Skip to content

T407375: Set healthcheck workarround with cron#252

Merged
don-vip merged 1 commit intomasterfrom
healthcheck
Oct 15, 2025
Merged

T407375: Set healthcheck workarround with cron#252
don-vip merged 1 commit intomasterfrom
healthcheck

Conversation

@chicocvenancio
Copy link
Member

It would be better to have it integrated into the code with a heartbeat and do some tracing to figure out why the workers are hanging, but this should get the workers back to working.

@don-vip don-vip merged commit bed32da into master Oct 15, 2025
@don-vip
Copy link
Collaborator

don-vip commented Oct 15, 2025

Mmm there's an error when applying it:

Error: Evaluation Error: Error while evaluating a Resource Statement, Cron::Job[v2chealthcheck]: parameter 'minute' expects a Cron::Minute = Variant[Integer[0, 59], Pattern[/(?x)\A(
    \* ( \/ ( [1-5][0-9]?|[6-9] ) )?
    |         [1-5]?[0-9] ( - [1-5]?[0-9] ( \/ ( [1-5][0-9]?|[6-9] ) )? )?
        ( ,   [1-5]?[0-9] ( - [1-5]?[0-9] ( \/ ( [1-5][0-9]?|[6-9] ) )? )? )*
  )\z/]] value, got String (file: /mnt/nfs/labstore-secondary-home/don-vip/backend-bookworm.pp, line: 295) on node encoding01.video.eqiad1.wikimedia.cloud

@don-vip
Copy link
Collaborator

don-vip commented Oct 15, 2025

Changed to */5 and applied on all encoder instances.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants