Cacti / documentation

Documentation for Cacti, Spine, RRDProxy and more
62 stars 52 forks source link

Update Plugins documentation to include procedure to update a plugin #88

Closed bmfmancini closed 3 years ago

netniV commented 3 years ago

Good idea for an update, needs one or two changes though.

bmfmancini commented 3 years ago

Hey Larry

Good point on the git pull

instead of mv plugin /tmp if you cp -R the folder to temp git pull will work

Also we should note that if you git pull the repo you will get the absolute latest so if you have been testing a release version it may be best to wget the release tar

On Sun, Dec 13, 2020, 18:44 Mark Brugnoli-Vinten notifications@github.com wrote:

Good idea for an update, needs one or two changes though.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Cacti/documentation/pull/88#issuecomment-744089888, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGEXTEMWOGAVXR5L6LM3KLSUVGU3ANCNFSM4UZWEMXQ .

bmfmancini commented 3 years ago

Git pull pull the commit code for the release ? I don't usually do that I usually wget the zip so when we are done testing in lab we move to prod with the same updates

On Sun, Dec 13, 2020, 18:50 Sean Mancini sean@seanmancini.com wrote:

Hey Larry

Good point on the git pull

instead of mv plugin /tmp if you cp -R the folder to temp git pull will work

Also we should note that if you git pull the repo you will get the absolute latest so if you have been testing a release version it may be best to wget the release tar

On Sun, Dec 13, 2020, 18:44 Mark Brugnoli-Vinten notifications@github.com wrote:

Good idea for an update, needs one or two changes though.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Cacti/documentation/pull/88#issuecomment-744089888, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGEXTEMWOGAVXR5L6LM3KLSUVGU3ANCNFSM4UZWEMXQ .

bmfmancini commented 3 years ago

@netniV all updated let me know if this works better

netniV commented 3 years ago

I will have to test the git commands. There is a way to fetch to the cache (stored in the .git folder), then checkout to a tag. There is no need to rename folders or anything else when you do that as long you haven't modified anything.

bmfmancini commented 3 years ago

I also added some updates to the FAQ document

TheWitness commented 3 years ago

Going to merge this. We can fix in subsequent pulls.