-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build python3.6 deb packages and install them into docker-snmp-sv2 #245
Conversation
src/python3/Makefile
Outdated
|
||
$(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% : | ||
## Obtaining the python3 | ||
wget http://http.debian.net/debian/pool/main/p/$(PYTHON_PNAME)/$(PYTHON_PNAME)_$(PYTHON_VER).orig.tar.xz -O $(PYTHON_PNAME)_$(PYTHON_VER).orig.tar.xz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python3.6 is in stretch meaning there will be updates, probably we should upload to the azure storage and download from there. #Resolved
# install libsnmp30 dependencies | ||
# install dependencies for sonic-cfggen | ||
# install libpython3.6-dev and pip dependencies | ||
# TODO: remove libpython3.6-dev, its and pip's dependencies if we can get pip3 directly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is 'its'? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ldconfig && \ | ||
cd .. && rm -rf Python-3.5.2 && rm Python-3.5.2.tgz && \ | ||
pip3 install --no-cache-dir /python-wheels/*py3*.whl hiredis && \ | ||
# clean up |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment shall be moved lower? #WontFix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All these comments are related to below single RUN statement. It's difficult to comment near the exact line, any idea?
In reply to: 99458299 [](ancestors = 99458299)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One option: you could split the RUN statement into three RUN statements, update, install and clean up. #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right. Both options have the same effect. The merging option is kind of manual 'docker squash', which remove the unnecessary intermediate layer storage from docker image's point of view. In future when 'docker squash' becomes mature, we can prevent this situation.
In reply to: 99679265 [](ancestors = 99679265)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are using docker --squash now, we can seperate into different RUNs now. #Resolved
rules/python3.mk
Outdated
@@ -0,0 +1,38 @@ | |||
# python3.5.2+ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3.6.0-1? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I know, but why not using a fixed version? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
snmpagent depends on python3.5.2+. Currently 3.6.0-1 was the only backport-able version for Debian Jessie fulfill our requirement. So I keep the filename and the comment as accurate as possible, in order not to mislead that 3.6.0-1 is the lowest require version.
In reply to: 99676879 [](ancestors = 99676879)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with shuotian, should remove 3.5.2+. we specify 3.6.0-1 that what we have validated. For example, we might use something in this package that is only available for 3.6.0-1, then 3.5.2 is no longer supported. But the questions is since we have no plan to validate/test 3.5.2 in the future, when are we gong to know it will 3.5.2. #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the answers
conflict with recent commit, can you resolve the conflict? #Resolved |
Squashed commit of the following: commit dfa48862586c6c56fc7425aeb802f67f3245077b Merge: 4a70bde f2eed8f Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sun Jan 29 20:38:22 2017 +0000 Merge branch 'github_master' into qiluo/python36 commit 4a70bde906bff141bcb1d6578826c009f9bb9168 Merge: 00ec4e1 9b940c3 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sun Jan 22 23:56:13 2017 +0000 Merge branch 'github_master' into qiluo/python36 commit 00ec4e1a9295dd9420da492be60a297d55bbdb5a Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sun Jan 22 23:40:23 2017 +0000 Fix python3 command commit 04daf5826dea73fa68a7c0ad6799e5f0fdafdded Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sat Jan 21 02:16:48 2017 +0000 Remove build shell script commit 5988810c4537f5a3350fc3d01e399ea25006e191 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sat Jan 21 02:16:27 2017 +0000 Refine python3 build.sh commit 1c4ba20ef944131e5d8c35d08f9454024d9eea14 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Sat Jan 21 02:15:08 2017 +0000 Add missing supervisor commit 70369a596099cbebb12a1052bb3dc322cb72caf1 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Fri Jan 20 23:23:52 2017 +0000 Fix DERIVED_TARGETS and cp command commit 5f52e884620c59239a55102fc1ce988ed85fe13d Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Fri Jan 20 06:37:13 2017 +0000 Reduce docker image size commit 3985310970bc8f8ab0b84358cef61834153a69fd Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Fri Jan 20 03:50:30 2017 +0000 snmp docker images builds with python debs commit 0c340ec460b7172fe3662371f51bfb13cb7dacc5 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Thu Jan 12 02:50:57 2017 +0000 (comment) commit 0c2353f6c93678b00a01ea2104e145bb822ad542 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Thu Jan 12 00:53:32 2017 +0000 (comment) commit 22836d960e97f070b99543f255f0a61ccdcfb1c8 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Wed Jan 11 23:12:55 2017 +0000 Fix bug: dpkg and force install commit d26da9607af702b8029a7b9aaf9f73ec1ae7734d Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Wed Jan 11 22:43:09 2017 +0000 Remove wget timestamp option commit 363aeba0aaf5bf53f557385355d34c3824a4a778 Author: Qi Luo <qiluo-msft@users.noreply.github.com> Date: Wed Jan 11 22:18:22 2017 +0000 Build debian package python3.6
RUN apt-get update && apt-get install -y supervisor \ | ||
libperl5.20 libpci3 libwrap0 \ | ||
libexpat1-dev \ | ||
python-lxml python-jinja2 python-netaddr python-ipaddr python-yaml \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python-lxml python-jinja2 python-netaddr python-ipaddr python-yaml [](start = 12, length = 67)
we should remove these now, included in the sonic-config-engine #Resolved
|
||
# install supervisor | ||
# install libsnmp30 dependencies | ||
# install dependencies for sonic-cfggen |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this #Resolved
libperl5.20 libpci3 libwrap0 \ | ||
libexpat1-dev \ | ||
python-lxml python-jinja2 python-netaddr python-ipaddr python-yaml \ | ||
curl gcc && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc [](start = 18, length = 2)
do we need gcc? #Resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove packages for docker-config-engine
When merging, please note to re-format the comments on the github website. Otherwise it will contain too much logs. |
|
sairedis * 248a095 2018-11-07 | Add best candidate search for acl table (sonic-net#371) [Kamil Cudnik] * d1e26c3 2018-11-07 | Ignore order when compare QOS MAP list entries (sonic-net#372) [Kamil Cudnik] * e8df347 2018-11-05 | Add best candidate search for router interface using tunnel term tabl… (sonic-net#370) [Kamil Cudnik] * 8ae173b 2018-11-01 | Refactor saidump graph generator (sonic-net#367) [Kamil Cudnik] common * 2a37c5c 2018-11-07 | Add system warm-reboot check in WarmRestart class (sonic-net#246) (HEAD, origin/master, origin/HEAD) [zhenggen-xu] * fb082b5 2018-11-06 | Fix ProducerStateTable::clear() to clear StateHash properly (sonic-net#244) [Taoyu Li] * 0ee636b 2018-11-05 | Remove local googletest submodule and link with system gtest (sonic-net#245) [Qi Luo] * b56ffba 2018-10-31 | Add NEIGH_RESTORE_TABLE in stateDB (sonic-net#243) [zhenggen-xu] * b58c69d 2018-10-28 | swss-common: add op 'create' case to avoid flush immediately (sonic-net#219) [Dong Zhang] swss * afdcf34 2018-11-12 | Support neighsyncd system warmreboot. (sonic-net#661) (HEAD, origin/master, origin/HEAD) [zhenggen-xu] * f380685 2018-11-09 | Routing-stack warm-reboot feature. (sonic-net#602) [Rodny Molina] * 9fbcb60 2018-11-09 | Add warm-reboot mode for teammgrd (sonic-net#678) [pavel-shirshov] * 8c60787 2018-11-09 | Don't add loopback ip2me route again if already configured (sonic-net#656) [Jipan Yang] * 6eb1613 2018-11-08 | [test]: Add interface IPv6 add/remove test case (sonic-net#677) [Shuotian Cheng] * 5de5054 2018-11-07 | [vrfmgrd]: Support VNET setting (sonic-net#657) [Marian Pritsak] * f666011 2018-11-06 | [teammgrd]: Add retry logic when enslaving member port into team (sonic-net#669) [Shuotian Cheng] * 36e304d 2018-11-05 | [vstest]: print output when runcmd returns error (sonic-net#672) [lguohan] * aede5d4 2018-11-05 | [test]: Fix clean up wrong interface IP (sonic-net#673) [Shuotian Cheng] * aeceaca 2018-11-02 | [portmgrd]: portmgrd shall be responsible for all ports update (sonic-net#668) [Shuotian Cheng] * 5796e54 2018-11-02 | Orchagent warm restart data restore requires three iterations (sonic-net#670) [Jipan Yang] * 15a2299 2018-11-01 | [vlanmgrd]: Do not bring up VLAN members by default (sonic-net#667) [Shuotian Cheng] * 44a4460 2018-11-01 | [test]: Clean up LAGs after finishing the test (sonic-net#666) [Shuotian Cheng] Signed-off-by: Guohan Lu <gulv@microsoft.com>
sairedis * 248a095 2018-11-07 | Add best candidate search for acl table (#371) [Kamil Cudnik] * d1e26c3 2018-11-07 | Ignore order when compare QOS MAP list entries (#372) [Kamil Cudnik] * e8df347 2018-11-05 | Add best candidate search for router interface using tunnel term tabl… (#370) [Kamil Cudnik] * 8ae173b 2018-11-01 | Refactor saidump graph generator (#367) [Kamil Cudnik] common * 2a37c5c 2018-11-07 | Add system warm-reboot check in WarmRestart class (#246) (HEAD, origin/master, origin/HEAD) [zhenggen-xu] * fb082b5 2018-11-06 | Fix ProducerStateTable::clear() to clear StateHash properly (#244) [Taoyu Li] * 0ee636b 2018-11-05 | Remove local googletest submodule and link with system gtest (#245) [Qi Luo] * b56ffba 2018-10-31 | Add NEIGH_RESTORE_TABLE in stateDB (#243) [zhenggen-xu] * b58c69d 2018-10-28 | swss-common: add op 'create' case to avoid flush immediately (#219) [Dong Zhang] swss * afdcf34 2018-11-12 | Support neighsyncd system warmreboot. (#661) (HEAD, origin/master, origin/HEAD) [zhenggen-xu] * f380685 2018-11-09 | Routing-stack warm-reboot feature. (#602) [Rodny Molina] * 9fbcb60 2018-11-09 | Add warm-reboot mode for teammgrd (#678) [pavel-shirshov] * 8c60787 2018-11-09 | Don't add loopback ip2me route again if already configured (#656) [Jipan Yang] * 6eb1613 2018-11-08 | [test]: Add interface IPv6 add/remove test case (#677) [Shuotian Cheng] * 5de5054 2018-11-07 | [vrfmgrd]: Support VNET setting (#657) [Marian Pritsak] * f666011 2018-11-06 | [teammgrd]: Add retry logic when enslaving member port into team (#669) [Shuotian Cheng] * 36e304d 2018-11-05 | [vstest]: print output when runcmd returns error (#672) [lguohan] * aede5d4 2018-11-05 | [test]: Fix clean up wrong interface IP (#673) [Shuotian Cheng] * aeceaca 2018-11-02 | [portmgrd]: portmgrd shall be responsible for all ports update (#668) [Shuotian Cheng] * 5796e54 2018-11-02 | Orchagent warm restart data restore requires three iterations (#670) [Jipan Yang] * 15a2299 2018-11-01 | [vlanmgrd]: Do not bring up VLAN members by default (#667) [Shuotian Cheng] * 44a4460 2018-11-01 | [test]: Clean up LAGs after finishing the test (#666) [Shuotian Cheng] Signed-off-by: Guohan Lu <gulv@microsoft.com>
* Added support to configure fec mode on interface * Modified checks to address comments
…nox SN2740 platform (sonic-net#245)
* build syncd-vs and syncd-vs-dbg package build syncd-vs package without vendor SAI package remove vssyncd package as its build still rely on a vendor SAI package. * install sai headers in libsaivs-dev package
* 6aa8d00 2021-11-12 | marvell-armhf]: add kconfig force inclusion for CONFIG_ARCH_NR_GPIO (sonic-net#245) (HEAD, origin/master, origin/HEAD) [dflynn-Nokia] * 39c7d18 2021-11-06 | [arm64]: add kconfig force inclusion for COMPAT_VSDO [Guohan Lu] * fafaaea 2021-11-06 | [backport] kernel-compat-always-include-linux-compat.h-from-net-compat.patch [Guohan Lu] * 6ac9b48 2021-10-19 | [ci]: use native arm64/armhf pool to build kernel [Guohan Lu] Signed-off-by: Guohan Lu <lguohan@gmail.com>
* 6aa8d00 2021-11-12 | marvell-armhf]: add kconfig force inclusion for CONFIG_ARCH_NR_GPIO (#245) (HEAD, origin/master, origin/HEAD) [dflynn-Nokia] * 39c7d18 2021-11-06 | [arm64]: add kconfig force inclusion for COMPAT_VSDO [Guohan Lu] * fafaaea 2021-11-06 | [backport] kernel-compat-always-include-linux-compat.h-from-net-compat.patch [Guohan Lu] * 6ac9b48 2021-10-19 | [ci]: use native arm64/armhf pool to build kernel [Guohan Lu] Signed-off-by: Guohan Lu <lguohan@gmail.com>
c2aac75 [SFP-Refactor] Fix LP mode API issue (#247) dba17c8 Firmware upgrade CLI support for QSFP-DD transceivers (#244) cd69212 [SFP-Refactor] Implement CMIS Low Power mode (#237) 9cea07f Fix RegGroupField decode (#245) 6ae1909 Add CMIS QSFP support (#246) c1f317d Gracefully handle CMIS APIs for passive modules (#238) ec7335d fix for firmware functions (#243) cf2ebe9 Fix RegBitField decode/encode (#242) ef4f2c6 Fix SFP_CABLE_TECH_FIELD (#240) e118644 remove time counting message in functions because function running time could be difficult to predict in unit tests (#241) Signed-off-by: Prince George <prgeor@microsoft.com>
c2aac75 [SFP-Refactor] Fix LP mode API issue (#247) dba17c8 Firmware upgrade CLI support for QSFP-DD transceivers (#244) cd69212 [SFP-Refactor] Implement CMIS Low Power mode (#237) 9cea07f Fix RegGroupField decode (#245) 6ae1909 Add CMIS QSFP support (#246) c1f317d Gracefully handle CMIS APIs for passive modules (#238) ec7335d fix for firmware functions (#243) cf2ebe9 Fix RegBitField decode/encode (#242) ef4f2c6 Fix SFP_CABLE_TECH_FIELD (#240) e118644 remove time counting message in functions because function running time could be difficult to predict in unit tests (#241) Signed-off-by: Prince George <prgeor@microsoft.com>
…onic-net#245) The bullseye kernel defaults the CONFIG_ARCH_NR_GPIO config option for marvell-armhf to 1024 which prevents the value from being overridden to 0. This commit uses the new force_inclusion file to override the value without performing any additional checks.
…lly (#19325) #### Why I did it src/sonic-gnmi ``` * a610784 - (HEAD -> master, origin/master, origin/HEAD) Improve timeout for GNMI dbus service (#265) (3 days ago) [ganglv] * b1c90c9 - Add unit test for vlan interface (#264) (6 days ago) [ganglv] * fbf0234 - Add unit test for syslog config (#263) (6 days ago) [ganglv] * 047fc29 - Add unit test for portchannel interface (#262) (6 days ago) [ganglv] * 52dd8e7 - Add unit test for pg headroom (#261) (6 days ago) [ganglv] * 8a10d9f - Add unit test for pfcwd status (#260) (6 days ago) [ganglv] * 65a43bc - Add unit test for pfcwd interval (#259) (6 days ago) [ganglv] * 0bbb9ef - Add unit test for ntp (#258) (7 days ago) [ganglv] * e5c43a4 - Add unit test for monitor config (#257) (7 days ago) [ganglv] * 8a8881e - Add unit test for mmu dynamic threshold (#256) (7 days ago) [ganglv] * 76857df - Add unit test for lo interface (#255) (7 days ago) [ganglv] * c35c69d - Add unit test for k8s config (#254) (8 days ago) [ganglv] * dd0bdcf - Add unit test for IPV6 (#253) (8 days ago) [ganglv] * 9a1133f - Add unit test for incremental qos (#252) (8 days ago) [ganglv] * 9bf7969 - Add unit test for eth interface (#251) (8 days ago) [ganglv] * 2e1eb05 - Add unit test for ECN config (#250) (8 days ago) [ganglv] * 56f17eb - Add unit test for dynamic ACL (#249) (8 days ago) [ganglv] * 388eba8 - Add unit test for DHCP relay (#245) (10 days ago) [ganglv] * fd2fdf6 - Add unit test for CACL (#243) (11 days ago) [ganglv] ``` #### How I did it #### How to verify it #### Description for the changelog
…lly (sonic-net#19325) #### Why I did it src/sonic-gnmi ``` * a610784 - (HEAD -> master, origin/master, origin/HEAD) Improve timeout for GNMI dbus service (sonic-net#265) (3 days ago) [ganglv] * b1c90c9 - Add unit test for vlan interface (sonic-net#264) (6 days ago) [ganglv] * fbf0234 - Add unit test for syslog config (sonic-net#263) (6 days ago) [ganglv] * 047fc29 - Add unit test for portchannel interface (sonic-net#262) (6 days ago) [ganglv] * 52dd8e7 - Add unit test for pg headroom (sonic-net#261) (6 days ago) [ganglv] * 8a10d9f - Add unit test for pfcwd status (sonic-net#260) (6 days ago) [ganglv] * 65a43bc - Add unit test for pfcwd interval (sonic-net#259) (6 days ago) [ganglv] * 0bbb9ef - Add unit test for ntp (sonic-net#258) (7 days ago) [ganglv] * e5c43a4 - Add unit test for monitor config (sonic-net#257) (7 days ago) [ganglv] * 8a8881e - Add unit test for mmu dynamic threshold (sonic-net#256) (7 days ago) [ganglv] * 76857df - Add unit test for lo interface (sonic-net#255) (7 days ago) [ganglv] * c35c69d - Add unit test for k8s config (sonic-net#254) (8 days ago) [ganglv] * dd0bdcf - Add unit test for IPV6 (sonic-net#253) (8 days ago) [ganglv] * 9a1133f - Add unit test for incremental qos (sonic-net#252) (8 days ago) [ganglv] * 9bf7969 - Add unit test for eth interface (sonic-net#251) (8 days ago) [ganglv] * 2e1eb05 - Add unit test for ECN config (sonic-net#250) (8 days ago) [ganglv] * 56f17eb - Add unit test for dynamic ACL (sonic-net#249) (8 days ago) [ganglv] * 388eba8 - Add unit test for DHCP relay (sonic-net#245) (10 days ago) [ganglv] * fd2fdf6 - Add unit test for CACL (sonic-net#243) (11 days ago) [ganglv] ``` #### How I did it #### How to verify it #### Description for the changelog
…lly (sonic-net#19325) #### Why I did it src/sonic-gnmi ``` * a610784 - (HEAD -> master, origin/master, origin/HEAD) Improve timeout for GNMI dbus service (sonic-net#265) (3 days ago) [ganglv] * b1c90c9 - Add unit test for vlan interface (sonic-net#264) (6 days ago) [ganglv] * fbf0234 - Add unit test for syslog config (sonic-net#263) (6 days ago) [ganglv] * 047fc29 - Add unit test for portchannel interface (sonic-net#262) (6 days ago) [ganglv] * 52dd8e7 - Add unit test for pg headroom (sonic-net#261) (6 days ago) [ganglv] * 8a10d9f - Add unit test for pfcwd status (sonic-net#260) (6 days ago) [ganglv] * 65a43bc - Add unit test for pfcwd interval (sonic-net#259) (6 days ago) [ganglv] * 0bbb9ef - Add unit test for ntp (sonic-net#258) (7 days ago) [ganglv] * e5c43a4 - Add unit test for monitor config (sonic-net#257) (7 days ago) [ganglv] * 8a8881e - Add unit test for mmu dynamic threshold (sonic-net#256) (7 days ago) [ganglv] * 76857df - Add unit test for lo interface (sonic-net#255) (7 days ago) [ganglv] * c35c69d - Add unit test for k8s config (sonic-net#254) (8 days ago) [ganglv] * dd0bdcf - Add unit test for IPV6 (sonic-net#253) (8 days ago) [ganglv] * 9a1133f - Add unit test for incremental qos (sonic-net#252) (8 days ago) [ganglv] * 9bf7969 - Add unit test for eth interface (sonic-net#251) (8 days ago) [ganglv] * 2e1eb05 - Add unit test for ECN config (sonic-net#250) (8 days ago) [ganglv] * 56f17eb - Add unit test for dynamic ACL (sonic-net#249) (8 days ago) [ganglv] * 388eba8 - Add unit test for DHCP relay (sonic-net#245) (10 days ago) [ganglv] * fd2fdf6 - Add unit test for CACL (sonic-net#243) (11 days ago) [ganglv] ``` #### How I did it #### How to verify it #### Description for the changelog
No description provided.