2507 Commits

Author SHA1 Message Date
Michael Johnson
63b5cfc14e Fix VIP plug failure if netns directory exists
In rare cases, the network namespace may already exist in the amphroa.
VIP plug should not fail if the directory is already present.

Change-Id: I33c2e1740bff1313ba6b8d3ef2ea4fe494263751
Story: 2004300
Task: 27856
2018-11-08 09:18:37 -08:00
Zuul
5bd7cc7cb4 Merge "Update README by adding Mailing List and Wiki URL" 2018-11-01 00:20:13 +00:00
Zuul
e4a6532cd2 Merge "Don't quote {posargs} in tox.ini" 2018-10-31 23:05:48 +00:00
Zuul
22c5028533 Merge "Redirect disk-image-builder logs, make verbose" 2018-10-31 14:28:39 +00:00
Kim Bao Long
95db992d00 Update README by adding Mailing List and Wiki URL
Currently, README.rst does not mention about Wiki page and Mailing
List as contact point to community. So this patch aims to add an
URL that links to Wiki page and Mailing List tag in `External
Resources` session.

Change-Id: I3d9fffbfe110c6515cf7f47835e2bcc32fa3be9d
2018-10-31 09:32:10 +07:00
Ian Wienand
5bec6f70c2 Redirect disk-image-builder logs, make verbose
Update the diskimage-create.sh to have a "-l" option to pass into
dib's "--logfile" option.  In the devstack job, use this to redirect
the dib output to a separate logfile.  Turn on verbose logging by
default for this log.

Copy this in the v1 jobs.  v2 jobs are covered by a similar change in
octavia-tempest-plugin: I5bc0d9a3b071733cbe31d618f7236a3c2285b3e5

Depends-On: https://review.openstack.org/612865
Change-Id: Ie639af4266152d4626f8312b1849deeba5048348
2018-10-30 11:39:12 +01:00
Michael Johnson
4b1c374bdc Fix diskimage-create README.rst
We missed updating the diskimage-create README.rst when we added
some new command line options. This patch corrects those omisions.

Change-Id: I28dbc5b83321a69e060ea7fef5a1889ec623a3be
2018-10-29 11:53:23 -07:00
Zuul
3cf910baa8 Merge "Simplify keepalived lvsquery parsing for UDP" 2018-10-28 19:02:11 +00:00
Zuul
2deadd3f83 Merge "Remove unused methods." 2018-10-27 08:39:50 +00:00
Zuul
47b24d7669 Merge "Add posibilities to set default timeouts" 2018-10-24 11:57:43 +00:00
Zuul
2c790796f5 Merge "Fix logging error in get_current_loadbalancer_from_db" 2018-10-22 19:03:23 +00:00
Nguyen Van Trung
809a383635 Don't quote {posargs} in tox.ini
Quotes around {posargs} cause the entire string to be combined into one
arg that gets passed to stestr. This prevents passing multiple args
(e.g. '--concurrency=16 some-regex')

Change-Id: I7c45e4525f255deec7c79ae567cf4b1dda111a4a
2018-10-22 16:34:59 +00:00
Vadim Ponomarev
be377afcca Remove unused methods.
Remove unused methods _reset_lb_and_listener_statuses, _reset_lb_status,
_reset_lb_listener_policy_statuses from controllers.

Change-Id: I8e5980489bc905daa946ff4e2e7c3ae698d5b0b4
Story: 1655768
2018-10-22 13:56:28 +00:00
Kamil Sambor
8923935df2 Add posibilities to set default timeouts
Default timeouts for backend member and frontend
client can be set now via config file.
Timeouts exposed in config in section haproxy_amphora:
 * timeout_client_data
 * timeout_member_connect
 * timeout_member_data
 * timeout_tcp_inspect

Change-Id: I6e1be42c5c15c4171b012734e4c2a1fded51dbc7
Partial-Bug: 1797130
Story: 2004042
Task: 27046
2018-10-22 12:55:08 +00:00
Michael Johnson
759cb75960 Update docs conf.py for openstackdocstheme change
A Starlingx patch[1] changed the signature for some openstackdocstheme
methods which is causing Octavia docs to not build.

This patch updates the octavia docs configuration for the new
openstackdocstheme.

[1] https://review.openstack.org/#/c/607298

Change-Id: Id09ab3b78291c28e116f1f4ffb8836eac0537d94
2018-10-19 16:18:20 -07:00
Zuul
8e373c8053 Merge "Remove deprecated parameters" 2018-10-19 20:59:02 +00:00
Michael Johnson
e644b9fdce Remove deprecated API settings
We moved "bind_host", "bind_port", "auth_strategy", and "api_handler"
into the api_settings section in the configuration file. These settings
have now reached the end of their deprecation cycle and are being removed.

Change-Id: I323a7bdceae5a8d0e3025800beaf1d0a13c12ef8
2018-10-18 19:42:21 +00:00
Luis Tomas Bolivar
feff3b472c Ensure pool object contains the listener_id if passed
When creating a pool with an associated listener_id, sometimes the
listener_id is not set. Problem comes from [1] not setting the listeners
property on db_pool pool object even if it is passed.

[1] https://github.com/openstack/octavia/blob/master/octavia/api/v2/controllers/pool.py#L244-L245

Story: 2003833
Change-Id: I414a3c6d2e97d30ff12eda4ecf71ec032998df7d
2018-10-16 10:56:26 +02:00
Zuul
e9bd56f094 Merge "Refactor the AAP driver to not depend on nova" 2018-10-16 06:23:20 +00:00
Zuul
d81257d4cf Merge "Increment versioning with pbr instruction" 2018-10-16 05:59:19 +00:00
German Eichberger
79cd851548 Refactor the AAP driver to not depend on nova
Replace calls to the nova client with calls to the compute driver.
This will help non vm efforts (e.g. zune) and also make the code
easier to break up later.

Change-Id: I7ee175c5ecb98af0ca5e299c2ac10e43eb40ed30
2018-10-15 13:48:42 -07:00
Zuul
ed301d7fa2 Merge "Add API versioning to the api-ref" 2018-10-14 13:43:59 +00:00
Sean McGinnis
dd1b1be815 Increment versioning with pbr instruction
With moving away from required milestone releases, the version numbers
calculated by PBR on the master branch will not work for those testing
upgrades from the last stable release. More details can be found in the
mailing list post here:

    http://lists.openstack.org/pipermail/openstack-dev/2018-October/135706.html

This is an empty commit that will cause PBR to increment its calculated
version to get around this.

PBR will see the following which will cause it to increment the version:

Sem-Ver: feature

Please merge this patch as soon as possible to support those testing
upgrades.

Change-Id: Id29359e81c002cc302b81a8779f363a4a2ba5474
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
2018-10-12 13:10:02 -05:00
Vadim Ponomarev
862f9f438e Fix logging error in get_current_loadbalancer_from_db
Change-Id: Ib69fe420658dee635045a47681a91fba921f7248
2018-10-12 11:32:13 +03:00
German Eichberger
df17903ab8 Delete zombie amphorae when detected
Zombie amphorae will be deleted by the system to lighten the load on
operators if the amphora record is still in the database.

Story: 2003912
Task: 26800

Change-Id: If133a3d36a9381bcca9f7d00f5c1531885907940
2018-10-12 00:00:06 +01:00
Jacky Hu
6d21b33cdc Remove deprecated parameters
This is a follow up of I70e5cb07a4435f58f5da3999be70162efa7f0bd8, which
eliminated the usage of the parameters removed here.

Change-Id: I9c52aa6ef73482e2f2d83a6b228407790e18ca74
2018-10-11 09:25:41 +08:00
Michael Johnson
9aea3501d1 Add API versioning to the api-ref
This patch documents the new parameters added, by API version greater
than 2.0.

Change-Id: Ieb2c30653e832b9fa726f8430da603945fd10cc2
2018-10-10 16:24:36 -07:00
Adam Harwell
2c88c553ae HTTPS HMs need the same validation path as HTTP++
Second take on this patch, didn't realize the Create and the Update
paths were different. Same exact change as the other patch, but in the
update validation method.

For reference: https://review.openstack.org/#/c/604924/

Change-Id: I916ccb7658d4849b3c208405ec40fc1a0eab4ba7
2018-10-04 16:08:00 -07:00
Zuul
2ea3ed8c48 Merge "HTTPS HMs need the same validation path as HTTP" 2018-10-03 16:17:20 +00:00
Zuul
a0112c4b37 Merge "Support REDIRECT_PREFIX action for L7Policy" 2018-10-02 11:27:03 +00:00
Zuul
687a0e8472 Merge "Separate the thread pool for health and stats update" 2018-10-01 19:41:57 +00:00
Michael Johnson
9b6aa47c03 Fix an upgrade issue for CentOS 7 amphora
A recent patch[1] (stein master) added the http-reuse option to the
haproxy template for pools. This feature is not available in the HAProxy
version included with CentOS 7, 1.5.x. This could cause an upgrade issue
if the control plane was upgraded to Stein, but the cloud still had older
CentOS based amphora.

This patch corrects that issue by checking the HAProxy version in the
amphora and adjusting the template if it finds an older HAProxy.

This patch also updates the test_health_check_stale_amphora test to
not wait (sleep) for the full heartbeat_timeout.

[1] https://review.openstack.org/#/c/598379/

Change-Id: I3d990d1d3cd93dbeced9edc53f9c166610dafcd0
Story: 2003901
Task: 26775
2018-09-28 16:37:22 -07:00
sapd
62192dba3a Support REDIRECT_PREFIX action for L7Policy
Currently, Octavia only support three actions for L7Policy,
in this patch we will implement new action for L7Policy.

Story: 2003700

Change-Id: Ie99591ede097b566294ebdb673c460442dd6d942
2018-09-28 18:10:39 +07:00
Zuul
7fa83ef8e6 Merge "Fix the API list performance regression" 2018-09-26 08:32:38 +00:00
Zuul
b4c323db16 Merge "Pass through DIB_LOCAL_ELEMENTS from localrc" 2018-09-26 04:11:02 +00:00
Zuul
89d992773e Merge "Fix health manager performance regression" 2018-09-25 23:16:49 +00:00
Michael Johnson
951f018efe Pass through DIB_LOCAL_ELEMENTS from localrc
This patch allows us to specify special DIB elements via the localrc.
An octavia-tempest-plugin patch[1] linked to this will use this
setting in the gates to configure DIB to use the infra package mirrors.

[1] https://review.openstack.org/#/c/601332

Change-Id: I7444e8941c7175d92c50f1714a87b4181b983829
2018-09-24 16:35:33 -07:00
Adam Harwell
7dd6f8b846 HTTPS HMs need the same validation path as HTTP
Change-Id: I2fd51664336dca51f134b3fccd3e8c936b809839
2018-09-24 15:14:17 -07:00
Michael Johnson
d08578d8fd Remove duplicate check gate jobs
We had a bad merge that duplicated some of the v2 check gate jobs.
This patch corrects that.

Change-Id: Ibe60e78d1c1dc6a978eaf80e16124b372aff6360
2018-09-24 11:24:03 -07:00
Zuul
48e0986051 Merge "Updates README-Vagrant.md to use OSC commands" 2018-09-20 22:27:35 +00:00
Michael Johnson
f15b43ddf4 Fix the API list performance regression
This patch fixes the Octavia v2 API "list" performance regression.

It also corrects some database model forward reference issues.

Co-Authored-By: Adam Harwell <flux.adam@gmail.com>
Change-Id: Ic110e0e73938743c1aba01aa28f393bae7141cbd
Story: 2002933
Task: 22920
2018-09-19 14:02:38 -07:00
German Eichberger
78ba1970f8 Updates the operator docs with the new lb failover command
Since Pike we have the failopver command on the load balancer and
this should be used instead of dsabling the port. Added other
minor updates.

Change-Id: I606518d8d9a52104872a08ee18bd6be62c100de3
2018-09-19 10:37:56 -07:00
Adam Harwell
8e886959c0 Simplify keepalived lvsquery parsing for UDP
The healthcheck parsing can be simplified a lot, which also fixes some
testing issues on OSX.
Fix a missing mock in the amphora server tests too, which was causing
the centos distro tests to run improperly / miss.

Change-Id: I7606822b820b501cd983e8975bcb3b7d5c58d904
2018-09-18 13:38:53 -07:00
ckji
756a17ce3a Updates README-Vagrant.md to use OSC commands
Change-Id: I255520474a38147722bc75f868ef6c7fccbcba97
2018-09-17 16:58:08 +08:00
Zuul
aee5c6e10e Merge "Gate on octavia-dsvm-base based jobs and housekeeping" 2018-09-15 02:29:09 +00:00
Zuul
182c942b5d Merge "Support DNF package manager in diskimage-create" 2018-09-15 00:46:10 +00:00
Zuul
d1168a496b Merge "Add the missing markup for the hyperlink title" 2018-09-14 23:37:17 +00:00
Zuul
4155922a74 Merge "Set some amphora driver optimizations" 2018-09-14 23:25:05 +00:00
Zuul
e2bdb0adf4 Merge "Update amphora-agent to report UDP listener health" 2018-09-14 23:14:31 +00:00
Zuul
1fd65c9acf Merge "Validate member address for batch update members" 2018-09-14 03:18:49 +00:00