New commit detected on ManageIQ/manageiq/gaprindashvili: https://github.com/ManageIQ/manageiq/commit/685c571bef061200b7a26b8aa3b4f3577014733e commit 685c571bef061200b7a26b8aa3b4f3577014733e Author: Greg McCullough <gmccullo> AuthorDate: Tue Jul 31 12:07:50 2018 -0400 Commit: Greg McCullough <gmccullo> CommitDate: Tue Jul 31 12:07:50 2018 -0400 Merge pull request #17748 from hsong-rh/add_internal_to_service_template Add internal column to service template for transformation plan (cherry picked from commit 2a66cb59e26816c7296896620b5b7731b350943d) https://bugzilla.redhat.com/show_bug.cgi?id=1610729 app/models/service_template.rb | 3 + app/models/service_template_transformation_plan.rb | 2 + spec/models/service_template_transformation_plan_spec.rb | 11 + 3 files changed, 16 insertions(+)
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili: https://github.com/ManageIQ/manageiq-ui-classic/commit/24bc296be8024b360cb14837ac60e34d3c741c93 commit 24bc296be8024b360cb14837ac60e34d3c741c93 Author: Dan Clarizio <dclarizi> AuthorDate: Tue Jul 31 18:24:17 2018 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Tue Jul 31 18:24:17 2018 -0400 Merge pull request #4274 from h-kataria/service_templates_to_exclude_migration_plans Changes to exclude service templates marked as service_type = internal (cherry picked from commit 3acd7630eadecd477892075a2d4b222a16547940) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1610729 app/controllers/catalog_controller.rb | 25 +- app/presenters/tree_builder_catalog_items.rb | 4 +- app/presenters/tree_builder_service_catalog.rb | 4 +- spec/controllers/catalog_controller_spec.rb | 6 +- 4 files changed, 23 insertions(+), 16 deletions(-)
Created attachment 1474664 [details] issue_5.9.4.3.mp4 Not fixed! I can still see plans in catalog_items, just that now it is little bit covered. So earlier, plans were showing under catalog_item accordion in a tree (under Unassigned section) and now it is not showing in tree; but you can still find it with small extra steps, *click on `Unassigned`* and you will find plan are still there. Also checked SSUI which is working as expected. Checked on: 5.9.4.3.20180808144238_a20955a Marking FailedQA as behavior is not expected. (P.S. Video attached)
Created attachment 1474667 [details] earlier_behavior_5.9.4.0.mp4
FYI, if you delete plan from catalog_item it actually deletes plan from migration page.
https://github.com/ManageIQ/manageiq-ui-classic/pull/4473
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili: https://github.com/ManageIQ/manageiq-ui-classic/commit/23574ded21541a4628e7aa05e0b8b8970f8b2fbd commit 23574ded21541a4628e7aa05e0b8b8970f8b2fbd Author: Dan Clarizio <dclarizi> AuthorDate: Tue Aug 14 12:36:20 2018 -0400 Commit: Dan Clarizio <dclarizi> CommitDate: Tue Aug 14 12:36:20 2018 -0400 Merge pull request #4473 from h-kataria/exclude_migration_template_unassigned_folder Added missing 'public_service_templates' scope (cherry picked from commit 0b219e9248c8814caf0ed25831cbf69e41a2fcf7) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1610729 app/controllers/catalog_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Created attachment 1476602 [details] 5.9.4.4_proof.png Fixed! Mentioned bug is patched and migration plans are not displaying in Unassigned catalog section (last time it was there). Also in SSUI, service catalog don't show migration plan, which is what we have expected. (My small concern is - ordered migration plan still get displayed in My Order section of SSUI but it is covering through another Bugzilla issue so verifying this issue) Verified on: 5.9.4.4.20180816162527_c00eb23
Created attachment 1476603 [details] My_Order_SSUI.png
New commit detected on ManageIQ/manageiq-schema/master: https://github.com/ManageIQ/manageiq-schema/commit/296d94413596d44908c5849a974ed6b3be9693f9 commit 296d94413596d44908c5849a974ed6b3be9693f9 Author: hsong-rh <hsong> AuthorDate: Fri Aug 10 11:22:34 2018 -0400 Commit: hsong-rh <hsong> CommitDate: Fri Aug 10 11:22:34 2018 -0400 Update default of internal attribute in service template https://bugzilla.redhat.com/show_bug.cgi?id=1610729 db/migrate/20180810144738_update_default_internal_attribute.rb | 11 + spec/migrations/20180810144738_update_default_internal_attribute_spec.rb | 23 + 2 files changed, 34 insertions(+)