Skip to content

Commit 4bcb91d

Browse files
keboliuCarl Keene
authored andcommitted
mount 'mellanox' folder only instead of create each sub folder (sonic-net#7830)
#### Why I did it Following the discussion in another PR sonic-net#7708 (comment) , since there will be multi subfolders under **/var/log/mellanox**, so we agreed to only mount this folder and the subfolders will be created afterward on demand. #### How I did it during the syncd docker creation, only mount folder **/var/log/mellanox** #### How to verify it build an Mellanox image and verify the related folder on the host and docker side.
1 parent aafc5f1 commit 4bcb91d

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

files/build_templates/docker_image_ctl.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ start() {
382382
{%- endif %}
383383
{%- if sonic_asic_platform == "mellanox" %}
384384
{%- if docker_container_name == "syncd" %}
385-
-v /var/log/mellanox/sniffer:/var/log/mellanox/sniffer:rw \
385+
-v /var/log/mellanox:/var/log/mellanox:rw \
386386
-v mlnx_sdk_socket:/var/run/sx_sdk \
387387
-v mlnx_sdk_ready:/tmp \
388388
-v /dev/shm:/dev/shm:rw \

0 commit comments

Comments
 (0)