Skip to content

Conversation

Artyop
Copy link
Contributor

@Artyop Artyop commented May 26, 2025

This commit fixes the usage of the hardcoded image directory by using the image_dir parameter instead.

@Artyop Artyop requested a review from a team as a code owner May 26, 2025 09:30
@Artyop Artyop added the release-note/misc This PR makes changes that have no direct user impact. label May 26, 2025
@Artyop Artyop requested a review from rolinh May 26, 2025 09:30
This commit fixes the usage of the hardcoded image directory by using the image_dir parameter instead.

Signed-off-by: Antony Reynaud <antony.reynaud@isovalent.com>
@Artyop Artyop force-pushed the pr/artyop/fix-image-tag-dir branch from b0b7dfa to 4d53cfb Compare May 26, 2025 09:35
@rolinh
Copy link
Member

rolinh commented May 27, 2025

@Artyop Does this fix need to be backported?

@Artyop
Copy link
Contributor Author

Artyop commented May 27, 2025

@Artyop Does this fix need to be backported?

Not really, hardcoded value was the correct one anyway so I'm not sure backporting it would be useful

Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Thanks!

@joestringer
Copy link
Member

/test

@joestringer joestringer enabled auto-merge May 27, 2025 19:23
@joestringer joestringer added this pull request to the merge queue May 28, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label May 28, 2025
Merged via the queue into main with commit 742af8f May 28, 2025
360 of 365 checks passed
@joestringer joestringer deleted the pr/artyop/fix-image-tag-dir branch May 28, 2025 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants