Skip to content

Commit

Permalink
fix: national DEM workflow should have a JIRA associated TDE-1162 (#897)
Browse files Browse the repository at this point in the history
#### Motivation

When the national DEM dataset is published, a GitHub PR is created in
https://github.com/linz/elevation/ and its branch name needs a JIRA
ticket number. Moreover, the JIRA ticket number is usefull to filter
workflows.

#### Modification

- Add a ticket label to the national-dem workflow
- specify a JIRA ticket in the cron workflow

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [ ] Issue linked in Title
  • Loading branch information
paulfouquet authored Dec 10, 2024
1 parent 0ad9728 commit 4fe8cc1
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
5 changes: 5 additions & 0 deletions workflows/cron/cron-national-dem.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ spec:
successfulJobsHistoryLimit: 3
failedJobsHistoryLimit: 3
suspend: false
workflowMetadata:
labels:
linz.govt.nz/ticket: 'TDE-1130'
workflowSpec:
podMetadata:
labels:
Expand All @@ -31,3 +34,5 @@ spec:
value: 'true'
- name: 'copy_option'
value: '--force-no-clobber'
- name: 'ticket'
value: 'TDE-1130'
1 change: 1 addition & 0 deletions workflows/raster/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -298,6 +298,7 @@ Publishing to the AWS Registry of Open Data is an optional step [publish-odr](#P

| Parameter | Type | Default | Description |
| -------------- | ---- | ------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| ticket | str | | Ticket ID e.g. 'TDE-1130' |
| config_file | str | https://raw.githubusercontent.com/linz/basemaps-config/master/config/tileset/elevation.json | Location of the configuration file listing the source datasets to merge. |
| odr_url | str | | (Optional) If an existing dataset add the S3 path to the dataset here to load existing metadata e.g. "s3://nz-elevation/new-zealand/new-zealand/dem_1m/2193/" |
| group | 2 | | How many output tiles to process in each standardising task "pod". Change if you have resource or performance issues when standardising a dataset. |
Expand Down
6 changes: 6 additions & 0 deletions workflows/raster/national-dem.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ spec:
workflowMetadata:
labels:
linz.govt.nz/region: 'new-zealand'
labelsFrom:
linz.govt.nz/ticket:
expression: workflow.parameters.ticket
podMetadata:
labels:
linz.govt.nz/category: raster
Expand All @@ -32,6 +35,9 @@ spec:
- name: version_topo_imagery
description: 'Specify a version of the topo-imagery image to use, e.g. "v4.8" or "latest"'
value: 'v7'
- name: ticket
description: 'Ticket ID, e.g. "TDE-1130"'
value: ''
- name: config_file
description: 'Location of the configuration file listing the source datasets to merge.'
value: 'https://raw.githubusercontent.com/linz/basemaps-config/master/config/tileset/elevation.json'
Expand Down

0 comments on commit 4fe8cc1

Please sign in to comment.