feat(mass-tag): script is using new koji api endpoint that checks permissions before making actions and added verifications #12827

Merged
jnsamyak merged 2 commits from mass_tag_improvement into main 2026-06-08 03:10:27 +00:00
Owner

Signed-off-by: Anton Medvedev amedvede@redhat.com

Signed-off-by: Anton Medvedev <amedvede@redhat.com>
Owner

I like the verifying part... but the checking side... this particular case required autosign, but also required secure-boot for kernel.

Perhaps instead calling checkTagAccess might be better?

I like the verifying part... but the checking side... this particular case required autosign, but also required secure-boot for kernel. Perhaps instead calling checkTagAccess might be better?
Author
Owner

rebased onto 4b0ee5ba43

rebased onto 4b0ee5ba438cbfcb02a99da0539422b09f4571ab
Author
Owner

rebased onto a104560c13

rebased onto a104560c139ee0c7b5799a5b4791d055269d5dc2
Author
Owner

I like the verifying part... but the checking side... this particular case required autosign, but also required secure-boot for kernel.

Perhaps instead calling checkTagAccess might be better?

Thanks for advice and I agree that the way you suggested is better, but this method is also not ideal, It's telling that I have tag access, but I couldn't proccess side tag merge since it required secure-boot for kernel

> I like the verifying part... but the checking side... this particular case required autosign, but also required secure-boot for kernel. > > Perhaps instead calling checkTagAccess might be better? Thanks for advice and I agree that the way you suggested is better, but this method is also not ideal, It's telling that I have tag access, but I couldn't proccess side tag merge since it required `secure-boot` for kernel
Author
Owner

1 new commit added

  • feat(mass-tag): added verification of tag access for user
**1 new commit added** * ``feat(mass-tag): added verification of tag access for user``
Owner

rebased onto 859dac161a

rebased onto 859dac161a639ee8a1d9f781d92bb30cefec42ab
Owner

I think it looks reasonable now.

I think it looks reasonable now.
Owner

Pull-Request has been merged by kevin

Pull-Request has been merged by kevin
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
releng/tickets!12827
No description provided.