From: Chuck Lever <cel@kernel.org>
To: <kdevops@lists.linux.dev>
Cc: Chuck Lever <chuck.lever@oracle.com>,
Luis Chamberlain <mcgrof@kernel.org>
Subject: [PATCH v3 05/36] terraform/azure: Make the resource group name a variable
Date: Thu, 4 Sep 2025 11:14:53 -0400 [thread overview]
Message-ID: <20250904151526.1596282-6-cel@kernel.org> (raw)
In-Reply-To: <20250904151526.1596282-1-cel@kernel.org>
From: Chuck Lever <chuck.lever@oracle.com>
Resource groups appear to be global. To run more than one instance
of kdevops in Azure at the same time, each instance needs a unique
resource group. Hoist the resource group name into Kconfig so that
it can be set to a unique value.
Reviewed-by: Luis Chamberlain <mcgrof@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
---
.../templates/azure/terraform.tfvars.j2 | 1 +
terraform/azure/kconfigs/Kconfig.location | 16 ++++++++++++++++
terraform/azure/main.tf | 2 +-
terraform/azure/vars.tf | 5 +++++
4 files changed, 23 insertions(+), 1 deletion(-)
diff --git a/playbooks/roles/gen_tfvars/templates/azure/terraform.tfvars.j2 b/playbooks/roles/gen_tfvars/templates/azure/terraform.tfvars.j2
index 8be48e800036..7ce0f6170e22 100644
--- a/playbooks/roles/gen_tfvars/templates/azure/terraform.tfvars.j2
+++ b/playbooks/roles/gen_tfvars/templates/azure/terraform.tfvars.j2
@@ -1,4 +1,5 @@
azure_subscription_id = "{{ terraform_azure_subscription_id }}"
+azure_resource_group_name = "{{ terraform_azure_resource_group_name }}"
azure_location = "{{ terraform_azure_location }}"
azure_vmsize = "{{ terraform_azure_vm_size }}"
azure_image_publisher = "{{ terraform_azure_image_publisher }}"
diff --git a/terraform/azure/kconfigs/Kconfig.location b/terraform/azure/kconfigs/Kconfig.location
index a7e8bae37507..2cc0e419d54d 100644
--- a/terraform/azure/kconfigs/Kconfig.location
+++ b/terraform/azure/kconfigs/Kconfig.location
@@ -134,3 +134,19 @@ config TERRAFORM_AZURE_LOCATION
default "westus2" if TERRAFORM_AZURE_REGION_WESTUS2
default "westus3" if TERRAFORM_AZURE_REGION_WESTUS3
default "westcentralus" if TERRAFORM_AZURE_REGION_WESTCENTRALUS
+
+config TERRAFORM_AZURE_RESOURCE_GROUP_NAME
+ string "Azure resource group name"
+ output yaml
+ default "kdevops_resource_group"
+ help
+ An Azure resource group is a container that holds related
+ resources so they can be managed as a single unit. These
+ resources share the same life cycle and are deployed,
+ updated, and deleted together. Resource groups are global
+ to your subscription.
+
+ To run concurrent kdevops jobs in Azure, each run must
+ have a unique resouce group name.
+
+ https://learn.microsoft.com/en-us/azure/azure-resource-manager/management/overview#resource-groups
diff --git a/terraform/azure/main.tf b/terraform/azure/main.tf
index c178c34d0180..8dcead78b5fd 100644
--- a/terraform/azure/main.tf
+++ b/terraform/azure/main.tf
@@ -1,7 +1,7 @@
# Azure terraform provider main
resource "azurerm_resource_group" "kdevops_group" {
- name = "kdevops_resource_group"
+ name = var.azure_resource_group_name
location = var.azure_location
}
diff --git a/terraform/azure/vars.tf b/terraform/azure/vars.tf
index 2d06857905e2..dd3c20ed05c7 100644
--- a/terraform/azure/vars.tf
+++ b/terraform/azure/vars.tf
@@ -33,6 +33,11 @@ variable "azure_managed_disks_tier" {
type = string
}
+variable "azure_resource_group_name" {
+ description = "Name of the set of resources for this run"
+ type = string
+}
+
variable "azure_subscription_id" {
description = "Your Azure subscription ID"
type = string
--
2.51.0
next prev parent reply other threads:[~2025-09-04 15:15 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-04 15:14 [PATCH v3 00/36] Various fixes Chuck Lever
2025-09-04 15:14 ` [PATCH v3 01/36] scripts: Update gen_ssh_key.sh Chuck Lever
2025-09-04 15:14 ` [PATCH v3 02/36] terraform/azure: Fix count of managed disks Chuck Lever
2025-09-04 15:14 ` [PATCH v3 03/36] terraform/azure: Remove tags from terraform configuration Chuck Lever
2025-09-04 15:14 ` [PATCH v3 04/36] terraform/azure: Enable network acceleration Chuck Lever
2025-09-04 15:14 ` Chuck Lever [this message]
2025-09-04 15:14 ` [PATCH v3 06/36] terraform/azure: Remove the provider version constraint Chuck Lever
2025-09-04 15:14 ` [PATCH v3 07/36] terraform/azure: Update the RHEL publishers Chuck Lever
2025-09-04 15:14 ` [PATCH v3 08/36] terraform/oci: Work around oracle/oci provider bug Chuck Lever
2025-09-04 15:14 ` [PATCH v3 09/36] terraform/oci: Add Kconfig choices for disk performance provisioning Chuck Lever
2025-09-04 15:14 ` [PATCH v3 10/36] terraform/oci: Enable preemptible compute instances Chuck Lever
2025-09-04 15:14 ` [PATCH v3 11/36] terraform/aws: Fix zone selection for us_west1 Chuck Lever
2025-09-04 15:15 ` [PATCH v3 12/36] Revert "terraform/aws: Add Oracle Linux AMI selections" Chuck Lever
2025-09-04 15:15 ` [PATCH v3 13/36] terraform/aws: Remove unused terraform tags Chuck Lever
2025-09-04 15:15 ` [PATCH v3 14/36] terraform/GCE: Fix the default disk throughput values Chuck Lever
2025-09-04 15:15 ` [PATCH v3 15/36] terraform/gce: Remove unused tags Chuck Lever
2025-09-04 15:15 ` [PATCH v3 16/36] terraform/gce: Remove support for Fedora Chuck Lever
2025-09-04 15:15 ` [PATCH v3 17/36] terraform/gce: Update existing machine types and introduce new ones Chuck Lever
2025-09-04 15:15 ` [PATCH v3 18/36] terraform: Create the user's .ssh/config if it does not already exist Chuck Lever
2025-09-04 15:15 ` [PATCH v3 19/36] pynfs: Don't install xdrlib3 on older RHEL Chuck Lever
2025-09-04 15:15 ` [PATCH v3 20/36] guestfs: Update the guestfs playbook description Chuck Lever
2025-09-04 15:15 ` [PATCH v3 21/36] gen_hosts: Add 'localhost' to the [all] group Chuck Lever
2025-09-04 15:15 ` [PATCH v3 22/36] guestfs: Don't run the 'destroy' tags on the controller Chuck Lever
2025-09-04 15:15 ` [PATCH v3 23/36] base_image: Skip "locales-all" package installation Chuck Lever
2025-09-04 15:15 ` [PATCH v3 24/36] update_etc_hosts: ipaddr Chuck Lever
2025-09-04 15:15 ` [PATCH v3 25/36] Add an Ansible requirements.yml file Chuck Lever
2025-09-04 15:15 ` [PATCH v3 26/36] terraform/oci: Clean up the route table resource Chuck Lever
2025-09-04 15:15 ` [PATCH v3 27/36] guestfs: Correct the spelling of "drives" Chuck Lever
2025-09-04 15:15 ` [PATCH v3 28/36] guestfs: Configure how many extra drives to provision Chuck Lever
2025-09-04 15:15 ` [PATCH v3 29/36] ltp: Replace /opt with a symlink to data partition Chuck Lever
2025-09-04 15:15 ` [PATCH v3 30/36] terraform: Squelch Ansible complaints about missing inventory file Chuck Lever
2025-09-04 15:15 ` [PATCH v3 31/36] terraform: Skip localhost when waiting for targets to reboot Chuck Lever
2025-09-04 15:15 ` [PATCH v3 32/36] devconfig: include the "service" group in the playbooks hosts list Chuck Lever
2025-09-05 10:25 ` Daniel Gomez
2025-09-04 15:15 ` [PATCH v3 33/36] Makefile: Remove CONFIG_TOPDIR_PATH_HAS_SHA256SUM Chuck Lever
2025-09-04 15:15 ` [PATCH v3 34/36] Makefile: Set TOPDIR_PATH and generate its sha256sum at run time Chuck Lever
2025-09-04 15:15 ` [PATCH v3 35/36] ansible.cfg: generate an ansible.cfg file in TOPDIR Chuck Lever
2025-09-04 15:15 ` [PATCH v3 36/36] inventory: generate the inventory pathname dynamically Chuck Lever
2025-09-05 13:21 ` Daniel Gomez
2025-09-04 17:16 ` [PATCH v3 00/36] Various fixes Luis Chamberlain
2025-09-05 13:06 ` Daniel Gomez
2025-09-05 16:37 ` Chuck Lever
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=20250904151526.1596282-6-cel@kernel.org \
--to=cel@kernel.org \
--cc=chuck.lever@oracle.com \
--cc=kdevops@lists.linux.dev \
--cc=mcgrof@kernel.org \
/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