search

TheNetworkGuy / netbox-zabbix-sync

Python script to synchronise Netbox devices to Zabbix.
MIT License
109 stars 27 forks source link

Adding docker-compose.yml and updated documentation #65

Open Kage1 opened 1 month ago

Kage1 commented 1 month ago

Added docker-compose.yml as proposed by the owner and updated the readme to reflect the new file with usage instructions.

TheNetworkGuy commented 1 month ago

Could you edit the location of the config file to .config.py instead of the /opt/netbox-zabbix? Not everyone will have the script running in that directory so it makes more sense to include the file from the directory in which both files are found. Since this is also the case in the Git repo :)

Kage1 commented 4 weeks ago

Could you edit the location of the config file to .config.py instead of the /opt/netbox-zabbix? Not everyone will have the script running in that directory so it makes more sense to include the file from the directory in which both files are found. Since this is also the case in the Git repo :)

The portion of the docker-compose volume map after the colon : is the internal map to the docker container which is mapped to /opt/netbox-zabbix. This is where the file lives in the docker container.

The host file system could be updated to reference the same directory where the compose file is run at.

Kage1 commented 4 weeks ago

I'm not very good with dealing with PR's... Do I need to close this one and create a new PR or is there a way to update this one to include the updated compose file?

Edit: Looks like after the edit it's now showing up in the PR.