-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add support for all new cloud models in Nautobot 2.3+ #434
Conversation
@joewesch - Starting this PR to spark a conversation about an issue I discovered. In the new cloud resources, a cloud_resource_type has a You can see how I overcame this in the |
We discussed this internally, but adding context here for transparency. I explained this sounds similar to how we dealt with name clashes in other modules. I suggested to review how we overcame this for |
Still need to add modules to support the many-to-many assignments.
|
This is complete and passing integration tests locally. Need PR approved and a release from pynautobot to update dependency for CI. PYNAUTOBOT PR: nautobot/pynautobot#248 |
tests/integration/targets/latest/tasks/cloud_network_prefix_assignment.yml
Outdated
Show resolved
Hide resolved
…ted cloud resource type tests
CI will pass once a patch is released in pynautobot and dependency is bumped. PR is submitted on that repo and awaiting approval: nautobot/pynautobot#253 |
Ok, got it now. Never use alternate name, use the alias instead. I should have got them all this time. |
This is the start of the work needed to close issues: #373 #375 #371 #372 #374
The PR has a dependency and is directly related to nautobot/pynautobot#248 in the https://github.com/nautobot/pynautobot repo.