Skip to content

azure-pipelines: install libyang3 instead of libyang1#386

Open
bhouse-nexthop wants to merge 1 commit into
sonic-net:masterfrom
bhouse-nexthop:bhouse.libyang3-debs
Open

azure-pipelines: install libyang3 instead of libyang1#386
bhouse-nexthop wants to merge 1 commit into
sonic-net:masterfrom
bhouse-nexthop:bhouse.libyang3-debs

Conversation

@bhouse-nexthop
Copy link
Copy Markdown
Contributor

Why I did it

sonic-buildimage no longer builds the libyang1 Debian packages (libyang_1.0.73, libyang-cpp, python3-yang); it now builds only libyang3. The CI pipeline here installs libyang_1.0.73_*.deb from the downloaded sonic-buildimage build artifacts, which no longer exist, so the dependency install step would fail.

Part of sonic-net/sonic-buildimage#22385.

How I did it

Updated azure-pipelines.yml to install the libyang3 deb using a versionless glob instead of the removed libyang1 deb:

  • sudo dpkg -i libyang_1.0.73_*.debsudo dpkg -i libyang3_*.deb

No libyang-cpp, python3-yang, or libyang-dev references exist in this repository, so no other changes were needed.

How to verify it

Run the Azure pipeline (or the "Install Debian dependencies" step) against artifacts from a current sonic-buildimage build. The libyang3_*.deb glob resolves to the libyang3 package produced by sonic-buildimage and installs successfully, where the old libyang_1.0.73_*.deb glob would match nothing.

Description for the changelog

azure-pipelines: install libyang3 instead of libyang1

sonic-buildimage no longer builds the libyang1 debs (libyang_1.0.73).
It now builds only libyang3. Update the CI dependency install step to
install the versionless libyang3 deb glob instead of the removed
libyang1 deb.

Part of sonic-net/sonic-buildimage#22385.

Signed-off-by: Brad House <bhouse@nexthop.ai>
@mssonicbld
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@rookie-who
Copy link
Copy Markdown

/azp run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 386 in repo sonic-net/sonic-host-services

Copy link
Copy Markdown

@rookie-who rookie-who left a comment

Choose a reason for hiding this comment

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

LGTM — clean libyang1 → libyang3 replacement.

@vaibhavhd
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@vaibhavhd
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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