Bad default location for REPORTS_ROOT about netbox-docker HOT 4 CLOSED

Hello,

at the moment we're not using reports but I also think /etc/netbox/reports would be a good location for that. But you're right. You can't map to /etc/netbox/reports when the configuration is mapped to /etc/netbox in Openshift.
So the proposal with mapping to /etc/netbox/config and /etc/netbox/reports is a good solution.

I won't be able to implement this now because I'm on holiday for the next weeks. If this can wait I will do the implementation.

@flipkill1985 Could you test this in our deployment?

from netbox-docker.

You're right, /etc/reports is a bad location for the reports. IMO this happened when we moved the 'main' configuration file to /etc/netbox and BASE_DIR now expands to /etc rather than /opt/netbox/netbox.

I agree that /etc/netbox/reports would be a good location. But I want to make sure this works out when using a configuration managed by Kubernetes/OpenShift. And I'm not entirely sure it does in a nice way.

IMO the nice way would be to have two separate ConfigMaps, one for reports and one for configuration. And when the one for configuration is mapped to /etc/netbox, I'm not sure if we can map the one for reports to /etc/netbox/reports.

So the main configuration might have to be moved to somewhere else, e.g. /etc/netbox/config, so that config and reports can co-exist.

Or am I over-complicating matters and it's fine to have one ConfigMap for config and reports?

I'd love to hear your opinion on this, and @tobiasge's as well, as he initially suggested to move the configuration to /etc/netbox.

from netbox-docker.

@tobiasge Have you had the chance to look into this? Would one ConfigMap for config and reports be fine or do you think they should better be separated?

from netbox-docker.

Hello,
I'm back now. I think it would be better to have them separated. This makes it possible for the user to also define a persistent volume in Openshift.

from netbox-docker.