Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow performing operations on Instances of a chosen Cloud Network #6458

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented Nov 27, 2019

Issue: #6309

This PR fixes the issue regarding Instances displayed through a Cloud Network's Relationships: nothing happened in the UI for almost all of the operations in the toolbar, for some operations error No template found for CloudNetworkController#button occurred.

The logic for operations on nested list of Instances was missing, in cloud_network controller. I'm adding appropriate logic by using button method from GenericButtonMixin.

In this PR, I'm also adding missing routes for managing policies of selected Instances.

Also Check Compliance of Last Known Configuration action will be fixed after merging this PR together with #6426 (MERGED).


This PR does not fix:


Before: (nothing happens in the UI, after clicking on Manage Policies in the toolbar)
manage-before

After: (Managing Policies for selected Instance works well)
manage-after

@hstastna
Copy link
Author

@miq-bot add_label bug

@miq-bot miq-bot added the bug label Nov 27, 2019
@hstastna hstastna changed the title Allow performing operations on Instances of a chosen Cloud Network [WIP] Allow performing operations on Instances of a chosen Cloud Network Nov 28, 2019
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch from 072e9ee to 0f95c73 Compare November 28, 2019 14:07
@hstastna hstastna changed the title [WIP] Allow performing operations on Instances of a chosen Cloud Network Allow performing operations on Instances of a chosen Cloud Network Nov 28, 2019
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch 2 times, most recently from ade62a3 to 55f3c4e Compare December 2, 2019 13:27
@hstastna hstastna changed the title Allow performing operations on Instances of a chosen Cloud Network [WIP] Allow performing operations on Instances of a chosen Cloud Network Dec 2, 2019
@miq-bot miq-bot added the wip label Dec 2, 2019
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch 15 times, most recently from cf970f0 to 8b50d56 Compare December 4, 2019 12:13
@hstastna hstastna changed the title [WIP] Allow performing operations on Instances of a chosen Cloud Network Allow performing operations on Instances of a chosen Cloud Network Dec 4, 2019
@miq-bot miq-bot removed the wip label Dec 4, 2019
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch from 8b50d56 to c9f8ec9 Compare December 4, 2019 14:13
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch from c9f8ec9 to 8b4d26b Compare December 4, 2019 15:49
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch from 8b4d26b to 0431835 Compare December 13, 2019 14:04
@hstastna hstastna force-pushed the Operations_Instances_of_CloudNetwork branch from 0431835 to f384298 Compare December 13, 2019 14:12
@miq-bot
Copy link
Member

miq-bot commented Dec 13, 2019

Checked commit hstastna@f384298 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. 👍

@martinpovolny martinpovolny merged commit 3254970 into ManageIQ:master Dec 13, 2019
@martinpovolny martinpovolny self-assigned this Dec 13, 2019
@martinpovolny martinpovolny added this to the Sprint 127 Ending Jan 6, 2020 milestone Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants