Differential D10206

[keyserver] Set a job that synchronizes links with blobs
ClosedPublic
Actions

Authored by tomek on Dec 6 2023, 6:39 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Nov 18, 11:54 AM

	Unknown Object (File)
	Fri, Nov 8, 5:54 AM

	Unknown Object (File)
	Sat, Nov 2, 10:15 AM

	Unknown Object (File)
	Tue, Oct 29, 8:48 AM

	Unknown Object (File)
	Mon, Oct 28, 10:23 PM

	Unknown Object (File)
	Oct 4 2024, 4:21 AM

	Unknown Object (File)
	Oct 4 2024, 4:21 AM

	Unknown Object (File)
	Oct 4 2024, 4:21 AM

View All 68 Files

Subscribers

Details

Reviewers

michal
bartek
rohan

Commits

rCOMM122195947a95: [keyserver] Set a job that synchronizes links with blobs

Summary

Run this script daily. It should happen after Blob service cleanup, which starts between midnight and 0:15 AM and currently takes up to a couple of minutes.

Depends on D10205

Test Plan

Modify the schedule and check if the script runs.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Dec 6 2023, 6:39 AM

Herald added a subscriber: ashoat. · View Herald TranscriptDec 6 2023, 6:39 AM

Fix

Harbormaster completed remote builds in B24823: Diff 34337.Dec 6 2023, 6:58 AM

Harbormaster completed remote builds in B24825: Diff 34339.Dec 6 2023, 7:17 AM

tomek requested review of this revision.Dec 6 2023, 7:17 AM

ashoat added inline comments.Dec 6 2023, 7:50 AM

keyserver/src/cron/cron.js
131 ↗	(On Diff #34339)	I had interpreted your previous request as wanting to include "in the keyserver's timezone" here, but seeing this I wonder if you are fine omitting it. Either way, I think we should be consistent

Looks good, I agree on having the comments consistent though

This revision is now accepted and ready to land.Dec 7 2023, 9:54 AM

tomek added a child revision: D10290: [lib] Modify link verification to call the proper keyserver.Dec 11 2023, 6:39 AM

Update comment

Harbormaster failed remote builds in B26416: Diff 36526!Feb 1 2024, 7:36 AM

Harbormaster completed remote builds in B26416: Diff 36526.Feb 1 2024, 7:51 AM

tomek mentioned this in D11076: [keyserver] Create invite link blobs in a migration.Feb 14 2024, 7:05 AM

Closed by commit rCOMM122195947a95: [keyserver] Set a job that synchronizes links with blobs (authored by tomek). · Explain WhyFeb 14 2024, 7:58 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM122195947a95: [keyserver] Set a job that synchronizes links with blobs.

tomek mentioned this in rCOMMa8b3efff9b29: [keyserver] Create invite link blobs in a migration.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

cron/

14 lines

Diff 37089

keyserver/src/cron/cron.js

Loading...