Skip to content

Install libyang to azure pipeline#11

Merged
rhalstea merged 1 commit intosonic-net:mainfrom
liuh-80:dev/liuh/install-libyang-pipeline
Sep 19, 2022
Merged

Install libyang to azure pipeline#11
rhalstea merged 1 commit intosonic-net:mainfrom
liuh-80:dev/liuh/install-libyang-pipeline

Conversation

@liuh-80
Copy link
Copy Markdown
Contributor

@liuh-80 liuh-80 commented Sep 15, 2022

Why I did it

sonic-swss-common lib will add dependency to libyang soon, so need install libyang lib to prevent build and UT break.

How I did it

Modify azure pipeline to install libyang in azure pipeline steps.

How to verify it

Pass all UT.

Which release branch to backport (provide reason below if selected)

Description for the changelog

Modify azure pipeline to install libyang in azure pipeline steps.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@liuh-80 liuh-80 marked this pull request as ready for review September 15, 2022 05:48
@liuh-80 liuh-80 requested a review from qiluo-msft September 15, 2022 05:48
@liuh-80 liuh-80 requested review from bocon13 and rhalstea September 19, 2022 01:44
@liuh-80
Copy link
Copy Markdown
Contributor Author

liuh-80 commented Sep 19, 2022

@bocon13, @rhalstea, sonic-swss-common will add dependency to libyang in this PR: sonic-net/sonic-swss-common#682

Which will break the azure pipeline of this repo, because pipeline need install libyang before install sonic-swss-common deb package, otherwise there will be a dependency error.

However current PR can't merge because following error: "The base branch restricts merging to authorized users."

So, could you please help review and merge this PR?

@rhalstea rhalstea merged commit add1cea into sonic-net:main Sep 19, 2022
@liuh-80 liuh-80 deleted the dev/liuh/install-libyang-pipeline branch September 20, 2022 02:34
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.

3 participants