Skip to content
This repository was archived by the owner on Jun 21, 2026. It is now read-only.

CronJob is in batch/v1beta1 not batch/v1 - #33

Merged
psFried merged 1 commit into
psFried:masterfrom
edwardgeorge:master
Aug 24, 2020
Merged

psFried merged 1 commit into
psFried:masterfrom
edwardgeorge:master

Conversation

@edwardgeorge

Copy link
Copy Markdown
Contributor

Modifies k8s_types.rs to have CronJob under version v1beta1 instead of v1.

Fixes #32

CronJob in k8s_types.rs is defined as under the v1 batch api, but it does not exist in this version. It exists under v1beta1 and is not in v1.

This has the effect that if you specify .with_child(k8s_types::batch::v1::CronJob, ...) on your operator config then you get no events at all being processed, the operator appears to hang and do nothing. (which may perhaps be a separate bug?)

This operator config now processes events as expected:

    let operator_config = OperatorConfig::new(OPERATOR_NAME, PARENT_TYPE)
        .with_child(k8s_types::batch::v1beta1::CronJob, ChildConfig::replace());

@psFried

psFried commented Aug 24, 2020

Copy link
Copy Markdown
Owner

Thanks a lot for fixing this!

@psFried
psFried merged commit 80bfc2b into psFried:master Aug 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CronJob in k8s_types is defined under the wrong version number.

2 participants