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
Add lifecycle to core dpl #1739
base: main
Are you sure you want to change the base?
Conversation
@MinerYang @zyyw |
Hi @derekcha , thanks for contributing to harbor-helm. Could you please help to resolve the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for consistancy reasons, It would make sense to support lifecycle on all pods not just the core.
2858fb3
to
59f9955
Compare
Signed-off-by: derek <derek@dunamu.com>
Signed-off-by: derek <derek@dunamu.com>
…, trivy Signed-off-by: derek <derek@dunamu.com>
4ab04f8
to
d0cee30
Compare
@zyyw Are you still discussing this PR? |
Hello,
I'm using Harbor so well, and I'm glad I was able to raise a PR like this.
What improvements does these changes make?
A 504 or EOF error occurred when rolling while using Harbor.
It looks like the pods aren't being cleaned up properly on shutdown (possibly because there is a port after AWS LB).
So I tested randomly setting the prestop(sleep 60) lifecycle for containers in Harbor Core and no errors occurred.
So, I am uploading a PR so that lifecycle settings can be done through this chart.
I will write down the link to the issue I registered below.
#1722