-
Notifications
You must be signed in to change notification settings - Fork 84
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 docs #634
Add proxy docs #634
Conversation
Changes Unknown when pulling 282e7bf on eriknelson:proxy-docs into ** on openshift:master**. |
282e7bf
to
2aa7d5c
Compare
Changes Unknown when pulling 2aa7d5c on eriknelson:proxy-docs into ** on openshift:master**. |
2aa7d5c
to
63a8350
Compare
Changes Unknown when pulling 63a8350 on eriknelson:proxy-docs into ** on openshift:master**. |
docs/proxy.md
Outdated
|
||
Finally, it's possible the services spawned by the APB may also require external | ||
network access via proxy. The APB *must* be authored to set these environment variables | ||
explicitly if if recognizes them in its own execution environment, or the cluster |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dup "if" in here: "f if recognizes them"
@eriknelson Others LGTM. Thx. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor changes. Otherwise, good doc.
docs/proxy.md
Outdated
|
||
It is recommended to review the [Openshift Documentation ](https://docs.openshift.com/container-platform/3.7/install_config/http_proxies.html) | ||
and to configure a cluster accordingly before attempting to run the broker behind | ||
a proxy. When running an Ansible Service Broker inside of a proxied Openshift cluster, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably best to start using "Ansible Broker".
TYPO: Openshift should be OpenShift.
docs/proxy.md
Outdated
[edit the broker's DeploymentConfig and set the HTTP_PROXY, HTTPS_PROXY, and NO_PROXY environment variables](https://docs.openshift.com/container-platform/3.7/install_config/http_proxies.html#setting-environment-variables-in-pods). | ||
It's common that APB pods themselves may require external access via proxy as well. | ||
If the broker recognizes it has a proxy configuration, it will transparently | ||
apply this env vars to the APB pods that it spawns. As long as the modules used |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TYPO: this -> these
742b004
to
dfb5325
Compare
dfb5325
to
479cbe3
Compare
No description provided.