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

jewel: rgw: account/container metadata not actually present in a request are deleted during POST through Swift API #9542

Merged
merged 2 commits into from Jun 30, 2016

Conversation

Abhishekvrshny
Copy link

@Abhishekvrshny Abhishekvrshny commented Jun 7, 2016

This patch rectifies an issue with handling of user metadata that are
actually stored by an updated resource (account or container).
The expected behaviour is they will be merged with new medadata coming
from an HTTP client.

Backport: Jewel
Fixes: http://tracker.ceph.com/issues/15779
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit 3f3b18b)
Introduced in: 3f3b18b.
Fixes: http://tracker.ceph.com/issues/15977
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

(cherry picked from commit d6129e6)
@Abhishekvrshny Abhishekvrshny added this to the jewel milestone Jun 7, 2016
@Abhishekvrshny Abhishekvrshny self-assigned this Jun 7, 2016
ghost pushed a commit that referenced this pull request Jun 16, 2016
…actually present in a request are deleted during POST through Swift API

Reviewed-by:
@ghost ghost assigned oritwas Jun 29, 2016
@ghost
Copy link

ghost commented Jun 29, 2016

@oritwas does this backport look good to merge ? It passed the rgw suite http://tracker.ceph.com/issues/16344#note-6 (except for one failure which I believe to be unrelated).

ghost pushed a commit that referenced this pull request Jun 30, 2016
…actually present in a request are deleted during POST through Swift API

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@oritwas
Copy link
Member

oritwas commented Jun 30, 2016

lgtm

@oritwas oritwas merged commit 4e1ebac into ceph:jewel Jun 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants