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

Fix `skipping test that requires root` issue #19

Open
wants to merge 2 commits into
base: master
from

Conversation

@chenrui333
Copy link
Contributor

chenrui333 commented Jan 6, 2020

@chenrui333 chenrui333 mentioned this pull request Jan 6, 2020
@chenrui333 chenrui333 force-pushed the chenrui333:fix-go-test-issue branch from 0c5a32c to 5a616cb Jan 6, 2020
.travis.yml Outdated Show resolved Hide resolved
@AkihiroSuda

This comment has been minimized.

Copy link
Member

AkihiroSuda commented Jan 15, 2020

@chenrui333

This comment has been minimized.

Copy link
Contributor Author

chenrui333 commented Jan 18, 2020

Thanks the reminder, picking it up now.

chenrui333 added 2 commits Jan 6, 2020
Signed-off-by: Rui Chen <chenrui333@gmail.com>
Signed-off-by: Rui Chen <chenrui333@gmail.com>
@chenrui333 chenrui333 force-pushed the chenrui333:fix-go-test-issue branch from 5a616cb to 96a9c5e Jan 18, 2020
@AkihiroSuda

This comment has been minimized.

Copy link
Member

AkihiroSuda commented Jan 20, 2020

CI still failing. maybe go test -exec sudo works.

@AkihiroSuda

This comment has been minimized.

Copy link
Member

AkihiroSuda commented Jan 20, 2020

Also, don't drop -test.root -test.v

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.