Skip to content

Fixed an issue with the list_fsxn_* script, and added a PowerShell version of the warm_performnace_tier script. - #287

Open
kcantrel wants to merge 8 commits into
mainfrom
update_terraform_checks
Open

kcantrel wants to merge 8 commits into
mainfrom
update_terraform_checks

Conversation

@kcantrel

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Partial reads, suppressed failures, invalid parameter handling, and early watchdog timeouts can produce incomplete or incorrect operation.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds Windows support for warming FSx for ONTAP performance tiers and improves regional listing scripts.

Changes:

  • Adds a parallel PowerShell warming utility.
  • Supports comma-separated regions and responsive watchdogs.
  • Expands usage documentation.
File summaries
File Description
WarmPerformanceTier.ps1 Adds Windows warming implementation.
warm_performance_tier/README.md Documents Windows usage.
list_fsxn_volumes Adds multi-region parsing and watchdog polling.
list_fsxn_svms Adds multi-region parsing and watchdog polling.
list_fsxn_filesystems Adds multi-region parsing and watchdog polling.
Review details

Suppressed comments (1)

Management-Utilities/warm_performance_tier/README.md:136

  • Correct the heading typo from “Runing” to “Running.”
### Runing the script on a Windows based operating system
  • Files reviewed: 5/5 changed files
  • Comments generated: 13
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread Management-Utilities/fsx-ontap-aws-cli-scripts/list_fsxn_filesystems Outdated
Comment thread Management-Utilities/fsx-ontap-aws-cli-scripts/list_fsxn_svms Outdated
Comment thread Management-Utilities/fsx-ontap-aws-cli-scripts/list_fsxn_volumes Outdated
Comment thread Management-Utilities/warm_performance_tier/WarmPerformanceTier.ps1
Comment thread Management-Utilities/warm_performance_tier/WarmPerformanceTier.ps1
Comment thread Management-Utilities/warm_performance_tier/WarmPerformanceTier.ps1
Comment thread Management-Utilities/fsx-ontap-aws-cli-scripts/list_fsxn_svms Outdated
Comment thread Management-Utilities/fsx-ontap-aws-cli-scripts/list_fsxn_volumes Outdated
Comment thread Management-Utilities/warm_performance_tier/README.md Outdated
Comment thread Management-Utilities/warm_performance_tier/README.md Outdated
@kcantrel
kcantrel requested a review from nichollri September 22, 2026 18:28
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