mirror of
https://github.com/ansible/awx-operator.git
synced 2026-03-26 21:33:14 +00:00
56f10cf966
Use backup_pvc for custom backup PVC name in templates When backup_pvc is specified with create_backup_pvc: true, the PVC template and ownerReference removal used the hardcoded default name (deployment_name-backup-claim) instead of the user-specified name. This caused the management pod to reference a PVC that didn't exist. Replace backup_claim variable with backup_pvc throughout the backup role so the resolved PVC name is used consistently in all templates. Authored By: Adam Knochowski <aknochow@redhat.com> Assisted By: Claude
96 lines
3.1 KiB
YAML
96 lines
3.1 KiB
YAML
---
|
|
- name: Delete any existing management pod
|
|
k8s:
|
|
name: "{{ ansible_operator_meta.name }}-db-management"
|
|
kind: Pod
|
|
namespace: "{{ backup_pvc_namespace }}"
|
|
state: absent
|
|
force: true
|
|
wait: true
|
|
|
|
# Check to make sure provided pvc exists, error loudly if not. Otherwise, the management pod will just stay in pending state forever.
|
|
- name: Check provided PVC exists
|
|
k8s_info:
|
|
name: "{{ backup_pvc }}"
|
|
kind: PersistentVolumeClaim
|
|
namespace: "{{ backup_pvc_namespace }}"
|
|
register: provided_pvc
|
|
when:
|
|
- backup_pvc != ''
|
|
|
|
- name: Surface error to user
|
|
block:
|
|
- name: Set error message
|
|
set_fact:
|
|
error_msg: "{{ backup_pvc }} does not exist, please create this pvc first or ensure create_backup_pvc is set to true (default) for automatic backup_pvc creation."
|
|
|
|
- name: Handle error
|
|
import_tasks: error_handling.yml
|
|
|
|
- name: Fail early if pvc is defined but does not exist
|
|
fail:
|
|
msg: "{{ backup_pvc }} does not exist, please create this pvc first or ensure create_backup_pvc is set to true (default) for automatic backup_pvc creation."
|
|
when:
|
|
- backup_pvc != ''
|
|
- provided_pvc.resources | length == 0
|
|
- not create_backup_pvc | bool
|
|
|
|
# If backup_pvc is defined, use in management-pod.yml.j2
|
|
- name: Set default pvc name
|
|
set_fact:
|
|
_default_backup_pvc: "{{ deployment_name }}-backup-claim"
|
|
|
|
# by default, it will re-use the old pvc if already created (unless a pvc is provided)
|
|
- name: Set PVC to use for backup
|
|
set_fact:
|
|
backup_pvc: "{{ backup_pvc | default(_default_backup_pvc, true) }}"
|
|
|
|
- block:
|
|
- name: Create PVC for backup
|
|
k8s:
|
|
kind: PersistentVolumeClaim
|
|
definition: "{{ lookup('template', 'backup_pvc.yml.j2') }}"
|
|
|
|
- name: Remove PVC ownerReference
|
|
k8s:
|
|
definition:
|
|
apiVersion: v1
|
|
kind: PersistentVolumeClaim
|
|
metadata:
|
|
name: "{{ backup_pvc }}"
|
|
namespace: "{{ backup_pvc_namespace }}"
|
|
ownerReferences: null
|
|
when:
|
|
- (backup_pvc == '' or backup_pvc is not defined) or (create_backup_pvc | bool)
|
|
|
|
- name: Set default postgres image
|
|
set_fact:
|
|
_default_postgres_image: "{{ _postgres_image }}:{{_postgres_image_version }}"
|
|
|
|
- name: Set user provided postgres image
|
|
set_fact:
|
|
_custom_postgres_image: "{{ postgres_image }}:{{ postgres_image_version }}"
|
|
when:
|
|
- postgres_image | default([]) | length
|
|
- postgres_image_version is defined and postgres_image_version != ''
|
|
|
|
- name: Set Postgres image URL
|
|
set_fact:
|
|
_postgres_image: "{{ _custom_postgres_image | default(lookup('env', 'RELATED_IMAGE_AWX_POSTGRES')) | default(_default_postgres_image, true) }}"
|
|
|
|
- name: Create management pod from the template
|
|
k8s:
|
|
name: "{{ ansible_operator_meta.name }}-db-management"
|
|
kind: Pod
|
|
state: present
|
|
definition: "{{ lookup('template', 'management-pod.yml.j2') }}"
|
|
wait: true
|
|
|
|
- name: Look up details for this deployment
|
|
k8s_info:
|
|
api_version: "{{ api_version }}"
|
|
kind: "AWX"
|
|
name: "{{ deployment_name }}"
|
|
namespace: "{{ ansible_operator_meta.namespace }}"
|
|
register: this_awx
|