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

Revert "bump cert_handler to v1 (#94)" #106

Merged
merged 3 commits into from
May 2, 2024
Merged

Conversation

sed-i
Copy link
Contributor

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

Issue

Grafana agent must also work with juju 2.9, but certhandler v1 uses secrets.

Solution

Switch back to certhandler v0.

This reverts commit 0cfca69.

Fixes #105.

In tandem with:

Testing

graph LR

subgraph lxd/juju-2.9.49
ubuntu --- grafana-agent
end

subgraph microk8s/juju-3.4.2
prometheus --- self-signed-certificates
traefik --- self-signed-certificates
prometheus --- traefik
end

grafana-agent --- self-signed-certificates
grafana-agent --- prometheus
Loading

"cos" model, Juju 3.4:

bundle: kubernetes
saas:
applications:
  prom:
    charm: prometheus-k8s
    channel: latest/edge
    revision: 182
    base: [email protected]/stable
    scale: 1
    trust: true
  ssc:
    charm: self-signed-certificates
    channel: latest/edge
    revision: 137
    scale: 1
  trfk:
    charm: traefik-k8s
    channel: latest/stable
    revision: 176
    base: [email protected]/stable
    scale: 1
    trust: true
relations:
- - prom:certificates
  - ssc:certificates
- - trfk:ingress-per-unit
  - prom:ingress
- - trfk:certificates
  - ssc:certificates
--- # overlay.yaml
applications:
  prom:
    offers:
      prom:
        endpoints:
        - receive-remote-write
        acl:
          admin: admin
  ssc:
    offers:
      certs:
        endpoints:
        - certificates
        - send-ca-cert
        acl:
          admin: admi

LXD model, Juju 2.9.49:

series: jammy
saas:
  certs:
    url: j34:admin/rwdrop.certs
  prom:
    url: j34:admin/rwdrop.prom
applications:
  ga:
    charm: ./grafana-agent_ubuntu-22.04-amd64.charm
  ub:
    # In cli I had to specify --series jammy
    charm: ubuntu
    channel: stable
    revision: 24
    num_units: 1
relations:
- - ga:juju-info
  - ub:juju-info
- - prom:receive-remote-write
  - ga:send-remote-write
- - ga:certificates
  - certs:certificates
- - ga:receive-ca-cert
  - certs:send-ca-cer

@sed-i sed-i merged commit e1e750f into main May 2, 2024
13 checks passed
@sed-i sed-i deleted the feature/revert-certhandler branch May 2, 2024 17:37
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.

grafana-agent no longer works with juju 2.9
1 participant