Merged
Conversation
…cross-compilation from Ubuntu
casibbald
force-pushed
the
advanced-filtering
branch
from
4021197 to
ccb3b79
Compare
- Fixed MissingContentMD5 errors in batch deletion operations - Added MD5 hash computation and base64 encoding for delete requests - Updated delete_objects_recursive and delete_all_versions in rm.rs - Updated delete_all_objects and delete_all_versions in bucket.rs - MinIO requires Content-MD5 headers for batch operations - AWS SDK Rust doesn't add these headers automatically - Successfully tested: cleared 10 buckets with 102 objects using only obsctl commands - Resolves phantom deletion success and batch operation failures - Maintains compatibility with AWS S3 and other S3-compatible services
Add comprehensive panic hook to detect broken pipe errors across all platforms. Supports Unix EPIPE, Linux ESHUTDOWN, Windows ERROR_NO_DATA error codes. Graceful exit with output flushing prevents ugly panic messages. Maintains professional CLI behavior when piping to head/tail commands.
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
1 participant
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.
🚀 Enterprise-Grade obsctl: Advanced Filtering, OTEL Observability & Production-Ready Infrastructure
📋 Overview
This massive PR transforms obsctl from a basic S3 CLI into a production-ready, enterprise-grade tool with advanced filtering capabilities, comprehensive observability, and robust infrastructure. The changes represent 38,521 additions across 119 files, delivering a complete enterprise solution.
🎯 Key Features & Achievements
🔍 Advanced Filtering System
--created-after,--modified-before)--min-size,--max-size)--head,--tail,--max-results)🎨 Intelligent Pattern Matching
--patternflag*-prod,backup-\d{4},user-[a-z]-bucket📊 OpenTelemetry Observability
🛠️ Production Infrastructure
🔧 Critical Bug Fixes
📈 Statistics & Impact
🔄 Key Commits
feat(cli): implement cross-platform broken pipe handlingfix: add Content-MD5 headers for MinIO batch deletion compatibilityfix: Windows CI build fixed - removed Windows runner dependencyFixed compilation issues🧪 Testing & Validation
📚 Documentation
🎯 Production Readiness
This PR delivers a production-ready enterprise CLI tool that:
🚀 Next Steps
After merge, obsctl will be ready for:
This PR represents a complete transformation of obsctl into an enterprise-grade S3 CLI tool with advanced capabilities, comprehensive observability, and production-ready infrastructure. Ready for immediate production deployment.
📋 PR Details for GitHub
Title:
🚀 Enterprise-Grade obsctl: Advanced Filtering, OTEL Observability & Production-Ready InfrastructureHead Branch:
advanced-filteringBase Branch:
masterLabels:
enhancement,feature,infrastructure,observabilityReviewers: (Add appropriate reviewers)
Assignees: (Add assignees as needed)