fix(*) make check -C api

[nickolaev]

Signed-off-by: Nikolay Nikolaev nikolay.nikolaev@konghq.com

Summary

Fixes check api test in master.

Documentation

• Link to the website documentation PR

Testing

• Unit tests
• E2E tests
• Manual testing on Universal
• Manual testing on Kubernetes

[jpeach]

@nickolaev since you are touching the protobufs, could you also apply this change:

diff --git api/mesh/v1alpha1/dataplane.proto api/mesh/v1alpha1/dataplane.proto
index e19af659..c7577961 100644
--- api/mesh/v1alpha1/dataplane.proto
+++ api/mesh/v1alpha1/dataplane.proto
@@ -114,7 +114,7 @@ message Dataplane {
 
       // DEPRECATED: use networking.outbound[].tags
       // Service name.
-      string service = 2 [ (validate.rules).string.hostname = true ];
+      string service = 2 [ (validate.rules).string.hostname = true, deprecated = true ];
 
       // Tags
       map<string, string> tags = 5;

🙏

[jakubdyszkiewicz]

can we add this as a part of a commit workflow? It's not the first time we are learning about linter mistake after merge to master. Can be separate PR