)]}'
{"id":"openstack%2Fcinderlib~798706","triplet_id":"openstack%2Fcinderlib~master~I121bc20c1ac4d7c07a1bcd13df19fdb868e65857","project":"openstack/cinderlib","branch":"master","topic":"datetime-serialization","hashtags":[],"change_id":"I121bc20c1ac4d7c07a1bcd13df19fdb868e65857","subject":"Fix test_global_setup unit test","status":"MERGED","created":"2021-06-29 15:50:40.000000000","updated":"2021-06-29 19:28:31.000000000","submitted":"2021-06-29 19:27:22.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":"798706-datetime-serialization","meta_rev_id":"72aa0b59e93308e19a11b9229a77ebe1dff11db9","_number":798706,"virtual_id_number":798706,"owner":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-06-29 19:27:22.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"all":[{"value":2,"date":"2021-06-29 16:49:49.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-06-29 18:05:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"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":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"all":[{"value":0,"date":"2021-06-29 16:50:36.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-06-29 18:05:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"recommended":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"all":[{"value":1,"date":"2021-06-29 16:50:36.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-06-29 16:49:49.000000000","updated_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"reviewer":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"state":"REVIEWER"},{"updated":"2021-06-29 17:21:34.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"},{"updated":"2021-06-29 18:05:34.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"}],"messages":[{"id":"1e5a85361e06fe451dcda45944866d00681e464c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"date":"2021-06-29 15:50:40.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2a695f8cd516398bfd6864aa0dd6f72fa4f1c029","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-06-29 16:49:49.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nThanks for the detailed commit message.\n\nZuul status page is showing everything passing with os-brick-src-tempest-lvm-lio-barbican still in progress (and that job passed earlier and this patch doesn\u0027t affect it).","accounts_in_message":[],"_revision_number":1},{"id":"e6dd53d6635c579ece90ceb43b0515c06d5e1727","tag":"autogenerated:gerrit:deleteVote","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-06-29 16:49:59.000000000","message":"Removed Workflow+1 by Brian Rosmaita \u003crosmaita.fossdev@gmail.com\u003e\n","accounts_in_message":[],"_revision_number":1},{"id":"33b4133bbe249bb86ba08c7b5c15fe76ae7137ce","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-06-29 16:50:36.000000000","message":"Patch Set 1: Review-Priority+1\n\nMeant to hit \"Review-Priority +1\" in the previous comment.","accounts_in_message":[],"_revision_number":1},{"id":"21d62b836af4d95e644f6f693f915edf433808b3","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-29 17:21:34.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f3ca2d78f1264850b99a242a6620658b : SUCCESS in 10m 58s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/44c45919832e4a4eb5b8cb84206c8f55 : SUCCESS in 8m 23s\n- cinderlib-tox-py36 https://zuul.opendev.org/t/openstack/build/1d53a040ce3a4739b7561c50548deca2 : SUCCESS in 10m 42s\n- cinderlib-tox-py37 https://zuul.opendev.org/t/openstack/build/257c8b0853f64e27b539e1f635861277 : SUCCESS in 10m 44s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/bc25304e338442ca9d6fcba8139ce986 : SUCCESS in 13m 36s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/09dd2bdacdf34698b78449c856ef558a : SUCCESS in 11m 45s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/ebe9812576e84cf099487520a224e1f8 : SUCCESS in 1h 29m 27s","accounts_in_message":[],"_revision_number":1},{"id":"2e414e2ac9e4fa5929dbd32b2320f36f466a9109","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2021-06-29 18:05:34.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1\n\nThis change makes sense to me.  Thanks!","accounts_in_message":[],"_revision_number":1},{"id":"b9044729dfc18eeb3cad097f17a1762dbb48ec88","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-29 18:05:53.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"654a4d742a458491d6d84196d659c1c773630c7d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-29 19:27:22.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/adf8138d8ae342d2b2ae7f990f6232e5 : SUCCESS in 10m 11s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b507cf035d32465a9d904f8960925d5a : SUCCESS in 7m 01s\n- cinderlib-tox-py36 https://zuul.opendev.org/t/openstack/build/61f339decdd748e3acec5aa7d99fa44b : SUCCESS in 9m 31s\n- cinderlib-tox-py37 https://zuul.opendev.org/t/openstack/build/e683e45d0c05480c8a20d9117b81db12 : SUCCESS in 9m 46s\n- cinderlib-lvm-functional https://zuul.opendev.org/t/openstack/build/a868223f4ff642bd9562b99bb98213f6 : SUCCESS in 13m 00s\n- cinderlib-ceph-functional https://zuul.opendev.org/t/openstack/build/028a7146a4e14c68898eb90e6a8a8e72 : SUCCESS in 12m 28s\n- os-brick-src-tempest-lvm-lio-barbican https://zuul.opendev.org/t/openstack/build/d41bf97470934ffa8b8203a2e8af3005 : SUCCESS in 1h 20m 24s","accounts_in_message":[],"_revision_number":1},{"id":"69344cf096c9b475b576aa95ad5671e404abeb57","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-29 19:27:22.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"72aa0b59e93308e19a11b9229a77ebe1dff11db9","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-29 19:28:31.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/22533953c7cb4a408add6d99861c7038 : SUCCESS in 53s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"42dbc4ac86dad64b19342aa491d4249bb1e36b10","revisions":{"42dbc4ac86dad64b19342aa491d4249bb1e36b10":{"kind":"REWORK","_number":1,"created":"2021-06-29 15:50:40.000000000","uploader":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"ref":"refs/changes/06/798706/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinderlib","ref":"refs/changes/06/798706/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/06/798706/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/06/798706/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinderlib refs/changes/06/798706/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinderlib refs/changes/06/798706/1"}}},"commit":{"parents":[{"commit":"4d859adcbd6556ede5aef9abca8c8a3c944082e9","subject":"Merge \"Continue wallaby development\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/4d859adcbd6556ede5aef9abca8c8a3c944082e9"}]}],"author":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2021-06-29 15:35:23.000000000","tz":120},"committer":{"name":"Gorka Eguileor","email":"geguileo@redhat.com","date":"2021-06-29 15:48:07.000000000","tz":120},"subject":"Fix test_global_setup unit test","message":"Fix test_global_setup unit test\n\nCurrent code in test_global_setup sets a non valid user_id and leaves it\nlike that for the rest of the tests, which can lead to error:\n\nValueError: A string is required in field user_id, not a _SentinelObject\n\nAn easy way to reproduce this error is to create a file called fail.txt\nwith the following content:\n\ncinderlib.tests.unit.test_cinderlib.TestCinderlib.test_global_setup\ncinderlib.tests.unit.test_serialization.TestSerialization.test_conn_to_and_from\n\nAnd then run from the tox virtual environment:\n\n    stestr run --serial --load-list\u003dfail.txt\n\nThis patch fixes this issue by restoring the values after the test is\nrun.\n\nChange-Id: I121bc20c1ac4d7c07a1bcd13df19fdb868e65857\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/42dbc4ac86dad64b19342aa491d4249bb1e36b10"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinderlib/commit/42dbc4ac86dad64b19342aa491d4249bb1e36b10"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"}},{"label":"Review-Priority","status":"MAY","applied_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
