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

Incorrect status reported in tlspolicy when issuerRef points to an issuer that doesn't exist #581

Closed
laurafitzgerald opened this issue Sep 20, 2023 · 0 comments · Fixed by #594
Assignees

Comments

@laurafitzgerald
Copy link
Contributor

laurafitzgerald commented Sep 20, 2023

Steps to reproduce non issuer
Follow https://docs.kuadrant.io/getting-started/
Follow https://docs.kuadrant.io/multicluster-gateway-controller/docs/how-to/multicluster-gateways-walkthrough/ until the creation of the TLSPolicy
Apply a tlspolicy that points to an issuer that doesn't exist

spec:
  issuerRef:
    group: cert-manager.io
    kind: Issuer
    name: something-else

Observe status

 status:
    conditions:
    - lastTransitionTime: "2023-09-20T09:04:40Z"
      message: Gateway is TLS Enabled
      reason: GatewayTLSEnabled
      status: "True"
      type: Ready
    observedGeneration: 2
@laurafitzgerald laurafitzgerald mentioned this issue Sep 20, 2023
18 tasks
@laurafitzgerald laurafitzgerald changed the title No status reported in tlspolicy when the targetRef points to a gateway that doesn't exist Incorrect status reported in tlspolicy when the targetRef points to a gateway that doesn't exist or issuer that doesn't exist Sep 20, 2023
@laurafitzgerald laurafitzgerald self-assigned this Sep 21, 2023
@mikenairn mikenairn changed the title Incorrect status reported in tlspolicy when the targetRef points to a gateway that doesn't exist or issuer that doesn't exist Incorrect status reported in tlspolicy when issuerRef points to an issuer that doesn't exist Sep 21, 2023
github-merge-queue bot pushed a commit that referenced this issue Sep 22, 2023
validate the presence of specified issuer or clusterisssuer in tlspolicy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
1 participant