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

Add explicit cluster_url with oc login #150

Merged
merged 1 commit into from Oct 3, 2017
Merged

Add explicit cluster_url with oc login #150

merged 1 commit into from Oct 3, 2017

Conversation

jwmatthews
Copy link
Member

This PR fixes the issue I've hit recently when trying to deploy latest 3.7 origin builds with our ec2 scripts which replace the cluster cert with letsencrypt signed certificates. I think the replacement of the CA may be causing issues.

I changed these variables in my_vars.yml
origin_image_tag: latest
openshift_client_version: latest

./run_setup_environment.sh
...

...

TASK [openshift_setup : Login as admin] *******************************************************************************
fatal: [34.233.221.91]: FAILED! => {"changed": true, "cmd": "/usr/bin/oc login -u admin -p admin", "delta": "0:00:00.177283", "end": "2017-10-01 13:56:55.907736", "failed": true, "rc": 1, "start": "2017-10-01 13:56:55.730453", "stderr": "error: x509: certificate signed by unknown authority", "stderr_lines": ["error: x509: certificate signed by unknown authority"], "stdout": "", "stdout_lines": []}

If I log into the instance I see same error trying to execute "oc login":

[ec2-user@ip-10-0-0-70 ~]$ sudo su -
[root@ip-10-0-0-70 ~]# oc login
error: x509: certificate signed by unknown authority

Part of the problem was that the default context specified the 127.0.0.1 entry in ~/.kube/config, so I added an explicit cluster_url.

Have tested this with both ec2 and local/linux deployments.

@jmrodri jmrodri self-requested a review October 3, 2017 13:17
Copy link
Contributor

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

ACK

@jmrodri jmrodri merged commit 6a70afc into master Oct 3, 2017
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.

None yet

3 participants