)]}'
{"id":"openstack%2Ftempest~50310","triplet_id":"openstack%2Ftempest~master~I4c4a2c993a9cedde3f8fa0f1a31b13d1eb4ed5e6","project":"openstack/tempest","branch":"master","topic":"add_uuid_generator","hashtags":[],"change_id":"I4c4a2c993a9cedde3f8fa0f1a31b13d1eb4ed5e6","subject":"Add a random uuid generator to data_utils","status":"ABANDONED","created":"2013-10-08 12:09:29.000000000","updated":"2013-10-17 06:03:08.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b6251fb8c3220f6018742dc77f061ef923091831","_number":50310,"virtual_id_number":50310,"owner":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2013-10-08 12:48:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-10-09 06:19:46.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"value":-1,"date":"2013-10-09 04:10:13.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"date":"2013-10-09 02:01:23.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"}],"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"date":"2013-10-09 04:10:13.000000000","_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"}],"values":{"-1":"Release Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-08 12:48:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-10-09 02:01:23.000000000","updated_by":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"reviewer":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"state":"REVIEWER"},{"updated":"2013-10-09 04:10:13.000000000","updated_by":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"reviewer":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"state":"REVIEWER"}],"messages":[{"id":"f5f05d55e29374f41a439681152ddee1008676ca","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-08 12:48:59.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-tempest-pep8 http://logs.openstack.org/10/50310/1/check/gate-tempest-pep8/7695c62 : SUCCESS in 1m 51s\n- gate-tempest-docs http://logs.openstack.org/10/50310/1/check/gate-tempest-docs/0d1b183 : SUCCESS in 2m 47s\n- check-tempest-devstack-vm-full http://logs.openstack.org/10/50310/1/check/check-tempest-devstack-vm-full/d4e1040 : SUCCESS in 39m 18s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/10/50310/1/check/check-tempest-devstack-vm-postgres-full/984b7ba : SUCCESS in 38m 18s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/10/50310/1/check/check-tempest-devstack-vm-neutron/b048510 : SUCCESS in 29m 04s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/10/50310/1/check/gate-tempest-devstack-vm-large-ops/a1d2495 : SUCCESS in 10m 53s\n- check-grenade-devstack-vm http://logs.openstack.org/10/50310/1/check/check-grenade-devstack-vm/7d9e55a : SUCCESS in 32m 24s\n","accounts_in_message":[],"_revision_number":1},{"id":"386ff4c16c5771d516cd12edeefd3894b988ca0b","author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"date":"2013-10-09 02:01:23.000000000","message":"Patch Set 1:\n\nDo we really need a help function? or directly use:\n\nimport uuid\n\nuuid.uuid4()\n\nwanna see others\u0027 opinion.","accounts_in_message":[],"_revision_number":1},{"id":"99a588af3fde77f49a165766109adf92a676523d","author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"date":"2013-10-09 04:10:13.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nI\u0027ll spare mine, this could have helped standardizing if we were to also provide random but invalid uuids\n\nI see there are a few negative tests proposed which attempt some \"string validation\", but I\u0027m not sure those are good and will be merged, see https://wiki.openstack.org/wiki/Meetings/QATeamMeeting#Proposed_Agenda_for_October_10_2013","accounts_in_message":[],"_revision_number":1},{"id":"b38704a834ecb775fce456d37dab9d8356225ff3","author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"date":"2013-10-09 06:19:46.000000000","message":"Patch Set 1:\n\n@Giulio: Thank you for your comment.\nHowever I may not understand what you mean.\n\nYou mean \"data_utils.py\u0027s change\" is good. But you are not sure other changes(tests) are good because of the Agenda\u0027s topic.\nRight?","accounts_in_message":[],"_revision_number":1},{"id":"d83914ccc471ce034bfeab72f2327406db8848d9","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-17 06:03:08.000000000","message":"Patch Set 1:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1},{"id":"4507ee08badb9118129dd303b6c490416edf1462","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-17 06:03:08.000000000","message":"Patch Set 1: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"1c0a1273d7b51854dcc0dedf697c7c26b11f80c2","revisions":{"1c0a1273d7b51854dcc0dedf697c7c26b11f80c2":{"kind":"REWORK","_number":1,"created":"2013-10-08 12:09:29.000000000","uploader":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"ref":"refs/changes/10/50310/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/10/50310/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/10/50310/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/10/50310/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/10/50310/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/10/50310/1"}}},"commit":{"parents":[{"commit":"d0181f647c4fdac223d9058022d471e4fb8b8a61","subject":"Merge \"add two negative tests for flavor-access\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/d0181f647c4fdac223d9058022d471e4fb8b8a61"}]}],"author":{"name":"Masayuki Igawa","email":"igawa@mxs.nes.nec.co.jp","date":"2013-10-08 11:59:05.000000000","tz":540},"committer":{"name":"Masayuki Igawa","email":"igawa@mxs.nes.nec.co.jp","date":"2013-10-08 12:06:33.000000000","tz":540},"subject":"Add a random uuid generator to data_utils","message":"Add a random uuid generator to data_utils\n\nIn some test cases, there are same code for using a random uuid string.\nThis commit introduces a random uuid generator(rand_uuid_str()) to\ndata_utils and changes for using the generator.\n\nChange-Id: I4c4a2c993a9cedde3f8fa0f1a31b13d1eb4ed5e6\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/1c0a1273d7b51854dcc0dedf697c7c26b11f80c2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/1c0a1273d7b51854dcc0dedf697c7c26b11f80c2"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
