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.

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

Set TTL for jobs #880

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Set TTL for jobs #880

wants to merge 1 commit into from

Conversation

vuldin
Copy link
Member

@vuldin vuldin commented Nov 20, 2023

This small change makes the redpanda-configuration and redpanda-post-upgrade jobs (and their associated pods) automatically be cleaned up after successful runs. This frees up their resources and is recommended (see note here.

Keeping in draft for now since I need to ensure the commit happens with the proper email address, and I need to add the change to the 2nd job.

@CLAassistant
Copy link

CLAassistant commented Nov 20, 2023

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@vuldin vuldin marked this pull request as draft November 20, 2023 02:52
@RafalKorepta
Copy link
Contributor

@vuldin Could you sign CLA?

@RafalKorepta
Copy link
Contributor

I still feel it might be not enough to scrape logs from those jobs in customer clusters. It will be very hard to debug such environment.

If you feel it should be available for customer then create as a variable in values.yaml

@alejandroEsc
Copy link
Contributor

I still feel it might be not enough to scrape logs from those jobs in customer clusters. It will be very hard to debug such environment.

If you feel it should be available for customer then create as a variable in values.yaml

I agree, this is why we have not done this yet, we would like logs in case of failures.

@vuldin
Copy link
Member Author

vuldin commented Dec 5, 2023

@vuldin Could you sign CLA?

Hmm, I've signed the CLA but this still shows up for some reason (no matter how many times I sign it).

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.

4 participants