Skip to content

nova: Expose use_serial to UI#793

Open
nicolasbock wants to merge 1 commit intocrowbar:masterfrom
nicolasbock:serial_console
Open

nova: Expose use_serial to UI#793
nicolasbock wants to merge 1 commit intocrowbar:masterfrom
nicolasbock:serial_console

Conversation

@nicolasbock
Copy link
Contributor

Right now the use_serial switch is hidden in the raw view but it's
useful enough to expose it in the UI.

Itxaka
Itxaka previously approved these changes Jan 24, 2017
@Itxaka
Copy link
Member

Itxaka commented Jan 24, 2017

May need to open a bug for the docs to be updated methinks

@nicolasbock
Copy link
Contributor Author

@Itxaka oh, right, totally forgot about that 😄

@nicolasbock
Copy link
Contributor Author

Filed bsc#1021701.

@dirkmueller
Copy link
Contributor

I'm totally not convinced that it should be exposed because;

  • It requires novnc to be disabled
  • It is completely unprotected
  • Only one client can access the console at any time

Was therr any request for this being exposed?

@nicolasbock
Copy link
Contributor Author

@dirkmueller Yes, someone asked me yesterday on #cloud if we could enable it. I figured I would just throw this out on GitHub since it was so straightforward to do 😄. It sounds like we rather shouldn't do it though.

aplanas
aplanas previously approved these changes Jan 25, 2017
@dirkmueller dirkmueller added this to the Cloud 7 Update1 milestone Jan 27, 2017
Copy link
Member

@vuntz vuntz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nicolasbock can you add a hint (we have some in the dns barclamp, if you need an example) to explain to the user the limitations, especially the part about disabling novnc?

Right now the `use_serial` switch is hidden in the raw view but it's
useful enough to expose it in the UI.
@nicolasbock
Copy link
Contributor Author

@vuntz I have added the hints @dirkmueller provided. Is there a way to format the hints more like a list? I tried inserting html, but that didn't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

7 participants