Skip to content

[Az.ServiceFabric] Add Enable-AzServiceFabricManagedClusterMaintenanceWindow cmdlet#29538

Open
iliu816 wants to merge 4 commits intoAzure:mainfrom
iliu816:enable-sfmc-maintenance-window
Open

[Az.ServiceFabric] Add Enable-AzServiceFabricManagedClusterMaintenanceWindow cmdlet#29538
iliu816 wants to merge 4 commits intoAzure:mainfrom
iliu816:enable-sfmc-maintenance-window

Conversation

@iliu816
Copy link
Copy Markdown
Member

@iliu816 iliu816 commented May 8, 2026

Description

Add new Enable-AzServiceFabricManagedClusterMaintenanceWindow cmdlet that wraps the applyMaintenanceWindow API for Service Fabric Managed Clusters. This allows users to apply the maintenance window, triggering any pending updates.

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately - Done
  • SHOULD regenerate markdown help files if there is cmdlet API change - Done (generated via platyPS)
  • SHOULD have proper test coverage for changes in pull request - Done (Record + Playback passing)
  • SHOULD NOT adjust version of module manually in pull request - Not changed

Add new cmdlet to apply the maintenance window on Service Fabric Managed
Clusters immediately. This wraps the applyMaintenanceWindow API which
triggers any pending updates to be applied right away.

Changes:
- New cmdlet EnableAzServiceFabricManagedClusterMaintenanceWindow.cs
- Help documentation and module help index
- Module manifest updated with new export
- ChangeLog entry
- Scenario test with session recording (deploys maintenance configuration
  and verifies applyMaintenanceWindow succeeds)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 17:31
@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.

@iliu816 iliu816 changed the title Add Enable-AzServiceFabricManagedClusterMaintenanceWindow cmdlet [Az.ServiceFabric] Add Enable-AzServiceFabricManagedClusterMaintenanceWindow cmdlet May 8, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new Az.ServiceFabric cmdlet, Enable-AzServiceFabricManagedClusterMaintenanceWindow, which wraps the Service Fabric Managed Clusters applyMaintenanceWindow API so users can immediately apply an assigned maintenance window (triggering pending updates).

Changes:

  • Added the new cmdlet implementation and exported it from the module manifest.
  • Added/updated markdown help content and updated the module help index.
  • Added a new scenario test (PowerShell + xUnit hook) to validate the cmdlet behavior.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/ServiceFabric/ServiceFabric/help/Enable-AzServiceFabricManagedClusterMaintenanceWindow.md New cmdlet reference help page.
src/ServiceFabric/ServiceFabric/help/Az.ServiceFabric.md Adds the cmdlet to the module help index.
src/ServiceFabric/ServiceFabric/Commands/ManagedClusters/ManagedClusters/EnableAzServiceFabricManagedClusterMaintenanceWindow.cs Implements the new cmdlet that calls ManagedApplyMaintenanceWindow.
src/ServiceFabric/ServiceFabric/ChangeLog.md Adds an Upcoming Release entry for the new cmdlet.
src/ServiceFabric/ServiceFabric/Az.ServiceFabric.psd1 Exports the new cmdlet from the module.
src/ServiceFabric/ServiceFabric.Test/ScenarioTests/ServiceFabricManagedClustersTests.ps1 Adds a scenario test that provisions a cluster, assigns a maintenance configuration, and applies the window.
src/ServiceFabric/ServiceFabric.Test/ScenarioTests/ServiceFabricManagedClustersTests.cs Wires the new PowerShell scenario test into the xUnit test runner.

@VeryEarly
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

fix shouldprocess message

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 17:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 4 comments.

Comment thread src/ServiceFabric/ServiceFabric/ChangeLog.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 17:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

@VeryEarly
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@VeryEarly
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@YanaXu
Copy link
Copy Markdown
Contributor

YanaXu commented May 9, 2026

Please resolve the conflicts.

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.

4 participants