Page MenuHomePhabricator

`become` error message is confusing when using recursive memberships
Open, Needs TriagePublicBUG REPORT


List of steps to reproduce (step by step, including full links if applicable):

  • Go to tool admin page, e.g.
  • Click "manage maintainers".
  • Under "Tools" add a tool with maintainers, e.g. isa.
  • Click "Update".
  • As a user that is maintainer of the second tool ("isa"), SSH to toolforge and run become isa-dev.

What happens?:
Fails to "become" tool with error message:

You are not a member of the group tools.isa-dev.
Any existing member of the tool's group can add you to that.

What should have happened instead?:
"Becomes" tool.

Additional information
I created isa-dev and added isa as maintainer. @Eugene233 was the user tried and failed to "become" the tool.

Event Timeline

If tools.isa is a maintainer of tools.isa-dev, then you become isa as your user and then become isa-dev from isa.

jjmc89@tools-sgebastion-11:~$ become jjmc89-bot
tools.jjmc89-bot@tools-sgebastion-11:~> become jjmc89-bot-dev

I see. I assumed it worked the same as if I added the maintainers of isa manually. If this is how it's intended to work I think it would be a good idea to clarify that on the maintainers page ( and/or in the instructions for "become".

taavi renamed this task from Maintainers of tools added as maintainers can't use `become` to `become` error message is confusing when using recursive memberships.Dec 17 2022, 5:04 PM