Skip to content

Conversation

Alex-Welsh
Copy link
Member

No description provided.

@Alex-Welsh Alex-Welsh requested a review from a team as a code owner February 28, 2023 10:19
@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from 4afb67e to 9c50bdb Compare February 28, 2023 10:33
This change allows ansible facts to be used in
group_vars/*/network-interfaces, which can be used to select the default
network interface. This means different flavors and images can be used
without needing to manually change the interface. It also reduces the
MTU of all networks except external.
@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from 9c50bdb to ba70aa7 Compare February 28, 2023 13:14
@Alex-Welsh Alex-Welsh changed the title Support VM controller and compute hosts in the ci-multinode environment Support VM controller and compute hosts in the ci-multinode environment & re-enable monitoring Feb 28, 2023
@Alex-Welsh Alex-Welsh changed the title Support VM controller and compute hosts in the ci-multinode environment & re-enable monitoring Various multinode fixes and features Feb 28, 2023
@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from 57c377f to fb15500 Compare February 28, 2023 14:00
This fix sets seed_enable_snat and fixes the external IP of the seed which (in
conjunction with an update to the multinode openstack configuration to
set the gateway on the external subnet) allows VMs to reach the
internet.
@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from fb15500 to aa72bfa Compare March 1, 2023 09:22
@Alex-Welsh
Copy link
Member Author

This PR is twinned with this change for the multinode openstack configuration which uses the seed as the gateway on the external subnet.

- name: Configure VXLAN
hosts: controllers,compute,seed,storage
gather_facts: false
gather_facts: true
Copy link
Contributor

Choose a reason for hiding this comment

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

There is potential for some incorrect behaviour here, since facts are gathered as the bootstrap user using the system python interpreter, some facts may be incorrect. In general we disable fact gathering during the user & venv bootstrap phase of host configure to avoid this. I think this is mostly an issue when using fact caching, but I can't remember all the details.

Let's try it like this, but keep an eye out for odd owner/permissions issues.

@@ -0,0 +1,2 @@
external_ips:
alex-mn-seed: 192.168.38.3 No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't going to work if you change the cluster name...

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, painfully obvious once I actually thought about it. Is there some magic way of getting the name of the seed in Kayobe or will it need to be terraform templated?

Copy link
Contributor

Choose a reason for hiding this comment

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

Possibly you could use {{ groups['seed'][0] }} as the key, but I don't think the ip-allocation playbook would handle this.

I think currently terraform does not know about the IP subnets in this config, and it should probably stay that way.

Alternatively we could rely on the fact that kayobe seed host configure runs first and will grab this IP.

Copy link
Member Author

@Alex-Welsh Alex-Welsh Mar 1, 2023

Choose a reason for hiding this comment

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

Possibly you could use {{ groups['seed'][0] }} as the key, but I don't think the ip-allocation playbook would handle this.

I gave that a go but it doesn't seem to template properly. Surprisingly it didn't error out but it also didn't apply the change.

I think we may just have to assume it's on 192.168.38.3 and change either the kayobe or openstack configuration if the environment is different

@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from 3e4a82a to 467153c Compare March 1, 2023 14:22
@Alex-Welsh Alex-Welsh requested a review from markgoddard March 1, 2023 14:23
@Alex-Welsh Alex-Welsh force-pushed the multinode_vm_overcloud branch from 467153c to ac6f534 Compare March 1, 2023 14:28
@Alex-Welsh
Copy link
Member Author

Closing this PR and swtiching target to stackhpc/xena

@Alex-Welsh Alex-Welsh closed this Mar 1, 2023
@Alex-Welsh Alex-Welsh deleted the multinode_vm_overcloud branch March 29, 2023 15:46
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.

2 participants