Skip to content

[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

Closed
wants to merge 1 commit into from

Conversation

srowen
Copy link
Member

@srowen srowen commented Sep 3, 2019

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.

@srowen
Copy link
Member Author

srowen commented Sep 3, 2019

@andygrove @shaneknapp @skonto not sure if you all know what to make of this test failure?

lsof: WARNING: can't stat() aufs file system /var/lib/docker/aufs/mnt/04f77d5c3184798b175981cb930d3537efa8ab1dac04588c7cbd418972f0688c
lsof: WARNING: can't stat() aufs file system /home/docker/aufs/mnt/04f77d5c3184798b175981cb930d3537efa8ab1dac04588c7cbd418972f0688c
lsof: WARNING: can't stat() tmpfs file system /var/lib/docker/containers/805a48533fb8d67174d0a268446a7886192a6b21fca4760c199bc9293273489c/mounts/shm
lsof: WARNING: can't stat() tmpfs file system /home/docker/containers/805a48533fb8d67174d0a268446a7886192a6b21fca4760c199bc9293273489c/mounts/shm

Seems more env-related than anything?

@andygrove
Copy link
Member

@srowen yes, I agree, this is something env-related

@SparkQA
Copy link

SparkQA commented Sep 3, 2019

Test build #110048 has finished for PR 25664 at commit bf41b94.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member Author

srowen commented Sep 3, 2019

Hm, it didn't post the link to the failed test:
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/15067/

(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.

@skonto
Copy link
Contributor

skonto commented Sep 4, 2019

@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.
This should also be backported to 2.4.3, 2.4.4 according to the related PRs.

@srowen
Copy link
Member Author

srowen commented Sep 4, 2019

How do I retrigger it?
Are you saying the issue you cite might cause this?
BTW see #25599

@shaneknapp
Copy link
Contributor

@andygrove @shaneknapp @skonto not sure if you all know what to make of this test failure?

lsof: WARNING: can't stat() aufs file system /var/lib/docker/aufs/mnt/04f77d5c3184798b175981cb930d3537efa8ab1dac04588c7cbd418972f0688c
lsof: WARNING: can't stat() aufs file system /home/docker/aufs/mnt/04f77d5c3184798b175981cb930d3537efa8ab1dac04588c7cbd418972f0688c
lsof: WARNING: can't stat() tmpfs file system /var/lib/docker/containers/805a48533fb8d67174d0a268446a7886192a6b21fca4760c199bc9293273489c/mounts/shm
lsof: WARNING: can't stat() tmpfs file system /home/docker/containers/805a48533fb8d67174d0a268446a7886192a6b21fca4760c199bc9293273489c/mounts/shm

Seems more env-related than anything?

you can safely ignore these...

@shaneknapp
Copy link
Contributor

test this please

@shaneknapp
Copy link
Contributor

@SparkQA
Copy link

SparkQA commented Sep 4, 2019

@SparkQA
Copy link

SparkQA commented Sep 4, 2019

@SparkQA
Copy link

SparkQA commented Sep 4, 2019

Test build #110138 has finished for PR 25664 at commit bf41b94.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen srowen closed this in ded23f8 Sep 5, 2019
@srowen
Copy link
Member Author

srowen commented Sep 5, 2019

Merged to master/2.4

srowen added a commit that referenced this pull request Sep 5, 2019
…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>
rluta pushed a commit to rluta/spark that referenced this pull request Sep 17, 2019
…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>
rodrigovedovato pushed a commit to elo7/spark that referenced this pull request Sep 20, 2019
…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)
nikunjb pushed a commit to nikunjb/spark that referenced this pull request Oct 1, 2019
…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>
@srowen srowen deleted the SPARK-28921.2 branch October 9, 2019 14:11
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.

5 participants