)]}'
{"id":"openstack%2Fcinder~7517","triplet_id":"openstack%2Fcinder~master~I64b18f709683ad323af864af3d3fedcc091a039b","project":"openstack/cinder","branch":"master","topic":"bp/cinder-notifications","hashtags":[],"change_id":"I64b18f709683ad323af864af3d3fedcc091a039b","subject":"Adding the volume notifications to cinder","status":"ABANDONED","created":"2012-05-16 18:35:00.000000000","updated":"2012-08-15 21:13:29.000000000","total_comment_count":11,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"cc0e7aebd16f4a5eea3421cb04ba83079c630812","_number":7517,"virtual_id_number":7517,"owner":{"_account_id":1175,"name":"Craig Vyvial"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},{"_account_id":1175,"name":"Craig Vyvial"},{"date":"2012-08-15 16:19:21.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":-1,"date":"2012-08-08 02:00:41.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},{"value":-1,"date":"2012-08-08 21:51:56.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1175,"name":"Craig Vyvial"},{"value":0,"date":"2012-08-15 16:19:21.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"value":-1,"date":"2012-08-08 20:43:23.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":1175,"name":"Craig Vyvial"},{"date":"2012-08-15 16:19:21.000000000","_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"date":"2012-08-08 20:43:23.000000000","_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},{"_account_id":1175,"name":"Craig Vyvial"},{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Branch 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":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},{"_account_id":1175,"name":"Craig Vyvial"},{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-05-16 19:09:11.000000000","updated_by":{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},"reviewer":{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},"state":"REVIEWER"},{"updated":"2012-05-16 19:09:36.000000000","updated_by":{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},"reviewer":{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},"state":"REVIEWER"},{"updated":"2012-08-08 02:00:41.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2012-08-08 20:43:23.000000000","updated_by":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"reviewer":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"state":"REVIEWER"},{"updated":"2012-08-08 21:51:56.000000000","updated_by":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"reviewer":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"state":"REVIEWER"},{"updated":"2012-08-15 16:19:21.000000000","updated_by":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"reviewer":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"state":"REVIEWER"}],"messages":[{"id":"0272e5fc6d3bebe7947d797d74489e3905a34add","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-05-16 18:38:40.000000000","message":"Patch Set 1: Verified\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/check-cinder-python26/10/ : SUCCESS \nhttps://jenkins.openstack.org/job/check-cinder-python27/10/ : SUCCESS \nhttps://jenkins.openstack.org/job/check-cinder-merge/10/ : SUCCESS \nhttps://jenkins.openstack.org/job/check-cinder-pep8/10/ : SUCCESS","accounts_in_message":[],"_revision_number":1},{"id":"04d019d4199f8e04267ff7656237fb9b7b8659fc","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2012-05-16 18:57:42.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nThanks Craig!!!","accounts_in_message":[],"_revision_number":1},{"id":"8fdd1f39db49daac4e426933534ce5f2d75c3ee5","author":{"_account_id":838,"name":"Nirmal Ranganathan","email":"rnirmal@gmail.com","username":"rnirmal"},"date":"2012-05-16 19:09:11.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nEverything else looks good, just the one inline comment. That function shouldn\u0027t be in exception.py","accounts_in_message":[],"_revision_number":1},{"id":"fb71df0c75cb555815d7bc58337e3b72a78287a8","author":{"_account_id":396,"name":"Mandell","email":"mdegerness@nvidia.com","username":"mdegerne"},"date":"2012-05-16 19:09:36.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nSome comments in line.","accounts_in_message":[],"_revision_number":1},{"id":"3afd53113741df273d48f7f6f150c6d0d7c19f43","author":{"_account_id":1175,"name":"Craig Vyvial"},"date":"2012-05-23 18:46:03.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"00436398535e29406006c00ac7cab470c746aa39","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-05-31 06:03:06.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},{"id":"792ae661803739ce1458e9bbeb3b059c4d0af0ec","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2012-05-31 06:03:06.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":"153af5998383a1102dcf6e76698cd2ebd6b62252","author":{"_account_id":1175,"name":"Craig Vyvial"},"date":"2012-08-08 01:49:44.000000000","message":"Patch Set 1: Restored\n\nbringing this back","accounts_in_message":[],"_revision_number":1},{"id":"1d28caad3ace8998fffaa643e7c23ea92ebaf251","author":{"_account_id":1175,"name":"Craig Vyvial"},"date":"2012-08-08 01:49:57.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2692cb55d0cc15b01f786a41d99c41090d53e887","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-08 02:00:41.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nBuild failed\n\n- https://jenkins.openstack.org/job/gate-cinder-merge/505/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-cinder-docs/431/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-cinder-pep8/470/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-cinder-python26/464/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-cinder-python27/471/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/6978/consoleFull : FAILURE\n","accounts_in_message":[],"_revision_number":2},{"id":"cf3437915209f19b80000ecd3b0714ce1f159d42","author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"date":"2012-08-08 20:43:23.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(5 inline comments)\n\nsurprised we missed these... think I\u0027ve pointed out most of them.  fixing these I was able to run in devstack.","accounts_in_message":[],"_revision_number":2},{"id":"0cb96f293a9d4e616f00622dce09d6c9f2d85789","author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"date":"2012-08-08 21:51:56.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"3b253cb52157cc5f2632bd53c3c4d5b0a7a336c1","author":{"_account_id":1175,"name":"Craig Vyvial"},"date":"2012-08-15 21:13:29.000000000","message":"Patch Set 2: Abandoned\n\nmade a new review for this","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"5eaf16c5486c15cd7d53135527b7a320a6f51522","revisions":{"62efe3aec4130a1c56aea5cfca92c4192047a83f":{"kind":"REWORK","_number":1,"created":"2012-05-16 18:35:00.000000000","uploader":{"_account_id":1175,"name":"Craig Vyvial"},"ref":"refs/changes/17/7517/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/17/7517/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/17/7517/1"}}},"commit":{"parents":[{"commit":"fe23db33801fa72665e480534d070ddae4a9c451","subject":"Remove instance Foreign Key in volumes table, replace with instance_uuid","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/fe23db33801fa72665e480534d070ddae4a9c451"}]}],"author":{"name":"Craig Vyvial","email":"cp16net@gmail.com","date":"2012-05-16 16:20:19.000000000","tz":-300},"committer":{"name":"Craig Vyvial","email":"cp16net@gmail.com","date":"2012-05-16 16:20:19.000000000","tz":-300},"subject":"Adding the volume notifications to cinder","message":"Adding the volume notifications to cinder\n\nadding the volume notifications\nadded unit tests for notifications\nadded context for notify with rabbit\n\nblueprint cinder-notifications\n\nChange-Id: I64b18f709683ad323af864af3d3fedcc091a039b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/62efe3aec4130a1c56aea5cfca92c4192047a83f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/62efe3aec4130a1c56aea5cfca92c4192047a83f"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"fe23db33801fa72665e480534d070ddae4a9c451","is_merged_in_target_branch":true,"change_id":"I54d723dcb9819731a58ec64095c54c99aa9e5dc4","change_number":7279,"patch_set_number":5,"change_status":"MERGED"}],"branch":"refs/heads/master"},"5eaf16c5486c15cd7d53135527b7a320a6f51522":{"kind":"REWORK","_number":2,"created":"2012-08-08 01:49:57.000000000","uploader":{"_account_id":1175,"name":"Craig Vyvial"},"ref":"refs/changes/17/7517/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cinder","ref":"refs/changes/17/7517/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cinder refs/changes/17/7517/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cinder refs/changes/17/7517/2"}}},"commit":{"parents":[{"commit":"e8db777de63e2308c12a8c3b54e7a0ed54f8a068","subject":"Merge \"Deprecate root_helper in favor of rootwrap_config\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/e8db777de63e2308c12a8c3b54e7a0ed54f8a068"}]}],"author":{"name":"Craig Vyvial","email":"cp16net@gmail.com","date":"2012-05-16 16:20:19.000000000","tz":-300},"committer":{"name":"Craig Vyvial","email":"cp16net@gmail.com","date":"2012-08-08 01:48:55.000000000","tz":-300},"subject":"Adding the volume notifications to cinder","message":"Adding the volume notifications to cinder\n\nadding the volume notifications\nadded unit tests for notifications\nadded context for notify with rabbit\n\nblueprint cinder-notifications\n\nChange-Id: I64b18f709683ad323af864af3d3fedcc091a039b\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/5eaf16c5486c15cd7d53135527b7a320a6f51522"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cinder/commit/5eaf16c5486c15cd7d53135527b7a320a6f51522"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"e8db777de63e2308c12a8c3b54e7a0ed54f8a068","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
