From e2c175f3114fb1857ab8ab4bb7e8ff2f8daf18bd Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Tue, 11 Dec 2018 15:56:21 +0100 Subject: [PATCH] Make MTU cluster-level configurable via IDF JIRA: FUEL-336 Change-Id: I1c8d22b8322f700eb727d9077035ba4c9f9f9753 Signed-off-by: Alexandru Avadanii --- docs/release/installation/installation.instruction.rst | 1 + .../0002-maas-region-allow-timeout-override.patch | 18 ++++-------------- .../cluster/all-mcp-arch-common/infra/maas.yml.j2 | 7 +++---- .../classes/cluster/all-mcp-arch-common/init.yml.j2 | 2 ++ .../classes/cluster/mcp-common-ha/infra/kvm_pdf.yml.j2 | 2 -- .../cluster/mcp-common-ha/openstack_compute.yml | 1 - .../cluster/mcp-common-ha/openstack_compute_pdf.yml.j2 | 1 + .../cluster/mcp-common-ha/openstack_control.yml.j2 | 3 +++ .../openstack_interface_vcp_biport.yml.j2 | 2 ++ .../openstack_interface_vcp_triport.yml.j2 | 3 +++ .../cluster/mcp-common-noha/openstack_compute.yml | 1 - .../cluster/mcp-common-noha/openstack_control.yml.j2 | 5 ++--- .../mcp-common-noha/openstack_control_pdf.yml.j2 | 1 + .../cluster/mcp-common-noha/openstack_gateway.yml | 1 - .../cluster/mcp-odl-noha/opendaylight/control.yml.j2 | 1 - mcp/scripts/pharos | 2 +- 16 files changed, 23 insertions(+), 28 deletions(-) diff --git a/docs/release/installation/installation.instruction.rst b/docs/release/installation/installation.instruction.rst index 8726d9bd0..183a6f668 100644 --- a/docs/release/installation/installation.instruction.rst +++ b/docs/release/installation/installation.instruction.rst @@ -609,6 +609,7 @@ Fuel installer: timeout_comissioning: 10 # commissioning timeout in minutes timeout_deploying: 15 # deploy timeout in minutes network: # Cluster nodes network (mandatory) + interface_mtu: 1500 # Cluster-level MTU (optional) ntp_strata_host1: 1.pool.ntp.org # NTP1 (optional) ntp_strata_host2: 0.pool.ntp.org # NTP2 (optional) node: ... # List of per-node cfg (mandatory) diff --git a/mcp/patches/salt-formula-maas/0002-maas-region-allow-timeout-override.patch b/mcp/patches/salt-formula-maas/0002-maas-region-allow-timeout-override.patch index fe5f4e671..58bed30f7 100644 --- a/mcp/patches/salt-formula-maas/0002-maas-region-allow-timeout-override.patch +++ b/mcp/patches/salt-formula-maas/0002-maas-region-allow-timeout-override.patch @@ -39,14 +39,14 @@ JIRA: FUEL-316 Signed-off-by: Alexandru Avadanii --- - maas/region.sls | 40 ++++++++++++++++++++++++++++++++++++++++ - 1 file changed, 40 insertions(+) + maas/region.sls | 30 ++++++++++++++++++++++++++++++ + 1 file changed, 30 insertions(+) diff --git a/maas/region.sls b/maas/region.sls -index e0f65b0..d151e3a 100644 +index e0f65b0..5da3a7f 100644 --- a/maas/region.sls +++ b/maas/region.sls -@@ -38,6 +38,46 @@ restore_maas_database_{{ region.database.name }}: +@@ -38,6 +38,36 @@ restore_maas_database_{{ region.database.name }}: {%- endif %} @@ -70,16 +70,6 @@ index e0f65b0..d151e3a 100644 + - require_in: + - service: maas_region_services + -+maas_ip_blksize_force: -+ file.replace: -+ - name: "/usr/lib/python3/dist-packages/tftp/bootstrap.py" -+ - pattern: 'int_blksize = min\(\(int_blksize, MAX_BLOCK_SIZE\)\)' -+ - repl: 'int_blksize = 1464' -+ - require: -+ - pkg: maas_region_packages -+ - require_in: -+ - service: maas_region_services -+ +maas_interface_default_mode_dhcp: + file.replace: + - name: "/usr/lib/python3/dist-packages/maasserver/models/node.py" diff --git a/mcp/reclass/classes/cluster/all-mcp-arch-common/infra/maas.yml.j2 b/mcp/reclass/classes/cluster/all-mcp-arch-common/infra/maas.yml.j2 index fb45b13fe..40a14c4a6 100644 --- a/mcp/reclass/classes/cluster/all-mcp-arch-common/infra/maas.yml.j2 +++ b/mcp/reclass/classes/cluster/all-mcp-arch-common/infra/maas.yml.j2 @@ -19,9 +19,6 @@ parameters: mcpcontrol_interface: ${_param:opnfv_fn_vm_primary_interface} primary_interface: ${_param:opnfv_fn_vm_secondary_interface} pxe_admin_interface: ${_param:opnfv_fn_vm_tertiary_interface} - interface_mtu: 1500 - # MaaS has issues using MTU > 1500 for PXE interface - pxe_admin_interface_mtu: 1500 linux_system_codename: xenial maas_admin_username: opnfv dns_server01: '{{ nm.dns_public[0] }}' @@ -113,6 +110,7 @@ parameters: name: ${_param:mcpcontrol_interface} type: eth proto: dhcp + mtu: ${_param:interface_mtu} primary_interface: enabled: true name: ${_param:primary_interface} @@ -134,7 +132,8 @@ parameters: pxe_admin_interface: enabled: true name: ${_param:pxe_admin_interface} - mtu: ${_param:pxe_admin_interface_mtu} + # MaaS has issues using MTU > 1500 for PXE interface + mtu: 1500 proto: static address: ${_param:single_address} netmask: ${_param:opnfv_net_admin_mask} diff --git a/mcp/reclass/classes/cluster/all-mcp-arch-common/init.yml.j2 b/mcp/reclass/classes/cluster/all-mcp-arch-common/init.yml.j2 index 84c82b738..dfa1d89cc 100644 --- a/mcp/reclass/classes/cluster/all-mcp-arch-common/init.yml.j2 +++ b/mcp/reclass/classes/cluster/all-mcp-arch-common/init.yml.j2 @@ -45,6 +45,8 @@ parameters: opnfv_vcp_vm_tertiary_interface: ens4 {%- endif %} + interface_mtu: {{ conf.idf.fuel.network.interface_mtu or 1500 }} + ntp_strata_host1: {{ conf.idf.fuel.network.ntp_strata_host1 or '1.pool.ntp.org' }} ntp_strata_host2: {{ conf.idf.fuel.network.ntp_strata_host2 or '0.pool.ntp.org' }} diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/infra/kvm_pdf.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-ha/infra/kvm_pdf.yml.j2 index 6754d13dd..484e53299 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/infra/kvm_pdf.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-ha/infra/kvm_pdf.yml.j2 @@ -13,8 +13,6 @@ {%- set vlans = { nm.vlan_admin: nm.ctl01.nic_admin, nm.vlan_mgmt: nm.ctl01.nic_mgmt, nm.vlan_public: nm.ctl01.nic_public } %} --- parameters: - _param: - interface_mtu: 1500 linux: network: interface: diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute.yml b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute.yml index c7338bb67..93acf13e8 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute.yml +++ b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute.yml @@ -32,7 +32,6 @@ parameters: cluster_node03_hostname: ${_param:openstack_control_node03_hostname} cluster_node03_address: ${_param:openstack_control_node03_address} nova_vncproxy_url: https://${_param:cluster_public_host}:6080 - interface_mtu: 1500 keepalived_vip_interface: br-ctl keepalived_vip_virtual_router_id: 69 linux_system_codename: xenial diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute_pdf.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute_pdf.yml.j2 index 3e33811a9..f1b369c66 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute_pdf.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_compute_pdf.yml.j2 @@ -75,3 +75,4 @@ parameters: type: ovs_port ovs_bridge: br-floating bridge: br-floating + mtu: ${_param:interface_mtu} diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_control.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_control.yml.j2 index edb20e3b6..41625e8ff 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_control.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_control.yml.j2 @@ -87,6 +87,9 @@ parameters: <<: *db_conn_recycle_time vlan_aware_vms: true root_helper_daemon: false + global_physnet_mtu: ${_param:interface_mtu} + backend: + external_mtu: ${_param:interface_mtu} keystone: server: <<: *db_conn_recycle_time diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_biport.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_biport.yml.j2 index cb4c233cd..3b302aca8 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_biport.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_biport.yml.j2 @@ -29,6 +29,7 @@ parameters: name_servers: - {{ nm.net_admin_gw }} noifupdown: true + mtu: ${_param:interface_mtu} single: enabled: true type: eth @@ -36,4 +37,5 @@ parameters: name: ${_param:single_nic} address: ${_param:single_address} netmask: ${_param:opnfv_net_public_mask} + mtu: ${_param:interface_mtu} {%- endif %} diff --git a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_triport.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_triport.yml.j2 index 38a436138..8815de99b 100644 --- a/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_triport.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-ha/openstack_interface_vcp_triport.yml.j2 @@ -28,6 +28,7 @@ parameters: address: ${_param:pxe_admin_address} netmask: ${_param:opnfv_net_admin_mask} noifupdown: true + mtu: ${_param:interface_mtu} single_int: enabled: true type: eth @@ -37,6 +38,7 @@ parameters: netmask: ${_param:opnfv_net_public_mask} gateway: ${_param:opnfv_net_public_gw} name_servers: {{ nm.dns_public }} + mtu: ${_param:interface_mtu} control_int: enabled: true type: eth @@ -44,6 +46,7 @@ parameters: name: ${_param:control_nic} address: ${_param:control_address} netmask: ${_param:opnfv_net_mgmt_mask} + mtu: ${_param:interface_mtu} {%- else %} {#- For NOVCP scenarios, base config is in kvm_pdf, only add/override gw #} parameters: diff --git a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_compute.yml b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_compute.yml index 0409974ba..f3c668f08 100644 --- a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_compute.yml +++ b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_compute.yml @@ -20,7 +20,6 @@ classes: - cluster.mcp-common-noha.openstack_compute_pdf parameters: _param: - interface_mtu: 9000 linux_system_codename: xenial nova: compute: diff --git a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control.yml.j2 index e9bb1357f..02a9168d0 100644 --- a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control.yml.j2 @@ -72,7 +72,6 @@ classes: - cluster.mcp-common-noha.openstack_control_pdf parameters: _param: - interface_mtu: 9000 linux_system_codename: xenial ceilometer_create_gnocchi_resources: 'True' barbican_integration_enabled: 'false' @@ -185,9 +184,9 @@ parameters: server: vlan_aware_vms: true root_helper_daemon: false - global_physnet_mtu: 9000 + global_physnet_mtu: ${_param:interface_mtu} backend: - external_mtu: 9000 + external_mtu: ${_param:interface_mtu} {%- if '-bgpvpn-' in conf.MCP_DEPLOY_SCENARIO %} bgp_vpn: enabled: True diff --git a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control_pdf.yml.j2 b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control_pdf.yml.j2 index 7a1c3532d..06df3b845 100644 --- a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control_pdf.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_control_pdf.yml.j2 @@ -22,6 +22,7 @@ parameters: type: eth address: ${_param:pxe_admin_address} netmask: ${_param:opnfv_net_admin_mask} + mtu: ${_param:interface_mtu} noifupdown: true {#- prevent duplicates for tagged mgmt on the same physical interface as PXE/admin #} diff --git a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_gateway.yml b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_gateway.yml index 0062adf16..f29cb867c 100644 --- a/mcp/reclass/classes/cluster/mcp-common-noha/openstack_gateway.yml +++ b/mcp/reclass/classes/cluster/mcp-common-noha/openstack_gateway.yml @@ -11,7 +11,6 @@ classes: - cluster.mcp-common-noha.openstack_gateway_pdf parameters: _param: - interface_mtu: 9000 linux_system_codename: xenial neutron: gateway: diff --git a/mcp/reclass/classes/cluster/mcp-odl-noha/opendaylight/control.yml.j2 b/mcp/reclass/classes/cluster/mcp-odl-noha/opendaylight/control.yml.j2 index 92abf9d8f..135b82d23 100644 --- a/mcp/reclass/classes/cluster/mcp-odl-noha/opendaylight/control.yml.j2 +++ b/mcp/reclass/classes/cluster/mcp-odl-noha/opendaylight/control.yml.j2 @@ -15,7 +15,6 @@ classes: - cluster.mcp-odl-noha.opendaylight.control_pdf parameters: _param: - interface_mtu: 1500 linux_system_codename: xenial opendaylight: server: diff --git a/mcp/scripts/pharos b/mcp/scripts/pharos index 7b870a0cf..8f43c9af3 160000 --- a/mcp/scripts/pharos +++ b/mcp/scripts/pharos @@ -1 +1 @@ -Subproject commit 7b870a0cf6306b3de451860829508c0633493284 +Subproject commit 8f43c9af3ca49498659fb3caa08162504b50ab07 -- 2.16.6