202205 #27
Closed
202205 #27
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* [Arista] Fix boot0 code for docker_inram Enable docker_inram for all systems with 4GB or less of flash. This is mandatory to allow these systems to store 2 SONiC images. This change also fixes the missing docker_inram attribute when installing a new image from SONiC. Because the SWI image can ship with additional kernel parameters within such as `sonic_fips=` this lead to a conflict. To prevent the conflict, the extra kernel parameters from the SWI are now stored in the file `kernel-cmdline-append` which isn't used anywhere. * Add optional zram compression for docker_inram Some devices running SONiC have a small storage device (2G and 4G mainly) The SONiC image growth over time has made it impossible to install 2 images on a single device. Some mitigations have been implemented in the past for some devices but there is a need to do more. One such mitigation is `docker_inram` which creates a `tmpfs` and extracts `dockerfs.tar.gz` in it. This all happens in the SONiC initramfs and by ensuring the installation process does not extract `dockerfs.tar.gz` on the flash but keep the file as is. This mitigation does a tradeoff by using more RAM to reduce the disk footprint. It however creates new issues for devices with 4G of system memory since the extracted `dockerfs.tar.gz` nears the 1.6G. Considering debian upgrades (with dual base images) and the continuous stream of features this is only going to get bigger. This change introduces an alternative to the `tmpfs` by allowing a system to extract the `dockerfs.tar.gz` inside a `zram` device thus bringing compression in play at the detriment of performance. Introduce 2 new optional kernel parameters to be consumed by SONiC initramfs. - `docker_inram_size` which represent the max physical size of the `zram` or `tmpfs` volume (defaults to DOCKER_RAMFS_SIZE) - `docker_inram_algo` which is the method to use to extract the `dockerfs.tar.gz` (defaults to `tmpfs`) other values are considered to be compression algorithm for `zram` (e.g `zstd`, `zlo-rle`, `lz4`) Refactored the logic to mount the docker fs in the SONiC initramfs under the `union-mount` script. Moved the code into a function to make it cleaner and separated the inram volume creation and docker extraction. On Arista platform with a flash smaller or equal to 4GB set `docker_inram_algo` to `zstd` which produces the best compression ratio at the detriment of a slower write performance and a similar read performance to other `zram` compression algorithms.
…15287) Why I did it Update cable length for uplink/downlink ports for chassis and and update PG/pool headroom size accordingly. Work item tracking 17880812 How I did it Updated cable length as well as buffer config in HWSKU files.
…lly (#15237) src/sonic-swss * 4d9fb93 - (HEAD -> 202205, origin/202205) [orchagent][neighorch] traffic convergence acceleration for staticroutebfd (#2769) (6 days ago) [Baorong Liu] * 43c88a3 - Fix pipeline issue caused by urllib3 v2 (6 days ago) [Liu Shilong]
…5112) (#15279) Why I did it Don't auto update package version for release branch. Work item tracking Microsoft ADO (number only): 22335854 How I did it How to verify it
…atically (#15238) src/sonic-utilities * cd08aa69 - (HEAD -> 202205, origin/202205) [show][muxcable] add some new commands health, reset-cause, queue_info support for muxcable (#2853) (2 days ago) [vdahiya12] * 4b96bd7e - Update pcieutil error message on loading common pcie module (#2786) (6 days ago) [cytsao1] * 77b725ca - Fix the show interface counters throwing exception on device with no external interfaces (#2851) (6 days ago) [abdosi]
…ded (#14933) (#15316)
…D automatically (#15318) src/sonic-platform-daemons * 616f13f - (HEAD -> 202205, origin/202205) [ycabled] fix bug for `show mux status` delayed response (#364) (6 hours ago) [vdahiya12]
…lly (#15319) src/sonic-swss * c781521 - (HEAD -> 202205, origin/202205) [swss][orchagent] fix srt-bfd ut (#2803) (6 hours ago) [Baorong Liu]
…atically (#15320) src/sonic-utilities * ec472146 - (HEAD -> 202205, origin/202205) fix show interface neighbor expected empty issue (#2465) (3 minutes ago) [jcaiMR] * d1f4413c - [vlan][dhcp_relay] Clear dhcpv6 relay counter while deleting vlan (#2852) (6 hours ago) [Yaqiang Zhu] * 051f28ce - [db-migrator] Fix hwsku match for 6100 and add errors when hwsku is None (#2821) (7 hours ago) [Vaibhav Hemant Dixit]
…D automatically (#15344) src/sonic-platform-daemons * 1537d6a - (HEAD -> 202205, origin/202205) [dualtor] Fix command `show mux status` (#371) (2 minutes ago) [Longxiang Lyu]
…D automatically (#15347) src/sonic-platform-daemons * a90bff5 - (HEAD -> 202205, origin/202205) [ycabled] correct the wrong function call for 'config hwmode state' (#372) (5 minutes ago) [vdahiya12]
…D automatically (#15366) src/sonic-platform-daemons * 18815c7 - (HEAD -> 202205, origin/202205) Revert "[ycabled] refactor code for onboarding async client changes;refactor (#355)" (3 minutes ago) [Ying Xie] * 5324554 - Revert "add async notification support in active-active topo; refactor code for ycable tasks for change events (#327)" (3 minutes ago) [Ying Xie] * cbbe2b5 - Revert "[ycabled] fix bug for `show mux status` delayed response (#364)" (3 minutes ago) [Ying Xie] * 9746709 - Revert "[dualtor] Fix command `show mux status` (#371)" (3 minutes ago) [Ying Xie] * 551ab3c - Revert "[ycabled] correct the wrong function call for 'config hwmode state' (#372)" (3 minutes ago) [Ying Xie]
…atically (#15367) src/sonic-utilities * 75ca81e4 - (HEAD -> 202205, origin/202205) [202205][config]Support multi-asic Golden Config override with fix (#2825) (#2862) (13 hours ago) [jingwenxie]
…atically (#15373) src/sonic-telemetry * 5a83c07 - (HEAD -> 202205, origin/202205) Merge pull request #109 from zbud-msft/backport-202205 (86 minutes ago) [Ying Xie] * 0f75a64 - [202012] Workaround gomonkey for armhf (#88) (3 weeks ago) [Zain Budhwani] * 1b0dc75 - Fix Makefile error (3 weeks ago) [Zain Budhwani] * 72e4540 - Add diff cov (#85) (3 weeks ago) [Zain Budhwani] * 11aace5 - Add get-update to azp yml (#79) (3 weeks ago) [Zain Budhwani] * 3eaaccc - Add net core and code coverage results (#77) (3 weeks ago) [Zain Budhwani] * 3cf3883 - Enable unit test (3 weeks ago) [ganglyu] * 2a0928f - Change dir name in pipeline (#75) (4 weeks ago) [Zain Budhwani] * baa845a - Update yml (4 weeks ago) [Zain Budhwani] * c533d52 - Fix format (4 weeks ago) [ganglyu]
… automatically (#15372) src/sonic-platform-common * 4969f70 - (HEAD -> 202205, origin/202205) Skip CDB and VDM for flat memory modules (#281) (2 minutes ago) [Prince George]
gechiang
requested review from
a team,
StormLiangMS,
qiluo-msft,
xumia and
lguohan
as code owners
jon-nokia
pushed a commit
to jon-nokia/sonic-buildimage-msft
that referenced
this pull request
Aug 9, 2023
Added below commits: 9b30690 jcaiMR Fri Dec 16 fix handleSwssNotification crash in dhcp6relay (Azure#28) 047afb7 jcaiMR Wed Dec 14 14:08:58 2022 +0800 Fix multiple vlan issue (Azure#27) ff6bec3 Vivek Thu Dec 8 09:44:15 2022 -0800 Made the Error log informative (Azure#22) 2fbe729 jcaiMR Wed Nov 30 14:41:53 2022 +0800 disable cfg dynamic change (Azure#25) 13d0805 Liu Shilong Wed Nov 30 10:54:11 2022 +0800 Use github code scanning instead of LGTM (Azure#26) 1e846f6 kellyyeh Wed Nov 23 14:36:02 2022 -0800 Fix packet range check for relay-reply packets (Azure#21) 4d19e13 kellyyeh Thu Nov 17 10:04:53 2022 -0800 Add unittest infrastructure (Azure#5) 7f4fdab jcaiMR Fri Nov 11 14:47:51 2022 +0800 fix packet range check issue (Azure#20) 257ecdf kellyyeh Thu Nov 3 11:34:11 2022 -0700 Add client packet UDP header length check (Azure#19)
r12f
pushed a commit
that referenced
this pull request
Dec 19, 2024
…tically (#20083) #### Why I did it src/sonic-dash-api ``` * 4919d9e - (HEAD -> master, origin/master, origin/HEAD) Allow `underlay_sip` to coexist with `vnet` in Route table (#27) (4 hours ago) [Lawrence Lee] ``` #### How I did it #### How to verify it #### Description for the changelog
mssonicbld
added a commit
that referenced
this pull request
Feb 18, 2025
… automatically (#632) #### Why I did it src/sonic-platform-common ``` * 43b7a46 - (HEAD -> 202405, origin/202405) [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#27) (7 hours ago) [mssonicbld] * 388067b - [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#25) (31 hours ago) [mssonicbld] * a89da71 - [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#23) (2 days ago) [mssonicbld] * 6d32a64 - [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#21) (3 days ago) [mssonicbld] * 68eca1d - [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#19) (4 days ago) [mssonicbld] * 3052012 - [cmis] Optimize cmis.get_error_description speed for passive module (#541) (#17) (5 days ago) [mssonicbld] ``` #### How I did it #### How to verify it #### Description for the changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why I did it
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
A picture of a cute animal (not mandatory but encouraged)