Fix pylint not found in CI

Author: furtib

test/pylint/BUILD

@@ -12,6 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+load("@rules_python//python:py_binary.bzl", "py_binary")
 load("//test/pylint:pylint_test.bzl", "pylint_test")
 
 exports_files([
@@ -20,6 +21,13 @@ exports_files([
     "requirements.txt",
 ])
 
+py_binary(
+    name = "pylint_bin",
+    srcs = ["pylint_bin.py"],
+    visibility = ["//visibility:public"],
+    deps = ["@pylint_deps//pylint"],
+)
+
 pylint_test(
     name = "pylint",
     exclude = [
@@ -31,6 +39,6 @@ pylint_test(
         "test",
         "__init__.py",
     ],
-    pylint = "@pylint_deps//pylint",
+    pylint = ":pylint_bin",
     pylintrc = ":.pylintrc",
 )

test/pylint/pylint_bin.py

@@ -0,0 +1,19 @@
+# Copyright 2026 Ericsson AB
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+"""Entry point wrapper for pylint, used as a Bazel py_binary."""
+
+from pylint import run_pylint
+
+run_pylint()

test/pylint/pylint_runner.py

@@ -18,17 +18,16 @@
 Uses a workspace marker file (like MODULE.bazel) to find the repo
 root.
 
-Pylint is imported as a Python dependency provided by Bazel,
-not invoked from PATH.
+The pylint executable is provided by Bazel as a py_binary and
+passed via the --pylint argument.
 """
 
 import argparse
 import os
 import pathlib
+import subprocess
 import sys
 
-from pylint import lint
-
 
 def workspace_root(marker):
     """Resolve the workspace root from a marker file."""
@@ -52,11 +51,33 @@ def find_py_files(root, paths, exclude_patterns):
     return files
 
 
+def resolve_runfile(path):
+    """Resolve a Bazel rootpath to an absolute path via runfiles."""
+    # In a py_test, the working directory is the runfiles tree
+    # so rootpaths can be resolved relative to cwd
+    resolved = os.path.realpath(path)
+    if os.path.exists(resolved):
+        return resolved
+    # Try via RUNFILES_DIR
+    runfiles_dir = os.environ.get("RUNFILES_DIR", "")
+    workspace = os.environ.get("TEST_WORKSPACE", "_main")
+    if runfiles_dir:
+        candidate = os.path.join(runfiles_dir, workspace, path)
+        if os.path.exists(candidate):
+            return os.path.realpath(candidate)
+    return resolved
+
+
 def main():
     """Parse arguments and run pylint on discovered sources."""
     parser = argparse.ArgumentParser(
         description="Run pylint on Python source files."
     )
+    parser.add_argument(
+        "--pylint",
+        required=True,
+        help="Path to the pylint executable.",
+    )
     parser.add_argument(
         "--workspace",
         required=True,
@@ -82,6 +103,12 @@ def main():
     )
     args = parser.parse_args()
 
+    # Resolve paths before changing directories
+    pylint_exe = resolve_runfile(args.pylint)
+    rcfile_path = None
+    if args.rcfile:
+        rcfile_path = os.path.realpath(args.rcfile)
+
     root = workspace_root(args.workspace)
     sources = find_py_files(root, args.paths, args.exclude)
 
@@ -93,15 +120,14 @@ def main():
     # (which uses os.getcwd()) can resolve project imports
     os.chdir(root)
 
-    pylint_args = []
-    if args.rcfile:
-        rcfile_path = os.path.realpath(args.rcfile)
-        pylint_args.append(f"--rcfile={rcfile_path}")
-    pylint_args.extend(str(s) for s in sources)
+    cmd = [pylint_exe]
+    if rcfile_path:
+        cmd.append(f"--rcfile={rcfile_path}")
+    cmd.extend(str(s) for s in sources)
 
-    print(f"Running: pylint {' '.join(pylint_args)}")
-    result = lint.Run(pylint_args, exit=False)
-    sys.exit(result.linter.msg_status)
+    print(f"Running: {' '.join(cmd)}")
+    result = subprocess.run(cmd, check=False)
+    sys.exit(result.returncode)
 
 
 if __name__ == "__main__":

test/pylint/pylint_test.bzl

@@ -26,8 +26,9 @@ Usage:
         ],
         paths = [
             "src",
+
         ],
-        pylint = "@pylint_deps//pylint",
+        pylint = ":pylint_bin",
         pylintrc = ":.pylintrc",
         workspace = "//:MODULE.bazel",
     )
@@ -48,7 +49,7 @@ def pylint_test(
 
     Args:
         name: Test name.
-        pylint: Label of the pylint pip package.
+        pylint: Label of the pylint py_binary executable.
         pylintrc: Label of the .pylintrc configuration file.
         paths: Directories or .py files to lint (relative to workspace root).
         workspace: Label of a file at the workspace root, used to locate
@@ -64,6 +65,7 @@ def pylint_test(
         pylint_tags.append("external")
 
     args = [
+        "--pylint=$(rootpath {})".format(pylint),
         "--workspace=$(rootpath {})".format(workspace),
         "--rcfile=$(rootpath {})".format(pylintrc),
     ]
@@ -79,10 +81,10 @@ def pylint_test(
         main = "//test/pylint:pylint_runner.py",
         args = args,
         data = [
+            pylint,
             pylintrc,
             workspace,
         ],
-        deps = [pylint],
         local = True,
         tags = pylint_tags,
         **kwargs