Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/7525/changed default value for dds in zos_mvs_raw #336

Merged
merged 5 commits into from
Aug 29, 2022

Conversation

rexemin
Copy link
Collaborator

@rexemin rexemin commented Jun 13, 2022

SUMMARY

Fixes item 7525 on Jira by changing the way the argument parser treats the default value for the dds parameter in the zos_mvs_raw module.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

zos_mvs_raw

ADDITIONAL INFORMATION

This bug was reported to happen while trying to run the IEFBR14 command in a playbook without specifying any DD statements. The command by itself doesn't need any DD statements but the module nonetheless throws an exception if you don't specify at least a dummy dataset. Additionally, the documentation states that the dds parameter is optional, so the argument parser was changed accordingly.

@rexemin rexemin changed the title Changed default value for dds Bugfix/7525/changed default value for dds in zos_mvs_raw Jun 15, 2022
@rexemin rexemin marked this pull request as ready for review June 15, 2022 18:06
Copy link
Collaborator

@richp405 richp405 left a comment

Choose a reason for hiding this comment

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

seems straightforward.

@ddimatos ddimatos self-requested a review August 29, 2022 17:15
@ddimatos
Copy link
Collaborator

Agree , straight forward, looks good, thanks @rexemin

@ddimatos
Copy link
Collaborator

image

Internal link/log

@ddimatos ddimatos merged commit 23392ae into dev Aug 29, 2022
@ddimatos ddimatos deleted the bugfix/7525/zos-mvs-raw-no-dds branch August 29, 2022 22:18
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.

3 participants