Skip to content

Commit

Permalink
Enable DPB dependency check using YANG model in VS container (#45) (#…
Browse files Browse the repository at this point in the history
…4990)

Added required packages to enabled YANG dependency check for Dynamic Port Breakout in VS container.

[sonic-utilities PR #766](sonic-net/sonic-utilities#766) has a dependency on it.
Getting error like the following without this fix: `ImportError: No module named yang - required module not found`

Signed-off-by: Sangita Maity <[email protected]>
  • Loading branch information
samaity authored Jul 21, 2020
1 parent 1870c18 commit ebdfb4b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
5 changes: 4 additions & 1 deletion platform/vs/docker-sonic-vs.mk
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,10 @@ $(DOCKER_SONIC_VS)_DEPENDS += $(SWSS) \
$(PYTHON_SWSSCOMMON) \
$(LIBTEAMDCTL) \
$(LIBTEAM_UTILS) \
$(SONIC_DEVICE_DATA)
$(SONIC_DEVICE_DATA) \
$(LIBYANG) \
$(LIBYANG_CPP) \
$(LIBYANG_PY2)

$(DOCKER_SONIC_VS)_PYTHON_DEBS += $(SONIC_UTILS)

Expand Down
1 change: 1 addition & 0 deletions platform/vs/docker-sonic-vs/Dockerfile.j2
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ RUN pip install crontab
# Install dependencies for Dynamic Port Breakout
RUN pip install xmltodict==0.12.0
RUN pip install jsondiff==1.2.0
RUN pip install ijson==2.6.1

{% if docker_sonic_vs_debs.strip() -%}
# Copy locally-built Debian package dependencies
Expand Down

0 comments on commit ebdfb4b

Please sign in to comment.