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
[CVP-1588] Set default baseDomain variable inside for ipi stage in step-registry #15135
[CVP-1588] Set default baseDomain variable inside for ipi stage in step-registry #15135
Conversation
@dirgim PTAL |
- name: BASE_DOMAIN | ||
default: "origin-ci-int-aws.dev.rhcloud.com" | ||
documentation: |- | ||
Base domain is used for setting baseDomain variable in Config template used in ipi-conf-aws-commands.sh |
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.
Maybe changing this to something like A fully-qualified domain or subdomain name. The base domain of the cloud provider is used for setting baseDomain variable of the install configuration of 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.
Done
8c4a91f
to
b5d5eb1
Compare
/retitle [CVP-1588] Set default baseDomain variable inside for ipi stage in step-registry |
Looks reasonable to me, but I'd like to see at least a few green rehearsals. |
Has some green now. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: samvarankashyap, wking The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
@samvarankashyap: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
@samvarankashyap: Updated the following 2 configmaps:
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
$subject,
These changes help up overriding the baseDomain variable inside the config template of ipi stage.