Skip to content

Add oauth to prow tracer#530

Merged
0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:trace-oauth
Jan 9, 2018
Merged

Add oauth to prow tracer#530
0xmichalis merged 1 commit into
openshift:masterfrom
0xmichalis:trace-oauth

Conversation

@0xmichalis
Copy link
Copy Markdown
Contributor

No description provided.

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 5, 2018
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smarterclayton I expected the proxy to pick up what I can perform in the cluster and allow me to access what it's proxying to. Instead:

2018/01/05 15:38:03 oauthproxy.go:657: 172.16.6.1:60776 Cookie "_oauth_proxy" not present
2018/01/05 15:38:03 provider.go:347: authorizer reason: User "system:anonymous" cannot get clusterroles at the cluster scope

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I've got it figured out. I was missing the oauth reference annotation on the service account.

@0xmichalis 0xmichalis changed the title [WIP] Add oauth to prow tracer Add oauth to prow tracer Jan 9, 2018
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 9, 2018
@0xmichalis 0xmichalis merged commit f9e8085 into openshift:master Jan 9, 2018
@0xmichalis 0xmichalis deleted the trace-oauth branch January 9, 2018 11:30
derekhiggins pushed a commit to derekhiggins/release that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants