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

Preparation for Netbox 3.5 #990

Merged
merged 5 commits into from
Apr 28, 2023

Conversation

tobiasge
Copy link
Member

@tobiasge tobiasge commented Apr 18, 2023

Related Issue: -

New Behavior

  • Reports and Scripts have changed in Netbox 3.5. They need to be uploaded now.
    Docker compose now creates volumes as it does for the media files.
  • Napalm has been removed from Netbox 3.5
    All configuration entries for Napalm were removed and napalm itself is removed from the requirements file
  • Removed Gunicorn from the image
    Nginx Unit has been used for a while now. No need to install Gunicorn
  • Removed BASE_PATH ENV variable
    Changing the BASE_PATH needs more than just setting the variable. To prevent issues it's no longer allowed to change it that easily.
  • Changed the default SECRET_KEY
    Netbox requires a SECRET_KEY with a minimum of 50 characters in version 3.5 and up

Contrast to Current Behavior

Discussion: Benefits and Drawbacks

Changes to the Wiki

Proposed Release Note Entry

  • See changes from above

Double Check

  • I have read the comments and followed the PR template.
  • I have explained my PR according to the information in the comments.
  • My PR targets the develop branch.

@tobiasge tobiasge force-pushed the prepare-for-nb-35 branch 3 times, most recently from 2ce7c02 to cbf0a97 Compare April 19, 2023 09:08
- Reports and Scripts have changed in Netbox 3.5. They need to be uploaded now.
  The Docker compose now creates a volume as it does for the media files
- Napalm has been removed from Netbox 3.5
  All configuration entries for Napalm were removed and napalm itself is removed from the requirements file
- Removed Gunicorn from the image
  Nginx Unit has been used for a while now. No need to install Gunicorn
Setting the BASE_PATH is a more involved process than just setting this variable.
To prevent surprises the option to set this via ENV variable was removed.
@tobiasge tobiasge marked this pull request as ready for review April 27, 2023 16:07
@tobiasge tobiasge requested a review from cimnine April 27, 2023 16:07
build.sh Show resolved Hide resolved
env/netbox.env Show resolved Hide resolved
@tobiasge tobiasge merged commit 02a9264 into netbox-community:develop Apr 28, 2023
@tobiasge tobiasge deleted the prepare-for-nb-35 branch April 28, 2023 05:41
This was referenced Apr 28, 2023
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.

2 participants