Skip to content

Separate e2e-user config file in end to end #1563

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

Merged
merged 1 commit into from
Apr 2, 2015
Merged

Separate e2e-user config file in end to end #1563

merged 1 commit into from
Apr 2, 2015

Conversation

liggitt
Copy link
Contributor

@liggitt liggitt commented Apr 2, 2015

@fabianofranz review

osc project master isn't working to switch back to the system:admin user from e2e-user in test-end-to-end.sh. I'm really not sure how it ever worked. It's resulting in these errors:

You do not have rights to view project "master" on server "https://10.45.6.153:8443".
Your projects are:
default
test

Once you start messing with the context of a cluster admin's config file, there's no way to get back using osc login and osc project commands... you need to do osc config use-context master --kubeconfig=.... That seems wonky enough that I just made e2e use separate config files for the cluster user and the e2e user.

@liggitt
Copy link
Contributor Author

liggitt commented Apr 2, 2015

[test]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test Running (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_openshift3/1639/)

@smarterclayton
Copy link
Contributor

Hrm, project should switch contexts normally (if you know the context name). So then it's just a matter of login not properly making the transition from the initial config object to a different user and context.

@smarterclayton
Copy link
Contributor

I'll merge it direct when the tests pass

@smarterclayton
Copy link
Contributor

[test]

@ncdc
Copy link
Contributor

ncdc commented Apr 2, 2015

I just manually merged this into my dev branch locally and e2e passes for me.

@ncdc
Copy link
Contributor

ncdc commented Apr 2, 2015

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin up to 16df0e8

danmcp added a commit that referenced this pull request Apr 2, 2015
Separate e2e-user config file in end to end
@danmcp danmcp merged commit 4b6d097 into openshift:master Apr 2, 2015
@liggitt liggitt deleted the fix_end_to_end branch April 2, 2015 13:24
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.

5 participants