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.

Sign up for GitHub

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

Jump to bottom

ArgoCD sync trying convert 87600h0m0s back to 87600h #190

Closed
kfirfer opened this issue Apr 5, 2023 · 3 comments · May be fixed by #273
Closed

ArgoCD sync trying convert 87600h0m0s back to 87600h #190

kfirfer opened this issue Apr 5, 2023 · 3 comments · May be fixed by #273
Assignees
@llamerada-jp
Labels
bug Something isn't working stale

Comments

@kfirfer
Copy link

kfirfer commented Apr 5, 2023

Hello

Im using argocd to deploy pvc-autoresizer
In argocd sync the certificate get into the cluster with the duration parameter as
duration: 87600h0m0s
Instead
duration: 87600h

What it means that the application in argo is always in a "not synced" state
Same goes to controller certificate

I guess to overcome it need to set minutes and seconds in the value in helm chart

Image attached:

Screenshot from 2023-04-05 11-41-26
@kfirfer kfirfer added the bug Something isn't working label Apr 5, 2023
@llamerada-jp
Copy link
Contributor

Hi,
I got the same problem. This is because of ArgoCD's problem. And an issue had been written. You can resolve it with the solution below.
argoproj/argo-cd#6008 (comment)

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2023

This issue has been automatically marked as stale because it has not had any activity for 30 days. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label May 6, 2023
@github-actions
Copy link
Contributor

This issue has been automatically closed due to inactivity. Please feel free to reopen this issue (or open a new one) if this still requires investigation. Thank you for your contribution.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale May 13, 2023
gozer added a commit to gozer/pvc-autoresizer that referenced this issue Jul 9, 2024
@gozer
fix: cert-manager duration expects XhYmZs format
4f2b90f 
It accepts shorter format like Yh, but the mutating webhook, if
installed/enabled will convert it to the full format, causing potential
endless reconciliation loops with tools like ArgoCD

Fixes topolvm#190
@gozer gozer mentioned this issue Jul 9, 2024
Open
gozer added a commit to gozer/pvc-autoresizer that referenced this issue Jul 9, 2024
@gozer
fix: cert-manager duration expects XhYmZs format
e3bc7c7 
It accepts shorter format like Yh, but the mutating webhook, if
installed/enabled will convert it to the full format, causing potential
endless reconciliation loops with tools like ArgoCD

Fixes topolvm#190

Signed-off-by: Philippe M. Chiasson <gozer@ectoplasm.org>
gozer added a commit to gozer/pvc-autoresizer that referenced this issue Jul 9, 2024
@gozer
fix: cert-manager duration expects XhYmZs format
054bff2 
It accepts shorter format like Yh, but the mutating webhook, if
installed/enabled will convert it to the full format, causing potential
endless reconciliation loops with tools like ArgoCD

Fixes topolvm#190

Signed-off-by: Philippe M. Chiasson <gozer@ectoplasm.org>
Signed-off-by: Philippe M. Chiasson <gozer@lacework.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@llamerada-jp llamerada-jp

Labels
bug Something isn't working stale
Projects
Development
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: cert-manager duration expects XhYmZs format gozer/pvc-autoresizer
2 participants
@llamerada-jp @kfirfer