)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"911fd76c7c4706ac0711f367c08e04c4582ad9de","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     yue.lg \u003cyuelg@chinaunicom.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-10-03 21:07:24 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"For containerized cni_ds set mountPropagation\u003dBidirectional to"},{"line_number":8,"context_line":"let containers aware of the host\u0027s netns changes"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Signed-off-by: yue.lg \u003cyuelg@chinaunicom.cn\u003e"},{"line_number":11,"context_line":"Change-Id: I312817db503002f918276d0ff41baeb816f39404"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"db3bebb5_c12250a1","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":48},"updated":"2021-10-05 10:43:13.000000000","message":"This is related to https://bugs.launchpad.net/kuryr-kubernetes/+bug/1945293, isn\u0027t it? If so you got to add following line to the commit message:\n\n Closes-Bug: 1945293","commit_id":"141a125d49b9804d6f795f3a77eacc6e21ceef7e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"d491b676dd8fad09f943c93c09849cce24b1a9ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"45ffc7b3_992c0045","updated":"2021-10-11 09:49:00.000000000","message":"Thanks for the fix. \n\nFor next time: It\u0027s beneficial to split the commit message into a one line short description and then more detail description in the following lines. Please check the docs https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"b4bd7312e4bcbca1dbb4b9a900e85dd6902daf78"}],"devstack/lib/kuryr_kubernetes":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"911fd76c7c4706ac0711f367c08e04c4582ad9de","unresolved":true,"context_lines":[{"line_number":657,"context_line":"        cat \u003e\u003e \"${output_dir}/cni_ds.yml\" \u003c\u003c EOF"},{"line_number":658,"context_line":"        - name: openvswitch"},{"line_number":659,"context_line":"          mountPath: /var/run"},{"line_number":660,"context_line":"          mountPropagation: Bidirectional"},{"line_number":661,"context_line":"EOF"},{"line_number":662,"context_line":"    fi"},{"line_number":663,"context_line":"    cat \u003e\u003e \"${output_dir}/cni_ds.yml\" \u003c\u003c EOF"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"15a23fe9_c403730d","line":660,"range":{"start_line":660,"start_character":28,"end_line":660,"end_character":41},"updated":"2021-10-05 10:43:13.000000000","message":"I see Kuryr is run with HostToContainer elsewhere [1]. Any reason to change it to Bidirectional? We\u0027re not really manipulating those mounts, so HostToContainer should probably be enough.\n\n[1] https://github.com/openshift/openshift-ansible/blob/release-3.11/roles/kuryr/templates/cni-daemonset.yaml.j2#L141\n[2] https://github.com/openshift/cluster-network-operator/blob/master/bindata/network/kuryr/005-daemon.yaml#L92","commit_id":"141a125d49b9804d6f795f3a77eacc6e21ceef7e"}]}
