Skip to content

Feat/python enhance scripts#54

Open
57388sp wants to merge 1 commit into
mainfrom
feat/python-enhance-scripts
Open

Feat/python enhance scripts#54
57388sp wants to merge 1 commit into
mainfrom
feat/python-enhance-scripts

Conversation

@57388sp

@57388sp 57388sp commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Enhances the Python ONTAP scripts with configurable timeouts, reliable cross-version intercluster LIF detection, richer cluster reporting, and clearer docs. Removes hardcoded connection details from example INPUTS/ENV blocks.

Changes

  • ontap_client.py — add ONTAP_TIMEOUT env var; document all ONTAP_* vars
  • cluster_info.py — list aggregates (state, used %); add cluster contact/location
  • SnapMirror scripts — server-side query filters for intercluster LIF detection (cross-version reliable); expanded prerequisites (mixed versions, IC LIF routing); clearer SVM peer / relationship messages
  • README.md — document SSL/CA-bundle config and ONTAP_TIMEOUT
  • All scripts — remove hardcoded IPs/credentials from INPUTS/ENV blocks

Notes

  • ONTAP REST API only (9.8+); no public interface changes
  • Tested against live ONTAP 9.20.1, 9.18.1, and 9.17.1 clusters

@57388sp 57388sp requested a review from hvinn as a code owner June 5, 2026 07:03
@github-actions github-actions Bot added python Pull requests that update python code docs needs-test-report labels Jun 5, 2026
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Test Report missing or unfilled.

This PR touches python/, ansible/, or terraform/, so a populated Test Report section is required in the PR body. See TESTING.md for what to capture (environment, platform version, first-run output, idempotency check, teardown).

This is a soft gate - your CI checks are unaffected - but reviewers will not approve until the report is filled in.

- Remove hardcoded IPs and credentials from INPUTS/ENV blocks
- Add ONTAP_TIMEOUT env var support to OntapClient
- Use server-side query filters for intercluster LIF detection (cross-version reliable)
- Add aggregate listing and contact/location to cluster_info
- Improve SnapMirror prerequisites docs (mixed versions, IC LIF routing)
- Add clearer messages for SVM peer state and relationship lookups
- Document SSL/CA-bundle configuration in README
@57388sp 57388sp force-pushed the feat/python-enhance-scripts branch from 08ce10d to ae497cb Compare June 5, 2026 07:14

@hvinn hvinn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

make sure this doesn't break any tested workflows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs needs-test-report python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants