Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(#3097): propagate translation failures for gateway listeners referred secrets #3147

Merged
merged 11 commits into from
Nov 9, 2022

Conversation

czeslavo
Copy link
Contributor

@czeslavo czeslavo commented Nov 9, 2022

What this PR does / why we need it:

It adds propagation of translation failures for Gateway.Listeners that:

  • specify more than one certReference,
  • refer to a secret that has no valid TLS key-pair embedded

Which issue this PR fixes:

Part of #3097.

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@czeslavo czeslavo added this to the KIC v2.8.0 milestone Nov 9, 2022
@czeslavo czeslavo self-assigned this Nov 9, 2022
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 10:08 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 10:38 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 11:06 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 11:30 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 11:30 Inactive
@pull-request-size pull-request-size bot added size/L and removed size/M labels Nov 9, 2022
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 15:57 Inactive
@czeslavo czeslavo changed the title feat(#3097): propagate translation failures for gateway listeners with multiple cert refs feat(#3097): propagate translation failures for gateway listeners referred secrets Nov 9, 2022
@czeslavo czeslavo marked this pull request as ready for review November 9, 2022 16:03
@czeslavo czeslavo requested a review from a team as a code owner November 9, 2022 16:03
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 16:22 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 16:22 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 17:14 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 17:14 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 17:22 Inactive
rainest and others added 2 commits November 9, 2022 18:58
Consolidate E2E environment builders into helper.

Add GKE support for E2E tests.

Add GKE E2E workflow and make target.

Refactor image loads to always succeed during builder construction.

Change final release test from integration to E2E.

Co-authored-by: Grzegorz Burzyński <czeslavo@gmail.com>
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:04 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:04 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:31 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:31 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:53 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:53 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 18:58 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 20:35 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 20:54 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 21:04 Inactive
@czeslavo czeslavo requested a review from rainest November 9, 2022 21:12
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 21:28 Inactive
@czeslavo czeslavo temporarily deployed to Configure ci November 9, 2022 21:28 Inactive
@czeslavo czeslavo merged commit 64bc2f0 into main Nov 9, 2022
@czeslavo czeslavo deleted the feat/multiple-cert-refs-gateway-listener branch November 9, 2022 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants