search

p4lang / p4runtime

Specification documents for the P4Runtime control-plane API
Apache License 2.0
140 stars 86 forks source link

Fix #434: Remove obsolete TODO section in README #447

Closed jafingerhut closed 10 months ago

jafingerhut commented 10 months ago

Update the link to the auto-generated versions of the P4Runtime specification on the P4.org web site.

Update the section "P4 Language Version Applicability" to version 1.2.4 of the P4_16 language specification, but list 3 known exceptions of features that have not been explicitly addressed yet.

jafingerhut commented 10 months ago

Intended to resolve issue https://github.com/p4lang/p4runtime/issues/434

Also to explicitly document that we are NOT yet resolving the issues mentioned here: https://github.com/p4lang/p4runtime/issues/424

jafingerhut commented 10 months ago

Note to everyone who reviewed and approved commit 1 of this PR: I have just added commit 2, which adds 7 more items to the bullet list in the language version compatibility section for features new in P4_16 language spec version 1.2.4 that might lead to future changes to the P4Runtime API specification to address compatibility. There are already open issues in the https://github.com/p4lang/p4runtime repo to remind us of each of them, and I would recommend that we try to remember to remove these bullet items from the text in this language compatibility section as we believe these compatibility issues have been resolved.

chrispsommers commented 10 months ago

Thanks @jafingerhut for the follow-up content!