Commit Graph

61 Commits

Author SHA1 Message Date
James Denton 908f96d5f4 Remove ironic_server from inventory
Nothing references this group, and it is empty in a deployment
running the ironic service.

Change-Id: Icede4166fa64b92572602103c0e9b8a455597bae
2022-08-04 12:13:57 -05:00
Marc Gariepy c22db848a9 Cleanup ansible_python_interpreter
ansible_pyhton_interpreter is set to auto, it's not needed anywhere now.

Change-Id: I09da7425e6794851b2aa88b8053530b8ce3bbb26
2020-06-17 11:14:07 -04:00
Dmitriy Rabotyagov 75725919ab Cleanup after repo_build and pip_install retirement
Change-Id: I5af4ea54dcd2d416570f17bad676b79a08b78903
2020-05-12 22:06:44 +03:00
Dmitriy Rabotyagov 39dcdeedc8 Start using uWSGI role
Move service to use uWSGI role instead of iternal task for uwsgi
deployment. This aims to ease the maintenance of uWSGI and speedup
metal deployments as the same uwsgi environment will be used
across all services.

Change-Id: Ie79a7ba7d62504e9e81edbb386f8e52ce0a03074
2019-09-04 20:13:20 +03:00
Zuul 035ffaf2dc Merge "Replace git.openstack.org URLs with opendev.org URLs" 2019-08-15 02:19:41 +00:00
Dmitriy Rabotyagov 64fdc9d685 Use systemd-journald instead of log files
This patch aims to migrate service from usage of regular syslog files
to journald. We also disable uwsgi logging, since it dublicates
requests that are logged by service itself.

Change-Id: Iaddb2c158a52d90025899d9bfa5576358bef92dd
2019-07-17 00:57:58 +03:00
melissaml 3f7ed9e51e Replace git.openstack.org URLs with opendev.org URLs
Change-Id: Ide9495e549aea76446d5ef03fa75955ae081385b
2019-07-10 22:58:32 +08:00
Logan V 073199dbdc Remove Neutron LBaaS from test inventory
Change-Id: I2f992cb2396ac6198a5a5ab2348d7af96da6d77e
2019-05-16 15:49:38 -05:00
Jimmy McCrory 26fdbcd102 Set container_name as host_var in test inventory
The connection plugin no longer falls back on using the
inventory_hostname as the container_name. Set container_name as a host
var for each container in the test inventory.

Change-Id: I405b64b8067e43309257d50a5c2acc915c789d24
2018-10-20 15:28:29 -07:00
Zuul 5208f1bbbd Merge "use include_tasks instead of include" 2018-08-13 08:33:40 +00:00
caoyuan f640002133
use include_tasks instead of include
include is marked as deprecated since ansible 2.4[0]

Switch to include_tasks or import_playbook as necessary

[0] https://docs.ansible.com/ansible/2.4/include_module.html#deprecated

Change-Id: Ifdc05d6382c6e2733c630cba685a4d7ec8e63a68
2018-08-12 03:03:08 -05:00
wangqi a01a75d0a1 Trivial: Fix the pep8 warning
The yaml should start with "---"

Change-Id: I2695c42c1ad5168826423754af260642ace6171b
2018-08-06 07:34:37 +00:00
Jesse Pretorius 52444d4d97 Remove localhost from neutron agents group
The agent is not needed on the host.

Depends-On: https://review.openstack.org/584033
Change-Id: Ie6c7d0a4c767b1e2826ba8f2fa1b7d57bde222ed
2018-07-19 18:41:58 +01:00
Jesse Pretorius 996fcb7dc7 Move MQ vhost/user creation into role
There is no record for why we implement the MQ vhost/user creation
outside of the role in the playbook, when we could do it inside the
role.

Implementing it inside the role allows us to reduce the quantity of
group_vars duplicated from the role, and allows us to better document
the required variables in the role. The delegation can still be done
as it is done in the playbook too.

In this patch we implement two new variables:
- ironic_oslomsg_rpc_setup_host
- ironic_oslomsg_notify_setup_host

These are used in the role to allow delegation of the MQ vhost/user
setup for each type to any host, but they default to using the first
member of the applicable oslomsg host group.

We also adjust some of the defaults to automatically inherit existing
vars set in group_vars form the integrated build so that we do not
need to do the wiring in the integrated build's group vars. We still
default them in the role too for independent role usage.

Depends-On: https://review.openstack.org/583430
Change-Id: I331be6ce2ca304f31844d8b0cd48e2640d8312d4
2018-07-18 03:19:10 +01:00
Jesse Pretorius cc4fa19f87 Execute service setup against a delegated host using Ansible built-in modules
In order to reduce the packages required to pip install on to the hosts,
we allow the service setup to be delegated to a specific host, defaulting
to the deploy host. We also switch as many tasks as possible to using the
built-in Ansible modules which make use of the shade library.

The 'virtualenv' package is now installed appropriately by the openstack_hosts
role, so there's no need to install it any more. The 'httplib2' package is a
legacy Ansible requirement for the get_url/get_uri module which is no longer
needed. The keystone client library is not required any more now that we're
using the upstream modules. As there are no required packages left, the task
to install them is also removed.

With the dependent patches, the openstack_openrc role is now executed once
on the designated host, so it is no longer required as a meta-dependency for
the role.

Depends-On: https://review.openstack.org/579233
Depends-On: https://review.openstack.org/579959
Depends-On: https://review.openstack.org/580156
Depends-On: https://review.openstack.org/580601
Change-Id: Ic346e642f52f064b002f5a2facead2fca5a7a03a
2018-07-06 13:47:22 +00:00
Andrew Smith f148283fdd Update to use oslo.messaging service for RPC and Notify
This introduces oslo.messaging variables that define the RPC and
Notify transports for the OpenStack services. These parameters
replace the rabbitmq values and are used to generate the messaging
transport_url for the service. The association of the messaging
backend server to the oslo.messaging services will then be
transparent to the ironic service.

This patch:
* Add oslo.messaging variables for RPC and Notify to defaults
* Update transport_url generation (added notifications)
* Add oslo.messaging to inventory
* Add release note

Depends-On: If4326a6848d2d32af284fdbb94798eb0b03734d5
Depends-On: I2b09145b60116c029fc85477399c24f94974b61d
Change-Id: I363a888980dfd84b4b586df01fb022dd453dc3de
2018-05-18 18:33:58 -04:00
Jesse Pretorius f1c09f62df Remove tests-repo-clone.sh
Now that run_tests.sh handles the tests repo clone, we can
remove the use of the older tests-repo-clone.sh script.

Change-Id: I5e5022cbcaff6de7abfc7d2a56612b59dddc97d7
2018-03-28 10:14:26 +01:00
Zuul 73ad6d2557 Merge "ansible-role-requirements: Use systemd_service role from openstack namespace" 2018-03-25 17:37:00 +00:00
Jesse Pretorius 9c18f24116 ansible-role-requirements: Use systemd_service role from openstack namespace
In order to ensure that the updated role is used for tests,
we adjust the a-r-r file to pull from the correct repo.

Change-Id: Ib86b8865d78821739644c2acb05904397e330cfd
2018-03-25 16:20:12 +01:00
Jesse Pretorius 80e46c350e ansible-role-requirements: Add common python_venv_build role
In order to allow tests to work as the use of the common
role is implemented in the various roles, we pre-implement
the addition of the role into the a-r-r file.

Change-Id: I64a86845896818d6222419960de8b075b9309b2e
Implements: blueprint python-build-install-simplification
Signed-off-by: Jesse Pretorius <jesse.pretorius@rackspace.co.uk>
2018-03-24 23:20:18 +00:00
Jesse Pretorius 09c6da75c8 ansible-role-requirements: Add common systemd_service role
In order to allow tests to work with https://review.openstack.org/551028
merged, add the role to the a-r-r file.

Change-Id: Idb1f184964d78f8a19282dbe52ad039d73c9b200
2018-03-13 16:50:40 +00:00
OpenStack Proposal Bot 651cdea9d0 Updated from OpenStack Ansible Tests
Change-Id: Ieae445a2c49b2b6b69b113b168ffda1bd9359b41
2018-02-19 09:12:55 +00:00
Zuul 6b4fb1a2a5 Merge "ansible-role-requirements: Switch ODL git repo to the github mirror" 2017-11-28 14:25:56 +00:00
Jesse Pretorius 3e7377b4df Resolve ansible-lint test failures
With the merge of https://review.openstack.org/520177 in the
tests repo some ansible-lint failures which previously were
not being picked up are now detected.

This adds the appropriate skip tags to the tasks so that they
are not evaluated by ansible-lint.

Depends-On: I76157ccedfbcb8b0c2cba852bfa6b78ba6981c6c
Change-Id: I9f5c71aa896f25cfece39e080c74f99fb7be57b2
2017-11-24 11:28:04 +00:00
Jesse Pretorius eee7670c21 ansible-role-requirements: Switch ODL git repo to the github mirror
Cloning from the official ODL git repository fails far too often with
the following error

fatal: unable to access 'https://git.opendaylight.org/gerrit/p/integration/packaging/ansible-opendaylight.git/': gnutls_handshake() failed: Error in the pull function

Lets switch to the github mirror which may be more reliable.

Change-Id: Ib6a902a857739b34f94104d389e7cdbc9cc6c5d7
2017-11-21 10:34:55 +00:00
Jesse Pretorius c69d80c4eb Initial OSA zuul v3 role jobs
This patch implements an initial set of jobs intended to match
the current job execution method. It does not intend to improve
how the jobs are executed - only to replicate what is currently
in openstack-infra/openstack-zuul-jobs and provide the platform
to iterate on.

Change-Id: I9763a35b0177693bce2904f0c9a0f70408c5d4de
2017-10-17 20:31:31 +00:00
OpenStack Proposal Bot 48745b9dbf Updated from OpenStack Ansible Tests
Change-Id: I479b5d04bf66378f5654a90a00e0ec1ce367e080
2017-10-13 14:48:10 +00:00
Jesse Pretorius 466bab3c6b Add opendaylight role to tests a-r-r
In order to pass the lint test, all meta-dep
roles for the roles we execute in the test must
be present. As the os_neutron role recently got
the opendaylight role added as a meta-dep it is
now required here too.

Change-Id: I44dec03565e5be74cf3a03961e47180e50cf88a8
2017-08-23 06:56:26 +01:00
ZhongShengping acc790ead6 Add ceph_client role
The role 'ceph_client' was not found in openstack-ansible-os_ironic,
so add it.

Change-Id: I202556f75008dddb64207bdaef79e932e69469d7
Closes-Bug: #1704294
2017-07-17 10:31:29 +08:00
OpenStack Proposal Bot f1f0bbb9a2 Updated from OpenStack Ansible Tests
Change-Id: I9cb80cf501a3129d650c8e578e40b953dc3d5ce4
2017-06-27 14:51:08 +00:00
ZhongShengping c7a1f783a0 Replace neutron module with os_network and os_subnet
The neutron library has been removed[0] in openstack-ansible-plugins.
We can use standard ansible module instead.

[0]https://review.openstack.org/#/c/467797/

Change-Id: Ia30a72bcd3d112d7dce7ebe4cdf5da659b983808
Closes-Bug: #1700700
2017-06-27 17:39:23 +08:00
Jesse Pretorius 7aeb1c65d3 Use zuul-cloner for tests repo in OpenStack-CI
When executing the tests repo clone in OpenStack-CI,
use zuul-cloner instead of git to enable cross-repo
testing. This ensures that if a dependent patch from
the tests repo is noted using 'Depends-On: <change-id>'
in the commit message, that patch will be included.

Change-Id: I46e8d71e36c37d916836fad2c0604183aab46a0b
Depends-On: Idce7abebf32f24c356a27e099fbca954d917402b
2017-05-04 16:51:46 +01:00
Jimmy McCrory 4995ed78f3 Fix role gate tests for Ansible 2.3
'ansible_become' defined as a host_var or in the inventory for a host
has a higher precedence than 'become' defined within a playbook.

In change Id5b76a87809f03951c954fc3d752419a673403f7, 'become' was
defined explicitly for each play running against localhost so
'ansible_become' should be removed from its host_vars.

Change-Id: I0a3019ed7f4e151625682d009e5359378152592d
2017-04-19 17:31:12 -07:00
Andy McCrae 3acc49c204 Add nova_placement service to test inventory
Change-Id: I73bd0a45bdb8f41119dac8d3fc05d1c96b492f6d
2017-02-08 11:28:18 +00:00
Andy McCrae cede11ace8 Fix ironic gate
This patch fixes a few minor issues:
* The ironic_service_user_name was not being passed to Nova
  - this value was fixed in Nova #1647253
* The br-flat network seems to already existing on some gate nodes, this
has been changed to named br-vlan

Change-Id: I12f68a1cda39a3d615fbcf6ad62694faca6d09f4
2017-01-17 13:32:49 +00:00
Andy McCrae 9f084a910c Add support for neutron network names for cleaning/provisioning
This patch adds the ability to specify a cleaning and provisioning
network for Ironic. If none are specified then Ironic continues to
function as it does now.

Ironic role will calculate the UUID of the neutron network assuming a
network name is provided.

Additionally, this is added to testing by configuring a network to add
with the network-name.

Change-Id: I9be6f351c0da292ac8b861d2168e73d1861e1603
2016-12-08 15:54:15 +00:00
Andy McCrae 2e46ac9589 Add rsyslog_client dependency for Swift role
The testing needs swift which needs rsyslog_client.
This PR updates the role requirements for testing.

Change-Id: Id1aafc80345f7f12caecf883d41916b26f1cd00a
2016-11-16 21:33:38 +00:00
Joshua Hesketh 1adaee4408 Replace github link with published spec
Change-Id: I497132fbe107019184b657645df78cc2c7d08258
2016-11-03 12:13:28 +11:00
Jesse Pretorius a404529ec0 Use centralised Ansible test scripts
This patch consumes the centralised Ansible test scripts
implemented in https://review.openstack.org/381853

Change-Id: I2623678bc73068a64178fafc94db124c8c62ba83
2016-10-05 13:06:55 +01:00
Andy McCrae 729fbca6bf Move to non stand-alone ironic
We want to add virtual bmc testing and a full stack test of Ironic, as a
first step lets set it up with all the integrated components.

Change-Id: If4db7ada81e3ce54a6f28db6a67e7e0b4d679cce
2016-09-29 08:27:03 +00:00
Jesse Pretorius 53491a3f6d Use centralised test scripts
This patch consumes the test scripts implemented by
https://review.openstack.org/375061 to ensure that
the tests and test preparation is consistent and
more maintainable.

Change-Id: Ie01392e33b86b9b61019cc733f9f3db15f36c19d
2016-09-28 08:51:21 +01:00
Michael Davies f24f266208 os_ironic mysql password variable not updated
Ironic in OSA is currently broken as the ironic database
user isn't created, and consequently the ironic services can't
connect to the database. It broke back in patch 91deb13.

This patch corrects the os_ironic side of the problem. Another
patch will fix the openstack-ansible side.

Change-Id: I30b04756258a3241bb2444fd98f3a6624c0d2309
Partial-Bug: #1625081
Signed-off-by: Michael Davies <michael@the-davies.net>
2016-09-21 17:31:57 +00:00
Travis Truman 5ee8b16437 Ansible 2.1.1 role testing
Change-Id: If01b3672825aae1cab690035b43cf2da92180ead
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
2016-09-21 10:01:36 -04:00
Tim Laszlo f75d7195e3 Define keystone region name.
If multiple regions exist, keystone_authtoken should contain region_name.
This prevents the service from authenticating across regions.

Change-Id: I3e08385d91f5f913209dc944875c843828da3c2b
2016-07-22 12:40:09 -05:00
Jenkins d86aeb65cb Merge "Support service-specific rabbitmq clusters" 2016-07-22 12:04:15 +00:00
Jimmy McCrory 6de135bfb1 Remove openstack_hosts from test requirements
The openstack_hosts role is not used in any of the test playbooks
or required by dependent roles and can be removed from the test
role requirements.

Change-Id: I4182c169c1fa01651bf2187182961187f23d27d4
2016-07-20 18:47:02 -07:00
Travis Truman ebbc044a48 Support service-specific rabbitmq clusters
All rabbitmq connection vars are now namespaced. Namespace support
was previously inconsistent which limited deployer override options.

Change-Id: Iccc4839f497ae4868d1234e8fd36220870ad7f04
Implements: blueprint multi-rabbitmq-clusters
2016-07-18 10:41:14 -04:00
Jesse Pretorius 2b7f402c42 Clean up container cache prep in tests
This patch cleans up the container cache preparation in the tests
due to the following patches:

- https://review.openstack.org/307856 (new image build process)
- https://review.openstack.org/315114 (new apt config process)
- https://review.openstack.org/322188 (new resolver config process)

Change-Id: I2da67e7bec7ad21959edd85e3dc80c97c857fb47
2016-06-17 19:28:27 +01:00
Jimmy McCrory f736ebb79f Remove pip_lock_down dependency
The pip_install and pip_lock_down roles have been merged.

Remove pip_lock_down from the role's meta dependencies and test
requirements.

Change-Id: I62d9f8dce930f4e0dfe56c872ca46995e0e78b6d
2016-06-08 21:08:25 -05:00
Andy McCrae 3113ef63af Fix Ironic gating
Ironic gating was failing due to missing pip packages, additionally
there were some structural changes ro the inventory file which were
needed.

Change-Id: I77c40e7a49ebfbe05aac6d6a3d11ba9329c60413
2016-05-31 10:57:04 +01:00