Commit Graph

4165 Commits

Author SHA1 Message Date
Zuul 93dd9916c3 Merge "devstack: Drop removed services" 2024-04-17 18:39:16 +00:00
Zuul ecbf4ebc8a Merge "Remove executable flag from some files" 2024-04-17 18:39:14 +00:00
Zuul 0c004ecbb2 Merge "Fix migration scripts with SQLAlchemy 2.0" 2024-04-17 00:17:51 +00:00
Zuul 12287c7075 Merge "SQLAlchemy 2.0: Drop use of removed autoload" 2024-04-16 10:18:17 +00:00
Zuul 603d437aef Merge "Add pyupgrade hook to pre-commit config" 2024-04-15 10:46:29 +00:00
Zuul b4e8727159 Merge "pyupgrade changes for Python3.8+ (7)" 2024-04-15 10:46:27 +00:00
Zuul 6f36be9cc6 Merge "pyupgrade changes for Python3.8+ (6)" 2024-04-15 10:46:25 +00:00
Zuul ce29065796 Merge "pyupgrade changes for Python3.8+ (5)" 2024-04-15 10:46:23 +00:00
Zuul 54495575ab Merge "pyupgrade changes for Python3.8+ (4)" 2024-04-15 09:46:01 +00:00
Zuul 314f2b60dc Merge "pyupgrade changes for Python3.8+ (3)" 2024-04-15 09:22:31 +00:00
Gregory Thiemonge 4e6121d9bc Fix migration scripts with SQLAlchemy 2.0
select no longer accepts list/tuple as first argument but it takes
multiple arguments [0]

[0] https://docs.sqlalchemy.org/en/14/changelog/migration_20.html#select-no-longer-accepts-varied-constructor-arguments-columns-are-passed-positionally

Closes-Bug: #2061348
Change-Id: I271986146f485b2cd4ae3391d1a27f4a57f97560
2024-04-15 09:05:57 +00:00
Takashi Kajinami d0c29009e6 SQLAlchemy 2.0: Drop use of removed autoload
The autoload arugment was removed[1] in SQLAlchemy and only
the autoload_with argument should be passed.

The autoload argument is set according to the autoload_with argument
automatically even in SQLAlchemy 1.x[2] so is not at all needed.

[1] c932123bac
[2] ad8f921e96

Closes-Bug: #2061303
Change-Id: Ic18044b8065d2350c180ad26f5307a77aa99037b
2024-04-15 18:05:33 +09:00
Sergey Kraynev 85cfb6c2ae Fix negative or 0 limit parameter in pagination
Octavia replace "limit" with None when it is less 1. (for example 0, -1)
However the further code failed to compare None and int values.
This patch fixes it by validation, that limit is None.

Co-Authored-By: Roman Goncharov <gadzhet007@gmail.com>

Closes-Bug: #2060917
Change-Id: I9bb45a1aca6b7b18644752a3dccc3ebfb7c106ef
2024-04-12 19:23:28 +00:00
Zuul f91ff6c5a6 Merge "pyupgrade changes for Python3.8+ (2)" 2024-04-12 18:03:28 +00:00
Zuul 29880ec667 Merge "pyupgrade changes for Python3.8+ (1)" 2024-04-12 18:03:26 +00:00
Sergey Kraynev 824b51a1da
Handle empty delay on update healthmonitor
Check that delay field is not UnsetType before further validation

Closes-Bug: #2059894
Change-Id: Ia853d43dc273019c76da09104f31aa7e1b154fec
2024-04-10 17:22:50 +04:00
Tom Weininger c19c69e668 Add pyupgrade hook to pre-commit config
Inspired by Nova's patch [1].

[1]: https://review.opendev.org/c/openstack/nova/+/896987

Change-Id: I7f8d2e820d539949e1724de7b82d685415ee3cd3
2024-04-10 10:35:39 +02:00
Tom Weininger 6bbdd48815 pyupgrade changes for Python3.8+ (7)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I9399730fed16b85686caa586788a1bc03ebd123a
2024-04-10 10:35:37 +02:00
Tom Weininger ba3bbfa866 pyupgrade changes for Python3.8+ (6)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I3924de7cd8e2d242006ec4f272ece0276053e2ed
2024-04-10 10:23:50 +02:00
Tom Weininger a0360f9719 pyupgrade changes for Python3.8+ (5)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I9aeb3d603a519c3029b5be1ab622a0250f628f7d
2024-04-10 10:23:50 +02:00
Tom Weininger 0009c7cda8 pyupgrade changes for Python3.8+ (4)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I7d69a002c4e4c79fc2f5a13e9056aa2461d0ab7a
2024-04-10 10:23:50 +02:00
Tom Weininger 3d3360953b pyupgrade changes for Python3.8+ (3)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: Ib160f988cea0c6a27b3c1efeb2b2953f8929dc5c
2024-04-10 10:23:50 +02:00
Tom Weininger 9b1b8a3b6f pyupgrade changes for Python3.8+ (2)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I4ad020fbedd8106a3a86768c25a0c4dc750ec88a
2024-04-10 10:23:50 +02:00
Tom Weininger 69552a2082 pyupgrade changes for Python3.8+ (1)
Result of running

$ pyupgrade --py38-plus $(git ls-files | grep ".py$")

This was inspired by Nova [1]

Fixed PEP8 errors introduced by pyupgrade by running:

$ autopep8 --select=E127,E128,E501 --max-line-length 79 -r \
  --in-place octavia

and manual updates.

[1]: https://review.opendev.org/c/openstack/nova/+/896986

Change-Id: I8560b5440c0c2338e468b12d8c4abcc47a8fb50a
2024-04-10 10:23:44 +02:00
Zuul fa7cc01787 Merge "Fix fully-populated API with allowed_cidrs" 2024-03-20 20:29:29 +00:00
OpenStack Release Bot e4d7186776 Update master for stable/2024.1
Add file to the reno documentation build to show release notes for
stable/2024.1.

Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/2024.1.

Sem-Ver: feature
Change-Id: I744c29c1a03aba952b98a9fa3b6772073fa3805c
2024-03-14 19:54:08 +00:00
Zuul 9f1a6e47d7 Merge "dib: Remove Ubuntu Forcal support" 2024-03-14 06:54:53 +00:00
Zuul 6b4975f5b2 Merge "dib: Remove remaining logic for CentOS/RHEL 8" 2024-03-14 06:28:05 +00:00
Gregory Thiemonge 5d9b23c602 Fix fully-populated API with allowed_cidrs
When creating a LB + a listener with an allowed_cidr with the
fully-populated API, an issue happened when Octavia validated that the
allowed_cidrs and the VIP ip address have the same IP version. The
vip.ip_address value was not updated in the load balancer object,
forcing the expiration of the DB object before entering _graph_create
fixes this issue.

Note: there's no change in the tests, the test function for this feature
exists, looks correct, and passes successfully, the bug is only
reproducible in octavia-api.

Closes-Bug: 2057751
Change-Id: Ia106d81c1b2588e5d938d2238c8a2f6660bf5ef1
2024-03-13 04:36:49 -04:00
Takashi Kajinami 4342c31ae2 dib: Remove Ubuntu Forcal support
Ubuntu Focal is no longer part of the tested environments, because of
newer LTS available now (Jammy).

Change-Id: I7a6df974762abdd94784416609304618ce702b6e
2024-03-13 12:16:59 +09:00
Takashi Kajinami 02a2195e54 dib: Remove remaining logic for CentOS/RHEL 8
... because CentOS 8 and RHEL 8 is no longer supported.

Change-Id: I90e5b85827a324c0a258fd30cf94b3e8ac8c841f
2024-03-13 12:14:36 +09:00
Michael Johnson 411e7c6dbc Check Amphora status on SR-IOV failover flows
As noted on an earlier patch[1], the "SetAmphoraFirewallRules" task was not
checking the Amphora status nor using an API timeout. This could cause failover
flows to take longer than necessary if one of the Amphora is missing.

This patch corrects that issue by honoring both the Amphora status and timeout.

[1] https://review.opendev.org/c/openstack/octavia/+/910101/13/octavia/controller/worker/v2/flows/amphora_flows.py

Change-Id: Ic5e8140b13164267236f0a5d9a48fbd84bcdd688
2024-03-12 17:12:25 +00:00
Zuul ffc6f83f07 Merge "Add --wait to Octavia cookbook" 2024-03-11 16:54:36 +00:00
Zuul 2abab95fa0 Merge "Fix neutron setting overrides" 2024-03-11 16:47:28 +00:00
Omer 6cc3e50a74 Add --wait to Octavia cookbook
So far we did not mention the --wait argument when we created Octavia
resources in the cookbook.

This argument will save the user some (loadbalancer show) API calls,
so one won't have to make sure the Octavia resources are ready every
now and then.

Change-Id: If066e420a7ada869f67fbea29c50dc896f8a72ea
2024-03-11 16:20:51 +00:00
Zuul 00e9eac7eb Merge "Use devstack helper functions in the plugin" 2024-03-08 20:05:08 +00:00
Zuul ef28b2e629 Merge "Drop direct execution of octavia/cmd/*.py" 2024-03-08 19:37:35 +00:00
Zuul 2b8af0dc62 Merge "When we failed to load pkcs12 cert print warning" 2024-03-08 14:11:17 +00:00
Zuul 46ce3ef2d0 Merge "Add release note about redis jobboard driver updates" 2024-03-08 14:11:15 +00:00
Zuul 91ee3d7c86 Merge "redis: Support multiple sentinel servers" 2024-03-08 14:11:14 +00:00
Zuul a26c452f6f Merge "fix: specify endpoint info. for neutron client" 2024-03-08 09:14:52 +00:00
Zuul c85bdad24d Merge "redis: Add username" 2024-03-08 08:13:44 +00:00
Takashi Kajinami b7c293656b Add release note about redis jobboard driver updates
This adds a release note to explain updates made recently in redis
jobboard driver[1][2].

[1] 16f6b2e8f6
[2] bd3ef61a0c

Change-Id: I6c43a0a810f01632696f254a31e9a17c2f2cd73d
2024-03-07 16:45:07 +00:00
Zuul 188ed9c46b Merge "Fix duplicate tasks in SRIOV LB Create flow" 2024-03-07 16:21:44 +00:00
Zuul 6b0ca25696 Merge "Add additional-vips to the feature matrix" 2024-03-07 11:30:47 +00:00
Zuul edfc9803f5 Merge "reno: Update master for unmaintained/victoria" 2024-03-06 09:09:49 +00:00
Zuul dc168aee83 Merge "reno: Update master for unmaintained/xena" 2024-03-06 08:47:57 +00:00
OpenStack Release Bot 5179cc3a2e reno: Update master for unmaintained/xena
Update the xena release notes configuration to build from
unmaintained/xena.

Change-Id: I611b21c6a03b01c8dff364b223da324d6e2fb1ac
2024-03-05 18:59:44 +00:00
OpenStack Release Bot 1d411539bb reno: Update master for unmaintained/wallaby
Update the wallaby release notes configuration to build from
unmaintained/wallaby.

Change-Id: If0336f59e8aa68298cd3efb9b8ee2dd391b90387
2024-03-05 18:58:35 +00:00
OpenStack Release Bot 959297dba0 reno: Update master for unmaintained/victoria
Update the victoria release notes configuration to build from
unmaintained/victoria.

Change-Id: I1e830e9c06f87125835afe2115d6b32a84fc2b80
2024-03-05 18:57:25 +00:00