-
Notifications
You must be signed in to change notification settings - Fork 15
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
config for disabling the beta_job
block or delaying on startup
#726
Labels
Comments
Would it be more straight forward to have an _enable_ flag instead of _disable_?
|
its about the default behaviour which should still be enabled? So the best default boolean is a false one. |
Gophers like to use false as the default value for Booleans :) But I would dare to have true as the default for an enabled flag.
Do we have enable(d)/disable(d) in other parts of the configuration? There is disable_access_control, which we justified with being an exception. Disabling a job is also an exception.
…________________________________
Von: Alex Schneider ***@***.***>
Gesendet: Monday, April 24, 2023 5:12:30 PM
An: avenga/couper ***@***.***>
Cc: Felix Hassert ***@***.***>; Mention ***@***.***>
Betreff: Re: [avenga/couper] config for disabling the `beta_job` block or delaying on startup (Issue #726)
https://github.com/alex-schneider/cron :)
—
Reply to this email directly, view it on GitHub<#726 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAD3OQSDHB2XKCYDFTPL2B3XC2J55ANCNFSM6AAAAAAVGX26L4>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
beta_job
does run on startup no matter how big theinterval
has been set. However, it may be required to disable a job via environment variable for some reason. A workaround could have been a huge interval which actually does not work with our startup run. So the proposal here would be to introduce adisable
bool attribute which could be set totrue
if required.Thoughts: how about to add an additional attribute which prevents the startup run?
e.g.:
/cc @filex
The text was updated successfully, but these errors were encountered: