{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3051","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3051/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3051/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3051/events","html_url":"https://github.com/ManageIQ/manageiq/pull/3051","id":84766660,"number":3051,"title":"Update orchestration stack through orchestration service reconfigure","user":{"login":"bzwei","id":7047579,"avatar_url":"https://avatars.githubusercontent.com/u/7047579?v=3","gravatar_id":"","url":"https://api.github.com/users/bzwei","html_url":"https://github.com/bzwei","followers_url":"https://api.github.com/users/bzwei/followers","following_url":"https://api.github.com/users/bzwei/following{/other_user}","gists_url":"https://api.github.com/users/bzwei/gists{/gist_id}","starred_url":"https://api.github.com/users/bzwei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bzwei/subscriptions","organizations_url":"https://api.github.com/users/bzwei/orgs","repos_url":"https://api.github.com/users/bzwei/repos","events_url":"https://api.github.com/users/bzwei/events{/privacy}","received_events_url":"https://api.github.com/users/bzwei/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/provisioning","name":"provisioning","color":"5319e7"}],"state":"open","locked":false,"assignee":{"login":"bzwei","id":7047579,"avatar_url":"https://avatars.githubusercontent.com/u/7047579?v=3","gravatar_id":"","url":"https://api.github.com/users/bzwei","html_url":"https://github.com/bzwei","followers_url":"https://api.github.com/users/bzwei/followers","following_url":"https://api.github.com/users/bzwei/following{/other_user}","gists_url":"https://api.github.com/users/bzwei/gists{/gist_id}","starred_url":"https://api.github.com/users/bzwei/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bzwei/subscriptions","organizations_url":"https://api.github.com/users/bzwei/orgs","repos_url":"https://api.github.com/users/bzwei/repos","events_url":"https://api.github.com/users/bzwei/events{/privacy}","received_events_url":"https://api.github.com/users/bzwei/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2015-06-03T20:51:46Z","updated_at":"2015-07-15T20:23:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/3051","html_url":"https://github.com/ManageIQ/manageiq/pull/3051","diff_url":"https://github.com/ManageIQ/manageiq/pull/3051.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/3051.patch"},"body":"The provider orchestration API allows one to update existing stacks. The update API accepts a new orchestration template and its corresponding parameters.\r\n\r\nThis work supports stack update through manageiq orchestration service. This means only stack created through manageiq orchestration service can be updated.\r\n\r\nManageIQ service already has the concept of reconfigure. This work utilize the reconfigure automate to update an existing stack. A new reconfiguration entry point together with its own state machine is introduced under Cloud/Orchestration namespace.\r\n  \r\n This work has been broken down into a few commits.\r\n\r\n1. The dialog generated by the tool enables reconfigurable option. This allows the parameter fields editable during service reconfigure.\r\n\r\n2. Model and corresponding MiqAe model changes to bridge the reconfigure automate and provider update api.\r\n\r\n3. New entry point `/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachine/Reconfigure/default` and ruby scripts exposing to user to use and edit.\r\n\r\n4. UI changes to add entry points for reconfigure and retirement with default values.\r\n\r\nFixes #3401"},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/121735753","html_url":"https://github.com/ManageIQ/manageiq/pull/3051#issuecomment-121735753","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3051","id":121735753,"user":{"login":"miq-bot","id":7806257,"avatar_url":"https://avatars.githubusercontent.com/u/7806257?v=3","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2015-07-15T20:23:07Z","updated_at":"2015-07-15T20:23:07Z","body":"<rubocop />Checked commits https://github.com/bzwei/manageiq/commit/a452d0022a40a65976b24be48d65e7e201757763 .. https://github.com/bzwei/manageiq/commit/cc4761ff47b17f0dda6311caa80a08425b5d961d with rubocop 0.32.0\n19 files checked, 18 offenses detected\n\n**app/controllers/catalog_controller.rb**\n- [ ] :small_blue_diamond: - [Line 1434](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/controllers/catalog_controller.rb#L1434), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [121/120]\n- [ ] :small_blue_diamond: - [Line 1435](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/controllers/catalog_controller.rb#L1435), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [136/120]\n\n**app/views/catalog/_form_basic_info.html.haml**\n- [ ] :red_circle: **Warn** - [Line 85](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L85), Col  -  - The = symbol should have one space separating it from code\n- [ ] :red_circle: **Warn** - [Line 87](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L87), Col  -  - The = symbol should have one space separating it from code\n- [ ] :red_circle: **Warn** - [Line 98](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L98), Col  -  - Line is too long. [145/120]\n- [ ] :red_circle: **Warn** - [Line 107](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L107), Col  -  - The = symbol should have one space separating it from code\n- [ ] :red_circle: **Warn** - [Line 109](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L109), Col  -  - The = symbol should have one space separating it from code\n- [ ] :red_circle: **Warn** - [Line 118](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/app/views/catalog/_form_basic_info.html.haml#L118), Col  -  - Line is too long. [144/120]\n\n**db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb**\n- [ ] :small_blue_diamond: - [Line 16](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb#L16), Col 1 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for emailrequester is too high. [31.69/15]\n- [ ] :small_blue_diamond: - [Line 49](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb#L49), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [135/120]\n- [ ] :small_blue_diamond: - [Line 51](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb#L51), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [173/120]\n- [ ] :small_blue_diamond: - [Line 60](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb#L60), Col 1 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for emailapprover is too high. [27.48/15]\n- [ ] :small_blue_diamond: - [Line 88](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/Email.class/__methods__/servicereconfigurerequest_approved.rb#L88), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [173/120]\n\n**db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/check_reconfigured.rb**\n- [ ] :small_blue_diamond: - [Line 20](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/check_reconfigured.rb#L20), Col 1 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for check_updated is too high. [36.25/15]\n\n**db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/postreconfigure.rb**\n- [ ] :red_circle: **Warn** - [Line 14](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/postreconfigure.rb#L14), Col 1 - [Lint/UselessAssignment](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Lint/UselessAssignment) - Useless assignment to variable - `stack`.\n\n**db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/prereconfigure.rb**\n- [ ] :small_blue_diamond: - [Line 24](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/db/fixtures/ae_datastore/ManageIQ/Cloud/Orchestration/Reconfiguration/StateMachines/Methods.class/__methods__/prereconfigure.rb#L24), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [142/120]\n\n**spec/models/service_orchestration_spec.rb**\n- [ ] :small_blue_diamond: - [Line 65](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/spec/models/service_orchestration_spec.rb#L65), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [126/120]\n- [ ] :small_blue_diamond: - [Line 126](https://github.com/bzwei/manageiq/blob/cc4761ff47b17f0dda6311caa80a08425b5d961d/spec/models/service_orchestration_spec.rb#L126), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.0/RuboCop/Cop/Metrics/LineLength) - Line is too long. [123/120]\n"}}