Skip to content

Update good_job in preparation for 4.0 upgrade#11374

Merged
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/update-good-job-3-99
Oct 21, 2024
Merged

Update good_job in preparation for 4.0 upgrade#11374
mitchellhenke merged 2 commits intomainfrom
mitchellhenke/update-good-job-3-99

Conversation

@mitchellhenke
Copy link
Contributor

🛠 Summary of changes

We're quite far behind on good_job versions, and a major version update was released back in July. The structure of jobs has changed, and we must complete all jobs scheduled or queued in the old format prior. This is described further in the v3 to v4 update guide.

changelog: Internal, Maintenance, Update good_job in preparation for 4.0 upgrade
@mitchellhenke mitchellhenke requested a review from a team October 21, 2024 19:31
@mitchellhenke mitchellhenke marked this pull request as ready for review October 21, 2024 19:31
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM! thanks for linking the upgrade guide

In a production console, run GoodJob.v4_ready? which should return true when safely upgradable.

What do you think of augmenting our HeartbeatJob (or something) to add GoodJob.v4_ready? to the log payload so we can get confirmation in the logs we're ready to migrate/deploy before we do?

@mitchellhenke
Copy link
Contributor Author

LGTM! thanks for linking the upgrade guide

In a production console, run GoodJob.v4_ready? which should return true when safely upgradable.

What do you think of augmenting our HeartbeatJob (or something) to add GoodJob.v4_ready? to the log payload so we can get confirmation in the logs we're ready to migrate/deploy before we do?

Yeah, that's a great idea. I added a new job to run hourly in b4d8d23. I think we could put it in the HeartbeatJob, but I think I'd rather leave that one as simple as possible since we have alerts related to it.

@mitchellhenke mitchellhenke merged commit 5421a73 into main Oct 21, 2024
@mitchellhenke mitchellhenke deleted the mitchellhenke/update-good-job-3-99 branch October 21, 2024 20:17
This was referenced Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants