search

p4lang / p4runtime

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

Update Index message in p4runtime.proto to support bytes #416

Open antoninbas opened 1 year ago

antoninbas commented 1 year ago

But I feel like we have opened a can of worms: we also need to address the case of counters / meters, for which the index field should probably be changed to bytes as well. It makes the API less convenient to use IMO, but it makes sense in a way since PSA typically uses bit<W> values to index counters in the datapath.

Originally posted by @antoninbas in https://github.com/p4lang/p4runtime/issues/329#issuecomment-761174829

smolkaj commented 1 year ago

Thanks for opening this for tracking. Agreed on all points.