Skip to content

Conversation

@Jarema
Copy link
Member

@Jarema Jarema commented Oct 24, 2025

When Dependabot bumps natsio/jetstream-controller version, the workflow will now automatically download the corresponding CRDs from the NACK release and commit them along with the version bump.

This ensures CRDs always stay synchronized with the controller version and eliminates manual CRD update steps.

Signed-off-by: Tomasz Pietrek [email protected]

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR automates CRD (Custom Resource Definition) downloads when Dependabot bumps the natsio/jetstream-controller version. The workflow now automatically fetches the corresponding CRDs from the NACK release and commits them alongside the version bump, ensuring CRDs stay synchronized with the controller version.

Key changes:

  • Added automatic CRD download logic for NACK controller updates
  • Enhanced commit messages to indicate when CRDs are included
  • Implemented error handling for failed CRD downloads

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

When Dependabot bumps natsio/jetstream-controller version, the workflow
will now automatically download the corresponding CRDs from the NACK
release and commit them along with the version bump.

Changes:
- Download CRDs from NACK releases when controller version is bumped
- Ensure CRD directory exists with mkdir -p
- Fail the job if CRD download fails (prevents version/CRD mismatch)
- Update commit message to indicate CRDs are included

This ensures CRDs always stay synchronized with the controller version
and eliminates manual CRD update steps.

Signed-off-by: Tomasz Pietrek <[email protected]>
@Jarema Jarema force-pushed the automate-crd-downloads branch from ee4ba46 to f502357 Compare October 24, 2025 10:18
Copy link
Member

@samuelattwood samuelattwood left a comment

Choose a reason for hiding this comment

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

LGTM

Replaces #1059

@Jarema Jarema merged commit 2fee2c2 into main Oct 24, 2025
4 checks passed
@Jarema Jarema deleted the automate-crd-downloads branch October 24, 2025 20:14
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.

3 participants