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

[JENKINS-39867] Upgrade fabric8 to 1.4.26 #101

Merged
merged 1 commit into from Dec 6, 2016

Conversation

Projects
None yet
3 participants
@carlossg

carlossg commented Dec 2, 2016

Fixes Unauthorized message from OpenShift API

@carlossg

This comment has been minimized.

Show comment
Hide comment
@carlossg

carlossg Dec 2, 2016

I haven't tested it yet, other than just compiling ok without errors

carlossg commented Dec 2, 2016

I haven't tested it yet, other than just compiling ok without errors

[JENKINS-39867] Upgrade fabric8 to 1.4.26
Fixes Unauthorized message from OpenShift API
@iocanel

This comment has been minimized.

Show comment
Hide comment
@iocanel

iocanel Dec 6, 2016

@carlossg: It's a long story, but this actually saves me from hours of troubleshooting :-)

It looks great and I will try it asap!

iocanel commented Dec 6, 2016

@carlossg: It's a long story, but this actually saves me from hours of troubleshooting :-)

It looks great and I will try it asap!

@iocanel

This comment has been minimized.

Show comment
Hide comment
@iocanel

iocanel Dec 6, 2016

It worked fine for me!

iocanel commented Dec 6, 2016

It worked fine for me!

@iocanel iocanel merged commit 52c0af5 into master Dec 6, 2016

1 check passed

Jenkins This pull request looks good
Details
@iocanel

This comment has been minimized.

Show comment
Hide comment
@iocanel

iocanel Dec 6, 2016

@carlossg: I think that after this, its a good time for a release, wdyt?

iocanel commented Dec 6, 2016

@carlossg: I think that after this, its a good time for a release, wdyt?

@carlossg carlossg deleted the JENKINS-39867 branch Dec 6, 2016

@dserodio

This comment has been minimized.

Show comment
Hide comment
@dserodio

dserodio Dec 6, 2016

Yes, please make a new release. I tried testing this PR locally with mvn hpi:run but I get a nasty java.lang.LinkageError: Failed to load jenkins.util.SystemProperties error which I'd rather not try to debug :)

dserodio commented Dec 6, 2016

Yes, please make a new release. I tried testing this PR locally with mvn hpi:run but I get a nasty java.lang.LinkageError: Failed to load jenkins.util.SystemProperties error which I'd rather not try to debug :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment