-
Notifications
You must be signed in to change notification settings - Fork 760
chore: improve event and docker-compose ,Improve the permissions of the endpoint health interface, upgrade otel from 0.30.0 to 0.31.0
#620
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…#592) * Initial plan * Fix ETag format to comply with HTTP standards by wrapping with quotes Co-authored-by: overtrue <[email protected]> * bufigx --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: overtrue <[email protected]> Co-authored-by: overtrue <[email protected]>
* improve lock Signed-off-by: Mu junxiang <[email protected]> * feat(tests): add wait_for_object_absence helper and improve lifecycle test reliability Signed-off-by: Mu junxiang <[email protected]> * chore: remove dirty docs Signed-off-by: Mu junxiang <[email protected]> --------- Signed-off-by: Mu junxiang <[email protected]>
…599) * feat(append): implement object append operations with state tracking Signed-off-by: junxiang Mu <[email protected]> * chore: rebase Signed-off-by: junxiang Mu <[email protected]> --------- Signed-off-by: junxiang Mu <[email protected]>
Co-authored-by: loverustfs <[email protected]>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found. |
* feature/audit-add-logger: fix improve code for `import` fix: validate mqtt broker build(deps): upgrade s3s (#595) feat(append): implement object append operations with state tracking (#599) Improve lock (#596) Fix ETag format to comply with HTTP standards by wrapping with quotes (#592) upgrade version upgrade starshard version feat: improve code for notify # Conflicts: # Cargo.lock # Cargo.toml # rustfs/src/storage/ecfs.rs
…he `endpoint` health interface
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request refactors RustFS to upgrade its dependencies and organize imports more efficiently. It focuses on updating OpenTelemetry to version 0.31, replacing std::collections with hashbrown for performance improvements, and reorganizing imports across multiple files.
Key Changes
- Upgraded OpenTelemetry dependencies from 0.30 to 0.31
- Replaced std::collections with hashbrown HashMap/HashSet throughout the codebase for better performance
- Reorganized and consolidated imports in multiple files for better readability
Reviewed Changes
Copilot reviewed 33 out of 36 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Cargo.toml | Updated OpenTelemetry and other dependency versions |
| rustfs/src/storage/ecfs.rs | Major import consolidation and organization |
| crates/notify/src/notifier.rs | Replaced DashMap with AsyncShardedHashMap for better performance |
| crates/utils/src/notify/net.rs | Added new network utilities module |
| rustfs/src/admin/handlers/event.rs | Commented out bucket notification handlers and improved MQTT validation |
| Multiple notify crates | Replaced std::collections with hashbrown equivalents |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 33 out of 36 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
endpoint health interface, upgrade otel from 0.30.0 to 0.31.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 33 out of 36 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Type of Change
Related Issues
#629
Summary of Changes
Checklist
make pre-commitImpact
Additional Notes
Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.