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

Add test for removing cerftificate relations #108

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented May 3, 2024

Issue

canonical/grafana-k8s-operator#328

Solution

Add itest for removing certificate relations.

Context

Ingress URL vs k8s fqdn vs loadbalancer IP -- which do we put in the cert?

Testing Instructions

Run integration test.
Inspect traefik's cert with e.g.

echo | openssl s_client -showcerts -connect 10.43.8.188:443 | openssl x509 -text -noout

@Abuelodelanada
Copy link
Contributor

@sed-i

Please take a look at this PR: canonical/traefik-k8s-operator#354

I managed to run full cos-lite itests several times in a row without failing.

I've packed Traefik charm with that patch and used it in bundle.yaml file in cos-lite-bundle:

  traefik:
    charm: ./traefik-k8s_ubuntu-20.04-amd64.charm
    base: [email protected]/stable
    scale: 1
    resources:
      traefik-image: docker.io/ubuntu/traefik:2-22.04
    constraints: arch=amd64
    storage:
      configurations: kubernetes,1,1024M
    trust: true

With that every time I run itest @ localhost pass.

@simskij
Copy link
Member

simskij commented Nov 29, 2024

@Abuelodelanada @sed-i is this still relevant?

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.

3 participants