Skip to content

Commit

Permalink
fix: Use Kubernetes 23 and update Ingress API
Browse files Browse the repository at this point in the history
  • Loading branch information
faucomte97 committed Oct 24, 2022
1 parent 2fc042c commit ce0d346
Show file tree
Hide file tree
Showing 10 changed files with 121 additions and 114 deletions.
4 changes: 2 additions & 2 deletions Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ black = "*"
django-test-migrations = "==1.2.0"

[packages]
aimmo = {editable = true,path = "."}
kubernetes = "==24.2.0"
aimmo = {editable = true, path = "."}
kubernetes = "==23.6.0"
eventlet = "*"
docker = "<6"
wheel = "*"
Expand Down
175 changes: 90 additions & 85 deletions Pipfile.lock

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion aimmo-game-creator/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ verify_ssl = true
name = "pypi"

[packages]
kubernetes = "==24.2.0"
kubernetes = "==23.6.0"
docker = "<6"
aimmo-game-creator = { editable = true, path = "." }
google-cloud-logging = "*"
Expand Down
14 changes: 7 additions & 7 deletions aimmo-game-creator/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 6 additions & 4 deletions aimmo-game-creator/game_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -181,8 +181,9 @@ def _create_game_name(game_id):

def _add_path_to_ingress(self, game_id):
game_name = KubernetesGameManager._create_game_name(game_id)
backend = kubernetes.client.V1IngressBackend(game_name, 80)
path = kubernetes.client.V1HTTPIngressPath(backend, f"/{game_name}(/|$)(.*)", path_type="exact")
game_service = kubernetes.client.V1IngressServiceBackend(game_name, 80)
backend = kubernetes.client.V1IngressBackend(service=game_service)
path = kubernetes.client.V1HTTPIngressPath(backend, f"/{game_name}(/|$)(.*)", path_type="Prefix")

patch = [{"op": "add", "path": "/spec/rules/0/http/paths/-", "value": path}]

Expand All @@ -194,8 +195,9 @@ def _add_path_to_ingress(self, game_id):

def _remove_path_from_ingress(self, game_id):
game_name = KubernetesGameManager._create_game_name(game_id)
backend = kubernetes.client.V1IngressBackend(game_name, 80)
path = kubernetes.client.V1HTTPIngressPath(backend, f"/{game_name}(/|$)(.*)", path_type="exact")
game_service = kubernetes.client.V1IngressServiceBackend(game_name, 80)
backend = kubernetes.client.V1IngressBackend(service=game_service)
path = kubernetes.client.V1HTTPIngressPath(backend, f"/{game_name}(/|$)(.*)", path_type="Prefix")
try:
ingress = self.networking_api.list_namespaced_ingress("default").items[0]
# These exceptions are usually triggered locally where there is no ingress.
Expand Down
2 changes: 1 addition & 1 deletion aimmo-game-worker/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ verify_ssl = true
name = "pypi"

[packages]
kubernetes = "==24.2.0"
kubernetes = "==23.6.0"
docker = "<6"
aimmo-game-worker = {editable = true, path = "."}
restrictedpython = "==4.0.b7"
Expand Down
14 changes: 7 additions & 7 deletions aimmo-game-worker/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion aimmo-game/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ black = "*"
[packages]
aimmo-game = {editable = true,path = "."}
oauthlib = "==3.2.0"
kubernetes = "==24.2.0"
kubernetes = "==23.6.0"
docker = "<6"

[requires]
Expand Down
8 changes: 4 additions & 4 deletions aimmo-game/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions aimmo_runner/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"six",
"hypothesis",
"psutil",
"docker >= 3.5, < 3.6",
"kubernetes == 10.0.0",
"docker<6",
"kubernetes==23.6.0",
],
zip_safe=False,
)

0 comments on commit ce0d346

Please sign in to comment.