1592 Commits

Author SHA1 Message Date
Nobuto Murata
22523e5b54 Allow overriding libvirt/num_pcie_ports
Especially with arm64/aarch64, the default value limits the number of
volume attachments to two usually. And when more than two volumes are to
be attached, it will fail with "No more available PCI slots". There is
no one-size-fits-all value here so let operators override the default
value.

Closes-Bug: #1944214
Change-Id: I9b9565873cbaeb575704b94a25d0a8556ab96292
2021-09-28 12:31:20 +00:00
James Troup
7d850b2754 Spelling, grammar and consistency fixes to config.yaml.
Change-Id: Iecb447eb94a47c72c9f0a1a749fa5628d02a8ef7
2021-09-24 01:04:07 +01:00
James Troup
0720a0a402 Clarify CPU pinning config options.
Change-Id: I646834db820b9bfe09784a4c659b2a4a69bb1c72
2021-09-21 18:37:02 +01:00
Arif Ali
8048e6c74a
Add ovmf package for uefi support
When creating an instance with uefi support it is looking for various
files that are typically provided by ovmf package. This is automatically
installed from train and above, but is lacking in queens. This will
solve it for any deployment.

Change-Id: I7665a596112d673624ea5edd7438b915834918df
2021-09-06 16:41:37 +01:00
Zuul
0a42fa9245 Merge "Added neutron_physnet and neutron_tunnel config" 2021-09-01 10:24:37 +00:00
Stephan Pampel
6ab0d0d5f6 Added neutron_physnet and neutron_tunnel config
Adding neutron_physnet and neutron_tunnel as config options
for numa affinity to the charm.

Implementation of [0] config options.
Only affects OpenStack releases >= rocky

[0]https://docs.openstack.org/nova/rocky/configuration/config.html#neutron.physnets

Closes-Bug: #1921067
Change-Id: Ib050c23f5e4d2da8262c37f136dbc66129141017
2021-08-31 10:24:39 +02:00
Peter Matulis
0bddb1d88f Replace NFV section with a link
The NFV section was previously copied to the CDG [1].
This PR replaces it with a summary and a link to the
new location.

[1]: I220ad1a178a253ca04e1667e8c5727f8a54aa5cd

Change-Id: I66987fbea9d9fa4fbc971f621726b5eca0e53024
2021-08-27 18:49:53 -04:00
James Troup
f526b1f4a9 Allow nova-compute to read /sys/module/kvm_amd/parameters/sev
Closes-Bug: #1935697
Change-Id: I4cb54c26f285e0ea283193416f58bda3080bb38d
2021-08-06 15:14:12 +00:00
Peter Matulis
b3c463b668 Reword security-checklist action definition
Be specific about what action 'security-checklist'
actually does.

Will be used for the other ten charms that support this
action.

Change-Id: I51916eed45c526991685e95b667804e2a490b5a9
2021-08-04 12:08:14 +02:00
Aurelien Lourot
50ca3cc76c Sync release-tools
In order to fix ResolutionImpossible on tox invocation.

https://github.com/openstack-charmers/release-tools/issues/151
https://github.com/openstack-charmers/release-tools/pull/152

Change-Id: I7613f4296a86c7889bdad41a68868117b4d93fdb
2021-07-22 14:59:02 +02:00
Zuul
cdb292ff39 Merge "Improve config help for aa-profile-mode and fix spelling mistakes." 2021-07-16 12:32:52 +00:00
Zuul
aff2a0b182 Merge "Added allocation-ratio config opts" 2021-07-15 23:36:30 +00:00
James Troup
cfa5bbd0da Improve config help for aa-profile-mode and fix spelling mistakes.
Change-Id: Ia99192226649585fd683eff3db3c525b0daa78e1
2021-07-09 20:13:01 +01:00
Zuul
f3fab948f8 Merge "Update action definitions" 2021-06-24 09:41:20 +00:00
Zuul
35185a612b Merge "Improve definition of virt-type option" 2021-06-23 11:57:51 +00:00
Zuul
6713703894 Merge "Port ncc pci-alias list fix to nova-compute" 2021-06-17 18:25:56 +00:00
Zuul
b168963f2e Merge "c-h sync - restore proxy env vars for add-apt-repository" 2021-06-15 03:03:54 +00:00
Rodrigo Barbieri
d51e010fca Port ncc pci-alias list fix to nova-compute
Port of https://review.opendev.org/535350

Fix for this bug already exists for
nova-cloud-controller charm, therefore
port it over to nova-compute.

Closes-bug: #1921147
Change-Id: I0c37d2bce3b195f5dc18111bb70a30de03b2bbc6
func-test-pr: https://github.com/openstack-charmers/zaza-openstack-tests/pull/587
2021-06-14 14:39:56 -03:00
Peter Matulis
a435b03e33 Update action definitions
Remove references to the charm's README for two
actions:

* register-to-cloud
* remove-from-cloud

Not only is this non-standard but, in particular, the
referenced information will be moved to the CDG

Drive-by: Improve the remaining defintions

Depends-On: I03493b30956eddcb77bd714360806aa53c126942
Change-Id: I80c212a223637e2b76e421085fef1b466f548c30
2021-06-11 14:44:55 -04:00
Zuul
369b67b28a Merge "apparmor: ensure multipath.conf is accessible" 2021-06-11 15:08:54 +00:00
Zuul
06fbe4aa6c Merge "Set cross_az_attach mode from ncc relation configuration" 2021-06-11 14:49:42 +00:00
Zuul
203140252c Merge " Add 'virsh-audit' action in nova-compute" 2021-06-11 14:36:41 +00:00
Peter Matulis
63273a4e1d Improve definition of virt-type option
The lxd hypervisor is no longer supported

I'm not sure whether the other types should be
there, even for testing.

Improve general wording

Related-Bug: #1931579
Change-Id: I8510382c256dbe731b810f09f25408f7fa1d7b15
2021-06-10 12:02:07 -04:00
James Page
9d9a74ddda apparmor: ensure multipath.conf is accessible
Allow access to main multipath configuration file from the
nova-compute daemon.

Change-Id: Ibaa5f45b7fd72fcc936986286939e1285bcdb945
Closes-Bug: 1906727
2021-06-08 13:29:22 +01:00
Aurelien Lourot
1f6d8c2aa7 Add impish to metadata.yaml
Change-Id: I97b9645ef3679e0c795602a53c445ca0cb2fc0bc
2021-06-03 11:22:14 +02:00
Brett Milford
43acdd4f80 Added allocation-ratio config opts
Nova supports setting allocation ratios at the nova-compute level from
Liberty onwards. Prior to this allocation ratios were set at the
nova-scheduler level.

Newton introduced the Placement API, and Ocata introduced the ability to
have compute resources (Core/RAM/Disk) precomputed before passing
candidates to the FilterScheduler [0]. Pike removed CoreFilter,
RAMFilter and DiskFilter scheduler filters.

From Pike onwards valid methods for settings these allocation ratios are via:
- A call to the Placement API [1].
- Config values to supplied to nova-compute (xxx_allocation_ratio).

Stein introduced initial_xxx_allocation_ratio in response to the runtime
behaviour of the ResourceTracker [2].

Currently, the precedence of resource ratio values are:
xxx_allocation_ratio > Placement API call > initial_xxx_allocation_ratio

That is a (compute) resource provider's allocation ratios will default
to initial_xxx_allocation_ratio which may be overridden at run time by a
call to the Placement API. If xxx_allocation_ratio is set it will
override all configurations for that provider.

When not otherwise configured, we set initial_xxx_allocation_ratio to
the values provided by ncc to maintain backwards compatibility. Where
initial_xxx_allocation_ratio is not available we set
xxx_allocation_ratio.

[0] https://specs.openstack.org/openstack/nova-specs/specs/ocata/implemented/resource-providers-scheduler-db-filters.html
[1] https://docs.openstack.org/api-ref/placement/#update-resource-provider-inventories
[2] https://specs.openstack.org/openstack/nova-specs/specs/stein/implemented/initial-allocation-ratios.html

Change-Id: Ifa314e9e23e0ae5d16113cd91a7507e61f9de704
Closes-Bug: #1677223
2021-05-24 15:39:47 +10:00
Peter Matulis
6d1450d565 Update README - scaling down
Remove section on scaling down as it has been documented
as a cloud operation in the CDG

Apply README template to the bottom of the README

Depends-On: I03493b30956eddcb77bd714360806aa53c126942
Change-Id: Iab6f0bdce05ee5115b0cdbb517cba79aa87dabf0
2021-05-14 17:15:33 -04:00
Corey Bryant
31108aefce c-h sync - restore proxy env vars for add-apt-repository
Change-Id: Ib8e1597a5b08bcb79e49773fc14b4c8e1173c654
2021-05-13 08:47:23 -04:00
Martin Kalcok
5eea014dd5 Add action that returns number of running VMs on the unit.
Closes-Bug: #1911011
Change-Id: If9ae646bc91b6389876830843bbf905596963054
func-test-pr: https://github.com/openstack-charmers/zaza-openstack-tests/pull/535
2021-05-12 09:44:17 +00:00
Zuul
ddc6662aba Merge "Add apparmor rule to support /usr/sbin" 2021-05-07 20:38:31 +00:00
Chris MacNaughton
d1bd93b6c5 Test bundles for focal-wallaby and hirsute-wallaby
These are the test bundles (and any associated changes) for
focal-wallaby and hirsute-wallaby support.

Libraries sync.

hisute-wallaby test is disabled (moved to dev) due to [1] as bundle may
reference a reactive charm.

[1] https://github.com/juju-solutions/layer-basic/issues/194

Change-Id: I238e8a36b033594c67ffcefa325998f2eba2a659
2021-05-06 10:16:49 +00:00
Alex Kavanagh
c22787d88f Updates to flip all libraries back to master
This patchset updates all the requirements for charms.openstack,
charm-helpers, charms.ceph, zaza and zaza-openstack-tests back
to master branch.

Change-Id: I453eafd76c005cd5f10041c08e4a943fe235d474
2021-05-03 16:09:13 +01:00
Liam Young
120235f359 Add apparmor rule to support /usr/sbin
It seems that as of Disco *1 /sbin is a symlink to /usr/sbin. This
patch adds support for file in either location.

*1 https://lists.ubuntu.com/archives/ubuntu-devel-announce/2018-November/001253.html

Change-Id: I66fa27f3f5e29d83cfea0f1afb33374303ab4669
Closes-Bug: #1925511
2021-04-26 13:51:06 +00:00
Billy Olsen
bfc3436a32 Use get_availability_zone() on cloud-compute relation
Commit 9f4369d9 added a feature to set the availability zone of
the nova-compute unit on the cloud-compute relation. This uses the
value of the JUJU_AVAILABILITY_ZONE environment variable, which is
not consistent with how the nova-compute service sets its availability
zone.

Use the nova_compute_utils.get_availability_zone() method instead.

Closes-Bug #1925412

Change-Id: Ie68ecd808a60baf0d5bfe526f4355ce3c7ae5c77
2021-04-21 20:48:46 -07:00
Liam Young
fc835f8c54 CH Sync to pick up fixes to bug/1923739
Change-Id: I7b828e8fc13ab61dac7057ca5270ccabbf4d218d
Closes-Bug: 1923739
2021-04-15 13:00:02 +00:00
Billy Olsen
17e44e12e3 Set cross_az_attach mode from ncc relation configuration
The cross_az_attach property needs to be configured on the compute
nodes. The policy is set on the ncc service and is propigated to the
compute nodes on the cloud-compute relation. Update the relevant cinder
config setting based on the value provided.

Note, the default value for cross_az_attach is aligned with the nova
default of True.

Closes-Bug: #1899084
Change-Id: I7d00b50acbfe05dfd943a3511126b507fc570aeb
2021-04-13 11:45:24 -07:00
Alex Kavanagh
75975ff917 21.04 sync - add 'hirsute' in UBUNTU_RELEASES
The 'hirsute' key in c-h/core/host_factory/ubuntu.py:
UBUNTU_RELEASES had been missed out, and is needed for
hirsute support in many of the charms.  This sync is to
add just that key.  See also [1]

Note that this sync is only for classic charms.

[1] https://github.com/juju/charm-helpers/pull/598

Change-Id: I60b208bbf5a04a9ab598b76ff0cf7f8baf216cbb
2021-04-11 16:58:39 +01:00
Zuul
3709741027 Merge "21.04 libraries freeze for charms on master branch" 2021-04-06 18:43:33 +00:00
Alex Kavanagh
557b63b33e 21.04 libraries freeze for charms on master branch
* charm-helpers sync for classic charms
* build.lock file for reactive charms
* ensure tox.ini is from release-tools
* ensure requirements.txt files are from release-tools
* On reactive charms:
  - ensure stable/21.04 branch for charms.openstack
  - ensure stable/21.04 branch for charm-helpers

Change-Id: I14762601bb124cfb03bd3f427fa4b1243ed2377b
2021-04-03 20:31:35 +01:00
Martin Kalcok
3f00b8f509 Add action that returns list of registered compute nodes.
List contains only nodes registered to the same nova-cloud-controller
as the nova-compute service running on targeted unit.

Closes-Bug: #1911013
Change-Id: I28d1a9bd18b3a87fc31ff4bca5bfe58449cdae57
2021-04-01 08:45:11 +02:00
Zuul
1989ec7890 Merge "Added action that returns nova-compute node name." 2021-03-31 20:55:40 +00:00
Martin Kalcok
38efefccdf Added action that returns nova-compute node name.
Closes-Bug: #1911012
Change-Id: Id9a7c3a675072ed4da3b9cb9fc997e70895205cb
2021-03-30 11:49:18 +02:00
Zuul
f28ae8a3e1 Merge "Cap the number of workers always when multiplier is not set" 2021-03-29 14:23:02 +00:00
Zuul
e76484fcd0 Merge "Added actions that enable clean removal of nova-compute unit from model" 2021-03-25 08:15:40 +00:00
Martin Kalcok
ceb8a68868 Added actions that enable clean removal of nova-compute unit from model
List of added actions:
* disable
* enable
* remove-from-cloud
* register-to-cloud

More detailed explanation of the process added to the README.md

Closes-Bug: #1691998
Change-Id: I45d1def2ca0b1289f6fcce06c5f8949ef2a4a69e
func-test-pr: https://github.com/openstack-charmers/zaza-openstack-tests/pull/470
2021-03-24 16:07:58 +01:00
Zuul
70af11583e Merge "Add new osci.yaml" 2021-03-19 11:11:14 +00:00
Chris MacNaughton
da4243b591 Add new osci.yaml
This change is preparatory to migration Ubuntu OpenStack CI
from Jenkins to Zuul

Change-Id: I996496c12e189257a27fdb39dd094ccb0a6df929
2021-03-18 07:21:52 +01:00
Edward Hope-Morley
433796779e Fix error when setting cpu-dedicated-set
Change-Id: Ic89d48f81af46682dc251cfdd6d88152c7039b0f
Closes-Bug: #1897275
2021-03-10 12:17:21 +00:00
Nobuto Murata
5db8e14d4d Cap the number of workers always when multiplier is not set
Previously the cap is only applied to the units in containers. However,
services in a bare metal also requires a sensible cap. Otherwise,
nova-compute, for example, will have 256 workers for nova-api-metadata
out of the box which is an overkill with the following system.

32 cores * 2 threads/core * 2 sockets * 2 (default multiplier) = 256

Let's cap the number of workers as 4 always, then let operators override
it with an explicit config to worker-multiplier.

Synced charm-helpers for
https://github.com/juju/charm-helpers/pull/553

Closes-Bug: #1843011
Change-Id: If98f12d7cf1a77fb267f1b55c44896a48a40909a
2021-03-05 13:28:50 +09:00
Alex Kavanagh
b49dbc252c Add hirsute and remove trusty from metadata.yaml
This update adds the new hirsute Ubuntu release (21.04) and
removes trusty support (14.04 which is EOL at 21.04).

Change-Id: I59840b672673aa4a8e253659300d9333c1b20a4b
2021-03-02 21:02:06 +00:00