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

We were still disabling snapsot revert for all but the current snapshot #1367

Merged

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Jan 19, 2018

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

Now all snapshots can be reverted to and from

because one pr ( #1358 ) isn't enough to solve the problem

puddin' PROOF

screen shot 2018-01-19 at 11 59 26 am

@AllenBW AllenBW requested a review from himdel as a code owner January 19, 2018 16:58
@AllenBW AllenBW added this to the Sprint 78 Ending Jan 29, 2018 milestone Jan 19, 2018
@AllenBW AllenBW force-pushed the bug/master/#1534059-revert-snapshot-fail branch from 403f131 to 76cd712 Compare January 19, 2018 17:10
@miq-bot
Copy link
Member

miq-bot commented Jan 19, 2018

Checked commit AllenBW@76cd712 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

#1358 made it possible to revert to snapshots without parent

this PR makes it possible to revert to the current snapshot

Ops UI doesn't seem to limit this in any way (except for filtering out cloud Openstack VMs) so this (almost) matches.

@himdel himdel merged commit 4f1c1e6 into ManageIQ:master Jan 19, 2018
@AllenBW AllenBW deleted the bug/master/#1534059-revert-snapshot-fail branch January 19, 2018 17:44
simaishi pushed a commit that referenced this pull request Jan 19, 2018
…shot-fail

We were still disabling snapsot revert for all but the current snapshot
(cherry picked from commit 4f1c1e6)

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

Gaprindashvili backport details:

$ git log -1
commit 4f9f2e8c97f593bd56bce6d6300bec80a6cd4284
Author: Martin Hradil <[email protected]>
Date:   Fri Jan 19 18:42:28 2018 +0100

    Merge pull request #1367 from AllenBW/bug/master/#1534059-revert-snapshot-fail
    
    We were still disabling snapsot revert for all but the current snapshot
    (cherry picked from commit 4f1c1e6c6c3ffe733e4a4c414512325af93e27a9)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534059

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.

4 participants