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.

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

Send channel id to HCP Packer #11861

Merged
merged 8 commits into from
Jul 29, 2022
Merged

Send channel id to HCP Packer #11861

merged 8 commits into from
Jul 29, 2022

Conversation

sylviamoss
Copy link
Contributor

@sylviamoss sylviamoss commented Jul 4, 2022

HCP Packer needs to track the channel that was queried to get the source image. This PR adds the required changes to start sending the information to the service.

These changes conflict with #11905

@sylviamoss sylviamoss force-pushed the hcp_packer_source_channel_id branch from 7281f31 to f2b8a39 Compare July 4, 2022 15:03
@sylviamoss sylviamoss force-pushed the hcp_packer_source_channel_id branch from ef5d2d1 to 38b0b2c Compare July 27, 2022 13:00
@sylviamoss sylviamoss changed the title source iteration channel id poc Send channel id to HCP Packer Jul 27, 2022
@sylviamoss sylviamoss marked this pull request as ready for review July 27, 2022 13:21
@sylviamoss sylviamoss requested a review from a team as a code owner July 27, 2022 13:21
Copy link
Contributor

@nywilken nywilken left a comment

Choose a reason for hiding this comment

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

So far this looks good to me. I left a few comments. I think the refactor to break out the code can help things once merged.

datasource/hcp-packer-image/data.go Show resolved Hide resolved
datasource/hcp-packer-image/data.go Show resolved Hide resolved
datasource/hcp-packer-iteration/data.go Show resolved Hide resolved
datasource/packer-image-iteration/data.go Outdated Show resolved Hide resolved
hcl2template/types.build.go Outdated Show resolved Hide resolved
internal/registry/service.go Outdated Show resolved Hide resolved
internal/registry/service.go Outdated Show resolved Hide resolved
@sylviamoss sylviamoss force-pushed the hcp_packer_source_channel_id branch from 867c962 to 6ebbab9 Compare July 27, 2022 16:04
@sylviamoss
Copy link
Contributor Author

@nywilken I addressed all of your comments! 🎉

Copy link
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

Aside from the few comments I have, LGTM!

@@ -99,7 +98,7 @@ func (d *Datasource) Configure(raws ...interface{}) error {
return nil
}

// Information from []*models.HashicorpCloudPackerImage with some information
// DatasourceOutput Information from []*models.HashicorpCloudPackerImage with some information
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm guessing this was added to make the linters happy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! Correct!

internal/registry/service.go Outdated Show resolved Hide resolved
Co-authored-by: Wilken Rivera <wilken@hashicorp.com>
@sylviamoss sylviamoss merged commit 5c3d5c1 into main Jul 29, 2022
@sylviamoss sylviamoss deleted the hcp_packer_source_channel_id branch July 29, 2022 13:15
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants