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: the arguments 'domain' should not be assigned when return false #5720

Merged
merged 1 commit into from Sep 24, 2015

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Aug 30, 2015

rgw:the arguments 'domain' should not be assigned when return false
    Hostnames: [B.A]
    Inputs: [X.BB.A]
    Return: [false]
    Output: [B.A] it is wrong.

Fixes: #12629
Signed-off-by: Ruifeng Yang <149233652@qq.com>
(cherry picked from commit 9420d24)

@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 pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

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

Merge pull request #5720: the arguments 'domain' should not be assign…
…ed when return false

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

This comment has been minimized.

ghost commented Sep 2, 2015

@yehudasa does this backport look good to merge ? 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/ ).

@ghost ghost assigned yehudasa and unassigned ghost Sep 6, 2015

@ghost

This comment has been minimized.

ghost commented Sep 13, 2015

@yehudasa ping ?

1 similar comment
@ghost

This comment has been minimized.

ghost commented Sep 24, 2015

@yehudasa ping ?

yehudasa added a commit that referenced this pull request Sep 24, 2015

Merge pull request #5720 from dachary/wip-12854-hammer
the arguments 'domain' should not be assigned when return false

@yehudasa yehudasa merged commit 19e549e into ceph:hammer Sep 24, 2015

@ghost ghost changed the title from the arguments 'domain' should not be assigned when return false to rgw: the arguments 'domain' should not be assigned when return false Sep 30, 2015

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