Project

Profile

Help

Issue #9589

Updated by lmjachky over 2 years ago

Recent CI failures (https://github.com/pulp/pulp_container/runs/4348993881?check_suite_focus=true#step:13:1449), [CI failures](4https://github.com/pulp/pulp_container/runs/4348993881?check_suite_focus=true#step:13:1449), that were triggered after merging the commit [ab6225b6cfd29864a51baac4d9ad385b37983529](https://github.com/pulp/pulpcore/commit/ab6225b6cfd29864a51baac4d9ad385b37983529), caused pulp_container to fail when [checking](https://github.com/pulp/pulp_container/blob/aa0f6783d7fcfbf0f886a09d1ac11b4f28da8e3c/pulp_container/app/viewsets.py#L141-L178) object permissions. 

 We should at least perform similar checks to those in [`access_policy.py`](https://github.com/pulp/pulp_container/blob/09a63966a2ace4bc15c61b5b1e455cd7018c20c9/pulp_container/app/access_policy.py#L15-L31) and add a fallback that does not rigidly check for `view_containerpushrepository` and `view_containerrepository` in a strict order.

Back