cli icon indicating copy to clipboard operation
cli copied to clipboard

cf enable-ssh requires a restart of the app

Open stefan0001 opened this issue 4 years ago • 4 comments

Please fill out the issue checklist below and provide ALL the requested information.

  • [x] I reviewed open and closed github issues that may be related to my problem.
  • [x] I tried updating to the latest version of the CF CLI to see if it fixed my problem.
  • [x] I attempted to run the command with CF_TRACE=1 to help debug the issue.
  • [x] I am reporting a bug that others will be able to reproduce.

Describe the bug and the command you saw an issue with SSH access does not work after enabling it, because it requires a restart. As stated by the documentation enable-ssh requires a restart of the app to work. However I would consider this as a bug, as it is an unexpected behaviour. https://docs.cloudfoundry.org/devguide/deploy-apps/ssh-apps.html#enable-disable-ssh

What happened SSH access does not work after enabling it, because it requires a restart.

Expected behavior SSH access works directly, without a restart of the app.

Exact Steps To Reproduce

  1. App is deployed with globally set default_app_ssh_access to false.
  2. Enable ssh with cf enable-ssh app
  3. cf ssh app is not working, a restart of the app is required.

Note: After ssh was enabled + app restarted it works, even the ssh will be disbaled & enabled again

Provide more context cf version 7.2.0+be4a5ce2b.2020-12-10

stefan0001 avatar Feb 08 '21 10:02 stefan0001

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/176843301

The labels on this github issue will be updated when the story is started.

cf-gitbot avatar Feb 08 '21 10:02 cf-gitbot

A hint provided by the cf enable-ssh <app-name> command will also help a lot here like with cf bs... provides. Is this something the CF CLI team will accept? I can provide the pull request.

beyhan avatar Jun 07 '21 13:06 beyhan

@beyhan I had a similar idea and added https://github.com/cloudfoundry/cli/pull/2314 which displays a TIP when enabling cf ssh

wanddynosios avatar Sep 09 '22 11:09 wanddynosios

I think this can be closed now because https://github.com/cloudfoundry/cli/pull/2314 got merged.

beyhan avatar Oct 12 '23 20:10 beyhan