Skip to content

Commit

Permalink
fix: loki s3 overrides (#365)
Browse files Browse the repository at this point in the history
## Description
Fixes the loki s3 setup by setting the proper overrides.

## Related Issue

Fixes #343 

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [ ] Test, docs, adr added or updated as needed
- [ ] [Contributor Guide
Steps](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md)(https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md#submitting-a-pull-request)
followed
  • Loading branch information
TristanHoladay authored Apr 23, 2024
1 parent da3eb5a commit 3545066
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 15 deletions.
14 changes: 9 additions & 5 deletions .github/bundles/uds-bundle.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,16 @@ packages:
path: credentials.useSecret
- name: VELERO_IRSA_ANNOTATION
description: "IRSA ARN annotation to use for Velero"
path: serviceAccount.server.annotations
path: serviceAccount.server.annotations.irsa/role-arn
loki:
loki:
values:
- path: loki.storage.s3.endpoint
value: ""
- path: loki.storage.s3.secretAccessKey
value: ""
- path: loki.storage.s3.accessKeyId
value: ""
variables:
- name: LOKI_CHUNKS_BUCKET
description: "The object storage bucket for Loki chunks"
Expand All @@ -39,12 +46,9 @@ packages:
- name: LOKI_ADMIN_BUCKET
description: "The object storage bucket for Loki admin"
path: loki.storage.bucketNames.admin
- name: LOKI_S3_ENDPOINT
description: "The S3 endpoint"
path: loki.storage.s3.endpoint
- name: LOKI_S3_REGION
description: "The S3 region"
path: loki.storage.s3.region
- name: LOKI_IRSA_ANNOTATION
description: "The irsa role annotation"
path: loki.serviceAccount.annotations
path: serviceAccount.annotations.irsa/role-arn
7 changes: 2 additions & 5 deletions .github/bundles/uds-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,9 @@ variables:
loki_ruler_bucket: ${ZARF_VAR_LOKI_S3_BUCKET}
loki_admin_bucket: ${ZARF_VAR_LOKI_S3_BUCKET}
loki_s3_region: ${ZARF_VAR_LOKI_S3_AWS_REGION}
loki_s3-endpoint: ""
loki_irsa_annotation:
eks.amazonaws.com/role-arn: ${ZARF_VAR_LOKI_S3_ROLE_ARN}
loki_irsa_annotation: ${ZARF_VAR_LOKI_S3_ROLE_ARN}
velero_use_secret: false
velero_irsa_annotation:
eks.amazonaws.com/role-arn: "${ZARF_VAR_VELERO_S3_ROLE_ARN}"
velero_irsa_annotation: "${ZARF_VAR_VELERO_S3_ROLE_ARN}"
velero_bucket: ${ZARF_VAR_VELERO_S3_BUCKET}
velero_bucket_region: ${ZARF_VAR_VELERO_S3_AWS_REGION}
velero_bucket_provider_url: ""
Expand Down
7 changes: 2 additions & 5 deletions tasks/iac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -141,12 +141,9 @@ tasks:
loki_ruler_bucket: ${LOKI_S3_BUCKET}
loki_admin_bucket: ${LOKI_S3_BUCKET}
loki_s3_region: ${LOKI_S3_AWS_REGION}
loki_s3-endpoint: ""
loki_irsa_annotation:
eks.amazonaws.com/role-arn: "${LOKI_S3_ROLE_ARN}"
loki_irsa_annotation: "${LOKI_S3_ROLE_ARN}"
velero_use_secret: false
velero_irsa_annotation:
eks.amazonaws.com/role-arn: "${VELERO_S3_ROLE_ARN}"
velero_irsa_annotation: ${VELERO_S3_ROLE_ARN}"
velero_bucket: ${VELERO_S3_BUCKET}
velero_bucket_region: ${VELERO_S3_AWS_REGION}
velero_bucket_provider_url: ""
Expand Down

0 comments on commit 3545066

Please sign in to comment.