p4lang / p4runtime

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

Replace references to 'master/slave' in the spec doc #320

Closed bocon13 closed 4 years ago

bocon13 commented 4 years ago

Using the following mapping:

TODO

antoninbas commented 4 years ago

As an additional TODO, some figures in the spec need to be updated

bocon13 commented 4 years ago

@antoninbas I made the one fix for the typo that you caught and rebased against latest master.

Unfortunately, I don't think that I have write access to the repo which is why I was using my personal branch. ERROR: Permission to p4lang/p4runtime.git denied to bocon13. If you grant write access to me, then I can stage to p4lang in the future.

Otherwise, I think this PR is ready to go from my side. Thanks!

antoninbas commented 4 years ago

@bocon13 My bad I thought you already had write access, I sent you an invite

antoninbas commented 4 years ago

Merging this PR as it has been open for a while, and there was no objection at the 10/23/2020 WG meeting.

bocon13 commented 4 years ago

Great, thanks @antoninbas! Looks like write access is working, too.