PB-8755: Add labels for backup schedule request

ansible-collection/examples/backup/create.yaml

@@ -45,6 +45,7 @@
             volume_snapshot_class_mapping: "{{ item.volume_snapshot_class_mapping | default(omit) }}"
             direct_kdmp: "{{ item.direct_kdmp | default(omit) }}"
             validate_certs: "{{ item.validate_certs | default(true) }}"
+            labels: "{{ item.labels | default(omit) }}"
           loop: "{{ backups }}"
           register: backup_result
           loop_control:

ansible-collection/examples/backup_schedule/create.yaml

@@ -36,10 +36,11 @@
             cluster_ref: "{{ item.cluster_ref }}"
             backup_object_type: "{{ item.backup_object_type }}"
             skip_vm_auto_exec_rules: "{{ item.skip_vm_auto_exec_rules | default(omit) }}"
-            volume_snapshot_class_mapping: "{{ item.volume_snapshot_class_mapping }}"
+            volume_snapshot_class_mapping: "{{ item.volume_snapshot_class_mapping | default(omit) }}"
             direct_kdmp: "{{ item.direct_kdmp }}"
             owner: "admin"
             exclude_resource_types: "{{ item.exclude_resource_types | default(omit) }}"
+            labels: "{{ item.labels | default(omit) }}"
           loop: "{{ backup_schedules }}"
           register: backup_result
           loop_control:

ansible-collection/examples/backup_schedule/update.yaml

@@ -5,7 +5,7 @@
 
   vars_files:
     - "{{ inventory_dir }}/group_vars/common/all.yaml"
-    - "{{ inventory_dir }}/group_vars/backup_schedule/update.yaml"
+    - "{{ inventory_dir }}/group_vars/backup_schedule/update_vars.yaml"
 
   pre_tasks:
     - name: Validate required variables
@@ -20,6 +20,8 @@
         current_timestamp: "{{ lookup('pipe', 'date +%Y-%m-%d-%H-%M-%S') }}"
 
   tasks:
+    - name: Login and fetch Px-Backup token
+      include_tasks: "{{ playbook_dir | dirname }}/auth/auth.yaml"
     - name: Backup and update schedules
       block:
         - name: Get current configurations
@@ -36,6 +38,7 @@
             backup_location_ref: "{{ item.backup_location_ref }}"
             pre_exec_rule_ref: "{{ item.pre_exec_rule_ref | default(omit) }}"
             post_exec_rule_ref: "{{ item.post_exec_rule_ref | default(omit) }}"
+            labels: "{{ item.labels | default(omit) }}"
           register: current_configs
           loop: "{{ backup_schedules_update }}"
           loop_control:
@@ -64,6 +67,7 @@
             backup_location_ref: "{{ item.backup_location_ref }}"
             pre_exec_rule_ref: "{{ item.pre_exec_rule_ref | default(omit) }}"
             post_exec_rule_ref: "{{ item.post_exec_rule_ref | default(omit) }}"
+            labels: "{{ item.labels | default(omit) }}"
           register: update_result
           loop: "{{ backup_schedules_update }}"
           loop_control:
@@ -92,6 +96,7 @@
         backup_location_ref: "{{ item.backup_location_ref }}"
         pre_exec_rule_ref: "{{ item.pre_exec_rule_ref | default(omit) }}"
         post_exec_rule_ref: "{{ item.post_exec_rule_ref | default(omit) }}"
+        labels: "{{ item.labels | default(omit) }}"
       register: final_validation
       loop: "{{ backup_schedules_update }}"
       loop_control:

ansible-collection/inventory/group_vars/backup_schedule/inspect_vars.yaml

@@ -0,0 +1,8 @@
+---
+# Backup schedules to inspect
+backup_schedules_inspect:
+  - name: "bs-1"
+    uid: "uid-1"
+
+  - name: "bl-2"
+    uid: "uid-1"

ansible-collection/inventory/group_vars/backup_schedule/update.yaml → ansible-collection/inventory/group_vars/backup_schedule/update_vars.yaml

@@ -20,7 +20,8 @@ backup_schedules_update:
     backup_location_ref:
       name: "s3-backup-loc"
       uid: "backup-loc-uid"
-
+    labels:
+      key: value
 
 # Global update configuration
 backup_configs: true

ansible-collection/plugins/modules/px_backup/backup_schedule.py

@@ -367,6 +367,9 @@ def backup_schedule_request_body(module):
     if module.params.get('volume_snapshot_class_mapping'):
         backup_schedule_request['volume_snapshot_class_mapping'] = module.params['volume_snapshot_class_mapping']
 
+    if module.params.get('labels'):
+        backup_schedule_request['metadata']['labels'] = module.params['labels']
+
     return backup_schedule_request
 
 def run_module():