Following config will break routing: {"type":"table", "table": "main"}
Bug #1867761 reported by
Alvaro Uria
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
charm-advanced-routing |
Fix Released
|
High
|
Unassigned |
Bug Description
{"type": "rule"} has an optional parameter "table". The value could be "main" as a fallback for a more generic rule (ie. from net1 to net1) via a specific table (ie. SF2). If {"type": "table"} does not exist for the selected table in the rule definition, the charm will raise an exception stating that the table is missing.
advanced-
{"type": "table", "table": "main"}
However, the "main" table is the default for rules that don't specify a table, and the charm will flush and delete the "main" table if the above {"type": "table"} definition (for the main table) exists.
Validation is needed to avoid:
{"type": "table", "table": "main"}
Related branches
~peter-sabaini/charm-advanced-routing:bug/1867761
- Alvaro Uria (community): Needs Fixing
-
Diff: 179 lines (+51/-27)6 files modifiedMakefile (+9/-3)
lib/routing_entry.py (+11/-19)
tests/functional/cfg_opts.py (+26/-1)
tests/functional/requirements.txt (+1/-1)
tests/functional/test_routing.py (+3/-2)
tox.ini (+1/-1)
Changed in charm-advanced-routing: | |
status: | New → Confirmed |
importance: | Undecided → Critical |
Changed in charm-advanced-routing: | |
assignee: | nobody → Peter Sabaini (peter-sabaini) |
Changed in charm-advanced-routing: | |
status: | Confirmed → In Progress |
Changed in charm-advanced-routing: | |
status: | In Progress → Fix Committed |
milestone: | none → 20.05 |
To post a comment you must log in.
Marked high because the workaround is to avoid defining {"type": "table", "table": "main"}