Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

fix(google): Add partner metadata on clone (backport #10161)#10162

Merged
edgarulg merged 1 commit intorelease-1.36.xfrom
mergify/bp/release-1.36.x/pr-10161
Jan 21, 2025
Merged

fix(google): Add partner metadata on clone (backport #10161)#10162
edgarulg merged 1 commit intorelease-1.36.xfrom
mergify/bp/release-1.36.x/pr-10161

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jan 21, 2025

ISSUE

When cloning a server group using the UI. The resourceManagerTags and partnerMetadata are not populated even if the deployment is configured with resourceManagerTags and partnerMetadata.

Expected Behavior
If the server group has resourceManagerTags and partnerMetadata configured. Cloning the server group should populate these fields automatically.

Captura de pantalla 2025-01-17 a la(s) 4 07 23 p m Captura de pantalla 2025-01-17 a la(s) 4 07 40 p m Captura de pantalla 2025-01-17 a la(s) 4 08 26 p m

The main issue comes from Clouddriver because the instanceTemplate doesn't include the partner metadata. See: spinnaker/clouddriver#6334

In Deck, we just populate the resourceManagerTags and partnerMetadata in the instanceTemplateProperties under the lauchConfiguration.


This is an automatic backport of pull request #10161 done by [Mergify](https://mergify.com).

* fix(google): populate resourceManagerTags & partnerMetadata when cloning server group

* fix(google): add logs for debug

* fix(google): populate resourceManagerTags & partnerMetadata from launchConfig when clone serverGroup

(cherry picked from commit 3005516)
@edgarulg edgarulg added the ready to merge Reviewed and ready for merge label Jan 21, 2025
@edgarulg edgarulg merged commit 5dde11f into release-1.36.x Jan 21, 2025
2 checks passed
@edgarulg edgarulg deleted the mergify/bp/release-1.36.x/pr-10161 branch January 21, 2025 17:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready to merge Reviewed and ready for merge target-release/1.36

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants