)]}'
{"id":"openstack%2Fmistral~573679","triplet_id":"openstack%2Fmistral~master~I4bf689b6b31036ff8dea522fc90f8e086b726cdf","project":"openstack/mistral","branch":"master","topic":"573679","hashtags":[],"change_id":"I4bf689b6b31036ff8dea522fc90f8e086b726cdf","subject":"Remove NotImplementedError exception","status":"ABANDONED","created":"2018-06-08 12:23:45.000000000","updated":"2018-09-04 06:42:13.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c03dc88c3fa07c8b38a710d7925808d3ae1474e7","_number":573679,"virtual_id_number":573679,"owner":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},{"date":"2018-06-13 06:23:36.000000000","_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"value":1,"date":"2018-06-09 06:17:28.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"}],"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":{"all":[{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-06-08 18:32:15.000000000","updated_by":{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},"reviewer":{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},"state":"REVIEWER"},{"updated":"2018-06-09 06:17:28.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":"2018-06-09 06:48:21.000000000","updated_by":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"reviewer":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"state":"REVIEWER"},{"updated":"2018-06-13 06:23:36.000000000","updated_by":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"reviewer":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"state":"REVIEWER"}],"messages":[{"id":"e386958ccd761ff9ded4decc7ddb6a573b334feb","author":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"date":"2018-06-08 12:23:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"7aa322ce2564c53f7585e13fae11d568683fffe4","author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"date":"2018-06-08 12:34:43.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"bf3e61dfa8932f62f061402f6273fcbad7781263","author":{"_account_id":27008,"name":"Vitalii Solodilov","email":"Mcdkr@yandex.ru","username":"mcdoker18"},"date":"2018-06-08 18:32:15.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":1},{"id":"eb8b0b819e15842843341c52e4625862a1eaf01b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-08 21:33:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/79/573679/1/check/openstack-tox-pep8/0878504/ : SUCCESS in 5m 10s\n- openstack-tox-py27 http://logs.openstack.org/79/573679/1/check/openstack-tox-py27/927d08c/ : SUCCESS in 10m 36s\n- openstack-tox-py35 http://logs.openstack.org/79/573679/1/check/openstack-tox-py35/7d666da/ : SUCCESS in 9m 28s\n- build-openstack-sphinx-docs http://logs.openstack.org/79/573679/1/check/build-openstack-sphinx-docs/6d8099f/html/ : SUCCESS in 5m 18s\n- openstack-tox-cover http://logs.openstack.org/79/573679/1/check/openstack-tox-cover/8cb8c7c/ : FAILURE in 9m 12s (non-voting)\n- mistral-devstack http://logs.openstack.org/79/573679/1/check/mistral-devstack/1c53163/ : SUCCESS in 40m 41s\n- mistral-devstack-non-apache http://logs.openstack.org/79/573679/1/check/mistral-devstack-non-apache/0058561/ : SUCCESS in 35m 44s\n- mistral-devstack-kombu http://logs.openstack.org/79/573679/1/check/mistral-devstack-kombu/2593432/ : SUCCESS in 34m 31s\n- mistral-tox-unit-mysql http://logs.openstack.org/79/573679/1/check/mistral-tox-unit-mysql/37a1197/ : SUCCESS in 27m 20s\n- mistral-tox-unit-postgresql http://logs.openstack.org/79/573679/1/check/mistral-tox-unit-postgresql/a2d11b6/ : SUCCESS in 27m 42s\n- tripleo-ci-centos-7-scenario003-multinode-oooq-container http://logs.openstack.org/79/573679/1/check/tripleo-ci-centos-7-scenario003-multinode-oooq-container/f4f3319/ : SUCCESS in 2h 31m 59s\n- mistral-rally-task http://logs.openstack.org/79/573679/1/check/mistral-rally-task/5b529e2/ : SUCCESS in 55m 34s\n- openstack-tox-lower-constraints http://logs.openstack.org/79/573679/1/check/openstack-tox-lower-constraints/97db731/ : SUCCESS in 11m 12s","accounts_in_message":[],"_revision_number":1},{"id":"b8bccd716429e80637dfd5160563acec63faf50d","author":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"date":"2018-06-09 03:59:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3da3998a2f2f98f837b41e506bbe17dee6de7173","author":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"date":"2018-06-09 04:00:45.000000000","message":"Patch Set 1: Code-Review+1\n\n\u003e (2 comments)\nThank you for your comments","accounts_in_message":[],"_revision_number":1},{"id":"78a50af1914887a8bd031d3cbb76237107975ca6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-06-09 06:17:28.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/79/573679/2/check/openstack-tox-pep8/14fcc87/ : SUCCESS in 4m 53s\n- openstack-tox-py27 http://logs.openstack.org/79/573679/2/check/openstack-tox-py27/62f55dc/ : SUCCESS in 8m 56s\n- openstack-tox-py35 http://logs.openstack.org/79/573679/2/check/openstack-tox-py35/78d3649/ : SUCCESS in 8m 45s\n- build-openstack-sphinx-docs http://logs.openstack.org/79/573679/2/check/build-openstack-sphinx-docs/9848ee3/html/ : SUCCESS in 5m 06s\n- openstack-tox-cover http://logs.openstack.org/79/573679/2/check/openstack-tox-cover/b2f7ffd/ : FAILURE in 8m 27s (non-voting)\n- mistral-devstack http://logs.openstack.org/79/573679/2/check/mistral-devstack/9086e72/ : SUCCESS in 35m 12s\n- mistral-devstack-non-apache http://logs.openstack.org/79/573679/2/check/mistral-devstack-non-apache/33636bc/ : SUCCESS in 35m 06s\n- mistral-devstack-kombu http://logs.openstack.org/79/573679/2/check/mistral-devstack-kombu/1df9279/ : SUCCESS in 35m 30s\n- mistral-tox-unit-mysql http://logs.openstack.org/79/573679/2/check/mistral-tox-unit-mysql/c62cc9a/ : SUCCESS in 27m 25s\n- mistral-tox-unit-postgresql http://logs.openstack.org/79/573679/2/check/mistral-tox-unit-postgresql/d74b730/ : SUCCESS in 26m 56s\n- tripleo-ci-centos-7-scenario003-multinode-oooq-container http://logs.openstack.org/79/573679/2/check/tripleo-ci-centos-7-scenario003-multinode-oooq-container/abb10c4/ : SUCCESS in 2h 15m 32s\n- mistral-rally-task http://logs.openstack.org/79/573679/2/check/mistral-rally-task/e9c8d1c/ : SUCCESS in 54m 32s\n- openstack-tox-lower-constraints http://logs.openstack.org/79/573679/2/check/openstack-tox-lower-constraints/e97541b/ : SUCCESS in 9m 58s","accounts_in_message":[],"_revision_number":2},{"id":"b2b774b8ec729ad1f3ab0e07b23bcb7c33950e4e","author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"date":"2018-06-09 06:48:21.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"23036ad0b3a579bd2c1c595a0cee875fee1baf82","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2018-06-13 06:23:36.000000000","message":"Patch Set 2:\n\nI honestly don\u0027t see much sense in this patch. IMO explicitly declaring an exception has a clear message to the reader of the code rather than just declaring \"pass\". Even if it\u0027s never raised. But I\u0027m not totally against this change too.","accounts_in_message":[],"_revision_number":2},{"id":"c9774e1d10396087b155b1c6e4f32b9ebd34e017","author":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"date":"2018-09-04 06:42:13.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"74c274450a033c013fe1eeb42ecbb9a1b5c4ccac","revisions":{"eceeae9a54f13a6aecdf88107f2e4c4d3febe798":{"kind":"REWORK","_number":1,"created":"2018-06-08 12:23:45.000000000","uploader":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"ref":"refs/changes/79/573679/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/79/573679/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/79/573679/1"}}},"commit":{"parents":[{"commit":"13b2b0fbaf9662606e9d04781c78b5109bfc8145","subject":"Merge \"Fix the call to start_workflow from events triggers\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/13b2b0fbaf9662606e9d04781c78b5109bfc8145"}]}],"author":{"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","date":"2018-06-08 12:26:14.000000000","tz":420},"committer":{"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","date":"2018-06-08 12:27:33.000000000","tz":420},"subject":"Remove NotImplementedError exception","message":"Remove NotImplementedError exception\n\nThis patch aim to change \u0027NotImplementedError\u0027 to \u0027pass\u0027 statement into\nmethods that already implemented\n\nChange-Id: I4bf689b6b31036ff8dea522fc90f8e086b726cdf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/eceeae9a54f13a6aecdf88107f2e4c4d3febe798"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/eceeae9a54f13a6aecdf88107f2e4c4d3febe798"}]},"branch":"refs/heads/master"},"74c274450a033c013fe1eeb42ecbb9a1b5c4ccac":{"kind":"REWORK","_number":2,"created":"2018-06-09 03:59:27.000000000","uploader":{"_account_id":28174,"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","username":"longkb.fvl"},"ref":"refs/changes/79/573679/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/79/573679/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/79/573679/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/79/573679/2"}}},"commit":{"parents":[{"commit":"13b2b0fbaf9662606e9d04781c78b5109bfc8145","subject":"Merge \"Fix the call to start_workflow from events triggers\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/13b2b0fbaf9662606e9d04781c78b5109bfc8145"}]}],"author":{"name":"Kim Bao Long","email":"longkb@vn.fujitsu.com","date":"2018-06-08 12:26:14.000000000","tz":420},"committer":{"name":"LongKB","email":"longkb@vn.fujitsu.com","date":"2018-06-09 03:59:24.000000000","tz":420},"subject":"Remove NotImplementedError exception","message":"Remove NotImplementedError exception\n\nThis patch aim to change \u0027NotImplementedError\u0027 to \u0027pass\u0027 statement into\nmethods that already implemented\n\nChange-Id: I4bf689b6b31036ff8dea522fc90f8e086b726cdf\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/74c274450a033c013fe1eeb42ecbb9a1b5c4ccac"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/74c274450a033c013fe1eeb42ecbb9a1b5c4ccac"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}