-
Notifications
You must be signed in to change notification settings - Fork 16
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
Downstream changes from stratum project #391
Conversation
ffoulkes
commented
Jan 13, 2024
•
edited
Loading
edited
- Modify for updated version of stratum, which integrates changes from the upstream repository.
f64fc64
to
a1fa927
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Disable gnmi-ctl and sgnmi_cli builds so we can build the rest of P4 Control Plane while the Stratum code is being updated. - Update stratum submodule pointer to reference the topic branch. Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
1af2834
to
2c87763
Compare
- Incorporate changes downstreamed from stratum project. Signed-off-by: Derek G Foster <[email protected]>
Verified that the Since Derek has already tested the build against the other targets (DPDK and Tofino), proceeding with the merge into main branch. |