-
Notifications
You must be signed in to change notification settings - Fork 86
Bump fabric8 kubernetes-client from 4.1.3 to 4.6.2 #3076
Conversation
@enmasse-ci run tests profile=shared-iot |
1 similar comment
@enmasse-ci run tests profile=shared-iot |
@enmasse-ci run tests testcase=SmokeTests |
8604f99
to
17c84b7
Compare
@enmasse-ci run tests profile=shared-iot |
3 similar comments
@enmasse-ci run tests profile=shared-iot |
@enmasse-ci run tests profile=shared-iot |
@enmasse-ci run tests profile=shared-iot |
17c84b7
to
853f561
Compare
❗ Build Failed ❗ |
1 similar comment
❗ Build Failed ❗ |
@enmasse-ci run tests testcase=SmokeTests |
❗ Build Failed ❗ |
36a2aa9
to
f483640
Compare
@enmasse-ci run tests testcase=SmokeTests |
❗ Build Failed ❗ |
185ead9
to
621cbd4
Compare
@enmasse-ci run tests profile=smoke |
❌ Test Summary ❌TEST_PROFILE: smoke ❗ Test Failures ❗
|
621cbd4
to
b026f2c
Compare
@enmasse-ci run tests testcase=SmokeTests |
❗ Build Failed ❗ |
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.
LGTM!
@enmasse-ci run tests profile=shared-iot |
@enmasse-ci run tests profile=shared-iot |
❌ Test Summary ❌TEST_PROFILE: shared ❗ Test Failures ❗
|
* Bump sundrio.version from 0.16.4 to 0.17.2 * Remove KubernetesDeserializer thread safety workaround. * Simplify code used to turn off HTTP/2, utilizing new method in KC 4.4.0 * Remove generic parameter from KubernetesResource (relates to fabric8io/kubernetes-client#1661) * Workaround Fabric8 API change by making cascade desire explict (see fabric8io/kubernetes-client#1840)
722ad2c
to
ee65eb2
Compare
@enmasse-ci run tests |
❌ Test Summary ❌TEST_PROFILE: pr ❗ Test Failures ❗
|
890611f
to
37f3baf
Compare
@enmasse-ci run tests |
❌ Test Summary ❌TEST_PROFILE: pr ❗ Test Failures ❗
|
@enmasse-ci run tests testcase=FirefoxWebConsoleTest,ServiceCatalogWebTest,CustomResourceDefinitionAddressesTest,CommonTest |
❌ Test Summary ❌TEST_PROFILE: pr ❗ Test Failures ❗
|
@lulf I believe this is good to be merged, if you could review the additional changes. |
This allows the workaround #2999 to be removed.
Also: