Port Chain Update Replaces the Existing Flow Classifiers
Bug #1533615 reported by
Prithiv
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
networking-sfc |
In Progress
|
Low
|
RaoFei |
Bug Description
The port-chain-update PC --flow-classifier actually replaces the new flow classifier with the existing one unless specified both existing and the new flow classifier in the port chain update. There is no way to add another flow classifier by providing what classifier i have to insert in the existing chain. Should the port-chain-update just add the new flow classifier to the existing chain rather than the requirement to provide both classifiers in update
description: | updated |
Changed in networking-sfc: | |
status: | New → Opinion |
Changed in networking-sfc: | |
status: | Opinion → Invalid |
status: | Invalid → Confirmed |
Changed in networking-sfc: | |
status: | Confirmed → In Progress |
assignee: | nobody → RaoFei (milo-frao) |
importance: | Undecided → Low |
Changed in networking-sfc: | |
importance: | Low → Wishlist |
importance: | Wishlist → Low |
summary: |
- Port Chain Update Replaces the Existing Flow Classifier + Port Chain Update Replaces the Existing Flow Classifiers |
To post a comment you must log in.
If we check the other object update operation implementation in Neutron, we can find almost all the object's list update will use the new replace the old.
eg,
1. create object['attr'] = [a,b], then object['attr'] = [a,b]
2. update object['attr'] = [c], then object['attr'] = [c]
3. update object['attr'] = [a,b,c], then object['attr'] = [a,b,c]
There are many object implement as this behavior above.
eg,
port object's security_groups list,
subnet object's allocation_pools, dns_nameservers, and host_routes list.
router object's external_fixed_ips, routers list
...
So, it's acceptable to replaces the new flow classifier with the existing one unless specified both existing and the new flow classifier in the port chain update.
We can discuss more about this.