Drop centos6 and update pyperf test

.github/workflows/build-base-images.yml

@@ -73,7 +73,7 @@ jobs:
             --build-arg PHPSPY_BUILDER_UBUNTU=@sha256:cf31af331f38d1d7158470e095b132acd126a7180a54f263d386da88eb681d93
             --build-arg AP_BUILDER_CENTOS=@sha256:0f4ec88e21daf75124b8a9e5ca03c37a5e937e0e108a255d890492430789b60e
             --build-arg AP_BUILDER_ALPINE=@sha256:69704ef328d05a9f806b6b8502915e6a0a4faa4d72018dc42343f511490daf8a
-            --build-arg AP_CENTOS_MIN=:6
+            --build-arg AP_CENTOS_MIN=:7
             --build-arg BURN_BUILDER_GOLANG=@sha256:f7d3519759ba6988a2b73b5874b17c5958ac7d0aa48a8b1d84d66ef25fa345f1
             --build-arg GPROFILER_BUILDER=@sha256:be65f488b7764ad3638f236b7b515b3678369a5124c47b8d32916d6487418ea4
             --build-arg PYPERF_BUILDER_UBUNTU=@sha256:cf31af331f38d1d7158470e095b132acd126a7180a54f263d386da88eb681d93

.github/workflows/build-test-deploy.yml

@@ -85,7 +85,6 @@ jobs:
           - ubuntu:20.04
           - ubuntu:20.10
           - ubuntu:22.04
-          - centos:6
           - centos:7
           - centos:8
           - debian:8

scripts/build_x86_64_executable.sh

@@ -39,17 +39,13 @@ UBUNTU_VERSION_1804=@sha256:dca176c9663a7ba4c1f0e710986f5a25e672842963d95b960191
 # phpspy & pyperf - ubuntu:20.04
 UBUNTU_VERSION=@sha256:cf31af331f38d1d7158470e095b132acd126a7180a54f263d386da88eb681d93
 # async-profiler glibc - centos:7
-# requires CentOS 7 so the built DSO can be loaded into machines running with old glibc (tested up to centos:6),
-# we do make some modifications to the selected versioned symbols so that we don't use anything from >2.12 (what centos:6
-# has)
 AP_BUILDER_CENTOS=@sha256:0f4ec88e21daf75124b8a9e5ca03c37a5e937e0e108a255d890492430789b60e
 # async-profiler musl - alpine
 AP_BUILDER_ALPINE=@sha256:69704ef328d05a9f806b6b8502915e6a0a4faa4d72018dc42343f511490daf8a
 # dotnet builder - mcr.microsoft.com/dotnet/sdk:6.0.428-1-focal
 DOTNET_BUILDER=@sha256:525ce79a6f545131df515ce34f7ee086eb18e4d707eff9676b2678f2f23b6d9e
-# minimum CentOS version we intend to support with async-profiler (different between x86_64, where we require
-# an older version)
-AP_CENTOS_MIN=:6
+# minimum CentOS version we intend to support with async-profiler
+AP_CENTOS_MIN=:7
 # burn - golang:1.16.3
 BURN_BUILDER_GOLANG=@sha256:f7d3519759ba6988a2b73b5874b17c5958ac7d0aa48a8b1d84d66ef25fa345f1
 # bcc & gprofiler - centos:7

tests/test_executable.py

@@ -55,10 +55,6 @@ def test_executable(
         pytest.xfail("Dotnet-trace doesn't work with alpine: https://github.com/intel/gprofiler/issues/795")
 
     if exec_container_image is not None:
-        if "centos:6" in exec_container_image.tags and any("pyperf" in flag for flag in profiler_flags):
-            # don't run PyPerf on the centos:6 image, it fails. And in any case PyPerf can't run on centos:6.
-            pytest.skip("PyPerf test on centos:6")
-
         gprofiler_inner_dir = Path("/app")
         inner_output_dir = Path("/app/output")
         cwd = Path(os.getenv("GITHUB_WORKSPACE", os.getcwd()))

tests/test_python.py

@@ -87,6 +87,7 @@ def test_python_select_by_libpython(
         "3.7-musl-uwsgi",
     ],
 )
+@pytest.mark.flaky(reruns=3, reruns_delay=2)
 @pytest.mark.parametrize("profiler_type", ["py-spy", "pyperf"])
 def test_python_matrix(
     application_pid: int,
@@ -119,7 +120,9 @@ def test_python_matrix(
         ):
             pytest.xfail("This combination fails, see https://github.com/Granulate/gprofiler/issues/714")
 
-    with PythonProfiler(1000, 2, profiler_state, profiler_type, True, None, False, python_pyspy_process=[]) as profiler:
+    # Use longer duration for pyperf to ensure enough samples are collected for reliable assertions
+    duration = 5 if profiler_type == "pyperf" else 2
+    with PythonProfiler(1000, duration, profiler_state, profiler_type, True, None, False, python_pyspy_process=[]) as profiler:
         try:
             profile = snapshot_pid_profile(profiler, application_pid)
         except TimeoutError:
@@ -183,7 +186,9 @@ def test_dso_name_in_pyperf_profile(
     if is_aarch64() and profiler_type == "pyperf":
         pytest.skip("PyPerf doesn't support aarch64 architecture, see https://github.com/intel/gprofiler/issues/499")
 
-    with PythonProfiler(1000, 2, profiler_state, profiler_type, True, None, True, python_pyspy_process=[]) as profiler:
+    # Use longer duration for pyperf to ensure enough samples are collected
+    duration = 5 if profiler_type == "pyperf" else 2
+    with PythonProfiler(1000, duration, profiler_state, profiler_type, True, None, True, python_pyspy_process=[]) as profiler:
         profile = snapshot_pid_profile(profiler, application_pid)
     python_version, _, _ = application_image_tag.split("-")
     interpreter_frame = "PyEval_EvalFrameEx" if python_version == "2.7" else "_PyEval_EvalFrameDefault"