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.

Sign up for GitHub

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

Jump to bottom

kraken: rgw: Realm set does not create a new period #14509

Merged
merged 3 commits into from Jul 7, 2017
Merged

kraken: rgw: Realm set does not create a new period #14509

merged 3 commits into from Jul 7, 2017

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm self-assigned this Apr 13, 2017
@smithfarm smithfarm added this to the kraken milestone Apr 13, 2017
@smithfarm
Copy link
Contributor Author 

/home/jenkins-build/build/workspace/ceph-pull-requests/qa/workunits/cephtool/test.sh:1801: test_mon_crushmap_validation:  expect_false ceph osd setcrushmap -i /tmp/cephtool.Bey/map
/home/jenkins-build/build/workspace/ceph-pull-requests/qa/workunits/cephtool/test.sh:31: expect_false:  set -x
/home/jenkins-build/build/workspace/ceph-pull-requests/qa/workunits/cephtool/test.sh:32: expect_false:  ceph osd setcrushmap -i /tmp/cephtool.Bey/map
2017-04-13 18:39:04.390789 7f83311ef700 -1 WARNING: the following dangerous and experimental features are enabled: *
2017-04-13 18:39:04.398400 7f83311ef700 -1 WARNING: the following dangerous and experimental features are enabled: *
Error EINVAL: Failed crushmap test: /tmp/cephtool.Bey/crushtool: exit status: 1

@smithfarm smithfarm changed the title kraken: rgw: Realm set does not create a new period [DNM] kraken: rgw: Realm set does not create a new period Apr 15, 2017
@smithfarm
Copy link
Contributor Author

Jenkins re-test this please

@tchaikov
Copy link
Contributor

@smithfarm the crushtool timeout recurs occasionally. it's a known issue. but we haven't managed to root cause it yet.

@smithfarm smithfarm changed the title [DNM] kraken: rgw: Realm set does not create a new period kraken: rgw: Realm set does not create a new period Apr 26, 2017
@oritwas @smithfarm
radosgw-admin: realm set should create a new realm
68d2d4b 
Fixes: http://tracker.ceph.com/issues/18333
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit e23339c)
@oritwas @smithfarm
radosgw-admin: relam set can use input redirection
ff888f8 
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit b8b3ae3)
@oritwas @smithfarm
radosgw-admin: check for name mistmatch in realm set
87811ff 
Signed-off-by: Orit Wasserman <owasserm@redhat.com>
(cherry picked from commit 4facc5f)
@smithfarm
Copy link
Contributor Author

Jenkins re-test this please

@smithfarm smithfarm requested a review from yehudasa July 6, 2017 06:43
@smithfarm
Copy link
Contributor Author

@yehudasa @mattbenjamin @oritwas @cbodley This PR passed an rgw suite at http://tracker.ceph.com/issues/19009#note-60 and, provided there are no objections, I'll merge it as soon as it gets a positive review from one of you.

@smithfarm smithfarm merged commit 6353508 into ceph:kraken Jul 7, 2017
@smithfarm smithfarm deleted the wip-18499-kraken branch July 7, 2017 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oritwas oritwas oritwas approved these changes

@yehudasa yehudasa Awaiting requested review from yehudasa

Assignees

@smithfarm smithfarm

Labels
bug-fix rgw
Projects
None yet
Milestone
kraken
3 participants
@smithfarm @tchaikov @oritwas