Skip to content

Commit

Permalink
Merge pull request #5107 from emissary-ingress/rlane/fix-disable-flag
Browse files Browse the repository at this point in the history
Fix go filter disable flag
  • Loading branch information
rick-a-lane-ii authored Jun 16, 2023
2 parents d4ce1e5 + cc2d8c0 commit f7ae900
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions python/ambassador/ir/irgofilter.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
from .ir import IR # pragma: no cover

GO_FILTER_LIBRARY_PATH = "/ambassador/filter.so"
AMBASSADOR_DISABLE_GO_FILTER = os.getenv("AMBASSADOR_DISABLE_GO_FILTER", False)
AMBASSADOR_DISABLE_GO_FILTER = os.getenv("AMBASSADOR_DISABLE_GO_FILTER", "false")


def go_library_exists(go_library_path: str) -> bool:
Expand Down Expand Up @@ -62,7 +62,7 @@ def __init__(
# We want to enable this filter only in Edge Stack
def setup(self, ir: "IR", _: Config) -> bool:
if ir.edge_stack_allowed:
if AMBASSADOR_DISABLE_GO_FILTER:
if AMBASSADOR_DISABLE_GO_FILTER.lower() in ("true", "yes", "1"):
self.logger.info(
"AMBASSADOR_DISABLE_GO_FILTER=%s, disabling Go filter...",
AMBASSADOR_DISABLE_GO_FILTER,
Expand Down

0 comments on commit f7ae900

Please sign in to comment.