search

howardjones / network-weathermap

Network Weathermap draws diagrams from data
http://www.network-weathermap.com/
MIT License
426 stars 94 forks source link

Features/vagrant sans cacti user #148

Closed RutgerLubbers closed 5 years ago

RutgerLubbers commented 6 years ago

Hi Howard,

Another pull request, next to (or as a replacement of) the previous one. Again, this one concerns the Vagrant setup. The previous request has a problem with rights. The cacti user cannot write in certain directories, for which the rights have to be fixed outside Vagrant.

So I changed the owner of the files from the cacti user to the vagrant user (within Vagrant). Yes, this is not ideal, however, this is a development setup and not a production environment.

Next to this, I changed the way PHP is installed. There is now a list of PHP versions to install and one to make active. The list now contains 5.6, 7.0, 7.1 and 7.2. In the future this list may become a bit shorter.

We would like to use this feature since we would like to see the new weathermap (cacti) 1.0 plugin working on cacti 1.3.38. For this, we are working on: adding groups; editing groups; adding maps to a group; removing maps from a group; and getting the overlay & click functionality working.

For this we will create a PR soon too.

Kind Regards,

Rutger

howardjones commented 5 years ago

Sorry for the long wait on this one - looks good! I'm merging it now. Thanks for working on it.