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 proxy settings to ova #32

Merged
merged 1 commit into from
Jan 18, 2020
Merged

Add proxy settings to ova #32

merged 1 commit into from
Jan 18, 2020

Conversation

rguske
Copy link
Contributor

@rguske rguske commented Jan 6, 2020

Signed-off-by: Robert Guske rguske@vmware.com

Fixes #40

Provides proxy settings to enable the provisioning of VEBA in environments that allows Internet connection through a proxy only. It is also necessary for functions that require an outgoing connection in such environments.

  • changes implemented
  • tested
  • validation that the provision of VEBA behind a proxy is successful

image
image

@lamw lamw self-requested a review January 7, 2020 14:07
files/setup.sh Outdated Show resolved Hide resolved
manual/photon.xml.template Outdated Show resolved Hide resolved
manual/photon.xml.template Outdated Show resolved Hide resolved
Copy link
Contributor

@lamw lamw left a comment

Choose a reason for hiding this comment

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

You may want to reconsider the Proxy input as mentioned in the review and see if breaking up the individual components of the Proxy URL would be better user experience.

manual/photon.xml.template Outdated Show resolved Hide resolved
@rguske
Copy link
Contributor Author

rguske commented Jan 16, 2020

I´ve tested all changes locally with proxy settings set and not set.
VEBA configuration at startup:
image
Configured Proxy settings:
image

- code implemented in the corresponding files
- test build created and validated
- discovered errors fixed from the review

Signed-off-by: Robert Guske <rguske@vmware.com>
@lamw lamw merged commit 27bced9 into vmware-samples:development Jan 18, 2020
@sofixa
Copy link
Contributor

sofixa commented Jan 21, 2020

@lamw @rguske actually that won't suffice since Docker has its own proxy settings that need to be set as well: https://docs.docker.com/config/daemon/systemd/#httphttps-proxy

@rguske
Copy link
Contributor Author

rguske commented Jan 21, 2020

Thanks for bringing it up @sofixa! I´ll look into it.

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.

Allow network proxy settings during deployment
4 participants