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.

Sign up for GitHub

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

Jump to bottom

Closes #8958: Add webhook support for jobs #11859

Merged

jeremystretch merged 4 commits into feature from 8958-job-webhooks

Mar 7, 2023

Member

jeremystretch commented Feb 28, 2023

Closes: #8958

Introduces two new event types for webhooks: job_start and job_end
Replace the set_status() method on JobResult with terminate()
Extend the start() and terminate() methods on JobResult to handle webhook queueing
Script and Report classes now inherit from WebhooksMixin
snapshots becomes a conditional key in webhook payload (as it pertains only to object changes)

jeremystretch added 4 commits

February 28, 2023 14:29


          Add type_job_start & type_job_end to Webhook

19c1691


          Replace JobResult.set_status() with terminate()

331d22b


          Add trigger_webhooks() to JobResult

d23ef2d


          Undo change to default status for object type_* fields on Webhook

1e027fc

jeremystretch requested a review from kkthxbye-code

March 1, 2023 00:35

jeremystretch merged commit e015da9 into feature

jeremystretch deleted the 8958-job-webhooks branch

March 7, 2023 23:01

github-actions bot locked as resolved and limited conversation to collaborators

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

Labels

None yet