Skip to content
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

[dnm] Add static routes to netConfig #731

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions tests/roles/dataplane_adoption/defaults/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,9 @@
start: 172.17.0.100
cidr: 172.17.0.0/24
vlan: 20
routes:
- destination: 172.17.1.0/24

Check failure on line 28 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

yaml[indentation]

Wrong indentation: expected 10 but found 8
nexthop: 172.17.0.100
- name: External
dnsDomain: external.example.com
subnets:
Expand Down Expand Up @@ -60,6 +63,9 @@
start: 172.19.0.100
cidr: 172.19.0.0/24
vlan: 22
routes:
- destination: 172.19.1.0/24

Check failure on line 67 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

yaml[indentation]

Wrong indentation: expected 10 but found 8
nexthop: 172.19.0.100
registry_name: "quay.io"
registry_namespace: "podified-antelope-centos9"
image_tag: "current-podified"
Expand Down Expand Up @@ -92,7 +98,7 @@
# on the OpenstackDataplaneNodeSet inventory. For adding a variable to the invetory,
# please add it to OpenstackDataplaneNodeSet.nodeTemplate.ansibleVars

edpm_bootstrap_command: |

Check warning on line 101 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

jinja[spacing]

Jinja2 spacing could be improved: # This is a hack to deploy RDO Delorean repos to RHEL as if it were Centos 9 Stream
# This is a hack to deploy RDO Delorean repos to RHEL as if it were Centos 9 Stream
set -euxo pipefail
curl -sL https://github.com/openstack-k8s-operators/repo-setup/archive/refs/heads/main.tar.gz | tar -xz
Expand Down Expand Up @@ -148,7 +154,7 @@
{% endraw %}
neutron_physical_bridge_name: br-ctlplane
neutron_public_interface_name: "{{ dataplane_public_iface | default('eth0') }}"
edpm_sshd_allowed_ranges: "{{ ['192.168.122.0/24'] if dataplane_os_net_config_set_route|default(true)|bool else ['0.0.0.0/0'] }}"

Check warning on line 157 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

jinja[spacing]

Jinja2 spacing could be improved: {{ ['192.168.122.0/24'] if dataplane_os_net_config_set_route|default(true)|bool else ['0.0.0.0/0'] }} -> {{ ['192.168.122.0/24'] if dataplane_os_net_config_set_route | default(true) | bool else ['0.0.0.0/0'] }}
edpm_neutron_sriov_agent_enabled: true
edpm_neutron_dhcp_agent_enabled: true
nova_libvirt_backend: local
Expand All @@ -159,7 +165,7 @@
prelaunch_test_instance: true
telemetry_adoption: true

dataplane_cr: |

Check warning on line 168 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

jinja[spacing]

Jinja2 spacing could be improved: apiVersion: dataplane.openstack.org/v1beta1
apiVersion: dataplane.openstack.org/v1beta1
kind: OpenStackDataPlaneNodeSet
metadata:
Expand Down Expand Up @@ -287,7 +293,7 @@
org_namespace: "openstack"
# adoption repo default location

networker_cr: |

Check warning on line 296 in tests/roles/dataplane_adoption/defaults/main.yaml

View workflow job for this annotation

GitHub Actions / pre-commit

jinja[spacing]

Jinja2 spacing could be improved: apiVersion: dataplane.openstack.org/v1beta1
apiVersion: dataplane.openstack.org/v1beta1
kind: OpenStackDataPlaneNodeSet
metadata:
Expand Down
Loading