Skip to content
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

V1.11.0 ovh #24

Open
wants to merge 564 commits into
base: master
Choose a base branch
from
Open

V1.11.0 ovh #24

wants to merge 564 commits into from

Conversation

Rom123Soleil
Copy link

Add quota sets for manila

pierreprinetti and others added 30 commits February 13, 2023 09:14
As per the [OpenStack object-storage
docs](https://docs.openstack.org/api-ref/object-store/#create-container),
container names must not contain a slash (`/`).

With this patch, objectstorage functions error when called with a containerName
containing a slash.
Even though this is a separate package, it's helpful to include some
minimal documentation for same here rather than insisting folks root
through the gophercloud/utils docs.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Manila: add List for share-access-rules API
Fix invalid baremetal-introspection service type
networking v2: add extraroutes Add and Remove methods
Version 0.10 changes the default GIT_BASE to point to github repos
rather than opendev.
The jobs are running in the Github infra and we should prefer github
repos when possible.
It's not mirrored on github.
In order to help debugging, it can be useful to have the devstack's
local.conf file around as well as an overview of devstack running
services.
The devstack environment was not setup properly to enable acceptance
tests developed for
gophercloud#2571.
…-tests

Enable tests for object versioning
With this commit, all `IsEmpty() (bool, error)` functions now check the
status code of the result. If it's `204`, they immediately confirm that
the page is empty.

This change was introduced in commit 64ed1bc in `objectstorage` to
support Swift instances running behind a reverse proxy that was
truncating `content-type` headers from `204 Empty` responses. With this
change, the same check is applied to all services.

While no functional impact is expected on non-proxied OpenStack modules,
this patch will prevent issues on proxied clouds. In general, it also
seems reasonable to expect `204 Empty` responses to be empty.

---

This change is the result of running:

```shell
find -name 'results.go' -exec sed -i 's|^\(func (\(.\+\) .\+) IsEmpty() (bool, error) {\)$|\1\nif \2.StatusCode==204 {\nreturn true,nil\n}\n|' {} \;
go fmt ./...
```
[all] IsEmpty to check for HTTP status 204
[neutron v2]: Add support for network segments update
pierreprinetti and others added 30 commits February 15, 2024 15:51
Also start using the commit hash of the release, rather than the tag.
[v1] build(deps): bump EmilienM/devstack-action from 0.11 to 0.14
Due to an error in implementing the addition of context.Context, the
default reauth function caught the context passed when generating the
ProviderClient, which could be long canceled when the reauthentication
takes place.
The volume attachments API operates on volumes, not attachments. Correct
the variable name.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Closes: gophercloud#2861
[v1] compute: Use volumeID, not attachmentID for volume attachments
When the Provider client contains a non-nil Context, the [`doRequest`
method](https://github.com/gophercloud/gophercloud/blob/5b44c45ec947301d531178302c4cecc0a923ad96/provider_client.go#L401-L405)
of the Provider client generated a new context by merging the Provider
client's and the per-call context. However the new context is too
short-lived to let the callers consume a response body.

This version reverts the inclusion of Context in the v1 branch.

This version replaces v1.9 and v1.10.
Co-authored-by: Maxime Delord <maxime.delord@ovh.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet