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

[Fixes] Fix deployment error with flexible server configurations #2394

Conversation

JPEasier
Copy link
Contributor

Description

After creating the databases, firewall rules are created. These must be deployed before configurations can be created, otherwise a deployment error may occur. Failed Pipeline
See also: https://learn.microsoft.com/en-us/answers/questions/1078895/deploying-postgresql-flexible-server-with-azureext.html

Solution:

Add dependsOn to the sub resources

Pipeline references

Pipeline
DbForPostgreSQL: FlexibleServers

Type of Change

  • 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

…users/jpeasier/fixDeploymentErrorWithFlexibleServerConfigurations
@JPEasier JPEasier added the bug Something isn't working label Nov 30, 2022
@JPEasier JPEasier added this to the Release v0.8.0 milestone Nov 30, 2022
@JPEasier JPEasier requested a review from a team as a code owner November 30, 2022 12:55
@JPEasier JPEasier self-assigned this Nov 30, 2022
@JPEasier JPEasier changed the title Users/jpeasier/fix deployment error with flexible server configurations Fix deployment error with flexible server configurations Nov 30, 2022
@AlexanderSehr AlexanderSehr changed the title Fix deployment error with flexible server configurations [Fixes] Fix deployment error with flexible server configurations Nov 30, 2022
@AlexanderSehr AlexanderSehr added the [cat] modules category: modules label Nov 30, 2022
@JPEasier JPEasier merged commit 58cbcbc into main Nov 30, 2022
@JPEasier JPEasier deleted the users/jpeasier/fixDeploymentErrorWithFlexibleServerConfigurations branch November 30, 2022 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working [cat] modules category: modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants