feat: Fill out more of the core autoconnect functions #378
Conversation
This file contains 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
Member
jrconlin
commented
May 11, 2023
jrconlin
commented
- add proper health check routine for status
* add proper health check routine for __status__
jrconlin
added a commit
that referenced
this pull request
Aug 2, 2023
This patch provides support for Google Cloud Platform Bigtable as a storage mechanism for Push This feature is enabled under the "bigtable" flag. It also includes work for: * #378 to improve the status check to perform a minimal database connection check and report available storage types * SYNC-3688 to include state machine changes * SYNC-3741 / #389 which removes message table rotation * SYNC-3449 which creates the bigtable module * SYNC-3809 which removes sortkey_timestamp restrictions for topic messages also: - kill save_channels (only needed by legacy autoconnect) - update mockall/cleanup old workaround - update openssl per RUSTSEC-2023-0044 - several bigtable explicit integration tests support scripts - fix autoconnect's db initialization to match autoendpoint's (don't panic when no DB_DSN is set) - don't unnecessarily remove expired messages in a background task during check storage - convert from circleci docker images to standard ones. - limit to `jobs=8` to prevent circleci OOM messages. --------- Co-authored-by: Philip Jenvey <pjenvey@underboss.org>
|
Merged as part of #364 |
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.