Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LibreNMS] Updated chart and dependencies to work with Librenms latest (24.3.0) #74

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

InputObject2
Copy link

@InputObject2 InputObject2 commented Apr 30, 2024

This PR modifies two things (aside from the versions in the chart.yaml)

Changing the disabled commands on Redis

In the default values, I changed the disabled commands list in redis since Librenms needs FLUSHDB on startup and it is in the default bitnami list.

Mounting the config.php file using an init-container

The handling of files in /opt/librenms/config.d seemingly changed between v22 and v24, there is a purge on startup and the config files are autogenerated from /data/config/*.php. There is also a chmod/chown that gets run on /data/config.

This means that the configmap for config.php cannot be placed in /opt/librenms/config.d and it also cannot be placed inside /data/config since a configmap mount is readonly.

In the statefulset template, I added an init-container to mount the configmap into /tmp/config and then perform a copy to /data/config since the librenms startup scripts needs all files to be read-writable in order to chmod them.

I removed the config from the app container since we move it into /data/config/ on startup and from there the templater picks it up and generates the config that gets put into /opt/librenms/config.d.

Same thing for the syslog since it also uses the librenms config and image.

So all this to say, it runs now:

image

Changed the disabled commands list in redis since Librenms needs FLUSHDB on startup and it is in the default bitnami list.
@InputObject2
Copy link
Author

@gerardsegarra would you be able to take a look at this PR?

@InputObject2 InputObject2 changed the title Updated chart and dependencies to work with Librenms latest (24.3.0) [LibreNMS] Updated chart and dependencies to work with Librenms latest (24.3.0) Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant