Release - Lucia Training Platform v1.3 #93
Conversation
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
Co-authored-by: Rui Gao <[email protected]>
* add cicd yaml * add deploy all and skip alert-manager * update * fix * update * update * update * fix * update * update * test * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * update * remove deploy all
* update: copilot readme for clarity * cleanup: unnecessary data * fix typo: prompt to summarize ltp_human_intervention questions * remove: unused knowledge from prompt * fix: typo * Update src/copilot-chat/src/copilot_agent/prompts/ltp/gen_result_summary_rejection.txt Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]>
Fix GPU and default Ethernet interface detection in job exporter.
This PR adds version checking and conditional updating for blobfuse2 in the VMSS installation script. The change ensures that blobfuse2 is upgraded to at least version 1.5.0 if an older version is already installed. Key changes: Added version detection for the installed blobfuse2 package Implemented conditional upgrade logic based on minimum version requirement
* improve seperate embedding model url setting env from llm endpoint, for flexible configuration when using sglang interface add default value for COPILOT_EMBEDDING_URL
Update /etc/hosts for dns records in cluster configuration.
…-email (#91) **Description** Add alert logging webhook to replace logging with admin-email implementation to prevent the alert parser fail to capture alerts due to email feature not enable. **Major Change** - Added new /alert-handler/log-alerts webhook endpoint with dedicated controller - Updated alert parsing regex pattern to match new log format - Configured logging webhook for all alert receivers in AlertManager
yukirora
had a problem deploying
to
auto-test
GitHub Actions
Failure
There was a problem hiding this comment.
Pull Request Overview
Merges bug fixes from v1.3 to dev branch, including environment variable fixes, URL configuration updates, GPU detection improvements, documentation cleanup, and alert logging enhancements.
- Fixed critical environment configuration issues including missing
CLUSTER_IDandREST_SERVER_URI - Updated GPU detection to use kernel modules instead of PCI bus for more reliable hardware identification
- Added comprehensive alert logging webhooks and cleaned up unused documentation files
Reviewed Changes
Copilot reviewed 30 out of 30 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/model-proxy/src/proxy/proxy.go | Added logging for request tracking and healthz monitoring |
| src/model-proxy/src/proxy/model_server.go | Fixed REST server URL configuration to use environment variable |
| src/model-proxy/deploy/model-proxy.yaml.template | Added REST_SERVER_URI environment variable configuration |
| src/job-exporter/src/Moneo/src/worker/exporters/node_exporter.py | Improved network interface detection and GPU hardware detection |
| src/job-exporter/build/moneo-gpu-exporter_entrypoint.sh | Updated GPU detection to use kernel modules |
| src/copilot-chat/src/copilot_agent/utils/llmsession.py | Added separate embedding URL configuration |
| src/copilot-chat/src/copilot_agent/prompts/ltp/* | Removed unused documentation file and fixed spelling errors |
| src/copilot-chat/deploy/copilot-chat-deployment.yaml.template | Added missing CLUSTER_ID environment variable |
| src/alert-manager/* | Added comprehensive alert logging functionality |
| contrib/aks/scripts/install-fuse.sh | Added blobfuse2 version check and upgrade logic |
| build/pai_build.py | Added command-line docker registry configuration options |
| .github/workflows/build-deploy-changes.yaml | Added CI/CD workflow for automated service deployment |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
__Description__ Add LTP release note. __Major Revision__ - Add release note for v1.3 v1.2 v1.1 v1.0
yukirora
had a problem deploying
to
auto-test
GitHub Actions
Failure
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Merge bug fixes from v1.3 to dev branch.
Major Revisions