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

Update create_pgsnapshot in purefa_pgsnap.py renaming throttle parame… #516

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

AheroX
Copy link
Contributor

@AheroX AheroX commented Dec 1, 2023

…ter of protection_group_snapshots_post to allow_throttle

"create_pgsnapshot" in plugins/modules/purefa_pgsnap.py calls "protection_group_snapshots_api.py" from the py-pure-client with the wrongly named parameter "throttle" instead of "allow_throttle" to fail if array health is not optimal.

This causes the error:
Traceback (most recent call last):
File "[...]/AnsiballZ_purefa_pgsnap.py", line 107, in
_ansiballz_main()
File "[...]/AnsiballZ_purefa_pgsnap.py", line 99, in _ansiballz_main
invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)
File "[...]/AnsiballZ_purefa_pgsnap.py", line 47, in invoke_module
runpy.run_module(mod_name='ansible_collections.purestorage.flasharray.plugins.modules.purefa_pgsnap', init_globals=dict(_module_fqn='ansible_collections.purestorage.flasharray.plugins.modules.purefa_pgsnap', _modlib_path=modlib_path),
File "/usr/lib64/python3.9/runpy.py",line 225, in run_module
return _run_module_code(code, init_globals, run_name, mod_spec)
File "/usr/lib64/python3.9/runpy.py", line 97, in _run_module_code
_run_code(code, mod_globals, init_globals,
File "/usr/lib64/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "[...]/purefa_pgsnap.py", line 615, in
File "[...]/purefa_pgsnap.py", line 593, in main
File "[...]/purefa_pgsnap.py", line 336, in create_pgsnapshot
TypeError: post_protection_group_snapshots() got an unexpected keyword argument 'throttle'

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

purefa_pgsnap.py

…ter of protection_group_snapshots_post to allow_throttle

"create_pgsnapshot" in plugins/modules/purefa_pgsnap.py calls "protection_group_snapshots_api.py" from the py-pure-client with the wrongly named parameter "throttle" instead of "allow_throttle" to fail if array health is not optimal.

This causes the error:
Traceback (most recent call last):
  File "[...]/AnsiballZ_purefa_pgsnap.py", line 107, in <module>
    _ansiballz_main()
  File "[...]/AnsiballZ_purefa_pgsnap.py", line 99, in _ansiballz_main
    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)
  File "[...]/AnsiballZ_purefa_pgsnap.py", line 47, in invoke_module
    runpy.run_module(mod_name='ansible_collections.purestorage.flasharray.plugins.modules.purefa_pgsnap', init_globals=dict(_module_fqn='ansible_collections.purestorage.flasharray.plugins.modules.purefa_pgsnap', _modlib_path=modlib_path),
  File "/usr/lib64/python3.9/runpy.py",line 225, in run_module
    return _run_module_code(code, init_globals, run_name, mod_spec)
  File "/usr/lib64/python3.9/runpy.py", line 97, in _run_module_code
_run_code(code, mod_globals, init_globals,
  File "/usr/lib64/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "[...]/purefa_pgsnap.py", line 615, in <module>
  File "[...]/purefa_pgsnap.py", line 593, in main
  File "[...]/purefa_pgsnap.py", line 336, in create_pgsnapshot
TypeError: post_protection_group_snapshots() got an unexpected keyword argument 'throttle'
@CLAassistant
Copy link

CLAassistant commented Dec 1, 2023

CLA assistant check
All committers have signed the CLA.

@sdodsley sdodsley added the bug Something isn't working label Dec 1, 2023
@sdodsley sdodsley self-requested a review December 1, 2023 16:52
@sdodsley
Copy link
Member

sdodsley commented Dec 1, 2023

@AheroX please create a doc fragment for this bug fix.

@AheroX
Copy link
Contributor Author

AheroX commented Dec 1, 2023

create a doc fragment

My apologies - no idea what i am to do.

@sdodsley
Copy link
Member

sdodsley commented Dec 1, 2023

No worries - I'll add it for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants