cnti-testcatalog / testsuite

📞📱☎️📡🌐 Cloud Native Telecom Initiative (CNTI) Test Catalog is a tool to check for and provide feedback on the use of K8s + cloud native best practices in networking applications and platforms
https://wiki.lfnetworking.org/display/LN/Test+Catalog
Apache License 2.0
169 stars 70 forks source link

[BUG] Outdated helm repo path in fluentd bitnami install #2016

Open svteb opened 1 month ago

svteb commented 1 month ago

Describe the bug

From /testsuite/src/tasks/utils/fluentd_bitnami.cr:

module FluentDBitnami
  def self.install
    #todo use embedded file to install fluentd values over fluent helm
    #chart
    Log.info {"Installing FluentD Bitnami daemonset "}
    # File.write("fluentd-values.yml", FLUENTD_VALUES)
    # Helm.helm_repo_add("fluent","https://fluent.github.io/helm-charts")
    Helm.helm_repo_add("bitnami","oci://registry-1.docker.io/bitnamicharts")          <---         offender
    #
    # # Install fluentd in the cnf-testsuite namespace
    # Helm.install("--values ./fluentd-values.yml -n #{TESTSUITE_NAMESPACE} fluentd fluent/fluentd")
    # Helm.helm_repo_add("fluent", "https://fluent.github.io/helm-charts")
    Helm.install("--set aggregator.enabled=false -n #{TESTSUITE_NAMESPACE} fluentd bitnami/fluentd")
    KubectlClient::Get.resource_wait_for_install("Daemonset", "fluentd", namespace: TESTSUITE_NAMESPACE)
  end

Unless some authorization is required the repo oci://registry-1.docker.io/bitnamicharts fails with:

Error: looks like "oci://registry-1.docker.io/bitnamicharts" is not a valid chart repository or cannot be reached: object required

To Reproduce

helm repo add bitnami oci://registry-1.docker.io/bitnamicharts

Expected behavior Some live bitnami helm repo should be used, i.e https://charts.bitnami.com/bitnami.