in_process_exporter_metrics: convert rss memory from pages to bytes
https://github.com/fluent/fluent-bit/issues/9343
Enter [N/A] in the box, if an item is not applicable to your change.
Testing Before we can approve your change; please submit the following in a comment:
- [x] Example configuration file for the change
- [x] Debug log output from testing the change
- [ ] Attached Valgrind output that shows no leaks or memory corruption was found
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
- [N/A] Run local packaging test showing all targets (including any new ones) build.
- [N/A] Set
ok-package-testlabel to test for all targets (requires maintainer to do).
Documentation
- [N/A] Documentation required for this feature
Backporting
- [ ] Backport to latest stable release.
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.
Example configuration: fluent-bit -i process_exporter_metrics -p metrics="memory" -o stdout
Output thats shows successful fix: process_memory_bytes{name="rsyslogd",pid="167",ppid="1",type="rss"} = 483328
cat /proc/167/status | grep Rss RssAnon: 472 kB RssFile: 0 kB RssShmem: 0 kB
Could you use signed-off in your commit? We use DCO and it complains signed-off is missing:
Summary
Commit sha: 09989e7, Author: Bettina Riedl, Committer: Bettina Riedl; The sign-off is missing.
Could you use signed-off in your commit? We use DCO and it complains signed-off is missing:
Summary
Commit sha: 09989e7, Author: Bettina Riedl, Committer: Bettina Riedl; The sign-off is missing.
Added sign-off :)
Force push was rebase on current main
Hi all, any chance to get this fix merged?