Skip to content

Commit

Permalink
Merge pull request #652 from umago/encap-tos
Browse files Browse the repository at this point in the history
Allow ovn-encap-tos to be configured
  • Loading branch information
openshift-merge-bot[bot] authored May 27, 2024
2 parents 5b00bea + 7a762ae commit c0f3080
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 0 deletions.
2 changes: 2 additions & 0 deletions roles/edpm_ovn/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ edpm_ovn_chassis_mac_mapping_prefixes:

edpm_ovn_chassis_mac_mapping_seed: "{{ ansible_machine_id }}"
edpm_ovn_encap_type: geneve
edpm_ovn_encap_tos: "0"
edpm_ovn_dbs: []
edpm_enable_chassis_gw: false
edpm_enable_chassis_extport: false
Expand Down Expand Up @@ -84,6 +85,7 @@ edpm_ovn_ovs_external_ids:
{{ chassis_mac_mappings | join(',') }}"
ovn-encap-ip: "{{ edpm_ovn_encap_ip }}"
ovn-encap-type: "{{ edpm_ovn_encap_type }}"
ovn-encap-tos: "{{ edpm_ovn_encap_tos }}"
ovn-match-northd-version: "{{ ovn_match_northd_version }}"
ovn-monitor-all: "{{ ovn_monitor_all }}"
ovn-remote: >-
Expand Down
5 changes: 5 additions & 0 deletions roles/edpm_ovn/meta/argument_specs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,11 @@ argument_specs:
description: >
Passed to the `edpm_ovn_ovs_external_ids` as value for the `ovn-encap-type`.
type: str
edpm_ovn_encap_tos:
default: '0'
description: >
Passed to the `edpm_ovn_ovs_external_ids` as value for the `ovn-encap-tos`.
type: str
edpm_ovn_multi_rhel:
default: false
description: ''
Expand Down
6 changes: 6 additions & 0 deletions roles/edpm_ovn/molecule/default/verify-tasks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@
register: output
failed_when: output.stdout != 'geneve'

- name: Verify ovn-encap-tos
ansible.builtin.shell: >
/usr/bin/ovs-vsctl get open_vswitch . external_ids:ovn-encap-tos
register: output
failed_when: output.stdout != '"0"'

- name: Verify ovn firewall rules are applied
block:
- name: Rule source file exists
Expand Down
6 changes: 6 additions & 0 deletions roles/edpm_ovn/molecule/noconfig/verify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,9 @@
/usr/bin/ovs-vsctl get open_vswitch . external_ids:ovn-encap-type
register: output
failed_when: output.stdout != 'vxlan'

- name: Verify ovn-encap-tos
ansible.builtin.shell: >
/usr/bin/ovs-vsctl get open_vswitch . external_ids:ovn-encap-tos
register: output
failed_when: output.stdout != '"0"'

0 comments on commit c0f3080

Please sign in to comment.