Skip to content

[Modules] Leave out not supported throughput field in Cosmos#1658

Merged
AlexanderSehr merged 1 commit intoAzure:mainfrom
itpropro:fix/documentdb
Jul 11, 2022
Merged

[Modules] Leave out not supported throughput field in Cosmos#1658
AlexanderSehr merged 1 commit intoAzure:mainfrom
itpropro:fix/documentdb

Conversation

@itpropro
Copy link
Copy Markdown
Contributor

Description

When CosmosDB is deployed serverless, it is not supported to provide throughput.
Therefore a filter was introduced in the options to conditionally provide the property.

...
    options: contains(databaseAccount.properties.capabilities, { name: 'EnableServerless' }) ? null : {
      throughput: throughput
    }
...

Type of Change

Please delete options that are not relevant.

  • Bugfix (non-breaking change which fixes an issue)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

@itpropro itpropro requested a review from a team as a code owner July 10, 2022 15:28
@eriqua eriqua added enhancement New feature or request [cat] modules category: modules labels Jul 10, 2022
@AlexanderSehr AlexanderSehr merged commit be75792 into Azure:main Jul 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[cat] modules category: modules enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants