public inbox for kdevops@lists.linux.dev
 help / color / mirror / Atom feed
From: Luis Chamberlain <mcgrof@kernel.org>
To: Chuck Lever <cel@kernel.org>, Daniel Gomez <da.gomez@kruces.com>,
	kdevops@lists.linux.dev
Cc: Luis Chamberlain <mcgrof@kernel.org>
Subject: [PATCH 09/23] kconfig: remove CONFIG_KERNEL_CI infrastructure
Date: Mon, 11 Aug 2025 15:24:36 -0700	[thread overview]
Message-ID: <20250811222452.2213071-10-mcgrof@kernel.org> (raw)
In-Reply-To: <20250811222452.2213071-1-mcgrof@kernel.org>

Remove the CONFIG_KERNEL_CI Kconfig infrastructure now that all
workflows have been converted to use their own internal loop
configurations or SOAK_DURATION:

- Remove Kconfig.kernel-ci file with all kernel-ci options
- Remove kernel-ci menu and source from Kconfig.kdevops
- Remove CONFIG_KERNEL_CI from kernel-testing defconfig
- Update fstests documentation to mention SOAK_DURATION instead

This completes the removal of the confusing CONFIG_KERNEL_CI feature
that was originally added for loop testing but has been superseded
by better architectural choices.

Generated-by: Claude AI
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
---
 defconfigs/kernel-testing  |   1 -
 docs/fstests.md            |  13 +--
 kconfigs/Kconfig.kdevops   |   3 -
 kconfigs/Kconfig.kernel-ci | 170 -------------------------------------
 4 files changed, 4 insertions(+), 183 deletions(-)
 delete mode 100644 kconfigs/Kconfig.kernel-ci

diff --git a/defconfigs/kernel-testing b/defconfigs/kernel-testing
index 833d1c5f..3147c0fa 100644
--- a/defconfigs/kernel-testing
+++ b/defconfigs/kernel-testing
@@ -2,5 +2,4 @@ CONFIG_KDEVOPS_TRY_REFRESH_REPOS=y
 CONFIG_KDEVOPS_TRY_UPDATE_SYSTEMS=y
 CONFIG_KDEVOPS_TRY_INSTALL_KDEV_TOOLS=y
 CONFIG_GIT_ALTERNATIVES=y
-CONFIG_KERNEL_CI=y
 CONFIG_WORKFLOW_KOTD_ENABLE=y
diff --git a/docs/fstests.md b/docs/fstests.md
index 3911004c..bc4a2d59 100644
--- a/docs/fstests.md
+++ b/docs/fstests.md
@@ -137,15 +137,10 @@ of a baseline by running fstests in a loop 100 times.  If no new failures have
 been found we consider that we have a high confidence in a baseline and then
 it can be used to verify if new development changes are not causing a regression.
 
-The amount of loops you choose to test with fstests is configurable
-with `CONFIG_KERNEL_CI_STEADY_STATE_GOAL`. We default to 100. This value
-is chosen for a few reasons:
-
-  * Running fstests 100 times takes about 1 week
-  * This is a sensible litmus test to ensure no regressions are
-    introduced with a new delta of patches, if you want to build confidence
-    in some possibly intrusive changes to Linux, or backport some patches
-    without regressing the kernel.
+The preferred method for long-term testing with fstests is to use the
+SOAK_DURATION feature, which allows specific tests designed for soak
+testing to run for extended periods. This provides a more targeted approach
+to stress testing compared to simply running all tests in a loop.
 
 The value originally comes from kdevops use at SUSE Linux enterprise
 distributions, as a litmus test to validate kernel releases, so to ensure
diff --git a/kconfigs/Kconfig.kdevops b/kconfigs/Kconfig.kdevops
index 70d4ec5d..c2362adf 100644
--- a/kconfigs/Kconfig.kdevops
+++ b/kconfigs/Kconfig.kdevops
@@ -187,9 +187,6 @@ menu "Ansible post-bring up provisioning configuration"
 source "kconfigs/Kconfig.ansible_provisioning"
 endmenu
 
-menu "Kernel continous integration configuration"
-source "kconfigs/Kconfig.kernel-ci"
-endmenu
 
 menu "Ansible Configuration"
 source "kconfigs/Kconfig.ansible_cfg"
diff --git a/kconfigs/Kconfig.kernel-ci b/kconfigs/Kconfig.kernel-ci
deleted file mode 100644
index 0ef71fc6..00000000
--- a/kconfigs/Kconfig.kernel-ci
+++ /dev/null
@@ -1,170 +0,0 @@
-config HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	bool
-	default n
-
-if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-
-config KERNEL_CI_DEFAULT_STEADY_STATE_GOAL
-	int
-	default 1
-
-endif
-
-config KERNEL_CI
-	bool "Enable kernel continous integration"
-	default y
-	help
-	  Each subsystem may have its own way to run a test for a longer
-	  period of time. Such is the case for fstests, for fstests this is
-	  the soak duration. Only a few tests leverage soak duration, that is
-	  a test must be designed to leverage soak duration to use it.
-
-	  For subsystems which don't have the same concept or in case you want
-	  to run a a full set of configured tests over and over again, you can
-	  enable the this mechanism.
-
-	  This is enabled by default so to leverage of optional subsystem
-	  specific watchdogs, which may reset a system in case of a crash. The
-	  default number of loops is just 1 so to account for this sensible
-	  default.
-
-	  Workflows support kdevops kernel-ci by implementing a Makefile target
-	  for a test with a "loop" postfix. For instance, to run the fstests
-	  kernel-ci loop you would just run:
-
-	    make fstests-baseline-loop
-
-	  Enable this option if you want to enable and configure certain
-	  kernel-ci features.
-
-if KERNEL_CI
-
-config KERNEL_CI_ENABLE_STEADY_STATE
-	bool "Bound kernel-ci tests with a steady-state goal"
-	default y
-	help
-	  Enable this so that tests don't run forever in a loop if no issues
-	  are found, instead, the kernel-ci loop will bail once a certain number
-	  of full loops have passed without failures. We refer to this goal as
-	  a kernel-ci steady-state test goal. Although the goal can be workflow
-	  specific, for now we define a generic goal which can be used by all
-	  workflows.
-
-	  If you don't enable this, and the test you are running won't ever run
-	  into a failure, you'll have the test run forever.
-
-if KERNEL_CI_ENABLE_STEADY_STATE
-
-config KERNEL_CI_STEADY_STATE_GOAL
-	int "Generic kernel-ci steady-state goal"
-	default 1 if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	default KERNEL_CI_DEFAULT_STEADY_STATE_GOAL if HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	help
-	  The maximum number of possitive successes to have before bailing out
-	  a kernel-ci loop and report success. This value is currently used for
-	  all workflows. A value of 100 means 100 tests will run before we
-	  bail out and report we have achieved steady state for the workflow
-	  being tested.
-
-config KERNEL_CI_STEADY_STATE_INCREMENTAL
-	bool "Bound kernel-ci tests with an incremental steady-state goal"
-	default y
-	help
-	  Enable this so that kernel-ci loop will bail once a certain number
-	  of full loops have passed without resetting the counter on failures.
-	  In this mode, when a test run has failures, the failed tests will be
-	  added to the local expunge lists and success counter will not be
-	  incremented, but the next time that loop will be run it will resume
-	  from the last success counter and stop when it reaches the kernel-ci
-	  steady-state test goal.
-
-endif # KERNEL_CI_ENABLE_STEADY_STATE
-
-config KERNEL_CI_EMAIL_REPORT
-	bool "Enable sending a report on failure"
-	default n if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	default KERNEL_CI_DEFAULT_ENABLE_EMAIL_REPORT if HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	help
-	  Enable this so that a report will be emailed in case of a failure.
-
-if KERNEL_CI_EMAIL_REPORT
-
-config KERNEL_CI_ADD_CUSTOM_SUBJECT_TOPIC
-	bool "Enable a custom kernel-ci subject topic for email reports"
-	default y if BOOTLINUX
-	help
-	  Enable this so that you can customize the subject slightly for
-	  kernel-ci email reports. The email subject prefix will always
-	  start with "kernel-ci: ", if this is enabled we allow you to
-	  slightly configure this. Otherwise the standard "kernel-ci: "
-	  subject prefix will be used.
-
-	  You want to enable this if you are using kernel-ci for different
-	  things, such as testing a distribution kernel but also a stable
-	  or linux-next kernel, so that you can distinguish for which kernel
-	  the report is for.
-
-if KERNEL_CI_ADD_CUSTOM_SUBJECT_TOPIC
-
-config KERNEL_CI_SUBJECT_TOPIC
-	string "Main email subject topic prefix to use"
-	default "" if !BOOTLINUX
-	default BOOTLINUX_TREE_NAME if BOOTLINUX
-	help
-	  Email kernel-ci subject will include the git tree name you are using.
-	  This is set in the BOOTLINUX_TREE_NAME kconfig variable. For instance,
-	  if you decided to build and boot into a linux stable tree the
-	  subject for the kernel-ci email will start with:
-
-	    kernel-ci linux-stable:
-
-config KERNEL_CI_ADD_CUSTOM_SUBJECT_TOPIC_TAG
-	bool "Add git tag to kernel-ci subject topic"
-	default y if BOOTLINUX
-	depends on BOOTLINUX
-	help
-	  Enable this so that the kernel-ci subject topic will also have
-	  the git tag used. For intance, if you have enabled to build and
-	  boot  into linux-next tag next-20210126, and you also enabled this
-	  option, the KERNEL_CI_SUBJECT_TOPIC will be set to "linux-next" and
-	  the subject for the kernel-ci email will start with:
-
-	    kernel-ci linux-next next-20210126:
-
-	  If you do not enable this, then the subject for the kernel-ci email
-	  will start only with:
-
-	    kernel-ci linux-next:
-
-endif # KERNEL_CI_ADD_CUSTOM_SUBJECT_TOPIC
-
-config KERNEL_CI_EMAIL_MODIFY_FROM
-	bool "Modify default from address"
-	select KDEVOPS_SSH_CONFIG_UPDATE_STRICT
-	default n if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	default KERNEL_CI_DEFAULT_EMAIL_MODIFY_FROM if HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	help
-	  If you would like to modify the default from address used to send
-	  the report enable this.
-
-config KERNEL_CI_EMAIL_FROM
-	string "Who to send the email from"
-	depends on KERNEL_CI_EMAIL_MODIFY_FROM
-	default $(shell, echo $USER) if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	default KERNEL_CI_DEFAULT_EMAIL_FROM if HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	  help
-	  The email to pass on to mailx -S from="" setting. This will be used
-	  to modify the from address of the email report sent.
-
-config KERNEL_CI_EMAIL_RCPT
-	string "Who to send email report to"
-	default $(shell, echo $USER) if !HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	default KERNEL_CI_DEFAULT_EMAIL_RCPT if HAVE_KERNEL_CI_CUSTOM_DEFAULTS
-	  help
-	  Comma separated list of users to send the kernel-ci loop email
-	  report to. These are the recipients of the kernel-ci test loop
-	  report.
-
-endif # KERNEL_CI_EMAIL_REPORT
-
-endif # KERNEL_CI
-- 
2.47.2


  parent reply	other threads:[~2025-08-11 22:24 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-11 22:24 [PATCH 00/23] remove old kernel-ci and enhance reboot-limit Luis Chamberlain
2025-08-11 22:24 ` [PATCH 01/23] fstests: remove CONFIG_KERNEL_CI support Luis Chamberlain
2025-08-11 22:24 ` [PATCH 02/23] fstests: remove kernel-ci script symlinks Luis Chamberlain
2025-08-11 22:24 ` [PATCH 03/23] blktests: remove CONFIG_KERNEL_CI support Luis Chamberlain
2025-08-11 22:24 ` [PATCH 04/23] gitr: " Luis Chamberlain
2025-08-11 22:24 ` [PATCH 05/23] ltp: " Luis Chamberlain
2025-08-11 22:24 ` [PATCH 06/23] nfstest: " Luis Chamberlain
2025-08-11 22:24 ` [PATCH 07/23] pynfs: " Luis Chamberlain
2025-08-11 22:24 ` [PATCH 08/23] reboot-limit: convert CONFIG_KERNEL_CI to internal loop feature Luis Chamberlain
2025-08-11 22:24 ` Luis Chamberlain [this message]
2025-08-11 22:24 ` [PATCH 10/23] scripts: remove kernel-ci loop infrastructure Luis Chamberlain
2025-08-11 22:24 ` [PATCH 11/23] reboot-limit: simplify what gets selected Luis Chamberlain
2025-08-11 22:24 ` [PATCH 12/23] reboot-limit: add graph visualization support for results Luis Chamberlain
2025-08-11 22:24 ` [PATCH 13/23] reboot-limit: save graphs in organized results/graphs directory Luis Chamberlain
2025-08-11 22:24 ` [PATCH 14/23] docs: add comprehensive reboot-limit workflow documentation Luis Chamberlain
2025-08-11 22:24 ` [PATCH 15/23] reboot-limit: add kexec-tools dependency installation Luis Chamberlain
2025-08-11 22:24 ` [PATCH 16/23] reboot-limit: add A/B testing support targets Luis Chamberlain
2025-08-11 22:24 ` [PATCH 17/23] reboot-limit: fix kexec and reboot connection handling Luis Chamberlain
2025-08-11 22:24 ` [PATCH 18/23] reboot-limit: add COUNT parameter to override reboot count Luis Chamberlain
2025-08-11 22:24 ` [PATCH 19/23] reboot-limit: fix wait_for tasks using wrong host reference Luis Chamberlain
2025-08-11 22:24 ` [PATCH 20/23] reboot-limit: use ansible reboot module for all reboot types Luis Chamberlain
2025-08-11 22:24 ` [PATCH 21/23] reboot-limit: fix COUNT parameter to properly override reboot count Luis Chamberlain
2025-08-11 22:24 ` [PATCH 22/23] reboot-limit: handle empty dev group gracefully Luis Chamberlain
2025-08-11 22:24 ` [PATCH 23/23] reboot-limit: add kexec comparison feature Luis Chamberlain
2025-08-12 15:06 ` [PATCH 00/23] remove old kernel-ci and enhance reboot-limit Chuck Lever
2025-08-13  1:28   ` Luis Chamberlain

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250811222452.2213071-10-mcgrof@kernel.org \
    --to=mcgrof@kernel.org \
    --cc=cel@kernel.org \
    --cc=da.gomez@kruces.com \
    --cc=kdevops@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox