Skip to content
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

Netbox 3.6 #175

Closed
wants to merge 13 commits into from
Closed

Netbox 3.6 #175

wants to merge 13 commits into from

Conversation

kbelokon
Copy link

Pull Request

Related Issue

This PR completes the work started in #167 for transitioning to NetBox version 3.6.

New Behavior

I have finished the implementation for transitioning to NetBox version 3.6, as initiated in PR #167. My changes ensure full compatibility and functionality with this version of NetBox.

Contrast to Current Behavior

Unlike the current state in the branch of PR #167, where the transition to NetBox 3.6 was incomplete, my changes fully support this version, resolving all known issues and incompatibilities.

Changes to the Documentation

No updates to the documentation are required as the changes pertain only to internal compatibility with the new version of NetBox.

Proposed Release Note Entry

"Added support for NetBox version 3.6, completing the work started in PR #167. Ensures full compatibility and functionality with the new version."

Double Check

  • I have explained my PR according to the information in the comments.
  • My PR targets the dev branch.

ryanmerolle and others added 12 commits October 14, 2023 15:04
…box_3.6

netbox 3.6 support (Sourcery refactored)
…dability

- Remove unnecessary variable assignments and use self.cleaned_data directly
- Replace forms.ValidationError with ValidationError for consistency
- Simplify error message assignment by raising ValidationError directly
- Remove redundant comments and add more meaningful ones
- Simplify object filtering by removing unnecessary get calls
- Use dictionary for ValidationError messages to specify field errors
- Simplify assignment of assigned_object_id and assigned_object_type_id
- Simplify error message assignment in ACLExtendedRuleForm clean method
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions

2 Security Hotspots

See analysis details on SonarCloud

@netravnen
Copy link

netravnen commented Mar 26, 2024

Installing this PR into Netbox 3.7.4 running under LXC container (Debian 12, Python 3.11) seems to work without obvious issues. (Not using docker!)

Install the plugin

/opt/netbox/venv/bin/pip3 install git+https://github.com/netbox-community/netbox-acls.git@refs/pull/175/head

Modify the maximum allowed version to < 3.8

sed --in-place --expression='s/max_version = ".*"/max_version = "3.7.99"/' /opt/netbox/venv/lib/python3.11/site-packages/netbox_acls/__init__.py

Run the migration script

/opt/netbox/venv/bin/python3 /opt/netbox/netbox/manage.py migrate netbox_acls

Enable the netbox_acls plugin in /opt/netbox/netbox/netbox/configuration.py

Restart netbox

systemctl restart netbox.service netbox-rq.service

@jeffgdotorg jeffgdotorg mentioned this pull request Mar 26, 2024
9 tasks
@ryanmerolle
Copy link
Collaborator

I merged the changes for 3.6 into the repo. I had issues with cherrypicking and such, but I named @kbelokon in the release notes given the support provided. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants