Remove debugs

Author: Ariouz

.github/scripts/extract_matrix.py

@@ -24,17 +24,11 @@
     "enterprise",
     "corporate-compliance",
 
-    # Jobs failing in GitHub Actions:buffer overflow, out of memory, auditwheel incompatible with runner ubuntu-latest's glibc version
+    # Jobs failing in GitHub Actions:buffer overflow, out of memory
     "python-svm-unittest",
     "cpython-gate",
 
-    # Python 3.8.10 (matrix.python_version) is not available for linux aarch64
-    # cf. https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json
-    #"linux-aarch64",
-
-    # WIP
     "darwin",
-    #"windows"
 )
 
 DOWNLOADS_LINKS = {

.github/workflows/ci-matrix-gen.yml

@@ -190,42 +190,6 @@ jobs:
         run: |
           ${{ matrix.setup_steps }}
 
-      - name: Check disk space and memory usage
-        if: ${{ runner.os == 'macOS' }}
-        run: |
-          echo "before purge"
-          df -h
-          pagesize=$(sysctl -n hw.pagesize)
-          vm_stat
-          vm_stat | awk -v ps=$pagesize '
-            /Pages free/      {free=$3}
-            /Pages active/    {active=$3}
-            /Pages inactive/  {inactive=$3}
-            /Pages speculative/ {speculative=$3}
-            /Pages wired/     {wired=$4}
-            END {
-              used=(active+inactive+wired+speculative)*ps/1024/1024;
-              free=free*ps/1024/1024;
-              printf "Used: %.2f MB\nFree: %.2f MB\n", used, free
-            }'
-          ps aux | sort -nrk 4 | head
-          sudo purge
-          echo "after purge"
-          pagesize=$(sysctl -n hw.pagesize)
-          vm_stat
-          vm_stat | awk -v ps=$pagesize '
-            /Pages free/      {free=$3}
-            /Pages active/    {active=$3}
-            /Pages inactive/  {inactive=$3}
-            /Pages speculative/ {speculative=$3}
-            /Pages wired/     {wired=$4}
-            END {
-              used=(active+inactive+wired+speculative)*ps/1024/1024;
-              free=free*ps/1024/1024;
-              printf "Used: %.2f MB\nFree: %.2f MB\n", used, free
-            }'
-
-
       - name: Free resources on macOS
         run: |
           echo "Disabling Spotlight indexing..."
@@ -248,7 +212,6 @@ jobs:
         working-directory: main
         if: ${{ matrix.run_steps && runner.os != 'Windows' }}
         run: |
-          env
           ${{ matrix.run_steps }}
       - name: Run on Windows
         working-directory: main
@@ -263,7 +226,6 @@ jobs:
         shell: bash
         working-directory: main
         run: |
-          ls && pwd
           tar cf ${{ matrix.provide_artifact[0] }}.tar ${{ matrix.provide_artifact[1] }}
       - name: Upload artifacts
         if: ${{ matrix.provide_artifact }}

.github/workflows/ci-unittest-retagger.yml

@@ -33,7 +33,6 @@ jobs:
       - name: Init Github branch
         working-directory: main
         run: |
-          echo "Set new repo origin url"
           git remote set-url origin https://[email protected]/${{ github.repository }}.git
           git remote -v
           git config --global user.name "Retagger Workflow" 
@@ -47,7 +46,6 @@ jobs:
         working-directory: main
         run: |
           mkdir -p ../retagger-reports ../diffs
-          ls && ls ..
           mv retagger-report*.json ../retagger-reports
 
           declare -a os_list=("linux-x86_64" "linux-aarch64" "win32-AMD64")

graalpython/com.oracle.graal.python.test/src/runner.py

@@ -497,8 +497,6 @@ def update_tags(test_file: 'TestFile', results: list[TestResult], tag_platform:
                 untag_failed=False, untag_skipped=False, untag_missing=False):
     current = read_tags(test_file, allow_exclusions=True)
     exclusions, current = partition_list(current, lambda t: t.is_exclusion)
-    print(f"exclusions tests: {exclusions}")
-    print(f"current tests: {current}")
     status_by_id = {r.test_id.normalized(): r.status for r in results}
     tag_by_id = {}
     for tag in current:
@@ -508,7 +506,6 @@ def update_tags(test_file: 'TestFile', results: list[TestResult], tag_platform:
             tag_by_id[tag.test_id] = tag
 
     for test_id, status in status_by_id.items():
-
         if tag := tag_by_id.get(test_id):
             if status == TestStatus.SUCCESS:
                 tag_by_id[test_id] = tag.with_key(tag_platform)
@@ -1292,7 +1289,6 @@ def read_tags(test_file: TestFile, allow_exclusions=False) -> list[Tag]:
                 excluded_keys = []
                 for key in keys:
                     if key.startswith('$'):
-                        print(f"[DEBUG] Found github excluded test {test} on platform {key}")
                         excluded_keys.append(key.removeprefix('$'))
 
                 tag = Tag(
@@ -1302,7 +1298,6 @@ def read_tags(test_file: TestFile, allow_exclusions=False) -> list[Tag]:
                     is_exclusion=is_exclusion,
                     comment=comment,
                 )
-               
                 if not is_exclusion or allow_exclusions:
                     tags.append(tag)
                 comment = None

mx.graalpython/mx_graalpython.py

@@ -253,6 +253,9 @@ def _is_overridden_native_image_arg(prefix):
 
 
 def github_ci_build_args():
+    # Determine memory and parallelism for GitHub CI builds
+    # Use 90% of available memory up to 14GB, but at least 8GB
+    # Set cores to number of CPUs if at least 4 cores and enough memory, otherwise 1
     total_mem = psutil.virtual_memory().total / (1024 ** 3)
     min_bound = 8
     max_mem = 14*1024
@@ -347,7 +350,6 @@ def libpythonvm_build_args():
         ]
     else:
         print(invert("Not using an automatically selected PGO profile"), file=sys.stderr)
-    print(f"[DEBUG] libpythonvm args: {build_args}")
     return build_args
 
 
@@ -849,7 +851,6 @@ def graalpy_standalone_home(standalone_type, enterprise=False, dev=False, build=
 
     print(f"[DEBUG] GITHUB_CI env: {os.environ.get('GITHUB_CI')}")
     if GITHUB_CI:
-        print("[DEBUG] Running in GitHub Ci")
         mx_args.append("--extra-image-builder-argument=-Ob")
     else:
         mx_args.append("--extra-image-builder-argument=-g")
@@ -1258,8 +1259,6 @@ def run_python_unittests(python_binary, args=None, paths=None, exclude=None, env
     else:
         args.append(os.path.relpath(_python_unittest_root()))
 
-    print(f"[DEBUG] args: {args}")
-
     mx.logv(shlex.join([python_binary] + args))
     if lock:
         lock.release()