Bug 1466032 - Adding an Automate Task schedule adds UTC to the last Attribute/Value pair
Summary: Adding an Automate Task schedule adds UTC to the last Attribute/Value pair
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.9.0
Assignee: Drew Bomhof
QA Contact: Milan Falešník
URL:
Whiteboard: automate:schedule
: 1490098 1493280 1493381 1503982 (view as bug list)
Depends On:
Blocks: 1479943 1480288
TreeView+ depends on / blocked
 
Reported: 2017-06-28 19:25 UTC by Ryan Spagnola
Modified: 2021-06-10 12:31 UTC (History)
12 users (show)

Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1479943 (view as bug list)
Environment:
Last Closed: 2018-03-06 14:39:22 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Ryan Spagnola 2017-06-28 19:25:31 UTC
Description of problem:
When adding a new schedule that is of type Automate Task the last attribute/value pair will have UTC added to the end when the schedule is saved. Every time the schedule is edited and saved UTC is added.
If there are no attribute/value pairs specified then UTC is added to the Request.

Version-Release number of selected component (if applicable):
5.8

How reproducible:
Always when adding or editing a schedule.

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
This prevents schedules from being modified.

Comment 2 CFME Bot 2017-07-11 13:37:26 UTC
New commit detected on ManageIQ/manageiq-providers-azure/master:
https://github.com/ManageIQ/manageiq-providers-azure/commit/255c954a313bf02099e860d72e5d7e4f03096a30

commit 255c954a313bf02099e860d72e5d7e4f03096a30
Author:     Drew Bomhof <dbomhof>
AuthorDate: Mon Jul 10 16:28:05 2017 -0400
Commit:     Drew Bomhof <dbomhof>
CommitDate: Mon Jul 10 16:28:05 2017 -0400

    Security Groups array_integer data type
    
    Adding the array_integer datatype to allow arrays to be passed into the security groups dialog
    
    Dependent on: https://github.com/ManageIQ/manageiq/pull/15409
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1466032

 content/miq_dialogs/miq_provision_azure_dialogs_template.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 3 Drew Bomhof 2017-07-31 17:34:01 UTC
(In reply to CFME Bot from comment #2)
> New commit detected on ManageIQ/manageiq-providers-azure/master:
> https://github.com/ManageIQ/manageiq-providers-azure/commit/
> 255c954a313bf02099e860d72e5d7e4f03096a30
> 
> commit 255c954a313bf02099e860d72e5d7e4f03096a30
> Author:     Drew Bomhof <dbomhof>
> AuthorDate: Mon Jul 10 16:28:05 2017 -0400
> Commit:     Drew Bomhof <dbomhof>
> CommitDate: Mon Jul 10 16:28:05 2017 -0400
> 
>     Security Groups array_integer data type
>     
>     Adding the array_integer datatype to allow arrays to be passed into the
> security groups dialog
>     
>     Dependent on: https://github.com/ManageIQ/manageiq/pull/15409
>     
>     https://bugzilla.redhat.com/show_bug.cgi?id=1466032
> 
>  content/miq_dialogs/miq_provision_azure_dialogs_template.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

This PR had the wrong BZ attached

Comment 5 CFME Bot 2017-07-31 20:23:30 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/626f59ecc7e4c87549f316207f92d94cd7d55076

commit 626f59ecc7e4c87549f316207f92d94cd7d55076
Author:     Drew Bomhof <dbomhof>
AuthorDate: Mon Jul 31 15:27:37 2017 -0400
Commit:     Drew Bomhof <dbomhof>
CommitDate: Mon Jul 31 15:27:37 2017 -0400

    Add the '&' separator for the time_zone parameter
    
    The encoded time_zone parameter was being sent over without an '&' separator which caused it to be included in the key value pairs returned
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1466032

 app/assets/javascripts/controllers/schedule/schedule_form_controller.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 7 Jeff Warnica 2017-08-15 16:19:15 UTC
Quoting @bevans

To work around this issue The schedule needs to be modified via the rails console. MiqSchedule is the class for the schedules.

s = MiqSchedule.where(:name => 'test schedule').first

s.filter[:ui][:ui_attrs][2][1] = 'ScheduleTest'

s.filter[:parameters]["instance"] = 'ScheduleTest'

s.save

Comment 8 Greg McCullough 2017-09-19 19:48:39 UTC
*** Bug 1493280 has been marked as a duplicate of this bug. ***

Comment 9 Greg McCullough 2017-09-21 14:33:15 UTC
*** Bug 1490098 has been marked as a duplicate of this bug. ***

Comment 10 Milan Falešník 2017-10-11 09:28:10 UTC
Verified in 5.9.0.1, this no longer happens

Comment 11 Harpreet Kataria 2017-10-23 20:40:01 UTC
*** Bug 1503982 has been marked as a duplicate of this bug. ***

Comment 12 Milan Zázrivec 2018-03-29 12:48:37 UTC
*** Bug 1493381 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.