Skip to content

[azp] Download and install libyang3 instead of libyang, add libnexthopgroup#1916

Merged
lolyu merged 2 commits into
sonic-net:masterfrom
theasianpianist:libyang3-support
May 29, 2026
Merged

[azp] Download and install libyang3 instead of libyang, add libnexthopgroup#1916
lolyu merged 2 commits into
sonic-net:masterfrom
theasianpianist:libyang3-support

Conversation

@theasianpianist

@theasianpianist theasianpianist commented May 28, 2026

Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Installing libyang3 in the CI pipeline. Also incorporate changes from #1908 to avoid circular dependency with this PR.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Documentation update
  • Test improvement

Approach

What is the motivation for this PR?

sonic-net/sonic-swss-common#973 changed swss-common to depend on libyang3

Work item tracking
  • Microsoft ADO (number only):

How did you do it?

When building in the PR check pipeline, download and install libyang3. A similar change was made for sonic-swss: sonic-net/sonic-swss#4618

How did you verify/test it?

Any platform specific information?

Documentation

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

prsunny
prsunny previously approved these changes May 28, 2026
@prsunny

prsunny commented May 28, 2026

Copy link
Copy Markdown
Contributor

@saiarcot895 , please help merge

saiarcot895
saiarcot895 previously approved these changes May 28, 2026
sonic-swss PR #4394 added -lnexthopgroup to fpmsyncd link
dependencies, but only updated swss's own CI pipeline. This causes
sairedis CI to fail when compiling swss because libnexthopgroup
is missing from the build environment.

Add libnexthopgroup deb download and installation to:
- build-swss-template.yml (swss compilation environment)
- build-docker-sonic-vs-template.yml (VS image build)
- docker-sonic-vs/Dockerfile (VS image runtime)

Signed-off-by: Yuqing Zhao <galadriel.zyq@alibaba-inc.com>
@theasianpianist theasianpianist dismissed stale reviews from saiarcot895 and prsunny via d3d561e May 28, 2026 22:35
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

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

@theasianpianist theasianpianist changed the title [azp] Download and install libyang3 instead of libyang [azp] Download and install libyang3 instead of libyang, add libnexthopgroup May 28, 2026

@GaladrielZhao GaladrielZhao 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.

Thanks, looks good :)

@lolyu lolyu merged commit e92402c into sonic-net:master May 29, 2026
19 checks passed
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.

7 participants