)]}'
{"id":"openstack%2Fnova~3335","triplet_id":"openstack%2Fnova~master~I4fa22980a28e4a051176f1426a571f37fb5f118e","project":"openstack/nova","branch":"master","topic":"affinity_filter","hashtags":[],"change_id":"I4fa22980a28e4a051176f1426a571f37fb5f118e","subject":"Add affinity filters updated to use scheduler_hints and have non-douchey names","status":"MERGED","created":"2012-01-24 02:25:46.000000000","updated":"2012-01-24 16:46:25.000000000","submitted":"2012-01-24 16:42:51.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9dedf5cde1d2f65d45ae71484550e0ded3f8d839","_number":3335,"virtual_id_number":3335,"owner":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},{"value":1,"date":"2012-01-24 16:42:51.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":{"approved":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"all":[{"value":0,"date":"2012-01-24 16:20:39.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},{"value":2,"date":"2012-01-24 16:22:43.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},{"value":0,"date":"2012-01-24 16:42:51.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"all":[{"value":0,"date":"2012-01-24 16:20:39.000000000","_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},{"value":1,"date":"2012-01-24 16:22:43.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-01-24 06:56:18.000000000","updated_by":{"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},"reviewer":{"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},"state":"REVIEWER"},{"updated":"2012-01-24 08:51:18.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2012-01-24 16:22:43.000000000","updated_by":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"reviewer":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"state":"REVIEWER"},{"updated":"2012-01-24 16:42:51.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"6160612e2bde7b6aab7182ef5f8a5cad6bedc96c","author":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"date":"2012-01-24 02:26:35.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"79852812352d1476c5d0bfbdb5bd7d564a157ee1","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2012-01-24 03:11:54.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"0af373cf81285ebb3cde4c1b704099f8d3397efe","author":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"date":"2012-01-24 06:31:53.000000000","message":"Patch Set 2: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"394a90fee405aceb3b8fca467f16ef3d80bc56d5","author":{"_account_id":70,"name":"Joshua McKenty","email":"joshua@pistoncloud.com","username":"joshua-mckenty"},"date":"2012-01-24 06:56:18.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\nYour commit messages could use some work, son.\nAnd somehow the /32 test feels cheap.\nBut this is decent overall.","accounts_in_message":[],"_revision_number":2},{"id":"9fc1360711fe375af5d75a1e9a752113cb37adf3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-01-24 07:42:00.000000000","message":"Patch Set 2: Fails; I would prefer that you didn\u0027t submit this\n\nBuild Failed \n \nhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/759/ : FAILURE \nhttps://jenkins.openstack.org/job/gate-nova-merge/1043/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-unittests/656/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-pep8/2419/ : UNSTABLE","accounts_in_message":[],"_revision_number":2},{"id":"6991e6a1e4853bb24db32486227b39a7525ba398","author":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"date":"2012-01-24 07:49:15.000000000","message":"Patch Set 2:\n\nErr… that wasn\u0027t supposed to have been approved yet… but the test fail is the https 500 error on git upstream for devstack.","accounts_in_message":[],"_revision_number":2},{"id":"a33040233e50aa959f4e7aba7a18f16d7133cd00","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-01-24 08:49:48.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\nI think this is fine as a first pass.  Yes there are optimizations available but I like keeping the scheduler plugin logic simple for now.","accounts_in_message":[],"_revision_number":2},{"id":"f106b88311820d76e62b215130fb0d8c9ce79c16","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-01-24 08:51:18.000000000","message":"Patch Set 2: No score\n\nhmm looks like a number of pep8 errors.  If you fix them and push, it is good from me.","accounts_in_message":[],"_revision_number":2},{"id":"d715618bb0519927a318c25c0cbcca8c4b2fa68f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-01-24 09:00:13.000000000","message":"Patch Set 2: No score\n\nBuild Unstable \n \nhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/760/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-merge/1044/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-unittests/657/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-pep8/2420/ : UNSTABLE","accounts_in_message":[],"_revision_number":2},{"id":"64fae139f4b6b10385a866a3aaec5d032ddb5e7c","author":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"date":"2012-01-24 16:19:51.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9a28bb777683acd676b222f1eb4e4f072f152571","author":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"date":"2012-01-24 16:20:39.000000000","message":"Patch Set 3:\n\nFixed the pep8 stuff.","accounts_in_message":[],"_revision_number":3},{"id":"e9e1d657ccf30c0cfd5455ef4d26eff78f32c3eb","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2012-01-24 16:22:43.000000000","message":"Patch Set 3: Looks good to me (core reviewer); Approved\n\nOnly spacing changes for pep8 fixes.  Approve!","accounts_in_message":[],"_revision_number":3},{"id":"435664311b34e535c9f289e3487429ec2d947086","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-01-24 16:42:51.000000000","message":"Patch Set 3: Verified\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/768/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-merge/1047/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-unittests/660/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-nova-pep8/2423/ : SUCCESS","accounts_in_message":[],"_revision_number":3},{"id":"202f7ddf03a6c97bc5ad740243f12f965f090ae9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-01-24 16:42:52.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3},{"id":"35d7099daf0c64275910de97b829c31ebbd1adea","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-01-24 16:46:25.000000000","message":"Patch Set 3:\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/nova-tarball/1748/ : SUCCESS","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"91bc67d81a9711fbf5a0f0c46bbf1d87232391f8","revisions":{"48387f4f015d85c61d40a52784de65789247b1e6":{"kind":"REWORK","_number":1,"created":"2012-01-24 02:25:46.000000000","uploader":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"ref":"refs/changes/35/3335/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/3335/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/3335/1"}}},"commit":{"parents":[{"commit":"b160d731b5dff65ba887578ad8fe4d7403426613","subject":"Merge \"Brings back keystone middleware\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b160d731b5dff65ba887578ad8fe4d7403426613"}]}],"author":{"name":"Christopher MacGown","email":"ignoti+github@gmail.com","date":"2012-01-24 02:25:34.000000000","tz":-480},"committer":{"name":"Christopher MacGown","email":"ignoti+github@gmail.com","date":"2012-01-24 02:25:41.000000000","tz":-480},"subject":"Cleaner affinity filters, updated to use scheduler_hints and have non-douchey names","message":"Cleaner affinity filters, updated to use scheduler_hints and have non-douchey names\n\nChange-Id: I4fa22980a28e4a051176f1426a571f37fb5f118e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/48387f4f015d85c61d40a52784de65789247b1e6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/48387f4f015d85c61d40a52784de65789247b1e6"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b160d731b5dff65ba887578ad8fe4d7403426613","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"f391e2772066e81ba2c918eb801226447fc31453":{"kind":"NO_CHANGE","_number":2,"created":"2012-01-24 02:26:35.000000000","uploader":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"ref":"refs/changes/35/3335/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/3335/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/3335/2"}}},"commit":{"parents":[{"commit":"b160d731b5dff65ba887578ad8fe4d7403426613","subject":"Merge \"Brings back keystone middleware\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b160d731b5dff65ba887578ad8fe4d7403426613"}]}],"author":{"name":"Christopher MacGown","email":"chris@pistoncloud.com","date":"2012-01-24 02:25:34.000000000","tz":-480},"committer":{"name":"Christopher MacGown","email":"ignoti+github@gmail.com","date":"2012-01-24 02:26:23.000000000","tz":-480},"subject":"Cleaner affinity filters, updated to use scheduler_hints and have non-douchey names","message":"Cleaner affinity filters, updated to use scheduler_hints and have non-douchey names\n\nChange-Id: I4fa22980a28e4a051176f1426a571f37fb5f118e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f391e2772066e81ba2c918eb801226447fc31453"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f391e2772066e81ba2c918eb801226447fc31453"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b160d731b5dff65ba887578ad8fe4d7403426613","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"91bc67d81a9711fbf5a0f0c46bbf1d87232391f8":{"kind":"REWORK","_number":3,"created":"2012-01-24 16:19:51.000000000","uploader":{"_account_id":693,"name":"Christopher MacGown","email":"chris@pistoncloud.com","username":"0x44"},"ref":"refs/changes/35/3335/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/35/3335/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/35/3335/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/35/3335/3"}}},"commit":{"parents":[{"commit":"0fc18408d93aa30a2cb2650951dc1171b4ef6bcd","subject":"Merge \"Add floating IP support to Quantum Manager\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0fc18408d93aa30a2cb2650951dc1171b4ef6bcd"}]}],"author":{"name":"Christopher MacGown","email":"chris@pistoncloud.com","date":"2012-01-24 02:25:34.000000000","tz":-480},"committer":{"name":"Christopher MacGown","email":"ignoti+github@gmail.com","date":"2012-01-24 16:19:46.000000000","tz":-480},"subject":"Add affinity filters updated to use scheduler_hints and have non-douchey names","message":"Add affinity filters\nupdated to use scheduler_hints and have non-douchey names\n\nChange-Id: I4fa22980a28e4a051176f1426a571f37fb5f118e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/91bc67d81a9711fbf5a0f0c46bbf1d87232391f8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/91bc67d81a9711fbf5a0f0c46bbf1d87232391f8"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"0fc18408d93aa30a2cb2650951dc1171b4ef6bcd","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}