)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c8d6e09691f7d297a306ebf8aef23ede1de1090a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d8bc5396_6ba2e9df","updated":"2025-01-13 17:03:24.000000000","message":"merging given previous +2","commit_id":"a9c94abf530f3ed36b9b3f10c89549f04af26df4"}],"docker/openvswitch/openvswitch-base/extend_start.sh":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7a4cc7cf35fa6af3531fab45897d0a731c0aa3de","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    chmod 755 /var/log/kolla/openvswitch"},{"line_number":8,"context_line":"fi"},{"line_number":9,"context_line":"if [[ $(stat -c %U:%G /var/log/kolla/openvswitch) !\u003d \"openvswitch:openvswitch\" ]]; then"},{"line_number":10,"context_line":"    chown -R openvswitch:openvswitch /var/log/kolla/openvswitch"},{"line_number":11,"context_line":"fi"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":". /usr/local/bin/kolla_openvswitch_extend_start"}],"source_content_type":"text/x-sh","patch_set":4,"id":"df3630dc_b157aad5","line":10,"updated":"2025-01-13 14:32:15.000000000","message":"better run this unconditionally, to always chown log files even if the dir is already correct?","commit_id":"912fa174bbc5f9003e3bce401b78eaa53c73632a"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"4b1935d5dcee178cda9870a71f7043b0e342e854","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    chmod 755 /var/log/kolla/openvswitch"},{"line_number":8,"context_line":"fi"},{"line_number":9,"context_line":"if [[ $(stat -c %U:%G /var/log/kolla/openvswitch) !\u003d \"openvswitch:openvswitch\" ]]; then"},{"line_number":10,"context_line":"    chown -R openvswitch:openvswitch /var/log/kolla/openvswitch"},{"line_number":11,"context_line":"fi"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":". /usr/local/bin/kolla_openvswitch_extend_start"}],"source_content_type":"text/x-sh","patch_set":4,"id":"7740d057_5e54e45c","line":10,"in_reply_to":"df3630dc_b157aad5","updated":"2025-01-13 14:47:14.000000000","message":"I want to say the same. Even \u0027mkdir -p\u0027 can be used without any condition check. and merged with \u0027-m 755\u0027 option to force create/fix with correct rights. anyway lgtm.","commit_id":"912fa174bbc5f9003e3bce401b78eaa53c73632a"}]}