Skip to content

Commit 5d7a345

Browse files
authored
[SAI-PTF][202012]Fix sai ptf 202012 (#12724)
* fix sai-ptf docker build error Signed-off-by: richardyu-ms <[email protected]> * correct the docker image version Signed-off-by: richardyu-ms <[email protected]> * update thrift package Signed-off-by: richardyu-ms <[email protected]> * fix version upgrade issue in 202012 Signed-off-by: richardyu-ms <[email protected]> * remove useless file Signed-off-by: richardyu-ms <[email protected]> Signed-off-by: richardyu-ms <[email protected]>
1 parent 10c2d60 commit 5d7a345

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

dockers/docker-ptf-sai/Dockerfile.j2

+4-5
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ RUN pip3 install crc16 \
1616
getmac \
1717
packet_helper \
1818
psutil \
19-
scapy==2.4.4 \
2019
scapy_helper \
2120
pysubnettree
2221

@@ -34,7 +33,7 @@ debs/{{ deb }}{{' '}}
3433
# Remove old ptf package
3534
RUN dpkg -r python-ptf
3635

37-
# Install new ptf package
38-
RUN git clone https://github.com/p4lang/ptf.git \
39-
&& cd ptf \
40-
&& python3.7 setup.py install --single-version-externally-managed --record /tmp/ptf_install.txt
36+
# Install new ptf package not supprt in 202012 branch
37+
# RUN git clone https://github.com/p4lang/ptf.git \
38+
# && cd ptf \
39+
# && python3.7 setup.py install --single-version-externally-managed --record /tmp/ptf_install.txt

platform/vs/docker-ptf-sai.mk

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
DOCKER_PTF_SAI = docker-ptf-sai.gz
44
DOCKER_PTF_BASE = docker-ptf.gz
55
$(DOCKER_PTF_SAI)_PATH = $(DOCKERS_PATH)/docker-ptf-sai
6-
$(DOCKER_PTF_SAI)_DEPENDS += $(LIBTHRIFT_2) $(PYTHON3_THRIFT_2)
6+
$(DOCKER_PTF_SAI)_DEPENDS += $(LIBTHRIFT_0_13_0) $(PYTHON3_THRIFT_0_13_0)
77
$(DOCKER_PTF_SAI)_LOAD_DOCKERS += $(DOCKER_PTF_BASE)
88
SONIC_DOCKER_IMAGES += $(DOCKER_PTF_SAI)
9-
SONIC_BUSTER_DOCKERS += $(DOCKER_PTF_SAI)
9+
SONIC_STRETCH_DOCKERS += $(DOCKER_PTF_SAI)

0 commit comments

Comments
 (0)