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

make sure [registry]admin covers [registry]edit #9004

Merged
merged 1 commit into from
May 27, 2016

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented May 24, 2016

Fixes #9000

@liggitt
[test]

@liggitt
Copy link
Contributor

liggitt commented May 24, 2016

Add a check to policy_test.go that makes sure all default policy rules set an API group?

@liggitt
Copy link
Contributor

liggitt commented May 24, 2016

extra test request, LGTM otherwise

@deads2k
Copy link
Contributor Author

deads2k commented May 24, 2016

Spoke in person. Will open a separate pull to bring our roles into sync.

@deads2k deads2k force-pushed the fix-policy-covers branch from ea64e8b to 5713642 Compare May 24, 2016 20:08
// 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{}
Copy link
Contributor Author

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.

Copy link
Contributor

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

@deads2k deads2k force-pushed the fix-policy-covers branch from 5713642 to 826479a Compare May 24, 2016 20:19
@liggitt
Copy link
Contributor

liggitt commented May 25, 2016

test failures:

  • integration test router flake: TestRouter integration flake #8502

  • authentication/scopedtokens.test/cmd/authentication.sh:30: executing 'oc get projects --token="${listprojecttoken}"' expecting success and text 'cmd-authentication'

    === BEGIN TEST CASE ===
    test/cmd/authentication.sh:30: executing 'oc get projects --token="${listprojecttoken}"' expecting success and text 'cmd-authentication'
    FAILURE after 0.188s: test/cmd/authentication.sh:30: executing 'oc get projects --token="${listprojecttoken}"' expecting success and text 'cmd-authentication': the command returned the wrong error code; the output content test failed
    There was no output from the command.
    Standard error from the command:
    Error from server: scopes [user:list-projects] prevent this action; User "scoped-user" cannot list all projects in the cluster
    === END TEST CASE ===
    
  • origin/pkg/authorization/authorizer.TestAPIGroupDeny

    --- FAIL: TestAPIGroupDeny (0.00s)
    panic: api.IsPersonalSubjectAccessReview is not a registered type and cannot be converted into version "unknown/unknown" [recovered]
    panic: api.IsPersonalSubjectAccessReview is not a registered type and cannot be converted into version "unknown/unknown"
    

@deads2k deads2k force-pushed the fix-policy-covers branch from 826479a to d764705 Compare May 25, 2016 13:31
@openshift-bot
Copy link
Contributor

Evaluated for origin test up to d764705

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4054/)

@deads2k
Copy link
Contributor Author

deads2k commented May 25, 2016

[merge]

@deads2k
Copy link
Contributor Author

deads2k commented May 25, 2016

@deads2k
Copy link
Contributor Author

deads2k commented May 25, 2016

@deads2k
Copy link
Contributor Author

deads2k commented May 25, 2016

and conformance flakes again.

@liggitt I'm not crazy, that test job is green, right?

@deads2k
Copy link
Contributor Author

deads2k commented May 25, 2016

this time integration hung: https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4085/

ending with


There was no error output from the command.
Running test/end-to-end/core.sh:176: executing 'oc delete pod cli-with-token-2' expecting success...
SUCCESS after 0.146s: test/end-to-end/core.sh:176: executing 'oc delete pod cli-with-token-2' expecting success
Standard output from the command:
pod "cli-with-token-2" deleted

There was no error output from the command.

@deads2k
Copy link
Contributor Author

deads2k commented May 26, 2016

And now a test and problem to look at tomorrow https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4091/

re[merge]

@deads2k
Copy link
Contributor Author

deads2k commented May 26, 2016

Yum failures

re[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to d764705

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/4108/)

@openshift-bot openshift-bot merged commit d764705 into openshift:master May 27, 2016
@deads2k deads2k deleted the fix-policy-covers branch September 6, 2016 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants