Skip to content

feat(Jobs): Allow to configure Schedules in kubernetes job configuration#250

Merged
guillaume-chervet merged 9 commits intomainfrom
feature/configure-schedules-in-configuration
Mar 17, 2026
Merged

feat(Jobs): Allow to configure Schedules in kubernetes job configuration#250
guillaume-chervet merged 9 commits intomainfrom
feature/configure-schedules-in-configuration

Conversation

@Silthian
Copy link
Collaborator

@Silthian Silthian commented Mar 6, 2026

Silthian added 2 commits March 6, 2026 17:27
Signed-off-by: David Majorczyk <jan.nachtingal@gmail.com>
Signed-off-by: David Majorczyk <jan.nachtingal@gmail.com>
@guillaume-chervet
Copy link
Collaborator

@Silthian did you manage to test it?

@Silthian
Copy link
Collaborator Author

Silthian commented Mar 11, 2026

@Silthian did you manage to test it?

Nope, try to test with podman desktop and minikube but got some errors on slimfaas pods.

@guillaume-chervet : I test it and it's OK

SlimFaas/DefaultVisibility: "Public"
SlimFaas/NumberParallelJob: "1"
SlimFaas/DependsOn: "fibonacci1,fibonacci2"
SlimFaas/Schedules: '[{"Schedule":"0 0 * * *","Args":[]}]'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SlimFaas/Schedules: '[{"Schedule":"0/2 0 * * *","Args":["39"]}]' would be better

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to 0/2 * * * * (every 2 minutes).
Update the jobs.md doc to reflect it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to */2 * * * * (every 2 minutes) as internal cron parser considers 0/2 * * * * as 0 * * * *
Also update the documentation.

Signed-off-by: David Majorczyk <jan.nachtingal@gmail.com>
@guillaume-chervet
Copy link
Collaborator

let discuss it in minutes slimfaas :)

Signed-off-by: David Majorczyk <jan.nachtingal@gmail.com>
@Silthian
Copy link
Collaborator Author

let discuss it in minutes slimfaas :)

There si still an error image_not_allowed when enqueuing the job.

@guillaume-chervet
Copy link
Collaborator

Unit test seems to fail @Silthian

Signed-off-by: David Majorczyk <jan.nachtingal@gmail.com>
@sonarqubecloud
Copy link

@Silthian
Copy link
Collaborator Author

Unit test seems to fail @Silthian

sorry, fix the old test and a new one to check that image is added to whitelist if not present

@guillaume-chervet guillaume-chervet merged commit af25e0c into main Mar 17, 2026
27 checks passed
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.

2 participants