Skip to content

Azure: Fix update_skus rountine for Gen1 default#56

Merged
JAVGan merged 1 commit into
mainfrom
fixazure
Oct 1, 2024
Merged

Azure: Fix update_skus rountine for Gen1 default#56
JAVGan merged 1 commit into
mainfrom
fixazure

Conversation

@JAVGan

@JAVGan JAVGan commented Sep 30, 2024

Copy link
Copy Markdown
Collaborator

This commit changes the ms_azure.utils.update_skus function to prevent failures on Azure when dealing with already existing skus.

The changes make this function behave as follow:

  1. If the plan has no skus it will create it for the first time using the default generation informed as the parameter generation.

  2. If the plan already has skus it will just detect which one is the default and then update the values accordingly, to avoid issues with The property 'PlanId' is locked by a previous submission. In this case, the argumentgeneration is not being activelly used.

Refers to SPSTRAT-397

@JAVGan

JAVGan commented Sep 30, 2024

Copy link
Copy Markdown
Collaborator Author

@lslebodn @jajreidy PTAL.

This commit changes the `ms_azure.utils.update_skus` function to prevent
failures on Azure when dealing with already existing skus.

The changes make this function behave as follow:

1. If the plan *has no skus* it will create it for the first time using
   the default generation informed as the parameter `generation`.

2. If the plan *already has skus* it will just detect which one is the
   default and then update the values accordingly, to avoid issues with
  `The property 'PlanId' is locked by a previous submission`. In this
   case, the argument`generation` is not being activelly used.

Refers to SPSTRAT-397

@jajreidy jajreidy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@JAVGan JAVGan merged commit 9035b10 into main Oct 1, 2024
@JAVGan JAVGan deleted the fixazure branch October 1, 2024 13:09
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.

2 participants