Skip to content

feat: add Ansible playbook for CIFS share provisioning workflow#7

Merged
kxvya-git merged 1 commit intomainfrom
feature/ansible-cifs-provision
Apr 14, 2026
Merged

feat: add Ansible playbook for CIFS share provisioning workflow#7
kxvya-git merged 1 commit intomainfrom
feature/ansible-cifs-provision

Conversation

@kxvya-git
Copy link
Copy Markdown
Collaborator

Summary

Add cifs_provision.yml playbook that creates a FlexVol with NTFS security style, creates a CIFS share, sets share ACL, and verifies the share. Update group_vars example with CIFS-specific defaults.

Changes

  • Add [cifs_provision.yml]— new playbook with 5 tasks:
    Create FlexVol volume with volume_security_style: ntfs
    Create CIFS share on the volume
    Set share ACL (user/group + permission)
    Verify the share via na_ontap_rest_info
    Print summary
    Update ontap.yml.example — reorganise shared defaults and add CIFS-specific variables (share_name, share_comment, acl_user, acl_permission)

Checklist

  • No secrets, credentials, or API tokens in code or config
  • Workflow YAML validates against workflow-spec/v1/schema.json
  • Linting passes (ruff check executors/python/orchestrio/)
  • Formatting passes (ruff format --check executors/python/)
  • Tests pass (pytest executors/python/tests/ -v)
  • Manually tested with an example workflow (if applicable)

Related issues

@kxvya-git kxvya-git requested a review from hvinn as a code owner April 13, 2026 05:36
Add ansible/cifs_provision.yml playbook that creates a FlexVol with NTFS security style, creates a CIFS share, sets share ACL, and verifies the share. Update group_vars example with CIFS-specific defaults and README with usage docs.
@kxvya-git kxvya-git force-pushed the feature/ansible-cifs-provision branch from ebbf9c3 to 14cb11d Compare April 13, 2026 19:52
@kxvya-git kxvya-git merged commit 0f788ed into main Apr 14, 2026
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants