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

Initialize the delete_backing flag for existing VM disks in the reconfigure form #1190

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Apr 27, 2017

Initialize the delete_backing flag for existing VM disks in the reconfigure form

https://bugzilla.redhat.com/show_bug.cgi?id=1445818

Before:

screenshot from 2017-05-05 11-20-51

After:

screenshot from 2017-05-05 10-46-06

@lgalis
Copy link
Contributor Author

lgalis commented Apr 27, 2017

@miq-bot add_label bug, fine/yes, euwe/yes

@lgalis lgalis force-pushed the initialize_delete_backing_for_vm_disk_reconfigure branch from 9e324ff to e34672c Compare May 4, 2017 21:47
@miq-bot
Copy link
Member

miq-bot commented May 4, 2017

Checked commits lgalis/manageiq-ui-classic@fc90caf~...e34672c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@lgalis lgalis changed the title [WIP] Initialize the delete_backing flag for existing VM disks in the reconfigure form Initialize the delete_backing flag for existing VM disks in the reconfigure form May 5, 2017
@miq-bot miq-bot removed the wip label May 5, 2017
@lgalis
Copy link
Contributor Author

lgalis commented May 5, 2017

@dclarizio or @h-kataria - please review

@h-kataria h-kataria merged commit 8a195f6 into ManageIQ:master May 8, 2017
@lgalis lgalis deleted the initialize_delete_backing_for_vm_disk_reconfigure branch May 8, 2017 12:56
@simaishi
Copy link
Contributor

@lgalis Is there a BZ for this? Can you please create if it doesn't exist?

@lgalis
Copy link
Contributor Author

lgalis commented May 12, 2017

@simaishi - added the BZ link in the PR description

@lgalis
Copy link
Contributor Author

lgalis commented May 12, 2017

@miq-bot remove_label bugzilla needed

@simaishi
Copy link
Contributor

@lgalis please remove the conflict label when Euwe PR is ready. Thanks!

@lgalis
Copy link
Contributor Author

lgalis commented May 24, 2017

@miq-bot remove_label euwe/conflict
@simaishi - EUWE PR : ManageIQ/manageiq#15223

@lgalis
Copy link
Contributor Author

lgalis commented May 24, 2017

The problem fixed in this PR does not exist in euwe. Removing the euwe/yes label
@miq-bot remove_label euwe/yes
@miq-bot add/label euwe/no

@miq-bot miq-bot removed the euwe/yes label May 24, 2017
@miq-bot
Copy link
Member

miq-bot commented May 24, 2017

@lgalis unrecognized command 'add', ignoring...

Accepted commands are: add_label, assign, close_issue, move_issue, remove_label, rm_label, set_milestone

simaishi pushed a commit that referenced this pull request Jun 6, 2017
…_disk_reconfigure

Initialize the delete_backing flag for existing VM disks in the reconfigure form
(cherry picked from commit 8a195f6)

https://bugzilla.redhat.com/show_bug.cgi?id=1459264
@simaishi
Copy link
Contributor

simaishi commented Jun 6, 2017

Fine backport details:

$ git log -1
commit a59cdb58d04471b396a6429a819e6013c8110c87
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Sun May 7 20:15:50 2017 -0400

    Merge pull request #1190 from lgalis/initialize_delete_backing_for_vm_disk_reconfigure
    
    Initialize the delete_backing flag for existing VM disks in the reconfigure form
    (cherry picked from commit 8a195f679191366461cab6bac6156316e6c1c72d)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459264

@chessbyte chessbyte added this to the Sprint 61 Ending May 22, 2017 milestone Oct 16, 2017
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.

None yet

6 participants