Open
Conversation
… shards (apple#11324) Co-authored-by: Dan Lambright <[email protected]> Co-authored-by: Jingyu Zhou <[email protected]>
) (apple#11341) Co-authored-by: Chaoguang Lin <[email protected]>
…_fix Hex decode add-prefix and remove-prefix for fdbrestore.
* ConsistencyCheckerUrgent repeated run * address comments * avoid trace SevError for TesterRecruitmentTimeout unless it keeps failure for over 1 day * address comments * address comments
In the current code, errors are retried in getConsistentReadVersion, so it's possible that the client has cancelled the GRV request, but readVersionBatcher continue retrying, which can lead to many clients DDoS GRV proxies, especially when the database has become unavailable for a while and clients are issuing many GRV requests.
Throw errors in getConsistentReadVersion [release-7.3]
* Add logging to waitForVersion * Respond to review comments. --------- Co-authored-by: Dan Lambright <[email protected]>
Cherry-pick Add logging to LogRouter's waitForVersion function (apple#11359)
…cherry_pick [Release-7.3] Add documentation about experimental features
context (useful for running ycsb against bare metal cluster). * packaging/docker/Dockerfile * packaging/docker/Dockerfile.eks Make the ycsb target inherit from foundationdb-base so we pick up libfdb_c.so. Add in a version of run_ycsb.sh that doesn't presume k8s. Use 'entrypoint' rather than 'cmd' so can override on 'docker run'. * packaging/docker/run_ycsb_standalone.sh Version of run_ycsb.sh w/o the presumption of k8s.
* Disable fdb_test.go:TestOpenNotExistTenant This is not supported yet. * fixup! Fix the comment
[release-7.3] Add a knob for merge parallelism and add log for max shard size.
CC's checkBetterSingletons() calls getUsedIds() that asserts proxy interfaces are present. However, when a GRV/commit proxy failed, before CC starts a new recovery, the proxy's processId becomes empty, thus triggering the failure. The fix is to cancel the caller while waiting for recovery. To reproduce 7.1 commit 725a08a clang build: ./fdbserver.6.0.15 -r simulation -f ./tests/restarting/from_5.2.0_until_6.3.0/ClientTransactionProfilingCorrectness-1.txt -s 900000399 -b on -f ./tests/restarting/from_5.2.0_until_6.3.0/ClientTransactionProfilingCorrectness-2.txt --restarting -s 900000400 -b on
…er (apple#12475) * counters for dirty tags in TLogServer (apple#12427) * Make DirtyTagsProcessed count only tags that touch persistentData (apple#12468) Co-authored-by: Dan Lambright <[email protected]> --------- Co-authored-by: Dan Lambright <[email protected]>
* Delete blob tests and move some tests Specifically, moving from_7.3.50 to from_7.3.50_until_7.4.0, because Joshua contains 7.4 binaries and may cause "downgrade" issues. * Limit downgrade to _until_7.4.0 to avoid "downgrading" 7.4 * fix CMakeLists.txt
* Pass encryption file name to backup to encrypt backup files (apple#12287) * Backup encryption fix on S3 (apple#12289) * Backup encryption fix * Add enable_read_cache option to be selected randomly to cover both code paths. * Add randomization in ClientKnobs for testing * Enable Backup/Restore Encryption/Decryption in Simulation Tests (apple#12293) * Enable Backup Encryption in Simulation Tests * Enable encryption in BackupNew/OldRestore test and add fix * Enable encryption in BackupAndParallelRestoreCorrectness workload * Enable in RestoreBackup workload * Add additional workloads * Fix for Snap restart where second is not able to find if first run has encryption key or not. * Change the encryption probability to 50% in worloads * Format * Disable it in test and re-enable it later * Add metadata for Encrypted Backups (apple#12354) * Add metadata for Encrypted backups * Format files * Update error message in case of encryption keys mismatch for backup and restore (apple#12375) * Fix Backup Encryption File Access Flags (apple#12220) * fix flags * format Co-authored-by: Jingyu Zhou <[email protected]> * Delete line --------- Co-authored-by: Milla Samuel <[email protected]> Co-authored-by: Jingyu Zhou <[email protected]> --------- Co-authored-by: Milla Samuel <[email protected]> Co-authored-by: Milla Samuel <[email protected]> Co-authored-by: Jingyu Zhou <[email protected]>
apple#12590) * Provide encryption support with fdbbackup modify command * Update backup help * format * Fix errors * Error out if different encryption key files are passed for same url * Addressed comments
* EncryptionBackup: Log encryption key file access failures at SevError severity (apple#12629) * Change the error to Sev40 * Fix formatting error * Display Encryption Key Info in status json (apple#12649) * Encryption in json * Addressed comments * Use unordered_set * Addressed comments for ctest
8c0f52d to
393da65
Compare
Contributor
Result of foundationdb-pr-clang-arm on Linux CentOS 7
|
Contributor
Result of foundationdb-pr-clang-ide on Linux RHEL 9
|
Contributor
Result of foundationdb-pr-clang-arm on Linux CentOS 7
|
Contributor
Result of foundationdb-pr-cluster-tests on Linux RHEL 9
|
Contributor
Result of foundationdb-pr-clang on Linux RHEL 9
|
Contributor
Result of foundationdb-pr on Linux RHEL 9
|
Contributor
Result of foundationdb-pr-clang-ide on Linux RHEL 9
|
Contributor
Result of foundationdb-pr-cluster-tests on Linux RHEL 9
|
Contributor
Result of foundationdb-pr-clang on Linux RHEL 9
|
Contributor
Result of foundationdb-pr on Linux RHEL 9
|
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.
Replace this text with your description here...
Code-Reviewer Section
The general pull request guidelines can be found here.
Please check each of the following things and check all boxes before accepting a PR.
For Release-Branches
If this PR is made against a release-branch, please also check the following:
release-branchormainif this is the youngest branch)