Update loadbalancer/v2/l7policies create and update#2324
Update loadbalancer/v2/l7policies create and update#2324mandre merged 1 commit intogophercloud:masterfrom
Conversation
97abf9f to
64179a6
Compare
|
Build failed.
|
|
recheck |
|
Build failed.
|
|
@EmilienM can you have a look? The failing tests seem to be "irrelevant" afaik, but maybe im missing something |
mandre
left a comment
There was a problem hiding this comment.
@EmilienM can you have a look? The failing tests seem to be "irrelevant" afaik, but maybe im missing something
Correct, the failing tests are all in acceptance/openstack/networking/v2/extensions/qos and unrelated to your patch.
Could you just add the redirect_prefix and redirect_http_code fields to results.go as well?
64179a6 to
bd63942
Compare
Add REDIRECT_PREFIX and REDIRECT_HTTP_CODE to l7policy CreateOpts and UpdateOpts. Docs: https://docs.openstack.org/api-ref/load-balancer/v2/?expanded=create-an-l7-policy-detail,update-a-l7-policy-detail#create-an-l7-policy For: gophercloud#2322
bd63942 to
27b4c75
Compare
@mandre done! Thanks :) |
mandre
left a comment
There was a problem hiding this comment.
All good now 👍
Let's just wait for CI to come back.
|
Build failed.
|
|
Same 5 failures in qos acceptance tests, something's off with that job but it's not related to your PR. |
Add REDIRECT_PREFIX and REDIRECT_HTTP_CODE to l7policy
CreateOpts and UpdateOpts.
Docs: https://docs.openstack.org/api-ref/load-balancer/v2/?expanded=create-an-l7-policy-detail,update-a-l7-policy-detail#create-an-l7-policy
Fixes: #2322