Skip to content

Commit

Permalink
This commit adds the patch fix
Browse files Browse the repository at this point in the history
  • Loading branch information
EmanElsaban committed Jan 16, 2024
1 parent bd13bab commit 8713032
Show file tree
Hide file tree
Showing 2 changed files with 83 additions and 1 deletion.
81 changes: 81 additions & 0 deletions contrib/patch-config-loggers.diff
Original file line number Diff line number Diff line change
@@ -0,0 +1,81 @@
--- a/debian/tron/opt/venvs/tron/lib/python3.8/site-packages/kubernetes/client/configuration.py
+++ b/debian/tron/opt/venvs/tron/lib/python3.8/site-packages/kubernetes/client/configuration.py
@@ -71,11 +71,11 @@
"""

_default = None
-
def __init__(self, host="http://localhost",
api_key=None, api_key_prefix=None,
username=None, password=None,
discard_unknown_keys=False,
+ is_logger_used=False,
):
"""Constructor
"""
@@ -106,26 +106,28 @@
"""Password for HTTP basic authentication
"""
self.discard_unknown_keys = discard_unknown_keys
+ self.is_logger_used = is_logger_used
self.logger = {}
- """Logging Settings
- """
- self.logger["package_logger"] = logging.getLogger("client")
- self.logger["urllib3_logger"] = logging.getLogger("urllib3")
- self.logger_format = '%(asctime)s %(levelname)s %(message)s'
- """Log format
- """
- self.logger_stream_handler = None
- """Log stream handler
- """
- self.logger_file_handler = None
- """Log file handler
- """
- self.logger_file = None
- """Debug file location
- """
- self.debug = False
- """Debug switch
- """
+ if self.is_logger_used:
+ """Logging Settings
+ """
+ self.logger["package_logger"] = logging.getLogger("client")
+ self.logger["urllib3_logger"] = logging.getLogger("urllib3")
+ self.logger_format = '%(asctime)s %(levelname)s %(message)s'
+ """Log format
+ """
+ self.logger_stream_handler = None
+ """Log stream handler
+ """
+ self.logger_file_handler = None
+ """Log file handler
+ """
+ self.logger_file = None
+ """Debug file location
+ """
+ self.debug = False
+ """Debug switch
+ """

self.verify_ssl = True
"""SSL/TLS verification
@@ -178,11 +180,12 @@
for k, v in self.__dict__.items():
if k not in ('logger', 'logger_file_handler'):
setattr(result, k, copy.deepcopy(v, memo))
- # shallow copy of loggers
- result.logger = copy.copy(self.logger)
- # use setters to configure loggers
- result.logger_file = self.logger_file
- result.debug = self.debug
+ if self.is_logger_used:
+ # shallow copy of loggers
+ result.logger = copy.copy(self.logger)
+ # use setters to configure loggers
+ result.logger_file = self.logger_file
+ result.debug = self.debug
return result

@classmethod
3 changes: 2 additions & 1 deletion debian/rules
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ override_dh_virtualenv:
--preinstall setuptools==46.1.3 \
--pip-tool pip-custom-platform \
--extra-pip-arg "-vvv"

@echo patching k8s client lib for configuration class
patch debian/tron/opt/venvs/tron/lib/python3.8/site-packages/kubernetes/client/configuration.py contrib/patch-config-loggers.diff
override_dh_installinit:
dh_installinit --noscripts

0 comments on commit 8713032

Please sign in to comment.