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

test: Cleanup check-machines #7118

Merged
merged 1 commit into from Jun 28, 2017

Conversation

Projects
None yet
3 participants
@stefwalter
Contributor

stefwalter commented Jun 27, 2017

There's a ton of duplicated info leaking from one test to
another, making it hard to write varied tests, with multiple
virtual machines, and/or extend logic. Cleanup the file

@stefwalter stefwalter referenced this pull request Jun 27, 2017

Merged

test: Boot real VMs for testing virtual machines #7117

1 of 1 task complete
test: Cleanup check-machines
There's a ton of duplicated info leaking from one test to
another, making it hard to write varied tests, with multiple
virtual machines, and/or extend logic. Cleanup the file

Closes #7118

@martinpitt martinpitt added the priority label Jun 28, 2017

@martinpitt

This comment has been minimized.

Show comment
Hide comment
@martinpitt

martinpitt Jun 28, 2017

Member

The "paused" stuff happens on Fedora-26 as well (so I remembered right) - we might just blacklist it there as well for the time being until #7117 happens?

Member

martinpitt commented Jun 28, 2017

The "paused" stuff happens on Fedora-26 as well (so I remembered right) - we might just blacklist it there as well for the time being until #7117 happens?

@martinpitt martinpitt merged commit cb0c277 into cockpit-project:master Jun 28, 2017

16 checks passed

avocado/fedora-24 Tests passed
Details
container/kubernetes Tests passed
Details
selenium/chrome Tests passed
Details
selenium/firefox Tests passed
Details
semaphoreci The build passed on Semaphore.
Details
verify/centos-7 Tests passed
Details
verify/debian-stable Tests passed
Details
verify/debian-testing Tests passed
Details
verify/fedora-26 Tests passed
Details
verify/fedora-atomic Tests passed
Details
verify/fedora-i386 Tests passed
Details
verify/rhel-7 Tests passed
Details
verify/rhel-7-4 Tests passed
Details
verify/rhel-atomic Tests passed
Details
verify/ubuntu-1604 Tests passed
Details
verify/ubuntu-stable Tests passed
Details

@stefwalter stefwalter referenced this pull request Jun 29, 2017

Merged

machines: Delete #7113

1 of 1 task complete

@stefwalter stefwalter deleted the stefwalter:cleanup-check-machines branch Oct 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment