Commit Graph

1387 Commits

Author SHA1 Message Date
OpenDev Sysadmins e0c3a80991 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:36:39 +00:00
Liam Young 40fd0c2fc9 Add support for cells v2
This change adds relations necessary for registering a compute cell
with the superconductor. For a cell to be registered this charm
must have relations with the compute cells conductor, database and
message queue. Only when all these relations are complete can the
registration happen. Below are major changes included in this PR.

* Add nova-cell-api relation for communicating with the
  nova-cell-conductor
* Add shared-db-cell relation for communicating with the
  a compute cells database.
* Add amqp-cell relation for communicating with the
  a compute cells message queue.
* Add methods for registering cells with the
  superconductors database.
* Charm helper sync

Change-Id: I54f98c1d420b6a6b1dd8ed7dbd00a2362b584b81
2018-07-30 10:39:57 +00:00
Liam Young 21b17020d8 Remove support for nova cells v1
This change removes support for nova cells v1.

Change-Id: Id86697b6a2e893e023769ea214671e958e7166ca
2018-06-15 05:57:52 +00:00
Frode Nordahl ba0b7f7e91
Remove orphan symlink
Support for dfs was dropped in commit 0f14eac672

Remove orphan symlink in the actions/ directory.

Change-Id: I7a4dc5cf2a20431581f51fb1e6cc48b6f172879a
2018-05-28 08:46:22 +02:00
Liam Young 57d13c665b Add support for tls-certificates relation
Add support for the charm to request and receive certificates from
the tls-certificates relation.

Change-Id: I54995aefbb31a845dbb46800d0444697ed057da7
2018-05-15 14:56:53 +00:00
David Ames f03ccf02b7 Enable Bionic as a gate test
Change bionic test from dev to gate for 18.05.

Change-Id: I5a82ac79b29181fabec41570cd5aa003d2bfb8ea
2018-05-11 16:12:20 -07:00
Ryan Beisner d6cf5285c1 Remove deprecated functional test targets
Change-Id: I4887e897a1ae5e4a181f6bdbc148fdb31a82655d
2018-05-09 16:37:04 -05:00
Frode Nordahl edc18d4937
Make charm agnostic of underlying init system
Replace charm custom init control functions with `service_pause`
and `service_resume` functions from charm-helpers.

Change-Id: I235af30a19294316f65fba0e13fe10ae50164a42
Closes-Bug: #1765215
2018-04-20 19:10:59 +02:00
David Ames 794b8ed0bb Charm-helpers sync to fix CA cert comparison
The comparison of bytes vs string of the CA certificate produces a
false negative. This leads to rewriting certificates and affecting
connectivity to services.

Read in the certificate as bytes as well for a bytes vs bytes
comparison.

Closes-Bug: #1762431

Change-Id: Ie2348a83671b9636bd94227e903b1a50bff7aecc
2018-04-11 14:26:49 -07:00
Ryan Beisner dc4f54a6e3 Update tox.ini to stop using unverified package
As of pip 10.0, --allow-unverified is not permitted.

Use of the flag in this repo was previously used to force
installation of python-apt to accommodate certain unit tests.

The unverified package, python-apt, is no longer necessary
for test execution.

Related-Bug: #1760720

Change-Id: I9d5b2d5139ed61ff9d7f116c33e3e145362b46e1
2018-04-03 16:33:35 +00:00
Corey Bryant 8893bace59 Ensure only the leader can add hosts to cell
Ensure that only the elected leader can add compute hosts to the
cell in cloud-compute-relation-changed as database commands should
only be run by the elected leader.

Change-Id: I4806580f58e2a2feba82c74d9d5ff29dfb220c9e
Closes-Bug: #1752402
2018-03-21 14:17:33 +00:00
Liam Young 1cd062efde Fix up amulet for queens
Switch to use Keystone v3 clients for Queens

Change-Id: Ic93b430fa12f9d283c507a51a58e0468b7d4a850
2018-02-27 09:01:48 +00:00
Ryan Beisner f5213021b7 Sync charm-helpers
Change-Id: If5ce5cd9b5aa88ec4b12b1087fc9ee4770a4be0a
2018-02-21 14:09:38 -06:00
Billy Olsen d43e264d02 Sync charm-helpers and use VolumeAPIContext
Sync in the charm-helpers to use the new VolumeAPIContext object
in order to determine the volume catalog info to use in the
configuration file.

This is simply an alternative implementation for commit 16ae06bf
which separates the concerns for volume endpoint context and
internal endpoint context.

Change-Id: I187bc8ef5644a59dc5a0b5f7ad66774812a78b78
Related-Bug: #1733566
2018-02-08 19:01:44 -07:00
Liam Young a0778a5304 Only run map_instances for Ocata
map_instances only needs to be run during the transition from
Newton to Ocata to map existing instances into a cell. All
new Ocata instances will be automatically mapped to a call.
This change limits the slow running map_instances command to
just run for upgrades to Ocata

Change-Id: Ic2e2df530504284d28cfcab26a71d211342203fa
Closes-Bug: #1743357
2018-01-23 13:27:00 +00:00
Ryan Beisner 33881c371b Sync charm-helpers
Notable issues resolved:

openstack_upgrade_available() broken for swift
https://bugs.launchpad.net/charm-swift-proxy/+bug/1743847

haproxy context doesn't consider bindings
https://bugs.launchpad.net/charm-helpers/+bug/1735421

regression in haproxy check
https://bugs.launchpad.net/charm-helpers/+bug/1743287

Change-Id: Ifb1d9e71924db6ab8a3d9ee34fcd1b40f910d68a
2018-01-19 16:18:19 +02:00
Liam Young c276dfdb96 Batch up map_instances call
nova-manage map_instances maps all instances into a cell in batches
of 50 if max-count is not set. Setting max-count causes the script
to run a single batch of size max-count. The return code of the
script shows if there are still more to do. This change runs
map_instances repeatedly with a batch size of 50000 while rc is 1
and then exists cleanly when a rc 0 is recieved.

Change-Id: Id1184778a5ae94bb3b57348b10d12077b093d6dd
Partial-Bug: #1742115
2018-01-17 14:43:10 +00:00
Zuul 8d7b98f48d Merge "Add amulet test for pci-alias" 2018-01-16 19:37:14 +00:00
Liam Young 0a9b1a21a7 Add amulet test for pci-alias
Add an amulet test to check pci-alias setting is propagated
properly into nova.conf. This is a prerequisite for work to support
setting multiple aliases.

Depends-On: I6ac49b0a2782df2ebc8f3d1fe299dec3cc078d7c
Change-Id: Ic8d51dfb3130670cd6810ca80378d6ef8c0d0346
2018-01-15 12:52:01 +00:00
Liam Young edd9b1face Add action for running archive-deleted-rows
Add an action for moving stale data to shadow tables using
nova-manage *1. This will speed up other operations such as
map_instances which no longer need to work against stale
data.

*1 https://docs.openstack.org/nova/pike/cli/nova-manage.html

Change-Id: I03f3d641b50cfc6f02262edb0f714ba6e9566775
Partial-Bug: #1742115
2018-01-12 14:11:16 +00:00
James Page 0f14eac672 Remove deploy from source support
Drop support for deployment from Git repositories, as deprecated
in the 17.02 charm release.  This feature is unmaintained and has
no known users.

Change-Id: I2fe15b648d485e5b03965a00dee6324669ebe9fa
2018-01-12 10:42:50 +00:00
Zuul 4a9209ea0f Merge "Update amulet tests for new s3/ec2/nova username" 2018-01-05 09:26:22 +00:00
Zuul 08d425ee07 Merge "Fix the misspelling of "migration"" 2018-01-04 13:48:53 +00:00
Liam Young c837c1412f Update amulet tests for new s3/ec2/nova username
Keystone now orders the elements of the username when constructing
a username for multiple services. Update amulet to expect the
new username (only actually affects icehouse).

Change-Id: I6ac49b0a2782df2ebc8f3d1fe299dec3cc078d7c
Closes-Bug: #1739409
Depends-On: I0f857d7c2d5c4abf4843bc3fe1a9848164048fe2
2018-01-03 13:00:53 +00:00
James Page 5c79af1bd1 Drop postgresql support
Remove postgresql DB support; This feature is untested as part
of the charms, is not in use and was deprecated as part of
the 1708 charms release.

Change-Id: I866559f519ace3476e0cc85661d99e5f5491227d
2017-12-21 12:01:58 +00:00
Guoqiang Ding c68b23524c Fix the misspelling of "migration"
Change-Id: I797913d5d13a8e7b2734d0e20b1717de3a4fa565
2017-12-21 13:57:16 +08:00
James Page 0196625084 Drop zeromq support
Support for the ZeroMQ messaging driver has bit-rotted over
the last few years across the OpenStack charms; drop support
for ZMQ inline with deprecation notices issued in 17.02 charm
release.

Change-Id: I39e464b289c9df842ca73e64c7576ba0531963ae
2017-12-15 17:27:01 +00:00
David Ames 373158b5cf Update HAProxy default timeout values
The default HAProxy timeout values are fairly strict. On a busy cloud
it is common to exceed one or more of these timeouts. The only
indication that HAProxy has exceeded a timeout and dropped the
connection is errors such as "BadStatusLine" or "EOF." These can be
very difficult to diagnose when intermittent.

This charm-helpers sync pulls in the change to update the default
timeout values to more real world settings. These values have been
extensively tested in ServerStack. Configured values will not be
overridden.

Partial Bug: #1736171

Change-Id: I0a3a8f0dd2dedcc8e02dd6af2f5486501698833e
2017-12-11 11:37:20 -08:00
Edward Hope-Morley 16ae06bf42 Use volumev3 for internal endpoint in >= Pike
volumev2 api is deprecated for >= Pike so switch
internal endpoint to v3 for Pike onwards.

Change-Id: Icb9861efbd650b079f507ca585818e95921dc42a
Partial-Bug: 1733566
2017-12-08 10:06:49 +00:00
Zuul 2adc46ac04 Merge "Adding http_proxy_to_wsgi support to mitaka." 2017-12-05 09:53:51 +00:00
Ryan Beisner 50fa95b4fb Add Bionic and remove Zesty series and tests
Bionic, being the next LTS, is important to enable for dev
and test as early as possible ahead of 18.02.

Zesty goes EOL in Jan 2018. The next stable charms release (18.02)
will not provide Zesty series support, as it was an interim
(non-LTS) release.

Change-Id: I4052507a8ff6bf24a4699cd8ab6bcee950157835
2017-12-05 05:29:36 +00:00
Ryan Beisner 1da186ac2d Sync charm-helpers
Change-Id: I7ffa8b100144e14b736a7f204977f0aee24aada3
2017-11-22 18:25:39 +00:00
Seyeong Kim d9f9431456 Adding http_proxy_to_wsgi support to mitaka.
nova mitaka doesn't have http_proxy_to_wsgi

but mitaka nova has issue related to this.

I backported commit 6051f30a, b609a3b32 to api-paste.ini.

Change-Id: Ibeb8f655504ecfc1763743faa52f7e3aa552c3c4
Related-Bug: #1573766
2017-11-22 09:39:12 +09:00
Andrew McLeod 93bd3d7279 Enable xenial-pike amulet test
Make default func27-smoke xenial-pike
Charm-helpers sync

Change-Id: If2de44a0661aaa4679c7ed4c8bfe99e56fb8c24d
2017-11-16 15:41:44 +13:00
Zuul 8d72cd1588 Merge "Resync charmhelpers for py3 fixes." 2017-11-15 07:59:14 +00:00
Liam Young f427dc5a48 Resync charmhelpers for py3 fixes.
Change-Id: I28457389a65a835204729c5070b420715664aa26
2017-11-13 16:13:50 +00:00
Edward Hope-Morley 8dc0b72a4b Set default value of disk-allocation-ratio to 1.0
This makes it inline with upstream Openstack defaults.

Change-Id: Id771c26a7b023bc5fdc330e03758e5425dc60e8e
2017-11-10 09:43:08 +00:00
Edward Hope-Morley e824e45ed2 Ensure enabled_filters are in [filter_scheduler]
For >= Ocata the nova.conf template is adding filters
to an incorrect section unless additional filters are
defined due to a typo in the template. This patch
ensures that filters are always set in the
[filter_scheduler] section.

Change-Id: Iad5d830dd530ea27ddd3b92a39c15524e48af0aa
Closes-Bug: 1731319
2017-11-09 21:30:51 +00:00
Zuul 0ab4dff278 Merge "Only use PciPassthroughFilter when sriov enabled" 2017-11-07 17:38:58 +00:00
Edward Hope-Morley f42591849d Fix unit tests missing mocks
Running the current unit tests you keep getting prompted
for root password because of a load of missing mocks.
This patch mocks out functions that there left unmocked
in unit tests that were not actually testing them.

Change-Id: I6d7dac4f900f133a4d1ac64e431ffcb6b2cfd58e
2017-11-06 12:31:50 +00:00
Edward Hope-Morley 05ea3a13f1 Only use PciPassthroughFilter when sriov enabled
Fixes code that inadvertently interprets strings as
booleans resulting in them always being True.
PciPassthroughFilter will now only be added to
scheduler_default_filters if sriov is enabled.

Change-Id: Ie2c79e98d7880e8172ac7e77febb51af04732bec
Closes-Bug: 1730369
2017-11-06 12:03:27 +00:00
Edward Hope-Morley af5a4ec747 Add disk-allocation-ration config option
We already support configuring ram and cpu overcommit so
this patch adds the ability to control disk overcommit.

Change-Id: Ib172e8b48cb34de4beb9cec1a277dc43ded36eed
Closes-Bug: 1726346
2017-10-23 12:11:35 +01:00
Zuul 5b85e562f4 Merge "Add nova-manage failure verbosity and clean up Cells V2 code" 2017-10-16 23:33:42 +00:00
Corey Bryant 3b873932be Add nova-manage failure verbosity and clean up Cells V2 code
Update all nova-manage commands to use subprocess.check_output()
and log subprocess.CalledProcessError.output on failure. This
will help capture nova-manage error details on first failure.

Specify cell1 uuid on discover_hosts call. This doesn't change
behavior, it is just more explicit and useful if we move to
multiple cells in the future.

Introduce an is_cellv2_init_ready() function that uses contexts to
check if cells_v2 is ready to initialize. This cleans up the
corresponding TODOs.

Move checks for cell v2 init readiness to update_cell_db_if_ready(),
also cleaning up corresponding TODOs.

Change-Id: I313edce84d3d249031e020a4fbb4baf216c01ddb
Related-Bug: 1720846
2017-10-16 12:19:55 +00:00
Liam Young 1fcbc69649 Return early from identity-service-relation-joined
After the previous commit landed it was suggested on that it would
be cleaner to return early from identity-service-relation-joined
if the cluster was partially formed rather than use a long if/else
. This change makes that style update.

Change-Id: I9dab5d930e13dda6cfe2c6e11648587c87bdb955
Partial-Bug: #1544959
2017-10-06 09:34:06 +00:00
Liam Young 36e9518921 Block endpoint reg if cluster partially formed
When an existing cluster of the service is scaled out the new unit
will join with keystone before it is fully clustered. In identity
joined hook the charmhelpers function canonical_url is called which
in turn uses another charmhelpers function, resolve_address.
resolve_address will only return the vip if the vip is set in config
AND the unit is clustered. This means that the units local address
is returned and that is then registered with keystone.

This change gates registering an endpoint if the cluster is
partially formed.

Change-Id: Ibe7d2286de130f045257ad6552f7a5b1279958b6
Partial-Bug: #1544959
2017-10-03 10:02:11 +00:00
Ryan Beisner a332621052 Update requirements for git charmhelpers
Change-Id: I57490579664d55aca17244afe5d9da635d2afff2
2017-10-03 04:33:15 +00:00
Ryan Beisner 0a70b50c89 Add Artful dev series metadata
Change-Id: I56c757ea0749c1003da068b7c23344800c38efb7
2017-09-26 14:58:09 -04:00
Frode Nordahl 7e8713b01b Update repo to do ch-sync from Git
Change-Id: Idc8eddcb2973736df76d7d7d8354fe681fbab64a
2017-09-26 09:55:32 +02:00
zhangyangyang 29f42fa5de change assert(Not)Equals to assert(Not)Equal
According to http://docs.python.org/2/library/unittest.html
assert(Not)Equals is a deprecated alias of assert(Not)Equal.

Change-Id: Iac2d5eefe47b96efc787c6eca5ca014ca3b3910c
Closes-Bug: #1329757
2017-09-12 15:50:42 +08:00