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

[JENKINS-59000] Upgrade kubernetes-client to 4.4.2 #582

Merged
merged 1 commit into from
Aug 26, 2019
Merged

Conversation

carlossg
Copy link
Contributor

To get the fix in fabric8io/kubernetes-client#1669

Origin header is set with port -1 when no port is present in the Kubernetes API url

Causing an error on exec

java.net.ProtocolException: Expected HTTP 101 response but was '403 Forbidden'

To get the fix in fabric8io/kubernetes-client#1669

Origin header is set with port -1 when no port is present in the Kubernetes API url

Causing an error on exec

```
java.net.ProtocolException: Expected HTTP 101 response but was '403 Forbidden'
```
Copy link
Member

@Vlatombe Vlatombe left a comment

Choose a reason for hiding this comment

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

lgtm

@Vlatombe Vlatombe added chore Maintenance bug Bug Fixes and removed chore Maintenance labels Aug 26, 2019
@Vlatombe Vlatombe merged commit 124214e into master Aug 26, 2019
@Vlatombe Vlatombe deleted the JENKINS-59000 branch August 26, 2019 13:29
@Vlatombe Vlatombe restored the JENKINS-59000 branch August 26, 2019 13:29
@Vlatombe Vlatombe deleted the JENKINS-59000 branch August 26, 2019 13:29
@Vlatombe Vlatombe added the dependencies Pull requests that update a dependency file label Aug 26, 2019
@jglick jglick removed the bug Bug Fixes label Aug 26, 2019
@jglick
Copy link
Member

jglick commented Aug 26, 2019

(can only have one category label for Release Drafter)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants