-
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
make sure [registry]admin covers [registry]edit #9004
Conversation
Add a check to policy_test.go that makes sure all default policy rules set an API group? |
extra test request, LGTM otherwise |
Spoke in person. Will open a separate pull to bring our roles into sync. |
ea64e8b
to
5713642
Compare
// TODO roundtrip roles to pick up defaulting for API groups. Without this, the covers check in reconcile-cluster-roles will fail. | ||
// we can remove this again once everything gets group qualified and we have unit tests enforcing that. other pulls are in | ||
// progress to do that. | ||
versionedRoles := []authorizationapiv1.ClusterRole{} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liggitt Removed the merge tag. I had to add this. Comment is above. its temporary, but significant enough to deserve a second look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as long as it's extremely temporary
5713642
to
826479a
Compare
test failures:
|
826479a
to
d764705
Compare
Evaluated for origin test up to d764705 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4054/) |
[merge] |
conformance wedged again: https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4069/ |
and conformance flakes again. @liggitt I'm not crazy, that test job is green, right? |
this time integration hung: https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4085/ ending with
|
And now a test and problem to look at tomorrow https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4091/ re[merge] |
Yum failures re[merge] |
Evaluated for origin merge up to d764705 |
continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4108/) |
Fixes #9000
@liggitt
[test]