Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
3 of 4 tasks
akselleirv opened this issue Jul 1, 2024 · 3 comments · Fixed by #2599
Closed
3 of 4 tasks

Comments

@akselleirv
Copy link

akselleirv commented Jul 1, 2024

Checklist

  • I've searched the issue queue to verify this is not a duplicate bug report.
  • I've included steps to reproduce the bug.
  • I've pasted the output of kargo version.
  • I've pasted logs, if applicable.

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
Copy link
Contributor

hiddeco commented Jul 1, 2024

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

@krancour
Copy link
Member

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

@krancour krancour added this to the v0.9.0 milestone Aug 19, 2024
@krancour
Copy link
Member

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

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

Successfully merging a pull request may close this issue.

4 participants