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

fix(RELEASE-1338): new build for old catalog index #253

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

theflockers
Copy link
Contributor

this PR make the iib-add-fbc-fragment to require a new IIB build when the resumed catalog is older than the upstream one to ensure the catalog integrity.

Signed-off-by: Leandro Mendes [email protected]

@theflockers theflockers requested a review from a team as a code owner December 9, 2024 11:54
@theflockers theflockers marked this pull request as draft December 9, 2024 11:59
@theflockers theflockers force-pushed the release-1338 branch 3 times, most recently from 8c7ce2f to 97a2fef Compare December 9, 2024 12:14
this PR makes the iib-add-fbc-fragment-to-index-image task
to call a new IIB build when the resumed catalog index is
older than the upstream one to ensure the catalog integrity.

Signed-off-by: Leandro Mendes <[email protected]>
@theflockers theflockers marked this pull request as ready for review December 9, 2024 15:36
@johnbieren
Copy link
Contributor

This is a new required parameter, so it is a breaking change to the task creating the InternalRequest, no?

@theflockers
Copy link
Contributor Author

This is a new required parameter, so it is a breaking change to the task creating the InternalRequest, no?

luckily it is not a breaking change because this parameter is already sent by the managed pipeline task. I suspect (did not look tbh) this parameter was dropped some time ago but it was not changed in the managed one.

https://github.com/konflux-ci/release-service-catalog/blob/1b1616f3bb7e14b2a85cb6d2989ab18b228b2fe9/tasks/add-fbc-contribution/add-fbc-contribution.yaml#L136C1-L137C1

@theflockers theflockers merged commit 0b8b959 into hacbs-release:main Dec 18, 2024
4 checks passed
@theflockers theflockers deleted the release-1338 branch December 18, 2024 16:59
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.

2 participants