CCI-MOC / esi

Elastic Secure Infrastructure project
6 stars 12 forks source link

Update MOC ESI with configuration for brocade switch #338

Closed tzumainn closed 1 year ago

tzumainn commented 1 year ago

This should include files and procedures that this configuration is part of any redeploy

joachimweyl commented 1 year ago

is this related to https://github.com/CCI-MOC/ops-issues/issues/777

naved001 commented 1 year ago

@joachimweyl related, but different.

tzumainn commented 1 year ago

@DanNiESh can you point naved to the post-install ansible scripts so he can file a PR to make any changes persistent? Thanks!

naved001 commented 1 year ago

@tzumainn I have updated /var/lib/config-data/puppet-generated/neutron/etc/neutron/plugins/ml2/ml2_conf.ini on all the 3 controllers to include the information about the brocade switch.

I guess the next step is to register a server that uses a switchport on that switch?

tzumainn commented 1 year ago

@naved001 correct! There's two-ish steps:

tzumainn commented 1 year ago

Ah, the command to create nodes from a json file is just openstack baremetal create <json file>; it might make sense to start a new one for these nodes?

naved001 commented 1 year ago

@tzumainn Sure, I can create another json file to the new set of nodes.

I noticed there's this file https://github.com/CCI-MOC/esi-pilot/blob/main/instackenv.json which also has similar information about the nodes?

tzumainn commented 1 year ago

Ah, instackenv.json is used by the undercloud.

naved001 commented 1 year ago

@tzumainn

I followed https://github.com/CCI-MOC/esi-pilot/blob/main/openstack_commands/node_manage.sh and registered my node, and it is in "enroll" state and for that reason I cannot run inspect on it. Am I missing something?

DanNiESh commented 1 year ago

@naved001 correct! There's two-ish steps:

  • 'openstack baremetal node create'; you can see the required information in esi-pilot/bm_inventory.json. I think that file can also be read by some script to create the nodes all at once, but I'm not entirely sure which; @DanNiESh do you know? Yes. 'openstack baremetal create bm_inventory.json' imports all nodes at once. This command should have been added to node_manage.sh but I forgot it.
  • the script esi-pilot/openstack_commands/node_manage.sh show the next steps - creating the baremetal port and running the requisite ironic commands to perform initial inspection. The big thing here would be to use 'switch2' instead of 'switch1' for the port's switch_info
DanNiESh commented 1 year ago

@DanNiESh can you point naved to the post-install ansible scripts so he can file a PR to make any changes persistent? Thanks!

The related files are: 1) https://github.com/CCI-MOC/esi-pilot/blob/main/ansible-deployment/switch/tasks/main.yaml. This task modifies ansible switch configuration. 2) https://github.com/CCI-MOC/esi-pilot/blob/main/ml2-ansible-hosts.yaml. However I don't think configuring things in this yaml file will make changes to ansible.ini file on overclouds. So actually only 1) takes effect to modify ini file.