Skip to content

ci: improve fetch Core manifest workflow#2205

Open
anmazzotti wants to merge 1 commit intomainfrom
improve_fetch_core_manifest_workflow
Open

ci: improve fetch Core manifest workflow#2205
anmazzotti wants to merge 1 commit intomainfrom
improve_fetch_core_manifest_workflow

Conversation

@anmazzotti
Copy link
Contributor

What this PR does / why we need it:

This PR introduces various improvements around the Core CAPI Provider manifest update workflow and related tooling.
In particular:

  1. go.mod updated to ignore other unrelated projects
  2. remove redundant crust-gather binary installation and default to krew
  3. always install the clusterctl-operator when configuring kubectl and krew
  4. cleanup fetch-core-capi.sh script
  5. add make update-core-capi-manifest for easy usage
  6. cleanup github action workflow and remove cron trigger, this is on-demand job only
  7. dump unzipped manifest as artifacts on update jobs

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

Special notes for your reviewer:

Checklist:

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

@anmazzotti anmazzotti self-assigned this Mar 11, 2026
@anmazzotti anmazzotti added area/ci kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. labels Mar 11, 2026
@anmazzotti anmazzotti moved this to In Progress (8 max) in CAPI / Turtles Mar 11, 2026
@anmazzotti anmazzotti force-pushed the improve_fetch_core_manifest_workflow branch 4 times, most recently from 7e4c274 to 58b3391 Compare March 11, 2026 11:28
@anmazzotti anmazzotti closed this Mar 11, 2026
@anmazzotti anmazzotti force-pushed the improve_fetch_core_manifest_workflow branch from 58b3391 to c8d07ae Compare March 11, 2026 11:30
@github-project-automation github-project-automation bot moved this from In Progress (8 max) to Done in CAPI / Turtles Mar 11, 2026
@anmazzotti anmazzotti reopened this Mar 11, 2026
@anmazzotti anmazzotti force-pushed the improve_fetch_core_manifest_workflow branch 5 times, most recently from 0c416e6 to 03e6638 Compare March 11, 2026 12:08
@anmazzotti anmazzotti moved this from Done to In Progress (8 max) in CAPI / Turtles Mar 11, 2026
@anmazzotti anmazzotti force-pushed the improve_fetch_core_manifest_workflow branch from 03e6638 to 0f59025 Compare March 12, 2026 09:28
Signed-off-by: Andrea Mazzotti <andrea.mazzotti@suse.com>
@anmazzotti anmazzotti force-pushed the improve_fetch_core_manifest_workflow branch from 0f59025 to e58ec94 Compare March 12, 2026 09:31
@anmazzotti anmazzotti marked this pull request as ready for review March 12, 2026 09:33
@anmazzotti anmazzotti requested a review from a team as a code owner March 12, 2026 09:33
@anmazzotti anmazzotti moved this from In Progress (8 max) to PR to be reviewed in CAPI / Turtles Mar 12, 2026
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.

It is way cleaner now.

@salasberryfin salasberryfin enabled auto-merge March 13, 2026 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Projects

Status: PR to be reviewed

Development

Successfully merging this pull request may close these issues.

ci: krew is installed in the root directory when running tests

2 participants