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

[v1.9.2][Bugfix] [zos_copy] Only issue opercmd to check for a locked data set if dest exists #1549

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

fernandofloresg
Copy link
Collaborator

@fernandofloresg fernandofloresg commented Jun 24, 2024

SUMMARY

zos_copy was issuing an opercmd to see if the destination data set is locked even if it does not exist. Fix only issues the command if destination exists.

This is the fix meant for 1.9.2 release, did not add a test case for this scenario since automating a test to validate the command was not issued would take me some time, basically that would need to check the syslog to make sure the command was not issued for that specific data set name, I can add it if requested.

Fixes #1544

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

zos_copy

ADDITIONAL INFORMATION

@fernandofloresg fernandofloresg changed the base branch from dev to staging-v1.9.2 June 28, 2024 18:47
@fernandofloresg
Copy link
Collaborator Author

Screenshot 2024-06-28 at 12 46 12 PM

@fernandofloresg fernandofloresg marked this pull request as ready for review June 28, 2024 18:54
@fernandofloresg fernandofloresg changed the title [Bugfix] [zos_copy] Only issue opercmd to check for a locked data set if dest exists [v1.9.2][Bugfix] [zos_copy] Only issue opercmd to check for a locked data set if dest exists Jul 11, 2024
@fernandofloresg fernandofloresg merged commit 3d37045 into staging-v1.9.2 Jul 11, 2024
@fernandofloresg fernandofloresg deleted the bugfix/1544/zos_copy-opercmd branch July 11, 2024 18:33
Copy link
Collaborator

@ddimatos ddimatos left a comment

Choose a reason for hiding this comment

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

Looks good, nice and easy :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants