Skip to content

fix: update log file paths and bump image tags to v2.0.0#53

Merged
a-klos merged 1 commit into
mainfrom
fix/path-in-k8s-template-files
Jul 16, 2025
Merged

fix: update log file paths and bump image tags to v2.0.0#53
a-klos merged 1 commit into
mainfrom
fix/path-in-k8s-template-files

Conversation

@a-klos
Copy link
Copy Markdown
Member

@a-klos a-klos commented Jul 16, 2025

Correct log file paths and permissions in deployment configurations while updating all relevant image tags to version 2.0.0.

@a-klos
Copy link
Copy Markdown
Member Author

a-klos commented Jul 16, 2025

this pr fixes:
#52

@a-klos a-klos mentioned this pull request Jul 16, 2025
@huhn511 huhn511 requested a review from Copilot July 16, 2025 07:47
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 PR corrects log file paths and permissions in deployment configurations while updating all component image tags from v1.0.0 to v2.0.0. The changes address inconsistencies in directory structure and improve the Docker build configuration.

  • Updates image tags to v2.0.0 across all services (backend, frontend, admin-backend, admin-frontend, extractor, mcp)
  • Corrects log file paths from /app/{service}/log/ to /app/services/{service}/log/ to match actual directory structure
  • Improves Docker build configuration by adding infrastructure directory to ignore patterns and fixing live update sync paths

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
infrastructure/rag/values.yaml Updates all service image tags from v1.0.0 to v2.0.0
infrastructure/rag/templates/extractor/deployment.yaml Corrects log file paths for document-extractor service
infrastructure/rag/templates/backend/deployment.yaml Corrects log file paths for rag-backend and mcp-server services
infrastructure/rag/templates/admin-backend/deployment.yaml Corrects log file paths for admin-backend service
Tiltfile Adds infrastructure directory to ignore patterns and fixes frontend live update sync paths
Comments suppressed due to low confidence (1)

Tiltfile:282

  • This line removes the sync for rag-core-lib which appears to be intentional based on the diff, but this could break the document extractor if it depends on this library. Verify that the document-extractor service doesn't require rag-core-lib or ensure it's included through another mechanism.
        sync(core_library_context +"/extractor-api-lib", "/app/libs/extractor-api-lib"),

@a-klos a-klos merged commit dab9b2d into main Jul 16, 2025
9 checks passed
@a-klos a-klos deleted the fix/path-in-k8s-template-files branch July 16, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants