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

async Runner executions can be killed prematurely by agent (missing timeout) #203

Closed
simonmeggle opened this issue Jul 28, 2022 · 1 comment
Assignees
Labels
c:bakery component: bakery p:high Priority: high t:bug Type: Bug
Milestone

Comments

@simonmeggle
Copy link
Collaborator

The V2 bakery script does not set the timeout for async plugin execution.
As a consequence, a default timeout of "check_interval (1min) + X" is used.
For most e2e web tests this is sufficient, but longer tests will be killed by the agent.
See
https://host/site/check_mk/plugin-api/cmk.base.cee.plugins.bakery.bakery_api/v1.html#cmk.base.cee.plugins.bakery.bakery_api.v1.Plugin

@simonmeggle simonmeggle added t:bug Type: Bug p:high Priority: high c:bakery component: bakery v:cmkv2.x labels Jul 28, 2022
@simonmeggle simonmeggle added this to the v1.2.9.1 milestone Jul 28, 2022
@simonmeggle simonmeggle self-assigned this Jul 28, 2022
@simonmeggle
Copy link
Collaborator Author

In V1 , timeout is set to the same value as cache_time (=execution interval).
This should be changed to 5sec less than execution interval in V1 and V2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:bakery component: bakery p:high Priority: high t:bug Type: Bug
Projects
None yet
Development

No branches or pull requests

1 participant