Resolve "AddLink crashes goSDN when Link without correct port config is send."
requested to merge 345-addlink-crashes-gosdn-when-link-without-correct-port-config-is-send into master
This MR extends Links within their .proto
definition. Additionally the topology store has been slightly adjusted so that the integration-tests for topology are runnable again. .proto
files are generated with protoc-gen-go v1.32.0
.
Description
Links have now an additional required field: Configuration
.
The topology store is now working with the binary representation of a uuid as identifier, since we do not work with a LoadedLink
struct as we normally do (e.g., LoadedUser
,LoadedRole
, ...). This is something that should be addressed but since there is also an open discussion about the future implementation of the topology (#346), this should be addressed after a decision has been made there.
Related Issue
Related to #345
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
-
Bug fix (non-breaking change which fixes an issue) -
New feature (non-breaking change which adds functionality) -
Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
-
My code follows the code style of this project. -
My change requires a change to the documentation. -
I have updated the documentation accordingly. -
I have read the CONTRIBUTING document. -
I have added tests to cover my changes. -
All new and existing tests passed.
Edited by Malte Bauch