)]}'
{"tempest/config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"cabd03e2edc04bcad5e7d9e89bd7411d8a2fd115","unresolved":false,"context_lines":[{"line_number":575,"context_line":"                default\u003dFalse,"},{"line_number":576,"context_line":"                help\u003d\"Does the test environment support port security?\"),"},{"line_number":577,"context_line":"    # TODO(ylobankov): Delete this option once the Liberty release is EOL."},{"line_number":578,"context_line":"    cfg.BoolOpt(\u0027dvr_extra_resources\u0027,"},{"line_number":579,"context_line":"                default\u003dTrue,"},{"line_number":580,"context_line":"                help\u003d\"Whether or not to create internal network, subnet, \""},{"line_number":581,"context_line":"                     \"port and add network interface to distributed router \""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a98d1f4_b2f616f7","line":578,"updated":"2016-09-30 15:19:45.000000000","message":"This is the wrong section for an option like this. It would go in the normal network section. network-feature-enabled is for setting optional features in neutron which this isn\u0027t. This dictates how a test should behave.\n\nTBH, I\u0027m not thrilled about the concept of adding a config option for just a single test like this. It makes it difficult for users to understand what is going, but I assume there is a reason this is needed.","commit_id":"1a02787a7374bc91cc4d4dd656cd4f3d75020e24"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"7260d053a75d29197aa6a09c520835bbc0623ea8","unresolved":false,"context_lines":[{"line_number":576,"context_line":"                help\u003d\"Does the test environment support port security?\"),"},{"line_number":577,"context_line":"    # TODO(ylobankov): Delete this option once the Liberty release is EOL."},{"line_number":578,"context_line":"    cfg.BoolOpt(\u0027dvr_extra_resources\u0027,"},{"line_number":579,"context_line":"                default\u003dTrue,"},{"line_number":580,"context_line":"                help\u003d\"Whether or not to create internal network, subnet, \""},{"line_number":581,"context_line":"                     \"port and add network interface to distributed router \""},{"line_number":582,"context_line":"                     \"in L3 agent scheduler test. Extra resources need to be \""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a8bc5a2_3f39d6d7","line":579,"updated":"2016-09-26 14:34:21.000000000","message":"default\u003dFalse ?","commit_id":"1a02787a7374bc91cc4d4dd656cd4f3d75020e24"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"0a70eef851defe4481e13b8acd0da061e5d62068","unresolved":false,"context_lines":[{"line_number":576,"context_line":"                help\u003d\"Does the test environment support port security?\"),"},{"line_number":577,"context_line":"    # TODO(ylobankov): Delete this option once the Liberty release is EOL."},{"line_number":578,"context_line":"    cfg.BoolOpt(\u0027dvr_extra_resources\u0027,"},{"line_number":579,"context_line":"                default\u003dTrue,"},{"line_number":580,"context_line":"                help\u003d\"Whether or not to create internal network, subnet, \""},{"line_number":581,"context_line":"                     \"port and add network interface to distributed router \""},{"line_number":582,"context_line":"                     \"in L3 agent scheduler test. Extra resources need to be \""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a8bc5a2_ec4f9768","line":579,"in_reply_to":"5a8bc5a2_3f39d6d7","updated":"2016-09-27 09:43:10.000000000","message":"I set this option to True to keep the default behavior of the test. So now I can fix the test on my envs by setting the option to True and it is enough for me :)","commit_id":"1a02787a7374bc91cc4d4dd656cd4f3d75020e24"},{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"39cd7cda56334dc4af1e20dcb47ba8058fe40af5","unresolved":false,"context_lines":[{"line_number":576,"context_line":"                help\u003d\"Does the test environment support port security?\"),"},{"line_number":577,"context_line":"    # TODO(ylobankov): Delete this option once the Liberty release is EOL."},{"line_number":578,"context_line":"    cfg.BoolOpt(\u0027dvr_extra_resources\u0027,"},{"line_number":579,"context_line":"                default\u003dTrue,"},{"line_number":580,"context_line":"                help\u003d\"Whether or not to create internal network, subnet, \""},{"line_number":581,"context_line":"                     \"port and add network interface to distributed router \""},{"line_number":582,"context_line":"                     \"in L3 agent scheduler test. Extra resources need to be \""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a8bc5a2_3da6fdad","line":579,"in_reply_to":"5a8bc5a2_ec4f9768","updated":"2016-09-27 14:11:29.000000000","message":"I see, I was having the same question as afazekas, but sounds good enough to me","commit_id":"1a02787a7374bc91cc4d4dd656cd4f3d75020e24"}]}