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

Adjust upgrade_snapshots to new snapper ls output #6021

Merged
merged 1 commit into from
Oct 23, 2018

Conversation

jknphy
Copy link
Contributor

@jknphy jknphy commented Oct 22, 2018

Adjust test to new snapper ls output. Regex updated to be able to move 4 or 5 columns to the right.

@asdil12
Copy link
Member

asdil12 commented Oct 23, 2018

Does this already deal with different column orderings (see #5995)?

@jknphy
Copy link
Contributor Author

jknphy commented Oct 23, 2018

It doesn't, but we have a different implementation for every snapper test, unfortunately. In this case I don't see the point in my opinion, it will just over-complicate the test.

@rwx788 rwx788 merged commit 6f3082c into os-autoinst:master Oct 23, 2018
@jknphy jknphy deleted the fix_upgrade_snapshots branch January 22, 2019 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants