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

allows you to specify the ip spice/vnc listens on #100

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

joshie
Copy link

@joshie joshie commented Jul 16, 2012

use the option "listen-ip": "some.ip" in the vm json to change which ip the vnc/spice connection is listening on,
Note: will continue to use the previous behavior if this is not specified in the json.

use the option "listen-ip": "some.ip" in the vm json
@rmustacc
Copy link
Contributor

Hi Josh, it doesn't appear that you have signed the CLA yet. Before we can take a look at this we'd need you to sign it. More information here: http://wiki.smartos.org/display/DOC/Contributing+to+SmartOS

@joshie
Copy link
Author

joshie commented Jul 16, 2012

Hi, my company (Elysium Digital), has signed the CLA. I was told this was enough. Is this not accurate?

@rmustacc
Copy link
Contributor

Ah, yes, if they have, then that covers it. Sorry, there's not a good way to map github user to company.

@ghost ghost assigned joshwilsdon Aug 9, 2012
@joshie
Copy link
Author

joshie commented Nov 21, 2012

I wanted to ping on this. What do I need to do to get this accepted? I remember getting a list and forgetting it. Someone on the mailing list could apparently use this :)

@nshalman
Copy link
Contributor

nshalman commented Dec 6, 2012

To get this patch accepted should we rebase it to the current master branch?

JohnSonnenschein and others added 20 commits January 18, 2013 11:50
use the option "listen-ip": "some.ip" in the vm json
@Licenser
Copy link

bump, this would be a very welcome addition!

@nshalman
Copy link
Contributor

The hope is for this patch to be re-implemented after IPv6 support makes it into SmartOS.
In particular it really should get some error checking.
With this patch, (at least on the version of vmadmd on which it was implemented), if you specify an IP address that doesn't exist in the GZ, vmadmd crashes.
We get by by having automation code that makes sure everything is correct, but it would be better for vmadm to do the check and bail rather than create something that causes vmadmd to crash.

@joshwilsdon joshwilsdon removed their assignment Dec 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants