)]}'
{"id":"openstack%2Fdevstack~184568","triplet_id":"openstack%2Fdevstack~master~I44e4490909a117930b1883dac859e0213827c834","project":"openstack/devstack","branch":"master","topic":"bug/1456428","hashtags":[],"change_id":"I44e4490909a117930b1883dac859e0213827c834","subject":"Passing correct tenant/user vars to context log","status":"ABANDONED","created":"2015-05-20 18:02:30.000000000","updated":"2015-12-11 13:06:32.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"71ae3df6a95c8a75cc84fbfc994cb0c150b1542c","_number":184568,"virtual_id_number":184568,"owner":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2015-05-21 06:09:33.000000000","_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},{"value":1,"date":"2015-05-20 19:57:29.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"date":"2015-05-20 18:56:22.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"date":"2015-05-20 19:01:29.000000000","_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-12-11 04:11:15.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":-1,"date":"2015-05-21 06:38:22.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true}],"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":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},{"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":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"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":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true}],"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":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},{"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-05-20 18:56:22.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2015-05-20 19:57:29.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2015-05-21 06:09:33.000000000","updated_by":{"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},"reviewer":{"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},"state":"REVIEWER"},{"updated":"2015-05-21 06:38:22.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-12-11 04:11:15.000000000","updated_by":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"reviewer":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"state":"REVIEWER"}],"messages":[{"id":"7e78222f01270a904b979e8306bb02053e5756f6","author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"date":"2015-05-20 18:02:30.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"242dab5d619570f968ffdaa207e8d45fb0717764","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-05-20 18:55:31.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/68/184568/1/check/check-tempest-dsvm-full/03c5c45/ : SUCCESS in 37m 46s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-postgres-full/e5b1a97/ : SUCCESS in 37m 14s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-neutron-full/361e7ef/ : FAILURE in 52m 52s\n- check-grenade-dsvm http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm/ad34542/ : SUCCESS in 44m 19s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/68/184568/1/check/gate-tempest-dsvm-large-ops/e67d420/ : SUCCESS in 30m 24s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/68/184568/1/check/gate-tempest-dsvm-neutron-large-ops/3ddb4f9/ : SUCCESS in 22m 12s\n- gate-devstack-bashate http://logs.openstack.org/68/184568/1/check/gate-devstack-bashate/3900cf1/ : SUCCESS in 18s\n- gate-devstack-unit-tests http://logs.openstack.org/68/184568/1/check/gate-devstack-unit-tests/c6172c0/ : SUCCESS in 33s\n- check-devstack-dsvm-cells http://logs.openstack.org/68/184568/1/check/check-devstack-dsvm-cells/79a6ec0/ : SUCCESS in 20m 50s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm-partial-ncpu/f83d4e4/ : SUCCESS in 38m 38s\n- check-swift-dsvm-functional http://logs.openstack.org/68/184568/1/check/check-swift-dsvm-functional/9b64fea/ : SUCCESS in 15m 58s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-ironic-pxe_ssh/4fc8149/ : SUCCESS in 36m 13s\n- check-devstack-dsvm-updown http://logs.openstack.org/68/184568/1/check/check-devstack-dsvm-updown/2c8c734/ : SUCCESS in 17m 35s\n- check-tempest-dsvm-f21 http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-f21/518abde/ : FAILURE in 42m 36s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-centos7/5129647/ : FAILURE in 48m 19s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/68/184568/1/check/gate-devstack-publish-docs/f1bad22//doc/build/html/ : SUCCESS in 1m 40s\n- check-grenade-dsvm-neutron http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm-neutron/3eb7bfd/ : SUCCESS in 44m 46s","accounts_in_message":[],"_revision_number":1},{"id":"3afd5a1cb83a560665fd60a68419691f3f8ee901","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2015-05-20 18:56:22.000000000","message":"Patch Set 1:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/1456771\n\nIf you believe we\u0027ve correctly identified the failure, feel free to leave a \u0027recheck\u0027 comment to run the tests again.\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":1},{"id":"d5b18302f48f47b1c864f1c156f6c2394c195cd8","author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"date":"2015-05-20 19:01:29.000000000","message":"Patch Set 1:\n\nrecheck","accounts_in_message":[],"_revision_number":1},{"id":"f4d29c0086e9a3cd891d9e78e3905b0fd4a6ce23","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-05-20 19:57:29.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-full/8178df7/ : SUCCESS in 46m 47s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-postgres-full/3d973eb/ : SUCCESS in 47m 01s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-neutron-full/b85a42e/ : SUCCESS in 55m 48s\n- check-grenade-dsvm http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm/db64bf2/ : SUCCESS in 40m 43s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/68/184568/1/check/gate-tempest-dsvm-large-ops/5582bc7/ : SUCCESS in 20m 50s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/68/184568/1/check/gate-tempest-dsvm-neutron-large-ops/60ebc0c/ : SUCCESS in 19m 29s\n- gate-devstack-bashate http://logs.openstack.org/68/184568/1/check/gate-devstack-bashate/5d649c5/ : SUCCESS in 12s\n- gate-devstack-unit-tests http://logs.openstack.org/68/184568/1/check/gate-devstack-unit-tests/3affea9/ : SUCCESS in 14s\n- check-devstack-dsvm-cells http://logs.openstack.org/68/184568/1/check/check-devstack-dsvm-cells/64e0985/ : SUCCESS in 21m 58s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm-partial-ncpu/82a0bd2/ : SUCCESS in 36m 56s\n- check-swift-dsvm-functional http://logs.openstack.org/68/184568/1/check/check-swift-dsvm-functional/71e815b/ : SUCCESS in 21m 53s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-ironic-pxe_ssh/49a47f0/ : SUCCESS in 29m 20s\n- check-devstack-dsvm-updown http://logs.openstack.org/68/184568/1/check/check-devstack-dsvm-updown/288a35a/ : SUCCESS in 21m 16s\n- check-tempest-dsvm-f21 http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-f21/510ffd1/ : FAILURE in 43m 58s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/68/184568/1/check/check-tempest-dsvm-centos7/4ef90a5/ : FAILURE in 46m 17s (non-voting)\n- gate-devstack-publish-docs http://docs-draft.openstack.org/68/184568/1/check/gate-devstack-publish-docs/03ca363//doc/build/html/ : SUCCESS in 1m 39s\n- check-grenade-dsvm-neutron http://logs.openstack.org/68/184568/1/check/check-grenade-dsvm-neutron/8452530/ : SUCCESS in 42m 23s","accounts_in_message":[],"_revision_number":1},{"id":"b36208dda4327f020305be7448417f60776512e4","author":{"_account_id":11042,"name":"chenglch","email":"chenglch@cn.ibm.com","username":"chenglch"},"date":"2015-05-21 06:09:33.000000000","message":"Patch Set 1:\n\nThanks Thiago Paiva Brito, I think modify the context structure in ironic may be a better choice.","accounts_in_message":[],"_revision_number":1},{"id":"bfc493a8ac805e0883d13a7388ea7ed525126c02","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2015-05-21 06:38:22.000000000","message":"Patch Set 1: Code-Review-1\n\nIt would be best to have ironic\u0027s context object subclass from the one in oslo.context to provide the values using the names expected by the defaults in oslo.log instead of changing the configuration here.","accounts_in_message":[],"_revision_number":1},{"id":"a3efb53bae64b7194541730eba262909ae22a218","author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"date":"2015-05-21 12:18:48.000000000","message":"Patch Set 1:\n\nShouldn\u0027t we be unifying the context object on oslo_context instead of creating one for each service just for logging reasons? The approach I used was the same other services are using (Glance, Heat, Cinder) and is using a variable already contained in oslo_context RequestContext instead of adding possibly duplicated data.","accounts_in_message":[],"_revision_number":1},{"id":"e33f2e2e69781c3503a04cdd70fcbb5bdebd6fd9","author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"date":"2015-12-11 04:11:15.000000000","message":"Patch Set 1: Code-Review-1\n\nWe\u0027re in the middle of moving ironic\u0027s devstack code to a plugin in ironic\u0027s tree. Could you please re-propose this patch there? https://review.openstack.org/#/c/255879/","accounts_in_message":[],"_revision_number":1},{"id":"749a56b35d4f668de6d0701633a75ac87283b12e","author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"date":"2015-12-11 13:06:32.000000000","message":"Abandoned\n\nHey Jim!\n\nIt appears that someone has already done that on the Ironic plugin. Abandoning this then.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"f70d89768bec9fee355d0c5c0d3a3c1882e67e1c","revisions":{"f70d89768bec9fee355d0c5c0d3a3c1882e67e1c":{"kind":"REWORK","_number":1,"created":"2015-05-20 18:02:30.000000000","uploader":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"ref":"refs/changes/68/184568/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/68/184568/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/184568/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/184568/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/68/184568/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/68/184568/1"}}},"commit":{"parents":[{"commit":"0b1b6470968380ca0ad7f3f2d944fbd9f781786f","subject":"Merge \"Add new options to baremetal config section\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/0b1b6470968380ca0ad7f3f2d944fbd9f781786f"}]}],"author":{"name":"Thiago Paiva","email":"thiagop@lsd.ufcg.edu.br","date":"2015-05-20 17:59:46.000000000","tz":-180},"committer":{"name":"Thiago Paiva","email":"thiagop@lsd.ufcg.edu.br","date":"2015-05-20 17:59:46.000000000","tz":-180},"subject":"Passing correct tenant/user vars to context log","message":"Passing correct tenant/user vars to context log\n\nCurrently the deployment of devstack + Ironic is breaking due to\nchanges to comply with oslo_context and the log formatting string\nthat expects, by default, user_name and project_name in the\ncontext. This patch aims to pass the current variables in the\ncontext of Ironic to generate a correct log formatting string.\n\nChange-Id: I44e4490909a117930b1883dac859e0213827c834\nCloses-bug: 1456428\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f70d89768bec9fee355d0c5c0d3a3c1882e67e1c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f70d89768bec9fee355d0c5c0d3a3c1882e67e1c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}