)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"3419298c602ed3c726698379d4ae641ec5f64739","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3f5c600e_3acd4751","updated":"2022-10-31 10:07:43.000000000","message":"I accept that patch because it allows to make next part smaller.\n\nStill wonder do we have a way to set it once instead of on each kolla_toolbox use.","commit_id":"c0273b1b5ea6ef19f73866267bd8628f4e0c37b1"}],"ansible/library/kolla_toolbox.py":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5e2cabe4130a1cc3f11796f3f5bccd4498eb0e7a","unresolved":true,"context_lines":[{"line_number":33,"context_line":"  - A module targerting at invoking ansible module in kolla_toolbox"},{"line_number":34,"context_line":"    container as used by Kolla project."},{"line_number":35,"context_line":"options:"},{"line_number":36,"context_line":"  container_engine:"},{"line_number":37,"context_line":"    description:"},{"line_number":38,"context_line":"      - Name of container engine to use"},{"line_number":39,"context_line":"    required: True"},{"line_number":40,"context_line":"    type: str"},{"line_number":41,"context_line":"  module_name:"},{"line_number":42,"context_line":"    description:"},{"line_number":43,"context_line":"      - The module name to invoke"}],"source_content_type":"text/x-python","patch_set":2,"id":"be27b351_89e92379","line":40,"range":{"start_line":36,"start_character":0,"end_line":40,"end_character":13},"updated":"2022-10-31 10:38:51.000000000","message":"Hmm, won\u0027t be better to set default value to docker ? What do you think ?","commit_id":"c0273b1b5ea6ef19f73866267bd8628f4e0c37b1"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"e596022b45c5ec88d8ff128ec68ea1724d81be4e","unresolved":true,"context_lines":[{"line_number":33,"context_line":"  - A module targerting at invoking ansible module in kolla_toolbox"},{"line_number":34,"context_line":"    container as used by Kolla project."},{"line_number":35,"context_line":"options:"},{"line_number":36,"context_line":"  container_engine:"},{"line_number":37,"context_line":"    description:"},{"line_number":38,"context_line":"      - Name of container engine to use"},{"line_number":39,"context_line":"    required: True"},{"line_number":40,"context_line":"    type: str"},{"line_number":41,"context_line":"  module_name:"},{"line_number":42,"context_line":"    description:"},{"line_number":43,"context_line":"      - The module name to invoke"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a8569a9_50619b79","line":40,"range":{"start_line":36,"start_character":0,"end_line":40,"end_character":13},"in_reply_to":"be27b351_89e92379","updated":"2022-10-31 10:44:28.000000000","message":"Still need all those places to set the value. And lack of default will allow to catch missing places.","commit_id":"c0273b1b5ea6ef19f73866267bd8628f4e0c37b1"}]}