-
Notifications
You must be signed in to change notification settings - Fork 60
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
Update DentOS to debian10 #216
Conversation
reverify |
1 similar comment
reverify |
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.
Thank you for working on this. Some questions:
- Why not go straight to Debian 11 or 12?
- It’d be great if you elaborated in the commit messages.
- Typo: Remove unsupported devises → devices
- It’d be great if you added a section about the devices you tested this on, and what difference you noticed (boot time, performance, …).
With Debian 11 or 12 a lot of python2 onl code has to be updated. |
Test results for AS5114 platform can be found here |
Fails to build for me on ubuntu 22.04:
|
Using a fresh checkout it worked. Must have been some leftovers somewhere from the previous build with debian 9 (though no idea where, since I did a git clean). |
Booted the image and was greeted with a scary:
in the boot log. Looking at the journal
it looks like it is stuck in a restart loop. |
Somehow with the switch to debian 10, FRR got downgraded from 8.5 to 6.0.2. |
I did a comparison via A few packages got removed (ignoring those with a version it in their name with a replacement package), some expectedly:
Of which probably the most interesting are
which probably should have been in the image. And a few packages got downgraded:
For completion sake, the following packages are new in the debian 10 image (some probably are just stuff broken out into their own package):
Of course there's the possibility I did something wrong when building my image, but this is what I got with a locally built one. |
...y/rootfs/buster/common/overlay/etc/systemd/system/networking.service.d/switchdev-online.conf
Show resolved
Hide resolved
builds/any/rootfs/buster/common/overlay/etc/systemd/system/switchdev-online@.service
Show resolved
Hide resolved
FRR is installed from http://deb.frrouting.org/frr for stretch and from Debian repos for buster |
Added all the missing repos from https://github.com/dentproject/dentOS/blob/main/builds/any/rootfs/stretch/standard/standard.yml to https://github.com/dentproject/dentOS/blob/main/builds/any/rootfs/buster/standard/standard.yml and changed the security location from Now building an image with hopefully latest FRR 8.5.2 included. |
yes this should include the latest stable frr from http://deb.frrouting.org/frr |
these packages are installed from pre-build binaries from dent-atrifacts repo for stretch and from Debian repos for buster |
https://github.com/lldpd/lldpd/blob/master/NEWS tells me we likely do want to have lldp 1.0.5 and not 1.0.3:
or maybe even upgrade to an even newer one, I see several CVEs mentioned in the changes of newer versions. |
I think we should start with 1.0.5 and possibly update to the latest. |
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.
If you change the builder to the expected upstream one (dentproject/builder10:1.3
) and stay at Debian 9 for now (until we get the builder built), I'll be fine to merge this.
c88a3db
to
705e92b
Compare
Include latest petunia image and fix for broken symlink Add packages: - mstpd - ifupdown2 - python3-pyroute2 Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
based on 1d553a9 Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
This docker image is based on opennetworklinux/builder10 Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
list of packages: - onl-kernel-5.15-lts-arm64-all-modules - frr - mrvl-fw-image - bridge-utils - bash-completion - man-db - dnsmasq - dnsutils - keepalived - lldpd Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Add dependencies for iproute2 and ethtool Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
list of packages: - mstpd - hostapd - stress-ng - python3-pyroute2 Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Set default config to get IP on ma1 using dhcp Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
replace ifupdown with ifupdown2 add lsb-release, dbus, python3-setuptools packages Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
- Add poed and petunia packages - rearrange base packages - update dent-artifacts pointer - use debian-keyring instead of debian-archive-keyring Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
705e92b
to
facf0d0
Compare
updated to use https://hub.docker.com/r/dentproject/builder10/tags |
@@ -23,7 +23,7 @@ g_builders = { | |||
7 : ('wheezy', 'opennetworklinux/builder7:1.2'), | |||
8 : ('jessie', 'opennetworklinux/builder8:1.11'), | |||
9 : ('stretch', 'dentproject/builder9:1.8' ), | |||
10 : ('buster', 'opennetworklinux/builder10:1.2'), | |||
10 : ('buster', 'dentproject/builder10:1.5'), |
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.
The Makefile you are adding is for 1.3 though. Why do we jump to 1.5? What are the differences to 1.3 and 1.4? Please add the Makefile used for the this builder.
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.
Bump version to 1.5 to sync with tag on dockehub https://hub.docker.com/r/dentproject/builder10/tags Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
reverify |
What is the recommended kernel for use with Debian 10? Is kernel 5.15 ready? |
Yes, Dent 3.0 has transitioned to kernel 5.15 back in March, and Dent 3.2 switched to Debian 10 this month. Please check the release notes at
https://github.com/dentproject/dentOS/releases
Thanks,
Taskin
…________________________________
From: Ashwin Ganesh ***@***.***>
Sent: Wednesday, September 20, 2023 11:35:58 AM
To: dentproject/dentOS ***@***.***>
Cc: taskin0003 ***@***.***>; Review requested ***@***.***>
Subject: Re: [dentproject/dentOS] Update DentOS to debian10 (PR #216)
What is the recommended kernel for us with Debian 10? Is kernel 5.15 ready?
—
Reply to this email directly, view it on GitHub<#216 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOITEXCY6F6AVAJFUCUYY7TX3MZQZANCNFSM6AAAAAAYCQ2HRA>.
You are receiving this because your review was requested.Message ID: ***@***.***>
|
Update DentOS to Debian 10:
Arm64 image can be found here:
https://repos.refinery.dev/repository/dent/snapshots/org/dent/dentos/dentos-verify-main/DENTOS-HEAD_ONL-OS10_2023-05-30.1233-2f62b8d_ARM64_INSTALLED_INSTALLER
Supported platforms: