Following config will break routing: {"type":"table", "table": "main"}

Bug #1867761 reported by Alvaro Uria
8
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-routing-config parameter now accepts:
{"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

Alvaro Uria (aluria)
Changed in charm-advanced-routing:
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
Alvaro Uria (aluria) wrote :

Marked high because the workaround is to avoid defining {"type": "table", "table": "main"}

Changed in charm-advanced-routing:
importance: Critical → High
Changed in charm-advanced-routing:
assignee: nobody → Peter Sabaini (peter-sabaini)
Changed in charm-advanced-routing:
status: Confirmed → In Progress
Alvaro Uria (aluria)
Changed in charm-advanced-routing:
status: In Progress → Fix Committed
milestone: none → 20.05
Revision history for this message
Andrea Ieri (aieri) wrote :

released as cs:~llama-charmers-next/advanced-routing-4

Changed in charm-advanced-routing:
assignee: Peter Sabaini (peter-sabaini) → Andrea Ieri (aieri)
status: Fix Committed → Fix Released
assignee: Andrea Ieri (aieri) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.