Hi Matthias,
On 23/06/2026 09:39, Matthias Fischer wrote:
On 22.06.2026 22:42, Peer Dietzmann wrote:
Hi Matthias,
Hi Peer,
thank you for adding this. Will try it again when it was merged but it
looks like the tests I did today, so it should work.
Sorry, but this isn't clear to me - blame the weather... ;-)
This patch hasn't been pushed, he is NOT in patchwork or GIT.
Shall I push it or will you?
Or does your message mean that it is no longer needed?
now I'm a bit confused, both patches I sent, must be merged into the
next release to make it work. The patch you provided also. If you know
where this is done, please do it, all three are required. What I meant
is, that when everything is in testing, then I will test it again to
confirm that the changed files are shipped correctly.
Best,
Peer
Best
Matthias
Best,
Peer
On 22/06/2026 19:18, Matthias Fischer wrote:
*Third try - fixed typo in patch line*
- "-Np1" should have been "-Np2".
Without the quotes, of course.
Sorry...its too warm and the observium-lfs has a different path variable
(DIR_APP) than others. I put both files in attachments.
##########################################################
*Second try with attachment in unix format (line breaks)!*
##########################################################
On 22.06.2026 16:20, Peer Dietzmann wrote:
Hello,
Hi,
there is also another patch required to make the module working. But I
just can't find the file in the repository that needs to be changed:
You won't find this file in the repository - that would be too easy. ;-)
This 'apache'-file is stored in the observium archive
(observium/scripts/agent-local/apache). And only there. It is extracted
during the build phase, and is then immediately deleted. This is
controlled via the lfs file:
Line 83 (to be sure: first delete and then unpack source):
...
@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar axf $(DIR_DL)/$(DL_FILE)
...
Line 103 (clean up):
...
@rm -rf $(DIR_APP)
...
IMHO the best way to apply your changes would be a patchfile (see
attachment) which needs to be copied into the repository in the
directory '/src/patches/observium', e.g.).
Then the lfs file needs something like this in a new line *84*:
...
cd $(DIR_APP) && patch -Np2 <
$(DIR_SRC)/src/patches/observium/[NAME_OF_PATCHFILE]
...
HTH
Matthias
Line 29 of /usr/lib/observium_agent/scripts-available/apache has to be
changed to:
close INFILE;
} else {
# grab the status URL (fresh data)
@data = split /(\n)/, LWP::Simple::get(
'http://localhost:81/server-status?auto' )
or die "Data fetch failure.\n";
# write file
The *localhost* has to be changed to *localhost:81*
The concerned bug that should be fixed through those three patches is:
https://bugzilla.ipfire.org/show_bug.cgi?id=14017
Best,
Peer
On 22/06/2026 16:15, Peer Dietzmann wrote:
Redirect requests to Apache's status to it's backend, force local requests.
This is required to allow Observium's agent to pull metrics from Apache.
Best,
Peer
Signed-off-by: Peer Dietzmann <[email protected]>
---
config/httpd/httpd.conf | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/config/httpd/httpd.conf b/config/httpd/httpd.conf
index c694bffe2..09b10b50f 100644
--- a/config/httpd/httpd.conf
+++ b/config/httpd/httpd.conf
@@ -49,4 +49,10 @@ Include /etc/httpd/conf/default-server.conf
# Virtual server configuration
Include /etc/httpd/conf/vhosts.d/*.conf
+# Allow local clients to pull Apache's status
+<Location /server-status>
+ SetHandler server-status
+ Require local
+</Location>
+
# EOF
--
Mit freundlichem Gruß
Peer Dietzmann
Brecht-IT | Administration und Support
