Skip to content

Use PTF version with bf_pktpy as default.#5418

Open
fruffy wants to merge 2 commits intomainfrom
fruffy/ptf_default_pkt_module
Open

Use PTF version with bf_pktpy as default.#5418
fruffy wants to merge 2 commits intomainfrom
fruffy/ptf_default_pkt_module

Conversation

@fruffy
Copy link
Collaborator

@fruffy fruffy commented Nov 8, 2025

Let's see what breaks...

@fruffy fruffy added the infrastructure Topics related to code style and build and test infrastructure. label Nov 8, 2025
@fruffy fruffy force-pushed the fruffy/ptf_default_pkt_module branch 2 times, most recently from f46a051 to 838cb83 Compare November 8, 2025 19:36
@fruffy fruffy added p4tc Topics related to the P4-TC back end. On PRs, also triggers p4tc CI tests to run. run-sanitizer Use this tag to run a Clang+Sanitzers CI run. run-static Use this tag to trigger static build CI run. breaking-change This change may break assumptions of compiler back ends. labels Nov 8, 2025
@fruffy fruffy force-pushed the fruffy/ptf_default_pkt_module branch from 8f653f9 to f6bcd3e Compare November 8, 2025 21:30
@fruffy
Copy link
Collaborator Author

fruffy commented Nov 8, 2025

@ChrisDodd Looks like the sanitizer is failing because of some unrelated changes. Might be caused by the recent JSON changes.

https://github.com/p4lang/p4c/actions/runs/19197872997/job/54881476767

@fruffy fruffy force-pushed the fruffy/ptf_default_pkt_module branch from f6bcd3e to c459cfc Compare November 8, 2025 22:13
@fruffy fruffy mentioned this pull request Nov 9, 2025
Open
@fruffy fruffy force-pushed the fruffy/ptf_default_pkt_module branch from c459cfc to 7a44092 Compare November 23, 2025 23:32
@fruffy fruffy marked this pull request as ready for review February 15, 2026 20:49
@fruffy
Use PTF version with bf_pktpy as default.
20d6ae6 
Signed-off-by: fruffy <fruffy@nyu.edu>
@fruffy
README.
50b6729 
Signed-off-by: fruffy <fruffy@nyu.edu>
@fruffy fruffy force-pushed the fruffy/ptf_default_pkt_module branch from 7a44092 to 50b6729 Compare February 15, 2026 20:49
@fruffy fruffy requested a review from jafingerhut February 15, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jafingerhut jafingerhut Awaiting requested review from jafingerhut

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

breaking-change This change may break assumptions of compiler back ends. infrastructure Topics related to code style and build and test infrastructure. p4tc Topics related to the P4-TC back end. On PRs, also triggers p4tc CI tests to run. run-sanitizer Use this tag to run a Clang+Sanitzers CI run. run-static Use this tag to trigger static build CI run.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@fruffy