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
Update declarative assets for Nutanix #5636
Update declarative assets for Nutanix #5636
Conversation
thunderboltsid
commented
Feb 10, 2022
•
edited
edited
- Update bootkube template to copy static resources for nutanix under data/data/bootstrap/files
- Add terraform files for Nutanix platform under data/data/nutanix
- Add terraform stages for Nutanix platform under pkg/terraform/stages/nutanix
4d25ef7
to
e8626da
Compare
/retest-required |
e8626da
to
c3bb9e9
Compare
/lgtm My only comment here is that it would have been helpful to have |
apologies, this comment is not relevant to this PR |
/lgtm cancel |
/retest-required |
1 similar comment
/retest-required |
@thunderboltsid would you mind taking a look at the merge conflict? |
c3bb9e9
to
affe38b
Compare
/retest-required |
1 similar comment
/retest-required |
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.
Found a nit and had a few questions on the terraform provider timeout value.
affe38b
to
cfb4d3d
Compare
/retest-required |
1 similar comment
/retest-required |
/lgtm |
Copy static assets for nutanix in bootkube.sh.template
/retest-required |
/retest-required |
1 similar comment
/retest-required |
/lgtm |
Add terraform files for Nutanix platform under data/data/nutanix as two stages: - Bootstrap uploads the bootstrap ignition image and provisions the bootstrap node. - Cluster creates the categories, uploads the rhcos image, and provisions the master nodes in the cluster.
Add stages for nutanix terraform files.
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: staebler 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-required Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
/retest-required Please review the full test history for this PR and help us cut down flakes. |
@thunderboltsid: 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. |