akuity / kargo

Application lifecycle orchestration
https://kargo.akuity.io/
Apache License 2.0
1.55k stars 133 forks source link

Kargo updates helm timestamp every 5 minute with a new commit on GitHub pull request #2226

Closed akselleirv closed 1 day ago

akselleirv commented 3 months ago

Checklist

Description

Kargo will constantly add commits to a pull request when using Helm chart.

It updates the Chart.lock file with a need timestamp on the generated field every five minute: generated: "2024-07-01T07:01:59.944058789Z"

Steps to Reproduce

apiVersion: kargo.akuity.io/v1alpha1
kind: Warehouse
metadata:
  name: example
  namespace: kargo
spec:
  subscriptions:
    - chart:
        repoURL: oci://my-acr.azurecr.io/helm-charts/my-app
---
apiVersion: kargo.akuity.io/v1alpha1
kind: Stage
metadata:
  name: test
  namespace: kargo
spec:
  subscriptions:
    warehouse: example
  promotionMechanisms:
    gitRepoUpdates:
      - repoURL: https://github.com/akselleirv/fake-example.git
        writeBranch: main
        pullRequest:
          github: { }
        helm:
          charts:
            - chartPath: my-app-dir
              name: my-app
              repository: oci://my-acr.azurecr.io/helm-charts

Version

v0.7.1

Logs

Paste any relevant application logs here.
hiddeco commented 3 months ago

I think this touches the same area/problem as #2178.

krancour commented 1 month ago

I am adding this to v0.9.0, as I anticipate that work on #2219 will naturally solve this.

krancour commented 4 days ago

@hiddeco will this still happen with the helm-template promotion step?