helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

Custom cronjobs template

Open jaceksocha opened this issue 11 months ago • 9 comments

Hi

Is there a known way to overwrite global cronjob template?

I saw only use case with "create-test" api call https://docs.testkube.io/articles/creating-tests/#changing-the-default-cronjob-template-used-for-scheduled-test-execution but it is required in every api call and IU does not support it (afaik).

Greetings

jaceksocha avatar Feb 27 '24 08:02 jaceksocha

hey @jaceksocha I beleive @ypoplavs should add it to values.yml

vsukhin avatar Feb 27 '24 09:02 vsukhin

Thank you for fast reaction. When do you plan to release chart ?

Greetings

jaceksocha avatar Feb 28 '24 12:02 jaceksocha

@ypoplavs ?

vsukhin avatar Feb 28 '24 14:02 vsukhin

@jaceksocha let me discuss this with the team.

ypoplavs avatar Mar 01 '24 11:03 ypoplavs

@ypoplavs of course. I will be grateful for message if you future knew something in this case.

Greetings

jaceksocha avatar Mar 03 '24 21:03 jaceksocha

Hello @jaceksocha! The change was released. Please upgrade to the latest helm chart version.

ypoplavs avatar Mar 04 '24 11:03 ypoplavs

Thank you @ypoplavs , we are testing new chart 1.16.51 .

We found some potentially strange behaviour, chart launched with values containing:

      ...
       cronJobTemplate: |
         apiVersion: batch/v1
         kind: CronJob
         metadata:
           name: "{{ .Name }}"
     ...

results with template (after base64 decoding env from operator pod) shifted by one space

       apiVersion: batch/v1 <----- shift one space
      kind: CronJob
      metadata:
        name: "{{ .Name }}"

but when cronJobTemplate contained in values.yaml begins with yaml comment

      ...
       cronJobTemplate: |
         # comment ...
         apiVersion: batch/v1
         kind: CronJob
         metadata:
           name: "{{ .Name }}"
     ...

everything seems to works fine. Looks like yaml parsing problem somewhere in the middle.

Greetings

jaceksocha avatar Mar 08 '24 13:03 jaceksocha

Thanks for raising @jaceksocha! I will check that

ypoplavs avatar Mar 11 '24 15:03 ypoplavs

@ypoplavs ?

vsukhin avatar Mar 23 '24 07:03 vsukhin

?

vsukhin avatar May 16 '24 05:05 vsukhin