Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

msp/runtime: add Job runtime#63016

Merged
jac merged 5 commits intomainfrom
jac/core-117
May 31, 2024
Merged

msp/runtime: add Job runtime#63016
jac merged 5 commits intomainfrom
jac/core-117

Conversation

@jac
Copy link
Member

@jac jac commented May 31, 2024

Currently we have a Service runtime entrypoint which parses env, creates a logger, initialises telemetry, creates the MSP contract, etc. according to MSP conventions.
However we don't have the equivalent for Jobs which requires operators to set up everything manually.
This PR remedies the situation by adding a Job entrypoint which performs similar setup to the Service entrypoint.

Closes CORE-117

Test plan

Successfully executed a mock job in GCP using this as an entrypoint

@jac jac requested a review from a team May 31, 2024 20:17
@cla-bot cla-bot bot added the cla-signed label May 31, 2024
Co-authored-by: Robert Lin <robert@bobheadxi.dev>
Copy link
Member

@bobheadxi bobheadxi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@bobheadxi
Copy link
Member

@jac should we file a follow-up issue to migrate https://github.com/sourcegraph/abuse-ban-bot to this new entrypoint?

Co-authored-by: Robert Lin <robert@bobheadxi.dev>
@jac
Copy link
Member Author

jac commented May 31, 2024

@jac should we file a follow-up issue to migrate https://github.com/sourcegraph/abuse-ban-bot to this new entrypoint?

Yeah, ideally we should migrate over all msp jobs if possible

@jac jac enabled auto-merge (squash) May 31, 2024 22:22
@jac jac merged commit 62a4195 into main May 31, 2024
@jac jac deleted the jac/core-117 branch May 31, 2024 22:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants