Automattic / team51-cli

A tool for provisioning and connecting GitHub repos, Pressable WordPress sites and DeployHQ.
MIT License
12 stars 0 forks source link

Issue granting access to some repos #177

Closed gin0115 closed 1 year ago

gin0115 commented 1 year ago

Had an issue this morning getting access to the JeffGoThelf, took 3 runs to get access to production.

Just tried again to get access to Om.co and again hitting the same problem, on some instances I can grant myself access (as expected), but in others it just errors out.

Ran in --dev mode and grabbed the actual error. It seems the colaborators\ batch_permission endpoint is returning a 502

image

Out of the 4 instances relating to Om.co, I have only been able to get access to 2 of them, no matter how many times I rerun the command.

Spoke to @ahegyes


Can confirm this was run while logged into 1Password OS - Pop!_OS 22.04 LTS x86_64

gin0115 commented 1 year ago

image

gin0115 commented 1 year ago

Appears the error was as a result of the repo being owned by the partner and contractors can not be added via Team51, only the Partner.

See https://a8c.slack.com/archives/C018UMRUN3V/p1671108386937929

Closing this issue as not really an issue.

Sorry for wasting anyones time

ahegyes commented 1 year ago

Even so @gin0115, we can probably enhance the error message to be more descriptive of what the issue is. I'm reopening this issue and assigning myself to it with the following scope:

Work on internal tools is paused until January, so that's most likely when this will commence :)

gin0115 commented 1 year ago

Awesome, as always let me know if you want me to test anything from Linux land :)

ahegyes commented 1 year ago

Closing this issue since the PR for better error handling has been merged.