Skip to content

Commit

Permalink
fix: k8s write secret proper exception (#2866)
Browse files Browse the repository at this point in the history
  • Loading branch information
talboren authored Dec 18, 2024
1 parent 8eb620d commit 6f6c50f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
7 changes: 4 additions & 3 deletions keep/secretmanager/kubernetessecretmanager.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

import kubernetes.client
import kubernetes.config
from kubernetes.client.rest import ApiException
from kubernetes.client.exceptions import ApiException

from keep.secretmanager.secretmanager import BaseSecretManager

Expand Down Expand Up @@ -32,7 +32,7 @@ def write_secret(self, secret_name: str, secret_value: str) -> None:
ApiException: If an error occurs while writing the secret.
"""
# k8s requirements: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
secret_name = secret_name.replace("_", "-")
secret_name = secret_name.replace("_", "-").lower()
self.logger.info("Writing secret", extra={"secret_name": secret_name})

body = kubernetes.client.V1Secret(
Expand Down Expand Up @@ -70,7 +70,7 @@ def write_secret(self, secret_name: str, secret_value: str) -> None:

def read_secret(self, secret_name: str, is_json: bool = False) -> str | dict:
# k8s requirements: https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names
secret_name = secret_name.replace("_", "-")
secret_name = secret_name.replace("_", "-").lower()
self.logger.info("Getting secret", extra={"secret_name": secret_name})
try:
response = self.api.read_namespaced_secret(
Expand All @@ -91,6 +91,7 @@ def read_secret(self, secret_name: str, is_json: bool = False) -> str | dict:
raise

def delete_secret(self, secret_name: str) -> None:
secret_name = secret_name.replace("_", "-").lower()
self.logger.info("Deleting secret", extra={"secret_name": secret_name})
try:
self.api.delete_namespaced_secret(
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[tool.poetry]
name = "keep"
version = "0.32.1"
version = "0.32.2"
description = "Alerting. for developers, by developers."
authors = ["Keep Alerting LTD"]
packages = [{include = "keep"}]
Expand Down

0 comments on commit 6f6c50f

Please sign in to comment.