Fix configure_haproxy() complexity as it's currently failing lint tests
Bug #1825084 reported by
Haw Loeung
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Content Cache Charm |
Triaged
|
Wishlist
|
Unassigned | ||
Bug Description
In my haste to bring up a new service, configure_haproxy() was updated implementing the features required. Unfortunately, this now fails the lint test with the following:
| ./reactive/
This bug is to fix that and remove the NOQA.
Related branches
~hloeung/content-cache-charm:master
- Content Cache Charmers: Pending requested
-
Diff: 51 lines (+6/-5)2 files modifiedlib/haproxy.py (+5/-1)
reactive/content_cache.py (+1/-4)
Revision history for this message
| Haw Loeung (hloeung) wrote | #1 |
| Changed in content-cache-charm: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| assignee: | nobody → Haw Loeung (hloeung) |
| summary: |
- Fix configure_haproxy() as it's currently failing lint tests + Fix configure_haproxy() complexity as it's currently failing lint tests |
| Changed in content-cache-charm: | |
| importance: | Medium → Low |
| Changed in content-cache-charm: | |
| importance: | Low → Wishlist |
| Changed in content-cache-charm: | |
| assignee: | Haw Loeung (hloeung) → nobody |
To post a comment you must log in.
Have assigned it to me, and will mark "In Progress" once work has started. Unless someone else wants to get their feet wet and pick this up.