fix(kbuild): Do not build kselftest if not set in jobfilter

kernelci/kbuild.py

@@ -39,14 +39,14 @@
 
 
 CROS_CONFIG_URL = \
     "https://chromium.googlesource.com/chromiumos/third_party/kernel/+archive/refs/heads/{branch}/chromeos/config.tar.gz"  # noqa
 LEGACY_CONFIG = [
     'config/core/build-configs.yaml',
     '/etc/kernelci/core/build-configs.yaml',
 ]
 FW_GIT = "https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git"  # noqa
 
 # TODO: find a way to automatically fetch this information
 LATEST_LTS_MAJOR = 6
 LATEST_LTS_MINOR = 6
 
@@ -156,6 +156,10 @@
                 self._kfselftest = False
             else:
                 self._kfselftest = True
+            if node['jobfilter'] and self._kfselftest is True:
+                kselftest_name = node['name'] + "-kselftest"
+                if kselftest_name not in node['jobfilter']:
+                    self._kfselftest = False
             self._apijobname = jobname
             self._steps = []
             self._artifacts = []
@@ -452,8 +456,8 @@
             sys.exit(1)
 
         print(f"Searching for fragment {fragname} in {cfg_path}")
         if 'fragments' in yml:
             frag = yml['fragments']
         else:
             print("No fragments section in config file")
             self.submit_failure("No fragments section in config file")
@@ -479,7 +483,7 @@
             elif fragment.startswith("CONFIG_"):
                 content = fragment + '\n'
             else:
                 # TODO: implement 'path' option properly
                 content = self.add_legacy_fragment(fragment)
 
             fragfile = os.path.join(self._fragments_dir, f"{num}.config")
@@ -528,8 +532,8 @@
         for i in range(0, fragnum):
             self.addcmd("./scripts/kconfig/merge_config.sh" +
                         f" -m .config {self._fragments_dir}/{i}.config")
         # TODO: olddefconfig should be optional/configurable
         # TODO: log all warnings/errors of olddefconfig to separate file
         self.addcmd("make olddefconfig")
         self.addcmd(f"cp .config {self._af_dir}/")
         self.addcmd("cd ..")
@@ -537,12 +541,12 @@
 
     def _generate_script(self):
         """ Generate shell script for complete build """
         # TODO(nuclearcat): Fetch firmware only if needed
         print("Generating shell script")
         fragnum = self._parse_fragments(firmware=True)
         self._merge_frags(fragnum)
         if not self._dtbs_check:
             # TODO: verify if CONFIG_EXTRA_FIRMWARE have any files
             # We can check that if fragments have CONFIG_EXTRA_FIRMWARE
             self._fetch_firmware()
             self._build_kernel()
@@ -640,7 +644,7 @@
         """ Add kernel image packaging steps """
         self.startjob("package_kimage")
         self.addcmd("cd " + self._srcdir)
         # TODO(nuclearcat): Not all images might be present
         for img in KERNEL_IMAGE_NAMES[self._arch]:
             self.addcmd("cp arch/" + self._arch + "/boot/" + img + " ../artifacts", False)
             # add image to artifacts relative to artifacts dir
@@ -709,7 +713,7 @@
 
     def serialize(self, filename):
         """ Serialize class to json """
         # TODO(nuclearcat): Implement to_json method?
         data = json.dumps(self, default=lambda o: o.__dict__,
                           sort_keys=True, indent=4)
         with open(filename, 'w') as f:
@@ -772,7 +776,7 @@
         '''
         Upload artifacts to storage
         '''
         # TODO: Upload not using upload_single, but upload as multiple files
         print("[_upload_artifacts] Uploading artifacts to storage")
         node_af = {}
         storage = self._get_storage()
@@ -854,7 +858,7 @@
             api.node.update(node)
         except requests.exceptions.HTTPError as err:
             err_msg = json.loads(err.response.content).get("detail", [])
             self.log.error(err_msg)
         return
 
     def submit(self, retcode, dry_run=False):