)]}'
{"id":"openstack%2Fdevstack-plugin-ceph~722427","triplet_id":"openstack%2Fdevstack-plugin-ceph~master~I2d4f23721107c4b5cb1c09719c55424eecd0c880","project":"openstack/devstack-plugin-ceph","branch":"master","topic":"bug/1872855","hashtags":[],"change_id":"I2d4f23721107c4b5cb1c09719c55424eecd0c880","subject":"Check that ceph-mon service has started to get its version","status":"MERGED","created":"2020-04-23 18:57:31.000000000","updated":"2020-05-04 19:21:54.000000000","submitted":"2020-05-04 19:18:24.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"722427-1588619904200-ae45bbcb","meta_rev_id":"4f49e7472fa4b02878788e7e03e920d01baa3102","_number":722427,"virtual_id_number":722427,"owner":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-05-04 19:18:23.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"all":[{"value":2,"date":"2020-04-24 07:42:58.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":2,"date":"2020-05-04 19:05:41.000000000","_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"all":[{"value":0,"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":1,"date":"2020-05-04 19:05:41.000000000","_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-04-24 07:42:58.000000000","updated_by":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"reviewer":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"state":"REVIEWER"},{"updated":"2020-05-04 19:05:41.000000000","updated_by":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"reviewer":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"state":"REVIEWER"},{"updated":"2020-05-04 19:18:23.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"c067871520a03144d164877ee2daf88e89d0bda2","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2020-04-23 18:57:31.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"996f993278ecf64fc4c83f0eb130b4f1fb92de00","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-23 19:11:57.000000000","message":"Patch Set 1: Verified-1\n\n(2 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/1dcc18b601644164abe6c60655bd94af : FAILURE in 7m 17s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/e208b172618c40d880a8aa4e5bf179e3 : FAILURE in 3m 37s\n- manila-tempest-minimal-dsvm-cephfs-native https://zuul.opendev.org/t/openstack/build/6d65cba348244cae94a770197723ce55 : FAILURE in 9m 48s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/00d4317660c34ab1aaa02a98c7388fe2 : POST_FAILURE in 9m 11s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"d9c814aa213386bdca8ce13875312a43334ce53d","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2020-04-23 19:51:41.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"56899bd50892daa8cef7d4a56a1b4d31113695b6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-23 20:16:12.000000000","message":"Patch Set 2: Verified-1\n\n(1 comment)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/b91d6b12edb346e8b25ccc2cf7551a63 : FAILURE in 10m 42s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/530aeb32e88743dca02c84bd758ad37e : FAILURE in 4m 14s\n- manila-tempest-minimal-dsvm-cephfs-native https://zuul.opendev.org/t/openstack/build/0dfeb4a899aa49f8bede0b69133a790e : FAILURE in 15m 17s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/1b32fcf5146f4dca808d649279dec9f6 : POST_FAILURE in 15m 01s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"f5bd127bcd65335e483ec1e181567897d8d4917d","author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"date":"2020-04-23 23:08:02.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"21bae39c6a13328389a1aada8ed18561a6a039b7","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-24 00:30:55.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- devstack-plugin-ceph-tempest-py3 https://zuul.opendev.org/t/openstack/build/891e64d88393442bbfbaf70358e961e9 : SUCCESS in 1h 00m 01s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/865c08d17fc24e83b580a61efafc4ce8 : SUCCESS in 3m 19s\n- manila-tempest-minimal-dsvm-cephfs-native https://zuul.opendev.org/t/openstack/build/b1dae852cecc45019db77cbfea40d955 : SUCCESS in 25m 43s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs https://zuul.opendev.org/t/openstack/build/d47738f3875343e4a2142f572b4ce17c : FAILURE in 1h 17m 20s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"82e71e0231927981562ae66abb716b6a4a868e0a","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2020-04-24 07:42:58.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"b255854757c38ba6b8eec7a399b1d8eaf30564b9","author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"date":"2020-05-04 19:05:41.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1\n\ncephfs-nfs failure is unrelated, in a shrink test way after devstack setup","accounts_in_message":[],"_revision_number":3},{"id":"9478949b8aa1e88b34099f69f67b8a44e6b246cc","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-04 19:09:52.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"8478cca1b863aa772580729f800ba4d0908128c4","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-04 19:18:23.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6572189d50784ccb964a38c9c394dd9b : SUCCESS in 3m 33s\n- openstack-tox-bashate https://zuul.opendev.org/t/openstack/build/2903dad35c324d6c8fa7cb3411342a9b : SUCCESS in 3m 07s","accounts_in_message":[],"_revision_number":3},{"id":"e22a43f343c94ece3482864bff8a11c6f29b4de5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-04 19:18:24.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"745056b4bd8f5176ed53f279ed7547363f910da2","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-05-04 19:21:54.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c8f8e7e8cd134bd7acf1f6abd89e4285 : SUCCESS in 1m 41s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"a97b1449cde413277b896854104645786d290a89","revisions":{"bc1f11bbca333e862c30f242dfdeec2daf210cd9":{"kind":"REWORK","_number":1,"created":"2020-04-23 18:57:31.000000000","uploader":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"ref":"refs/changes/27/722427/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/27/722427/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/1"}}},"commit":{"parents":[{"commit":"e6591702d84f1c85c19a1dbeb482fad342b32a6f","subject":"Removes upstart support","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/e6591702d84f1c85c19a1dbeb482fad342b32a6f"}]}],"author":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 18:52:55.000000000","tz":0},"committer":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 18:52:55.000000000","tz":0},"subject":"Check that ceph-mon service has started to get its version","message":"Check that ceph-mon service has started to get its version\n\nAdd an extra check to verify that ceph-mon service has started\nbefore trying to retrieve its version.\n\nAn already existing function is being used, wait_for_daemon\n\nSince we now guarantee that we are using systemctl, we can use\nthe is-active feature. If the service is running, the exit code\nwill be 0.\n\nAlso, change the command used to get the mon version. We can use\nthe ceph version command provided by the new cli instead of using\nceph daemon command which requires more privileges.\n\nChange-Id: I2d4f23721107c4b5cb1c09719c55424eecd0c880\nCloses-Bug: #1872855\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/bc1f11bbca333e862c30f242dfdeec2daf210cd9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/bc1f11bbca333e862c30f242dfdeec2daf210cd9"}]},"branch":"refs/heads/master"},"ec79c76e5f06a096996a9b033ce58c6ce0233b36":{"kind":"REWORK","_number":2,"created":"2020-04-23 19:51:41.000000000","uploader":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"ref":"refs/changes/27/722427/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/27/722427/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/2"}}},"commit":{"parents":[{"commit":"e6591702d84f1c85c19a1dbeb482fad342b32a6f","subject":"Removes upstart support","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/e6591702d84f1c85c19a1dbeb482fad342b32a6f"}]}],"author":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 18:52:55.000000000","tz":0},"committer":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 19:51:30.000000000","tz":0},"subject":"Check that ceph-mon service has started to get its version","message":"Check that ceph-mon service has started to get its version\n\nAdd an extra check to verify that ceph-mon service has started\nbefore trying to retrieve its version.\n\nAn already existing function is being used, wait_for_daemon\n\nSince we now guarantee that we are using systemctl, we can use\nthe is-active feature. If the service is running, the exit code\nwill be 0.\n\nAlso, change the command used to get the mon version. We can use\nthe ceph version command provided by the new cli instead of using\nceph daemon command which requires more privileges.\n\nChange-Id: I2d4f23721107c4b5cb1c09719c55424eecd0c880\nCloses-Bug: #1872855\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ec79c76e5f06a096996a9b033ce58c6ce0233b36"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ec79c76e5f06a096996a9b033ce58c6ce0233b36"}]},"branch":"refs/heads/master"},"a97b1449cde413277b896854104645786d290a89":{"kind":"REWORK","_number":3,"created":"2020-04-23 23:08:02.000000000","uploader":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"ref":"refs/changes/27/722427/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/27/722427/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/27/722427/3"}}},"commit":{"parents":[{"commit":"e6591702d84f1c85c19a1dbeb482fad342b32a6f","subject":"Removes upstart support","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/e6591702d84f1c85c19a1dbeb482fad342b32a6f"}]}],"author":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 18:52:55.000000000","tz":0},"committer":{"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","date":"2020-04-23 23:07:51.000000000","tz":0},"subject":"Check that ceph-mon service has started to get its version","message":"Check that ceph-mon service has started to get its version\n\nAdd an extra check to verify that ceph-mon service has started\nbefore trying to retrieve its version.\n\nAn already existing function is being used, wait_for_daemon\n\nSince we now guarantee that we are using systemctl, we can use\nthe is-active feature. If the service is running, the exit code\nwill be 0.\n\nAlso, change the command used to get the mon version. We can use\nthe ceph version command provided by the new cli instead of using\nceph daemon command which requires more privileges.\n\nChange-Id: I2d4f23721107c4b5cb1c09719c55424eecd0c880\nCloses-Bug: #1872855\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/a97b1449cde413277b896854104645786d290a89"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/a97b1449cde413277b896854104645786d290a89"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}