-
Notifications
You must be signed in to change notification settings - Fork 76
LCORE-302: User Data Collection #178
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
Closed
Closed
Changes from all commits
Commits
Show all changes
129 commits
Select commit
Hold shift + click to select a range
bfec4f3
Add data collector configuration and integrate service startup/shutdown
jrobertboos 9f0a47f
LCORE-311: build dev image from main with tag dev-latest (#181)
matysek 4330f7a
LCORE-217: utilities for endpoints handlers
tisnik d198124
Merge pull request #184 from tisnik/utilities-for-endpoints-handlers
tisnik 0cc48c5
Service version v. 0.1.0
tisnik a3b7d51
Merge pull request #185 from tisnik/service-version-0.1.0
tisnik 4ab572b
changes in the makefile
omkarjoshi0304 eaa27fb
Merge pull request #186 from omkarjoshi0304/dev
tisnik 4cfe44a
LCORE-311: Update readme
matysek b6cc25b
Merge pull request #188 from matysek/lcore-311-3
tisnik 37c6181
Configuration loading check in /config endpoint
tisnik f3e00c7
Updated unit tests accordingly
tisnik 25cab0d
Updated OpenAPI schema
tisnik 280d7be
Merge pull request #190 from tisnik/lcore-217-updated-config-endpoint
tisnik b7d13ed
Updated /query endpoint handler
tisnik 8982769
Updated unit tests accordingly
tisnik 11c75a0
Updated OpenAPI schema
tisnik be17079
Merge pull request #191 from tisnik/lcore-217-updated-query-endpoint
tisnik 7cb9308
Additional checks in /models endpoint
tisnik ce89d9d
Unit tests for /models endpoint handler
tisnik 0cb8486
Updated OpenAPI schema
tisnik 0738e68
Merge pull request #192 from tisnik/lcore-217-updated-models-endpoint
tisnik 4299586
Removed unused lockfile
tisnik 2e73753
Merge pull request #193 from tisnik/remove-unused-lockfile
tisnik fe4b0fc
Updated streaming query endpoint
tisnik c1efd8f
Added new unit tests
tisnik cc2d649
Merge pull request #194 from tisnik/lcore-217-updated-streaming-query…
tisnik 0024e06
LCORE-317: tooling to generate list of scenarios
tisnik 9acb538
Generated list of scenarioss
tisnik 9859c0f
Merge pull request #195 from tisnik/lcore-317-tooling-to-generate-lis…
tisnik 48248a0
Do not create new session if conversation_id is provided
rawagner 34a5224
Pass mcp config and auth headers in streaming_query too
rawagner 5737dbf
Merge pull request #179 from rawagner/streaming_mcp
tisnik d11c09d
Introduce authentication to LS-stack API
umago 2aed129
Removed unused dependency
tisnik 71c3730
Merge pull request #196 from tisnik/removed-unused-dependency
tisnik 07abc44
LCORE-303: silent improper Pyright warning in e2e's common.py
tisnik 0b3fdad
Merge pull request #197 from tisnik/lcore-303-silent-improper-pyright…
tisnik 4da4893
Fix K8SAuthDependency docstring
umago c717127
Add data collector service and update Makefile
jrobertboos dbccaa9
Referenced documents support
TamiTakamiya b2385be
Check metadata before building referenced documents list
TamiTakamiya 35feb6b
Merge pull request #183 from lightspeed-core/TamiTakamiya/referenced_…
tisnik dba0c08
Enhance data collector configuration and update related services
jrobertboos da036c0
Add types-requests to development dependencies
jrobertboos 95ab581
Enhance unit tests for DataCollectorService
jrobertboos 9c1783f
Merge pull request #189 from umago/auth
tisnik 7372aa5
Add tests for get_agent
rawagner 1b8fa21
LCORE-293: config file utils
tisnik 548ef80
Added unit tests for the new code
tisnik 0177946
Merge pull request #201 from tisnik/lcore-293-config-file-utils
tisnik fbf3440
LCORE-293: support for customization options
tisnik 5eeb059
Updated unit tests accordingly
tisnik 303846d
Merge pull request #203 from tisnik/lcore-293-support-for-customization
tisnik 8e24e8b
Merge pull request #200 from rawagner/get_agent_tests
tisnik f080a4b
LCORE-326: Add linux/arm64 arch for image building (support Mac users…
matysek 5f4340c
LCORE-293: system prompt retrieval refactoring
tisnik 2bdda1a
Merge pull request #204 from tisnik/lcore-293-system-prompt-refactoring
tisnik e66e1c8
LCORE-326: Add restore dev tagging for image building
matysek a86837c
Merge pull request #206 from matysek/lcore-326-1
tisnik f6adecf
Support MCP Headers
ldjebran 1e28b2d
Merge pull request #202 from ldjebran/add-mcp-headers
ldjebran 480587d
Add CORS middleware to FastAPI application
jrobertboos b2f72b2
Merge pull request #207 from jrobertboos/lcore-335
tisnik dacd970
LCORE-323: LlamaStackClient should be a singleton
manstis e60d7df
LCORE-293: use customized system prompt if provided
tisnik 371b985
Updated unit tests accordingly
tisnik fec6903
Documentation about system prompt
tisnik 0c7de14
Merge pull request #208 from tisnik/lcore-293-use-customized-system-p…
tisnik bf4a166
LCORE-316: bump-up dependencies #1
tisnik d678ede
Merge pull request #209 from tisnik/lcore-316-1
tisnik 2cc5e93
Merge pull request #187 from manstis/LCORE-323
manstis 621b610
Implement the /authorized endpoint
umago 0596fbf
Add work-around for tool_calls with granite models.
manstis a881753
Document and provide test for embedding system_prompt in customizatio…
manstis b78ee00
Merge pull request #212 from manstis/system_prompt_docs
tisnik acdaa3b
Refactor DataCollectorConfiguration and enhance error handling
jrobertboos 98e69de
Merge pull request #211 from manstis/granite_tool_parser
tisnik 7b41edf
Updated OpenAPI schema
tisnik 17bf0c3
Merge pull request #217 from tisnik/updated-openapi-schema
tisnik f5aa9c6
LCORE-326: Add arm64 image building for releases
matysek 5e9f001
Merge pull request #218 from matysek/lcore-326-2
tisnik ecb63cb
Add an option to disable query system prompt
omertuc d501148
Merge pull request #216 from omertuc/disablequerysystemprompt
tisnik 4ad2636
Add work-around for tool_calls with granite models and /query endpoint.
manstis c8ed06e
Merge pull request #219 from manstis/granite_tool_parser
manstis 2f71c65
Merge pull request #210 from umago/authorized-endpoint
tisnik 751d103
Add data collector constants and refactor configuration model
jrobertboos 8dd0435
Allow mcp headers to contain mcp servers names
ldjebran 71d3667
Merge pull request #221 from ldjebran/allow-mcp-headers-with-mcp-serv…
tisnik 2755fea
New Makefile targets
tisnik e4805cc
Howto prepare and publish Python package
tisnik 9640295
Merge pull request #222 from tisnik/lcore-240
tisnik 9fcd408
LCORE-240: build dependencies
tisnik b69c20b
Merge pull request #224 from tisnik/lcore-240-build-dependencies-
tisnik d44e3c6
Add `pyright` configuration file
omertuc 54dee42
Merge pull request #225 from omertuc/pyright
tisnik b1cb492
Unpack auth tuple to get token
omertuc b074aea
Merge pull request #220 from omertuc/token
tisnik 4b9b39c
Containerfile: replace cmd with entrypoint
frzifus 85941bf
Fixed Pylinter/Pyright issues in common_http.py
tisnik a3f4a1d
Fixed Pylinter/Pyright issues in llm_query_response.py
tisnik 8dfdd6e
Better script to list e2e scenarios
tisnik 36225c9
Merge pull request #228 from tisnik/lcore-303-fix-e2e-tests
tisnik 4a4e90c
LCORE-303: updated docstrings
tisnik fcbaaa9
Merge pull request #230 from tisnik/lcore-303-updated-docstrings
tisnik 21d153b
LCORE-316: update deps
tisnik c454da0
Merge pull request #231 from tisnik/lcore-316-update-deps-
tisnik 6fbe07a
LCORE-303: fix integration tests
tisnik 0578b21
Merge pull request #232 from tisnik/lcore-303-fix-integration-tests
tisnik 741d9a7
Unused return values fix
tisnik 6bf1102
Minor issues fix
tisnik 90e48a2
Added missing docstrings
tisnik f570ac2
New tests + fixed existing tests
tisnik a59ba0e
Merge pull request #233 from tisnik/lcore-303-fixed-pylint-issues
tisnik d6eb0d2
Merge pull request #227 from frzifus/main
tisnik 98fed29
Add ingress content service name to data collector configuration
jrobertboos e05b8ae
LCORE-369: Align service URLs with road-core
manstis 2c0acc8
Merge pull request #236 from manstis/LCORE-369
tisnik 4887e2a
Add data collector configuration and integrate service startup/shutdown
jrobertboos 0c7c878
Add data collector service and update Makefile
jrobertboos cdf7907
Enhance data collector configuration and update related services
jrobertboos 773374c
Add types-requests to development dependencies
jrobertboos e33a51b
Enhance unit tests for DataCollectorService
jrobertboos 51ef7f4
Refactor DataCollectorConfiguration and enhance error handling
jrobertboos b866386
Add data collector constants and refactor configuration model
jrobertboos 2e8d59a
Add ingress content service name to data collector configuration
jrobertboos 636b6c9
Merge branch 'lcore-302' of github.com:jrobertboos/lightspeed-stack i…
jrobertboos f8becbf
Update package versions in lock file and clean up config imports
jrobertboos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,61 @@ | ||
| name: Build image, main branch push quay.io | ||
|
|
||
| on: | ||
| push: | ||
| branches: [ main ] | ||
|
|
||
| env: | ||
| IMAGE_NAME: lightspeed-stack | ||
| IMAGE_NAMESPACE: lightspeed-core | ||
| IMAGE_REGISTRY: quay.io | ||
| LATEST_TAG: dev-latest | ||
| CONTAINER_FILE: Containerfile | ||
|
|
||
| jobs: | ||
| build-and-push-dev: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| # Required for image pushing to a registry | ||
| packages: write | ||
| steps: | ||
| - name: Install buildah | ||
| run: | | ||
| sudo apt update | ||
| # qemu is required for arm64 builds | ||
| sudo apt install -y buildah qemu-user-static | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| - name: Create dev image tag | ||
| run: | | ||
| echo "DEV_TAG=dev-$(date +%Y%m%d)-$(git rev-parse --short HEAD)" >> $GITHUB_ENV | ||
| echo "dev image tag: ${{ env.DEV_TAG }}" | ||
| - name: Build image with Buildah | ||
| id: build_image | ||
| uses: redhat-actions/buildah-build@v2 | ||
| with: | ||
| image: ${{ env.IMAGE_NAME }} | ||
| tags: | | ||
| ${{ env.DEV_TAG }} | ||
| ${{ env.LATEST_TAG }} | ||
| containerfiles: | | ||
| ${{ env.CONTAINER_FILE }} | ||
| archs: amd64, arm64 | ||
| oci: true | ||
| - name: Check images | ||
| run: | | ||
| buildah images | grep '${{ env.IMAGE_NAME }}' | ||
| echo '${{ steps.build_image.outputs.image }}' | ||
| echo '${{ steps.build_image.outputs.tags }}' | ||
| - name: Check manifest | ||
| run: | | ||
| set -x | ||
| buildah manifest inspect ${{ steps.build_image.outputs.image }}:${{ env.LATEST_TAG }} | ||
| - name: Push image to Quay.io | ||
| uses: redhat-actions/push-to-registry@v2 | ||
| with: | ||
| image: ${{ steps.build_image.outputs.image }} | ||
| tags: ${{ steps.build_image.outputs.tags }} | ||
| registry: ${{ env.IMAGE_REGISTRY }}/${{ env.IMAGE_NAMESPACE }} | ||
| username: ${{ secrets.QUAY_REGISTRY_USERNAME }} | ||
| password: ${{ secrets.QUAY_REGISTRY_PASSWORD }} | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| name: Check image building | ||
|
|
||
| on: | ||
| - pull_request | ||
|
|
||
| env: | ||
| IMAGE_NAME: lightspeed-stack | ||
| IMAGE_NAMESPACE: lightspeed-core | ||
| LATEST_TAG: dev-latest | ||
| CONTAINER_FILE: Containerfile | ||
|
|
||
| jobs: | ||
| build-pr: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| # Required for image pushing to a registry | ||
| packages: write | ||
| steps: | ||
| - name: Install qemu and buildah | ||
| run: | | ||
| sudo apt update | ||
| # qemu is required for arm64 builds | ||
| sudo apt install -y buildah qemu-user-static | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| - name: Build image with Buildah | ||
| id: build_image | ||
| uses: redhat-actions/buildah-build@v2 | ||
| with: | ||
| image: ${{ env.IMAGE_NAME }} | ||
| tags: | | ||
| ${{ env.LATEST_TAG }} | ||
| containerfiles: | | ||
| ${{ env.CONTAINER_FILE }} | ||
| archs: amd64, arm64 | ||
| oci: true | ||
| - name: Check images | ||
| run: | | ||
| buildah images | grep '${{ env.IMAGE_NAME }}' | ||
| echo '${{ steps.build_image.outputs.image }}' | ||
| echo '${{ steps.build_image.outputs.tags }}' | ||
| - name: Check manifest | ||
| run: | | ||
| set -x | ||
| buildah manifest inspect ${{ steps.build_image.outputs.image }}:${{ env.LATEST_TAG }} |
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
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
Oops, something went wrong.
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.
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.
Bug:
${{ env.DEV_TAG }}isn’t available inside the same stepThe variable is written to
$GITHUB_ENVduring this step and becomes available only after the step finishes, so the echo prints an empty string. Use the shell variable instead:🤖 Prompt for AI Agents