Skip to content

Prevent load balancer creation errors #16

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 4, 2023
Merged

Prevent load balancer creation errors #16

merged 1 commit into from
Sep 4, 2023

Conversation

priteau
Copy link
Member

@priteau priteau commented Aug 10, 2023

Octavia load balancers are launched using the service project. With default quotas, load balancer creation can quickly fail:

Quota exceeded for instances: Requested 1, but already used 10 of 10 instances (HTTP 403)

Set some service project quotas to unlimited to avoid these failures.

@priteau priteau self-assigned this Aug 10, 2023
@priteau priteau requested a review from a team as a code owner August 10, 2023 07:49
markgoddard
markgoddard reviewed Aug 10, 2023
View reviewed changes
Copy link

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice addition. Would it make sense to separate it from the demo project so that it's clear that it's for octavia?

@priteau
Copy link
Member Author

priteau commented Aug 11, 2023

Nice addition. Would it make sense to separate it from the demo project so that it's clear that it's for octavia?

Do you mean building openstack_projects from two separate lists?

@markgoddard
Copy link

Nice addition. Would it make sense to separate it from the demo project so that it's clear that it's for octavia?

Do you mean building openstack_projects from two separate lists?

I meant a separate file. I see the examples as snippets to be combined, but I suppose that's open to discussion.

@priteau
Prevent load balancer creation errors
2f32964 
Octavia load balancers are launched using the service project. With
default quotas, load balancer creation can quickly fail:

    Quota exceeded for instances: Requested 1, but already used 10 of 10 instances (HTTP 403)

Add a new example setting required service project quotas to unlimited
to avoid these failures.
@priteau
Copy link
Member Author

priteau commented Aug 16, 2023

Nice addition. Would it make sense to separate it from the demo project so that it's clear that it's for octavia?

Do you mean building openstack_projects from two separate lists?

I meant a separate file. I see the examples as snippets to be combined, but I suppose that's open to discussion.

Thank you for clarifying. I didn't really notice the layout used for the examples. I am pushing an updated change.

@priteau priteau requested a review from markgoddard August 16, 2023 13:42
@markgoddard markgoddard merged commit 961f102 into main Sep 4, 2023
@markgoddard markgoddard deleted the octavia-quotas branch September 4, 2023 09:06
scrungus pushed a commit that referenced this pull request Nov 30, 2023
@markgoddard
Merge pull request #16 from stackhpc/octavia-quotas
806d7d4 
Prevent load balancer creation errors
scrungus pushed a commit that referenced this pull request Feb 8, 2024
Squashed commit of the following:
78726b7 
commit bcab111
Author: scrungus <33693738+scrungus@users.noreply.github.com>
Date:   Wed Dec 6 17:32:04 2023 +0000

    role install no longer needed

commit 16aacaa
Author: scrungus <33693738+scrungus@users.noreply.github.com>
Date:   Mon Dec 4 17:07:10 2023 +0000

    comments

commit 5d09004
Author: scrungus <tyler@stackhpc.com>
Date:   Mon Dec 4 17:02:35 2023 +0000

    change to official collection

commit 4a7350e
Author: scrungus <tyler@stackhpc.com>
Date:   Thu Nov 30 16:01:12 2023 +0000

    separate venv is unnecessary now as all roles updated to new sdk

commit f652f13
Author: scrungus <tyler@stackhpc.com>
Date:   Thu Nov 30 16:08:32 2023 +0000

    changes for ansible-collection-openstack

commit 8429a57
Merge: 961f102 8ea98a4
Author: Bartosz Bezak <bartosz@stackhpc.com>
Date:   Tue Sep 5 14:18:13 2023 +0200

    Merge pull request #18 from stackhpc/domains

    add domains

commit 8ea98a4
Author: Bartosz Bezak <bartosz@stackhpc.com>
Date:   Tue Sep 5 11:27:17 2023 +0200

    add domains

commit 961f102
Merge: 382918a 2f32964
Author: Mark Goddard <mark@stackhpc.com>
Date:   Mon Sep 4 10:06:27 2023 +0100

    Merge pull request #16 from stackhpc/octavia-quotas

    Prevent load balancer creation errors

commit 382918a
Merge: ebab9f7 8c5c7c4
Author: Matt Crees <matthew.crees1@gmail.com>
Date:   Thu Aug 24 09:26:12 2023 +0100

    Merge pull request #17 from stackhpc/os-networks-rbac

    Add RBAC support into openstack-networks.yml

commit 8c5c7c4
Author: Matt Crees <mattc@stackhpc.com>
Date:   Wed Aug 23 17:31:59 2023 +0100

    Add RBAC support into openstack-networks.yml

commit 2f32964
Author: Pierre Riteau <pierre@stackhpc.com>
Date:   Wed Aug 16 15:39:36 2023 +0200

    Prevent load balancer creation errors

    Octavia load balancers are launched using the service project. With
    default quotas, load balancer creation can quickly fail:

        Quota exceeded for instances: Requested 1, but already used 10 of 10 instances (HTTP 403)

    Add a new example setting required service project quotas to unlimited
    to avoid these failures.

commit ebab9f7
Merge: 798f72a 4c23c55
Author: Mark Goddard <mark@stackhpc.com>
Date:   Thu Aug 10 09:53:11 2023 +0100

    Merge pull request #15 from stackhpc/noeol

    Wrap lines and add missing new line at end of file

commit 4c23c55
Author: Pierre Riteau <pierre@stackhpc.com>
Date:   Thu Aug 10 09:06:54 2023 +0200

    Wrap lines and add missing new line at end of file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@markgoddard markgoddard markgoddard approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@priteau priteau

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@priteau @markgoddard