This updates all references of let(:chef_run) to cached(:chef_run) to
speed up tests. By doing this, we have to create a new cached(:chef_run)
block whenever we need to adjust node attributes for testing.
Additional fixes:
- Pull in openstack-network::ml2_core_plugin in server spec so that we get the
default attributes set
- Remove unused shared_examples
Speed was improved from 4 minutes 18.7 seconds to 1 minute 2.73 seconds
Change-Id: Ib10a6828e6886a57527a5e5a506cc57364ec0c2a
- Switched default linter to cookstyle
- Renamed rake tasks to better conform with Chef conventions
Change-Id: I5dd1971392fdf282d0f214dfce9dcbecc18542a8
- Now use cookbook-openstackclient to create endpoints role service and
user
- added domain creation and access granting
- added values to work with identity_v3
- rewrote specs to work again
- updated readme
- updated neutron-package dependencies for debian
Change-Id: I2d404a424bd79a6e9b282304e21591fa33a48981
Depends-On: I0f8955f05de9b33711c54b9a198f45018cceb8e1
* removed the creation of ovs bridges (except br-int) from all recipes,
since this can not be done in a sufficient generic way or only with a lot of
case switches to cope with all possible network setups
* added an example recipe to create all default ovs bridges from the
networking guide for legacy ovs setups (we should also create one for dvr later)
* splittet recipe ml2_openvswitch into seperate recipes for ml2_openvswitch config,
openvswitch_agent and openvswitch to allow bridge creation from wrapper recipe inbetween
and seperate configs from package installation
Change-Id: I6383575862ba110b3f3b5cba227288dc026fce77
* added new logic to render plugin templates
* refactored recipe names to be more consistence
* moved version up to 13.0.0 for mitaka release
* removed suse as supported platform
* added verisionbumb for refactored os-identity and common
* adapted optimized endpoint logic
* added endpoint attributes to fit new endpoint logic
* adapted the specs (unit tests) to work again
* refactored attributes throughout all recipes that were connected to the
attributes used for the neutron.conf.erb template to adapt the new template
attribute syntax
* removed some attributes that were set to non default values, since the
defaults from neutron cloud and should be used instead
* moved all attributes form attributes/default.rb that were used in
neutron.conf.erb to attributes/neutron_conf.rb
* refactored attributes to fit new template logic
* refactored recipes/default.rb to fit new template logic
* removed all attributes set to default values in attribtues/default and
template
* replaced static plugin logic and templates with new config logic, following
the same principles as for neutron.conf
* renamed recipes to fit attributes and actual service names
* added recipes for ml2_core_plugin, ml2_openvswitch and ml2_linuxbridge as well
as a recipe for the creation of all plugin configs (plugin_conf) like plugin.ini
Change-Id: I9cc1b5cc069987ac83e064322c2291772505ff5f
Implements: blueprint cookbook-refactoring
Depends-On: I0547182085eed91d05384fdd7734408a839a9a2c
Depends-On: I3262b2e6f792f37c32a446e6567790b82bdd4613
Cleaned up all the minor rubocop issues, the ones left relate to
complex logic and what I think is a bug in rubocop for nested
vs compact modules/class definitions.
Change-Id: I6451b7593f36287d4efe4d7542f97d0a090fb1d1
Remove deprecated keys and use identity_uri via the new transform
helper method.
Also, cleanup specs for endpoint testing to make sure Common is
fully tested.
Change-Id: Iad3da2be74823b8d706eb9b4e12311cbf655ab7f
Implements: blueprint identity-uri
Used the existing ovs conf as a partial template to simply include
in the ml2 conf. This keeps all the attributes the same.
A more detailed spec will have to be done after some refactoring
to allow for sections to be tested.
Please give this idea a try, it seems to allow me to setup neutron
on ubuntu with ml2. I'm working toward using this as a basis for
getting the repo aio_neutron test working.
Change-Id: Ia988f3d7ef1c280c40b55cea4f455710f673dd32
Closes-Bug: #1314751
Nova metadata api endpoint was missing from common endpoints.
This patch will be follow by one for compute and network to
make use of these new endpoint.
Change-Id: Ie3e9f27f7d6eec4f4ebb0b1dbebd0b12a03a0b65
Partial-Bug: #1425633
Now that admin_endpoint, public_endpoint, and internal_endpoint
in the common library are working, these are the changes to use
them in the openstack-network recipes.
Note that some calls to 'endpoint' that can't have separate
admin, internal, public endpoints. (E.g. network-api-bind,
network-linuxbridge, network-openvswitch, etc.) Those
will continue to use the general endpoint routine.
Partial-Bug: 1412919
Change-Id: Ia08983f6f8da8a96da22969cdb660b44f82780d6
This patch updates the version of dnsmasq so dhcp-agent
can start correctly. This was found attempting to start
the testing framework with centos and neutron. This is
not triggered if the machine is anything other than
centos and 6.5 and above.
Change-Id: I622e51f8f9f912ca7bc4185614b3098208d29d11
Closes-Bug: 1397610
If keystone are configured with SSL, neutron need to connect keystone
with certs(or insecure) to load the admin_tenant_id. This commit will
add these arguments for the call.
Related openstack-common cookbook fix:
https://review.openstack.org/#/c/132282/
Closes-Bug: #1388138
Change-Id: I2407055a71340aa6ac8f2d586a84cfb681d093ed
Also added tests for the default attribute values used
in the plugin templates
Change-Id: I91146c2254381572734bffb831896f522527e446
Closes-Bug: #1380243
This commit adds nova_ca_certificates_file and
nova_api_insecure so that they can be configured.
It also makes the nova_url use the right scheme.
Change-Id: I00a9f900e2d722529808e10e946beae8f374c4fa
Closes-Bug: #1383701
This change makes the l3_agent recipe lookup the ID of the network used
for the gateway external network by name. For example, specify
node[openstack][network][l3][gateway_external_network_name] = public
and the recipe will lookup the network by name. It will use
openstack-common/libraries/cli.rb network_uuid method, effectively
querying `neutron net-external-list`. The result is saved to
gateway_external_network_id and used on subsequent runs.
To have the recipe check the UUID again (e.g. to update it), reset the
_id attribute to nil.
Change-Id: I5df6f9989df7b18121a8050835a192da9e1a762e
Enable attributes for neutron interaction with nova.
This uses the new Common CLI method in change:
I2b2f1d94dbd67550ad3a352c2205a532173f6065
The default is to make a call into keystone CLI to query the service tenant
UUID and use that in the neuton.conf file. If the UUID is overridden, no
CLI call is made. This does not pre-req any Compute recipes, just new Common
method.
Also see this bug: 1298640 for how the nova_url is handled, not obvious.
Added TODO's to look at handling this type of url situation better from Common.
Implements: blueprint neutron-icehouse-default-changes
Change-Id: I89dd47c802a544bdd998059b36884cb10f628223
The new get_secret method allows the 'secrets' databag to be renamed,
just like all other databags.
Change-Id: Ic8be5d19a112d78e22716c0cd571efc7186ac9ce
Related-Bug: #1288784
This change gives us new capabilities, such as matching notifies and
subscribes of resources.
Implements: blueprint chefspec-3-4
Change-Id: Ic2ffd83c398c93ec2f410b90b7305be7885b1471
Currently, qpid's password is stored in an attribute in the common
cookbook. This change causes the password to be looked up via
get_password method, which is how we obtain the rabbitmq password.
Change-Id: Id09820dafc67910f9f4d6c350ece6fe93fb21787
Implements: blueprint use-data-bag-for-qpid-password
On some plateform or verion of ovs the ouput of ovs-vsctl show have quote or not.
This patch implement a other method to check if bridge exist.
Co-Authored-By: Stephan Renatus <s.renatus@x-ion.de>
Change-Id: Iaae683213f281529e2167ac326d8e9573f026f12
Closes-Bug: #1266558
Added double quotes around the interface name in the grep pattern
because the output of ovs-vsctl quotes the interface name.
Closes-Bug: 1266558
Change-Id: I1a3dcbafe14db133ed02c4d9b7aacdb485fc2de1
Now that almost all of the rubocop blueprints have been completed, make
one final pass through all of the cookbooks ensuring they're all in
sync with each other.
- Upgrade rubocop to 0.18.1
- Fix violations caused by 0.18.1 upgrade
- Add Includes for non-existent folders to Includes so they
automatically cover future additions
Change-Id: Ib3220a0ab8281bc0eafaf9898c233d90490cfde6
Implements: blueprint lint-and-unit-testing-for-havana
- Update .rubocop.yml to include spec/**
- Update spec files to be rubocop compliant
- Inlcude 'chefspec/berkshelf' in spec_helper.rb for ease of use
Change-Id: Ief01cc34cc4f9f04e6ba6437f67bfd347975c3a8
Addresses: blueprint rubocop-for-network
- Adjust the stubs in spec/spec_helper.rb to specify the appropriate
number of expected variables.
- Add parenthensis around return values
Partial-Bug: #1271535
Change-Id: I1aa473e406059fec66b0b2c9a307e8eeb41b0996
The user_password, service_password and db_password functions are redundant
since they simply call "secret". Creates a get_password function that
will accept a "type" of db, service or user.
All instances of these calls have been changed to call get_password.
Change-Id: I480c654de44dc83e78f9ec4085f1b83edadd91d6
Partial-Bug: #1195915
all cops now pass except for the following
(which are silenced in .rubocop.yml )
* LineLength, MethodLength: not sure if we care about these
* WordArray: I prefer [] vs %w{} for readability.
Conflicts:
metadata.rb
recipes/common.rb
recipes/openvswitch.rb
spec/openvswitch_spec.rb
Change-Id: I92996b5db9f292e9ba93c7619ca727ffa92ddd71
Since in OpenStack Havana release the Quantum project was renamed to the
Neutron, for the cookbooks we also need do the same things to support
the Neutron. I do the %s/Quantum/Neutron/g in all the files under
cookbook-openstack-netwrok. It will effect other cookbooks:
openstack-compute, openstack-ops-database which should be covered in
others patches.
Implemented: blueprint neutron-support
Change-Id: I59f3f2d8dfad678f70b97d4cbc4e10dfa163e9b2
Ghanshyam Mann