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

Add env variable to control CENSUS_REPORTING_ENABLED #999

Open
candlerb opened this issue Apr 26, 2023 · 0 comments
Open

Add env variable to control CENSUS_REPORTING_ENABLED #999

candlerb opened this issue Apr 26, 2023 · 0 comments
Labels
enhancement The issue describes an enhancement that we would like to implement in the future.

Comments

@candlerb
Copy link

Desired Behavior

Netbox v3.4.9 quietly introduced a log of user installations. This can be controlled by CENSUS_REPORTING_ENABLED in configuration.py (defaulting to True). netbox-community/netbox#12337

I would like this to be exposed as an environment variable.

Contrast to Current Behavior

From v3.4.9, every time I spin up a test or lab instance of Netbox using docker, it's liable to report itself externally as a new instance.

Required Changes

Presumably in https://github.com/netbox-community/netbox-docker/blob/release/configuration/configuration.py similar to ENFORCE_GLOBAL_UNIQUE

Discussion: Benefits and Drawbacks

I could configure this in extra.py, but that makes it more difficult to deploy.

@tobiasge tobiasge added the enhancement The issue describes an enhancement that we would like to implement in the future. label Apr 27, 2023
kindlich added a commit to kindlich/netbox-docker that referenced this issue Jan 16, 2024
kindlich added a commit to kindlich/netbox-docker that referenced this issue Jan 16, 2024
infracaninophile added a commit to oxcert/netbox-docker that referenced this issue May 23, 2024
* Update dependency psycopg to v3.1.17

* Add env var for CENSUS_REPORTING_ENABLED

See netbox-community#999

* Add Redis Sentinel config as environment vars

* Update dependency psycopg to v3.1.18

* fix segfault

* docker-entrypoint.sh: clarify default token message

When a default admin API token is found, a warning is displayed. As it is only called "token", some users might not know what token is referred to. Also the message should give a hint or link to a documentation on how to remove it.

* Update Ubuntu and Nginx Unit

* Use ARM64 self-hosted runner

* Update dependency django-auth-ldap to v4.7.0

* Avoid duplicate checks for Renovate PRs

* Fix netbox-community#1127: Added sentry-sdk to requirements

* Update dependency sentry-sdk to v1.44.0

* Removed version line from the yaml file as it is obsolete

* Update dependency sentry-sdk to v1.44.1

* Update dependency django-auth-ldap to v4.8.0

* Update dependency sentry-sdk to v1.45.0

* Prepare for Netbox 4.0

* Cancel workflow runs for in progress PRs

* Update dependency dulwich to v0.22.0

* Update dependency dulwich to v0.22.1

* Adds SECURE_* parameters introduced in 3.7.6

See netbox-community/netbox#15644

* Update github/super-linter action to v6

* Update dependency sentry-sdk to v2

* Disabled new tests

* Update dependency django-storages to v1.14.3

* Fixed regex for social-auth-core replacement

* Update dependency sentry-sdk to v2.1.1

* Preparation for 2.9.0

* Fix netbox-community#1222: Use /login/ as health check URL

* Install Sentry Django integration correctly

* Preparation for 2.9.1

* Github -> GitHub

Trying to assuage textlint

* Temporarily comment out the netbox_dns plugin addition

Looks like this is causing the upstream netbox CI tests to fail.

* Partially revert upstream CI changes

We don't have a self-hosted linux/arm64 platform to use, so reinstate
the use of QEMU.  Yes, it's slow, but slow and working always beats
broken at any speed.

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Tobias Genannt <[email protected]>
Co-authored-by: kindlich <[email protected]>
Co-authored-by: NdFeB <[email protected]>
Co-authored-by: tbotnz <[email protected]>
Co-authored-by: Florian Knodt <[email protected]>
Co-authored-by: Tobias Genannt <[email protected]>
Co-authored-by: Chandan Kumar <[email protected]>
Co-authored-by: Christian Mäder <[email protected]>
Co-authored-by: Christian Mäder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue describes an enhancement that we would like to implement in the future.
Projects
None yet
Development

No branches or pull requests

2 participants