Skip to content

Conversation

mayankmani-sde
Copy link
Collaborator

SUMMARY

A new boolean parameter, noscratch, was introduced to the module to give users control over whether data sets are deleted using scratch or non-scratch logic. This affects operations when state == "absent".

#2053

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

zos_data_set
data_set
test_zos_data_set_func

ADDITIONAL INFORMATION

mayankmani-sde and others added 12 commits July 7, 2025 21:23
* add nonscratch parameter in main module

* added support for nonscratch in module_utils

* added test case

* added fragments

* FIXED

* changed in fragments

* Updated changelogs

* reviewed comments

* fixed

* removed print statement

* removed extra validation

---------

Co-authored-by: Fernando Flores <[email protected]>
@mayankmani-sde
Copy link
Collaborator Author

Already merged in branch staging-v1.16 as part of PR 2202

@mayankmani-sde mayankmani-sde marked this pull request as ready for review July 18, 2025 06:54
* adding sysin support

* Adding the test case

* changelog addition

* sanity issue resolving

* removing redundant files

* review comments incorporation
richp405
richp405 previously approved these changes Jul 29, 2025
Rohitcodes28 and others added 2 commits July 30, 2025 16:38
…ooks in test cases (#2213)

* editing and testing modules

* changelog addition
…2210-zos_data_set-Support-noscratch-options.yml

as this branch was already merged in staging now creating new PR to merge in dev so i chnage the pull request no in fragments
@mayankmani-sde mayankmani-sde changed the title Enhancement/2053/zos data set Enhancement[2053]Support for NOSCRATCH option when deleting datasets Aug 6, 2025
@mayankmani-sde mayankmani-sde changed the title Enhancement[2053]Support for NOSCRATCH option when deleting datasets [Enhancement][zos_data_set]Support for NOSCRATCH option when deleting datasets Aug 6, 2025
rexemin and others added 2 commits August 7, 2025 10:23
…ling (#2229)

* Fix TYPRUN handling

* Add changelog fragment

* Remove commented code

* Update module documentation
@rexemin rexemin added the Do not Merge When a pull request should not be merged for issue noted reasons label Aug 7, 2025
@fernandofloresg fernandofloresg removed the Do not Merge When a pull request should not be merged for issue noted reasons label Aug 15, 2025
@fernandofloresg
Copy link
Collaborator

Fixed the merge issues and tested using ansible 2.17

Screenshot 2025-08-15 at 3 18 29 PM Screenshot 2025-08-15 at 3 18 46 PM

Forced tests for zos_data_set and zos_job_submit bu that is ok because development branch is currently unstable.

@fernandofloresg fernandofloresg marked this pull request as draft August 15, 2025 21:20
@fernandofloresg fernandofloresg marked this pull request as ready for review August 15, 2025 21:20
Copy link
Collaborator

@fernandofloresg fernandofloresg left a comment

Choose a reason for hiding this comment

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

LGTM! I'm cheating a bit approving my own changes but it was needed lol

@fernandofloresg fernandofloresg merged commit 08a27e3 into dev Aug 15, 2025
15 checks passed
@fernandofloresg fernandofloresg deleted the Enhancement/2053/zos_data_set branch August 15, 2025 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants