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

rgw: Swift API: X-Trans-Id header is wrongly formatted #5721

Merged
2 commits merged into from Sep 2, 2015

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Aug 30, 2015

@ghost ghost self-assigned this Aug 30, 2015

@ghost ghost added this to the hammer milestone Aug 30, 2015

@ghost ghost added bug fix rgw labels Aug 30, 2015

@ghost

This comment has been minimized.

ghost commented Aug 30, 2015

@rzarzynski I would very much appreciate your input about this backport. I had to resolve conflicts that look harmless but you know more than I do ;-)

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

Conflicts:
	src/rgw/rgw_rest.cc

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

Conflicts:
	src/rgw/rgw_rest.cc

abhidixit and others added some commits Jun 8, 2015

Transaction Id added in response
Signed-off-by: Abhishek Dixit dixitabhi@gmail.com

(cherry picked from commit b711e31)

Conflicts:
	src/rgw/rgw_rest.cc
             the prototype of the end_header( function
             in the context of the diff changed
rgw: rework X-Trans-Id header to be conform with Swift API.
Fixes: #12108
Signed-off-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>
(cherry picked from commit d1735a4)

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

Conflicts:
	src/rgw/rgw_rest.cc

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@rzarzynski

This comment has been minimized.

Contributor

rzarzynski commented Aug 31, 2015

Hi @dachary. This backport looks good to me!

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5721: RGW Swift API: X-Trans-Id header is wrongly…
… formatted

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

ghost commented Sep 2, 2015

It passed a run of the hammer rgw suite ( see http://pulpito.ceph.com/loic-2015-09-01_17:25:32-rgw-hammer-backports---basic-multi/ ). @rzarzynski thanks for your review !

ghost pushed a commit that referenced this pull request Sep 2, 2015

Loic Dachary
Merge pull request #5721 from dachary/wip-12853-hammer
RGW Swift API: X-Trans-Id header is wrongly formatted

Reviewed-by: Radoslaw Zarzynski <rzarzynski@mirantis.com>

@ghost ghost merged commit e7c2269 into ceph:hammer Sep 2, 2015

@ghost ghost changed the title from RGW Swift API: X-Trans-Id header is wrongly formatted to rgw: Swift API: X-Trans-Id header is wrongly formatted Sep 13, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment