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

Enable CAAPF by default #1004

Merged

Conversation

Danil-Grigorev
Copy link
Contributor

@Danil-Grigorev Danil-Grigorev commented Jan 15, 2025

What this PR does / why we need it:

Enable CAAPF by default for all imported CAPI clusters.

Cluster or namespace may specify cluster-api.cattle.io/disable-fleet-auto-import label to exclude it from automatic import process with fleet.

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 #1002

Special notes for your reviewer:

Checklist:

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

@Danil-Grigorev Danil-Grigorev requested a review from a team as a code owner January 15, 2025 13:32
@Danil-Grigorev Danil-Grigorev force-pushed the caapf-enabled-by-default branch from 936b6e5 to 0d27b9d Compare January 15, 2025 13:41
Signed-off-by: Danil-Grigorev <danil.grigorev@suse.com>
@Danil-Grigorev Danil-Grigorev force-pushed the caapf-enabled-by-default branch from 0d27b9d to 9f433ae Compare January 15, 2025 13:44
@Danil-Grigorev Danil-Grigorev merged commit 2522b5f into rancher:main Jan 15, 2025
12 of 14 checks passed
@Danil-Grigorev
Copy link
Contributor Author

Seems the nightly failure is not related to the missing CAAPF installation. Still seeing this error.

Failed to apply CAPI cluster definition to cluster via Fleet

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.

Enable CAAPF by default
3 participants