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

revert fabric8 bump, work around 3.6 api svr issue #174

Merged

Conversation

gabemontero
Copy link

(short term) Fixes #173

@openshift/devex fyi

@gabemontero
Copy link
Author

[test]

@openshift-bot
Copy link

Evaluated for jenkins sync plugin test up to a4a406e

@openshift-bot
Copy link

continuous-integration/openshift-jenkins-sync-plugin/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_jenkins_sync_plugin/29/) (Base Commit: ab5442e) (PR Branch Commit: a4a406e)

@gabemontero
Copy link
Author

@bparees given the frequency of encounters, I'm inclined to get this change out there, even if we continue to sort out longer term work arounds for supporting 3.6 / where the openshift/origin bug occurred.

Thoughts?

@bparees
Copy link

bparees commented Sep 25, 2017

@gabemontero this version of the fabric plugin is compatible with both 3.6 and the current 3.7? if so, i'm definitely in agreement.

@gabemontero
Copy link
Author

gabemontero commented Sep 25, 2017

the PR test that just ran confirms current 3.7

I am bringing up oc cluster up --version=v3.6.0 as I type .... will update with results when I have them.

@gabemontero
Copy link
Author

Confirmed .... reproduced the problem with openshift/origin v3.6.0; then upgraded jenkins to my locally built version of this PR, and the error goes away.

@gabemontero
Copy link
Author

[merge]

@openshift-bot
Copy link

openshift-bot commented Sep 25, 2017

continuous-integration/openshift-jenkins-sync-plugin/merge Waiting: You are in the build queue at position: 1

@openshift-bot
Copy link

Evaluated for jenkins sync plugin merge up to a4a406e

@openshift-bot openshift-bot merged commit 546037b into openshift:master Sep 25, 2017
@gabemontero gabemontero deleted the work-around-3.6-api-issue branch September 25, 2017 16:25
waveywaves pushed a commit to waveywaves/jenkins-sync-plugin that referenced this pull request Oct 30, 2019
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.

3 participants