Skip to content

Conversation

MSpencer87
Copy link
Contributor

No description provided.

Comment on lines 5 to 12
registry_ip:
type: string
readOnly: true
description: "Public IP address of the proxy bastion node"
registry_private_ip:
type: string
readOnly: true
description: "Private IP address of the proxy bastion node"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this needed?

Comment on lines 7 to 11
export HTTP_PROXY=http://${CORRAL_registry_private_ip}:3219
export HTTPS_PROXY=http://${CORRAL_registry_private_ip}:3219
export http_proxy=http://${CORRAL_registry_private_ip}:3219
export https_proxy=http://${CORRAL_registry_private_ip}:3219
export proxy_host=${CORRAL_registry_private_ip}:3219
Copy link
Contributor

Choose a reason for hiding this comment

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

We already implemented half of the registry -> bastion rename output. Would this work if we used bastion_private_ip instead of registry_private_ip?

Copy link
Contributor

@markusewalker markusewalker left a comment

Choose a reason for hiding this comment

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

LGTM, awaiting offline confirmation that this is good to merge.

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.

3 participants