Skip to content

chore(plugindefinitions): increase helm chart and repository interval to 24hours#1908

Merged
abhijith-darshan merged 1 commit intomainfrom
chore/helm-chart-interval
Apr 10, 2026
Merged

chore(plugindefinitions): increase helm chart and repository interval to 24hours#1908
abhijith-darshan merged 1 commit intomainfrom
chore/helm-chart-interval

Conversation

@abhijith-darshan
Copy link
Copy Markdown
Contributor

Description

Increase HelmRepository and HelmChart interval to 24hrs.

HelmRepository .spec.internval is to periodical check the helm repository index (index.yaml)

This is not needed for OCI registries as there is no index.yaml

HelmChart .spec.interval is beneficial to periodically update the Helm chart when using version ranges or floating tags
ex >=1.2.0 or latest

In greenhouse these are pinned versions of OCI references so both resource's interval can be set to a long interval such as 24hrs.

The versions are propagated via (Cluster)PluginDefinition, so a change in version will trigger a reconcile on demand.

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

  • Related Issue # (issue)

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help
  • Separate ticket for tests # (issue/pr)

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Added to documentation?

  • 📜 README.md
  • 🤝 Documentation pages updated
  • 🙅 no documentation needed
  • (if applicable) generated OpenAPI docs for CRD changes

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@abhijith-darshan abhijith-darshan requested a review from a team as a code owner April 10, 2026 09:47
@abhijith-darshan abhijith-darshan changed the title (chore): increase chart timeout interval to 24hours chore(plugindefinitions): increase helm chart and repository interval to 24hours Apr 10, 2026
Comment thread go.mod
Signed-off-by: abhijith-darshan <abhijith.ravindra@sap.com>
@abhijith-darshan abhijith-darshan force-pushed the chore/helm-chart-interval branch from 0dae5e2 to 666af37 Compare April 10, 2026 09:57
@abhijith-darshan abhijith-darshan merged commit c7c0130 into main Apr 10, 2026
21 checks passed
@abhijith-darshan abhijith-darshan deleted the chore/helm-chart-interval branch April 10, 2026 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants