feat(gcal_sync): 0.5.0 build and valid custom-data namespace - #415
Open
canvas-investigator[bot] wants to merge 1 commit into
Open
feat(gcal_sync): 0.5.0 build and valid custom-data namespace#415canvas-investigator[bot] wants to merge 1 commit into
canvas-investigator[bot] wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🧵 Allison Leitner cc @allisonleitner
What
Updates the
gcal_syncplugin package to the 0.5.0 build and fixes an invalid custom-data namespace that blocks fresh installs.Why
Installing this plugin from
mainonto a current-platform instance fails during custom-data provisioning:The platform requires custom-data namespaces to match
^[a-z][a-z0-9_]*__[a-z][a-z0-9_]*$(a two-partprefix__name, enforced incanvas-plugins/plugin_runner/namespace.py). The manifest shipped a baregcal_sync, which fails validation, so provisioning never creates the schema or populatesnamespace_read_write_access_key. This sets a valid generic namespace,gcalsync__core.The 0.5.0 build
Brings
mainup to the 0.5.0 build already running in production deployments — general robustness/scale hardening, no deployment-specific logic:PendingHoldCreatemodel +InboundEventMapping.last_applied_hashclose a cross-calendar race that could mint a second hold for a shared event on webhook replay.reconcile.py): per-calendar locks +last_outbound_synced_atordering so capped runs rotate across the fleet and converge.ReimportDrainCron, every 2 min): a whole-roster rebuild is enqueued and drained a few providers per tick so each returned effect batch stays small enough to apply reliably.routes/google_admin.py,templates/google_admin.html).list_all_events,find_event_by_private_property).Schema note: 0.5.0 adds the
PendingHoldCreatetable and thelast_outbound_synced_at/last_applied_hashfields vs 0.4.5. Existing deployments install from their own copies and are unaffected by this namespace value.Tests
Carries the 0.5.0 pytest suite (updated to match the code). The only edit on top of the production 0.5.0 build is the namespace string, which the fully-mocked unit suite does not exercise — behavior is unchanged from the build already in production. README/docs refresh deferred to a follow-up.
Generated by the Investigator