-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Fix invalid API version from openshift yml #7913
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving this because that's compatible with versions 3.5, 3.6 and 3.7 of OpenShift but that's not a bug, it's the new DeploymentConfig
API endpoint introduced with OpenShift 3.6 (i.e. if you update your client and server the script should work fine).
@Themodem You should sign Eclipse ECA and amend your commit with ECA Signed-Off by footer (currently the ip validation of this PR is failing) |
Hi @benoitf Slightly confused about the ECA, the link provided by jenkins says my email address has not signed the ECA but if I look at the eclipse website that says I already have. Am i missing another step? Also the link from the jenkins webhook links to an out of date page
|
@Themodem yes you need to amend your commit with Signed-Off: footer |
Signed-off-by: Lee Fenlan <lee@fenlan.uk>
@benoitf ahh my bad, figured it would use gpg key signing. Amended the commit. Cheers |
What does this PR do?
Quick sed fix for deployment to minishift.
Should probably be fixed in the
maven2/io/fabric8/tenant/apps/che
repo.What issues does this PR fix or reference?
Closes #7803
Changelog
sed apps.openshift.io/ out of the version as its invalid when running
deploy_che.sh
Release Notes
N/A