Skip to content

Conversation

@pedro-avalos
Copy link
Collaborator

@pedro-avalos pedro-avalos commented Jan 5, 2026

Description

  • This PR replaces NGINX with Traefik as the ingress for the server charm.
  • NGINX is deprecated and most of our projects are turning to traefik as the ingress of choice.

Related Issue(s)

Resolves HWAPI-61

Testing

  • Integration tests

  • Manual testing steps:

    1. Set up Juju environment
    2. Pack charm
    3. uvx tox run -e integration

Checklist

  • I have followed the contribution guidelines.
  • I have signed the Canonical CLA.
  • I have added necessary tests.
  • I have added or updated any relevant documentation (if needed).
  • I have tested the changes.

Additional Notes

This PR adds traefik-k8s and lego as components to be deployed in the product Terraform modules. These are to ensure that we can pull in a TLS certificate (NGINX pulled it from a K8s secret, but we are moving towards using LEGO credentials).

@codecov
Copy link

codecov bot commented Jan 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.30%. Comparing base (77d1117) to head (81c8356).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #491      +/-   ##
==========================================
+ Coverage   89.20%   89.30%   +0.09%     
==========================================
  Files          28       28              
  Lines        1390     1403      +13     
  Branches        1        4       +3     
==========================================
+ Hits         1240     1253      +13     
  Misses        150      150              
Flag Coverage Δ *Carryforward flag
charm 100.00% <ø> (ø) Carriedforward from 7c3f8be
client 80.24% <ø> (ø) Carriedforward from 7c3f8be
server 97.01% <ø> (-0.03%) ⬇️ Carriedforward from 7c3f8be

*This pull request uses carry forward flags. Click here to find out more.

Components Coverage Δ
Server 96.85% <ø> (ø)
Charm 100.00% <100.00%> (ø)
Client 80.24% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pedro-avalos pedro-avalos force-pushed the replace-ingress branch 10 times, most recently from 3f46b2f to d7a0cef Compare January 9, 2026 23:09
@pedro-avalos pedro-avalos force-pushed the replace-ingress branch 4 times, most recently from d396194 to be5eb2b Compare January 14, 2026 16:06
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