From 778001089eba56b2cec5cd87a24cce4f16039773 Mon Sep 17 00:00:00 2001 From: Michael Polenchuk Date: Mon, 23 Jan 2017 15:24:03 +0400 Subject: [PATCH] Upload interfaces config before attributes Enable dpdk on an interface before upload specific settings in order to meet the validator requirements. JIRA: FUEL-247 Change-Id: Id1248b391257b07b26edb5630da47f4dcbafb156 Signed-off-by: Michael Polenchuk --- deploy/cloud/configure_nodes.py | 21 +++++++-------------- .../no-ha_nfv-ovs_heat_ceilometer_scenario.yaml | 3 +-- 2 files changed, 8 insertions(+), 16 deletions(-) diff --git a/deploy/cloud/configure_nodes.py b/deploy/cloud/configure_nodes.py index eabec397c..a50973af6 100644 --- a/deploy/cloud/configure_nodes.py +++ b/deploy/cloud/configure_nodes.py @@ -32,27 +32,20 @@ class ConfigureNodes(object): def config_nodes(self): log('Configure nodes') - # Super dirty fix since Fuel 7 requires user defined roles to be - # assigned before anything else (BUG fixed in Fuel 8)! + # Assign nodes to environment with given roles for node_id, roles_blade in self.node_id_roles_dict.iteritems(): - if "opendaylight" in roles_blade[0] or "onos" in roles_blade[0] or "contrail" in roles_blade[0]: - exec_cmd('fuel node set --node-id %s --role %s --env %s' - % (node_id, roles_blade[0], self.env_id)) + exec_cmd('fuel node set --node-id %s --role %s --env %s' + % (node_id, roles_blade[0], self.env_id)) for node_id, roles_blade in self.node_id_roles_dict.iteritems(): - if "opendaylight" not in roles_blade[0] and "onos" not in roles_blade[0] and "contrail" not in roles_blade[0]: - exec_cmd('fuel node set --node-id %s --role %s --env %s' - % (node_id, roles_blade[0], self.env_id)) - - for node_id, roles_blade in self.node_id_roles_dict.iteritems(): - # Modify node attributes - self.download_attributes(node_id) - self.modify_node_attributes(node_id, roles_blade) - self.upload_attributes(node_id) # Modify interfaces configuration self.download_interface_config(node_id) self.modify_node_interface(node_id, roles_blade) self.upload_interface_config(node_id) + # Modify node attributes + self.download_attributes(node_id) + self.modify_node_attributes(node_id, roles_blade) + self.upload_attributes(node_id) # Currently not used, we use default deployment facts # which are generated by fuel based on type segmentation diff --git a/deploy/scenario/no-ha_nfv-ovs_heat_ceilometer_scenario.yaml b/deploy/scenario/no-ha_nfv-ovs_heat_ceilometer_scenario.yaml index 3436e74f9..ba3c57901 100644 --- a/deploy/scenario/no-ha_nfv-ovs_heat_ceilometer_scenario.yaml +++ b/deploy/scenario/no-ha_nfv-ovs_heat_ceilometer_scenario.yaml @@ -76,8 +76,7 @@ dea-override-config: attributes_1: hugepages: dpdk: - value: 256 - min: 256 + value: 1024 nova: value: '2048': 1024 -- 2.16.6