Skip to content

Feature/gen2 qualification playbooks - #114

Merged
Saksham-Nautiyal merged 3 commits into
mainfrom
feature/gen2-qualification-playbooks
Aug 17, 2026
Merged

Feature/gen2 qualification playbooks#114
Saksham-Nautiyal merged 3 commits into
mainfrom
feature/gen2-qualification-playbooks

Conversation

@Saksham-Nautiyal

Copy link
Copy Markdown
Contributor

Description

A few sentences describing the overall goals of the pull request's commits.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, pep8, linting, or security issues
  • I have performed Ansible Sanity test using --docker default
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Saksham-Nautiyal and others added 2 commits August 5, 2026 03:39
Updated example playbooks for Gen2 modules as part of Story-24775-powerflex-gen2-connectivity-crud:
- device_group.yml
- info_v2.yml
- nvme_host.yml
- protection_domain_v2.yml
- sdc.yml
- sdt.yml
- snapshot_policy.yml
- snapshot_v2.yml
- storagepool_v2.yml
- thin_clone_example.yml
- volume_v2.yml

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
- README.md: Updated PowerFlex 5.1 Gen2 (Core 5.1) compatibility matrix
- playbooks/modules/device_v2.yml: Adjusted example for Gen2 qualification

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@Saksham-Nautiyal Saksham-Nautiyal self-assigned this Aug 14, 2026
- Replace `set_fact` with FQCN `ansible.builtin.set_fact`
- Replace `ignore_errors: true` with `failed_when: false`
- Capitalize task name in thin_clone example

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.32%. Comparing base (52fce50) to head (99ece9c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   93.32%   93.32%           
=======================================
  Files          79       79           
  Lines       12037    12037           
  Branches     1439     1439           
=======================================
  Hits        11233    11233           
  Misses        391      391           
  Partials      413      413           
Flag Coverage Δ
units 93.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Saksham-Nautiyal
Saksham-Nautiyal merged commit dd33b12 into main Aug 17, 2026
20 checks passed
@Saksham-Nautiyal
Saksham-Nautiyal deleted the feature/gen2-qualification-playbooks branch August 17, 2026 05:41
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