Skip to content

Commit

Permalink
Adjusting the list of ignored warnings.
Browse files Browse the repository at this point in the history
Removing all those related to the v2 bootstraps that are no longer
present.

Signed-off-by: Jakub Sobon <[email protected]>
  • Loading branch information
mum4k committed Dec 3, 2020
1 parent 2512c33 commit d79df50
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions test/integration/nighthawk_test_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,19 +45,16 @@ def _substitute_yaml_values(runfiles_instance, obj, params):
#
# Other messages logged at either of these severities fail the test.
_TEST_SERVER_WARN_ERROR_IGNORE_LIST = frozenset([
# TODO(#580) Remove these once our use of Envoy's API is updated to v3.
"Configuration does not parse cleanly as v3",
"Deprecated field: type envoy.api.v2.Cluster",
"Deprecated field: type envoy.api.v2.listener.Filter",
"Deprecated field: type envoy.config.filter.network.http_connection_manager.v2.HttpFilter",

# TODO(#582): Identify these and file issues or add explanation as necessary.
"Unable to use runtime singleton for feature envoy.http.headermap.lazy_map_min_size",
"Using deprecated extension name 'envoy.http_connection_manager' for 'envoy.filters.network.http_connection_manager'.",
"Using deprecated extension name 'envoy.listener.tls_inspector' for 'envoy.filters.listener.tls_inspector'.",
"Using deprecated extension name 'envoy.router' for 'envoy.filters.http.router'.",
"there is no configured limit to the number of allowed active connections. Set a limit via the runtime key overload.global_downstream_max_connections",

# A few of our filters use the same typed configuration, specifically
# 'test-server', 'time-tracking' and 'dynamic-delay'.
# For now this is by design.
"Double registration for type: 'nighthawk.server.ResponseOptions'",

# Logged for normal termination, not really a warning.
"caught SIGTERM",
])
Expand Down

0 comments on commit d79df50

Please sign in to comment.