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

Disable sudo on unit tests #6879

Merged
merged 1 commit into from Feb 21, 2018
Merged

Disable sudo on unit tests #6879

merged 1 commit into from Feb 21, 2018

Conversation

lamont-granquist
Copy link
Contributor

@lamont-granquist lamont-granquist commented Feb 20, 2018

Seems to be somewhat more stable in travis, and we do have coverage in jenkins (and the func and integ tests still run with sudo)

@lamont-granquist lamont-granquist requested a review from a team February 20, 2018 20:46
@lamont-granquist lamont-granquist force-pushed the lcg/travis-teting branch 2 times, most recently from 5bead59 to 2f50d3a Compare February 20, 2018 23:10
@lamont-granquist lamont-granquist requested a review from a team as a code owner February 20, 2018 23:10
@lamont-granquist lamont-granquist force-pushed the lcg/travis-teting branch 4 times, most recently from 5e74e38 to e511c35 Compare February 21, 2018 03:47
This seems to make unit tests on travis somewhat more stable

Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
@lamont-granquist lamont-granquist changed the title comment out some janky specs Disable sudo on unit tests Feb 21, 2018
@lamont-granquist
Copy link
Contributor Author

@tas50 @thommay this appears to make the unit tests green. smash merge if you approve of green tests...

@thommay thommay merged commit 9e69e65 into master Feb 21, 2018
@lock
Copy link

lock bot commented Apr 22, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Apr 22, 2018
@tas50 tas50 deleted the lcg/travis-teting branch July 10, 2018 22:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants