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

[submodule] Advance sonic-p4rt/sonic-pins pointer #10566

Merged
merged 2 commits into from
May 6, 2022

Conversation

rhalstea
Copy link
Contributor

Why I did it

To pick up new commits:

  • 60d2467 Add depends to p4rt debian package

How I did it

update sonic-p4rt/sonic-pins submodule pointer

How to verify it

should be able to build with p4rt enabled.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@rhalstea rhalstea requested a review from lguohan as a code owner April 13, 2022 14:54
@rhalstea
Copy link
Contributor Author

For info: @bocon13 @donNewtonAlpha

bocon13
bocon13 previously approved these changes Apr 21, 2022
@rhalstea
Copy link
Contributor Author

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 10566 in repo Azure/sonic-buildimage

@rhalstea
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@rhalstea
Copy link
Contributor Author

It looks like a test is failing to establish a SSH connection?

generic_config_updater/test_dhcp_relay.py::test_dhcp_relay_tc5_rm[config_add_default] ERROR [ 81%]
...
<10.250.0.101> ESTABLISH SSH CONNECTION FOR USER: admin

Which I'm not sure is related to this change. Can someone help rerun the buildimage tests? I don't seem to have the right permissions.

@rhalstea
Copy link
Contributor Author

Looks like I forgot to document this in the original comment, but this PR is the followup mentioned in #10182, and is need for P4RT to run in bullseye.

donNewtonIntel
donNewtonIntel previously approved these changes Apr 21, 2022
@rhalstea rhalstea dismissed stale reviews from donNewtonIntel and bocon13 via 9581adb May 2, 2022 18:33
@rhalstea rhalstea force-pushed the sonic_p4rt_bullseye branch 2 times, most recently from 9581adb to b0f5f79 Compare May 4, 2022 14:45
    60d2467 Add depends to p4rt debian package
bocon13
bocon13 previously approved these changes May 5, 2022
@rhalstea
Copy link
Contributor Author

rhalstea commented May 5, 2022

@lguohan would you mind reviewing?

@qiluo-msft qiluo-msft merged commit 7e190c7 into sonic-net:master May 6, 2022
liushilongbuaa pushed a commit to liushilongbuaa/sonic-buildimage that referenced this pull request Jun 20, 2022
#### Why I did it

To pick up new commits:
*  60d2467 Add depends to p4rt debian package

#### How I did it

update sonic-p4rt/sonic-pins submodule pointer

#### How to verify it

should be able to build with p4rt enabled.
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.

None yet

5 participants