Skip to content

fix: harbor retention rule latestPushed is optional#4108

Open
rocketeerbkw wants to merge 1 commit into
mainfrom
fix-optional-latestpushed
Open

fix: harbor retention rule latestPushed is optional#4108
rocketeerbkw wants to merge 1 commit into
mainfrom
fix-optional-latestpushed

Conversation

@rocketeerbkw

Copy link
Copy Markdown
Member

General Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated
  • PR title is ready for inclusion in changelog

Database Migrations

n/a

Description

Follow up to #4074. latestPushed is supposed to be optional, but leaving it off throws an error:

createHarborRetentionPolicy(input:{
    name: "harbor-policy"
    enabled: true
    rules: [
      {
        name: "all branches, excluding pullrequests"
        pattern: "[^pr-]*/*"
        latestPulled: 3
      },
      {
        name: "pullrequests"
        pattern: "pr-*"
        latestPulled: 1
      }
    ]
    schedule: "3 * * * *"
  }) {
    id
    name
    configuration {
      enabled
      rules {
        name
        pattern
        latestPulled
      }
      schedule
    }
    created
    updated
  }
Error: Provided harbor configuration is not valid: Error: all branches, excluding pullrequests: latestPushed must be a number

Closing issues

n/a

@github-actions

Copy link
Copy Markdown

Overview

Image reference testlagoon/api:main lagoon/api:ci-latest
- digest 7d7560b8cd0d 3552effdec88
- tag main ci-latest
- vulnerabilities critical: 5 high: 83 medium: 73 low: 11 critical: 5 high: 83 medium: 73 low: 11
- platform linux/amd64 linux/amd64
- size 152 MB 183 MB (+31 MB)
- packages 1568 1568
Base Image node:22-alpine
also known as:
22-alpine3.23
22.22-alpine
22.22-alpine3.23
22.22.3-alpine
22.22.3-alpine3.23
jod-alpine
jod-alpine3.23
node:22-alpine
also known as:
22-alpine3.23
22.22-alpine
22.22-alpine3.23
22.22.3-alpine
22.22.3-alpine3.23
jod-alpine
jod-alpine3.23
- vulnerabilities critical: 1 high: 9 medium: 9 low: 2 critical: 1 high: 9 medium: 9 low: 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants