Skip to content

Remove simple_content_access field from Organization entity#1405

Merged
ogajduse merged 1 commit intoSatelliteQE:masterfrom
zjhuntin:remove_simple_content_access
Feb 17, 2026
Merged

Remove simple_content_access field from Organization entity#1405
ogajduse merged 1 commit intoSatelliteQE:masterfrom
zjhuntin:remove_simple_content_access

Conversation

@zjhuntin
Copy link
Contributor

The simple_content_access field was removed from Katello's Organization API response in Katello PR #11616 as SCA is now the only content access mode. This field is no longer returned by the API, causing KeyError when reading Organization entities.

Description of changes

Remove simple_content_access from the organization entity as it was removed in Katello.

Upstream API documentation, plugin, or feature links

Katello/katello#11616

The simple_content_access field was removed from Katello's Organization
API response in Katello PR #11616 as SCA is now the only content access
mode. This field is no longer returned by the API, causing KeyError when
reading Organization entities.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@zjhuntin zjhuntin added the No-CherryPick PR doesnt need CherryPick to previous branches label Feb 17, 2026
Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • Consider whether this removal needs any compatibility handling for environments still talking to older Katello versions that may still return simple_content_access, and if so, gate the field definition or parsing accordingly.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Consider whether this removal needs any compatibility handling for environments still talking to older Katello versions that may still return `simple_content_access`, and if so, gate the field definition or parsing accordingly.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@ogajduse ogajduse merged commit 0f00bba into SatelliteQE:master Feb 17, 2026
14 of 15 checks passed
@zjhuntin zjhuntin deleted the remove_simple_content_access branch February 17, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No-CherryPick PR doesnt need CherryPick to previous branches Stream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants