FDio / govpp

Go toolset for the VPP.
Apache License 2.0
192 stars 81 forks source link

Improve compatibility checking #146

Open ondrej-fabry opened 1 year ago

ondrej-fabry commented 1 year ago

The current implementation of the compatibility checking does not make distinction between a missing message and message with different CRC.

This has recently caused issue in the last VPP release 23.06 where new messages where added to interface API which caused the compatibility check in examples fail. As a quick workaround the examples now only log warning message on failing this check for interface API.

The proper solution for this is to fail only if the messages that are being checked actually exist in the connected VPP and are truly incompatible - have different CRC.