Skip to content

[MINOR][DOC] Bump minikube minimum version to 1.7.3 in README.md#32403

Closed
williamhyun wants to merge 1 commit intoapache:masterfrom
williamhyun:k8s
Closed

[MINOR][DOC] Bump minikube minimum version to 1.7.3 in README.md#32403
williamhyun wants to merge 1 commit intoapache:masterfrom
williamhyun:k8s

Conversation

@williamhyun
Copy link
Copy Markdown
Member

@williamhyun williamhyun commented Apr 29, 2021

What changes were proposed in this pull request?

This PR updates the k8s README.md to be consistent with the Apache Spark website.

Why are the changes needed?

The current minimum version in the README.md does not match the minimum version of the website.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

N/A

@SparkQA
Copy link
Copy Markdown

SparkQA commented Apr 29, 2021

Test build #138091 has finished for PR 32403 at commit 3b8055e.

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

@williamhyun
Copy link
Copy Markdown
Member Author

Could you review this, @attilapiros ?

@SparkQA
Copy link
Copy Markdown

SparkQA commented Apr 30, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42611/

@SparkQA
Copy link
Copy Markdown

SparkQA commented Apr 30, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42611/

@attilapiros
Copy link
Copy Markdown
Contributor

The change is fine but this is also part of #31829
Where we are blocked by https://issues.apache.org/jira/browse/SPARK-34738.

@dongjoon-hyun
Copy link
Copy Markdown
Member

Oh, in this case, could you close this PR, @williamhyun ? Sorry for asking this.

@dongjoon-hyun
Copy link
Copy Markdown
Member

According to our discussion, I'll close this PR, @williamhyun . Thank you for making a PR.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants