Skip to content

fix(BA-1150): Broken single image rescan on HarborRegistryV2 #4161

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

Merged
merged 2 commits into from
Apr 14, 2025

Conversation

jopemachine
Copy link
Member

@jopemachine jopemachine commented Apr 14, 2025

resolves #4160 (BA-1150).

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue

@github-actions github-actions bot added size:XS ~10 LoC comp:manager Related to Manager component labels Apr 14, 2025
@jopemachine jopemachine changed the title fix: Broken single image rescan on HarborRegistryV2 fix(BA-1150): Broken single image rescan on HarborRegistryV2 Apr 14, 2025
@jopemachine jopemachine added this to the 23.09 milestone Apr 14, 2025
@jopemachine jopemachine marked this pull request as ready for review April 14, 2025 05:52
@Copilot Copilot AI review requested due to automatic review settings April 14, 2025 05:52
Copy link
Contributor

@Copilot 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.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/ai/backend/manager/container_registry/harbor.py:325

  • Ensure that the 'headers' key in rqst_args is always defined to avoid a potential KeyError if it is missing. Consider using rqst_args.get("headers", {}) to safely check for the 'Accept' header.
if "Accept" in rqst_args["headers"]:

@HyeockJinKim HyeockJinKim added this pull request to the merge queue Apr 14, 2025
Merged via the queue into main with commit 96a4019 Apr 14, 2025
21 checks passed
@HyeockJinKim HyeockJinKim deleted the fix/broken_single_image_rescan_on_harbor branch April 14, 2025 06:10
lablup-octodog pushed a commit that referenced this pull request Apr 14, 2025
Backported-from: main (25.5)
Backported-to: 24.03
Backport-of: 4161
lablup-octodog pushed a commit that referenced this pull request Apr 14, 2025
Backported-from: main (25.5)
Backported-to: 23.09
Backport-of: 4161
lablup-octodog pushed a commit that referenced this pull request Apr 14, 2025
Backported-from: main (25.5)
Backported-to: 24.09
Backport-of: 4161
github-merge-queue bot pushed a commit that referenced this pull request Apr 14, 2025
github-merge-queue bot pushed a commit that referenced this pull request Apr 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:manager Related to Manager component size:XS ~10 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Single image rescan not working on HarborRegistry
2 participants