Skip to content
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

feat: add ephemeral kubernetes provider #4927

Merged
merged 42 commits into from Sep 13, 2023
Merged

Conversation

shumailxyz
Copy link
Contributor

@shumailxyz shumailxyz commented Aug 8, 2023

What this PR does / why we need it:
Adds a new provider ephemeral-kubernetes that allows to use ephemeral clusters.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

  • I have added a basic example examples/ephemeral-cluster-demo showcasing use of ephemera-kubernetes provider. Will be adding the vote example using the new provider in separate PR.
  • Please also try this out when you review it.

@shumailxyz shumailxyz changed the title WIP: ephemeral cluster provider feat: add ephemeral kubernetes provider Sep 7, 2023
@shumailxyz shumailxyz marked this pull request as ready for review September 7, 2023 16:32
10ko
10ko previously requested changes Sep 12, 2023
Copy link
Member

@10ko 10ko left a comment

Choose a reason for hiding this comment

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

Woop woop! I added a bunch of corrections and questions, mostly around the docs! Thanks for the PR @shumailxyz !!

core/src/plugins/kubernetes/container/handlers.ts Outdated Show resolved Hide resolved
examples/ephemeral-cluster-demo/README.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Outdated Show resolved Hide resolved
docs/basics/ephemeral-clusters.md Show resolved Hide resolved
@shumailxyz
Copy link
Contributor Author

@10ko Thanks for all the suggestions. I couldn't directly apply suggestions because I had some local changes to docs but manually used those.

Copy link
Collaborator

@twelvemo twelvemo left a comment

Choose a reason for hiding this comment

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

Looks really good! Just one small nitpick and a couple of comments on the existing threads from my side.

core/test/unit/src/plugins/kubernetes/ephemeral.ts Outdated Show resolved Hide resolved
@10ko
Copy link
Member

10ko commented Sep 13, 2023

Good to go for me, at least as a first version, we can iterate on it. I'll leave it to the others to approve, so all requests are taken into consideration.

@shumailxyz shumailxyz dismissed stale reviews from 10ko and worldofgeese September 13, 2023 12:45

dismissing due to being stale

@shumailxyz shumailxyz added this pull request to the merge queue Sep 13, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 13, 2023
@shumailxyz shumailxyz added this pull request to the merge queue Sep 13, 2023
Merged via the queue into main with commit ed0ab01 Sep 13, 2023
42 checks passed
@shumailxyz shumailxyz deleted the feat-ephemeral-cluster branch September 13, 2023 13:41
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.

None yet

5 participants