-
Notifications
You must be signed in to change notification settings - Fork 7
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
updates from migration example qa/review #89
Conversation
e57e33d
to
d491b16
Compare
-L 19440:$nutanix_cluster1_cvim_ip_address:9440 \ | ||
-i $nutanix_cluster1_ssh_private_key \ | ||
-J root@$nutanix_cluster1_bastion_public_ip \ | ||
admin@$nutanix_cluster1_cvim_ip_address |
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.
If the Nutanix clusters see the bastion, .2, as their gateway, adding this route to the bastion should work, but it would add an extra hop.
We could have 0 hops with https://github.com/equinix-labs/terraform-equinix-metal-nutanix-cluster/pull/89/files#r1684973671
I'm just calling this out. Not expecting a change in the root module at this time.
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.
When I was testing, CVIM was not available. If it was up, I imagine we could make this a one-liner, with \
at the end of the admin
line and sudo ip route
as the command run directly from the single ssh
command line.
We should explain that we are using ssh -J
to use the bastion as a Jump Host.
d491b16
to
515236d
Compare
CI is acting really wonky. It appears to be pulling from old revisions that are neither the target nor HEAD of this branch.
I didn't have that line, Update: I didn't notice that I was behind main where the generated section had already been added. rebased. |
91648a2
to
c73c523
Compare
I verified and updated the changes. Using an odd Nutanix node count, I successfully provisioned clusters with node counts of 1 and 3. I followed the steps and performed a VM migration using a node count of 1." Let me know if any adjustments are needed! |
7450f11
to
6e44f9c
Compare
Signed-off-by: Marques Johansson <[email protected]>
Signed-off-by: Marques Johansson <[email protected]>
Signed-off-by: Marques Johansson <[email protected]>
6e44f9c
to
e7e7195
Compare
METAL_AUTH_TOKEN and METAL_ORGANIZATION_ID secrets have been added |
Signed-off-by: Marques Johansson <[email protected]>
5fdde78
to
88d29bc
Compare
WIP to review the migration example
Follow up to #71
Part of #68
Fixes #74