Skip to content

[submodule][master] Update submodule sonic-host-services to the latest HEAD automatically #22160

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

Merged

Conversation

mssonicbld
Copy link
Collaborator

@mssonicbld mssonicbld commented Mar 27, 2025

Why I did it

src/sonic-host-services

* aa971f6 - (HEAD -> master, origin/master, origin/HEAD) Revert "[featured] fix non existing feature start (#234)" (#272) (29 hours ago) [Qi Luo]
* 04e49f8 - Add iptables rules to drop all bgp packets destined for loopback1 IP addresses in dual ToR (#262) (11 days ago) [Yawen]
* 727e21b - [Caclmgrd]Fix key error issue about acl_source_ip_map (#268) (11 days ago) [Changrong Wu]
* 4cb6b97 - Restart caclmgrd whenever catch exception in child thread or in main thread (#194) (13 days ago) [Zhaohui Sun]
* d7842aa - config_db should not mandate True and False for featured (#252) (2 weeks ago) [Brad House - NextHop]
* 70de4d1 - Fix PyGObject required version (#264) (3 weeks ago) [Qi Luo]
* 953816b - Modify cacl rules to allow incoming packets to dash-ha (#256) (3 weeks ago) [yue-fred-gao]
* 5460c29 - Improve reboot status tracking and add reboot count & method to status (#259) (3 weeks ago) [Vasundhara Volam]
* e46e36e - Update ubuntu version in azure pipeline (#260) (4 weeks ago) [Vasundhara Volam]
* 23b81f9 - Fix reboot without message (#250) (4 weeks ago) [jkmar]
* 1ee792e - Implement Download and Delete For File Service (#242) (5 weeks ago) [Dawei Huang]
* 00393e3 - [kdump-config] Update kdump default config with pre-defined value in /proc/cmdline (installer.conf)  (#237) (7 weeks ago) [Marty Y. Lok]
* d028745 - [Radius] Add require_message_authenticator option to fix BlastRadius issue. (#172) (7 weeks ago) [Julian Chang]
* 8237b5d - Monitor rsyslog feature flag change and update rsyslog config by restart rsyslog-config service (#245) (8 weeks ago) [Hua Liu]
* 55cee71 - Improve Detection Logic for HALT Command in is_halt_command_running() (#247) (8 weeks ago) [Vasundhara Volam]
* 29b8be8 - [featured] fix non existing feature start (#234) (3 months ago) [Stepan Blyshchak]
* fe90cb1 - Adding support for filtering by destination IP and/or incoming interface to caclmgrd. (#195) (3 months ago) [nyanto]
* 6006e05 - caclmgrd interface rules patch 1 (#197) (3 months ago) [Nanma Purushotam]
* 3ba0c59 - Pin PyGObject to 3.50.0 (#228) (3 months ago) [cpackham-atlnz]
* 5445cf6 - Radius Access request to use configured src interface (#230) (3 months ago) [lakshmi-nexthop]

How I did it

How to verify it

Description for the changelog

@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from e43b909 to 5646143 Compare March 28, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 5646143 to e2b1373 Compare March 29, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from e2b1373 to 3103f82 Compare March 30, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 3103f82 to 106b3a1 Compare March 31, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn
Copy link
Contributor

kperumalbfn commented Mar 31, 2025

@cpackham-atlnz @dgsudharsan

With pygobject=3.5.0, we are seeing build failures(marvell_prestera_armhf), could you please check.

Collecting PyGObject==3.50.0
Downloading pygobject-3.50.0.tar.gz (1.1 MB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.1/1.1 MB 7.0 MB/s eta 0:00:00
Installing build dependencies: started
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: still running...
Installing build dependencies: finished with status 'error'
error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.
│ exit code: 1
╰─> [793 lines of output]
Collecting meson-python>=0.12.1
Downloading meson_python-0.17.1-py3-none-any.whl (27 kB)
Collecting pycairo>=1.16
Downloading pycairo-1.27.0.tar.gz (661 kB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 661.5/661.5 kB 5.5 MB/s eta 0:00:00
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'done'
Installing backend dependencies: started
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: still running...
Installing backend dependencies: finished with status 'error'
error: subprocess-exited-with-error

    × pip subprocess to install backend dependencies did not run successfully.
    │ exit code: 1

@cpackham-atlnz
Copy link

I assume you're referring to this result looks like the actual error is

                        -- Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY OPENSSL_INCLUDE_DIR)
                        CMake Error at Utilities/cmcurl/CMakeLists.txt:943 (message):
                          Could not find OpenSSL.  Install an OpenSSL development package or
                          configure CMake with -DCMAKE_USE_OPENSSL=OFF to build without OpenSSL.

I don't think it's related to the pygobject pinning since the other targets all succeeded.

@cpackham-atlnz
Copy link

cpackham-atlnz commented Mar 31, 2025

So I think there is a problem that we have

files/build/versions/dockers/sonic:slave-bookworm/versions-py3:112:pygobject==3.42.2
files/build/versions/host:image/versions-py3-all-armhf:1:pygobject==3.42.2

Which obviously causes a conflict with the sonic-host-services change but then I'm not sure why we had the original build failure that my change was attempting to fix.

I'm not sure if changing that to match will fix the actual problem but it probably should be done. How can I add that to the auto generated bumps?

@cpackham-atlnz
Copy link

I'm having problems building the armhf target locally even without the host-services change

make configure PLATFORM=marvell PLATFORM_ARCH=armhf
+++ Cheking platform/marvell +++
+++ Making configure +++
BLDENV=buster make -f Makefile.work configure
make[1]: Entering directory '/home/chrisp/src/sonic-buildimage'
"SECURE_UPGRADE_PROD_SIGNING_TOOL": ""
~/src/sonic-buildimage/src/sonic-build-hooks ~/src/sonic-buildimage
make[2]: Entering directory '/home/chrisp/src/sonic-buildimage/src/sonic-build-hooks'
dpkg-deb: building package 'sonic-build-hooks' in 'buildinfo/sonic-build-hooks_1.0_all.deb'.
make[2]: Leaving directory '/home/chrisp/src/sonic-buildimage/src/sonic-build-hooks'
~/src/sonic-buildimage
...
Killing march docker 
setfacl: /var/run/march/docker.sock: No such file or directory
make[1]: *** [Makefile.work:605: configure] Error 1
make[1]: Leaving directory '/home/chrisp/src/sonic-buildimage'
make: *** [Makefile:112: configure] Error 2

@cpackham-atlnz
Copy link

So my first problem was caused because my local /etc/docker/daemon.json has a data-root: entry that points docker at an alternate location for storage (due to having a small / and a large /home) that causes the cross docker daemons to fail with unable to configure the Docker daemon with file /etc/docker/daemon.json: the following directives are specified both as a flag and in the configuration file: data-root: (from flag: /tmp/d/d/var/lib/docker/, from file: /home/docker).

I've kicked off a build now but looks like it'll take a while

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 106b3a1 to adc3629 Compare April 1, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@cpackham-atlnz
Copy link

Eventually my local make configure failed with

 docker: Error response from daemon: Duplicate mount point: /var/lib/docker

so I still can't get to a point of seeing the failure from the pipeline

I think the fix might be as simple as updating files/build/versions/host-image/versions-py3-all-armhf:2:pygobject==3.42.2 but it's kind of hard for me to confirm that

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from adc3629 to 9db04b0 Compare April 2, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 9db04b0 to 0f6a327 Compare April 3, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 3c2434d to 0679678 Compare June 15, 2025 03:06
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 0679678 to 95f11c4 Compare June 16, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from 95f11c4 to a9066ad Compare June 17, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from c2aefc4 to a967204 Compare June 19, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from a967204 to e2296b5 Compare June 20, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from e2296b5 to cebb789 Compare June 21, 2025 03:02
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld force-pushed the submodule-master-sonic-host-services branch from cebb789 to 5d1fc81 Compare June 22, 2025 03:05
@mssonicbld
Copy link
Collaborator Author

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 512a05a into sonic-net:master Jun 22, 2025
18 checks passed
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.

6 participants