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

tests: Add RPMOSTREE_TEST_NO_OVERLAY to skip overlay #1251

Closed
wants to merge 1 commit into from

Conversation

cgwalters
Copy link
Member

The use case here is to run our tests against the shipped tree;
this could be used by a-h-t for example:
projectatomic/atomic-host-tests#74

I tried this with just TESTS=basic and it failed for usroverlay. So we'd
have to start adding feature detection to the test suite to make this truly
useful, but let's at least start with the basic bits now to play with it.

The use case here is to run our tests against the shipped tree;
this could be used by a-h-t for example:
projectatomic/atomic-host-tests#74

I tried this with just `TESTS=basic` and it failed for `usroverlay`. So we'd
have to start adding feature detection to the test suite to make this truly
useful, but let's at least start with the basic bits now to play with it.
@jlebon
Copy link
Member

jlebon commented Feb 14, 2018

@rh-atomic-bot r+ b4569d8

@rh-atomic-bot
Copy link

⌛ Testing commit b4569d8 with merge 32328f8...

rh-atomic-bot pushed a commit that referenced this pull request Feb 14, 2018
The use case here is to run our tests against the shipped tree;
this could be used by a-h-t for example:
projectatomic/atomic-host-tests#74

I tried this with just `TESTS=basic` and it failed for `usroverlay`. So we'd
have to start adding feature detection to the test suite to make this truly
useful, but let's at least start with the basic bits now to play with it.

Closes: #1251
Approved by: jlebon
@rh-atomic-bot
Copy link

💔 Test failed - status-atomicjenkins

@jlebon
Copy link
Member

jlebon commented Feb 14, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit b4569d8 with merge c5939b7...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing c5939b7 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