Skip to content

Do not set role-label in expand, rely on module embedded labels#1904

Merged
mr0re1 merged 1 commit into
GoogleCloudPlatform:developfrom
mr0re1:remove_role
Nov 3, 2023
Merged

Do not set role-label in expand, rely on module embedded labels#1904
mr0re1 merged 1 commit into
GoogleCloudPlatform:developfrom
mr0re1:remove_role

Conversation

@mr0re1

@mr0re1 mr0re1 commented Oct 30, 2023

Copy link
Copy Markdown
Collaborator

Release note:


ghpc will no longer populate ghpc_role label based on module source path for third-party modules. Please take care of setting it yourself if you rely on this label.


  • Do not set role-label in expand, rely on module embedded labels;
  • Add ghpc labels to packer module.

* Do not set role-label in expand, rely on module embedded labels;
* Add ghpc labels to packer module.
@mr0re1 mr0re1 added the release-chore To not include into release notes label Oct 30, 2023
@mr0re1 mr0re1 requested a review from nick-stroud October 30, 2023 23:51
Comment thread pkg/config/expand.go
@nick-stroud nick-stroud assigned mr0re1 and unassigned nick-stroud Nov 1, 2023
@mr0re1 mr0re1 added release-deprecations Added to release notes under the "Deprecations" heading. and removed release-chore To not include into release notes labels Nov 2, 2023
@mr0re1

mr0re1 commented Nov 2, 2023

Copy link
Copy Markdown
Collaborator Author

@nick-stroud , updated label to "deprecation", added release note snippet. Please review.

@mr0re1 mr0re1 assigned nick-stroud and unassigned mr0re1 Nov 2, 2023
@mr0re1 mr0re1 requested a review from nick-stroud November 2, 2023 21:42
@nick-stroud nick-stroud assigned mr0re1 and unassigned nick-stroud Nov 3, 2023
@mr0re1 mr0re1 merged commit 70b775e into GoogleCloudPlatform:develop Nov 3, 2023
@mr0re1 mr0re1 deleted the remove_role branch November 3, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-deprecations Added to release notes under the "Deprecations" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants