Dependency checking for Northbound API
(NEUTRON-158)
|
|
| Status: | Open |
| Project: | neutron |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Sub-task | Priority: | Medium |
| Reporter: | Michael Vorburger | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Comments |
| Comment by Michael Vorburger [ 02/Aug/18 ] |
|
as discussed today, based on dependency_validations.py : def _generate_sfc_port_pair_deps(data):
object_ids = []
ingress_port = data.get('ingress')
if ingress_port is not None:
object_ids.append(ingress_port)
egress_port = data.get('egress')
if egress_port is not None:
object_ids.append(egress_port)
return object_ids
thapar, mpeterson, jhershbe, mkolesni@redhat.com anything else to consider for this entity? |
| Comment by Michel Peterson [ 12/Aug/18 ] |
|
there seems to be a mismatch between the title and the code you put in the comment. The right one for this would be _generate_sfc_port_pair_group_deps of the same file. |
| Comment by Michael Vorburger [ 13/Aug/18 ] |
|
Oups, yeah - above is for NEUTRON-186 wherewas for this (port pair GROUP) it's just: def _generate_sfc_port_pair_group_deps(data):
return data['port_pairs']
|
| Comment by Michel Peterson [ 13/Aug/18 ] |
|
In general also deletion deps need to be considered |
| Comment by Michael Vorburger [ 13/Aug/18 ] |
|
FYI we agreed that we first want to finally make progress for create/update, and then do delete after - watch NEUTRON-195. |
| Comment by Michel Peterson [ 13/Aug/18 ] |
|
gotcha, missed that bit. Thanks |