Skip to content

fix: pin e2e coraza-waf to a pullable fixed image#264

Merged
ecv merged 1 commit into
mainfrom
fix/263-e2e-coraza-waf-pin
Jul 13, 2026
Merged

fix: pin e2e coraza-waf to a pullable fixed image#264
ecv merged 1 commit into
mainfrom
fix/263-e2e-coraza-waf-pin

Conversation

@ecv

@ecv ecv commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

What

Point the downstream-gateway e2e coraza-waf initContainer at a real, published,
fixed image:

- image: coraza-waf:v1.1.1-dev2
+ image: ghcr.io/datum-labs/coraza-envoy-go-filter/coraza-waf:v1.3.0-alpha10

Why

The old pin was a bare tag (coraza-waf:v1.1.1-dev2) — no registry, so CI
can't pull it — and predates the TPP Enforce fixes. The downstream Envoy
listener never programs:

status.(conditions[?type == 'Programmed'])[0].status: Invalid value: "Unknown": Expected value: "True"

So chainsaw/trafficprotectionpolicy-enforce-serves-traffic fails on main
and gates every open PR (the whole E2E Tests workflow is red).

v1.3.0-alpha10 carries the response-phase / listener fixes tracked in the
edge repo (https://github.com/datum-cloud/infra/issues/3319,
https://github.com/datum-cloud/infra/issues/3324,
https://github.com/datum-cloud/infra/issues/3329).

Verification

  • E2E Tests green
  • trafficprotectionpolicy-enforce-serves-traffic passes; downstream Gateway Programmed=True

Tracked by #263. Root bug #242.

The downstream gateway e2e config pinned the WAF filter as the bare tag
`coraza-waf:v1.1.1-dev2`, which has no registry and cannot be pulled in CI,
and predates the TPP Enforce listener-reject / response-phase fixes. The Envoy
listener never programs (Programmed=Unknown), so the
`trafficprotectionpolicy-enforce-serves-traffic` chainsaw test fails on main
and gates every PR.

Point the initContainer at the published, fixed image.

Key changes:
- Pin coraza-waf initContainer to
  ghcr.io/datum-labs/coraza-envoy-go-filter/coraza-waf:v1.3.0-alpha10
@ecv
ecv merged commit b7f5d29 into main Jul 13, 2026
10 of 11 checks passed
@ecv
ecv deleted the fix/263-e2e-coraza-waf-pin branch July 13, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants