Skip to content

Fix sonic-swss-common build#1206

Merged
yxieca merged 1 commit into
sonic-net:masterfrom
nexthop-ai:fix-ci
Jun 13, 2026
Merged

Fix sonic-swss-common build#1206
yxieca merged 1 commit into
sonic-net:masterfrom
nexthop-ai:fix-ci

Conversation

@bgallagher-nexthop

@bgallagher-nexthop bgallagher-nexthop commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Background / Motivation

I noticed that the sonic-swss-common build is failing during compilation because libyang python dependencies are not correctly installed by the build pipeline prior to attempting to compile.

This PR contains a short-term fix. The proper fix is in sonic-buildimage (sonic-net/sonic-buildimage#27836), but won't resolve the problem until the next nightly build fires and publishes the correct packages. This PR can be reverted once that fix takes effect.

Testing

The compilation step in the build pipeline now works as expected: https://dev.azure.com/mssonic/build/_build/results?buildId=1136291&view=logs&j=a5b8631f-d2a8-5bf4-c5b7-ed5dd46e146c&t=b99e062e-9dda-5ff0-ad0b-d7a7533b64b9

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@bgallagher-nexthop bgallagher-nexthop marked this pull request as ready for review June 11, 2026 21:59
Signed-off-by: Brian Gallagher <bgallagher@nexthop.ai>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR applies a short-term CI workaround in sonic-swss-common to unblock compilation when the build pipeline lacks the python3-libyang bindings, by installing the libyang Python CFFI bindings from PyPI after installing the libyang3 Debian packages.

Changes:

  • Install libyang==3.3.0 via pip3 in the Azure pipeline after installing downloaded .deb artifacts.
  • Add inline rationale comments explaining why the PyPI install is needed.

Comment thread .azure-pipelines/build-template.yml
Comment thread .azure-pipelines/build-template.yml
Comment thread .azure-pipelines/build-template.yml
@bgallagher-nexthop

bgallagher-nexthop commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Resolving Copilot feedback: this is intentionally a short-term fix. The "proper" fix is in sonic-buildimage PR sonic-net/sonic-buildimage#27836 but after merging, sonic-swss-common builds will still fail until the next periodic full build completes and publishes the right packages we need. This PR can be reverted once that has happened.

@yxieca yxieca merged commit a13d47f into sonic-net:master Jun 13, 2026
19 checks passed
@mssonicbld

Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202605: #1209

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.

4 participants