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

Configure default concurrency policy for CronJob #814

Closed
iliapolo opened this issue Jun 9, 2022 · 0 comments · Fixed by #1052
Closed

Configure default concurrency policy for CronJob #814

iliapolo opened this issue Jun 9, 2022 · 0 comments · Fixed by #1052
Assignees
@vinayak-kukreja
Labels
effort/small 1 day tops feature-request New/Enhanced functionality wanted priority/p1 Should be on near term plans
Projects
General Availability | cdk8s-plus

Comments

@iliapolo
Copy link
Member

iliapolo commented Jun 9, 2022

Should be set to Forbid.

Additional resources
@iliapolo iliapolo added priority/p1 Should be on near term plans effort/small 1 day tops feature-request New/Enhanced functionality wanted labels Jun 9, 2022
@iliapolo iliapolo added this to To do in General Availability | cdk8s-plus via automation Jun 9, 2022
@iliapolo iliapolo moved this from To do to In progress in General Availability | cdk8s-plus Aug 15, 2022
@vinayak-kukreja vinayak-kukreja mentioned this issue Aug 19, 2022
Merged
@mergify mergify bot closed this as completed in #1052 Sep 11, 2022
@mergify mergify bot closed this as completed in 716738c Sep 11, 2022
General Availability | cdk8s-plus automation moved this from In progress to Done Sep 11, 2022
cdk8s-automation pushed a commit that referenced this issue Sep 11, 2022
@vinayak-kukreja
feat(cronjob): introduce CronJob construct (#1052)
d737618 
This PR adds the Kubernetes [CronJob Resource L2](https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/cron-job-v1/) to cdk8s-plus. This is part of the effort to make cdk8s+ GA.

Having this L2 abstraction would enable the customer to provision Kubernetes CronJob Resource using cdk8s+ and that would generate/add it to the Kubernetes manifest.

Resolves #376
Resolves #813
Resolves #814

## Next Steps

Since timezone is behind a feature gate and works with K8s-24, we would need to fix the backport PR.

## DCO

Signed-off-by: Vinayak Kukreja <vinakuk@amazon.com>
(cherry picked from commit 716738c)
Signed-off-by: Vinayak Kukreja <78971045+vinayak-kukreja@users.noreply.github.com>
cdk8s-automation pushed a commit that referenced this issue Sep 11, 2022
@vinayak-kukreja
feat(cronjob): introduce CronJob construct (#1052)
d5f1d38 
This PR adds the Kubernetes [CronJob Resource L2](https://kubernetes.io/docs/reference/kubernetes-api/workload-resources/cron-job-v1/) to cdk8s-plus. This is part of the effort to make cdk8s+ GA.

Having this L2 abstraction would enable the customer to provision Kubernetes CronJob Resource using cdk8s+ and that would generate/add it to the Kubernetes manifest.

Resolves #376
Resolves #813
Resolves #814

## Next Steps

Since timezone is behind a feature gate and works with K8s-24, we would need to fix the backport PR.

## DCO

Signed-off-by: Vinayak Kukreja <vinakuk@amazon.com>
(cherry picked from commit 716738c)
Signed-off-by: Vinayak Kukreja <78971045+vinayak-kukreja@users.noreply.github.com>
@iliapolo iliapolo moved this from Done to Sep | 4-9 in General Availability | cdk8s-plus Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vinayak-kukreja vinayak-kukreja

Labels
effort/small 1 day tops feature-request New/Enhanced functionality wanted priority/p1 Should be on near term plans
Projects
No open projects
General Availability | cdk8s-plus
Sep | 4-9
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(cronjob): introduce CronJob construct cdk8s-team/cdk8s-plus
2 participants
@iliapolo @vinayak-kukreja