Skip to content
This repository was archived by the owner on Jul 10, 2024. It is now read-only.

SUBMARINE-935. Support test-k8s on both KinD and minikube#668

Closed
KUAN-HSUN-LI wants to merge 2 commits intoapache:masterfrom
KUAN-HSUN-LI:SUBMARINE-935
Closed

SUBMARINE-935. Support test-k8s on both KinD and minikube#668
KUAN-HSUN-LI wants to merge 2 commits intoapache:masterfrom
KUAN-HSUN-LI:SUBMARINE-935

Conversation

@KUAN-HSUN-LI
Copy link
Copy Markdown
Member

@KUAN-HSUN-LI KUAN-HSUN-LI commented Jul 13, 2021

What is this PR for?

Support running the test-k8s on both KinD and minikube
image

What type of PR is it?

[Improvement]

Todos

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-935

How should this be tested?

KinD is tested in CI.
minikube should be tested following the document

Screenshots (if appropriate)

Questions:

  • Do the license files need updating? No
  • Are there breaking changes for older versions? No
  • Does this need new documentation? No

@KUAN-HSUN-LI
Copy link
Copy Markdown
Member Author

@kevin85421 Can you help me review this PR. Thanks in advance!

Copy link
Copy Markdown
Member

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

It looks good to me. Can you add a screenshot about running test-k8s with minikube? In addition, add a hyperlink to IntegrationTestK8s.md.

@KUAN-HSUN-LI
Copy link
Copy Markdown
Member Author

@kevin85421 Thanks for your review.
I add a result image of passing the test-k8s with minikube, but I am not sure whether this image is clear enough.

Copy link
Copy Markdown
Member

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

LGTM

@asfgit asfgit closed this in 65b785e Jul 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants