netbox-demo-data icon indicating copy to clipboard operation
netbox-demo-data copied to clipboard

Fix docker instructions

Open zba opened this issue 1 year ago • 2 comments

fixed:

docker-compose is very old depricated , misslead to update database from netbox container , why it shall not be postgres ?

improve:

easer way to deploy sql , just from stdin instead of trics with docker exec

zba avatar Nov 07 '24 19:11 zba

honestly that looks a lot more chaotic than the original one sure - you can change the tires while driving - it's faster, saves on space etc. but it's also just more fragile and you'll not be able to tell what went wrong because it's just only got one failure more: FAIL CARCRASH. the same, here you're putting two operations into one so some traceability is gone.

FlorianHeigl avatar Mar 11 '25 22:03 FlorianHeigl

honestly that looks a lot more chaotic than the original one sure - you can change the tires while driving - it's faster, saves on space etc. but it's also just more fragile and you'll not be able to tell what went wrong because it's just only got one failure more: FAIL CARCRASH. the same, here you're putting two operations into one so some traceability is gone.

Because it not uses docker ps command which makes things broken, it is just wrong way to do this this way (docker cp....) , also old instruction just outdated. not works, new one worked at moment of PR. Also you not need to have netbox-demo repo pulled, just take sql.

In other words why you need to change tires, if it just traffic ligh stop ?

zba avatar Mar 21 '25 18:03 zba