code review

HewlettPackard · Dec 15, 2023 · ba0cc0c · ba0cc0c
1 parent 43fbfcb
commit ba0cc0c
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 5 deletions.
diff --git a/docs/installation/kubernetes.md b/docs/installation/kubernetes.md
@@ -11,12 +11,12 @@ The Kubernetes deployment is wrapped by Ansible. The code has been tested with A
 Install Ansible dependencies:
 
 ```bash
-ansible-galaxy collection install kubernetes.core
+ansible-galaxy install -r k8s/requirements.yml
 ```
 
 Install Python dependencies:
 ```bash
-pip3 install kubernetes
+pip3 install -r k8s/requirements.txt
 ```
 
 ## Ansible inventory

diff --git a/k8s/backup.yml b/k8s/backup.yml
@@ -13,7 +13,7 @@
   tasks:
     - name: Generate job name
       ansible.builtin.set_fact:
-        job_name: backup-{{ ansible_date_time.date }}
+        job_name: "backup-{{ ansible_date_time.iso8601 }}"
 
     - name: Create Job from backup cronjob
       shell: "kubectl --namespace {{ squest_namespace }} create job --from=cronjob/squest-backup {{ job_name }}"
@@ -42,7 +42,7 @@
       block:
         - name: Generate job name
           ansible.builtin.set_fact:
-            job_name: rsync-backup-{{ ansible_date_time.date }}
+            job_name: "rsync-backup-{{ ansible_date_time.iso8601 }}"
 
         - name: Create Job from rsync backup cronjob
           shell: "kubectl --namespace {{ squest_namespace }} create job --from=cronjob/squest-rsync-backup {{ job_name }}"

diff --git a/k8s/deploy.yml b/k8s/deploy.yml
@@ -2,6 +2,5 @@
   hosts: localhost
   gather_facts: False
 
-
   roles:
     - role: squest_k8s
diff --git a/k8s/requirements.txt b/k8s/requirements.txt
@@ -0,0 +1 @@
+kubernetes==24.2.0
diff --git a/k8s/requirements.yml b/k8s/requirements.yml
@@ -0,0 +1,3 @@
+collections:
+  - name: kubernetes.core
+    version: 2.3.2