Feature/update rules#7
Open
ashwinrookie wants to merge 38 commits into
Open
Conversation
- Added /rule-agent/test_rules POST endpoint to test deployed Drools rules - Fixed decision field parsing in test_rules endpoint response - Added comprehensive Swagger/OpenAPI 3.0 documentation with 7 test examples - Created TESTING_RULES.md guide with curl examples for all test scenarios - Updated .gitignore to exclude .env files with secrets - Added underwriting workflow components (TextractService, S3Service, etc.) - Enhanced Docker setup with Maven and Java for automated KJar builds - Added documentation: API testing guide, Docker setup, implementation summary 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Changed 'as' to 'AS' on line 16 to match FROM keyword casing. This resolves the Docker build warning: FromAsCasing: 'as' and 'FROM' keywords' casing do not match
Added ACL: 'public-read' to Excel file uploads in S3Service. This allows anyone with the S3 URL to view/download the Excel files without requiring AWS authentication. Other file types (JAR, DRL) remain private by default.
Handle buckets that don't support ACLs by attempting public-read ACL first, then falling back to standard upload if AccessControlListNotSupported error occurs. This allows Excel files to be uploaded to both: - Buckets with ACL support (uploaded with public-read ACL) - Buckets without ACL support (uploaded with bucket-level permissions) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove public-read ACL from Excel uploads to keep files private. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add complete infrastructure for running one dedicated Drools container per rule set, providing complete isolation, independent scaling, and better multi-tenancy. Architecture: - Container Orchestrator dynamically creates Docker/K8s containers - Service Registry tracks container endpoints - Request Router automatically routes to correct container - Supports both Docker Compose and Kubernetes deployments Key Features: - One JVM per rule set for complete fault isolation - Dynamic container creation on rule deployment - Automatic service discovery and request routing - Platform-agnostic (Docker or Kubernetes) - Independent scaling per rule set - Resource limits per container Components Added: - ContainerOrchestrator.py: Manages container lifecycle - Kubernetes manifests: Namespace, RBAC, Storage, Deployments - CONTAINER_PER_RULESET.md: 500+ line comprehensive guide - MINIKUBE_SETUP.md: Local Kubernetes development guide Configuration: - Docker: USE_CONTAINER_ORCHESTRATOR=true (enabled by default) - Kubernetes: Built-in support with proper RBAC Updated: - DroolsService.py: Request routing to correct container - DroolsDeploymentService.py: Integrated container creation - docker-compose.yml: Docker socket mount, orchestrator config - requirements.txt: Added docker and kubernetes libraries - CLAUDE.md: Updated architecture documentation This enables production-grade isolation while maintaining development simplicity. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add network lookup logic to find the correct Docker network name, including support for docker-compose project-prefixed network names. Fixes: Network 'underwriting-net' not found error when creating new Drools containers from backend. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Handle case where Docker container exists but not yet in registry. Prevents KeyError when checking existing containers. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…ule-based-llms into feature/update-rules
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
3 participants
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.
No description provided.