fluent-bit icon indicating copy to clipboard operation
fluent-bit copied to clipboard

tests: runtime: in_podman_metrics: fix warnings

Open nokute78 opened this issue 1 year ago • 6 comments

This patch is to fix warnings. Note: in_podman_metrics test failed on a system using cgroup v2. It doesn't relate this patch.


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:

  • [N/A] Example configuration file for the change
  • [X] Debug log output from testing the change
  • [X] 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-test label to test for all targets (requires maintainer to do).

Documentation

  • [N/A] Documentation required for this feature

Backporting

  • [N/A] Backport to latest stable release.

Debug/Valgrind output

Note: This test doesn't work on a system using cgroup v2. It doesn't relate this patch.

taka@taka-VirtualBox:~/git/fluent-bit/build$ bin/flb-rt-in_podman_metrics 
Test regular...                                 [2023/04/22 10:23:13] [ info] [fluent bit] version=2.1.1, commit=840d52649b, pid=6091
[2023/04/22 10:23:13] [ info] [storage] ver=1.2.0, type=memory, sync=normal, checksum=off, max_chunks_up=128
[2023/04/22 10:23:13] [ info] [cmetrics] version=0.6.1
[2023/04/22 10:23:13] [ info] [ctraces ] version=0.3.0
[2023/04/22 10:23:13] [ info] [input:podman_metrics:podman_metrics.0] initializing
[2023/04/22 10:23:13] [ info] [input:podman_metrics:podman_metrics.0] storage_strategy='memory' (memory only)
[2023/04/22 10:23:13] [ info] [input:podman_metrics:podman_metrics.0] Using config file /home/taka/git/fluent-bit/tests/runtime/data/podman/regular/config.json
[2023/04/22 10:23:13] [ info] [input:podman_metrics:podman_metrics.0] Detected cgroups v2
[2023/04/22 10:23:13] [ info] [input:podman_metrics:podman_metrics.0] Generating podman metrics (initial scrape)
(snip)
[ OK ]
FAILED: 2 of 6 unit tests have failed.
==6109== 
==6109== HEAP SUMMARY:
==6109==     in use at exit: 0 bytes in 0 blocks
==6109==   total heap usage: 7,537 allocs, 7,537 frees, 3,135,453 bytes allocated
==6109== 
==6109== All heap blocks were freed -- no leaks are possible
==6109== 
==6109== For lists of detected and suppressed errors, rerun with: -s
==6109== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

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.

nokute78 avatar Apr 22 '23 01:04 nokute78

I rebased to merge #7267

nokute78 avatar Apr 28 '23 22:04 nokute78

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Jul 30 '23 01:07 github-actions[bot]

ping

nokute78 avatar Oct 14 '23 23:10 nokute78

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Jan 16 '24 01:01 github-actions[bot]

ping. 🏓

cosmo0920 avatar Jan 17 '24 02:01 cosmo0920

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Apr 17 '24 01:04 github-actions[bot]