-
-
Notifications
You must be signed in to change notification settings - Fork 867
Description
This issue will track backwards-incompatible changes made as part of the transition to Dex (#1888).
We'll decide here how/whether to resolve them and/or just document them.
fly set-team
flags UX
Currently there are new flags for configuring things like GitHub auth, in the form of --github-group org:team
. This is a little confusing since it introduces a "group" term, and it's unclear whether it's a Concourse concept, a Dex concept, a GitHub concept, or a general term. I think we should just keep the flags as-is and convert them under the hood to whatever format is appropriate.
There are some flags that no longer make sense to configure in fly set-team
- notably, the client ID/secret and auth URL endpoints. Let's just remove these. The resolution on the user's end is simple - just stop passing them. The rest of the flags can then stay the same.
We restored the UX to how it was before, caveat being that some flags are now globally-configured.
No longer able to have "no auth"
Now that logging in us user-centric, you have to log in as someone. The --no-really-i-dont-want-any-auth
flag is no more. We're OK with this, and will just change Quick Start, tests, and other reference material to just configure a dummy username and password, or even encourage configuring things like GitHub auth. This is probably better in the long run anyway.