)]}'
{"id":"openstack%2Fdevstack-plugin-ceph~332484","triplet_id":"openstack%2Fdevstack-plugin-ceph~master~I25d8816740c6008b076986f3677c299ad9843c30","project":"openstack/devstack-plugin-ceph","branch":"master","topic":"Work","hashtags":[],"change_id":"I25d8816740c6008b076986f3677c299ad9843c30","subject":"Support systemd and ceph versions \u003e\u003d 9.2","status":"MERGED","created":"2016-06-21 21:31:59.000000000","updated":"2016-07-05 22:13:02.000000000","submitted":"2016-07-05 22:13:02.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":36,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2808bfe6b946f3e02a977a998ce396d5a0208b09","_number":332484,"virtual_id_number":332484,"owner":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2016-07-05 22:13:02.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"},{"value":0,"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"value":0,"_account_id":8868,"name":"tellesnobrega","inactive":true},{"value":0,"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},{"value":0,"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},{"value":0,"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},{"value":0,"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"recommended":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"},{"value":0,"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"value":0,"_account_id":8868,"name":"tellesnobrega","inactive":true},{"value":0,"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},{"value":1,"date":"2016-07-04 04:18:44.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},{"value":2,"date":"2016-07-05 12:38:32.000000000","_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},{"value":0,"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":1,"date":"2016-07-04 21:08:13.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"},{"value":0,"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"value":0,"_account_id":8868,"name":"tellesnobrega","inactive":true},{"value":0,"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},{"value":0,"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},{"value":1,"date":"2016-07-05 12:38:32.000000000","_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},{"value":0,"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},{"_account_id":8868,"name":"tellesnobrega","inactive":true},{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},{"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-21 21:34:39.000000000","updated_by":{"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"},"reviewer":{"_account_id":18643,"name":"John Spray","email":"jspray@redhat.com","username":"jcsp"},"state":"REVIEWER"},{"updated":"2016-06-21 21:35:13.000000000","updated_by":{"_account_id":8868,"name":"tellesnobrega","inactive":true},"reviewer":{"_account_id":8868,"name":"tellesnobrega","inactive":true},"state":"REVIEWER"},{"updated":"2016-06-21 21:35:27.000000000","updated_by":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"reviewer":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"state":"REVIEWER"},{"updated":"2016-06-22 04:14:07.000000000","updated_by":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"reviewer":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"state":"REVIEWER"},{"updated":"2016-07-04 04:18:44.000000000","updated_by":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"reviewer":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"state":"REVIEWER"},{"updated":"2016-07-04 21:08:13.000000000","updated_by":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"reviewer":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"state":"REVIEWER"},{"updated":"2016-07-05 12:38:32.000000000","updated_by":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"reviewer":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"state":"REVIEWER"},{"updated":"2016-07-05 22:13:02.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"2c0deb933e0847f81ac73da0c69ad6c4e0900d97","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-21 21:31:59.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"d4d583546cdd62dd463d39ad5b067427f9c522f1","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-21 22:01:00.000000000","message":"Patch Set 2: Published edit on patch set 1","accounts_in_message":[],"_revision_number":2},{"id":"49acae8fdfa97a805073f255ba037ad9a8a0f42d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-21 22:58:12.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/2/check/gate-devstack-plugin-ceph-bashate/9bd9fc6/ : FAILURE in 3m 16s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/2/check/gate-tempest-dsvm-full-devstack-plugin-ceph/cb80d9a/ : SUCCESS in 56m 44s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/2/check/gate-manila-tempest-dsvm-cephfs-native/0c9e11b/ : SUCCESS in 42m 20s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f42927c4bae0b6c31009ea564ba8c97317c917fb","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-21 23:29:08.000000000","message":"Patch Set 2: Workflow-1\n\nI welcome reviews but know that this needs more work before it should merge.","accounts_in_message":[],"_revision_number":2},{"id":"d248e0245cf384ad034bf64c33e72c717783e948","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 02:11:28.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"4550928d793b0fe4ba98b5a826cc412d972023ab","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 02:14:15.000000000","message":"Patch Set 3: Workflow-1\n\njust removed some tabs.","accounts_in_message":[],"_revision_number":3},{"id":"c4bcfd3300834a4b425739f6f89bf7e89c32706a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-22 03:04:29.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/3/check/gate-devstack-plugin-ceph-bashate/5106e39/ : SUCCESS in 2m 45s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph/ab3f3b8/ : SUCCESS in 46m 18s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/3/check/gate-manila-tempest-dsvm-cephfs-native/29991d3/ : SUCCESS in 23m 20s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"2b1bcb1ae2cd625efa02509612154757a5f14286","author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"date":"2016-06-22 04:14:07.000000000","message":"Patch Set 3:\n\nThe chown\u0027s are required because the daemons run as the \u0027ceph\u0027 user on redhat and root on ubuntu?\n\nSince it doesn\u0027t effect the gate voting, I support this change even if you have additional changes to make in the future.","accounts_in_message":[],"_revision_number":3},{"id":"44c842d3122496d4d716f3411255be3e638d6182","author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"date":"2016-06-22 09:54:04.000000000","message":"Patch Set 3: Code-Review-1\n\nThanks for this patch Tom, why don\u0027t we simply re-use some of the logic we have to detect which Ceph version got installed?\nI think it\u0027s easier so we don\u0027t restrict to a certain OS.\nBoth Ubuntu 16.04 and Fedora 24 have the version of Ceph.\n\nI think we should detect which version of Ceph got installed and then do the proper chown etc.","accounts_in_message":[],"_revision_number":3},{"id":"1eba08ad30a9e32d4bfec0b349e042e5f9a5adc6","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 11:06:48.000000000","message":"Patch Set 3:\n\n@Jon: \"The chown\u0027s are required because the daemons run as the \u0027ceph\u0027 user on redhat and root on ubuntu?\"\n\nI think it\u0027s the ceph version (post-hammer) rather than the platform that determines this.  There was already a one check for post-hammer\ninside an is_ubuntu branch inside configure_ceph() where ther is a\n\u0027chown -R ceph. ${CEPH_DATA_DIR}\u0027.\n\n@Sebastien: \"why don\u0027t we simply re-use some of the logic we have to detect which Ceph version got installed? I think it\u0027s easier so we don\u0027t restrict to a certain OS.\"\n\nI\u0027ll submit a new patch that attempts to do this.  Please let me know if it looks correct or not.","accounts_in_message":[],"_revision_number":3},{"id":"156dbf4a20c0dae1c7ed23d600d2d5cfb959793b","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 11:25:57.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"e13073dcd621fa51c12e0942714c134a7a0f0e2a","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 11:26:43.000000000","message":"Patch Set 4: Workflow-1\n\nI haven\u0027t tested this yet but want to see if the version check looks sane.","accounts_in_message":[],"_revision_number":4},{"id":"6920626ac84a23f29f6ba869926e4062e1a6c1d2","author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"date":"2016-06-22 11:41:32.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"6a383622f6e4d068b307da93e7ca66da4faa7d43","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-22 12:13:05.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/4/check/gate-devstack-plugin-ceph-bashate/c85f23e/ : FAILURE in 2m 18s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/4/check/gate-tempest-dsvm-full-devstack-plugin-ceph/9024423/ : SUCCESS in 39m 57s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/4/check/gate-manila-tempest-dsvm-cephfs-native/4eb14d2/ : SUCCESS in 31m 36s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"30b2f47d47c39a139b80c46de68982cfcd4c3913","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 15:41:37.000000000","message":"Patch Set 4:\n\n@Sebastien: \"You can use: grep -qs systemd /proc/1/comm.\"\n\nYeah, that makes sense.\n\nLet me check though: we will still do (a) the post-hammer version check to determine whether we need to do chowns and --setuser/--setgroup, and then do (b) the systemd check to determine whether we do systemctl style commands or not.\n\nIf that is correct, then it leads me to ask ...\n\nFor this patch, should we scope the systemd checks within non-ubuntu branches, so that we are checking systemd vs sysvinit?  This would imply that we need a followup patch later for post-trusty ubuntu but may make sense in order to limit the scope of change in a single patch.\n\nOr would you prefer that we go on and deal with the systemd vs upstart dimension in this patch as well?  In that case we would refactor to get rid of the ubuntu/vs non-ubuntu branches and just check:\n\n   - post-hammer or not\n   - systemd or upstart or sysvinit\n\nI\u0027m happy to take either approach and look forward to your guidance :)","accounts_in_message":[],"_revision_number":4},{"id":"efa15701d8e4081c144a19b2272f3f27b357475b","author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"date":"2016-06-22 16:20:58.000000000","message":"Patch Set 4:\n\nAlthough it would make sense to separate commits and I have one that refactors init system check and another one that brings the support for post-hammer releases this is entirely up to you.\nThat\u0027s one thing I don\u0027t like with gerrit is that it\u0027s not really practical to send multiple commits...\nAnyway, I\u0027m fine having both on this commit :)","accounts_in_message":[],"_revision_number":4},{"id":"42004fa27bf34e9476dbb9b7d66f2c453f34b5e1","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-22 17:28:00.000000000","message":"Patch Set 4: Code-Review-1\n\n(1 comment)\n\nHi Tom, from your commit message it seems like you were (subconsciously?) already targeting a OS agnostic solution. It\u0027d be great if you can pull this off, check if post-hammer or not, and then, systemd/upstart/sysvinit. \n\nLet\u0027s also bear in mind that for now,\n\u0027gate-manila-tempest-dsvm-cephfs-native\u0027 CI tests \u0027Ubuntu Trusty + Ceph Infernalis\u0027, and \u0027gate-tempest-dsvm-full-devstack-plugin-ceph\u0027 CI tests \u0027Ubuntu Trusty + Ceph Hammer\u0027(I could be wrong about this one.). So do you think the current CIs would sufficiently test your code? If not we\u0027d have to at least make a note of it in the README.","accounts_in_message":[],"_revision_number":4},{"id":"1994c4dc7e88e666577154eee641f06066341320","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 22:42:42.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"4c008c4807c16dd9349d7d6c24ac7300a8bfff61","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-22 22:45:10.000000000","message":"Patch Set 5: Workflow-1\n\nStill WIP, but I want to get the setting of INIT_VERSION up there\nchecked, and tested.\n\nIf it checks out then we\u0027ll need to check the code in detail to see\nthat we do the right thing for (a) post-hammer ceph, and (b) upstart\nvs systemd vs sysvinit.","accounts_in_message":[],"_revision_number":5},{"id":"40b01ad59f4b031621358e4575a01b548af12967","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-22 23:26:36.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/5/check/gate-devstack-plugin-ceph-bashate/f04469f/ : SUCCESS in 2m 11s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/5/check/gate-tempest-dsvm-full-devstack-plugin-ceph/7715028/ : SUCCESS in 43m 42s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/5/check/gate-manila-tempest-dsvm-cephfs-native/68d3ace/ : SUCCESS in 38m 53s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"1354350c7455d0f94311d8887de4cde87dcce688","author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"date":"2016-06-23 12:38:33.000000000","message":"Patch Set 5: Code-Review-1\n\n(9 comments)","accounts_in_message":[],"_revision_number":5},{"id":"0caa967a2b23a1b79f3dcdb136561ea655938f4c","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 16:46:12.000000000","message":"Patch Set 4:\n\n(3 comments)\n\nSome comments/questions about the ceph version test.","accounts_in_message":[],"_revision_number":4},{"id":"5442c0f7fcefcc13f04174795b6fa01a559bd981","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-23 17:04:49.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"024fa62eae68b1b5d4f466da5db91a4b2cab7f4e","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-23 18:19:06.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"eb5a2790e08128f1be176d1a9a40780fa31e36d7","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 18:24:07.000000000","message":"Patch Set 4:\n\n(1 comment)\n\nThanks Ramana.","accounts_in_message":[],"_revision_number":4},{"id":"b53bcab1b6b8fbbd7747e2a834f6b984e72e5327","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 21:24:39.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"a732a5f51cff1fbe52b2e7babb484b9d1a16e8a7","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 21:25:24.000000000","message":"Patch Set 5:\n\n(9 comments)\n\nThanks for the review, another iteration on the way.","accounts_in_message":[],"_revision_number":5},{"id":"e64c034b1b5c6edaa5fa99d9ae7bdd5be8e8385b","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 21:28:08.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"60f9491345ecfbc851b436aa835bafac2f303734","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-23 21:32:25.000000000","message":"Patch Set 7:\n\npatch set 6 makes changes in light of review comments; patch set 7\nis b/c i forgot to run tox locally before posting 6 and got some bashate errors.\n\nThis changeset builds a manila devstack fine with the stock repos that come with fedora 24 now.\n\nNeed to retest with candidate RH 2.0 ceph code, ubuntu trusty, ubuntu xenial, probably others i haven\u0027t thought of yet.  Code inspection may suggest other changes too of course, but since this enables our ongoing development my main concern at this point is that we have\nconfidence that it won\u0027t break upstream gate with trusty.","accounts_in_message":[],"_revision_number":7},{"id":"d6b8f91100c6a1c30f4002672febad9d6218a389","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-23 22:21:25.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/7/check/gate-devstack-plugin-ceph-bashate/76dd20a/ : SUCCESS in 2m 39s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/7/check/gate-tempest-dsvm-full-devstack-plugin-ceph/259eb64/ : SUCCESS in 51m 33s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/7/check/gate-manila-tempest-dsvm-cephfs-native/d777bf5/ : FAILURE in 6m 53s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"e74d6c00fba10203d5df7f77315cad1f4a2df803","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-24 10:43:24.000000000","message":"Patch Set 7: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"c0700f6f2ef68137081d6718f554b56cef5a3925","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-24 10:53:57.000000000","message":"Patch Set 7:\n\n(1 comment)\n\nThanks, Ramana.  I was just looking that the test run and also noticed the difference iin the versions of packages installed in ceph-common.  I will move the check on running as root or ceph as you suggest.","accounts_in_message":[],"_revision_number":7},{"id":"82e95ef0eaec18fbfd5a39a94b724d195c8fb061","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-24 11:00:48.000000000","message":"Patch Set 7:\n\n(1 comment)\n\nThanks, Tom! A couple of clarifications.","accounts_in_message":[],"_revision_number":7},{"id":"e190e1d15a5c87b2f08b6879f74f45669ea29294","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-24 11:38:23.000000000","message":"Patch Set 7:\n\n(2 comments)\n\nWill push up a new package that moves the RUN_AS check as Ramana suggests.","accounts_in_message":[],"_revision_number":7},{"id":"d49d92fc9dc2d769dd592436731302986604dbf8","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-24 12:28:39.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"945f23690f6397b73c3bc9aa8ca45a4307700d6c","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-24 12:30:03.000000000","message":"Patch Set 8:\n\nF24 stacks OK with patch set 8.  Let\u0027s see how trusty does now.","accounts_in_message":[],"_revision_number":8},{"id":"79ac257c25b31b426019558a86c64bfab75fcacc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-24 13:16:21.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/8/check/gate-devstack-plugin-ceph-bashate/55bfa02/ : SUCCESS in 2m 23s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/8/check/gate-tempest-dsvm-full-devstack-plugin-ceph/0298749/ : SUCCESS in 46m 46s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/8/check/gate-manila-tempest-dsvm-cephfs-native/5e23bb3/ : SUCCESS in 40m 36s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"b4011ccd3f9ac701821e3d4e0e9c6c74fd0bd577","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-24 13:31:41.000000000","message":"Patch Set 8: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"a0742f9fa3a718daf1a7add5e847424681559d39","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-27 16:06:33.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"11056e2a2ce35274e18a17a2ac76c3ffb933085e","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-27 16:10:03.000000000","message":"Patch Set 9:\n\nLet\u0027s see if jenkins still likes this patch.  Changes since the last\nget radosgw working too.  I\u0027ve tested locally and am able to stack/unstack and get relevant services including mon, osd, mds, and radosgw on trusty, xenial, and Fedora 24 using this local.conf:\n[[local|localrc]]\n\nADMIN_PASSWORD\u003d123456\nDATABASE_PASSWORD\u003d$ADMIN_PASSWORD\nRABBIT_PASSWORD\u003d$ADMIN_PASSWORD\nSERVICE_PASSWORD\u003d$ADMIN_PASSWORD\nSERVICE_TOKEN\u003da682f596-76f3-11e3-b3b2-e716f9080d50\n\nenable_service c_bak\ndisable_service tempest\n\nMANILA_SERVICE_IMAGE_ENABLED\u003dFalse\nENABLE_ISOLATED_METADATA\u003dTrue\n\nMANILA_GIT_BASE\u003dhttps://git.openstack.org\nMANILA_REPO_ROOT\u003dopenstack\nMANILACLIENT_BRANCH\u003dmaster\nenable_plugin manila $MANILA_GIT_BASE/$MANILA_REPO_ROOT/manila $MANILA_CLIENT_BRANCH\n#### Run with this version of the plugin.\n#enable_plugin devstack-plugin-ceph https://github.com/openstack/devstack-plugin-ceph\nenable_plugin devstack-plugin-ceph file:///opt/stack/devstack-plugin-ceph\nENABLE_CEPH_MANILA\u003dTrue\nENABLE_CEPH_NOVA\u003dTrue\nENABLE_CEPH_GLANCE\u003dTrue\nENABLE_CEPH_CINDER\u003dTrue\nENABLE_CEPH_C_BAK\u003dTrue\nENABLE_CEPH_RGW\u003dTrue\n\nLOGFILE\u003d/opt/stack/logs/devstacklog.txt","accounts_in_message":[],"_revision_number":9},{"id":"aae2d4cafbd0048bb0df423adcee89b5bffa0f2c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-27 17:53:20.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/9/check/gate-devstack-plugin-ceph-bashate/f7f57e3/ : SUCCESS in 3m 56s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/9/check/gate-tempest-dsvm-full-devstack-plugin-ceph/35d90ca/ : SUCCESS in 49m 10s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/9/check/gate-manila-tempest-dsvm-cephfs-native/e8a23e9/ : SUCCESS in 34m 48s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"f2eef6eb31c5fd8e2da26d93a84e9fb1f9639fa6","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-30 17:32:47.000000000","message":"Patch Set 9:\n\ncheck experimental\n\npabelanger says this will cause a run against xenial","accounts_in_message":[],"_revision_number":9},{"id":"5aedfe851d77652aa46effeecd63a8276c060396","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-06-30 19:17:37.000000000","message":"Patch Set 9: Code-Review+1\n\n(1 comment)\n\nThanks a lot, Tom!","accounts_in_message":[],"_revision_number":9},{"id":"21b6746653150027ae4a46c14a4d9e8a7346a216","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2016-06-30 20:50:11.000000000","message":"Patch Set 9:\n\ncheck experimental","accounts_in_message":[],"_revision_number":9},{"id":"bf83dde81f783b2a29e83034c1793189844b138b","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2016-06-30 21:01:58.000000000","message":"Patch Set 9: Code-Review-1\n\n(1 comment)\n\nLooks good Tom! Spotted a typo, commented inline. Also, why ceph. and not ceph on the chown? Thanks!","accounts_in_message":[],"_revision_number":9},{"id":"8c78b3dcea579ee58af6caa541219a137e20dd59","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-30 23:38:07.000000000","message":"Patch Set 9:\n\n(2 comments)\n\nI\u0027m going to push a new patch that corrects the spelling error that vkmc found and then try a \u0027run experimental\u0027 since \u0027check experimental\u0027 doesn\u0027t seem to be doing anything.","accounts_in_message":[],"_revision_number":9},{"id":"d450cb27990f6cb521bd357f69df1badeb411977","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-30 23:38:20.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"5a3ff2bc834c2f7447a6224fd967fcde3f96a76c","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-06-30 23:38:45.000000000","message":"Patch Set 10:\n\nrun experimental","accounts_in_message":[],"_revision_number":10},{"id":"0e0511f33c70d789c5f4dc6d2d3959810203cb09","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-01 00:27:42.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/10/check/gate-devstack-plugin-ceph-bashate/6f6e5d6/ : SUCCESS in 2m 33s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/10/check/gate-tempest-dsvm-full-devstack-plugin-ceph/793f73d/ : SUCCESS in 43m 22s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/10/check/gate-manila-tempest-dsvm-cephfs-native/ca61bf3/ : SUCCESS in 37m 42s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"805ba7b54421caa590ad516a60d0096d20e1e2f6","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-07-01 02:25:02.000000000","message":"Patch Set 10:\n\n@vkmc: \"Also, why ceph. and not ceph on the chown?\"\n\n\u0027chown ceph.ceph\u0027, \u0027chown ceph:ceph\u0027, and \u0027chown ceph.\u0027 all do the same thing, namely change owner and group to ceph.  The last form has a certan je ne sais quois and conforms with the way it was already done before I started making changes.","accounts_in_message":[],"_revision_number":10},{"id":"ed14d8edef4a2eb4b81d2594a92f22eebf16a4c6","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-07-01 08:19:29.000000000","message":"Patch Set 10: Code-Review+1\n\nThanks, Tom!","accounts_in_message":[],"_revision_number":10},{"id":"a1784aefe248191bdc00b5cb894ef3cbfebc9db4","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-07-01 18:26:51.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"b7270815cdbd56bae4cf24ba797940287a038986","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-01 19:20:18.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/check/gate-devstack-plugin-ceph-bashate/3ed9510/ : SUCCESS in 2m 31s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/check/gate-tempest-dsvm-full-devstack-plugin-ceph/04a3b9e/ : SUCCESS in 49m 02s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/11/check/gate-manila-tempest-dsvm-cephfs-native/d84d8b7/ : SUCCESS in 37m 11s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"f51dc02af66f9c75bb4c925e754ac361c3e9e17b","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-07-04 04:12:28.000000000","message":"Patch Set 11:\n\nTom wrote:\n\"\"\"\nRamana,\n\nI have a question about this point in the README: \"Rados Gateway with Keystone for Swift - works on Ubuntu only.\"\n\nDo you know what the issue is here?\n\nWith https://review.openstack.org/#/c/332484/ changes applied and ENABLE_CEPH_RGW\u003dTrue in local.conf, radosgw now runs on F24:\n\n[stack@devstack devstack]$ pgrep rados\n18947\n[stack@devstack devstack]$ ps 18947\n  PID TTY      STAT   TIME COMMAND\n18947 ?        Ssl    0:06 /usr/bin/radosgw -f --cluster ceph --name client.rgw.devstack --setuser ceph --setgroup ceph\n[stack@devstack devstack]$ \n\nI haven\u0027t done any functional testing of it though.\n\nOh, in /var/log/ceph/radosgw-devstack.log I see:\n\n2016-06-26 12:46:54.883314 7f01107d8700  0 revoked tokens response is missing signed section\n2016-06-26 12:46:54.883327 7f01107d8700  0 ERROR: keystone revocation processing returned error r\u003d-22\n\nbut that\u0027s in my trusty devstack too :(\n\nSo, is the main patch breaking trusty radosgw??  We seem to have parity :)\n\"\"\"\n\nSorry, I\u0027ve no idea about the status of RGW. This is something that Sebastien might know more about. I think it\u0027d be safe if we leave the RGW warnings in the README as is.","accounts_in_message":[],"_revision_number":11},{"id":"a8544a64a018eb44adf33e86aa7041598d316b30","author":{"_account_id":8056,"name":"Ramana Raja","email":"rraja@redhat.com","username":"Ram_Raja"},"date":"2016-07-04 04:18:44.000000000","message":"Patch Set 11: Code-Review+1\n\nTom wrote:\n\u003e I don\u0027t see any baseline runs for radosgw without the patch on trusty in gate, I guess because there swift is enabled and you can\u0027t have both.\n\nYes, from the logs looks like RGW is not tested by \"gate-tempest-dsvm-full-devstack-plugin-ceph\". Since RGW provides object interface to Ceph, I think it should\u0027ve to be enabled instead of swift.","accounts_in_message":[],"_revision_number":11},{"id":"8b08c99e3876a8ce612b3b71662c7a0b2f4220d5","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2016-07-04 21:08:13.000000000","message":"Patch Set 11: Code-Review+1\n\nThanks Tom! Code looks good and it works for me.","accounts_in_message":[],"_revision_number":11},{"id":"e281a193f7ef27050b859544dcd4a4c234660316","author":{"_account_id":6984,"name":"Sébastien Han","email":"seb@redhat.com","username":"leseb"},"date":"2016-07-05 12:38:32.000000000","message":"Patch Set 11: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":11},{"id":"d1329b56c335df664a69b7f85b326e53d88b4027","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 12:38:43.000000000","message":"Patch Set 11: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":11},{"id":"4b696f2c86b11311ff10df8911a739ca1664c339","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 13:54:52.000000000","message":"Patch Set 11: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/gate/gate-devstack-plugin-ceph-bashate/f58d9bf/ : FAILURE in 34m 46s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/gate/gate-tempest-dsvm-full-devstack-plugin-ceph/bdd59ae/ : SUCCESS in 1h 05m 08s","accounts_in_message":[],"_revision_number":11},{"id":"38f4e78204511504ddae0b5973c27e8c629d071b","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-07-05 14:31:19.000000000","message":"Patch Set 11:\n\nreverify","accounts_in_message":[],"_revision_number":11},{"id":"0caee1c56a6f3aa3fce4cc48f1b3f71c2d9354cd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 15:20:58.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/check/gate-devstack-plugin-ceph-bashate/6921598/ : SUCCESS in 3m 07s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/check/gate-tempest-dsvm-full-devstack-plugin-ceph/cefc949/ : SUCCESS in 38m 07s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/11/check/gate-manila-tempest-dsvm-cephfs-native/ca6dae2/ : SUCCESS in 26m 49s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"27564d1af0547218f9f556a2845437dff16c818f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 15:21:08.000000000","message":"Patch Set 11: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":11},{"id":"7c9beaa2f2543ab86308f330622c42a5cc7a777d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 16:31:44.000000000","message":"Patch Set 11: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/gate/gate-devstack-plugin-ceph-bashate/f84e3b3/ : SUCCESS in 1m 45s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/gate/gate-tempest-dsvm-full-devstack-plugin-ceph/5bbd99e/ : FAILURE in 50m 05s","accounts_in_message":[],"_revision_number":11},{"id":"5c76db291c95805bdc58c04caec5b273eeb5b5a0","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2016-07-05 16:32:21.000000000","message":"Patch Set 11:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- gate-tempest-dsvm-full-devstack-plugin-ceph: https://bugs.launchpad.net/bugs/1497006 https://bugs.launchpad.net/bugs/1449136\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":11},{"id":"bb6806be65fcd939ed1f949371d15cb8bdcaf6c0","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2016-07-05 16:49:50.000000000","message":"Patch Set 11:\n\nreverify","accounts_in_message":[],"_revision_number":11},{"id":"83f4691a6b4ff4fb49adc68aa7af30c2f9b8be61","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 18:01:06.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/check/gate-devstack-plugin-ceph-bashate/76702bd/ : SUCCESS in 3m 45s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/check/gate-tempest-dsvm-full-devstack-plugin-ceph/917b593/ : SUCCESS in 54m 57s\n- gate-manila-tempest-dsvm-cephfs-native http://logs.openstack.org/84/332484/11/check/gate-manila-tempest-dsvm-cephfs-native/b8d18c1/ : SUCCESS in 23m 29s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"889ac1eb3960a760ea681e4cc2c03e782a7d6ee0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 18:01:16.000000000","message":"Patch Set 11: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":11},{"id":"05027df7e312984cda594d5aeea305c2ce4e5003","date":"2016-07-05 22:13:02.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":11},{"id":"18cca65eb6112139da0c554aa9718a1b86e990f7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-07-05 22:13:02.000000000","message":"Patch Set 11: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-devstack-plugin-ceph-bashate http://logs.openstack.org/84/332484/11/gate/gate-devstack-plugin-ceph-bashate/4e0705e/ : SUCCESS in 1m 56s\n- gate-tempest-dsvm-full-devstack-plugin-ceph http://logs.openstack.org/84/332484/11/gate/gate-tempest-dsvm-full-devstack-plugin-ceph/17b414b/ : SUCCESS in 48m 33s","accounts_in_message":[],"_revision_number":11}],"current_revision_number":11,"current_revision":"19051aa4cd779d9f6265a1a0a9fd00617ac72dab","revisions":{"e35a2184984b3f0d9efd6ca02046e5c8b8690141":{"kind":"REWORK","_number":1,"created":"2016-06-21 21:31:59.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/1"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:31:26.000000000","tz":-240},"subject":"Support RedHat Ceph2.0 in devstack","message":"Support RedHat Ceph2.0 in devstack\n\nCurrently non-ubuntu releases use hammer and use Ceph\ninstallation, configuration, and startup commands that\ndo not work correctly with later releases.\n\nUse more appropriate Ceph installation, configuration and\nand startup (systemd rather than sysvinit) commands if\n$DISTRO \u003d\u003d \u0027rhel7\u0027 \u0026\u0026 $CEPH_RELEASE \u003d\u003d \u0027jewel\u0027.\n\n---\nNote that for this to work, since Ceph 2.0 has not yet\nshipped, appropriate repositories must be set up in\n/etc/yum.repos.d and FORCE_CEPH_INSTALL\u003dyes must be set\nin devstack local.conf.  Also CEPH_RELEASE\u003djewel must be\nset in local.conf.\n\nIt seems likely that the test for RedHat distro\nand Ceph release are too specific as we\u0027ll want to cover\nCentos and Fedora and Xenial, on the one hand, and Infernalis\nand Kraken and after, on the other.  And I doubt the systemd\ncommands used here are optimal.\n\nThe point of this review is to get code into gerrit so that\nwe can refer to and download it as we do development work in\nmanila and cinder with jewel codebase, as well as to get the ball\nrolling on devstack support for more than ubuntu trusty.  No\nneed for a fast-track merge - let\u0027s use this review to figure\nout how to do the right thing and settle on an appropriate\nfix for when Red Hat Ceph 2.0 ships.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/e35a2184984b3f0d9efd6ca02046e5c8b8690141"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/e35a2184984b3f0d9efd6ca02046e5c8b8690141"}]},"branch":"refs/heads/master"},"904788c171cc0ba29bf2b9b5d074400694f61497":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2016-06-21 22:01:00.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/2"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 22:00:54.000000000","tz":0},"subject":"Support RedHat Ceph2.0 in devstack","message":"Support RedHat Ceph2.0 in devstack\n\nCurrently devstack uses whatever version of Ceph the distro ships for\nnon-ubuntu distros and uses installation, configuration, and startup\ncommands that do not work correctly for later releases.\n\nUse more appropriate Ceph installation, configuration and\nand startup (systemd rather than sysvinit) commands if\n$DISTRO \u003d\u003d \u0027rhel7\u0027 \u0026\u0026 $CEPH_RELEASE \u003d\u003d \u0027jewel\u0027.\n\n---\nNote that for this to work, since Ceph 2.0 has not yet\nshipped, appropriate repositories must be set up in\n/etc/yum.repos.d and FORCE_CEPH_INSTALL\u003dyes must be set\nin devstack local.conf.  Also CEPH_RELEASE\u003djewel must be\nset in local.conf.\n\nIt seems likely that the test for RedHat distro\nand Ceph release are too specific as we\u0027ll want to cover\nCentos and Fedora and Xenial, on the one hand, and Infernalis\nand Kraken and after, on the other.  And I doubt the systemd\ncommands used here are optimal.\n\nThe point of this review is to get code into gerrit so that\nwe can refer to and download it as we do development work in\nmanila and cinder with jewel codebase, as well as to get the ball\nrolling on devstack support for more than ubuntu trusty.  No\nneed for a fast-track merge - let\u0027s use this review to figure\nout how to do the right thing and settle on an appropriate\nfix for when Red Hat Ceph 2.0 ships.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/904788c171cc0ba29bf2b9b5d074400694f61497"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/904788c171cc0ba29bf2b9b5d074400694f61497"}]},"branch":"refs/heads/master"},"d645faca95d2be91e2275387c0a91e53f52ce89f":{"kind":"REWORK","_number":3,"created":"2016-06-22 02:11:28.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/3"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-22 02:11:06.000000000","tz":-240},"subject":"Support RedHat Ceph2.0 in devstack","message":"Support RedHat Ceph2.0 in devstack\n\nCurrently devstack uses whatever version of Ceph the distro ships for\nnon-ubuntu distros and uses installation, configuration, and startup\ncommands that do not work correctly for later releases.\n\nUse more appropriate Ceph installation, configuration and\nand startup (systemd rather than sysvinit) commands if\n$DISTRO \u003d\u003d \u0027rhel7\u0027 \u0026\u0026 $CEPH_RELEASE \u003d\u003d \u0027jewel\u0027.\n\n---\nNote that for this to work, since Ceph 2.0 has not yet\nshipped, appropriate repositories must be set up in\n/etc/yum.repos.d and FORCE_CEPH_INSTALL\u003dyes must be set\nin devstack local.conf.  Also CEPH_RELEASE\u003djewel must be\nset in local.conf.\n\nIt seems likely that the test for RedHat distro\nand Ceph release are too specific as we\u0027ll want to cover\nCentos and Fedora and Xenial, on the one hand, and Infernalis\nand Kraken and after, on the other.  And I doubt the systemd\ncommands used here are optimal.\n\nThe point of this review is to get code into gerrit so that\nwe can refer to and download it as we do development work in\nmanila and cinder with jewel codebase, as well as to get the ball\nrolling on devstack support for more than ubuntu trusty.  No\nneed for a fast-track merge - let\u0027s use this review to figure\nout how to do the right thing and settle on an appropriate\nfix for when Red Hat Ceph 2.0 ships.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/d645faca95d2be91e2275387c0a91e53f52ce89f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/d645faca95d2be91e2275387c0a91e53f52ce89f"}]},"branch":"refs/heads/master"},"bc7cfa9ecae33e72eb36adcc4b3f442176bff839":{"kind":"REWORK","_number":4,"created":"2016-06-22 11:25:57.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/4"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-22 11:24:04.000000000","tz":-240},"subject":"Support Ceph versions \u003e\u003d 9.4","message":"Support Ceph versions \u003e\u003d 9.4\n\nCurrently the devstack ceph plugin frequently uses installation,\nconfiguration, startup, and shutdown commands that do not work\ncorrectly for Ceph releases after hammer.\n\nCheck the ceph version and use appropriate commands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/bc7cfa9ecae33e72eb36adcc4b3f442176bff839"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/bc7cfa9ecae33e72eb36adcc4b3f442176bff839"}]},"branch":"refs/heads/master"},"5baba647e0cbef4e5b4e94f9b2b9987548893303":{"kind":"REWORK","_number":5,"created":"2016-06-22 22:42:42.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/5"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-22 22:42:08.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.4","message":"Support systemd and ceph versions \u003e\u003d 9.4\n\n** WIP ** - needs testing for one thing.\n\nCurrently the devstack ceph plugin frequently uses installation,\nconfiguration, startup, and shutdown commands that do not work\ncorrectly for Ceph releases after hammer or with systemd init\nsystem.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/5baba647e0cbef4e5b4e94f9b2b9987548893303"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/5baba647e0cbef4e5b4e94f9b2b9987548893303"}]},"branch":"refs/heads/master"},"259788db1ca6ce511ef7897f8fb2f86b520ce161":{"kind":"REWORK","_number":6,"created":"2016-06-23 21:24:39.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/6"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-23 21:23:49.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\n** WIP ** - needs testing for one thing.\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/259788db1ca6ce511ef7897f8fb2f86b520ce161"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/259788db1ca6ce511ef7897f8fb2f86b520ce161"}]},"branch":"refs/heads/master"},"23b0edff8548c04439b7fa7f298d2872f3647dbd":{"kind":"REWORK","_number":7,"created":"2016-06-23 21:28:08.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/7"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-23 21:28:01.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\n** WIP ** - needs testing for one thing.\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/23b0edff8548c04439b7fa7f298d2872f3647dbd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/23b0edff8548c04439b7fa7f298d2872f3647dbd"}]},"branch":"refs/heads/master"},"84987f9ddf3e5a2069f83e1c6ca851e6b5c80991":{"kind":"REWORK","_number":8,"created":"2016-06-24 12:28:39.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/8"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-24 12:25:25.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\n** WIP ** - needs testing for one thing.\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/84987f9ddf3e5a2069f83e1c6ca851e6b5c80991"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/84987f9ddf3e5a2069f83e1c6ca851e6b5c80991"}]},"branch":"refs/heads/master"},"a261561bc598b8126c5623fb81b1ba083e399cd1":{"kind":"REWORK","_number":9,"created":"2016-06-27 16:06:33.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/9"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-27 16:05:02.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/a261561bc598b8126c5623fb81b1ba083e399cd1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/a261561bc598b8126c5623fb81b1ba083e399cd1"}]},"branch":"refs/heads/master"},"1711f7b3a3af81e3723e29a23893b8f27bac7755":{"kind":"REWORK","_number":10,"created":"2016-06-30 23:38:20.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/10"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-30 23:29:01.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/1711f7b3a3af81e3723e29a23893b8f27bac7755"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/1711f7b3a3af81e3723e29a23893b8f27bac7755"}]},"branch":"refs/heads/master"},"19051aa4cd779d9f6265a1a0a9fd00617ac72dab":{"kind":"REWORK","_number":11,"created":"2016-07-01 18:26:51.000000000","uploader":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"ref":"refs/changes/84/332484/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/84/332484/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/84/332484/11"}}},"commit":{"parents":[{"commit":"28ae5aeae317eebd14398227074042f523b7ed6c","subject":"Enable multiple ceph filesystems for manila","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/28ae5aeae317eebd14398227074042f523b7ed6c"}]}],"author":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-06-21 21:28:47.000000000","tz":-240},"committer":{"name":"Tom Barron","email":"tpb@dyncloud.net","date":"2016-07-01 18:23:24.000000000","tz":-240},"subject":"Support systemd and ceph versions \u003e\u003d 9.2","message":"Support systemd and ceph versions \u003e\u003d 9.2\n\nCurrently the devstack ceph plugin uses installation,\nconfiguration, startup, and shutdown commands that do not always\nwork correctly for Ceph releases after hammer or with systemd\ninit system.\n\nCheck the ceph version and the init system and use appropriate\ncommands.\n\nCo-Authored-By: Ramana Raja \u003crraja@redhat.com\u003e\n\nChange-Id: I25d8816740c6008b076986f3677c299ad9843c30\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/19051aa4cd779d9f6265a1a0a9fd00617ac72dab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/19051aa4cd779d9f6265a1a0a9fd00617ac72dab"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}