Multinode Wallaby #246
Merged
Multinode Wallaby #246
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Maybe the VXLAN role could assert required values are provided.
This reduction is required as VMs placed on the `admin-vxlan` network cannot be accessed.
jackhodgkiss
added a commit
that referenced
this pull request
Nov 24, 2022
jackhodgkiss
added a commit
that referenced
this pull request
Nov 24, 2022
jackhodgkiss
added a commit
that referenced
this pull request
Nov 25, 2022
jackhodgkiss
added a commit
that referenced
this pull request
Nov 25, 2022
jackhodgkiss
added a commit
that referenced
this pull request
Nov 25, 2022
jackhodgkiss
added a commit
that referenced
this pull request
Nov 25, 2022
markgoddard
suggested changes
Dec 1, 2022
| # External network | ||
| external_cidr: 192.168.38.0/24 | ||
| external_mtu: 1400 | ||
| external_mtu: 1350 |
There was a problem hiding this comment.
I think @sd109 hit this with his aufn work, and possibly @darmach with his magnum work. It's because we're using VXLAN networks with OVN, but internally it sometimes uses GENEVE tunnels which have a larger header size.
One option is to switch to geneve tenant networks when using OVN, which generally work better.
Comment on lines
+18
to
+21
| # Octavia load balancer configuration | ||
| octavia_auto_configure: "no" | ||
| octavia_provider_drivers: "ovn:OVN provider" | ||
| octavia_provider_agents: "ovn" |
There was a problem hiding this comment.
We probably only want this when kolla_enable_ovn is true.
| kolla_enable_cinder: true | ||
| kolla_enable_cinder_backup: true | ||
| kolla_enable_neutron_provider_networks: true | ||
| kolla_enable_ovn: true |
There was a problem hiding this comment.
Unsure whether we should change this default. Ideally it should be possible & easy to use either OVS or OVN.
| kolla_enable_cinder_backup: true | ||
| kolla_enable_neutron_provider_networks: true | ||
| kolla_enable_ovn: true | ||
| kolla_enable_octavia: true |
There was a problem hiding this comment.
Perhaps only enable octavia when OVN is enabled, since we don't have the config for the amphora provider in place currently.
markgoddard
pushed a commit
that referenced
this pull request
Jan 31, 2023
* feat: copy `ci-multinode` from PR `#246` * feat: copy `ansible` from PR `#246` * revert: configure-aio-resources.yml to `da05599` * feat: actually copy contents of `ci-multinode` For some reason the previous commit does not contain the important changes to the configuration. This commit fixes that. * feat: include `.automation.conf` from PR `#246` * revert: `stackhpc-ci` back to `stackhpc/xena` * feat: update address of `pulp-server` * feat: bump `stackhpc.cephadm` version * feat: `haproxy` group has been replaced with `loadbalancer` * fix: ensure `cephadm_host_labels` is defined * feat: reduce concurrency and remove load list * feat: increase `max_microversion` * fix: restore order to `requirements.yml` * fix: README has been moved * fix: remove unused collection * fix: add missing new line * feat: disable some unwanted service within ci-multinode * fix: remove extra blank lines * feat: revert `TEMPEST_CONCURRENCY` to two * feat: load `ci-multinode` tempest settings conditionally Firstly, the skip-list has been renamed from `tempest-full` to `ci-multinode` as this list would be environment specific. Secondly the `config.sh` script now contains a conditional for the `ci-multinode` environment allowing for the test list and concurrency settings to be loaded without impacting other environments. * fix: use `HTTP` dashboard URL over `HTTPS` * fix: reduce multinode tempest concurrency * feat: bump `stackhpc.cephadm` collection version * feat: remove unecessary comment
markgoddard
reviewed
Feb 6, 2023
markgoddard
reviewed
Feb 6, 2023
|
You'll probably also want to merge in stackhpc/wallaby, maybe in a separate PR. |
markgoddard
approved these changes
Feb 6, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
_bootstrap_userfor all groups to usecloud-userseedwithingrowroot_groupasseeduses LVM imagesMTUonexternal_networkto enable communication with VMs onadmin-vxlan