-
Notifications
You must be signed in to change notification settings - Fork 4
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
Only build and publish new ROCK version if rockfile.yaml changed #16
Only build and publish new ROCK version if rockfile.yaml changed #16
Conversation
branch run https://github.com/canonical/fluent-bit-rock/actions/runs/10278894931 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@maci3jka I'm not sure this will work like this.
If you only build the Rocks that have changed then the integration tests down the line will likely fail.
It is fine to build those ROCKs. Just don't publish them.
Looking at the linked test run they would still be published:
https://github.com/canonical/fluent-bit-rock/actions/runs/10278858107/job/28443209885?pr=9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.