)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"25dcd97e810e1f0382c877d77d85e07c22fc56fe","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  8b2b0b7 Use hacking import_exceptions for gettextutils._"},{"line_number":19,"context_line":"  6d0a6c3 Correct invalid docstrings"},{"line_number":20,"context_line":"  fe18bfa Remove keystone from default_log_levels default"},{"line_number":21,"context_line":"  13eb01c Adding domain to context and log"},{"line_number":22,"context_line":"  27b21bc Unify different names between Python2/3 with six.moves"},{"line_number":23,"context_line":"  12bcdb7 Remove vim header"},{"line_number":24,"context_line":"  a6f40ac Don\u0027t log to stdout when log_dir is set"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAUH%2F%2Bohg%3D","line":21,"updated":"2014-01-23 17:57:32.000000000","message":"Hmm, this probably needs to come out since we don\u0027t sync context.py and don\u0027t use it in nova, so the user_identity change in nova.conf.sample is probably wrong.\n\nhttps://review.openstack.org/#/c/55938/","commit_id":"d501dad038edb760c9a6cad174c58b988901aa41"},{"author":{"_account_id":8810,"name":"Francisco Demontiê dos Santos Junior","email":"demontie@lsd.ufcg.edu.br","username":"demontiejr"},"change_message_id":"3d659b53c3bae7df539858125f0d8e2990a3de51","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  a6f40ac Don\u0027t log to stdout when log_dir is set"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Note that change 13eb01c \"Adding domain to context and log\" was"},{"line_number":26,"context_line":"intentionally ommitted since it requires context.py to be synced and"},{"line_number":27,"context_line":"changes nova.conf.sample, but nova uses it\u0027s own context module so we"},{"line_number":28,"context_line":"don\u0027t want that change pulled in here."},{"line_number":29,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2Bhi0%3D","line":26,"updated":"2014-01-24 19:06:24.000000000","message":"s/ommitted/omitted","commit_id":"a38fccb2047f8dc54bd5e82bfc46a247c7e2809c"},{"author":{"_account_id":8810,"name":"Francisco Demontiê dos Santos Junior","email":"demontie@lsd.ufcg.edu.br","username":"demontiejr"},"change_message_id":"3d659b53c3bae7df539858125f0d8e2990a3de51","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Note that change 13eb01c \"Adding domain to context and log\" was"},{"line_number":26,"context_line":"intentionally ommitted since it requires context.py to be synced and"},{"line_number":27,"context_line":"changes nova.conf.sample, but nova uses it\u0027s own context module so we"},{"line_number":28,"context_line":"don\u0027t want that change pulled in here."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Change-Id: Iebfdf484b9fcac5dbb689c4f7ec4f43a082a50c9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2Bhio%3D","line":28,"updated":"2014-01-24 19:06:24.000000000","message":"I wouldn\u0027t say this is the reason. Actually I\u0027ve proposed the change Ib11e0156 which would sync the one the you omitted. The reason why the change was not included is because the community decided that we would add the domain_id to the Nova context just when we have some code really using it.","commit_id":"a38fccb2047f8dc54bd5e82bfc46a247c7e2809c"},{"author":{"_account_id":8810,"name":"Francisco Demontiê dos Santos Junior","email":"demontie@lsd.ufcg.edu.br","username":"demontiejr"},"change_message_id":"bae70c33f5419ddeb1261b6b95d0526c32879fbb","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Note that change 13eb01c \"Adding domain to context and log\" was"},{"line_number":26,"context_line":"intentionally ommitted since it requires context.py to be synced and"},{"line_number":27,"context_line":"changes nova.conf.sample, but nova uses it\u0027s own context module so we"},{"line_number":28,"context_line":"don\u0027t want that change pulled in here."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Change-Id: Iebfdf484b9fcac5dbb689c4f7ec4f43a082a50c9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2BhMg%3D","line":28,"in_reply_to":"AAAAUH%2F%2Bhbo%3D","updated":"2014-01-24 20:15:54.000000000","message":"I understand your reason, I\u0027m just explaining about the patch. The context probably won\u0027t be synced, but when we need domain_id in the nova context we\u0027ll need add the information.\n\nAnyway, thanks for responding. I\u0027m gonna remove my -1.\nBut if you want, just correct the typo in the line 26. :)","commit_id":"a38fccb2047f8dc54bd5e82bfc46a247c7e2809c"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2b0bf886c4a76069930184db846691d08f8d1abe","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Note that change 13eb01c \"Adding domain to context and log\" was"},{"line_number":26,"context_line":"intentionally ommitted since it requires context.py to be synced and"},{"line_number":27,"context_line":"changes nova.conf.sample, but nova uses it\u0027s own context module so we"},{"line_number":28,"context_line":"don\u0027t want that change pulled in here."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Change-Id: Iebfdf484b9fcac5dbb689c4f7ec4f43a082a50c9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAUH%2F%2Bhbo%3D","line":28,"in_reply_to":"AAAAUH%2F%2Bhio%3D","updated":"2014-01-24 19:32:28.000000000","message":"Right, but until then, nova isn\u0027t using the oslo-incubator context, which is why I\u0027m leaving it out here because otherwise that change will modify nova.conf.sample but won\u0027t work since the context with the logging change isn\u0027t used. Make sense?","commit_id":"a38fccb2047f8dc54bd5e82bfc46a247c7e2809c"}]}
