public inbox for kdevops@lists.linux.dev
 help / color / mirror / Atom feed
From: Luis Chamberlain <mcgrof@kernel.org>
To: kdevops@lists.linux.dev, chuck.lever@oracle.com,
	da.gomez@samsung.com, jlayton@kernel.org
Cc: mcgrof@kernel.org
Subject: [PATCH 2/3] kdevops: remove duplicate symbols and use scripts/append-makefile-vars.sh
Date: Wed,  4 Sep 2024 16:30:36 -0700	[thread overview]
Message-ID: <20240904233037.1516218-3-mcgrof@kernel.org> (raw)
In-Reply-To: <20240904233037.1516218-1-mcgrof@kernel.org>

A few kdevops kconfig duplicate symbols exist, remove them so to ensure
we can work with the latest linux-next version of kconfig. Additionally
we need to ensure to use scripts/append-makefile-vars.sh for when we
want to work with Makefile inherited variables. The KDEVOPS_PLAYBOOK_DIR
can also just be simplified, while we do provide for variability in
practice we never use this so just this a silent default for now.

Without this we end up in a hung state if we run 'make menuconfig'
with the latest kconfig. Fix this.

Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
---
 kconfigs/Kconfig.ansible_provisioning | 24 ++++++------------------
 terraform/aws/Kconfig                 | 14 --------------
 vagrant/Kconfig                       |  3 ++-
 vagrant/Kconfig.opensuse              | 22 ----------------------
 4 files changed, 8 insertions(+), 55 deletions(-)

diff --git a/kconfigs/Kconfig.ansible_provisioning b/kconfigs/Kconfig.ansible_provisioning
index 82a6f822c3f5..63f04306e7b2 100644
--- a/kconfigs/Kconfig.ansible_provisioning
+++ b/kconfigs/Kconfig.ansible_provisioning
@@ -1,12 +1,6 @@
-if !KDEVOPS_ANSIBLE_PROVISION_ENABLE
-
 config KDEVOPS_PLAYBOOK_DIR
 	string
-	default $(shell, echo $KDEVOPS_PLAYBOOKS_DIR)
-	help
-	  The directory where ansible playbooks are found.
-
-endif # KDEVOPS_ANSIBLE_PROVISION_ENABLE
+	default $(shell, scripts/append-makefile-vars.sh $(KDEVOPS_PLAYBOOKS_DIR))
 
 config HAVE_DISTRO_PREFERS_DISABLE_ANSIBLE
 	bool
@@ -41,7 +35,7 @@ config KDEVOPS_DEVCONFIG_ENABLE_CONSOLE
 	bool "Enable console setup"
 	default y
 	depends on KDEVOPS_DEVCONFIG_ENABLE
-	  help
+	help
 	  Enable this so that we configure and enable console serial settings
           for you on /etc/default/grub.
 
@@ -50,7 +44,7 @@ if KDEVOPS_DEVCONFIG_ENABLE_CONSOLE
 config KDEVOPS_DEVCONFIG_KERNEL_CONSOLE_SETTINGS
 	string "Kernel command line console kernel parameters to use"
 	default "console=tty0 console=tty1 console=ttyS0,115200n8"
-	  help
+	help
           Linux kernel console kernel parameters to use. This will be appended
 	  to the GRUB configuration file /etc/default/grub on the variable
           GRUB_CMDLINE_LINUX_DEFAULT, if and only if you did not have any
@@ -59,7 +53,7 @@ config KDEVOPS_DEVCONFIG_KERNEL_CONSOLE_SETTINGS
 config KDEVOPS_DEVCONFIG_GRUB_SERIAL_COMMAND
 	string "GRUB specific serial settings"
 	default "serial --speed=115200 --unit=0 --parity=no --stop=1"
-	  help
+	help
           GRUB console settings to use. This will be appended to the GRUB
           configuration file /etc/default/grub on the variable
           GRUB_SERIAL_COMMAND, if and only if we did change the your console
@@ -68,7 +62,7 @@ config KDEVOPS_DEVCONFIG_GRUB_SERIAL_COMMAND
 config KDEVOPS_GRUB_TIMEOUT
 	int "GRUB timeout"
 	default 2
-	  help
+	help
           GRUB timeout to wait for user interaction to let you pick another
           kernel or boot options.
 
@@ -104,15 +98,9 @@ config KDEVOPS_DEVCONFIG_SYSTEMD_WATCHDOG_TIMEOUT_KEXEC
 
 endif # KDEVOPS_DEVCONFIG_ENABLE_SYSTEMD_WATCHDOG
 
-config KDEVOPS_PLAYBOOK_DIR
-	string "The directory for the ansible playbooks to use"
-	default $(shell, echo $KDEVOPS_PLAYBOOKS_DIR)
-	help
-	  The directory where ansible playbooks are found.
-
 config KDEVOPS_ANSIBLE_INVENTORY_FILE
 	string "The ansible inventory file to use"
-	default $(shell, echo $KDEVOPS_HOSTFILE)
+	default $(shell, scripts/append-makefile-vars.sh $(KDEVOPS_HOSTFILE))
 	help
 	  The file to use for the ansible inventory.
 
diff --git a/terraform/aws/Kconfig b/terraform/aws/Kconfig
index dafacaeada15..a94045afcfee 100644
--- a/terraform/aws/Kconfig
+++ b/terraform/aws/Kconfig
@@ -280,13 +280,6 @@ config TERRAFORM_AWS_INSTANCE_IS4GEN_MEDIUM
 	  6 GiB RAM, 1vCPU, 1 x 937 GiB NVMe SSD, 25 Gbps Net, 9.6 Gbps EBS.
 	  https://aws.amazon.com/ec2/instance-types/i4g/
 
-config TERRAFORM_AWS_INSTANCE_IS4GEN_8XLARGE
-	bool "is4gen.8xlarge"
-	depends on TARGET_ARCH_ARM64
-	help
-	  192 GiB RAM, 32 vCPUs, 4 x 7500 GiB NVMe SSD, 50 Gbps Net, 19 Gbps EBS.
-	  https://aws.amazon.com/ec2/instance-types/i4g/
-
 config TERRAFORM_AWS_INSTANCE_IS4GEN_8XLARGE
 	bool "is4gen.8xlarge"
 	depends on TARGET_ARCH_ARM64
@@ -308,13 +301,6 @@ config TERRAFORM_AWS_INSTANCE_IM4GN_4XLARGE
 	  64 GiB RAM, 16 vcpus, 1 x 7500 NVMe SSD, 25 Gbps Net, 9.5 Gbps EBS.
 	  https://aws.amazon.com/ec2/instance-types/i4g/
 
-config TERRAFORM_AWS_INSTANCE_IM4GN_4XLARGE
-	bool "im4gn.4xlarge"
-	depends on TARGET_ARCH_ARM64
-	help
-	  64 GiB RAM, 16 vcpus, 1 x 7500 NVMe SSD, 25 Gbps Net, 9.5 Gbps EBS.
-	  https://aws.amazon.com/ec2/instance-types/i4g/
-
 config TERRAFORM_AWS_INSTANCE_I4I_LARGE
 	bool "i4i.large"
 	depends on TARGET_ARCH_X86_64
diff --git a/vagrant/Kconfig b/vagrant/Kconfig
index 93480274885e..ecf7bcafe1ad 100644
--- a/vagrant/Kconfig
+++ b/vagrant/Kconfig
@@ -1,4 +1,5 @@
 if VAGRANT
+
 choice
 	prompt "Vagrant virtualization technology to use"
 	default VAGRANT_LIBVIRT_SELECT
@@ -89,7 +90,7 @@ config VIRTUALBOX_EXTRA_DRIVE_FORMAT
 	default "vmdk" if VIRTUALBOX_EXTRA_DRIVE_VMDK
 	default "vhd" if VIRTUALBOX_EXTRA_DRIVE_VHD
 
-endif
+endif # VAGRANT_VIRTUALBOX
 
 config HAVE_SUSE_VAGRANT
 	bool
diff --git a/vagrant/Kconfig.opensuse b/vagrant/Kconfig.opensuse
index 2ffb1f31c02b..6da402755c26 100644
--- a/vagrant/Kconfig.opensuse
+++ b/vagrant/Kconfig.opensuse
@@ -56,28 +56,6 @@ config VAGRANT_OPENSUSE_X86_64_LEAP_15_3
 	help
 	  This enables the OpenSUSE Leap 15.3 release.
 
-config VAGRANT_OPENSUSE_X86_64_LEAP_15_4
-	bool "Leap-15.4.x86_64"
-	depends on TARGET_ARCH_X86_64
-	select HAVE_VAGRANT_BOX_URL
-	select HAVE_DISTRO_BTRFS_DISABLES_RAID56  if FSTESTS_BTRFS
-	select HAVE_DISTRO_XFS_SUPPORTS_CRC if FSTESTS_XFS
-	select HAVE_DISTRO_XFS_SUPPORTS_EXTERNAL_LOG if FSTESTS_XFS
-	select HAVE_DISTRO_XFS_SUPPORTS_REFLINKS if FSTESTS_XFS
-	select HAVE_DISTRO_XFS_SUPPORTS_BIGBLOCKS if FSTESTS_XFS
-	select HAVE_DISTRO_PREFERS_FSTESTS_WATCHDOG
-	select HAVE_DISTRO_PREFERS_FSTESTS_WATCHDOG_KILL
-	select HAVE_DISTRO_BLKTESTS_TEST_BLOCK if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
-	select HAVE_DISTRO_BLKTESTS_TEST_LOOP if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
-	select HAVE_DISTRO_BLKTESTS_TEST_NBD if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
-	select HAVE_DISTRO_PREFERS_BLKTESTS_WATCHDOG if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
-	select HAVE_DISTRO_PREFERS_BLKTESTS_WATCHDOG_KILL if KDEVOPS_WORKFLOW_ENABLE_BLKTESTS
-	select HAVE_DISTRO_PREFERS_REFRESH
-	select HAVE_DISTRO_PREFERS_UPDATE
-	select HAVE_DISTRO_PREFERS_INSTALL_KDEVTOOLS
-	help
-	  This enables the OpenSUSE Leap 15.4 release.
-
 endchoice
 
 if VAGRANT_OPENSUSE_X86_64_TW
-- 
2.43.0


  parent reply	other threads:[~2024-09-04 23:30 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-04 23:30 [PATCH 0/3] kdevops: update to latest kconfig Luis Chamberlain
2024-09-04 23:30 ` [PATCH 1/3] scripts/kconfig/kconfig.Makefile: move _refs-default Luis Chamberlain
2024-09-04 23:30 ` Luis Chamberlain [this message]
2024-09-04 23:30 ` [PATCH 3/3] scripts/gen-refs-default.Makefile: support direct make requests 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=20240904233037.1516218-3-mcgrof@kernel.org \
    --to=mcgrof@kernel.org \
    --cc=chuck.lever@oracle.com \
    --cc=da.gomez@samsung.com \
    --cc=jlayton@kernel.org \
    --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