Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Dec 10, 2020

What changes were proposed in this pull request?

This is a backport of #30700 .

This PR changes Minikube.scala for Kubernetes integration tests to work with Minikube 1.9+.
Minikube.scala assumes that apiserver.key and apiserver.crt are in ~/.minikube/.
But as of Minikube 1.9, they are in ~/.minikube/profiles/<profile>.

Why are the changes needed?

Currently, Kubernetes integration tests doesn't work with Minikube 1.9+.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

I confirmed the following test passes.

$ build/sbt -Pkubernetes -Pkubernetes-integration-tests package 'kubernetes-integration-tests/testOnly -- -z "SparkPi with no"'

@sarutak
Copy link
Member Author

sarutak commented Dec 10, 2020

cc: @dongjoon-hyun

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

Thanks!

@SparkQA
Copy link

SparkQA commented Dec 10, 2020

Test build #132556 has finished for PR 30704 at commit 6dc60b6.

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

asfgit pushed a commit that referenced this pull request Dec 10, 2020
…ork with Minikube 1.9+ #30702

### What changes were proposed in this pull request?

This is a backport of #30700 .

This PR changes `Minikube.scala` for Kubernetes integration tests to work with Minikube 1.9+.
`Minikube.scala` assumes that `apiserver.key` and `apiserver.crt` are in `~/.minikube/`.
But as of Minikube 1.9, they are in `~/.minikube/profiles/<profile>`.

### Why are the changes needed?

Currently, Kubernetes integration tests doesn't work with Minikube 1.9+.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

I confirmed the following test passes.
```
$ build/sbt -Pkubernetes -Pkubernetes-integration-tests package 'kubernetes-integration-tests/testOnly -- -z "SparkPi with no"'
```

Closes #30704 from sarutak/minikube-1.9-branch-2.4.

Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: Kousuke Saruta <[email protected]>
@sarutak
Copy link
Member Author

sarutak commented Dec 10, 2020

Merged to branch-2.4.

@sarutak sarutak closed this Dec 10, 2020
@dongjoon-hyun
Copy link
Member

Thank you so much!

@sarutak sarutak deleted the minikube-1.9-branch-2.4 branch June 4, 2021 20:42
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.

3 participants