Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] sonic-config-engine now depends on SONiC YANG packages #1675

Merged
merged 1 commit into from
Jun 16, 2021
Merged

[CI] sonic-config-engine now depends on SONiC YANG packages #1675

merged 1 commit into from
Jun 16, 2021

Conversation

jleveque
Copy link
Contributor

@jleveque jleveque commented Jun 15, 2021

sonic-config-engine now depends on sonic-yang-mgmt and sonic-yang-models, so we now need to install them before we install sonic-config-engine as part of continuous integration to get the CI working again

@jleveque jleveque added the CI label Jun 15, 2021
@jleveque jleveque self-assigned this Jun 15, 2021
@jleveque jleveque added the Bug label Jun 15, 2021
@jleveque
Copy link
Contributor Author

This PR fixes the CI, but now the failures appear to be real issues.

@jleveque jleveque requested a review from lguohan June 15, 2021 02:59
@lguohan lguohan merged commit 25669c3 into sonic-net:master Jun 16, 2021
@lguohan
Copy link
Contributor

lguohan commented Jun 16, 2021

test not related to the pr, unblock.

@jleveque jleveque deleted the fix_ci branch June 16, 2021 17:13
xumia pushed a commit to xumia/sonic-utilities that referenced this pull request Jun 28, 2021
…t#1675)

sonic-config-engine now depends on sonic-yang-mgmt and sonic-yang-models, so we now need to install them before we install sonic-config-engine as part of continuous integration to get the CI working again
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-utilities that referenced this pull request Aug 10, 2021
…t#1675)

sonic-config-engine now depends on sonic-yang-mgmt and sonic-yang-models, so we now need to install them before we install sonic-config-engine as part of continuous integration to get the CI working again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants