Skip to content

Conversation

@jamengual
Copy link
Contributor

This pull request introduces a small conditional improvement to the workflow configuration. The change ensures that the Atmos installation step only runs when an Atmos version is explicitly provided.

Make atmos install conditional
@github-actions
Copy link

github-actions bot commented Nov 14, 2025

Changes Found for foobar/changes in plat-ue2-sandbox


create

Plan: 1 to add, 0 to change, 0 to destroy.
To reproduce this locally, run:

atmos terraform plan foobar/changes -s plat-ue2-sandbox

Create

+ random_id.foo[0]
Terraform Plan Summary
  # random_id.foo[0] will be created
  + resource "random_id" "foo" {
      + b64_std     = (known after apply)
      + b64_url     = (known after apply)
      + byte_length = 8
      + dec         = (known after apply)
      + hex         = (known after apply)
      + id          = (known after apply)
      + keepers     = {
          + "seed" = "foo-plat-ue2-sandbox-blue"
        }
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Warning: Test warning summary

  with data.validation_warning.warn[0],
  on main.tf line 20, in data "validation_warning" "warn":
  20: data "validation_warning" "warn" {

Test warning details

Warning

Warning: Test warning summary

  with data.validation_warning.warn[0],
  on main.tf line 20, in data "validation_warning" "warn":
  20: data "validation_warning" "warn" {

Test warning details

Removed COMPONENT_CACHE_PATH variable assignment and its usage in output.
The test was expecting an outdated error message format with trailing
spaces. Updated to match the current error output format which includes:

- Proper markdown formatting with **Error:** prefix
- Clean line breaks without trailing spaces

This fixes the test-failed-plan-drift.yml test failure.
@goruha goruha merged commit 9e3bc38 into main Nov 21, 2025
59 of 61 checks passed
@goruha goruha deleted the atmos-install-conditional branch November 21, 2025 14:16
@github-actions
Copy link

These changes were released in v5.5.0.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants