Skip to content
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

teleport app start fails #31496

Closed
mdwn opened this issue Sep 6, 2023 · 4 comments
Closed

teleport app start fails #31496

mdwn opened this issue Sep 6, 2023 · 4 comments
Labels
bug test-plan-problem Issues which have been surfaced by running the manual release test plan

Comments

@mdwn
Copy link
Contributor

mdwn commented Sep 6, 2023

Expected behavior:

teleport app start should succeed.

Current behavior:

teleport app start fails with an error message. There appears to be no way to supply the expected proxy-server variable.

Bug details:

  • Teleport version: 14.0.0-alpha.2
  • Recreation steps: Run teleport app start, for example:
./teleport app start --name demo-app-2 --uri "http://localhost:9000" --proxy-server punch.teleport.localhost:443 --config /Users/mike/tmp/teleport-ent/config/teleport.yaml
  • Debug logs
➜  teleport-ent ./teleport app start --name demo-app-2 --uri "http://localhost:9000" --auth-server punch.teleport.localhost:443 --config /Users/mike/tmp/teleport-ent/config/teleport.yaml
ERROR: configuration error
initialization failed
config: when app_service is enabled, proxy_server must be specified instead of auth_server
...
➜  teleport-ent ./teleport app start --name demo-app-2 --uri "http://localhost:9000" --proxy-server punch.teleport.localhost:443 --config /Users/mike/tmp/teleport-ent/config/teleport.yaml
teleport: error: unknown long flag '--proxy-server'
@mdwn mdwn added bug test-plan-problem Issues which have been surfaced by running the manual release test plan labels Sep 6, 2023
@zmb3
Copy link
Collaborator

zmb3 commented Sep 6, 2023

Does this repro on v13 as well?

@mdwn
Copy link
Contributor Author

mdwn commented Sep 7, 2023

Does this repro on v13 as well?

Just wanted to acknowledge that I saw this, I'll attempt to repro when I'm able.

@mdwn
Copy link
Contributor Author

mdwn commented Sep 14, 2023

Hey all, sorry for the delay. Yes, this does repro on v13.

@r0mant
Copy link
Collaborator

r0mant commented Sep 14, 2023

I think we wanted to deprecate all teleport xxx start commands anyway in favor of teleport config + teleport start so I'm not sure we should even spend time adding this flag. I propose we just close this as "won't fix".

@r0mant r0mant closed this as not planned Won't fix, can't repro, duplicate, stale Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug test-plan-problem Issues which have been surfaced by running the manual release test plan
Projects
None yet
Development

No branches or pull requests

3 participants