Fix quota set to target admin project instead of default - #1438
Conversation
|
EDIT: sorry i now read the commit message and i see what's going on :). Still i think it would be good to get a review from @rebtoor :) |
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
cefd5de to
4b4dd56
Compare
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
4b4dd56 to
3497806
Compare
abhibongale
changed the title
PR openstack-k8s-operators#1412 added a quota increase for pre-adoption Ironic test instances, but targeted project 'default' instead of 'admin'. The OS_CLOUD=overcloud authenticates as project admin (clouds.yaml: project_name: admin), so quota set should target admin to match the project instances are created under. Since adoption migrates the TripleO DB, this quota carries over to the adopted cloud, providing headroom for post-adoption instance creation as well. Related-Bug: openstack-k8s-operators#1412 Signed-off-by: Abhishek Bongale <abhishekbongale@outlook.com>
3497806 to
f49f9a2
Compare
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
/recheck |
|
@abhibongale don't forget to remove the |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jistr, rebtoor The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
89b86f9
into
openstack-k8s-operators:main
3 participants
PR #1412 added a quota increase for pre-adoption Ironic test
instances, but targeted project 'default' instead of 'admin'.
The OS_CLOUD=overcloud authenticates as project admin
(clouds.yaml: project_name: admin), so quota set should
target admin to match the project instances are created under.
Since adoption migrates the TripleO DB, this quota carries
over to the adopted cloud, providing headroom for
post-adoption instance creation as well.