-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Role needs to be updated to latest NetBox #132
Comments
I can confirm that role (as of now) works perfectly for 2.10 and 2.11 (I'm currently using netbox 2.11.11) |
I'm not too surprised that it works as-is for later versions, but if there were new features introduced they might need configuration support within the role. |
Hello, I am interesting to help you on this topic to be able to install Netbox 3 with your role. What do you think ? Regards, |
I did an upgrade from 2.10.10 to 3.1.11 using #141 and it went smoothly. |
Guy may I suggest to add to the readme as prequisite to make this role work ? (for those who use ansible-core) ansible-galaxy collection install community.postgresql |
Hey guys. I haven't managed to proactively keep this role in sync with upstream (living circumstances over the past year and a half haven't been very conducive to many activities), so I could use some help if anyone's willing to pitch in.
Basically, the major version release notes and all minor version update notes for NetBox need to be reviewed and then this role needs to be updated to support new features or use updated configurations, etc. Some changes that might affect earlier versions negatively may need to be version-gated, as well.
This should be done separately for the latest 2.10 and latest 2.11 releases. Refer to the following PRs for examples of how it's been done in the past.
Please feel free to open a PR like this to keep track of your work if you'd like to work on this. I'd greatly appreciate it. Whenever possible, I'll try to provide my own code reviews as well.
In the worst case that this isn't able to be completed sooner, I'll probably be able to focus more on this hopefully by the end of the year.
The text was updated successfully, but these errors were encountered: