Skip to content

Commit

Permalink
upstream: fix the header file path typo (envoyproxy#8266)
Browse files Browse the repository at this point in the history
Fix the header path error introudced in pr envoyproxy#8166

Risk Level: LOW
Testing: N/A

Signed-off-by: Xin Zhuang <[email protected]>
  • Loading branch information
stevenzzzz committed Sep 17, 2019
1 parent 63f622f commit 8049e24
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion source/common/upstream/health_checker_impl.h
Original file line number Diff line number Diff line change
@@ -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"

Expand All @@ -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 {
Expand Down
3 changes: 3 additions & 0 deletions tools/check_format.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 8049e24

Please sign in to comment.