Skip to content

Audit storage statistic#10155

Merged
kakaiu merged 1 commit into
apple:mainfrom
kakaiu:add-statistic-for-audit-storage
May 5, 2023
Merged

Audit storage statistic#10155
kakaiu merged 1 commit into
apple:mainfrom
kakaiu:add-statistic-for-audit-storage

Conversation

@kakaiu

@kakaiu kakaiu commented May 5, 2023

Copy link
Copy Markdown
Member

This PR introduces AuditStorageStatistic trace event to:
(1) Understand the progress of audit storage;
(2) Understand the fragment of serverKeys introduced by encode_shard_location_metadata.

100K correctness test with 2 irrelevant failures
20230504-223330-zhewang-b9c720dd6272566a compressed=True data_size=33126674 duration=6473460 ended=100000 fail=2 fail_fast=10 max_runs=100000 pass=99998 priority=100 remaining=0 runtime=1:12:45 sanity=False started=100000 stopped=20230504-234615 submitted=20230504-223330 timeout=5400 username=zhewang

100K ValidateStorage test
20230504-223406-zhewang-eaa2bcbcd54aaef1 compressed=True data_size=33155985 duration=3378096 ended=100000 fail_fast=10 max_runs=100000 pass=100000 priority=100 remaining=0 runtime=0:52:40 sanity=False started=100000 stopped=20230504-232646 submitted=20230504-223406 timeout=5400 username=zhewang

This PR is based on #9628

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.

  • The PR has a description, explaining both the problem and the solution.
  • The description mentions which forms of testing were done and the testing seems reasonable.
  • Every function/class/actor that was touched is reasonably well documented.

For Release-Branches

If this PR is made against a release-branch, please also check the following:

  • This change/bugfix is a cherry-pick from the next younger branch (younger release-branch or main if this is the youngest branch)
  • There is a good reason why this PR needs to go into a release branch and this reason is documented (either in the description above or in a linked GitHub issue)

@kakaiu
kakaiu requested a review from liquid-helium May 5, 2023 00:44
@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-clang-ide on Linux CentOS 7

  • Commit ID: 12f93c5
  • Duration 0:20:00
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@fdb-windows-ci

Copy link
Copy Markdown
Collaborator

Doxense CI Report for Windows 10

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-clang on Linux CentOS 7

  • Commit ID: 12f93c5
  • Duration 0:44:53
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr on Linux CentOS 7

  • Commit ID: 12f93c5
  • Duration 1:20:42
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)

@foundationdb-ci

Copy link
Copy Markdown
Contributor

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

  • Commit ID: 12f93c5
  • Duration 1:27:53
  • Result: ✅ SUCCEEDED
  • Error: N/A
  • Build Log terminal output (available for 30 days)
  • Build Workspace zip file of the working directory (available for 30 days)
  • Cluster Test Logs zip file of the test logs (available for 30 days)

@kakaiu
kakaiu merged commit 3e3ed19 into apple:main May 5, 2023
kakaiu added a commit to kakaiu/foundationdb that referenced this pull request May 11, 2023
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.

4 participants