Hello! I've been struggling with the "maximum numb...
# ask-community
g
Hello! I've been struggling with the "maximum number of scheduled runs per flow" problem. I'm aware that this is supposed to be solved as in https://github.com/PrefectHQ/server/pull/281, but I can't seem to put it to work with the Helm chart. I'm trying to set it in the
values.yaml
file such as
Copy code
prefectConfig:
  services:
    towel:
      max_scheduled_runs_per_flow: 50
and when I try to upgrade my deployments with this, the towel pod doesn't get redeployed and the issue is still there. Is this way correct or should I do anything else?
k
Could you try with “50” as a string? I know that has caused issues in the past
I am also wondering if the hardcoding in the
config.toml
is overriding that but I don’t think it should. Will look
should be fine
g
sure, I'll try that. just wondering, do I need to re-register the flows or it should work out of the box?
k
I believe this one should work out of the box when the service gets re-deployed because this is a background service that is creating the schedule for the Flows that runs every 90 or 120 seconds and inserting them to the database so unrelated to the actual Flows
Maybe you have to toggle schedule on/off too to force it to take effect but I don’t think so
g
did that, but only 10 runs keeps showing 😞 is there a way I can double check these configs within any pods ?
k
I guess check for the environment variables in the
towel
pod?
g
btw everytime I do
helm upgrade..
only the graphql pod get recreated. towel pod hasn't changed, thus no environment variables for that
k
You can try redeploying the towel pod?
g
I've set a
timestamp
annotation in order to force the following pods to be recreated every upgrade • apollo • graphql • hasura • towel but it still doesn't seem to be working, no envs have been set on the towel pod, unfortunately
it seems like
{{- (include "prefect-server.envConfig" .) | nindent 12 }}
is missing from multiple deployments in the Helm chart
adding it just below this https://github.com/PrefectHQ/server/blob/master/helm/prefect-server/templates/towel/deployment.yaml#L57 (i.e. appending it to the other environment variables) seems to do the trick
can we open an issue on this ?
k
Ah I see. Yes this sounds like a good issue to open! Could you open one with your fix? I can also take care of it in a bit if you don’t have time
g
@Kevin Kho just did it: https://github.com/PrefectHQ/server/pull/346 sorry it took so long
k
I noticed. No worries man, we should be the ones thanking you for contributing 🙂
1