)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23235,"name":"Lihi Wishnitzer","email":"lihiwish@gmail.com","username":"lihiwish"},"change_message_id":"03ecbcb34d5ae6cf9db436803e03c99abdfebf83","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I37889dcb232f3c1557f241f82ade41520eb2de4e"},{"line_number":21,"context_line":"Co-Authored-By: Omer Anson \u003comer.anson@toganetworks.com\u003e"},{"line_number":22,"context_line":"Related-Bug: #1744558"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7f96bb07_706c239c","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":11},"updated":"2018-01-22 15:17:39.000000000","message":"Closes-bug?","commit_id":"ccb33323edc979c3718ed586b5b41fe0f13639e6"},{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"d8de6fbd5f7e15af38cffc449d41ae8076605c8f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"All our eventlets and cli apps should call the"},{"line_number":10,"context_line":"eventlet_utils.monkey_patch()"},{"line_number":11,"context_line":"It is now in all tests and eventlets, but missing from the cli apps."},{"line_number":12,"context_line":"This will also prevent the issue in bug #1744558. Currently the issue"},{"line_number":13,"context_line":"does not exist in the cli apps because they exit very quickly, but if"},{"line_number":14,"context_line":"they will be delayed and the subscriber mechanism will finish its"},{"line_number":15,"context_line":"initialization, they will get stuck too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f96bb07_419fd7b3","line":12,"updated":"2018-01-22 16:43:17.000000000","message":"Is the code snippet in bug #1744558 run from cli or import cli ? \nI don\u0027t sure that i understood why its will solve the bug","commit_id":"dffb94d80cf33fa0bb37159bf4cb5fcb952f3807"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"c6aa6a033c6dbe0ecd0d7941d71bdd4fe7b0ee9d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"All our eventlets and cli apps should call the"},{"line_number":10,"context_line":"eventlet_utils.monkey_patch()"},{"line_number":11,"context_line":"It is now in all tests and eventlets, but missing from the cli apps."},{"line_number":12,"context_line":"This will also prevent the issue in bug #1744558. Currently the issue"},{"line_number":13,"context_line":"does not exist in the cli apps because they exit very quickly, but if"},{"line_number":14,"context_line":"they will be delayed and the subscriber mechanism will finish its"},{"line_number":15,"context_line":"initialization, they will get stuck too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f96bb07_5378f8f6","line":12,"in_reply_to":"7f96bb07_419fd7b3","updated":"2018-01-23 08:13:11.000000000","message":"When a module in a package is imported (e.g. df_db.py in cli), then the __init__.py in that package is imported first (i.e. cli/__init__.py). This behaviour is recursive","commit_id":"dffb94d80cf33fa0bb37159bf4cb5fcb952f3807"},{"author":{"_account_id":26131,"name":"Eyal Leshem","email":"eyal.leshem@toganetworks.com","username":"leyal"},"change_message_id":"70fc28114b6582692fc70b687622d452dcb8491f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"All our eventlets and cli apps should call the"},{"line_number":10,"context_line":"eventlet_utils.monkey_patch()"},{"line_number":11,"context_line":"It is now in all tests and eventlets, but missing from the cli apps."},{"line_number":12,"context_line":"This will also prevent the issue in bug #1744558. Currently the issue"},{"line_number":13,"context_line":"does not exist in the cli apps because they exit very quickly, but if"},{"line_number":14,"context_line":"they will be delayed and the subscriber mechanism will finish its"},{"line_number":15,"context_line":"initialization, they will get stuck too."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7f96bb07_febf8bf9","line":12,"in_reply_to":"7f96bb07_5378f8f6","updated":"2018-01-23 08:30:46.000000000","message":"sure , \njust in the code snippet in #1744558 i think there no module from cli that imported.","commit_id":"dffb94d80cf33fa0bb37159bf4cb5fcb952f3807"}]}