-
Notifications
You must be signed in to change notification settings - Fork 486
Closed
Description
/kind feature
Describe the solution you'd like
[A clear and concise description of what you want to happen.]
As discussed in #1861, we must remove codes installing Go v1.17 on the kubeflow-katib-presubmit
env after the changes on kubeflow/testing#974 are reflected in the public.ecr.aws
registry.
katib/test/e2e/v1beta1/scripts/setup-katib.sh
Lines 79 to 84 in faeb84f
# TODO (tenzen-y): Once the changes on https://github.com/kubeflow/testing/pull/974 are reflected in the `public.ecr.aws` registry, we must remove this process. | |
# To avoid the `../../../../pkg/mod/k8s.io/[email protected]/plugin/pkg/client/auth/exec/metrics.go:21:2: package io/fs is not in GOROOT (/usr/local/go/src/io/fs)` error, | |
# we must use Go v1.16 or later, but as described in https://github.com/kubeflow/training-operator/issues/1581, | |
# we do not have permission to update `public.ecr.aws/j1r0q0g6/kubeflow-testing:latest` so we need to update it in this. | |
rm -rf /usr/local/go | |
wget -O /tmp/go.tar.gz https://dl.google.com/go/go1.17.10.linux-amd64.tar.gz && tar -C /usr/local -xzf /tmp/go.tar.gz |
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Love this feature? Give it a 👍 We prioritize the features with the most 👍