refactored logging levels in jira.py, service_desk.py, confluence.py, insights.py (#1468)

* refactoring log levels
* refactoring log.warn logging levels
---------

Co-authored-by: gkowalc <>
Co-authored-by: Greg <gkowalc>

Author: 0x89g09sa823

atlassian/confluence.py

@@ -762,7 +762,6 @@ def get_all_pages_by_space_ids_confluence_cloud(
     @deprecated(version="2.4.2", reason="Use get_all_restrictions_for_content()")
     def get_all_restictions_for_content(self, content_id):
         """Let's use the get_all_restrictions_for_content()"""
-        log.warning("Please, be informed that is deprecated as typo naming")
         return self.get_all_restrictions_for_content(content_id=content_id)
 
     def get_all_restrictions_for_content(self, content_id):
@@ -2284,7 +2283,7 @@ def remove_group(self, name):
         :param name: str
         :return:
         """
-        log.warning("Removing group...")
+        log.info("Removing group:  %s  during Confluence remove_group method execution", name)
         url = "rest/api/admin/group/{groupName}".format(groupName=name)
 
         try:
@@ -2714,7 +2713,7 @@ def get_atl_request(url: str):
                 }
             elif export_type == "pdf":
                 url = "spaces/flyingpdf/doflyingpdf.action?key=" + space_key
-                log.info("Initiate PDF  space export from space " + str(space_key))
+                log.info("Initiated PDF  space export")
                 return self.get_pdf_download_url_for_confluence_cloud(url)
             else:
                 raise ValueError("Invalid export_type parameter value. Valid values are: 'html/csv/xml/pdf'")

atlassian/insight.py

@@ -118,7 +118,7 @@ def upload_attachment_to_object(self, object_id, filename):
         """
         if self.cloud:
             raise NotImplementedError
-        log.warning("Adding attachment...")
+        log.info("Adding attachment...")
         url = "rest/insight/1.0/attachments/object/{objectId}".format(objectId=object_id)
         with open(filename, "rb") as attachment:
             files = {"file": attachment}
@@ -131,7 +131,7 @@ def delete_attachment(self, attachment_id):
         """
         if self.cloud:
             raise NotImplementedError
-        log.warning("Adding attachment...")
+        log.info("Deleting attachment...")
         url = "rest/insight/1.0/attachments/{attachmentId}".format(attachmentId=attachment_id)
         return self.delete(url)
 
@@ -373,7 +373,6 @@ def get_object_history(self, object_id, asc=False, abbreviate=True):
     @deprecated(version="3.29.0", reason="Use get_object_reference_info()")
     def get_object_referenceinfo(self, object_id):
         """Let's use the get_object_reference_info()"""
-        log.warning("Please, be informed that is deprecated as typo naming")
         self.get_object_reference_info(object_id)
 
     def get_object_reference_info(self, object_id):

atlassian/jira.py

@@ -599,7 +599,7 @@ def get_component_related_issues(self, component_id):
         return self.get(url)
 
     def create_component(self, component):
-        log.warning('Creating component "%s"', component["name"])
+        log.info('Creating component "%s"', component["name"])
         base_url = self.resource_url("component")
         url = "{base_url}/".format(base_url=base_url)
         return self.post(url, data=component)
@@ -610,7 +610,7 @@ def update_component(self, component, component_id):
         return self.put(url, data=component)
 
     def delete_component(self, component_id):
-        log.warning('Deleting component "%s"', component_id)
+        log.info('Deleting component "%s"', component_id)
         base_url = self.resource_url("component")
         return self.delete("{base_url}/{component_id}".format(base_url=base_url, component_id=component_id))
 
@@ -965,7 +965,7 @@ def remove_group(self, name, swap_group=None):
         :param swap_group: str - swap group
         :return:
         """
-        log.warning("Removing group...")
+        log.info("Removing group: %s ", name)
         url = self.resource_url("group")
         if swap_group is not None:
             params = {"groupname": name, "swapGroup": swap_group}
@@ -1030,7 +1030,7 @@ def remove_user_from_group(self, username=None, group_name=None, account_id=None
         :param group_name: str
         :return:
         """
-        log.warning("Removing user from a group...")
+        log.info("Removing user: %s from a group: %s", username, group_name)
         url = self.resource_url("group/user")
         url_domain = self.url
         if "atlassian.net" in url_domain:
@@ -1422,7 +1422,7 @@ def add_attachment_object(self, issue_key, attachment):
         :param issue_key: str
         :param attachment: IO Object
         """
-        log.warning("Adding attachment...")
+        log.info("Adding attachment:  %s", attachment)
         base_url = self.resource_url("issue")
         url = "{base_url}/{issue_key}/attachments".format(base_url=base_url, issue_key=issue_key)
         if attachment:
@@ -1472,13 +1472,13 @@ def delete_issue(self, issue_id_or_key, delete_subtasks=True):
         else:
             params["deleteSubtasks"] = "false"
 
-        log.warning("Removing issue %s...", issue_id_or_key)
+        log.info("Removing issue %s...", issue_id_or_key)
 
         return self.delete(url, params=params)
 
     # @todo merge with edit_issue method
     def issue_update(self, issue_key, fields):
-        log.warning('Updating issue "%s" with "%s"', issue_key, fields)
+        log.info('Updating issue "%s" with "%s"', issue_key, fields)
         base_url = self.resource_url("issue")
         url = "{base_url}/{issue_key}".format(base_url=base_url, issue_key=issue_key)
         return self.put(url, data={"fields": fields})
@@ -1512,7 +1512,7 @@ def issue_add_watcher(self, issue_key, user):
         :param user:
         :return:
         """
-        log.warning('Adding user %s to "%s" watchers', user, issue_key)
+        log.info('Adding user %s to "%s" watchers', user, issue_key)
         data = user
         base_url = self.resource_url("issue")
         return self.post(
@@ -1527,7 +1527,7 @@ def issue_delete_watcher(self, issue_key, user):
         :param user:
         :return:
         """
-        log.warning('Deleting user %s from "%s" watchers', user, issue_key)
+        log.info('Deleting user %s from "%s" watchers', user, issue_key)
         params = {"username": user}
         base_url = self.resource_url("issue")
         return self.delete(
@@ -1613,7 +1613,7 @@ def create_issues(self, list_of_issues_data):
 
     # @todo refactor and merge with create_issue method
     def issue_create(self, fields):
-        log.warning('Creating issue "%s"', fields["summary"])
+        log.info('Creating issue "%s"', fields["summary"])
         url = self.resource_url("issue")
         return self.post(url, data={"fields": fields})
 
@@ -2141,7 +2141,7 @@ def user_create(self, username, email, display_name, password=None, notification
                              Default:false.
         :return:
         """
-        log.warning("Creating user %s", display_name)
+        log.info("Creating user %s", display_name)
         data = {
             "name": username,
             "emailAddress": email,

atlassian/service_desk.py

@@ -56,7 +56,7 @@ def create_customer(self, full_name, email):
         :param email: str - email address
         :return: New customer
         """
-        log.warning("Creating customer...")
+        log.info("Creating customer using create_customer method...")
         data = {"fullName": full_name, "email": email}
 
         return self.post(
@@ -236,7 +236,7 @@ def perform_transition(self, issue_id_or_key, transition_id, comment=None):
         :param comment: OPTIONAL: str
         :return: None
         """
-        log.warning("Performing transition...")
+        log.info("Performing transition for issue: " + issue_id_or_key)
         data = {"id": transition_id, "additionalComment": {"body": comment}}
         url = "rest/servicedeskapi/request/{}/transition".format(issue_id_or_key)
 
@@ -373,7 +373,7 @@ def create_organization(self, name):
         :param name: str
         :return: Organization data
         """
-        log.warning("Creating organization...")
+        log.info("Creating organization: " + name)
         url = "rest/servicedeskapi/organization"
         data = {"name": name}
 
@@ -387,7 +387,7 @@ def add_organization(self, service_desk_id, organization_id):
         :param organization_id: int
         :return:
         """
-        log.warning("Adding organization...")
+        log.info("Adding organization...")
         url = "rest/servicedeskapi/servicedesk/{}/organization".format(service_desk_id)
         data = {"organizationId": organization_id}
 
@@ -401,7 +401,7 @@ def remove_organization(self, service_desk_id, organization_id):
         :param organization_id: int
         :return:
         """
-        log.warning("Removing organization...")
+        log.info("Removing organization...")
         url = "rest/servicedeskapi/servicedesk/{}/organization".format(service_desk_id)
         data = {"organizationId": organization_id}
 
@@ -414,7 +414,7 @@ def delete_organization(self, organization_id):
         :param organization_id:
         :return:
         """
-        log.warning("Deleting organization...")
+        log.info("Deleting organization: %s ", organization_id)
         url = "rest/servicedeskapi/organization/{}".format(organization_id)
 
         return self.delete(url, headers=self.experimental_headers)
@@ -430,7 +430,7 @@ def add_users_to_organization(self, organization_id, users_list=[], account_list
         :param users_list: list
         :return:
         """
-        log.warning("Adding users...")
+        log.info("Adding users: %s ", str(users_list))
         url = "rest/servicedeskapi/organization/{}/user".format(organization_id)
         data = {"usernames": users_list, "accountIds": account_list}
 
@@ -447,7 +447,7 @@ def remove_users_from_organization(self, organization_id, users_list=[], account
         :param account_list: list
         :return:
         """
-        log.warning("Removing users...")
+        log.info("Removing users: %s", str(users_list))
         url = "rest/servicedeskapi/organization/{}/user".format(organization_id)
         data = {"usernames": users_list, "accountIds": account_list}
 
@@ -901,7 +901,7 @@ def create_request_type(
         :param request_name: str
         :param request_description: str
         """
-        log.warning("Creating request type...")
+        log.info("Creating request type")
         data = {
             "issueTypeId": request_type_id,
             "name": request_name,