Skip to content

Commit b4e9af0

Browse files
committed
nixos/librenms: fix optimizations and view cache on new installations
The optimizations and view-cache require an initialized database to work, so the current state breaks new installations. This commit fixes that by performing the optimizations and view caching after the database migrations.
1 parent 7796d95 commit b4e9af0

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

nixos/modules/services/monitoring/librenms.nix

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -603,15 +603,10 @@ in
603603
''
604604
)
605605
+ ''
606-
# clear cache after update and regenerate optimizations
606+
# clear cache after update (before migrations)
607607
OLD_VERSION=$(cat ${cfg.dataDir}/version)
608608
if [[ $OLD_VERSION != "${package.version}" ]]; then
609609
rm -r ${cfg.dataDir}/cache/*
610-
${artisanWrapper}/bin/librenms-artisan view:clear
611-
${artisanWrapper}/bin/librenms-artisan optimize:clear
612-
${artisanWrapper}/bin/librenms-artisan view:cache
613-
${artisanWrapper}/bin/librenms-artisan optimize
614-
echo "${package.version}" > ${cfg.dataDir}/version
615610
fi
616611
617612
# convert rrd files when the oneMinutePolling option is changed
@@ -623,6 +618,15 @@ in
623618
624619
# migrate db
625620
${artisanWrapper}/bin/librenms-artisan migrate --force --no-interaction
621+
622+
# regenerate cache after migrations after update
623+
if [[ $OLD_VERSION != "${package.version}" ]]; then
624+
${artisanWrapper}/bin/librenms-artisan view:clear
625+
${artisanWrapper}/bin/librenms-artisan optimize:clear
626+
${artisanWrapper}/bin/librenms-artisan view:cache
627+
${artisanWrapper}/bin/librenms-artisan optimize
628+
echo "${package.version}" > ${cfg.dataDir}/version
629+
fi
626630
'';
627631
};
628632

0 commit comments

Comments
 (0)