)]}'
{"id":"openstack%2Fnova~12694","triplet_id":"openstack%2Fnova~master~I613da53876e6cd548566e27ab1693287023e9861","project":"openstack/nova","branch":"master","topic":"bp/trusted-filter-cache","hashtags":[],"change_id":"I613da53876e6cd548566e27ab1693287023e9861","subject":"Add trust level cache to trusted_filter","status":"MERGED","created":"2012-09-10 11:59:57.000000000","updated":"2013-01-22 11:18:21.000000000","submitted":"2013-01-22 11:18:18.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":35,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"30abd414f623de60f0ec20c4b899421c372de939","_number":12694,"virtual_id_number":12694,"owner":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},{"value":0,"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},{"value":0,"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":2,"date":"2013-01-22 11:18:18.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true}],"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":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":0,"date":"2013-01-22 10:07:14.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},{"value":0,"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},{"value":0,"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":2,"date":"2013-01-22 10:44:16.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true}],"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":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":0,"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},{"value":0,"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},{"value":0,"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":1,"date":"2013-01-22 10:44:16.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},{"value":0,"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},{"value":0,"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true}],"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":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true},{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-09-11 08:38:33.000000000","updated_by":{"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true},"reviewer":{"_account_id":99,"name":"Donald Dugger","username":"n0ano-ddd","inactive":true},"state":"REVIEWER"},{"updated":"2012-09-12 15:04:36.000000000","updated_by":{"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},"reviewer":{"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},"state":"REVIEWER"},{"updated":"2012-12-06 14:27:53.000000000","updated_by":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"reviewer":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"state":"REVIEWER"},{"updated":"2012-12-28 01:01:14.000000000","updated_by":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"reviewer":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"state":"REVIEWER"},{"updated":"2012-12-28 03:08:08.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2013-01-07 16:54:34.000000000","updated_by":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"reviewer":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"state":"REVIEWER"},{"updated":"2013-01-22 10:44:16.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":"2013-01-22 11:18:18.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"d56c4ba097c860dca212b5a1a84644789a054215","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-10 12:16:06.000000000","message":"Patch Set 1: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/1/check/gate-nova-merge/8394 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-nova-docs/4193 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-nova-pep8/9505 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-nova-python26/5656 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-nova-python27/5715 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-tempest-devstack-vm/10992 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-tempest-devstack-vm-cinder/973 : SUCCESS\n- http://logs.openstack.org/12694/1/check/gate-nova-pylint/637 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"2b2a6fe992e713704ecad15ddaa14b15fdd4a4a9","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-09-10 12:19:51.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/33851\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/33849\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/33850\n","accounts_in_message":[],"_revision_number":1},{"id":"7acfdc08c2eeb54ba36b2080094cfd40db20fa52","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-09-10 16:40:11.000000000","message":"Patch Set 1: Work In Progress\n\nI think this one should wait for grizzly.","accounts_in_message":[],"_revision_number":1},{"id":"b3752b21d18ae65c1f0ffdf5f1255e40677b8948","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-09-10 18:02:47.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\ninline stuff. Also this probably has to wait until grizzly.","accounts_in_message":[],"_revision_number":1},{"id":"73796dfe6183a159fa5dc8774c18521ffd6691c1","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-09-11 02:56:07.000000000","message":"Patch Set 1: (2 inline comments)\n\nRussell and Vish, thanks for your comments. \n\n\nAs to wait till grizzly, I will try my best to enhance the patches, but it\u0027s up to you core reviewers to make the decision :)\n\nThanks\n\n--jyh","accounts_in_message":[],"_revision_number":1},{"id":"90170bc54ba00d4ec68654064ce28a0e0fa14908","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-09-11 06:01:26.000000000","message":"Patch Set 1:\n\nthe data from the services and compute node table is already passed into the filter. Isn\u0027t that enough? You shouldn\u0027t need to make another db request.","accounts_in_message":[],"_revision_number":1},{"id":"64be83338079998c959f0f5aff379f4534d26a9a","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-09-11 08:36:02.000000000","message":"Patch Set 1:\n\nThanks for response. I think the filter_scheduler only pass data about one host to the filter\u0027s function, the data of host that would be filtered. Not whole data from services and compute node table.\n\nOr you mean we can change the filter_scheduler.py to passing the unfiltered_hosts_dict or filtered_hosts through filter_property to filter functions?","accounts_in_message":[],"_revision_number":1},{"id":"0cd2567ab9a7c8832539b48ecbbfcde47ee530d5","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-09-11 10:13:44.000000000","message":"Patch Set 1:\n\nWhy do you need more than the current host? The method is called for every host, so each call can update/check the cast for this host without having to update all hosts at once.","accounts_in_message":[],"_revision_number":1},{"id":"17233341f96a9436a4b86c66d5bae73b856bba81","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-09-11 14:31:21.000000000","message":"Patch Set 1:\n\nVish, the reason is, currently each time the trusted_filter is called, it will need poll the OAT service to check the trust level. As described in previous reply, polling OAT service may take several hundred ms to several seconds, depends on the OAT service configuration and strategy.\n\n This will be a huge problem if the host number is big, like 200 host machine. It may takes one minutes in good situation (200 * 300 ms),or 10 minutes in bad situation (200 * 3s), to finish the polling for all host, for only one instance launch.","accounts_in_message":[],"_revision_number":1},{"id":"cd890e8b1b2a244fcb305611fe9a062899c7a26a","author":{"_account_id":1981,"name":"Don Dugger","email":"n0ano@n0ano.com","username":"n0ano"},"date":"2012-09-12 15:04:36.000000000","message":"Patch Set 1:\n\nTo expand upon what Yunhong said, the problem is that each call to the attestation service takes upwards of a few seconds, whether you are attesting 1 host or a 100.  By calling the DB to find all of the compute nodes we can then fill up the cache with one call to the attestation service and satisfy subsequent sequential scheduler calls to the filter immediately out of the cache.","accounts_in_message":[],"_revision_number":1},{"id":"a22a3c6b37e2279c7501889304bb80753aeaa9ef","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-09-13 01:34:47.000000000","message":"Patch Set 1: No score\n\nDon: ah ok, I finally understand the rationale for the db call. It is so you can attest the whole list at once. In that case I\u0027m ok with this, but it needs to wait for grizzly to open.","accounts_in_message":[],"_revision_number":1},{"id":"abe54fcac5d4aea1f9227e119c03bd24bfcb9091","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-09-26 00:13:58.000000000","message":"Patch Set 1: Abandoned\n\nAbandon till open for review again.","accounts_in_message":[],"_revision_number":1},{"id":"85739e37609dcd9716be3c8cef40d2ad75b1e1dd","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-11-06 17:35:19.000000000","message":"Patch Set 1:\n\nThis needs to be resurrected.","accounts_in_message":[],"_revision_number":1},{"id":"2a1eba474cf3cbde79f8e1c5da92709c69709f04","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-07 12:42:39.000000000","message":"Patch Set 1: Restored\n\nRestore changes after tree is open now.","accounts_in_message":[],"_revision_number":1},{"id":"b10ec8589dd888a765442afde7c57575d193ea56","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-07 12:43:19.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"1a7c073be9411f8dcd5f4bae6c1bde336e4b55a3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-07 13:28:58.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nBuild failed\n\n- http://logs.openstack.org/12694/2/check/gate-nova-merge/11579 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-docs/7313 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-pep8/12623 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python26/8764 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python27/8833 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-tempest-devstack-vm/17484 : FAILURE\n- http://logs.openstack.org/12694/2/check/gate-nova-pylint/2562 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"c72c8305f6c40498abeba8f8920ec8580ccb6cc2","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-11-07 15:12:08.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/44841\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/44839\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/44840\n","accounts_in_message":[],"_revision_number":2},{"id":"edbc340a732170a7053dfec989b2468de2d05c23","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-08 02:12:44.000000000","message":"Patch Set 2:\n\nRecheck","accounts_in_message":[],"_revision_number":2},{"id":"e2ea44ba78e53517db23a8db5df1fd7a7bff9fce","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-08 02:42:18.000000000","message":"Patch Set 2: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/2/check/gate-nova-merge/11615 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-docs/7349 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-pep8/12659 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python26/8800 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python27/8869 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-tempest-devstack-vm/17612 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-pylint/2589 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"528fbdacc55f1c2559d6ae69400896ba091f2688","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-08 02:58:37.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"604c06082fce369e2be89e9c0cd688ec3be26538","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-08 03:32:54.000000000","message":"Patch Set 2:\n\nBuild successful\n\n- http://logs.openstack.org/12694/2/check/gate-nova-merge/11617 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-docs/7351 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-pep8/12661 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python26/8802 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-python27/8871 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-tempest-devstack-vm/17616 : SUCCESS\n- http://logs.openstack.org/12694/2/check/gate-nova-pylint/2590 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"f533e19ea97f25e32bc0e77e5c521e69664a3f41","author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"date":"2012-11-14 00:39:51.000000000","message":"Patch Set 2: (1 inline comment)\n\nI assume OAT stands for Open Attestation  Service, but that is not mentioned anywhere in the commit msg or the code.","accounts_in_message":[],"_revision_number":2},{"id":"4426e35cc7c4b815ea117369b380b26abe64cbf6","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-14 01:03:50.000000000","message":"Patch Set 2:\n\nJoe, thanks for the review. Yes, I need rebase the patch for the flag/conf changes and also update the commit message.","accounts_in_message":[],"_revision_number":2},{"id":"e04881dbfbf1a5f7b3098d211d8019004f733d93","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-14 11:49:29.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"fb085d578d087d59747ad5e5b7254a20656cb83c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-14 12:12:27.000000000","message":"Patch Set 3: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/3/check/gate-nova-merge/12012 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-nova-docs/7745 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-nova-pep8/13055 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-nova-python26/9196 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-nova-python27/9265 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-tempest-devstack-vm/18416 : SUCCESS\n- http://logs.openstack.org/12694/3/check/gate-nova-pylint/2889 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"1f8c4283912dc928d849f1f5ea6f4a3d7abbaa84","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-11-14 12:19:10.000000000","message":"Patch Set 3: Works for me\n\nSmokeStack Results (patch set 3):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/47063\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/47061\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/47062\n","accounts_in_message":[],"_revision_number":3},{"id":"faa036880fd288515942832a43c541efb1506f35","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2012-11-20 17:01:20.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nI think the time handling in this needs to be cleaned up a bit, I\u0027m concerned in the tz jumping this is going to just go funny. I\u0027d also encourage extensive additional testing of that.","accounts_in_message":[],"_revision_number":3},{"id":"b06b956d1199b67c4bbc8c5001294b47fc256d4f","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-22 09:11:27.000000000","message":"Patch Set 3: (1 inline comment)\n\nThanks for review. \nFor TZ handling, as all time is normalized, it should be ok. But it\u0027s a good idea to add test case for the TZ situation and I will add a test case.","accounts_in_message":[],"_revision_number":3},{"id":"fefb0fe2f6f1ae5c85c79d120429b2b976c6b80f","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-22 09:51:16.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"af662650eac3399f69494eaa4ccee271f359d30f","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-11-22 09:55:12.000000000","message":"Patch Set 4:\n\nSean Dague, I think I get your point. Yes, the originally code does not does not include \"Z\" in the end of the time. Fixed in the new patchset. For the vtime from OAT, it\u0027s ISO format per the OAT service requirement and should be ok.\n\nNew patchset also add a new test case to cover the timezone situation.","accounts_in_message":[],"_revision_number":4},{"id":"346662a91ebd3e9c151788686f3bb0d0322f5552","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-22 10:15:23.000000000","message":"Patch Set 4: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/4/check/gate-nova-merge/12769 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-nova-docs/8500 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-nova-pep8/13810 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-nova-python26/9950 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-nova-python27/10020 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-tempest-devstack-vm/19727 : SUCCESS\n- http://logs.openstack.org/12694/4/check/gate-nova-pylint/3464 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"4556473894f496453c635957d072794d7da50d1d","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-11-22 10:21:08.000000000","message":"Patch Set 4: Works for me\n\nSmokeStack Results (patch set 4):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/49803\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/49801\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/49802\n","accounts_in_message":[],"_revision_number":4},{"id":"256cdb8facd22fa27910348f2454e12c35d8be61","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2012-11-28 18:03:58.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\n(13 inline comments)\n\nLots of stylistic comments; see in-line.","accounts_in_message":[],"_revision_number":4},{"id":"6ac09b193fe76ea4298801ca66168ecf695c7e58","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2012-11-30 12:13:42.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nKevin\u0027s comments should be addressed.\n\nI\u0027m also not really sure why so much time manip is happening. Instead of doing that with arbitrary time in the past when this was first written, it would be good to make that relative to the time when the tests are run.","accounts_in_message":[],"_revision_number":4},{"id":"3c45823389e57f560c6cb4549db8ee52b5aaddc5","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-06 02:28:03.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"48779e319dacf12f58d11a4fef8ffacc351ec6d5","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-06 02:30:21.000000000","message":"Patch Set 5:\n\nThanks for review. All comments are addressed now.","accounts_in_message":[],"_revision_number":5},{"id":"94807319d5660fc16d0724e29fb218f139415d1f","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-12-06 02:46:10.000000000","message":"Patch Set 5: Works for me\n\nSmokeStack Results (patch set 5):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/53133\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/53131\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/53132\n","accounts_in_message":[],"_revision_number":5},{"id":"f2d0b45b1e9a00adeba9d703227c5ae7086f6918","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-12-06 02:54:08.000000000","message":"Patch Set 5: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/5/check/gate-nova-merge/13502 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-nova-docs/9220 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-nova-pep8/14530 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-nova-python26/10668 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-nova-python27/10740 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-tempest-devstack-vm/21239 : SUCCESS\n- http://logs.openstack.org/12694/5/check/gate-nova-pylint/3952 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":5},{"id":"44183011d5e485c18c813019cf45a87ad7e90da4","author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"date":"2012-12-06 14:27:53.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\nlgtm","accounts_in_message":[],"_revision_number":5},{"id":"daa560a6df63b11e66a374aac2d4309f509d4357","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2012-12-06 16:19:41.000000000","message":"Patch Set 5: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":5},{"id":"e52d4cd466a3a28cdcae78e40b9fd595266f7322","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-17 09:25:48.000000000","message":"Patch Set 5: (2 inline comments)\n\nSorry for slow response because busy on another patch. See inline comments please.","accounts_in_message":[],"_revision_number":5},{"id":"40f2d7d7aef5b420f12c6e941e839602d89441a9","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-17 09:27:33.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"810331a17788a4d05ed19a25a2bfc87fcdd38127","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-12-17 09:50:32.000000000","message":"Patch Set 6: Doesn\u0027t seem to work\n\nBuild failed\n\n- http://logs.openstack.org/12694/6/check/gate-nova-merge/14098 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-docs/9816 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-pep8/15126 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-python26/11264 : FAILURE\n- http://logs.openstack.org/12694/6/check/gate-nova-python27/11336 : FAILURE\n- http://logs.openstack.org/12694/6/check/gate-tempest-devstack-vm/22508 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-pylint/4415 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":6},{"id":"3a7e5451c5d431f47ddd74b809b54e57d0085363","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-12-17 16:19:58.000000000","message":"Patch Set 6: Works for me\n\nSmokeStack Results (patch set 6):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/55648\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/55646\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/55647\n","accounts_in_message":[],"_revision_number":6},{"id":"538530bedc9d5efc0389987cbe328d13f4571ab4","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2012-12-17 16:44:57.000000000","message":"Patch Set 6: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":6},{"id":"09a735873e7550dc2158203124a29fe1562388ca","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-18 00:29:56.000000000","message":"Patch Set 6: (1 inline comment)\n\nKevin, thanks.","accounts_in_message":[],"_revision_number":6},{"id":"973acee452fbb5e52f569e87032edad0415bd42d","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-18 00:30:02.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"b3c9d944013ca5757ae376ab1133d8d36eac142a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-12-18 00:55:31.000000000","message":"Patch Set 6: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/6/check/gate-nova-docs/9883 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-pep8/15193 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-python26/11331 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-python27/11403 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-tempest-devstack-vm/22681 : SUCCESS\n- http://logs.openstack.org/12694/6/check/gate-nova-pylint/4467 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":6},{"id":"1ad82770adfebc1cb58dbfce39ffb72762accc7a","author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"date":"2012-12-28 01:01:14.000000000","message":"Patch Set 6:\n\nTesting shows that filters are reinitialized for each VM that is scheduled.  Meaning that as far as I can tell adding a cache here wouldn\u0027t reduce the amount of calls to the OAT service.","accounts_in_message":[],"_revision_number":6},{"id":"32d3c17508761f59fb5b837e9b81115821e8faf7","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-28 01:27:50.000000000","message":"Patch Set 6:\n\nJoe, thanks for the review. \n\nIt\u0027s known that the cache is only valid for one VM instance scheduling. However, without this cache support, we need consult the OAT service for each host checking even in the one VM instance scheduling. This is a big impact.\n\nWe also discussed internally if we need global cache and the conclusion is global cache has benefit but also has side effect, so not support it at least in the first step. Also the OAT service itself can be enhanced to keep cache, so that the global cache benefit in openstack will be small.","accounts_in_message":[],"_revision_number":6},{"id":"f1a9fbf4b2b778918491081a0e4d122dd6326117","author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"date":"2012-12-28 01:55:37.000000000","message":"Patch Set 6:\n\nIf the cache is only used for scheduling a single VM, why have a \u0027attestation_auth_timeout\u0027?\n\nFurthermore could you clarify why this helps scheduling? Don\u0027t you still have the same number of calls to OAT?\n\nAnd lastly what is the benefit of having the caching logic here instead of OAT?","accounts_in_message":[],"_revision_number":6},{"id":"f1fb729895c2db681a010cbc09977d174a0ae914","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-28 02:10:48.000000000","message":"Patch Set 6:\n\n\u003e If the cache is only used for scheduling a single VM, why have a \u0027attestation_auth_timeout\u0027?\n\nIf the host farm is large enough (like \u003e1k), we will possibly hit the timeout even for one VM scheduling.\n\n\u003e Furthermore could you clarify why this helps scheduling? Don\u0027t you still have the same number of calls to OAT?\n\nCurrently, the host_passes() will be invoked for each host in a single VM scheduling, and for each invocation, we have to consult the OAT. With this cache, we get all hosts in the host farm during the __init__() of ComputeAttestationCache , thus, in the first consult to the OAT, we will fetch back the attestation status for all host, thus don\u0027t need consult OAT services anymore for followed host_passes() invocation.\n\n\n\u003e And lastly what is the benefit of having the caching logic here instead of OAT?\n\nYes, OAT can have cache also, but that\u0027s a separated story to this one. The OAT can have cache support to enhance the performance, while OAT can have cache also to reduce in OAT side.","accounts_in_message":[],"_revision_number":6},{"id":"d7bd25a747b5fc11a78a7435a6de4e4a34d6c03f","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-28 02:11:48.000000000","message":"Patch Set 6:\n\nFor the last sentence in previous comments, I mean\"\nThe OAT can have cache support to enhance the performance, while Openstack can have cache also to reduce in Openstack side\"","accounts_in_message":[],"_revision_number":6},{"id":"d43e2e7bd08ddbd50125b8a2be7887f0c6573323","author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"date":"2012-12-28 02:26:07.000000000","message":"Patch Set 6: I would prefer that you didn\u0027t merge this\n\nAhh I see,  so you are batching the calls to OAT.  Instead of N requests for one machine each, you have 1 request for N machines.\n\nLGTM,   The commit msg could be clearer though.","accounts_in_message":[],"_revision_number":6},{"id":"f1e766f1184f6700f18ed9dd8cf4534bb0ef482c","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-28 02:45:41.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"dd9b5e4505b9b4dd5808a2330124f65d88bcfb9c","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2012-12-28 02:47:55.000000000","message":"Patch Set 7:\n\nThanks for the review and suggestion. I updated the commit message accordingly.","accounts_in_message":[],"_revision_number":7},{"id":"41f1aec379b15faeddbc86e9de49c47d4b06df46","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-12-28 03:08:08.000000000","message":"Patch Set 7: Works for me\n\nSmokeStack Results (patch set 7):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/57721\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/57719\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/57720\n","accounts_in_message":[],"_revision_number":7},{"id":"c19525da0c271f826fead6ba6e0781ec590860c8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-12-28 03:19:21.000000000","message":"Patch Set 7: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/7/check/gate-nova-docs/10423 : SUCCESS\n- http://logs.openstack.org/12694/7/check/gate-nova-pep8/15733 : SUCCESS\n- http://logs.openstack.org/12694/7/check/gate-nova-python26/11871 : SUCCESS\n- http://logs.openstack.org/12694/7/check/gate-nova-python27/11943 : SUCCESS\n- http://logs.openstack.org/12694/7/check/gate-tempest-devstack-vm/23639 : SUCCESS\n- http://logs.openstack.org/12694/7/check/gate-devstack-vm-quantum/77 : SUCCESS (non-voting)\n- http://logs.openstack.org/12694/7/check/gate-nova-pylint/4901 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":7},{"id":"eef8be21ba1060b6c925700693935023c06368a8","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-01-04 20:45:47.000000000","message":"Patch Set 7: I would prefer that you didn\u0027t merge this\n\n(3 inline comments)\n\nI have grave concerns about the use of global variables in test_host_filters.py; please see inline comments.","accounts_in_message":[],"_revision_number":7},{"id":"3767928a52d1d61009c6052695f1676f3f110d0c","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2013-01-05 07:32:54.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"d44928567872019d67de0a1efdbd1c29fce225e2","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2013-01-05 07:38:13.000000000","message":"Patch Set 7: (3 inline comments)\n\nKevin, thanks for your review and I have updated the patch accordingly. A new patch is created to remove the DATA global variable.","accounts_in_message":[],"_revision_number":7},{"id":"3b070691bd1b76da312182cb1869bcbacfd21bc4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-05 07:57:33.000000000","message":"Patch Set 8: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12694/8/check/gate-nova-docs/10940 : SUCCESS\n- http://logs.openstack.org/12694/8/check/gate-nova-pep8/16250 : SUCCESS\n- http://logs.openstack.org/12694/8/check/gate-nova-python26/12402 : SUCCESS\n- http://logs.openstack.org/12694/8/check/gate-nova-python27/12460 : SUCCESS\n- http://logs.openstack.org/12694/8/check/gate-tempest-devstack-vm/24467 : SUCCESS\n- http://logs.openstack.org/12694/8/check/gate-devstack-vm-quantum/1005 : FAILURE (non-voting)\n- http://logs.openstack.org/12694/8/check/gate-nova-pylint/5288 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":8},{"id":"79c0eba31a762618da5d2b2613515db1314b96dd","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-01-07 16:52:36.000000000","message":"Patch Set 7: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":7},{"id":"3076c210092fd032f0b3f9dc74f03721387d2f6d","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-01-07 16:54:34.000000000","message":"Patch Set 8: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":8},{"id":"a27dd61f9f9bdd3687900975d8c516d4382586b0","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-01-22 06:03:01.000000000","message":"Patch Set 8: Abandoned\n\ncode review expired after 2 weeks of no activity, it can be restored using the `Restore Change` button  under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":8},{"id":"24c2165d42d7f8776fc0b09c2b958739f8bdd6e7","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-01-22 06:03:01.000000000","message":"Patch Set 8:\n\ncode review expired after 2 weeks of no activity, it can be restored using the `Restore Change` button  under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":8},{"id":"b4229fc4514aa85c98200c3058f94fe0569b6cef","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2013-01-22 06:04:47.000000000","message":"Patch Set 8: Restored\n\nCan anyone help to review the patch?","accounts_in_message":[],"_revision_number":8},{"id":"665303bb012553fb3e2eb806a05cb97e375f5b2c","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-01-22 07:08:35.000000000","message":"Patch Set 8: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":8},{"id":"de0e8f2515d690a8fdf9a8109fdc30bfe97a9bc0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 07:08:37.000000000","message":"Patch Set 8: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":8},{"id":"4a54c1c515f0ffcdf9abda5b62609df85ad41065","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 07:41:51.000000000","message":"Patch Set 8: Fails\n\nBuild failed.  For information on how to proceed, see http://wiki.openstack.org/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/12694/8/gate/gate-nova-docs/12890 : SUCCESS\n- http://logs.openstack.org/12694/8/gate/gate-nova-pep8/18196 : FAILURE\n- http://logs.openstack.org/12694/8/gate/gate-nova-python26/14354 : SUCCESS\n- http://logs.openstack.org/12694/8/gate/gate-nova-python27/14404 : SUCCESS\n- http://logs.openstack.org/12694/8/gate/gate-tempest-devstack-vm/27708 : SUCCESS\n- http://logs.openstack.org/12694/8/gate/gate-tempest-devstack-vm-quantum/2063 : SUCCESS\n","accounts_in_message":[],"_revision_number":8},{"id":"f3e4e7045f7ab7f09522faf16f1c64553d67dde2","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2013-01-22 09:52:40.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"2fffabb0ab8ef5589f8c16797392d0113f495cf6","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2013-01-22 09:52:54.000000000","message":"Patch Set 9:\n\nNew patchset patch-id matches previous patchset, but whitespace content has changed.","accounts_in_message":[],"_revision_number":9},{"id":"d5c67666b20ea64d7b0ffd78ff9b6679de97a3d6","author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"date":"2013-01-22 10:07:14.000000000","message":"Patch Set 9:\n\nOnly remove one white space to meet PEP8 requirement.","accounts_in_message":[],"_revision_number":9},{"id":"6901970efc180aecad87f50a64a097c13b20066c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 10:22:07.000000000","message":"Patch Set 9: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/12694/9/check/gate-nova-docs/12895 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-nova-pep8/18201 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-nova-python26/14359 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-nova-python27/14409 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-tempest-devstack-vm/27721 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-tempest-devstack-vm-quantum/2080 : SUCCESS\n- http://logs.openstack.org/12694/9/check/gate-nova-pylint/6711 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":9},{"id":"951dd0778bd3f0ea0d01df5d644161532bb636a9","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-01-22 10:44:16.000000000","message":"Patch Set 9: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":9},{"id":"fd8fc859d3e4938186817f764714e8e842deca20","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 10:44:18.000000000","message":"Patch Set 9: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":9},{"id":"5b0945e035f8a27d46ba1aa0ffa255b805509dc0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 11:18:18.000000000","message":"Patch Set 9: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/12694/9/gate/gate-nova-docs/12899 : SUCCESS\n- http://logs.openstack.org/12694/9/gate/gate-nova-pep8/18205 : SUCCESS\n- http://logs.openstack.org/12694/9/gate/gate-nova-python26/14363 : SUCCESS\n- http://logs.openstack.org/12694/9/gate/gate-nova-python27/14413 : SUCCESS\n- http://logs.openstack.org/12694/9/gate/gate-tempest-devstack-vm/27726 : SUCCESS\n- http://logs.openstack.org/12694/9/gate/gate-tempest-devstack-vm-quantum/2087 : SUCCESS\n","accounts_in_message":[],"_revision_number":9},{"id":"876f3c23a58d0c3d87436316e851fbec472a170a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-01-22 11:18:21.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":9}],"current_revision_number":9,"current_revision":"fab8af583bf6c363d2cebbc360ae2709325d80bd","revisions":{"2296bdafe5001ef2312762e0ccfa3932a0e2f161":{"kind":"REWORK","_number":1,"created":"2012-09-10 11:59:57.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/1"}}},"commit":{"parents":[{"commit":"28a5b31bdbff7ed4492f8a6ca8e7e53a7aba0893","subject":"Merge \"Specify the conf file when creating a volume.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/28a5b31bdbff7ed4492f8a6ca8e7e53a7aba0893"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 11:59:03.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls OAT service to get the trusted level for the host. This solution is\nnot good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2296bdafe5001ef2312762e0ccfa3932a0e2f161"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/2296bdafe5001ef2312762e0ccfa3932a0e2f161"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"28a5b31bdbff7ed4492f8a6ca8e7e53a7aba0893","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"9872c55cda681880159433c449ca157f71e02dcf":{"kind":"REWORK","_number":2,"created":"2012-11-07 12:43:19.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/2"}}},"commit":{"parents":[{"commit":"562b3abc843942a096e5750161c1aae2402d595b","subject":"Merge \"libvirt: Regenerates xml instead of using on-disk\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/562b3abc843942a096e5750161c1aae2402d595b"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-11-07 12:27:21.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls OAT service to get the trusted level for the host. This solution is\nnot good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9872c55cda681880159433c449ca157f71e02dcf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9872c55cda681880159433c449ca157f71e02dcf"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"562b3abc843942a096e5750161c1aae2402d595b","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"1a82b372bb4e5672b1b0e75ec76efa2b70492c68":{"kind":"REWORK","_number":3,"created":"2012-11-14 11:49:29.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/3"}}},"commit":{"parents":[{"commit":"aa2bb87a5e28cb19a0e9244807daacb16f59ce03","subject":"Merge \"Update vol mount smoketest to wait for volume.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/aa2bb87a5e28cb19a0e9244807daacb16f59ce03"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-11-14 11:41:30.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a82b372bb4e5672b1b0e75ec76efa2b70492c68"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1a82b372bb4e5672b1b0e75ec76efa2b70492c68"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"aa2bb87a5e28cb19a0e9244807daacb16f59ce03","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"8cb7f719ff4d5942c92ecc7d2809094d602f1102":{"kind":"REWORK","_number":4,"created":"2012-11-22 09:51:16.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/4"}}},"commit":{"parents":[{"commit":"b5c104bc7d5c3a6e5ee953e86416b575a34e0da3","subject":"Merge \"Add a CONTRIBUTING file.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b5c104bc7d5c3a6e5ee953e86416b575a34e0da3"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-11-22 09:48:46.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8cb7f719ff4d5942c92ecc7d2809094d602f1102"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/8cb7f719ff4d5942c92ecc7d2809094d602f1102"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"b5c104bc7d5c3a6e5ee953e86416b575a34e0da3","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"9cdb0af47277c326434353768a000d63c40a0d7e":{"kind":"REWORK","_number":5,"created":"2012-12-06 02:28:03.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/5"}}},"commit":{"parents":[{"commit":"4d648c4b3d480dc3cc2dbb4d77f561f0bd4de426","subject":"Merge \"Ensure prep_resize arguments can be serialized.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4d648c4b3d480dc3cc2dbb4d77f561f0bd4de426"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-12-06 02:17:26.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9cdb0af47277c326434353768a000d63c40a0d7e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9cdb0af47277c326434353768a000d63c40a0d7e"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"4d648c4b3d480dc3cc2dbb4d77f561f0bd4de426","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"516cb25a3ee0aecd7f977980a626c881c57d5861":{"kind":"REWORK","_number":6,"created":"2012-12-17 09:27:33.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/6"}}},"commit":{"parents":[{"commit":"f04722ee22feed97aca23cd16864626d9672671c","subject":"Merge \"Ensure that sql_dbpool_enable is a boolean value\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f04722ee22feed97aca23cd16864626d9672671c"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-12-17 09:21:08.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability.\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/516cb25a3ee0aecd7f977980a626c881c57d5861"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/516cb25a3ee0aecd7f977980a626c881c57d5861"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f04722ee22feed97aca23cd16864626d9672671c","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"afabf91659e5f750a15845e3252f1b1db0f8c277":{"kind":"TRIVIAL_REBASE_WITH_MESSAGE_UPDATE","_number":7,"created":"2012-12-28 02:45:41.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/7"}}},"commit":{"parents":[{"commit":"bde5420564e9abfc656b48af1798eabd65b30234","subject":"Merge \"Add API samples tests for the coverage extension.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/bde5420564e9abfc656b48af1798eabd65b30234"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-12-28 02:41:32.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability. The cache is setup when the first time the filter consulting\nthe OAT service.\n    The trusted_filters are reinitialized for each VM that is\nscheduled, thus the cache is valid only for one VM instance scheduing.\n\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/afabf91659e5f750a15845e3252f1b1db0f8c277"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/afabf91659e5f750a15845e3252f1b1db0f8c277"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"bde5420564e9abfc656b48af1798eabd65b30234","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"0a8ee920bed6f0e0dcbf477ec7a8513df1748b7c":{"kind":"REWORK","_number":8,"created":"2013-01-05 07:32:54.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/8"}}},"commit":{"parents":[{"commit":"1e117ab425c27805f78b976311728f05782fd5bf","subject":"Remove the global DATA","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1e117ab425c27805f78b976311728f05782fd5bf"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2013-01-05 07:26:24.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability. The cache is setup when the first time the filter consulting\nthe OAT service.\n    The trusted_filters are reinitialized for each VM that is\nscheduled, thus the cache is valid only for one VM instance scheduing.\n\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0a8ee920bed6f0e0dcbf477ec7a8513df1748b7c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0a8ee920bed6f0e0dcbf477ec7a8513df1748b7c"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"1e117ab425c27805f78b976311728f05782fd5bf","is_merged_in_target_branch":true,"change_id":"Id75077575917e501d51f83d3c4b1c4250fbc84d0","change_number":19029,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"fab8af583bf6c363d2cebbc360ae2709325d80bd":{"kind":"REWORK","_number":9,"created":"2013-01-22 09:52:40.000000000","uploader":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"ref":"refs/changes/94/12694/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/94/12694/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/94/12694/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/94/12694/9"}}},"commit":{"parents":[{"commit":"a4d608fa33b328d7ed77c7f9c40ffbb43c0ade6b","subject":"Merge \"Directly copy a file URL from glance.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a4d608fa33b328d7ed77c7f9c40ffbb43c0ade6b"}]}],"author":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2012-09-10 08:20:15.000000000","tz":480},"committer":{"name":"Yunhong, Jiang","email":"yunhong.jiang@intel.com","date":"2013-01-22 09:49:43.000000000","tz":480},"subject":"Add trust level cache to trusted_filter","message":"Add trust level cache to trusted_filter\n\n    Currently each time the trusted filter is called to check host_pass, it\npolls Open Attestation Service to get the trusted level for the host. This\nsolution is not good on scalability.\n    With a cache for the host trust level, trusted filter don\u0027t need to\nconsult OAT service if the cache is still valid, thus improves the\nscalability. The cache is setup when the first time the filter consulting\nthe OAT service.\n    The trusted_filters are reinitialized for each VM that is\nscheduled, thus the cache is valid only for one VM instance scheduing.\n\n    Implements blueprint trusted-filter-cache\n\nChange-Id: I613da53876e6cd548566e27ab1693287023e9861\nSigned-off-by: Yunhong, Jiang \u003cyunhong.jiang@intel.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fab8af583bf6c363d2cebbc360ae2709325d80bd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fab8af583bf6c363d2cebbc360ae2709325d80bd"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a4d608fa33b328d7ed77c7f9c40ffbb43c0ade6b","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}