Skip to content
This repository has been archived by the owner on Mar 8, 2023. It is now read-only.

#108 Support for local HTTP server #116

Merged
merged 7 commits into from
Dec 23, 2018

Conversation

scbjans
Copy link
Contributor

@scbjans scbjans commented May 30, 2018

fix #108

If the http_directory key is set in the Packer template an HTTP server is started, which is accessible by the created VM.
The IP address on which the HTTP server is started is determined as the first non-loopback interface IP address; if needed this can be overridden by setting the http_ip key in the Packer template.

@scbjans scbjans changed the title Sander/issue 108 #108 Support for local HTTP server May 30, 2018
@scbjans
Copy link
Contributor Author

scbjans commented May 30, 2018

@mkuzmin could you please review this as PR for #108

@mkuzmin mkuzmin added the next label Nov 1, 2018
@mkuzmin mkuzmin changed the base branch from master to http-server December 23, 2018 00:17
@mkuzmin mkuzmin merged commit 9cf0420 into jetbrains-infra:http-server Dec 23, 2018
@mkuzmin mkuzmin removed the next label Dec 23, 2018
mkuzmin added a commit that referenced this pull request Dec 23, 2018
@mkuzmin
Copy link
Contributor

mkuzmin commented Dec 23, 2018

Thank you!

@mkuzmin
Copy link
Contributor

mkuzmin commented Dec 24, 2018

I’ll quote my comment from #108 (comment) here:

But I would still prefer floppy approach if possible.
HTTP connection is easy in VirtualBox when Packer and VM are within localhost.

Our scenario is remote, and there are multiple issues:

  • Desktop firewall can block incoming connections to Packer process.
  • Using Docker containers, Packer is put behind NAT. You can configure port forwarding, but then you have to explicitly announce its IP address.
  • When Packer machine has multiple network interfaces, the wrong one may be selected. This could be improved, but requires even more logic.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants