Skip to content

Conversation

@jpogran-hashi
Copy link
Collaborator

@jpogran-hashi jpogran-hashi commented Aug 1, 2025

Related Issue

Fixes #

Description

This provides a structured way to handle component addresses
in the Terraform registry, enabling validation and error
reporting. The new types and functions facilitate the
installation workflow for components.

  • Added Component and ComponentPackage structs to represent registry
    components and their addresses.
  • Implemented ParseComponentSource function to parse and validate
    component registry addresses.
  • Created unit tests for ParseComponentSource to ensure correct parsing
    and error handling.

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

This provides a structured way to handle component addresses
in the Terraform registry, enabling validation and error
reporting. The new types and functions facilitate the
installation workflow for components.

- Added Component and ComponentPackage structs to represent registry
  components and their addresses.
- Implemented ParseComponentSource function to parse and validate
  component registry addresses.
- Created unit tests for ParseComponentSource to ensure correct parsing
  and error handling.
@hashicorp-cla-app
Copy link

hashicorp-cla-app bot commented Aug 1, 2025

CLA assistant check
All committers have signed the CLA.

@hashicorp-cla-app
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@jpogran-hashi jpogran-hashi self-assigned this Aug 4, 2025
mjyocca
mjyocca previously approved these changes Aug 4, 2025
@jpogran-hashi jpogran-hashi marked this pull request as ready for review August 4, 2025 18:35
@jpogran-hashi jpogran-hashi requested review from a team as code owners August 4, 2025 18:35
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

If I was to put a hat of a public consumer with no further/insider context I'd be puzzled about what is "component" and how it fits within the current model of modules and providers.

On that note - shall we add some brief explanation with examples to the Readme.md?

@jpogran-hashi
Copy link
Collaborator Author

On that note - shall we add some brief explanation with examples to the Readme.md?

@radeksimko I've updated the readme inline with existing content for modules. Does this suffice?

Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

This is perfect, thanks for the added docs @jpogran-hashi 👍🏻

@jpogran-hashi jpogran-hashi merged commit 4cf680c into main Aug 7, 2025
8 checks passed
@jpogran-hashi jpogran-hashi deleted the tf-27832-parse-component-address branch August 7, 2025 16:08
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.

4 participants