614 Commits

Author SHA1 Message Date
James Denton
4c6578538a Remove glance_api_servers from ironic.conf
The [glance]glance_api_servers configuration option has been deprecated
since Pike and removed in 2019 per Ironic commit
dcfb4f10f31c11afa350ef6418a0e12e6be6e5b2. Removing from ironic.conf
template.

Change-Id: I3b09514635ee0c8665c425bd1fe42fb594361a0e
2021-12-23 16:24:43 -06:00
James Denton
fc6902e811 Update Ironic Documentation
This patch updates OSA Ironic documentation to be more current
and to provide working examples.

Change-Id: I98cb73f49c526d0699446c0325e05a10ffca3e01
2021-12-23 14:37:22 +00:00
Zuul
52a014c56f Merge "Remove default_boot_interfaces" 2021-12-18 10:37:12 +00:00
OpenStack Proposal Bot
cd684b1175 Updated from OpenStack Ansible Tests
Change-Id: Id739c620a54b67616ebd8501703294e64e152903
2021-12-17 16:46:14 +00:00
James Denton
4cd32a2127 Remove default_boot_interfaces
The value for default_boot_interfaces must be supported by all
configured drivers, otherwise conductor won't start. This configuration
can be applied as an override by the operator, if necessary, rather
than OSA.

Change-Id: I780a1f130b0c695822ba50066247688bf4874dac
2021-12-17 15:13:49 +00:00
Zuul
fc0df045ef Merge "Add Support for Deploying UEFI Baremetal Nodes" 2021-12-15 12:54:00 +00:00
James Denton
2a24dd59ba Add Support for Deploying UEFI Baremetal Nodes
This patchset adds support for deploying instances using UEFI baremetal
nodes. UEFI may replace Legacy BIOS mode in future Ironic releases. Tested
with Ubuntu Focal 20.04 LTS.

Change-Id: I0fa6234ec7321e1d69901175baeab4ddb08afc50
2021-12-12 14:24:30 +00:00
OpenStack Proposal Bot
8bf07336bc Updated from OpenStack Ansible Tests
Change-Id: Iea19ef0b94e9bb402b1adf2b2a79fb777f5c308a
2021-12-04 17:40:09 +00:00
Damian Dabrowski
9355e1dd2d Database connection pooling improvements
- Implemented new variable ``connection_recycle_time`` responsible for SQLAlchemy's connection recycling
- Set new default values for db pooling variables which are inherited from the global ones.

Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819424
Change-Id: I1c90ce68ce218d538cd89b111dc1ee4142f14eef
2021-12-02 19:18:25 +01:00
Zuul
18690acd54 Merge "Use config_template as a collection" 2021-12-02 13:44:23 +00:00
Dmitriy Rabotyagov
d446c8a0fb Use config_template as a collection
Since we still use ceph-ansible that has their own implementation of
config_template module it's worth to use mentioned module as a collection
explicitly.

Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819814
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/820110


Change-Id: I30a19fc655e2b173a9623297a88cd874d800d13e
2021-12-02 04:01:06 +00:00
Dmitriy Rabotyagov
301900f373 Refactor definition of lock path
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819300
Depends-On: https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/819298
Change-Id: I4d2fe4659fed362ab0bb1c179339a623d7ce3e08
2021-11-30 12:14:13 +02:00
James Denton
621d769b30 Add [nova] section to ironic.conf
This patch fixes at least one issue related to ironic-conductor and
its inability to report to Nova based on lack of Nova-related
configuration in ironic.conf.

Change-Id: I96aac9f0f2c1e9bdb6ebd601ee3546ad13e0d927
2021-11-16 09:51:31 -06:00
Dmitriy Rabotyagov
590541adc1 Refactor galera_use_ssl behaviour
With PKI role in place in most cases you don't need to explicitly
provide path to the CA file because PKI role ensures that CA is trusted
by the system overall. In the meanwhile in PyMySQL [1] you must either
provide CA file or cert/key or enable verify.

Since current behaviour is to provide path to the custom CA we expect
certificate being trusted overall. Thus we enable cert verification when
galera_use_ssl is True.

[1] 78f0cf99e5/pymysql/connections.py (L267)

Change-Id: I79e43119830da22f09d7666b25054c6c14c28ffb
2021-10-06 11:59:30 +00:00
Dmitriy Rabotyagov
302acd77a0 Remove iscsi deploy
Since iSCSI deploy has been removed [1] we should also replace iscsi deploy with direct
since it's suggested deploy option.

[1] https://review.opendev.org/c/openstack/ironic/+/789382

Change-Id: I72f166b2fdd808b623e698f44c4d77747cc987bb
2021-10-06 11:56:57 +03:00
likui
14792030b5 Changed minversion in tox to 3.18.0
The patch bumps min version of tox to 3.18.0 in order to
replace tox's whitelist_externals by allowlist_externals option:
https://github.com/tox-dev/tox/blob/master/docs/changelog.rst#v3180-2020-07-23

Change-Id: I8692f2ad6c997de8ee2170b4b3623482969e0bfb
2021-07-03 21:19:01 +08:00
Zuul
fb96a661ff Merge "Add variables for rabbitmq ssl configuration" 2021-05-31 14:33:43 +00:00
Dmitriy Rabotyagov
82ab901fa8 Replace linters test with integarted one
We've created integrated linters check job a while back and it's successfully
working for several releases. At the moment we experience difficulties
with future maintenance of the linters check from the openstack-ansible-tests
repo. So instead of fixing current one, we replace it with modern version of
the test.


Change-Id: I409671bd6c28dc5fa21b292cabf082b5671532ef
2021-05-21 15:52:52 +03:00
Jonathan Rosser
029292a02d Add variables for rabbitmq ssl configuration
Change-Id: Icfe4696b0924abcf04d7f501186bf14d31c6fbdd
2021-05-17 10:49:26 +00:00
Zuul
a1112c610c Merge "Use global service variables" 2021-04-20 10:30:14 +00:00
Dmitriy Rabotyagov
43f3f35afd Use global service variables
Instead of overriding each service separatelly it might make
sense for deployers to define some higher level variable that
will be used first or fallback to default variable.

Change-Id: Ib2928efaa35bd82f6e6f65fecf32cf8da2eb9f5d
2021-04-19 19:18:51 +00:00
OpenStack Proposal Bot
55545c4a51 Updated from OpenStack Ansible Tests
Change-Id: I88a34ccad5b4c070d3e8b487f9c902e5845e1863
2021-04-19 09:58:16 +00:00
Zuul
4c2ea1c3c4 Merge "[goal] Deprecate the JSON formatted policy file" 2021-04-06 14:15:26 +00:00
Dmitriy Rabotyagov
109698e942 [goal] Deprecate the JSON formatted policy file
As per the community goal of migrating the policy file
the format from JSON to YAML[1], we need to replace policy.json to
policy.yaml and remove deprecated policy.json.

config_template has been choosen instead of the copy, since it can
properly handle content that has been lookuped.

We make a separate task not to restart service when it's not needed.

[1] https://governance.openstack.org/tc/goals/selected/wallaby/migrate-policy-format-from-json-to-yaml.html

Change-Id: I701473f4d99e0be06dea494eee4f08212bb7d853
2021-03-24 12:14:22 +00:00
OpenStack Proposal Bot
013780a230 Updated from OpenStack Ansible Tests
Change-Id: I98061e7212b9759703448825e451c87e134c0b95
2021-03-22 08:47:10 +00:00
Zuul
ffb3549732 Merge "[reno] Stop publishing release notes" 2021-03-18 10:59:05 +00:00
Jonathan Rosser
e5920f628e Use ansible_facts[] instead of fact variables
See https://github.com/ansible/ansible/issues/73654

Change-Id: If731b98c03bae2b38eb494c060699abea3cda9e0
2021-03-16 08:04:54 +00:00
OpenStack Proposal Bot
d3c97a45d8 Updated from OpenStack Ansible Tests
Change-Id: I680c412e711a6dd733c25d3f6fd1ce1ed8fc2b8e
2021-03-12 22:19:14 +00:00
Zuul
af098c4a04 Merge "Remove references to unsupported operating systems" 2021-03-10 17:54:17 +00:00
Jonathan Rosser
a0a07a43ac Remove references to unsupported operating systems
All references to Gentoo, SUSE, Debian stretch and Centos-7  are removed.
Conditional tasks, ternary operators and variables are simplified where possible
OS specific variables files are generalised where possible

Change-Id: Ia73e88947f52a74d9c03a17a7b1201a6346b5ac3
2021-03-10 12:16:38 +00:00
Jonathan Rosser
9adda06789 Switch default virtualenv to python3
Change-Id: Id28e42c69d2e0a7f9688080d9bc8943c71744e26
2021-03-10 08:54:14 +00:00
Jonathan Rosser
b4df743cfe Move ironic pip packages from constraints to requirements
This is necessary to use the new pip resolver

Depends-On: I6003a1004b6042a804ea142684b42b17f16d6da8
Change-Id: I06958da8881c93abde25c114d06943d777ea2b0a
2021-01-28 16:36:20 +00:00
dmitriy
321ee06231 [reno] Stop publishing release notes
Since we copy all release notes to the integrated repo there is not need
in publishing release notes for each repository. We should only verify their
validity and linting.


Change-Id: I406244b8d8bfe109e9db74bd6cf5cfb9f7a7bbf7
2021-01-22 18:25:51 +02:00
Dmitriy Rabotyagov
67733c8f0c Define condition for the first play host one time
We use the same condition, which defines against what host some "service"
tasks should run against, several times. It's hard to keep it the same
across the role and ansible spending additional resources to evaluate
it each time, so it's simpler and better for the maintenance to set
a boolean variable which will say for all tasks, that we want to run
only against signle host, if they should run or not now.

Depends-On: https://review.opendev.org/758953
Change-Id: Iab3194322e133282fcb71830f2b94e1279106ebd
2020-12-02 08:41:27 +00:00
Zuul
5ba9a1c6e8 Merge "Update Inspector listener address and network" 2020-11-10 13:57:30 +00:00
zhoulinhui
a842b99adc Replace deprecated UPPER_CONSTRAINTS_FILE variable
UPPER_CONSTRAINTS_FILE is deprecated and TOX_CONSTRAINTS_FILE is
the new environment variable name that replaces it [1].

This allows to use upper-constraints file as more
readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>.

[1] https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file
[2] https://review.opendev.org/#/c/722814/

Change-Id: Ia5d8154c5c52c73a63f3ff75e724a4bd03a3e5b6
2020-11-10 05:16:31 +00:00
James Denton
7900aeb223 Update Inspector listener address and network
This patch allows ironic-inspector to listen on host IP
rather than 0.0.0.0, as well as allows an existing Neutron-managed
inspection network to be used for inspection traffic.

Change-Id: I645857ad62954f08b160e5889f93dc1f6423def2
2020-11-09 11:35:08 -06:00
Dmitriy Rabotyagov
d509a5798f Reduce number of processes on small systems
Even the most modest 4C/8T system would run with the maximum 16 processes
due to the calculation being VCPU*2.

We devide amount of CPUs to number of threads for hyperthreaded CPUs

Change-Id: I67c151181cc358b1adb2ee4dc7aab8fbff2c25ba
2020-10-30 18:30:11 +02:00
Zuul
af9cf08525 Merge "Add iPXE support to Ironic Conductor" 2020-10-26 12:36:22 +00:00
James Denton
8f3a9e207c Add iPXE support to Ironic Conductor
This patchset adds support for iPXE, which can speed up baremetal
provisioning considerably due to the use of HTTP versus TFTP.

Change-Id: I8b49ae37a0380cd7a2191f050a52c85cc373026b
2020-10-22 15:24:19 +00:00
Zuul
09825dc934 Merge "Updated from OpenStack Ansible Tests" 2020-10-21 23:41:51 +00:00
Dmitriy Rabotyagov
1fb838443e Trigger uwsgi restart
When we were migrating service to uwsgi usage, we clean forgot to
trigger uwsgi restart on service config change.

Depends-On: https://review.opendev.org/758953
Change-Id: Iaf42be11b69bd2630c3f2e929ccff8b9ad9f0639
2020-10-21 07:49:16 +00:00
OpenStack Proposal Bot
a7f4df532f Updated from OpenStack Ansible Tests
Depends-On: https://review.opendev.org/758953
Change-Id: Ie75b18dd4399030089f8048a53688f5c1d52f35e
2020-10-21 07:31:06 +00:00
Jonathan Rosser
09960d6ad6 Fix linter errors
Depends-On: https://review.opendev.org/755585
Change-Id: I373999aef6d5056be76e69eb0a13183846ee084a
2020-10-01 17:16:53 +01:00
OpenStack Proposal Bot
3964a0107f Updated from OpenStack Ansible Tests
Change-Id: Ie3d656800426605cbaaa9ff4514d568edbec251f
2020-09-24 16:56:17 +00:00
Mohammed Naser
bb97551847 Fix ironic ramdisk logs
Change-Id: Id545c8cda0de7bab73d26e2001e57004f65af10a
2020-08-28 10:32:28 -04:00
Zuul
de902a0ba8 Merge "Use the utility host for db setup tasks" 2020-08-21 15:56:05 +00:00
Dmitriy Rabotyagov
9a5cbb3dfc Use the utility host for db setup tasks
Move it to the service setup host (defaults to utility[0]) instead
of the galera[0] host, and use galera_address (defaults to internal VIP)
as the endpoint instead of a local connection on the db host.

Change-Id: I87d61c619920a945c7a0b0887e3902f39f2b1b3c
2020-08-20 18:20:08 +03:00
Dmitriy Rabotyagov
bea6f664bd Add ironic-inspector to constraints
We missed adding ironic-inspector to constraints which resulted
in installing always the latest avaialbe version in the pypi.

Depends-On: https://review.opendev.org/746931
Change-Id: I9a6e7273f752b6bdc1639be299e00933889b6cc9
2020-08-19 17:12:05 +03:00
Zuul
b329d94104 Merge "Updated from OpenStack Ansible Tests" 2020-08-14 18:02:51 +00:00