openstack/kayobe
Code Issues Proposed changes

Revert "CI: Disable bare metal testing on RL9/c9s"

Browse Source 
This requires disabling libvirt_vm_trust_guest_rx_filters, which when
enabled triggers the following errors when booting baremetal instances
with Tenks on Libvirt 9 (and most likely since 8.9.0):

    Cannot set interface flags on 'macvtap1': Value too large for defined data type

This is apparently triggered by a Libvirt commit refreshing rx-filters
more often [1].

As explained in I71a2051d8acd63379bd70bc1287a059d4a7f6387, this setting
was added to allow traffic destined for other MAC addresses to reach VMs
when using a macvtap interface.

This will prevent multicast from working, but we don't need it for
baremetal tests in CI.

This setting will be enabled again once the issue is resolved in either
Libvirt or Tenks.

This reverts commit 21c68bbfafe529e1c337ba242c2e501c75bfedaa.

[1] 060d4c83ef

Change-Id: I2cfd2667abb1ae8988b7a7fd9761b75c20a0eaa4
(cherry picked from commit 990370a3673b8bdf4882816926868dd7b422db60)
(cherry picked from commit 46d15a76304f51b02836e37c5bebf1ebab875db5)
This commit is contained in:
Pierre Riteau 2023-10-04 08:57:52 +02:00
parent c4a6883f1b
commit 8b7b8e3739
4 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
dev/tenks-deploy-config-compute-libvirt-on-host.yml
View File

@ -54,3 +54,9 @@ libvirt_vm_emulator: "{% if ansible_facts.os_family == 'RedHat' %}/usr/libexec/q
# Specify a log path in the kolla_logs Docker volume. It is accessible on the # Specify a log path in the kolla_logs Docker volume. It is accessible on the
# host at the same path. # host at the same path.
libvirt_vm_default_console_log_dir: "/var/log/kolla/tenks" libvirt_vm_default_console_log_dir: "/var/log/kolla/tenks"
# NOTE(priteau): Disable libvirt_vm_trust_guest_rx_filters, which when enabled
# triggers the following errors when booting baremetal instances with Tenks on
# Libvirt 9: Cannot set interface flags on 'macvtap1': Value too large for
# defined data type
libvirt_vm_trust_guest_rx_filters: false

6
dev/tenks-deploy-config-compute.yml
View File

@ -55,3 +55,9 @@ libvirt_vm_default_console_log_dir: "/var/log/kolla/tenks"
# Console logs are owned by the ID of the Nova user in the nova_libvirt # Console logs are owned by the ID of the Nova user in the nova_libvirt
# container. # container.
libvirt_vm_log_owner: 42436 libvirt_vm_log_owner: 42436
# NOTE(priteau): Disable libvirt_vm_trust_guest_rx_filters, which when enabled
# triggers the following errors when booting baremetal instances with Tenks on
# Libvirt 9: Cannot set interface flags on 'macvtap1': Value too large for
# defined data type
libvirt_vm_trust_guest_rx_filters: false

2
playbooks/kayobe-overcloud-base/run.yml
View File

@ -30,5 +30,3 @@
cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-baremetal cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-baremetal
chdir: "{{ kayobe_src_dir }}" chdir: "{{ kayobe_src_dir }}"
executable: /bin/bash executable: /bin/bash
# FIXME(priteau): Bare metal testing fails on RL9/c9s with Libvirt 9.0.0
when: ansible_facts.os_family == 'Debian'

4
playbooks/kayobe-overcloud-upgrade-base/run.yml
View File

@ -80,8 +80,6 @@
cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-bm-pre-upgrade cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-bm-pre-upgrade
chdir: "{{ previous_kayobe_src_dir }}" chdir: "{{ previous_kayobe_src_dir }}"
executable: /bin/bash executable: /bin/bash
# FIXME(priteau): Bare metal testing fails on RL9/c9s with Libvirt 9.0.0
when: ansible_facts.os_family == 'Debian'
# Upgrade Kayobe, and use it to perform an upgrade of the control plane. # Upgrade Kayobe, and use it to perform an upgrade of the control plane.
@ -109,8 +107,6 @@
cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-bm-post-upgrade cmd: dev/overcloud-test-baremetal.sh &> {{ logs_dir }}/ansible/overcloud-test-bm-post-upgrade
chdir: "{{ kayobe_src_dir }}" chdir: "{{ kayobe_src_dir }}"
executable: /bin/bash executable: /bin/bash
# FIXME(priteau): Bare metal testing fails on RL9/c9s with Libvirt 9.0.0
when: ansible_facts.os_family == 'Debian'
environment: environment:
KAYOBE_CONFIG_SOURCE_PATH: "{{ kayobe_config_src_dir }}" KAYOBE_CONFIG_SOURCE_PATH: "{{ kayobe_config_src_dir }}"