-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[QE][AWS] Add support for installing private&disconnected cluster #24888
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
[QE][AWS] Add support for installing private&disconnected cluster #24888
Conversation
|
@yunjiang29 we already have |
9c08810 to
c9022db
Compare
b49746c to
88f54c4
Compare
bbb48a5 to
ef06df0
Compare
|
/test pj-rehearse |
1 similar comment
|
/test pj-rehearse |
|
/retest |
f7886af to
4b1c7a6
Compare
|
/test pj-rehearse |
1 similar comment
|
/test pj-rehearse |
c966e47 to
21143f3
Compare
|
@jianli-wei @jianlinliu all comments have been addressed, PTAL, thanks. |
21143f3 to
497ccf8
Compare
|
/retest |
...erator/step-registry/aws/deprovision/appsdnsrecord/aws-deprovision-appsdnsrecord-commands.sh
Outdated
Show resolved
Hide resolved
af1e0fe to
4842730
Compare
|
@jianli-wei @jianlinliu PTAL |
|
PR looks good to me, but seem like installer-rehearse-installer-rehearse-aws-disconnected-private job failed. |
4842730 to
b54ea4e
Compare
yes, it's a flake while pulling images, re-testing |
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.
Why not just "aws-provision-vpc"?
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.
following some kinds of manner, we call it customer/existing/shared VPCs
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.
In GCP, the step "gcp-provision-vpc" would write gcp network settings into file "${SHARED_DIR}/customer_vpc_subnets.yaml", which would be used in the step "ipi-conf-gcp". The questions, is it really necessary to add the step "ipi-conf-aws-custom-vpc" besides the existing step "ipi-conf-aws"?
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.
it does an option, I'd like to keep the step as simple as possible, ipi-conf-aws-custom-vpc just focus on subnets/AZs' settings.
ci-operator/step-registry/aws/deprovision/s3buckets/aws-deprovision-s3buckets-ref.yaml
Outdated
Show resolved
Hide resolved
ci-operator/step-registry/aws/deprovision/stacks/aws-deprovision-stacks-ref.yaml
Outdated
Show resolved
Hide resolved
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.
How about renaming as "aws-provision-static-iam-users" (i.e. remove the word "cco-manual")?
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.
cco-manual means these users are for CCO in manual mode, I think it's more clear.
b54ea4e to
b740442
Compare
|
/lgtm |
|
/lgtm |
b740442 to
2a5e339
Compare
|
@yunjiang29: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/lgtm |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: droslean, jianli-wei, jianlinliu, yunjiang29 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@yunjiang29: Updated the following 3 configmaps:
DetailsIn 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. |
Allow creating a cluster in a disconnected network: