Unable to insert a periodic job from external service

[odyfey]

During the implementation of enqueuing a periodic job from external service, a panic occurred. The external service does not contain any code for registering workers, it stores data and passes it to the worker as parameters when enqueuing periodic jobs.

How to reproduce

Initialize in an external service a client without worker registration:

client, err := river.NewClient(riverpgxv5.New(workerPool), &river.Config{})

Because of this condition, internal client.periodicJobs will not be initialized, which leads to panic when trying to insert a periodic job:

client.PeriodicJobs().Add(j)

Is this behavior intentional, or is it a bug?

[bgentry]

@odyfey are you saying you're setting up periodic jobs from an insert-only client? This won't work as the actual enqueueing of the jobs relies on leader election, which happens only on full running clients. I think we have some documentation gaps and maybe some missing edge case handling around this though.

[brandur]

Tried to improve the error message on this one a bit in #938.

[odyfey]

are you saying you're setting up periodic jobs from an insert-only client?

@bgentry yes, I am trying to add a periodic job to the queue using an insert-only client. Agreed, the case with periodic jobs should be mentioned in the section about insert-only clients.