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

vmcheck/misc-2: Make compatible with staged default #1438

Closed
wants to merge 1 commit into from

Conversation

cgwalters
Copy link
Member

First the pinning tests would try to pin a staged deployment,
and some of the later tests here depend on a subtle way on the
state of the system. It's tempting to do a reset before each one
and reboot but this makes things work.

There's some additional assertions here as I went through and
was debugging.

Prep for making staging the default.

First the pinning tests would try to pin a staged deployment,
and some of the later tests here depend on a subtle way on the
state of the system.  It's tempting to do a `reset` before each one
and reboot but this makes things work.

There's some additional assertions here as I went through and
was debugging.

Prep for making staging the default.
@jlebon
Copy link
Member

jlebon commented Jul 3, 2018

@rh-atomic-bot r+ eed1c5f

@rh-atomic-bot
Copy link

⌛ Testing commit eed1c5f with merge c9b8e15...

rh-atomic-bot pushed a commit that referenced this pull request Jul 3, 2018
First the pinning tests would try to pin a staged deployment,
and some of the later tests here depend on a subtle way on the
state of the system.  It's tempting to do a `reset` before each one
and reboot but this makes things work.

There's some additional assertions here as I went through and
was debugging.

Prep for making staging the default.

Closes: #1438
Approved by: jlebon
@cgwalters
Copy link
Member Author

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit eed1c5f with merge ea29cf8...

rh-atomic-bot pushed a commit that referenced this pull request Jul 3, 2018
First the pinning tests would try to pin a staged deployment,
and some of the later tests here depend on a subtle way on the
state of the system.  It's tempting to do a `reset` before each one
and reboot but this makes things work.

There's some additional assertions here as I went through and
was debugging.

Prep for making staging the default.

Closes: #1438
Approved by: jlebon
@jlebon
Copy link
Member

jlebon commented Jul 3, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit eed1c5f with merge 222d730...

@jlebon
Copy link
Member

jlebon commented Jul 3, 2018

OK, tests are running for real this time!
Thanks for bringing this up to my attention. I need to set aside some serious time to push forward on projectatomic/papr#96 and related.

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing 222d730 to master...

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