goSDN merge requestshttps://code.fbi.h-da.de/danet/gosdn/-/merge_requests2024-03-15T10:45:27Zhttps://code.fbi.h-da.de/danet/gosdn/-/merge_requests/794Resolve "Change the way user credentials for authz are provided to not be in ...2024-03-15T10:45:27ZFabian SeidlResolve "Change the way user credentials for authz are provided to not be in context"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Should work now, tested manually like this:
![Bildschirmfoto_2024-03-15_um_08.20.04](/uploads/8299feefb3437...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Should work now, tested manually like this:
![Bildschirmfoto_2024-03-15_um_08.20.04](/uploads/8299feefb3437ad42185a7000d1342cd/Bildschirmfoto_2024-03-15_um_08.20.04.png)
Related to #357https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/567Add test for role api inputs2023-10-26T06:11:31ZAndré SterbaAdd test for role api inputs## Description
This is an attempt to make the api more testable by exposing the earlier added validation errors as proper types instead of plain strings.
## Related Issue
#232
## Motivation and Context
We should be able to provide p...## Description
This is an attempt to make the api more testable by exposing the earlier added validation errors as proper types instead of plain strings.
## Related Issue
#232
## Motivation and Context
We should be able to provide proper error messages, when an user input is not as intended and fails validation.
This was already in place with the implementation of #232, but only as a string not as typed outputs.
Therefore the usability in tests and by application was rather non existend.
## Types of changes
- [ ] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
## Checklist:
<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
- [ ] 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.https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/469Resolve "Clean up duplicated, not clear functionality in NetworkElement NBI s...2023-10-20T14:57:43ZFabian SeidlResolve "Clean up duplicated, not clear functionality in NetworkElement NBI server"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
#277<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
#277https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/572Add validation tests for user entity2023-10-18T10:10:26ZAndré SterbaAdd validation tests for user entityAdding more tests for https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/567Adding more tests for https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/567https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/543Refactor gRPC error handling in northbound interface2023-09-25T11:49:11ZAndre SterbaRefactor gRPC error handling in northbound interface## Description
This is the implementation of this proposal https://code.fbi.h-da.de/danet/gosdn/-/issues/314.
## Related Issue
https://code.fbi.h-da.de/danet/gosdn/-/issues/314
## Motivation and Context
## How Has This Been Tested?
...## Description
This is the implementation of this proposal https://code.fbi.h-da.de/danet/gosdn/-/issues/314.
## Related Issue
https://code.fbi.h-da.de/danet/gosdn/-/issues/314
## 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.https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/468Resolve "PND related operations via NBI are located in CoreServer"2023-05-22T08:14:48ZFabian SeidlResolve "PND related operations via NBI are located in CoreServer"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
#276<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
#276https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/376Add additional example application hostname-checker2022-10-11T09:25:57ZAndre SterbaAdd additional example application hostname-checkerAdd additional example application hostname-checker.Add additional example application hostname-checker.https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/355Adjust northbound service naming2022-07-19T14:28:47ZAndre SterbaAdjust northbound service namingApply feedback about service naming from !349Apply feedback about service naming from !349https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/308Improve test coverage of rbac stuff2022-05-18T15:08:06ZGhost UserImprove test coverage of rbac stuff<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Trying to improve coverage. 🥴
## Related Issue
<!--- This project only accepts merge requests related to op...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Trying to improve coverage. 🥴
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!--- Please link to the issue here: -->
#207https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/305Resolve "Inconsistency and wrong definitions of requests/responses in proto f...2022-05-09T13:31:55ZGhost UserResolve "Inconsistency and wrong definitions of requests/responses in proto files"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Proto definitions have been updated to be in line with semantics e.g. GetPnd returns only one Pnd instead of...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
Proto definitions have been updated to be in line with semantics e.g. GetPnd returns only one Pnd instead of one or more. Changing proto files did require some minor changes in the way of preparing ONDs and SBIs for the response. Checked the CLI manually to see if it still works and if the presentation of returned objects looks okay.
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!--- Please link to the issue here: -->
#203.https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/298Refactor controller grpc api for better separation of concern2022-05-04T11:04:48ZAndre SterbaRefactor controller grpc api for better separation of concernResolves #199Resolves #199https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/270178 Progress on authz, not properly implemented yet2022-03-30T08:21:53ZGhost User178 Progress on authz, not properly implemented yet<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggest...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!--- Please link to the issue here: -->
See #178 for changes and information about them.
Closes #178https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/265Resolve "Update NB API for user managment"2022-03-28T09:05:22ZGhost UserResolve "Update NB API for user managment"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
API updated with dummy requests for user management and implemented dummy interceptor for future authenticat...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
API updated with dummy requests for user management and implemented dummy interceptor for future authentication/authorization.
Implemented dummy requests for login and logout in cli.
Further steps / functions will be implemented in #178 & #179.
Closes #177https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/257Resolve "Creating a new device with type plugin just returns a normal device"2022-03-23T16:57:31ZMalte BauchResolve "Creating a new device with type plugin just returns a normal device"- This commit is a interim solution that allows to create new devices with the `--type plugin` flag.
The orchestrator will only request capabilities of the provided device and build/return all the necessary files.
Therefore the api has ...- This commit is a interim solution that allows to create new devices with the `--type plugin` flag.
The orchestrator will only request capabilities of the provided device and build/return all the necessary files.
Therefore the api has been adjusted to allow to 'only' create gostructs and do not fire up a whole csbi.
- Additionaly the issue #176 has been addressed and the bug while creating multiple cSBIs has been fixed.
Appending to the `.Methods` string of the global `SouthboundStruct` was the
root cause for this bug. Creating a copy of `SouthboundStruct` and using
the copy in the process fixes this issue.
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!--- Please link to the issue here: -->
#175 , #176
Closes #175Malte BauchMalte Bauchhttps://code.fbi.h-da.de/danet/gosdn/-/merge_requests/248Resolve "A requested Device via gRPC is missing SBI"2022-03-10T12:54:45ZGhost UserResolve "A requested Device via gRPC is missing SBI"<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
See https://code.fbi.h-da.de/danet/gosdn/-/issues/167, for information on changes
Closes #167<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
See https://code.fbi.h-da.de/danet/gosdn/-/issues/167, for information on changes
Closes #167https://code.fbi.h-da.de/danet/gosdn/-/merge_requests/155Northbound Interface2021-06-01T16:37:22ZGhost UserNorthbound Interface<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
This merge request provides a new gRPC based interface and replaces the deprecated HTTP API. It brings cand...<!--- Provide a general summary of your changes in the Title above -->
## Description
<!--- Describe your changes in detail -->
This merge request provides a new gRPC based interface and replaces the deprecated HTTP API. It brings candy and love to everyone. And the old API was dead from the beginning anyway.
This MR is for implementation aspects only. Discussion of the actual API definition with protobuf should happen [here](https://code.fbi.h-da.de/cocsn/api/-/merge_requests/5)
## Related Issue
<!--- This project only accepts merge requests related to open issues -->
<!--- If suggesting a new feature or change, please discuss it in an issue first -->
<!--- If fixing a bug, there should be an issue describing it with steps to reproduce -->
<!--- Please link to the issue here: -->
#83
## Motivation and Context
<!--- Why is this change required? What problem does it solve? -->
gRPB based NBI was planned from the beginning.
## How Has This Been Tested?
<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran to -->
<!--- see how your change affects other areas of the code, etc. -->
Not yet yolo :fire:
## Types of changes
<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Breaking change (fix or feature that would cause existing functionality to change)
## Checklist:
<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
- [x] My code follows the code style of this project.
- [x] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [x] I have read the **CONTRIBUTING** document.
- [x] I have added tests to cover my changes.
- [x] All new and existing tests passed.v0.1.0 Codename Threadbare