Skip to content

[PostgreSQL] Deprecate Az.PostgreSql module#29649

Merged
VeryEarly merged 3 commits into
Azure:PostgreSql/release-2026-06-02from
nachoalonsoportillo:deprecate-Az-PostgreSql
May 26, 2026
Merged

[PostgreSQL] Deprecate Az.PostgreSql module#29649
VeryEarly merged 3 commits into
Azure:PostgreSql/release-2026-06-02from
nachoalonsoportillo:deprecate-Az-PostgreSql

Conversation

@nachoalonsoportillo

Copy link
Copy Markdown
Member

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings May 20, 2026 10:39
@azure-client-tools-bot-prd

Copy link
Copy Markdown
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@VeryEarly

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

@YanaXu

YanaXu commented May 21, 2026

Copy link
Copy Markdown
Contributor

Hi @nachoalonsoportillo, the status of this PR is not clear. Let's discuss about it.

@github-actions

Copy link
Copy Markdown

‼️ DO NOT MERGE THIS PR ‼️
This PR was labeled "Do Not Merge" because it contains code change that cannot be merged. Please contact the reviewer for more information.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also remove this from tools/Az/Az.psd1

@YanaXu

YanaXu commented May 21, 2026

Copy link
Copy Markdown
Contributor

Please check files:

  • tools/Az/Az.psd1
  • tools/GeneratedModuleList.txt
  • tools/TestFx/Coverage/Baseline.csv

@VeryEarly

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings May 25, 2026 06:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@VeryEarly

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 3 pipeline(s).

@nachoalonsoportillo nachoalonsoportillo changed the title Deprecate Az.PostgreSql module [PostgreSQL] Deprecate Az.PostgreSql module May 25, 2026
@VeryEarly VeryEarly changed the base branch from main to PostgreSql/release-2026-06-02 May 26, 2026 12:30
@VeryEarly VeryEarly merged commit 7f7f3f4 into Azure:PostgreSql/release-2026-06-02 May 26, 2026
1 of 6 checks passed
MaddyMicrosoft added a commit to MaddyMicrosoft/azure-powershell that referenced this pull request Jun 16, 2026
PostgreSql was deprecated and deleted in Azure#29649 but remained in
.ci-config.json selectModuleList. This causes build failures when
CIFilterTask includes PostgreSql in the test module list but the
src/PostgreSql folder no longer exists.

Remove PostgreSql from selectModuleList to prevent it from being
selected for builds/tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants