-
Notifications
You must be signed in to change notification settings - Fork 595
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[occm] add octavia loadbalancer flavor option #1160
[occm] add octavia loadbalancer flavor option #1160
Conversation
Flavor can be specified in config or via annotation. Annotation overrides config default.
Welcome @phiphi282! |
Hi @phiphi282. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @zetaab |
Build failed.
|
Build failed.
|
Build failed.
|
Build failed.
|
Build failed.
|
Build succeeded.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good. However, we need fix tests first
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: zetaab The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
/ok-to-test |
/retest |
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Build succeeded.
|
Tests are working now again :) |
/lgtm |
This reverts commit 7203e42.
Flavor was introduced in Octavia since version v2.6 (on 2019-01-25), for those deployments using Octavia prior to v2.6, load balancer creation will fail as the parameter @phiphi282 Please submit another PR to add the version check (example https://github.com/kubernetes/cloud-provider-openstack/blob/master/pkg/util/openstack/loadbalancer.go#L88), otherwise, we have to revert this PR. |
* Add support to specify octavia flavor. Flavor can be specified in config or via annotation. Annotation overrides config default. * Add short doc entry for flavor-id
What this PR does / why we need it:
This pr adds an option to specify the flavor-id used for the octavia loadbalancer.
The flavor-id can be set in the config or via annotation.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:
Flavors were being introduced in octavia version 4.0.0 if I am not mistaken.
I haven't found a way to check the octavia version from within the occm so if a check is needed here I would appreciate if someone can point me in the right direction here.
Release note: