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

Add ssh_no_proxy parameter (via @gaserre to address issue #84) #87

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

Conversation

chqr
Copy link

@chqr chqr commented Jul 5, 2017

Via @gaserre:

If ssh_no_proxy is specified, do not use a proxy (port forwarding)
when executing the provision and shutdown steps.
The reason to avoid proxying is that the current implementation does not
handle ssh disconnects properly.
The VNC boot command still uses the proxy.

Glenn Serre and others added 2 commits July 5, 2017 11:15
If ssh_no_proxy is specified, do not use a proxy (port forwarding)
when executing the provision and shutdown steps.
The reason to avoid proxying is that the current implementation does not
handle ssh disconnects properly.
The VNC boot command still uses the proxy.
@rdobson
Copy link
Contributor

rdobson commented Oct 18, 2017

@chqr - can you elaborate on the ssh disconnect issues you're referring to? Perhaps we could fix that?

@chqr
Copy link
Author

chqr commented Oct 20, 2017

@rdobson thanks for the reply! The discussion in #84 has the background on this PR.

The ssh disconnect issues were observed by @gaserre, and were his reason for creating this patch in his fork of packer-builder-xenserver.

My team needs this option for a different reason. In the XenServer cluster where we build our Packer vms, the vms use a different vlan than the Xen hosts. It is impossible to connect to the new vm's IP from the Xen host. It is possible, however, for the Packer process to directly connect to the vm. That's why we need to have a way to disable the ssh proxy.

        [vm vlan]
+--------+    +--------+
| packer | -> | new vm |
+--------+    +--------+
   |
===|===firewall=========
   v
+---------+
| xenhost |
+---------+
        [host vlan]

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.

2 participants