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
[JUJU-4619] Missing oracle cloud regions and list not being updated #16287
[JUJU-4619] Missing oracle cloud regions and list not being updated #16287
Conversation
This commit adds 51 new region for Oracle cloud provider. Regions list was taken from `oci` cli-tool output.
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.
QA has a failure, I did a juju bootstrap oci-canonical/us-ashburn-1
and the controller was still created in Phoenix, which is the default. Same for added models, cloud/region works
$ juju add-model five oci-canonical/us-ashburn-1
Added 'five' model on oci-canonical/us-ashburn-1 with credential 'default' for user 'admin'
But region only, per the juju add-model --help
fails.
$ juju add-model five us-luke-1
ERROR cloud us-luke-1 not found
Use 'juju clouds' to see a list of all available clouds or 'juju add-cloud' to a add one.
Even though I added a model in us-ashburn-1
, machine appeared in us-phoenix-1
region.
This commit removes the gov regions, if needed we will add them as saparate clouds (like did with AWS).
90dbbc7
to
9d31376
Compare
/merge |
#16498 Forward merges: - #16459 - #16467 - #16473 - #16474 - #16475 - #16476 - #16393 - #16480 - #16481 - #16287 - #16477 - #16486 - #16434 - #16487 - #16488 - #16491 - #16492 - #16494 - #16489 - #16497 Conflicts: - api/client/machinemanager/machinemanager.go - api/client/machinemanager/machinemanagernew_test.go - apiserver/facades/client/modelconfig/modelconfig.go - apiserver/facades/client/modelconfig/modelconfig_test.go - apiserver/stateauthenticator/context.go - cmd/juju/application/refresh_test.go - scripts/win-installer/setup.iss - snap/snapcraft.yaml - version/version.go Mostly trivially solved. `cmd/juju/application/refrest_test.go` was the only one that required any original code. Verify by ensuring unit tests pass
This PR adds 51 new region for Oracle cloud provider. Regions list was taken from
oci
cli-tool output.Checklist
QA steps
Commands to run to verify that the change works.
go test github.com/juju/juju/cloud/... -gocheck.v juju regions oracle
Links
Launchpad bug: https://pad.lv/2031468
Jira card: JUJU-4619