Merged
Conversation
ZePan110
requested review from
Spycsh,
chensuyue and
lvliang-intel
as code owners
ZePan110
force-pushed
the
helm-ci
branch
3 times, most recently
from
f35b502 to
3608743
Compare
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
ZePan110
force-pushed
the
helm-ci
branch
3 times, most recently
from
986a3a6 to
0fd0b34
Compare
ZePan110
force-pushed
the
helm-ci
branch
7 times, most recently
from
7467d72 to
435fa22
Compare
ZePan110
force-pushed
the
helm-ci
branch
2 times, most recently
from
88daec6 to
076a1c1
Compare
ZePan110
force-pushed
the
helm-ci
branch
3 times, most recently
from
e52d376 to
1f2a168
Compare
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
ZePan110
force-pushed
the
helm-ci
branch
8 times, most recently
from
4ab30c3 to
89f23f6
Compare
chensuyue
approved these changes
Jan 20, 2025
yongfengdu
requested changes
Jan 20, 2025
yongfengdu
approved these changes
Jan 20, 2025
lianhao
reviewed
Jan 20, 2025
| exit 0 | ||
| fi | ||
|
|
||
| if ! helm install --create-namespace --namespace $NAMESPACE $RELEASE_NAME oci://ghcr.io/opea-project/charts/${CHART_NAME} --set global.HUGGINGFACEHUB_API_TOKEN=${HFTOKEN} --set global.modelUseHostPath=/home/sdp/.cache/huggingface/hub -f comps/${{ inputs.service }}/deployment/kubernetes/${value_file} --version 0-latest --wait; then |
Collaborator
There was a problem hiding this comment.
we need to --set GOOGLE_CSE_ID=${GOOGLE_CSE_ID },GOOGLE_API_KEY=${GOOGLE_API_KEY} to make web-retriever helm CI happy.
Collaborator
Author
There was a problem hiding this comment.
I'll create another PR to add them.
Signed-off-by: ZePan110 <ze.pan@intel.com>
Signed-off-by: ZePan110 <ze.pan@intel.com>
smguggen
pushed a commit
to opea-aws-proserve/GenAIComps
that referenced
this pull request
Jan 23, 2025
* Add helm-chart CI test workflow. --------- Signed-off-by: ZePan110 <ze.pan@intel.com>
madison-evans
pushed a commit
to SAPD-Intel/GenAIComps
that referenced
this pull request
May 12, 2025
* Add helm-chart CI test workflow. --------- Signed-off-by: ZePan110 <ze.pan@intel.com>
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
4 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.
Description
Add helm-chart CI test workflow.
Issues
List the issue or RFC link this PR is working on. If there is no such link, please mark it as
n/a.Type of change
List the type of change like below. Please delete options that are not relevant.
Dependencies
List the newly introduced 3rd party dependency if exists.
Tests
Describe the tests that you ran to verify your changes.