Skip to content

submit: continue superseding on per-SR HTTP 403 errors#2135

Open
SAY-5 wants to merge 1 commit into
openSUSE:masterfrom
SAY-5:fix/supersede-403-2130
Open

submit: continue superseding on per-SR HTTP 403 errors#2135
SAY-5 wants to merge 1 commit into
openSUSE:masterfrom
SAY-5:fix/supersede-403-2130

Conversation

@SAY-5
Copy link
Copy Markdown

@SAY-5 SAY-5 commented May 5, 2026

Catch HTTP 403 on each individual change_request_state call when superseding existing SRs, log a warning, and continue with the remaining requests instead of aborting the whole loop. Fixes #2130.

@dmach
Copy link
Copy Markdown
Contributor

dmach commented May 11, 2026

Wouldn't be better to extend change_request_state() with an argument, say can_fail: bool = False instead of adding a new function?

@SAY-5
Copy link
Copy Markdown
Author

SAY-5 commented May 11, 2026

Good point, happy to refactor as a can_fail: bool = False kwarg on change_request_state(). I'll push that shortly.

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.

osc does not supersede older SR, if one of the SR lacks permission

2 participants