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

oc version detection fails #49

Closed
benemon opened this issue Sep 27, 2018 · 1 comment · Fixed by #50
Closed

oc version detection fails #49

benemon opened this issue Sep 27, 2018 · 1 comment · Fixed by #50

Comments

@benemon
Copy link

benemon commented Sep 27, 2018

Using branch 1.4.8, and the official Fuse Online install docs at

https://access.redhat.com/documentation/en-us/red_hat_fuse/7.1/html-single/integrating_applications_with_fuse_online/#installing-on-ocp

The install_ocp script does:

oc version | grep oc | tr -d oc\ v | cut -f1 -d "+"

to do some version checking.

This renders the output:

3.10.45
Sererhttps://skylake.demlab.lal:8443

From the basic output:

oc v3.10.45
kubernetes v1.10.0+b81c8f8
features: Basic-Auth

Server https://skylake.demolab.local:8443
openshift v3.10.14
kubernetes v1.10.0+b81c8f8

The install script then tries to use this output to do min/test versioning, which results in an execution failure, and the error message:

Testing oc version '3.10.45' against required minimum 'Sererhttps://skylake.demlab.lal:8443' ... ERROR: oc version (3.10.45) should be at least Sererhttps://skylake.demlab.lal:8443

Which is just all kinds of wrong :)

valdar added a commit that referenced this issue Sep 27, 2018
in order to avoid false positives and hopefully fix #49
@benemon
Copy link
Author

benemon commented Sep 27, 2018

I've just realised it's because my Server identifier has the characters 'oc' in the URL. Ouch.

@pure-bot pure-bot bot closed this as completed in #50 Sep 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant