Skip to content

Objectstore v1: allow to remove account and object metadata#2063

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
ovh:feature/objectstore-account-remove-metadata
Dec 2, 2020
Merged

Objectstore v1: allow to remove account and object metadata#2063
jtopjian merged 1 commit intogophercloud:masterfrom
ovh:feature/objectstore-account-remove-metadata

Conversation

@reenjii
Copy link
Copy Markdown
Contributor

@reenjii reenjii commented Nov 24, 2020

For #2062

Documentation about metadata handling in middlewares.
https://github.com/openstack/swift/blob/master/doc/source/development_middleware.rst#user-metadata

Actual code that handles X-Remove-xxx-Meta-name haders.
https://github.com/openstack/swift/blob/master/swift/proxy/controllers/base.py#L1647

Documentation update account.
https://docs.openstack.org/api-ref/object-store/#create-update-or-delete-account-metadata

X-Remove-Account-name (Optional) | header | string | Removes the metadata item named name. For example, X-Remove-Account-Meta-Blue removes custom metadata.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.005%) to 79.662% when pulling 88f38ca on ovh:feature/objectstore-account-remove-metadata into b47fd44 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Nov 24, 2020

Build succeeded.

Copy link
Copy Markdown
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

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

I apologize for the delay with reviewing this.

LGTM - thank you!

@jtopjian jtopjian merged commit 11768ba into gophercloud:master Dec 2, 2020
clementblaise pushed a commit to clementblaise/gophercloud that referenced this pull request Dec 7, 2020
@Benzhaomin Benzhaomin deleted the feature/objectstore-account-remove-metadata branch November 2, 2021 16:05
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.

3 participants