Skip to content

Commit

Permalink
Update logging outputs for consistency
Browse files Browse the repository at this point in the history
Since "Successfully" was removed to improve readability for the output in
ManageOperators._update_cyhy_ops_users(), I have mirrored that change in the
rest of the class.
  • Loading branch information
mcdonnnj committed Apr 6, 2021
1 parent c16418f commit 96fed19
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/manage_cyhy_ops/manageoperators.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ def add_user(self, username: str, ssh_key: str, overwrite: bool = False) -> int:
Overwrite=overwrite,
)
logging.info(
'Successfully added "%s"\'s SSH key to the Parameter Store in "%s".',
'Added "%s"\'s SSH key to the Parameter Store in "%s".',
username,
self.region,
)
Expand Down Expand Up @@ -148,7 +148,7 @@ def remove_user(self, username: str, full: bool = False) -> int:
# Response is an empty dictionary on success.
self._client.delete_parameter(Name=parameter_name)
logging.info(
'Successfully removed SSH key for user "%s" in region "%s".',
'Removed SSH key for user "%s" in region "%s".',
username,
self.region,
)
Expand Down

0 comments on commit 96fed19

Please sign in to comment.