diff options
| author | Leela Venkaiah G <gleelavenkaiah@gmail.com> | 2021-01-13 16:02:25 +0530 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-13 16:02:25 +0530 |
| commit | 31895408f080c5b367c957901cdc9bb73c5c18cd (patch) | |
| tree | 5b9e6095e50b15fb43f059752ca9fd62ce5df06a /events | |
| parent | 2d8c88db4e39f6434f9d385bc5a47ad41b8fc2cb (diff) | |
| download | glusterfs-31895408f080c5b367c957901cdc9bb73c5c18cd.tar.gz glusterfs-31895408f080c5b367c957901cdc9bb73c5c18cd.tar.xz glusterfs-31895408f080c5b367c957901cdc9bb73c5c18cd.zip | |
glusterfs-events: Fix incorrect attribute access (#2002)
Issue: When GlusterCmdException is raised, current code try to access
message atrribute which doesn't exist and resulting in a malformed
error string on failure operations
Code Change: Replace `message` with `args[0]`
Fixes: #2001
Change-Id: I65c9f0ee79310937a384025b8d454acda154e4bb
Signed-off-by: Leela Venkaiah G <lgangava@redhat.com>
Diffstat (limited to 'events')
| -rw-r--r-- | events/src/peer_eventsapi.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/events/src/peer_eventsapi.py b/events/src/peer_eventsapi.py index 4d2e5f35b1..795ad5f9d5 100644 --- a/events/src/peer_eventsapi.py +++ b/events/src/peer_eventsapi.py @@ -174,9 +174,9 @@ def sync_to_peers(args): sync_file_to_peers(WEBHOOKS_FILE_TO_SYNC) except GlusterCmdException as e: # Print stdout if stderr is empty - errmsg = e.message[2] if e.message[2] else e.message[1] + errmsg = e.args[0][2] if e.args[0][2] else e.args[0][1] handle_output_error("Failed to sync Webhooks file: [Error: {0}]" - "{1}".format(e.message[0], errmsg), + "{1}".format(e.args[0][0], errmsg), errcode=ERROR_WEBHOOK_SYNC_FAILED, json_output=args.json) @@ -185,9 +185,9 @@ def sync_to_peers(args): sync_file_to_peers(CUSTOM_CONFIG_FILE_TO_SYNC) except GlusterCmdException as e: # Print stdout if stderr is empty - errmsg = e.message[2] if e.message[2] else e.message[1] + errmsg = e.args[0][2] if e.args[0][2] else e.args[0][1] handle_output_error("Failed to sync Config file: [Error: {0}]" - "{1}".format(e.message[0], errmsg), + "{1}".format(e.args[0][0], errmsg), errcode=ERROR_CONFIG_SYNC_FAILED, json_output=args.json) |