Fix and Improve fix pg size & shard identical layout

[Hooper9973]

Bug Fixes:

• Resolved issues in create_shard and seal_shard functions to ensure correct shard management. Enhancements:
• Added validation to prevent creation of placement groups with sizes smaller than the chunk size. Test Improvements:
• pg test:
  • Added tests to verify that creating a placement group with a size smaller than the chunk size results in failure.
• shard test:
  • Created shard but shard size is larger than pg space left which should be failed.
  • Verified that successfully created shards have identical virtual chunk layouts.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 64.70588% with 6 lines in your changes missing coverage. Please review.

Project coverage is 65.30%. Comparing base (1746bcc) to head (6e5e52e).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
...ib/homestore_backend/replication_state_machine.cpp	0.00%	2 Missing and 1 partial ⚠️
src/lib/homestore_backend/heap_chunk_selector.cpp	50.00%	1 Missing and 1 partial ⚠️
src/lib/homestore_backend/hs_shard_manager.cpp	85.71%	1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #235      +/-   ##
==========================================
+ Coverage   63.15%   65.30%   +2.14%     
==========================================
  Files          32       32              
  Lines        1900     1977      +77     
  Branches      204      213       +9     
==========================================
+ Hits         1200     1291      +91     
+ Misses        600      578      -22     
- Partials      100      108       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[JacksonYao287]

NIT: I prefer to cast homestore::chunk_num_t(uinit_16) to uint_64 here, and then cast it back.

[Hooper9973]

I'll merge first, and if necessary, I will refine in next pr.
BTW, what are the pros in this way? 😃

[JacksonYao287]

hs_repl_test_helper.hpp is the framework, it does not know what is a shard_id_t , pg_id_t , blob_id_t and chunk_id_t. these types are only know application layer.

if we add v_chunk_id here, should we also add shard_id_t, pg_id_t and others?

so I think we can use this 64bit as a common shared data across different processed, and application layer should translate this to what it wants

[JacksonYao287]

looks good, only a minor comment