Open DingoEatingFuzz opened 3 years ago
Modifying the current shell by exporting a variable is not possible, so printing the message would be way to go and it provides user two important information: what is the expected environment variable name and which value they should set.
-json
and -t
to commands. When these flags are used the NOMAD_TOKEN
message should not displayed. With a structure output option users will be able to modify their current shell session by doing something like:$ eval "export NOMAD_TOKEN=$(nomad acl bootstrap -json | jq ".SecretID")"
Setting an environment variable is also OS specific, so instead of providing an exact command to run (which may not work for the user), the message should explain in broad terms that a NOMAD_TOKEN
environment variable should be set.
Proposal
Either mention the
NOMAD_TOKEN
env var or automatically exportNOMAD_TOKEN
as part ofnomad acl bootstrap
Or
Use-cases
This is a missed opportunity to teach the ACL system inline rather than relying on guides/docs.