Skip to content

[prometheus-artifactory-exporter] - bump helm chart version to 0.9.0#29

Merged
peimanja merged 6 commits into
peimanja:mainfrom
joawin:bump_helm_version_0.9.0
Sep 5, 2025
Merged

[prometheus-artifactory-exporter] - bump helm chart version to 0.9.0#29
peimanja merged 6 commits into
peimanja:mainfrom
joawin:bump_helm_version_0.9.0

Conversation

@joawin

@joawin joawin commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

This PR updates the Artifactory Exporter Version in the Helm Chart to v1.16.0

Which issue this PR fixes

Application Changelog peimanja/artifactory_exporter@v1.15.1...v1.16.0

Special notes for your reviewer:

Checklist

  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-artifactory-exporter])

@joawin

joawin commented Aug 11, 2025

Copy link
Copy Markdown
Contributor Author

@peimanja please approve :-)

@peimanja

Copy link
Copy Markdown
Owner

@joawin thanks for the contribution. We might wanna add the feature and flags that were added in https://github.com/peimanja/artifactory_exporter/pull/168/files#diff-fe44f09c4d5977b5f5eaea29170b6a0748819c9d02271746a20d81a5f3efca17R26-R28

This can be added in options like:

options:
  useCache: false
  cacheTimeout: 30s
  cacheTTL: 5m

and then implement the conditions in deployment args

@joawin

joawin commented Sep 4, 2025

Copy link
Copy Markdown
Contributor Author

Hi @peimanja ,
now it looks good.

Comment thread charts/prometheus-artifactory-exporter/values.yaml Outdated

@peimanja peimanja left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution

@peimanja peimanja merged commit 2c66710 into peimanja:main Sep 5, 2025
3 checks passed
@joawin joawin deleted the bump_helm_version_0.9.0 branch October 23, 2025 11:52
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