From 9a0c15dd179396c36037a4595ac2907128af773c Mon Sep 17 00:00:00 2001 From: Andrew Jacombs Date: Wed, 23 Aug 2023 14:38:30 +1200 Subject: [PATCH] feat: update default target values for basemaps import BM-822 (#156) #### Description Changes the defaults for basemaps imagery import cogify workflow to be full s3:// locations. #### Intention The previous values were taken from the non-cogify import workflow, which used the `make-cog` command. This has a magic `--aws` flag which transforms the targets into s3:// bucket locations. The equivalent cogify commands on the other hand require the actual path itself. #### Checklist - [ ] Tests updated - [ ] Docs updated - [x] Issue linked in Title --- workflows/basemaps/imagery-import-cogify.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/workflows/basemaps/imagery-import-cogify.yml b/workflows/basemaps/imagery-import-cogify.yml index 3154e91e..5f64ec38 100644 --- a/workflows/basemaps/imagery-import-cogify.yml +++ b/workflows/basemaps/imagery-import-cogify.yml @@ -30,10 +30,10 @@ spec: - name: target description: Target location for output COGs - value: "linz-basemaps" + value: "s3://linz-basemaps/" enum: - - "linz-basemaps" - - "linz-basemaps-staging" + - "s3://linz-basemaps/" + - "s3://linz-basemaps-staging/" - name: tile_matrix description: Output tile matrix, ";" separated list