-
Notifications
You must be signed in to change notification settings - Fork 9
[Feature]: add NSISTP product and workflows to example orchestrator #53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
[Feature]: add NSISTP product and workflows to example orchestrator #53
Conversation
|
This seems to downgrade NetBox version which I guess is not desirable? |
3b9f7e5 to
e63cf08
Compare
I rebased against master, NetBox is now at 4.4.1, which was the original/latest version, right? |
Removed forms volume from orchestrator service.
Updated the Netbox base image to version 4.4.1.
e63cf08 to
1367bfe
Compare
This reverts commit 0a9c1cc. At least PortsValueError is needed ...
migrations/versions/schema/2025-08-28_0e8d17ce0f06_reconcile_workflows_l2vpn.py
Show resolved
Hide resolved
| return TerminateNsistpForm | ||
|
|
||
|
|
||
| @step("Delete subscription from OSS/BSS") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be replaced by a @step("Remove VLANs from IMS") that actually removes the VLAN's from NetBox, as is done in terminate_l2vpn.py.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added this to issue #54.
Summary
This PR implements NSISTP products, product blocks and workflows to add NSI support to the example orchestrator.
Included
Blocked by VlanRange issue: #54
Issue: #41