Hi all,

I’ve been consolidating the policer to use the common ip definitions for DSCP 
in its marking parameters rather than having its own duplicates. As part of 
that I tried a minor change to the .api:

diff --git a/src/vnet/policer/policer_types.api b/src/vnet/policer/policer_types
.api
index d48dc646e..2377790bf 100644
--- a/src/vnet/policer/policer_types.api
+++ b/src/vnet/policer/policer_types.api
@@ -14,6 +14,8 @@
  */
 option version = "1.0.0";
 
+import "vnet/ip/ip_types.api";
+
 enum sse2_qos_rate_type : u8
 {
   SSE2_QOS_RATE_API_KBPS = 0,
@@ -53,7 +55,7 @@ enum sse2_qos_action_type : u8
 typedef sse2_qos_action
 {
   vl_api_sse2_qos_action_type_t type;
-  u8 dscp;
+  vl_api_ip_dscp_t dscp;
 };


The dscp api type is

enum ip_dscp : u8 {
  IP_API_DSCP_CS0 =  0,
  IP_API_DSCP_CS1 = 8,
  IP_API_DSCP_AF11 = 10,
etc


so it ought to be ok but ...

modified: ** policer_add_del
modified: ** policer_details
crcchecker: Changing production APIs in an incompatible way
make: *** [Makefile:643: checkstyle-api] Error 255

Is this a bug in the checkstyle-api script?

Thanks,

Brian
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#18698): https://lists.fd.io/g/vpp-dev/message/18698
Mute This Topic: https://lists.fd.io/mt/80483218/21656
Group Owner: vpp-dev+ow...@lists.fd.io
Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to