Skip to content

Commit

Permalink
Add Bookworm build to PR checks (#1371)
Browse files Browse the repository at this point in the history
* Add Bookworm build to PR checks

Add a step to verify the build for Bookworm for amd64, armhf, and arm64.

Signed-off-by: Saikrishna Arcot <[email protected]>

* Enable unit tests

Signed-off-by: Saikrishna Arcot <[email protected]>

* Manually start and stop rsyslogd

For Bookworm, the init.d service file for rsyslogd is removed, and the
only script to start and stop rsyslogd is the systemd service file.
However, that can only be used if systemd is being used as the init
manager, which isn't the case for our docker containers.

As an alternative, manually start and stop rsyslogd ourselves.

Signed-off-by: Saikrishna Arcot <[email protected]>

---------

Signed-off-by: Saikrishna Arcot <[email protected]>
  • Loading branch information
saiarcot895 authored Apr 12, 2024
1 parent d9ba01b commit be47489
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .azure-pipelines/build-template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ jobs:
sudo mkdir -m 755 /var/run/sswsyncd
sudo apt-get install -y rsyslog
sudo service rsyslog start
sudo rsyslogd
cat /etc/apt/sources.list
dpkg --list | grep libnl
Expand Down Expand Up @@ -167,7 +167,8 @@ jobs:
displayName: "Compile sonic sairedis with coverage enabled"
- script: |
sudo cp azsyslog.conf /etc/rsyslog.conf
sudo service rsyslog restart
sudo killall rsyslogd
sudo rsyslogd
displayName: "Update rsyslog.conf"
- ${{ if eq(parameters.run_unit_test, true) }}:
- script: |
Expand Down
36 changes: 36 additions & 0 deletions azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,42 @@ stages:
syslog_artifact_name: sonic-sairedis.syslog.arm64
debian_version: ${{ parameters.debian_version }}

- stage: BuildBookworm
dependsOn: BuildArm
condition: succeeded('BuildArm')
jobs:
- template: .azure-pipelines/build-template.yml
parameters:
arch: amd64
sonic_slave: sonic-slave-bookworm
swss_common_artifact_name: sonic-swss-common-bookworm
artifact_name: sonic-sairedis-bookworm
syslog_artifact_name: sonic-sairedis-bookworm.syslog
run_unit_test: true
debian_version: ${{ parameters.debian_version }}

- template: .azure-pipelines/build-template.yml
parameters:
arch: armhf
timeout: 180
pool: sonicbld-armhf
sonic_slave: sonic-slave-bookworm-armhf
swss_common_artifact_name: sonic-swss-common-bookworm.armhf
artifact_name: sonic-sairedis-bookworm.armhf
syslog_artifact_name: sonic-sairedis-bookworm.syslog.armhf
debian_version: ${{ parameters.debian_version }}

- template: .azure-pipelines/build-template.yml
parameters:
arch: arm64
timeout: 180
pool: sonicbld-arm64
sonic_slave: sonic-slave-bookworm-arm64
swss_common_artifact_name: sonic-swss-common-bookworm.arm64
artifact_name: sonic-sairedis-bookworm.arm64
syslog_artifact_name: sonic-sairedis-bookworm.syslog.arm64
debian_version: ${{ parameters.debian_version }}

- stage: BuildSwss
dependsOn: Build
condition: succeeded('Build')
Expand Down

0 comments on commit be47489

Please sign in to comment.