Skip to content

Upgrade ifupdown2 to version 1.2.8#3180

Merged
lguohan merged 2 commits intosonic-net:masterfrom
rajendra-dendukuri:ifupdown2_upgrade
Jul 20, 2019
Merged

Upgrade ifupdown2 to version 1.2.8#3180
lguohan merged 2 commits intosonic-net:masterfrom
rajendra-dendukuri:ifupdown2_upgrade

Conversation

@rajendra-dendukuri
Copy link
Copy Markdown
Contributor

- What I did
Instead of using ifupdown2 package from Debian stretch repository, source compiled and installed lates
version of ifupdown2. The latest version is required to support ZTP over IPv6 transport in a stable and
quick manner.

- How I did it

Created a new rules/ifupdown2.mk file and compiled ifupdown2 as a SONIC_MAKE_DEBS.
Modified slave.mk to have the ifupdown2 as a dependency to create sonic installer image.
Modified "files/build_templates/sonic_debian_extension.j2" to install source compiled ifupdown2 deb file and its dependencies.

- How to verify it
Load an image with these changes and check ifupdown2 version using the command dpkg -l ifupdown2

Desired=Unknown/Install/Remove/Purge/Hold
| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
||/ Name Version Architecture Description
+++-==============-============-============-=================================
ii ifupdown2 1.2.8-1 all Network Interface Management tool

Verified that "systemctl restart interfaces-config" also works.
Verified if eth0 IP is picked up via DHCP without any issues.

- Description for the changelog

Upgrade ifupdown2 to version 1.2.8

- A picture of a cute animal (not mandatory but encouraged)

Required by ZTP to support ZTP over IPv6 transport

Signed-off-by: Rajendra Dendukuri <rajendra.dendukuri@broadcom.com>
@lguohan lguohan requested a review from pavel-shirshov July 18, 2019 13:24
Comment thread files/build_templates/sonic_debian_extension.j2
Comment thread src/ifupdown2/Makefile Outdated
Copy link
Copy Markdown
Contributor

@pavel-shirshov pavel-shirshov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As comments

Remove git statements as no patches are being applied.

Signed-off-by: Rajendra Dendukuri <rajendra.dendukuri@broadcom.com>
@pavel-shirshov
Copy link
Copy Markdown
Contributor

retest mellanox please

@pavel-shirshov
Copy link
Copy Markdown
Contributor

retest vsimage please

1 similar comment
@rajendra-dendukuri
Copy link
Copy Markdown
Contributor Author

retest vsimage please

@lguohan lguohan merged commit 40c8bc1 into sonic-net:master Jul 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants