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

If a remote copy link already exists, the IP validation is ignored. #9

Open
leenaj0hn opened this issue Oct 4, 2018 · 2 comments
Open
Labels
bug Something isn't working fixed The bug has been fixed, but needs verification high The bug breaks the functionality

Comments

@leenaj0hn
Copy link

If a remote copy link already exists, and the user provides an incomplete IP address, the playbook does not fail with invalid IP error. The same succeeds with message remote copy already exists.

ok: [localhost] => {
"changed": false,
"invocation": {
"module_args": {
"different_secondary_wwn": false,
"discard_new_data": false,
"domain": null,
"full_sync": false,
"keep_snap": false,
"local_groups_direction": false,
"local_remote_volume_pair_list": null,
"local_snap_cpg": null,
"local_user_cpg": null,
"no_resync_snapshot": false,
"no_snapshot": false,
"recovery_action": null,
"remote_copy_group_name": null,
"remove_secondary_volume": false,
"skip_initial_sync": false,
"skip_promote": false,
"skip_start": false,
"skip_sync": false,
"snapshot_name": null,
"source_port": "1:3:1",
"starting_snapshots": null,
"state": "admitlink",
"stop_groups": false,
"storage_system_ip": "192.168.67.5",
"storage_system_password": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
"storage_system_username": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
"target_mode": null,
"target_name": "CSSOS-SSA06",
"target_port_wwn_or_ip": "192.168.2.",
"targets": null,
"unset_snap_cpg": false,
"unset_user_cpg": false,
"volume_auto_creation": false,
"volume_name": null
}
},
"msg": "Admit remote copy link 1:3:1:192.168.2. already exists."

@farhan7500
Copy link
Owner

Fixed!

Kindly verify

@farhan7500 farhan7500 added bug Something isn't working fixed The bug has been fixed, but needs verification high The bug breaks the functionality labels Oct 9, 2018
@ansarars
Copy link
Collaborator

Fixed in PR farhan7500/python-3parclient#4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed The bug has been fixed, but needs verification high The bug breaks the functionality
Projects
None yet
Development

No branches or pull requests

3 participants