Skip to content

Support NetBox v3 #22

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Guillaume001
Copy link

@Guillaume001 Guillaume001 commented Jul 24, 2021

  1. Create netbox/static folder before collecting static files otherwise there is a permission error to create this folder as netbox user.
  2. UserWarning: The CACHE_TIMEOUT configuration parameter was removed in v3.0.0 and no longer has any effect.
  3. Python 3.6 is no longer supported. NetBox v3.0 supports Python 3.7, 3.8, and 3.9.

python36,
python36-devel,
python39,
python39-devel,
Copy link

Choose a reason for hiding this comment

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

Most distros don't have packages named python39 or python39-devel, at least not available in their main repos. The only match I could find is in CentOS 8 AppStream. It would be best if these could be overridden. For example, in CentOS 7, the SCL package is named rh-python38-python.

Copy link
Author

Choose a reason for hiding this comment

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

This module supports CentOS and RedHat 8 only... So I bump this version to match with NetBox v3 requirements (with main repositories of these distros). If you want to make this module compatible with CentOS 7, indeed you must have as a prerequisite the SCL repository and specify the version 3.8. I don't think so this feature should be in this PR...

Copy link

Choose a reason for hiding this comment

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

Ok, I filed #24

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