Cannot update/patch protected branch allowed_to_push/push_access_levels state #2850
Comments
|
I think historically we have left issues like this up to the user to handle. As I think the idea of python-gitlab is to be a thin wrapper over the GitLab API. The positive to doing this is it makes things easier for the users. The negative is that it adds more work for the volunteer developers of python-gitlab to add this and there are other inconsistencies in the upstream API that would thus wanted to be added. And then the maintenance of this if GitLab changes the API. This might be a good thing to raise upstream and ask GitLab to resolve this inconsistency. Interested to get @nejch opinion about this. @nickbroon Off topic, I was in Edinburgh just two days ago 😄 But now back in the Portland, Oregon area. |
|
This issue was marked stale because it has been open 60 days with no activity. Please remove the stale label or comment on this issue. Otherwise, it will be closed in 15 days. |
|
The only work around I can think to allow modifying branch permissions is for the user to GET |
@nickbroon this is what I do, yes, I call an I'm honestly just afraid we'd deviate from upstream parameters if we were making aliases/mappings between GET and PUT/POST/PATCH endpoints, and then confuse users even more. You wouldn't really be updating I'm assuming we'd be doing this in a generic way and not for every possible endpoint, similar to how we have |
|
This issue was marked stale because it has been open 60 days with no activity. Please remove the stale label or comment on this issue. Otherwise, it will be closed in 15 days. |
|
I don't have permissions to remove the 'stale' label. But feel free to add the 'wont-fix' label and close this; I understand the libraries design decision to not translate between non-symmetric GET and PUSH/PATCH rest API in save/update mixins. |
|
This issue was marked stale because it has been open 60 days with no activity. Please remove the stale label or comment on this issue. Otherwise, it will be closed in 15 days. |
|
I don't have permissions to remove the 'stale' label. But feel free to add the 'wont-fix' label and close this; I understand the libraries design decision to not translate between non-symmetric |
Thank you. Will do. |
Description of the problem, including code/CLI snippet
Breaking out discussion from this Pull Request thread into its own issue.
The Save and Update mixins for
protectedbranchdoesn't really work for most of it's attributes, as the PATCH/POST api use different values from those return by the API.The PATCH api requires to send
allowed_to_push attribute, but the returned attribute ispush_access_levels, as documented in the API and examples: https://docs.gitlab.com/ee/api/protected_branches.html#example-update-a-push_access_level-recordExpected Behavior
To be able to update the `allowed_to_push' GitLab protected branch state.
Actual Behavior
Using the python-gitlab library to modify the
push_access_levelsattribute that the object has and then calling.save()appears to do nothing, and the python object has noallowed_to_pushattribute to modify.It does work for the basic
.allow_force_pushprotectedbranch attribute, as added and tested in the original Pull Request.Specifications
v4gitlab.comThe text was updated successfully, but these errors were encountered: