Skip to content
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

[code sync] Merge code from sonic-net/sonic-buildimage:202205 to 202205 #64

Merged
merged 7 commits into from
Jul 14, 2023

Conversation

mssonicbld
Copy link
Collaborator

* fb14b987b - (head/202205) Add health check probe for k8s upgrade containers. (#15223) (2023-07-14) [lixiaoyuner]
* 9cd331949 - Pick dependency files in submodules. (#15142) (#15827) (2023-07-14) [mssonicbld]
* e73924ebc - Upgrade XGS SAI version to 7.1.54.4 (#15820) (2023-07-13) [zitingguo-ms]
* 2d39daf95 - [submodule] Update submodule sonic-utilities to the latest HEAD automatically (#15809) (2023-07-13) [mssonicbld]
* 7c6a1612d - [ci/build]: Upgrade SONiC package versions (#15766) (2023-07-13) [mssonicbld]
* c39e37059 - Update cisco-8000.ini (#15813) (2023-07-13) [James An]<br>```

jamesan47 and others added 7 commits July 13, 2023 08:26
…atically (#15809)

src/sonic-utilities

* 20853a6f - (HEAD -> 202205, origin/202205) Revert "[GCU Feature Update] Cherry-pick Platform Validator PR to 202205  (#2883)" (#2908) (6 hours ago) [isabelmsft]
Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
#### Why I did it
After k8s upgrade a container, k8s can only know the container is running, don't know the service's status inside container. So we need a probe inside container, k8s will call the probe to check whether the container is really ready.
##### Work item tracking
- Microsoft ADO **(number only)**: 22453004
#### How I did it
Add a health check probe inside config engine container, the probe will check whether the start service exit normally or not if the start service exists and call the python script to do container self-related specific checks if the script is there. The python script should be implemented by feature owner if it's needed.

more details: [design doc](https://github.com/sonic-net/SONiC/blob/master/doc/kubernetes/health-check.md)
#### How to verify it
Check path /usr/bin/readiness_probe.sh inside container.

#### Which release branch to backport (provide reason below if selected)

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [x] 202205
- [x] 202211

#### Tested branch (Please provide the tested image version)
- [x] 20220531.28
@mssonicbld mssonicbld merged commit 9b25536 into sonic-net:202205 Jul 14, 2023
3 checks passed
jon-nokia pushed a commit to jon-nokia/sonic-buildimage-msft that referenced this pull request Aug 9, 2023
Update sonic-mgmt-common submodule pointer to include the following:

68e0341 Remove LOGLEVEL DB since is no longer used (sonic-net#64)

Signed-off-by: dprital <drorp@nvidia.com>
jon-nokia pushed a commit to jon-nokia/sonic-buildimage-msft that referenced this pull request Aug 9, 2023
Incorporates following commits:

43a9179 Zain Budhwani Mon Dec 5 13:44:16 2022 -0800 Call evtc_stop after error (sonic-net#64)
5712679 pettershao-ragilenetworks Fri Dec 2 11:04:08 2022 +0800 Fix the cfg variable configuration bug. (sonic-net#65)
mssonicbld added a commit to mssonicbld/sonic-buildimage-msft that referenced this pull request Dec 7, 2023
…ically (#17418)

#### Why I did it
src/sonic-dbsyncd
```
* 68baf40 - (HEAD -> 202305, origin/202305) [lldp-syncd] Fix unexpected exception in snmp-subagent (sonic-net#64) (18 hours ago) [Zhaohui Sun]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit to mssonicbld/sonic-buildimage-msft that referenced this pull request Dec 13, 2023
…ically (#17449)

src/sonic-dbsyncd

* cde84fa - (HEAD -> 202205, origin/202205) [lldp-syncd] Fix unexpected exception in snmp-subagent (sonic-net#64) (4 days ago) [Zhaohui Sun]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants