att-comdev / openstack-helm

PROJECT HAS MOVED TO OPENSTACK
https://github.com/openstack/openstack-helm
69 stars 41 forks source link

Initial commit of Neutron Configuration Overrides #340

Closed alanmeadows closed 7 years ago

alanmeadows commented 7 years ago

What is the purpose of this pull request?:

Initial Commit of Neutron Configuration Overrides

This commit is an evolution of the initial keystone, and nova configuration overrides work:

https://github.com/att-comdev/openstack-helm/pull/319 https://github.com/att-comdev/openstack-helm/pull/262

This also resolved a bug with nova bootstrap.

Unlike the other two pull requests, for the sake of expediency, this PR lacks the usual mount append effort involved in configuration override work for each chart. That can be done as a separate pull request against neutron.

What issue does this pull request address?: Fixes #

Notes for reviewers to consider:

This initial pull request adds in several configuration files for services not yet in use such as the metering agent, sriov agent, and so on. We can build upon these new configuration files by adding conditional containers to run these services in the future.

Also, it is well known even with configuration management the chart is very much geared toward openvswitch right now. This PR is not an attempt to solve that but eventually loosing that dependency can leverage this effort.

Lastly, the mount append effort missing from this PR. Can be done separately.

Specific reviewers for pull request:

@v1k0d3n @intlabs @larryrensing @wilkers-steve @wilreichert

intlabs commented 7 years ago

@alanmeadows Could you please rebase against master?

alanmeadows commented 7 years ago
po/neutron-dhcp-agent-df5sc             1/1       Running   0          1m
po/neutron-l3-agent-2nhlm               1/1       Running   0          1m
po/neutron-metadata-agent-xmzp5         1/1       Running   0          1m
po/neutron-server-3962975493-npcpm      1/1       Running   0          1m

and

alan@hpdesktop:~/Workbench/att/attcomdev/openstack-helm$ neutron agent-list
+--------------------------------------+----------------+-----------+-------------------+-------+----------------+------------------------+
| id                                   | agent_type     | host      | availability_zone | alive | admin_state_up | binary                 |
+--------------------------------------+----------------+-----------+-------------------+-------+----------------+------------------------+
| 24bb6d14-0120-4fe4-932e-8ce2526811c6 | L3 agent       | hpdesktop | nova              | :-)   | True           | neutron-l3-agent       |
| cde796dc-46d0-44ba-bb19-b0ec76eff452 | DHCP agent     | hpdesktop | nova              | :-)   | True           | neutron-dhcp-agent     |
| e744988d-c8f2-46c5-a206-9440e8b00b92 | Metadata agent | hpdesktop |                   | :-)   | True           | neutron-metadata-agent |
+--------------------------------------+----------------+-----------+-------------------+-------+----------------+------------------------+
alan@hpdesktop:~/Workbench/att/attcomdev/openstack-helm$ 

It works as far as the control plane is concerned. I would normally like to do a VM add test, but given the time constraints: merge little hamsters, merge!