Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add examples and tests for !include Atmos YAML function #1080

Merged
merged 14 commits into from
Feb 20, 2025
Merged

Add examples and tests for !include Atmos YAML function #1080

merged 14 commits into from
Feb 20, 2025

Conversation

aknysh
Copy link
Member

@aknysh aknysh commented Feb 20, 2025

what

  • Add examples and tests for !include Atmos YAML function
  • Update atmos terraform commands that require processing of Go templates and Atmos YAML functions

why

  • Test Atmos YAML function !include and show usage examples

  • The following atmos terraform commands require processing of Go templates and Atmos YAML functions in Atmos manifests:

init
plan
apply
deploy
destroy
generate
output
clean
shell
write
force-unlock
import
refresh
show
taint
untaint
validate
state list
state mv
state pull
state push
state replace-provider
state rm
state show

@aknysh aknysh added the minor New features that do not break anything label Feb 20, 2025
@aknysh aknysh requested a review from osterman February 20, 2025 02:39
@aknysh aknysh self-assigned this Feb 20, 2025
@aknysh aknysh requested review from a team as code owners February 20, 2025 02:39
coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 20, 2025
View reviewed changes
@aknysh @osterman
Update tests/test-cases/atmos-include-yaml-function.yaml
2c4163a 
Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>
coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 20, 2025
View reviewed changes
@aknysh aknysh requested a review from osterman February 20, 2025 02:50
@aknysh
updates
4ba680a
@aknysh
Merge remote-tracking branch 'origin/update-include-yaml-func' into u…
864d2af 
…pdate-include-yaml-func

# Conflicts:
#	tests/test-cases/atmos-include-yaml-function.yaml
osterman
osterman requested changes Feb 20, 2025
View reviewed changes
Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

See comments

@aknysh @osterman
Update tests/test-cases/atmos-include-yaml-function.yaml
9258c4f 
Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <erik@cloudposse.com>
coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 20, 2025
View reviewed changes
@aknysh aknysh requested a review from osterman February 20, 2025 16:55
@aknysh aknysh merged commit 056746c into main Feb 20, 2025
46 checks passed
@aknysh aknysh deleted the update-include-yaml-func branch February 20, 2025 19:48
@github-actions GitHub Actions
Copy link

These changes were released in v1.163.0.

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

@osterman osterman osterman approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

@aknysh aknysh

Labels
minor New features that do not break anything
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aknysh @osterman