Skip to content

Use safe dict access for yanked fields - #1323

Merged
jobselko merged 1 commit into
pulp:mainfrom
jobselko:yank_access
Aug 7, 2026
Merged

Use safe dict access for yanked fields#1323
jobselko merged 1 commit into
pulp:mainfrom
jobselko:yank_access

Conversation

@jobselko

@jobselko jobselko commented Aug 6, 2026

Copy link
Copy Markdown
Member

Follow-up to #1282

This is needed to avoid a KeyError at runtime for pulp-service, whose patch 0060 passes release dicts to write_simple_detail_json without yanked and yanked_reason keys.

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@jobselko jobselko self-assigned this Aug 6, 2026
@jobselko
jobselko merged commit dab5e2b into pulp:main Aug 7, 2026
18 of 24 checks passed
@jobselko
jobselko deleted the yank_access branch August 7, 2026 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gerrod3 gerrod3 gerrod3 approved these changes

Assignees

@jobselko jobselko

Labels

no-changelog no-issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jobselko @gerrod3