)]}'
{"id":"openstack%2Fdevstack-plugin-ceph~668128","triplet_id":"openstack%2Fdevstack-plugin-ceph~master~I1aa87fc16a37b6051e55c35ae69ef911a87d4c11","project":"openstack/devstack-plugin-ceph","branch":"master","topic":"bug/1653953","hashtags":[],"change_id":"I1aa87fc16a37b6051e55c35ae69ef911a87d4c11","subject":"tempest: re-enable shelve tests for stable rocky","status":"MERGED","created":"2019-06-28 09:31:30.000000000","updated":"2019-06-30 17:06:24.000000000","submitted":"2019-06-30 17:06:24.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"668128-1561914384853-971b6051","meta_rev_id":"b2ea8df043060dc8056bbdad4fcf53aa946acbdc","_number":668128,"virtual_id_number":668128,"owner":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":2,"date":"2019-06-30 17:06:24.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}],"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":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"all":[{"value":2,"date":"2019-06-28 17:57:40.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2019-06-28 15:29:32.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":2,"date":"2019-06-30 16:58:46.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}],"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":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":0,"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"value":1,"date":"2019-06-30 16:58:46.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-28 15:29:32.000000000","updated_by":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"reviewer":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"state":"REVIEWER"},{"updated":"2019-06-28 17:57:40.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2019-06-30 16:58:46.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2019-06-30 17:06:24.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":"a6e567e9a38c6dc86733271be29b9b8db468eae0","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2019-06-28 09:31:30.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"787e2d83f95c8452eff54cc14bca2a554bf360aa","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-28 11:08:11.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- devstack-plugin-ceph-tempest http://logs.openstack.org/28/668128/1/check/devstack-plugin-ceph-tempest/5db2b8d/ : SUCCESS in 1h 20m 58s (non-voting)\n- devstack-plugin-ceph-tempest-py3 http://logs.openstack.org/28/668128/1/check/devstack-plugin-ceph-tempest-py3/5cd906c/ : SUCCESS in 1h 26m 30s (non-voting)\n- openstack-tox-bashate http://logs.openstack.org/28/668128/1/check/openstack-tox-bashate/56ce997/ : SUCCESS in 2m 32s\n- manila-tempest-minimal-dsvm-cephfs-native-centos-7 http://logs.openstack.org/28/668128/1/check/manila-tempest-minimal-dsvm-cephfs-native-centos-7/6c15ac8/ : SUCCESS in 33m 13s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs-centos-7 http://logs.openstack.org/28/668128/1/check/manila-tempest-minimal-dsvm-cephfs-nfs-centos-7/8c13283/ : SUCCESS in 33m 21s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-native http://logs.openstack.org/28/668128/1/check/manila-tempest-minimal-dsvm-cephfs-native/3e8ab5f/ : SUCCESS in 24m 48s (non-voting)\n- manila-tempest-minimal-dsvm-cephfs-nfs http://logs.openstack.org/28/668128/1/check/manila-tempest-minimal-dsvm-cephfs-nfs/26db2f0/ : SUCCESS in 1h 06m 11s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b995d79f33069dcda66090fcc8beb805e57690b9","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-28 15:28:24.000000000","message":"Patch Set 1: Code-Review+1\n\nLooks fine. Will wait for ack from Matt that the necessary fixes went in to nova.","accounts_in_message":[],"_revision_number":1},{"id":"6294e63ec65256180362319d9dbaf65dd8e28c00","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-06-28 15:29:32.000000000","message":"Patch Set 1: Code-Review+1\n\nThe change itself here looks good, depends on the nova stable/rocky backport and the test is passing:\n\nhttp://logs.openstack.org/28/668128/1/check/devstack-plugin-ceph-tempest-py3/5cd906c/job-output.txt.gz#_2019-06-28_10_40_46_087286\n\nWill approve the nova stable/rocky backport once the stable/stein one merges.","accounts_in_message":[],"_revision_number":1},{"id":"f9581459b5be6f6bdf603e9b3591ca920632770d","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2019-06-28 17:57:40.000000000","message":"Patch Set 1: Code-Review+2\n\n\u003e The change itself here looks good, depends on the nova stable/rocky\n \u003e backport and the test is passing:\n \u003e \n \u003e http://logs.openstack.org/28/668128/1/check/devstack-plugin-ceph-tempest-py3/5cd906c/job-output.txt.gz#_2019-06-28_10_40_46_087286\n \u003e \n \u003e Will approve the nova stable/rocky backport once the stable/stein\n \u003e one merges.\n\nthose test result was with nova master. You mean these - http://logs.openstack.org/41/668141/1/check/devstack-plugin-ceph-tempest/30f0b35/testr_results.html.gz\n\nlet\u0027s add stable jobs as experimental to tests the changes on stable within same patch.","accounts_in_message":[],"_revision_number":1},{"id":"9b59eb1479b43fad398ae81215bfff76e7ae5e71","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2019-06-28 18:08:35.000000000","message":"Patch Set 1:\n\n\u003e \u003e The change itself here looks good, depends on the nova\n \u003e stable/rocky\n \u003e \u003e backport and the test is passing:\n \u003e \u003e\n \u003e \u003e http://logs.openstack.org/28/668128/1/check/devstack-plugin-ceph-tempest-py3/5cd906c/job-output.txt.gz#_2019-06-28_10_40_46_087286\n \u003e \u003e\n \u003e \u003e Will approve the nova stable/rocky backport once the stable/stein\n \u003e \u003e one merges.\n \u003e \n \u003e those test result was with nova master. You mean these -\n \u003e http://logs.openstack.org/41/668141/1/check/devstack-plugin-ceph-tempest/30f0b35/testr_results.html.gz\n \u003e \n \u003e let\u0027s add stable jobs as experimental to tests the changes on\n \u003e stable within same patch.\n\nadding the stable version of devstack-plugin-ceph-tempest-py3 in experimental pipeline- https://review.opendev.org/#/c/668223/","accounts_in_message":[],"_revision_number":1},{"id":"5f9c50ac3f7d022480e4c8950b4a7d5a55a90a8a","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-06-28 18:10:42.000000000","message":"Patch Set 1:\n\n\u003e those test result was with nova master. You mean these - http://logs.openstack.org/41/668141/1/check/devstack-plugin-ceph-tempest/30f0b35/testr_results.html.gz\n\nNo, what I pasted were results from the job in *this* change (668128) which depends on the nova stable/rocky backport. You linked to the DNM nova change on top of the stable/rocky backport, which depends on this change. We\u0027re both essentially showing the same thing and the DNM nova patches are redundant.","accounts_in_message":[],"_revision_number":1},{"id":"6d53e17d2ad1b474f47d8e399a1a7f9369b14a05","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-06-28 18:16:01.000000000","message":"Patch Set 1:\n\nOh I see what you were saying now - this is on master and thus only pulls in the nova master change even though it depends on the nova stable/rocky change. So the DNM patches in nova are necessary to test the fix on those branches (until your stable branch change for this repo merges).","accounts_in_message":[],"_revision_number":1},{"id":"95d80ca9ceb304a04e68e3d8546dcb92b953eb05","author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"date":"2019-06-28 18:17:21.000000000","message":"Patch Set 1:\n\nI\u0027m +2 on the nova stable/rocky change now.","accounts_in_message":[],"_revision_number":1},{"id":"f179aa3ed27dc63477fd18d56e9108aed648bcc0","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2019-06-30 16:58:46.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"89d217fc401b34364cea24352ee4b6855b99a5de","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-30 16:58:57.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"54da2370a5b38852bb69d9687ddac7e53add7ead","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-30 17:06:24.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"c157afd717ef25b0604f2130f2605da87c85e063","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-30 17:06:24.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-bashate http://logs.openstack.org/28/668128/1/gate/openstack-tox-bashate/3bad0ca/ : SUCCESS in 2m 17s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"ffcba4bb55b897fb521f3fc8cf7806de95f49aac","revisions":{"ffcba4bb55b897fb521f3fc8cf7806de95f49aac":{"kind":"REWORK","_number":1,"created":"2019-06-28 09:31:30.000000000","uploader":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"ref":"refs/changes/28/668128/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-plugin-ceph","ref":"refs/changes/28/668128/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/28/668128/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/28/668128/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/28/668128/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-plugin-ceph refs/changes/28/668128/1"}}},"commit":{"parents":[{"commit":"2590c305f164012dca427e54f74eb3d50f72b854","subject":"tempest: re-enable shelve tests for stable stein","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/2590c305f164012dca427e54f74eb3d50f72b854"}]}],"author":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2019-06-28 09:30:54.000000000","tz":60},"committer":{"name":"Lee Yarwood","email":"lyarwood@redhat.com","date":"2019-06-28 09:31:17.000000000","tz":60},"subject":"tempest: re-enable shelve tests for stable rocky","message":"tempest: re-enable shelve tests for stable rocky\n\nRelated-Bug: #1653953\nDepends-On: https://review.opendev.org/#/c/668119/\nChange-Id: I1aa87fc16a37b6051e55c35ae69ef911a87d4c11\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ffcba4bb55b897fb521f3fc8cf7806de95f49aac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-plugin-ceph/commit/ffcba4bb55b897fb521f3fc8cf7806de95f49aac"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}