Skip to content

Commit

Permalink
Merge pull request #114 from kube-logging/chore/remove-chart-tgz
Browse files Browse the repository at this point in the history
  • Loading branch information
csatib02 authored Dec 6, 2024
2 parents 99620c6 + 935b41c commit e9335ce
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 2 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/artifacts.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,9 @@ jobs:
id: oci-chart-name
run: echo "value=${{ steps.oci-registry-name.outputs.value }}/${{ steps.chart-name.outputs.value }}" >> "$GITHUB_OUTPUT"

- name: Helm dependency update
run: helm dependency update charts/${{ steps.chart-name.outputs.value }}

- name: Helm lint
run: helm lint charts/${{ steps.chart-name.outputs.value }}

Expand Down
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ coverage.html
*~

crddir

charts/telemetry-controller/charts/*.tgz
.licensei.cache

.DS_Store
Expand Down
Binary file not shown.
4 changes: 3 additions & 1 deletion e2e/e2e_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,15 @@ function load_images()

function helm_install_telemetry_controller()
{
local chart_dir="charts/telemetry-controller"
helm dependency update "${chart_dir}"
helm upgrade --install \
--debug \
--wait \
--create-namespace \
-f "e2e/values.yaml" \
telemetry-controller \
"charts/telemetry-controller/"
"${chart_dir}"
}

function test_one_tenant_two_subscriptions()
Expand Down

0 comments on commit e9335ce

Please sign in to comment.