Skip to content

Conversation

szachovy
Copy link
Member

@szachovy szachovy commented Sep 5, 2025

Description

Part of https://github.com/SUSE/spacewalk/issues/24541.
Please refer to the Automatic testing flow to gain initial understanding of this PR purpose.

@szachovy szachovy marked this pull request as draft September 8, 2025 08:51
@szachovy
Copy link
Member Author

szachovy commented Sep 10, 2025

In the sumaform validation the following error is present:

│ Error: Unsupported argument
│ 
│   on main.tf line 115, in module "cucumber_testsuite":
│  115:   cc_ptf_username = var.SCC_PTF_USER
│ 
│ An argument named "cc_ptf_username" is not expected here.
╵
╷
│ Error: Unsupported argument
│ 
│   on main.tf line 116, in module "cucumber_testsuite":
│  116:   cc_ptf_password = var.SCC_PTF_PASSWORD
│ 
│ An argument named "cc_ptf_password" is not expected here.

Issue is caused because the PR from sumaform has not yet been merged (see: uyuni-project/sumaform#1905)
If I reference my branch and even provide empty variable values, I still get an error from other undefined values:

Terraform has been successfully initialized!

You may now begin working with Terraform. Try running "terraform plan" to see
any changes that are required for your infrastructure. All Terraform commands
should now work.

If you ever set or change modules or backend configuration for Terraform,
rerun this command to reinitialize your working directory. If you forget, other
commands will detect it and remind you to do so if necessary.
Success! The configuration is valid.

│ Error: No value for required variable
│ 
│   on main.tf line 48:
│   48: variable "ENVIRONMENT" {
│ 
│ The root module input variable "ENVIRONMENT" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 139:
│  139: variable "IMAGE" {
│ 
│ The root module input variable "IMAGE" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 143:
│  143: variable "GIT_PROFILES_REPO" {
│ 
│ The root module input variable "GIT_PROFILES_REPO" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 147:
│  147: variable "IMAGES" {
│ 
│ The root module input variable "IMAGES" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 151:
│  151: variable "SERVER_IMAGE" {
│ 
│ The root module input variable "SERVER_IMAGE" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 155:
│  155: variable "PROXY_IMAGE" {
│ 
│ The root module input variable "PROXY_IMAGE" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 159:
│  159: variable "SUSE_MINION_IMAGE" {
│ 
│ The root module input variable "SUSE_MINION_IMAGE" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 163:
│  163: variable "RHLIKE_MINION_IMAGE" {
│ 
│ The root module input variable "RHLIKE_MINION_IMAGE" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 167:
│  167: variable "DEBLIKE_MINION_IMAGE" {
│ 
│ The root module input variable "DEBLIKE_MINION_IMAGE" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 171:
│  171: variable "PRODUCT_VERSION" {
│ 
│ The root module input variable "PRODUCT_VERSION" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 180:
│  180: variable "USE_MIRROR_IMAGES" {
│ 
│ The root module input variable "USE_MIRROR_IMAGES" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 185:
│  185: variable "DOMAIN" {
│ 
│ The root module input variable "DOMAIN" is not set, and has no default
│ value. Use a -var or -var-file command line argument to provide a value for
│ this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 189:
│  189: variable "ADDITIONAL_REPOS_ONLY" {
│ 
│ The root module input variable "ADDITIONAL_REPOS_ONLY" is not set, and has
│ no default value. Use a -var or -var-file command line argument to provide
│ a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 193:
│  193: variable "ENVIRONMENT_CONFIGURATION" {
│ 
│ The root module input variable "ENVIRONMENT_CONFIGURATION" is not set, and
│ has no default value. Use a -var or -var-file command line argument to
│ provide a value for this variable.
╵
╷
│ Error: No value for required variable
│ 
│   on main.tf line 198:
│  198: variable "DOWNLOAD_ENDPOINT" {
│ 
│ The root module input variable "DOWNLOAD_ENDPOINT" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
Error: Terraform exited with code 1.
Error: Process completed with exit code 1.

I wonder whether this should be ignored, since the deployment is split across multiple repositories. Let me know in the comments.

@szachovy szachovy marked this pull request as ready for review September 11, 2025 09:55
@szachovy szachovy self-assigned this Sep 11, 2025
@Bischoff Bischoff self-requested a review September 11, 2025 10:18
Bischoff

This comment was marked as outdated.

Copy link
Contributor

@Bischoff Bischoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See remarks

@Bischoff Bischoff self-requested a review October 14, 2025 12:55
Copy link
Contributor

@Bischoff Bischoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants