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/hybrid cloud #902

Merged
merged 121 commits into from
Jul 7, 2023
Merged

Feat/hybrid cloud #902

merged 121 commits into from
Jul 7, 2023

Conversation

MiroslavRepka
Copy link
Contributor

@MiroslavRepka MiroslavRepka commented Jun 26, 2023

This PR implements static node pools for Claudie. It works under three assumptions:

  1. Claudie can access nodes on specified endpoints
  2. Static nodes can access each other
  3. Claudie can ssh into them with root privileges with the specified key referenced from the secret.

The testing of the static nodes included

  • replacement of cloud nodepools with static nodepools (for both, control plane and compute plane)
  • addition and removal of static nodepools (for both, control plane and compute plane)
  • resizing of static nodepools (for both, control plane and compute plane)
  • building cluster from entirely static nodes (no cloud instances)

MiroslavRepka and others added 30 commits June 20, 2023 16:35
Copy link
Contributor

@cloudziu cloudziu left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@MiroslavRepka
Copy link
Contributor Author

Hey guys, could you please check the docs? Once thats done, I would like to merge this PR :).

cc @Despire @cloudziu @MarioUhrik

@katapultcloud
Copy link
Contributor

I was wondering whether the renaming of frontend is compatible with this PR? I still see some residue of the frontend name sprinkled. Otherwise LGTM 👍 nice work

@MiroslavRepka
Copy link
Contributor Author

I still see some residue of the frontend name sprinkled.

I think I would go with the approach where whichever PR gets merged first, the other will deal with the changes. :)

Copy link
Contributor

@cloudziu cloudziu left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Great docs

Copy link
Contributor

@MarioUhrik MarioUhrik left a comment

Choose a reason for hiding this comment

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

LGTM

@MiroslavRepka MiroslavRepka added this pull request to the merge queue Jul 7, 2023
Merged via the queue into master with commit e60fc58 Jul 7, 2023
1 check was pending
@MiroslavRepka MiroslavRepka deleted the feat/hybrid-cloud branch July 7, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants