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

Add Azure ClusterClass #585

Closed
wants to merge 1 commit into from

Conversation

anmazzotti
Copy link
Contributor

@anmazzotti anmazzotti commented Feb 27, 2025

What this PR does / why we need it:

This is a ClusterClass generic example for RKE2 on Azure.
It allows the user to provision a simple cluster using Community images.
It supports generic but sizeable MachineDeployments.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@anmazzotti anmazzotti force-pushed the add_azure_cluster_class branch from 311a78a to 3fdcd18 Compare February 27, 2025 14:17
@anmazzotti anmazzotti self-assigned this Feb 27, 2025
@anmazzotti anmazzotti added the area/documentation Issues or PRs related to documentation label Feb 27, 2025
@anmazzotti anmazzotti force-pushed the add_azure_cluster_class branch 4 times, most recently from 64ebc43 to 5644bf1 Compare February 28, 2025 16:18
@anmazzotti anmazzotti mentioned this pull request Feb 28, 2025
4 tasks
Signed-off-by: Andrea Mazzotti <andrea.mazzotti@suse.com>
@anmazzotti anmazzotti force-pushed the add_azure_cluster_class branch from 5644bf1 to 976edf7 Compare March 3, 2025 09:02
@anmazzotti anmazzotti marked this pull request as ready for review March 3, 2025 09:02
@anmazzotti anmazzotti requested a review from a team as a code owner March 3, 2025 09:02
Copy link
Contributor

@salasberryfin salasberryfin left a comment

Choose a reason for hiding this comment

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

Thanks @anmazzotti. The content of the PR LGTM. If you're okay with it, let's first find an agreement on what we discussed on #586 and then we can move with merging both PRs.

@anmazzotti
Copy link
Contributor Author

Closing this PR in favor of rancher/turtles#1121

@anmazzotti anmazzotti closed this Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants