Skip to content

Commit

Permalink
Fix backup application path (redhat-appstudio#2723)
Browse files Browse the repository at this point in the history
The original change introducing backup component was using clusterDir
which is not set. Instead of defining the clusterDir for each possible
cluster as it is done in other places:
```
- list:
    elements:
      - nameNormalized: stone-stg-m01
        values.clusterDir: stone-stg-m01
      - nameNormalized: stone-stg-rh01
        values.clusterDir: stone-stg-rh01
      - nameNormalized: stone-prd-m01
        values.clusterDir: stone-prd-m01
      - nameNormalized: stone-prd-rh01
        values.clusterDir: stone-prd-rh01
```
Simply use the normalized name of the cluster which correspond to the
folder structure within the backup component. This is a better solution
as we do not need to hardcode the cluster names in the backup component.

RHTAPSRE-215

Signed-off-by: Hugo Ares <hares@redhat.com>
  • Loading branch information
hugares authored and jhutar committed Nov 22, 2023
1 parent 50841fb commit 3b5bb05
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ spec:
values:
sourceRoot: components/backup
environment: staging
clusterDir: ""
- list:
elements: []
template:
Expand All @@ -21,7 +20,7 @@ spec:
spec:
project: default
source:
path: '{{values.sourceRoot}}/{{values.environment}}/{{values.clusterDir}}'
path: '{{values.sourceRoot}}/{{values.environment}}/{{nameNormalized}}'
repoURL: https://github.com/redhat-appstudio/infra-deployments.git
targetRevision: main
destination:
Expand Down

0 comments on commit 3b5bb05

Please sign in to comment.