-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[SPARK-28921][K8S][FOLLOWUP] Also bump K8S client version in integration-tests #25664
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
Conversation
@andygrove @shaneknapp @skonto not sure if you all know what to make of this test failure?
Seems more env-related than anything? |
@srowen yes, I agree, this is something env-related |
Test build #110048 has finished for PR 25664 at commit
|
Hm, it didn't post the link to the failed test: (The pass above is just for the rest of Spark which should not be affected.) I know we need to harmonize this version though. I'm tempted to just merge it as we don't have a reason to believe it's a legit failure, and the update is important. |
@srowen we should trigger the K8s integrations tests just in case here... would also be good to verify once more https://issues.apache.org/jira/browse/SPARK-28953 is not happening upstream. |
How do I retrigger it? |
you can safely ignore these... |
test this please |
k8s integration tests running here: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/15147/ |
Kubernetes integration test starting |
Kubernetes integration test status success |
Test build #110138 has finished for PR 25664 at commit
|
Merged to master/2.4 |
…ion-tests ### What changes were proposed in this pull request? Per #25640 (comment) also bump K8S client version in integration-tests module. ### Why are the changes needed? Harmonize the version as intended. ### Does this PR introduce any user-facing change? No ### How was this patch tested? Existing tests. Closes #25664 from srowen/SPARK-28921.2. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com> (cherry picked from commit ded23f8) Signed-off-by: Sean Owen <sean.owen@databricks.com>
…ion-tests ### What changes were proposed in this pull request? Per apache#25640 (comment) also bump K8S client version in integration-tests module. ### Why are the changes needed? Harmonize the version as intended. ### Does this PR introduce any user-facing change? No ### How was this patch tested? Existing tests. Closes apache#25664 from srowen/SPARK-28921.2. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com> (cherry picked from commit ded23f8) Signed-off-by: Sean Owen <sean.owen@databricks.com>
…ion-tests ### What changes were proposed in this pull request? Per apache#25640 (comment) also bump K8S client version in integration-tests module. ### Why are the changes needed? Harmonize the version as intended. ### Does this PR introduce any user-facing change? No ### How was this patch tested? Existing tests. Closes apache#25664 from srowen/SPARK-28921.2. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com> (cherry picked from commit ded23f8) Signed-off-by: Sean Owen <sean.owen@databricks.com> (cherry picked from commit 0566fd0)
…ion-tests ### What changes were proposed in this pull request? Per apache#25640 (comment) also bump K8S client version in integration-tests module. ### Why are the changes needed? Harmonize the version as intended. ### Does this PR introduce any user-facing change? No ### How was this patch tested? Existing tests. Closes apache#25664 from srowen/SPARK-28921.2. Authored-by: Sean Owen <sean.owen@databricks.com> Signed-off-by: Sean Owen <sean.owen@databricks.com> (cherry picked from commit ded23f8) Signed-off-by: Sean Owen <sean.owen@databricks.com>
What changes were proposed in this pull request?
Per #25640 (comment) also bump K8S client version in integration-tests module.
Why are the changes needed?
Harmonize the version as intended.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests.