)]}'
{"id":"openstack%2Fdevstack~180051","triplet_id":"openstack%2Fdevstack~master~Ifabd060540404eba1a02071b5cbf44ae2eb54fc9","project":"openstack/devstack","branch":"master","topic":"fix_sahara_log","hashtags":[],"change_id":"Ifabd060540404eba1a02071b5cbf44ae2eb54fc9","subject":"Fix logging_context_format_string input for sahara","status":"ABANDONED","created":"2015-05-05 07:56:37.000000000","updated":"2015-05-18 18:14:08.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"d3cdf037753f2fd69c5b111653631b58a378cb8c","_number":180051,"virtual_id_number":180051,"owner":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2015-05-18 18:14:08.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":-1,"date":"2015-05-05 18:55:26.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2015-05-06 01:05:06.000000000","_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"date":"2015-05-05 13:08:59.000000000","_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"}],"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":{"recommended":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"disliked":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},{"value":1,"date":"2015-05-05 08:55:50.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},{"value":1,"date":"2015-05-05 08:44:59.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"value":1,"date":"2015-05-05 11:42:48.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},{"value":-1,"date":"2015-05-05 12:28:57.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":-1,"date":"2015-05-05 14:01:36.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-05-05 08:44:59.000000000","updated_by":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"reviewer":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"state":"REVIEWER"},{"updated":"2015-05-05 08:55:50.000000000","updated_by":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"reviewer":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"state":"REVIEWER"},{"updated":"2015-05-05 11:42:48.000000000","updated_by":{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},"reviewer":{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},"state":"REVIEWER"},{"updated":"2015-05-05 12:28:57.000000000","updated_by":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"reviewer":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"state":"REVIEWER"},{"updated":"2015-05-05 13:08:59.000000000","updated_by":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"reviewer":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"state":"REVIEWER"},{"updated":"2015-05-05 14:01:36.000000000","updated_by":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"reviewer":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"state":"REVIEWER"},{"updated":"2015-05-05 18:55:26.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-05-18 18:14:08.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"d86c09ec2c6e97c9df1a7a40a5086c5a943194f8","author":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"date":"2015-05-05 07:56:37.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8d8a72210d0e0de1ba24aa6927d6814e8e745ac2","author":{"_account_id":12038,"name":"Vitaly Gridnev","email":"gridnevvvit@gmail.com","username":"vgridnev"},"date":"2015-05-05 08:44:59.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"69337ac41b0350850cc885cef49272f1c2d8382b","author":{"_account_id":7125,"name":"Alexander Ignatov","email":"aignatov@mirantis.com","username":"aignatov"},"date":"2015-05-05 08:55:50.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"a8564cff2da8f5a60c30391390ad5cfe568a7116","author":{"_account_id":12039,"name":"Andrey Pavlov","email":"apavlov@mirantis.com","username":"apavlov"},"date":"2015-05-05 11:42:48.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"ee535408a819429c950e1eda4d2bb67c8e817011","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2015-05-05 12:28:57.000000000","message":"Patch Set 1: Code-Review-1\n\nPlease update sahara to follow the logging conventions defined by the defaults for oslo.log instead -- set \"user_name\" in your context. There is a base class in oslo.context to define all of the expected settings. You can add more, to maintain compatibility with your existing code that uses \"username\", but please provide the minimum standard set of names so that deployers do not need to know  how to configure the logging instructions for different applications differently.","accounts_in_message":[],"_revision_number":1},{"id":"be809797eabfded94a24b7575446996910bba576","author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"date":"2015-05-05 13:08:59.000000000","message":"Patch Set 1:\n\nAgreed with Doug","accounts_in_message":[],"_revision_number":1},{"id":"2b7abeab99fd6fe99e05a92891d14535302d6de9","author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"date":"2015-05-05 14:01:36.000000000","message":"Patch Set 1: Code-Review-1\n\nagreed, unfortunate that it broke but long term we should make fixes on the sahara side to stay compatible with oslo defaults.  Much less chaotic, I think.","accounts_in_message":[],"_revision_number":1},{"id":"feae7d206d10ecaa767a03c330f426dbe0f7725c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-05-05 18:55:26.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- check-tempest-dsvm-full http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-full/26344c1/ : FAILURE in 16m 09s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-postgres-full/28c947a/ : FAILURE in 22m 12s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-neutron-full/ebfbc08/ : FAILURE in 19m 16s\n- check-grenade-dsvm http://logs.openstack.org/51/180051/1/check/check-grenade-dsvm/44b080d/ : FAILURE in 27m 30s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/51/180051/1/check/gate-tempest-dsvm-large-ops/26b5e86/ : SUCCESS in 30m 45s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/51/180051/1/check/gate-tempest-dsvm-neutron-large-ops/5e30396/ : SUCCESS in 25m 01s\n- gate-devstack-bashate http://logs.openstack.org/51/180051/1/check/gate-devstack-bashate/0b33c29/ : SUCCESS in 31s\n- gate-devstack-unit-tests http://logs.openstack.org/51/180051/1/check/gate-devstack-unit-tests/1d1ca4c/ : SUCCESS in 25s\n- check-devstack-dsvm-cells http://logs.openstack.org/51/180051/1/check/check-devstack-dsvm-cells/e4e09e8/ : FAILURE in 20m 57s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/51/180051/1/check/check-grenade-dsvm-partial-ncpu/acda7f7/ : FAILURE in 26m 21s\n- check-swift-dsvm-functional http://logs.openstack.org/51/180051/1/check/check-swift-dsvm-functional/4e3a211/ : FAILURE in 22m 50s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-ironic-pxe_ssh/f4dfede/ : FAILURE in 16m 57s\n- check-devstack-dsvm-updown http://logs.openstack.org/51/180051/1/check/check-devstack-dsvm-updown/e3a00cc/ : FAILURE in 17m 59s\n- check-tempest-dsvm-f21 http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-f21/a1202bb/ : NOT_REGISTERED (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/51/180051/1/check/check-tempest-dsvm-centos7/da1c70c/ : FAILURE in 43m 31s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/51/180051/1/check/gate-devstack-publish-docs/0d367d0//doc/build/html/ : SUCCESS in 1m 53s\n- check-grenade-dsvm-neutron http://logs.openstack.org/51/180051/1/check/check-grenade-dsvm-neutron/ae54467/ : FAILURE in 25m 05s","accounts_in_message":[],"_revision_number":1},{"id":"b1886941d521bbef7e72405bc1580049de1b398b","author":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"date":"2015-05-06 01:05:06.000000000","message":"Patch Set 1:\n\noslo_context only has \"user\" \u0026 \"tenant\":\n      \nhttps://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L55-L56\n\nIf the defaults for oslo.log using members already in oslo_context, things would be much easier, right ?","accounts_in_message":[],"_revision_number":1},{"id":"be15169043f1697b4fc1039115a94787f318ea59","author":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"date":"2015-05-06 01:08:09.000000000","message":"Patch Set 1:\n\nSubmitted fix at https://review.openstack.org/#/c/180372/","accounts_in_message":[],"_revision_number":1},{"id":"60b4a02648432a5c809d938bc4b1a6a542dd46e7","author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"date":"2015-05-07 12:40:56.000000000","message":"Patch Set 1:\n\noslo_context uses \"user\" and \"tenant\" fields but devstack uses \"user_name\" and \"project_name\" fields in logging. It\u0027s very strange.\n\nDoug, i think should change \u0027setup_colorized_logging\u0027 function.","accounts_in_message":[],"_revision_number":1},{"id":"4563bfbf60e4f569be9f0e44c73790ee1d56e053","author":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"date":"2015-05-12 05:15:32.000000000","message":"Patch Set 1:\n\nAs sahara already merged the related patch, and there are other changes related to logging_context_format_string under review, such as : https://review.openstack.org/#/c/172509/\n\nIt\u0027s time to abandon this patch no matter what members have been used by default for oslo.log.","accounts_in_message":[],"_revision_number":1},{"id":"8a8e6da13b645d5d5ea7f85fc85f41508b100723","author":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"date":"2015-05-12 05:15:48.000000000","message":"Abandoned\n\nAs sahara already merged the related patch, and there are other changes related to logging_context_format_string under review, such as : https://review.openstack.org/#/c/172509/\n\nIt\u0027s time to abandon this patch no matter what members have been used by default for oslo.log.","accounts_in_message":[],"_revision_number":1},{"id":"76b0ef1d017967ba4a13d8b5e3721960fd44562f","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2015-05-18 18:14:08.000000000","message":"Patch Set 1:\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/51/180051/1/16045/results.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"e25f610d995bed892c27e1c4d427d00d8c6d5172","revisions":{"e25f610d995bed892c27e1c4d427d00d8c6d5172":{"kind":"REWORK","_number":1,"created":"2015-05-05 07:56:37.000000000","uploader":{"_account_id":6116,"name":"li,chen","email":"shchenli@cn.ibm.com","username":"chen-li"},"ref":"refs/changes/51/180051/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/51/180051/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/51/180051/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/51/180051/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/51/180051/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/51/180051/1"}}},"commit":{"parents":[{"commit":"e210d26a1944ce5e2e8e41201496d93a3d351ddc","subject":"Merge \"clean up logging around run_process\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e210d26a1944ce5e2e8e41201496d93a3d351ddc"}]}],"author":{"name":"Li, Chen","email":"chen.li@intel.com","date":"2015-05-05 07:54:00.000000000","tz":480},"committer":{"name":"Li, Chen","email":"chen.li@intel.com","date":"2015-05-05 07:57:45.000000000","tz":480},"subject":"Fix logging_context_format_string input for sahara","message":"Fix logging_context_format_string input for sahara\n\nChange-Id: Ifabd060540404eba1a02071b5cbf44ae2eb54fc9\nCloses-Bug: #1451631\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e25f610d995bed892c27e1c4d427d00d8c6d5172"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/e25f610d995bed892c27e1c4d427d00d8c6d5172"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}