upstream: fix the header file path typo (envoyproxy#8266)

Fix the header path error introudced in pr envoyproxy#8166 Risk Level: LOW Testing: N/A Signed-off-by: Xin Zhuang <[email protected]>
danzh2010 · Sep 24, 2019 · 4bf49e5 · 4bf49e5
1 parent 73020c0
commit 4bf49e5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/source/common/upstream/health_checker_impl.h b/source/common/upstream/health_checker_impl.h
@@ -1,6 +1,7 @@
 #pragma once
 
 #include "envoy/access_log/access_log.h"
+#include "envoy/api/api.h"
 #include "envoy/api/v2/core/health_check.pb.h"
 #include "envoy/grpc/status.h"
 
@@ -11,7 +12,6 @@
 #include "common/stream_info/stream_info_impl.h"
 #include "common/upstream/health_checker_base_impl.h"
 
-#include "include/envoy/api/_virtual_includes/api_interface/envoy/api/api.h"
 #include "src/proto/grpc/health/v1/health.pb.h"
 
 namespace Envoy {

diff --git a/tools/check_format.py b/tools/check_format.py
@@ -500,6 +500,9 @@ def checkSourceLine(line, file_path, reportError):
     if invalid_construct in line:
       reportError("term %s should be replaced with standard library term %s" %
                   (invalid_construct, valid_construct))
+  # Do not include the virtual_includes headers.
+  if re.search("#include.*/_virtual_includes/", line):
+    reportError("Don't include the virtual includes headers.")
 
   # Some errors cannot be fixed automatically, and actionable, consistent,
   # navigable messages should be emitted to make it easy to find and fix