Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: parse uds_bundle_metadata resource #1059

Merged
merged 2 commits into from
Jan 22, 2025
Merged

feat: parse uds_bundle_metadata resource #1059

merged 2 commits into from
Jan 22, 2025

Conversation

catsby
Copy link
Collaborator

@catsby catsby commented Jan 21, 2025

Description

This PR adds to the tfparser to extract the bundle metadata from the uds_bundle_metadata resource, defined like so:

resource "uds_bundle_metadata" "basic" {
  kind        = "UDSBundle"
  version     = "0.1.0"
  description = "This is a basic bundle"
}

NOTE: also fix an issue where the parser would panic if something it thought was required was missing (ex version in the required_providers block)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

Signed-off-by: catsby <clint@defenseunicorns.com>
@catsby catsby requested a review from YrrepNoj January 21, 2025 19:41
@catsby catsby requested a review from a team as a code owner January 21, 2025 19:41
Signed-off-by: catsby <clint@defenseunicorns.com>
Copy link
Member

@YrrepNoj YrrepNoj left a comment

Choose a reason for hiding this comment

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

LGTM

@catsby catsby merged commit d74ef0d into uds-tofu Jan 22, 2025
16 checks passed
@catsby catsby deleted the tfparse-metadata branch January 22, 2025 15:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants