The bug is fixed #6
Closed
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
…orms (sonic-net#6304) Mellanox already supports multiple destination IPs in IPinIP tunnel configuration, thus removing mellanox exception for IPinIP configuration. - How I did it Removed "dst_ip" field generation in mellanox platform condition. Sorted the "dst_ip" list, so that it is easier to test against sample configuration in unit tests. Aligned unit test sample. Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
instruct azure pipeline to use node.js in the docker. https://docs.microsoft.com/en-us/azure/devops/pipelines/process/container-phases?view=azure-devops#bring-your-own-nodejs Signed-off-by: Guohan Lu <lguohan@gmail.com>
…et#6301) - No longer install Python 2 packages in host: - libpython2.7-dev - docker - ipaddress - netifaces - azure-storage - watchdog - futures - Install Python 3 versions of the following packages in host: - docker - azure-storage - watchdog - redis - swsssdk (install unconditionally)
- Convert system-health scripts to Python 3 - Build and install system-health as a Python 3 wheel - Also convert newlines from DOS to UNIX
use bash -t test flag to check if input device is tty or not Signed-off-by: Guohan Lu <lguohan@gmail.com>
…onic-net#6319) sonic-slave tag only allows all lower case. In case the user name is mixed case, we need to change user name to all lower case. Signed-off-by: Guohan Lu <lguohan@gmail.com>
…rk (sonic-net#6315) Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Depending on the performance characteristics of a given hardware platform, it's possible to exceed the default 120 second kernel timeout during I/O intensive operations like image installation. This can cause a kernel panic like so: kernel:[ 852.441781] Kernel panic - not syncing: hung_task: blocked tasks If this happens during image installation, it's possible for the install to become corrupted and leave the device in an unreachable state that requires a power cycle to resolve. This risk increases as image size continues to increase. So, we need to increase the timeout so that we don't encounter kernel panics on devices with lower disk throughput. Signed-off-by: Danny Allen <daall@microsoft.com>
- Why I did it Make EEPROM platform APIs Python3 compliant in Nokia platform. - How I did it Handle bytearray type returned by read_eeprom and read_eeprom_bytes methods. - How to verify it Boot Nokia ixs7215 and verify PMON docker running and show platform syseeprom Co-authored-by: Carl Keene <keene@nokia.com>
Why I did it To support FG_ECMP scenarios - How I did it Modified minigraph parser to parse ECMP fields in the case they are present in minigraph - How to verify it Loaded ensuing config_db file on a DUT to verify the fields are parsed and configure device correctly
…or hardware_reboot_cause (sonic-net#6325) What: Modify unit test to not call any platform dependent api in test_find_hardware_reboot_cause. - Why I did it MELLANOX build is failing for the recent PRs. The errors are due to platform library being invoked in a unit test for determine-reboot-cause script. Verified by running unit tests and a successful Mellanox build. Co-authored-by: Vaibhav Hemant Dixit <vadixit@microsoft.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
…sonic-net#6298) Enable the notify mode of rsyslogd imfile module used for supervisord logs in docker container
Including commits in sonic-swss-common repo: b423b9c Add support for hexists call (sonic-net#432) [Tony Titus] 0982996 Remove extension of tableNameSeparatorMap (sonic-net#430) [Qi Luo] d16cc76 [build]: add azure pipeline build badge (sonic-net#429) [lguohan] f2aaf55 Set up CI with Azure Pipelines (sonic-net#428) [lguohan]
…net#6333) Features: Spectrum-3 | Systems | Added GA-level support for SN4700 A0 system All | Shared headroom | Added GA-level support for Shared headroom between PGs Bugs fixes: All | Counters | Sent traffic in certain size is wrongly increase to a smaller size counter, because port extended counter has a counter for sent traffic per packet-size range All | Shared buffer | Configuring shared buffer on the fly may, on occasion, cause the chip to get stuck Spectrum-2 | Modules | On occasion, link down is experienced with INPHI COLORZ PAM4 100G optic cables on SN3700 systems
Accton util applies lsmod to check if drivers are installed. But lsmod may return error on startup and skip module installation. Signed-off-by: Brandon Chuang <brandon_chuang@edge-core.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
fix a bug when there are multiple debian packages to be uninstalled Signed-off-by: Guohan Lu <lguohan@gmail.com>
in case conflicting packages have same name but different version, the install step needs to wait till package with the conflicting version is uninstalled Signed-off-by: Guohan Lu <lguohan@gmail.com>
no instalation of frr during the build process Signed-off-by: Guohan Lu <lguohan@gmail.com>
frr build requires libyang 1.0.184 which conflicts with libyang 1.0.73. Solution here is to compile frr and libyang 1.0.184 first, and then uninstall libyang 1.0.184 after frr build. Then, compile libyang 1.0.73 and all packages depend on it later. Signed-off-by: Guohan Lu <lguohan@gmail.com>
…ion (sonic-net#6309) * [sonic-py-common] add an API to get file path containing SONiC version so that the API can be mocked for unit tests.
…on (sonic-net#6339) the control file version is 4.2.1.5-7 sonic$ sudo dpkg -i target/debs/buster/libsaibcm-dev_4.2.1.5-8_amd64.deb (Reading database ... 175880 files and directories currently installed.) Preparing to unpack .../libsaibcm-dev_4.2.1.5-8_amd64.deb ... Unpacking libsaibcm-dev (4.2.1.5-7) over (4.2.1.5-7) ... Setting up libsaibcm-dev (4.2.1.5-7) ... lgh@491d842369cf:/sonic$ sudo dpkg -i target/debs/buster/libsaibcm_4.2.1.5-8_amd64.deb (Reading database ... 175880 files and directories currently installed.) Preparing to unpack .../libsaibcm_4.2.1.5-8_amd64.deb ... Unpacking libsaibcm (4.2.1.5-7) over (4.2.1.5-7) ... Setting up libsaibcm (4.2.1.5-7) ... Processing triggers for libc-bin (2.28-10) ... Signed-off-by: Guohan Lu <lguohan@gmail.com>
… image (sonic-net#6340) build frr dbg image force to install frr in the build process which breaks the current build and is uneccessary. Signed-off-by: Guohan Lu <lguohan@gmail.com>
* Add changes to allow migration of bfn syncd to buster * Update BFN packages for Debian 10 Signed-off-by: Denys Petryshyn <denysx.petryshyn@intel.com>
In the emulated armhf environment, the function readdir()returns NULL on a ext4 file system directory. When running the libyang1 test cases, it will require to load the plugins from the files (such as metadata.so), because the readdir() is failing, the plugins can’t be loaded in the emulated armhf environment, so it causes libyang1 test error. This error is a combination of the following reasons. • Emulation of a 32-bit target from a 64-bit host –> qemu from x86_64 to armhf • Glibc version > 2.27 – Debian buster is using glibc 2.28 - How I did it Enabled large file support by setting _FILE_OFFSET_BITS=64 for libyang1. Signed-off-by: Sabareesh Kumar Anandan <sanandan@marvell.com>
Fix syntax errors during xcvrd start with Python 3 daemons
Includes below commits ``` 9e91e0d 2021-02-04 | Fix: RedisClient.set() should return bool (sonic-net#454) [Qi Luo] 244a0bf 2021-02-04 | [swig] Add missing functions for ConfigDBConnector (sonic-net#453) [Qi Luo] e659698 2021-01-22 | [dbconnector] Make sure json is declared as non null object (sonic-net#452) [Kamil Cudnik] ```
…rse_dom_capability' (sonic-net#6695) **- Why I did it** PR sonic-net/sonic-platform-common#102 modified the name of the SFF-8436 (QSFP) method to align the method name between all drivers, renaming it from `parse_qsfp_dom_capability` to `parse_dom_capability`. Once the submodule was updated, the callers using the old nomenclature broke. This PR updates all callers to use the new naming convention. **- How I did it** Update the name of the function globally for all calls into the SFF-8436 driver. Note that the QSFP-DD driver still uses the old nomenclature and should be modified similarly. I will open a PR to handle this separately.
**- Why I did it** To support nightly build for master branch **- How to verify it** confirm from azure pipeline page.
Port_config update for hwsku 7050CX3-32S-C3 - add two 10G ports. This change is added to fix issue of "PortsOrch initialization failure" seen by previous removal of these 10G ports. Tested on the device with new minigraph, and the PortsOrch initialization failure is not seen.
Signed-off-by: Guohan Lu <lguohan@gmail.com>
* Remove makefile and As part of sonic-net#6046 we have moved to python3 of dbsyncd. Cleanup the python2 makefile and dependency. Signed-off-by: Abhishek Dosi <abdosi@microsoft.com> * Revert "Remove makefile and dependency for python2 of dbsyncd." This reverts commit ceaa4f8. * Fix Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
…6696) prerm is needed for platform modules package to be properly removed. Added prerm to remove installed in postinst wheel packages. Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>
official build only use scheduled build Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
check the uid before modify local user account.
when run sudo, the process the invoke nss_tacplus library
does not have priviledge to modify the user profile, and
will generate below error messages:
user_rw@sonic:~$ sudo bash
usermod: Permission denied.
usermod: cannot lock /etc/passwd; try again later.
usermod: Permission denied.
usermod: cannot lock /etc/passwd; try again later.
Signed-off-by: Guohan Lu <lguohan@gmail.com>
sonic-utilities will become dependent upon sonic-platform-common as of sonic-net/sonic-utilities#1386.
…ic-net#6321) Accton util applies lsmod to check if drivers are installed. But lsmod may return error on startup and skip module installation. Signed-off-by: roy_lee <roy_lee@edge-core.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
- [ci]: add default values to build template - [ci]: add marvel armhf official build Signed-off-by: Guohan Lu <lguohan@gmail.com>
* [device] add platform.json hwsku.json for Montara * [device] add autonge, fec fields to hwsku
**- Why I did it** After migrating to python3, the operator '/' always get a float result, but it gets integer result in python2. Need fix this in thermal_conditions. **- How I did it** 1. cast float value to int 2. change the unit test case to cover this situation **- How to verify it** Manually test and regression test
proper bool comparision Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
…ic-net#6713) Fix sonic-net#6711 the requirement was introduced in commit 75104bb to support sflow in stretch build. in buster build, the requirement is met, no need to pin down the version. Signed-off-by: Guohan Lu <lguohan@gmail.com>
restructure repo clean up to make it robust Signed-off-by: Guohan Lu <lguohan@gmail.com>
… IN_PORTS for TD3 (sonic-net#6718) ACL entry set attribute updates all the entries in the table. The correct behavior is to set the attribute on single entry. - How I did it Current SDK code, while setting the new attribute, is going through all the entries and updating it. Added a logic to check for requested entry and only allow for that ACL entry. A case has filed with BRCM. Once an official fix is provided by BRCM, we will then remove this in house fix and apply the official fix.
) - Why I did it To move ‘sonic-host-service’ which is currently built as a separate package to ‘sonic-host-services' package. - How I did it - Moved 'sonic-host-server' to 'src/sonic-host-services' and included it as part of the python3 wheel. - Other files were moved to 'src/sonic-host-services-data' and included as part of the deb package. - Changed build option ‘INCLUDE_HOST_SERVICE’ to ‘ENABLE_HOST_SERVICE_ON_START’ for enabling sonic-hostservice at boot-up by default.
xumia
pushed a commit
that referenced
this pull request
Jun 29, 2023
…sonic-net#15634) #### Why I did it src/dhcpmon ``` * 824a144 - (HEAD -> master, origin/master, origin/HEAD) replace atoi with strtol (#6) (3 hours ago) [Mai Bui] * 32c0c3f - Fix libswsscommon package installation for non-amd64 (#7) (6 hours ago) [Saikrishna Arcot] ``` #### How I did it #### How to verify it #### Description for the changelog
xumia
pushed a commit
that referenced
this pull request
Feb 8, 2024
Why I did it Advance dhcpmon to a3c5381 in 202305 branch. a3c5381 - (HEAD, origin/master, origin/HEAD, master) Merge pull request src: Add libnl3 build.sh script #11 from jcaiMR/dev/jcai_fix_err_log (11 days ago) [StormLiangMS] c5ef7e7 - Change common_libs dependencies from buster to bullseye (Updating docker-orchagent/syncd Dockerfile and start.sh #9) 824a144 - replace atoi with strtol (Rename hostname #6) (10 weeks ago) [Mai Bui] 32c0c3f - Fix libswsscommon package installation for non-amd64 (README.md leaves out docker-database #7) (10 weeks ago) [Saikrishna Arcot] Work item tracking Microsoft ADO (25048723): How I did it How to verify it Run test_dhcp_relay.py, no failure
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.
Everything works again