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
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.

1 participant