-
Notifications
You must be signed in to change notification settings - Fork 134
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeError: 'NoneType' object is not subscriptable #19
Comments
Thanks for reporting this! What version of GitLab are you on? Also, if you launch marge-bot with |
Looking at the docs the docs, their example call to |
It looks like only the call that gets a single project will return permissions. |
The behaviour of the GitLab REST API can change with versions and this can be the reason why we see different results. What version of GitLab are you? You should be able to see it under |
9.3.5 ee |
Ok, that's a recent one. What are the permissions the Also, when you run it with |
This is curious... after looking at the debug output gitlab is sending the |
Sometimes though the |
We use group level permissions for our users (since we have several projects). Could that be part of it? |
Believe it or not but some projects the user has access to come back with an empty permissions array:
Even though the user is a master on the group. We tracked this down to a gitlab bug that we have not reported(or found the issue tracking) where subgroups getting moved do not cascade proper permissions down. So we removed our user from these groups and manually re-added. Now we still have group level access
I see two things needing to happen here:
|
Ok, it makes sense now, thank you both for looking into this! It is definitely a bug that marge-bot only checks the The part where GitLab returns both |
I got interested. I can push the first requirement |
…vel access. Handle none access more gracefully
Yes, please, just send us a PR. I can do the second part on top of that. |
Ah, just saw you are doing both, even better :) |
I believe this can be closed now - How does the docker pipeline work? |
We plan to push a new release to docker hub tomorrow, but If you want to build the docker image yourself, something like
should do the trick, but it will take a while to run the first time (and install the nix build system/package manager on your system in |
Sounds good. I noticed the makefile but didn't see anything in travis for automating it. LMK if you need someone to take it While we are in the same sphere of discussion - I wrote a small ktmpl k8s template for a deploy and a secrets file for this. is this a pr you are interested in? it relies upon ktmpl as a polyfilly until templates are part of core |
I wasn't aware of ktmpl as templating solution for k8 and yes, certainly interested in seeing this! Concerning properly automating the docker build, it wasn't in the critical path for getting out the initial release – but it's something we definitely want to do; I really need to go to bed now, but I'll open an issue tomorrow and we can discuss it there. |
I think we can close this now, as the fix has made it into master. |
We're trying to get marge-bot up and running on our own gitlab instance, and might have found a bug?
After looking into this a bit, it looks like gitlabs api will return permissions only on the
/projects/:id
request, but not for/projects
.The text was updated successfully, but these errors were encountered: