charts icon indicating copy to clipboard operation
charts copied to clipboard

Add time and remove isCA for a clean argo sync

Open zoe-tek opened this issue 7 months ago • 1 comments

right now when syncing this plugin the kind: Certificate duration and renewBefore adds the minutes and seconds. I changed it to duration: 2160h0m0s renewBefore: 360h0m0s

It also removes "isCA: false" as that is the default

zoe-tek avatar Oct 08 '25 17:10 zoe-tek

PR

zoe-tek avatar Oct 08 '25 18:10 zoe-tek

Hi, @zoe-tek. I'm Dosu, and I'm helping the charts team manage their backlog and am marking this issue as stale.

Issue Summary:

  • You proposed updating the Argo sync configuration to use hour-based durations for Certificate duration and renewBefore fields.
  • You suggested removing the default "isCA: false" field for a cleaner configuration.
  • You linked a related pull request (#686) that addresses these changes.
  • There have been no further comments or reactions on this issue since then.
  • The linked PR appears to resolve the proposed changes.

Next Steps:

  • Please let me know if this issue is still relevant to the latest version of the charts repository by commenting here.
  • Otherwise, I will automatically close this issue in 7 days.

Thank you for your understanding and contribution!

dosubot[bot] avatar Jan 23 '26 16:01 dosubot[bot]

PR is still waiting to be merged.

zoe-tek avatar Jan 26 '26 18:01 zoe-tek

@sxd The user mentioned that the PR is still waiting to be merged and the issue remains relevant. Could you please assist with this?

dosubot[bot] avatar Jan 26 '26 18:01 dosubot[bot]