-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Extended test flake due to user not having permissions #8491
Comments
@smarterclayton I assume this surfaced in #8483, does anyone know if it has surfaced before? Is it limited to this specific extended test or does it appear in other places? Am I correct in assuming there is no way to reproduce reliably? The failure is happening here: https://github.com/openshift/origin/blob/master/test/extended/localquota/local_fsgroup_quota.go#L25 As far as I can tell the test is correct but I'm curious what these other issues are alluded to above. (policy race cace, permission assignment flake) Could anyone point me to info on these? From openshift.log:
If the timestamps can be relied on we've failed before this section:
|
I think this may just be a general flaw with our e2e tests, they need to use something similar to https://github.com/openshift/origin/blob/master/test/util/policy.go#L20 in all e2e tests (to ensure permissions converge before use) |
Nm, put a fix up |
https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_conformance/148/consoleFull
@deads2k looks like a policy cache race, or possibly the permission assignment flake @ncdc spotted
The text was updated successfully, but these errors were encountered: