chia-blockchain icon indicating copy to clipboard operation
chia-blockchain copied to clipboard

[CHIA-299] DL separate DAT files in folders by store id

Open fchirica opened this issue 1 year ago • 11 comments

Purpose:

Current Behavior:

New Behavior:

Testing Notes:

fchirica avatar Mar 12 '24 00:03 fchirica

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Mar 14 '24 17:03 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Mar 15 '24 05:03 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Mar 15 '24 16:03 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Mar 18 '24 09:03 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Mar 25 '24 15:03 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Mar 25 '24 16:03 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Mar 27 '24 16:03 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Mar 27 '24 18:03 github-actions[bot]

File Coverage Missing Lines
chia/data_layer/data_layer_server.py 0.0% lines 105-113, 118
chia/data_layer/download_data.py 85.7% lines 195-198, 203-205, 357
chia/data_layer/s3_plugin_service.py 0.0% lines 149, 151, 154-156, 215, 232, 258-259, 262
Total Missing Coverage
163 lines 28 lines 82%

github-actions[bot] avatar Mar 29 '24 12:03 github-actions[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Apr 17 '24 16:04 github-actions[bot]

This PR has been flagged as stale due to no activity for over 60 days. It will not be automatically closed, but it has been given a stale-pr label and should be manually reviewed by the relevant parties.

github-actions[bot] avatar Jun 28 '24 11:06 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Jul 02 '24 00:07 github-actions[bot]

Pull Request Test Coverage Report for Build 9753178686

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 135 of 163 (82.82%) changed or added relevant lines in 6 files are covered.
  • 69 unchanged lines in 15 files lost coverage.
  • Overall coverage decreased (-0.03%) to 90.964%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chia/data_layer/download_data.py 48 56 85.71%
chia/data_layer/data_layer_server.py 0 10 0.0%
chia/data_layer/s3_plugin_service.py 0 10 0.0%
<!-- Total: 135 163
Files with Coverage Reduction New Missed Lines %
chia/_tests/simulation/test_simulation.py 1 96.5%
chia/daemon/keychain_proxy.py 1 72.57%
chia/data_layer/data_layer_server.py 1 0.0%
chia/farmer/farmer.py 1 72.23%
chia/daemon/server.py 1 84.13%
chia/_tests/core/util/test_file_keyring_synchronization.py 1 97.1%
chia/data_layer/s3_plugin_service.py 1 0.0%
chia/daemon/client.py 2 73.33%
chia/_tests/core/test_farmer_harvester_rpc.py 2 98.02%
chia/rpc/rpc_server.py 3 88.04%
<!-- Total: 69
Totals Coverage Status
Change from base Build 9719381064: -0.03%
Covered Lines: 101719
Relevant Lines: 111778

💛 - Coveralls

coveralls-official[bot] avatar Jul 03 '24 17:07 coveralls-official[bot]

Pull Request Test Coverage Report for Build 10001146000

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 167 of 252 (66.27%) changed or added relevant lines in 6 files are covered.
  • 1948 unchanged lines in 78 files lost coverage.
  • Overall coverage decreased (-0.1%) to 90.869%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chia/data_layer/data_layer.py 14 19 73.68%
chia/data_layer/data_layer_server.py 0 10 0.0%
chia/data_layer/download_data.py 52 67 77.61%
chia/data_layer/s3_plugin_service.py 0 55 0.0%
<!-- Total: 167 252
Files with Coverage Reduction New Missed Lines %
chia/_tests/wallet/test_wallet_retry.py 1 98.55%
chia/types/eligible_coin_spends.py 1 98.06%
chia/_tests/wallet/db_wallet/test_dl_wallet.py 1 99.42%
chia/_tests/wallet/db_wallet/test_dl_offers.py 1 99.24%
chia/util/keyring_wrapper.py 1 73.83%
chia/daemon/keychain_proxy.py 1 72.57%
chia/full_node/mempool.py 1 99.71%
chia/_tests/wallet/dao_wallet/test_dao_wallets.py 1 99.95%
chia/data_layer/data_layer_server.py 1 0.0%
chia/farmer/farmer.py 1 72.23%
<!-- Total: 1948
Totals Coverage Status
Change from base Build 9719381064: -0.1%
Covered Lines: 72256
Relevant Lines: 79499

💛 - Coveralls

coveralls-official[bot] avatar Jul 15 '24 17:07 coveralls-official[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Jul 24 '24 17:07 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Jul 31 '24 21:07 github-actions[bot]

This PR is awaiting a coverage-diff override (or coverage of the diff) prior to merge

Starttoaster avatar Jul 31 '24 23:07 Starttoaster

Coverage exemption

emlowe avatar Aug 01 '24 15:08 emlowe