Kubernetes: Elimine as falhas do shellcheck

Criado em 16 jan. 2019  ·  120Comentários  ·  Fonte: kubernetes/kubernetes

O que aconteceu : Depois de # 24614 / # 68438, tivemos 283 *.sh arquivos com falha sendo excluídos do shellcheck lints.

O que você esperava que acontecesse : devemos lintar todos os arquivos. Quando os arquivos estão listados em hack/.shellcheck_failures podemos perder erros reais neles. Inicialmente, para colocar o linter no lugar, tivemos que listar muitos arquivos para manter o PR de tamanho razoável, mas o ideal é que possamos consertá-los com o tempo e eliminar esse arquivo.

Como reproduzi-lo (o mínimo e precisamente possível) : hack/verify-shellcheck.sh

Mais alguma coisa que precisamos saber? : Pretendo continuar trabalhando nisso, EG # 72861, # 72955, ...

/ sig testando
/atribuir

Arquivos com falhas conhecidas TODO:

  • [x] ./build/build-image/rsyncd.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/common.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73117)
  • [x] ./build/copy-output.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/lib/release.sh
  • [x] ./build/make-build-image.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/make-clean.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/package-tarballs.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/release-images.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/release-in-a-container.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/release.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/run.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/shell.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./build/util.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72955)
  • [x] ./cluster/addons/addon-manager/kube-addons.sh
  • [x] ./cluster/addons/fluentd-elasticsearch/es-image/run.sh
  • [x] ./cluster/addons/fluentd-elasticsearch/fluentd-es-image/run.sh
  • [x] ./cluster/centos/build.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/config-build.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/config-default.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/config-test.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/deployAddons.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/make-ca-cert.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/master/scripts/flannel.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/node/bin/mk-docker-opts.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/node/scripts/flannel.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/centos/util.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73253)
  • [x] ./cluster/clientbin.sh
  • [x] ./cluster/common.sh
  • [x] ./cluster/gce/config-common.sh
  • [x] ./cluster/gce/config-default.sh
  • [x] ./cluster/gce/config-test.sh
  • [x] ./cluster/gce/delete-stranded-load-balancers.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74192)
  • [] ./cluster/gce/gci/configure-helper.sh
  • [] ./cluster/gce/gci/configure.sh
  • [x] ./cluster/gce/gci/flexvolume_node_setup.sh
  • [x] ./cluster/gce/gci/health-monitor.sh
  • [] ./cluster/gce/gci/master-helper.sh
  • [x] ./cluster/gce/gci/mounter/stage-upload.sh
  • [x] ./cluster/gce/gci/node-helper.sh
  • [x] ./cluster/gce/gci/shutdown.sh
  • [x] ./cluster/gce/list-resources.sh
  • [x] ./cluster/gce/upgrade-aliases.sh
  • [x] ./cluster/gce/upgrade.sh
  • [] ./cluster/gce/util.sh
  • [x] ./cluster/get-kube-binaries.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72861)
  • [x] ./cluster/get-kube-local.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72861)
  • [x] ./cluster/get-kube.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72861)
  • [x] ./cluster/images/conformance/run_e2e.sh
  • [x] ./cluster/images/etcd-empty-dir-cleanup/etcd-empty-dir-cleanup.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74192)
  • [x] ./cluster/juju/prereqs/ubuntu-juju.sh
  • [x] ./cluster/juju/util.sh
  • [x] ./cluster/kube-down.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74060)
  • [x] ./cluster/kube-up.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74449)
  • [x] ./cluster/kube-util.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74449)
  • [x] ./cluster/kubeadm.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74060)
  • [x] ./cluster/kubectl.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74449)
  • [x] ./cluster/kubemark/gce/config-default.sh
  • [x] ./cluster/kubemark/iks/config-default.sh
  • [x] ./cluster/kubemark/util.sh
  • [x] ./cluster/local/util.sh
  • [] ./cluster/log-dump/log-dump.sh
  • [x] ./cluster/pre-existing/util.sh
  • [x] ./cluster/restore-from-backup.sh
  • [x] ./cluster/test-e2e.sh
  • [x] ./cluster/test-network.sh
  • [x] ./cluster/test-smoke.sh
  • [x] ./cluster/update-storage-objects.sh
  • [x] ./cluster/validate-cluster.sh
  • [x] ./hack/benchmark-go.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/build-cross.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/build-go.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/build-ui.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72924)
  • [x] ./hack/cherry_pick_pull.sh
  • [x] ./hack/dev-build-and-push.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73151)
  • [x] ./hack/dev-build-and-up.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73151)
  • [x] ./hack/dev-push-conformance.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73151)
  • [x] ./hack/dev-push-hyperkube.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73151)
  • [x] ./hack/e2e-internal/e2e-cluster-size.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-internal/e2e-down.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-internal/e2e-grow-cluster.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-internal/e2e-shrink-cluster.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-internal/e2e-status.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-internal/e2e-up.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/e2e-node-test.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74230)
  • [x] ./hack/generate-bindata.sh
  • [x] ./hack/generate-docs.sh
  • [x] ./hack/get-build.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74060)
  • [x] ./hack/ginkgo-e2e.sh
  • [x] ./hack/godep-restore.sh
  • [x] ./hack/godep-save.sh
  • [x] ./hack/grab-profiles.sh
  • [x] ./hack/install-etcd.sh
  • [x] ./hack/jenkins/benchmark-dockerized.sh
  • [x] ./hack/jenkins/build.sh
  • [x] ./hack/jenkins/test-dockerized.sh
  • [x] ./hack/jenkins/upload-to-gcs.sh
  • [x] ./hack/jenkins/verify-dockerized.sh
  • [x] ./hack/lib/etcd.sh
  • [x] ./hack/lib/golang.sh
  • [x] ./hack/lib/init.sh
  • [x] ./hack/lib/logging.sh
  • [x] ./hack/lib/protoc.sh
  • [x] ./hack/lib/swagger.sh
  • [x] ./hack/lib/test.sh
  • [x] ./hack/lib/util.sh
  • [x] ./hack/lib/version.sh
  • [x] ./hack/list-feature-tests.sh
  • [x] ./hack/local-up-cluster.sh
  • [x] ./hack/make-rules/build.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/clean.sh
  • [x] ./hack/make-rules/cross.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/helpers/cache_go_dirs.sh
  • [x] ./hack/make-rules/make-help.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/test-cmd.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/test-e2e-node.sh
  • [x] ./hack/make-rules/test-integration.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/test-kubeadm-cmd.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74242)
  • [x] ./hack/make-rules/test.sh
  • [x] ./hack/make-rules/update.sh
  • [x] ./hack/make-rules/verify.sh
  • [x] ./hack/make-rules/vet.sh
  • [x] ./hack/print-workspace-status.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73257)
  • [x] ./hack/run-in-gopath.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73257)
  • [x] ./hack/test-go.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/test-integration.sh
  • [x] ./hack/test-update-storage-objects.sh
  • [x] ./hack/update-all.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/update-api-reference-docs.sh (feito em: https://github.com/kubernetes/kubernetes/pull/72924)
  • [x] ./hack/update-bazel.sh
  • [x] ./hack/update-codegen.sh
  • [x] ./hack/update-generated-device-plugin-dockerized.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-device-plugin.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-docs.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-kms-dockerized.sh
  • [x] ./hack/update-generated-kms.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-kubelet-plugin-registration-dockerized.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-kubelet-plugin-registration.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-pod-resources-dockerized.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-pod-resources.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-protobuf-dockerized.sh
  • [x] ./hack/update-generated-protobuf.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74286)
  • [x] ./hack/update-generated-runtime-dockerized.sh
  • [x] ./hack/update-generated-runtime.sh
  • [x] ./hack/update-generated-swagger-docs.sh
  • [x] ./hack/update-godep-licenses.sh
  • [x] ./hack/update-gofmt.sh
  • [x] ./hack/update-openapi-spec.sh
  • [x] ./hack/update-staging-godeps-dockerized.sh
  • [x] ./hack/update-staging-godeps.sh
  • [x] ./hack/update-swagger-spec.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72924)
  • [x] ./hack/update-translations.sh
  • [x] ./hack/update-workspace-mirror.sh
  • [x] ./hack/verify-all.sh (feito em: https://github.com/kubernetes/kubernetes/pull/74299)
  • [x] ./hack/verify-api-groups.sh
  • [x] ./hack/verify-api-reference-docs.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72924)
  • [x] ./hack/verify-bazel.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74259)
  • [x] ./hack/verify-boilerplate.sh
  • [x] ./hack/verify-cli-conventions.sh
  • [x] ./hack/verify-codegen.sh
  • [x] ./hack/verify-description.sh
  • [x] ./hack/verify-generated-device-plugin.sh
  • [x] ./hack/verify-generated-docs.sh
  • [x] ./hack/verify-generated-files-remake.sh
  • [x] ./hack/verify-generated-files.sh
  • [x] ./hack/verify-generated-kms.sh
  • [x] ./hack/verify-generated-kubelet-plugin-registration.sh
  • [x] ./hack/verify-generated-pod-resources.sh
  • [x] ./hack/verify-generated-protobuf.sh
  • [x] ./hack/verify-generated-runtime.sh
  • [x] ./hack/verify-generated-swagger-docs.sh
  • [x] ./hack/verify-godep-licenses.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74259)
  • [x] ./hack/verify-godeps.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74259)
  • [x] ./hack/verify-gofmt.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74259)
  • [x] ./hack/verify-golint.sh
  • [x] ./hack/verify-govet.sh
  • [x] ./hack/verify-import-boss.sh
  • [x] ./hack/verify-imports.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74259)
  • [x] ./hack/verify-linkcheck.sh
  • [x] ./hack/verify-no-vendor-cycles.sh
  • [x] ./hack/verify-openapi-spec.sh
  • [x] ./hack/verify-pkg-names.sh
  • [x] ./hack/verify-readonly-packages.sh
  • [x] ./hack/verify-spelling.sh
  • [x] ./hack/verify-staging-godeps.sh
  • [x] ./hack/verify-staging-meta-files.sh
  • [x] ./hack/verify-swagger-spec.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/72924)
  • [x] ./hack/verify-symbols.sh
  • [x] ./hack/verify-test-featuregates.sh
  • [x] ./hack/verify-test-images.sh
  • [x] ./hack/verify-test-owners.sh
  • [x] ./hack/verify-typecheck.sh
  • [x] ./pkg/kubectl/cmd/edit/testdata/record_testcase.sh
  • [x] ./pkg/util/verify-util-pkg.sh
  • [x] ./plugin/pkg/admission/imagepolicy/gencerts.sh
  • [x] ./staging/src/k8s.io/apiextensions-apiserver/examples/client-go/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73252)
  • [x] ./staging/src/k8s.io/apiextensions-apiserver/examples/client-go/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73252)
  • [x] ./staging/src/k8s.io/apiextensions-apiserver/hack/build-image.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73252)
  • [x] ./staging/src/k8s.io/apiextensions-apiserver/hack/update-codegen.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73252)
  • [x] ./staging/src/k8s.io/apiextensions-apiserver/hack/verify-codegen.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73252)
  • [x] ./staging/src/k8s.io/apiserver/pkg/admission/plugin/webhook/testcerts/gencerts.sh
  • [x] ./staging/src/k8s.io/apiserver/pkg/util/webhook/gencerts.sh
  • [x] ./staging/src/k8s.io/apiserver/plugin/pkg/authenticator/token/oidc/testdata/gen.sh
  • [x] ./staging/src/k8s.io/apiserver/plugin/pkg/authorizer/webhook/gencerts.sh
  • [x] ./staging/src/k8s.io/code-generator/generate-groups.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73246)
  • [x] ./staging/src/k8s.io/code-generator/generate-internal-groups.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73246)
  • [x] ./staging/src/k8s.io/code-generator/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73246)
  • [x] ./staging/src/k8s.io/code-generator/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73246)
  • [x] ./staging/src/k8s.io/csi-api/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73219)
  • [x] ./staging/src/k8s.io/csi-api/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73219)
  • [x] ./staging/src/k8s.io/kube-aggregator/hack/build-image.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73218)
  • [x] ./staging/src/k8s.io/kube-aggregator/hack/local-up-kube-aggregator.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73218)
  • [x] ./staging/src/k8s.io/kube-aggregator/hack/register-all-apis-from.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73218)
  • [x] ./staging/src/k8s.io/kube-aggregator/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73218)
  • [x] ./staging/src/k8s.io/kube-aggregator/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73218)
  • [x] ./staging/src/k8s.io/metrics/hack/update-codegen.sh
  • [x] ./staging/src/k8s.io/metrics/hack/verify-codegen.sh
  • [x] ./staging/src/k8s.io/node-api/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73213)
  • [x] ./staging/src/k8s.io/node-api/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73213)
  • [x] ./staging/src/k8s.io/sample-apiserver/hack/build-image.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73210)
  • [x] ./staging/src/k8s.io/sample-apiserver/hack/update-codegen.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73210)
  • [x] ./staging/src/k8s.io/sample-apiserver/hack/verify-codegen.sh (feito em: https://github.com/kubernetes/kubernetes/pull/73210)
  • [x] ./staging/src/k8s.io/sample-controller/hack/update-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73213)
  • [x] ./staging/src/k8s.io/sample-controller/hack/verify-codegen.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/73213)
  • [x] ./test/cmd/apply.sh
  • [x] ./test/cmd/apps.sh
  • [x] ./test/cmd/authorization.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/batch.sh
  • [x] ./test/cmd/certificate.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/core.sh
  • [x] ./test/cmd/crd.sh
  • [x] ./test/cmd/create.sh
  • [x] ./test/cmd/delete.sh
  • [x] ./test/cmd/diff.sh
  • [x] ./test/cmd/discovery.sh
  • [x] ./test/cmd/generic-resources.sh
  • [x] ./test/cmd/get.sh
  • [x] ./test/cmd/initializers.sh
  • [x] ./test/cmd/legacy-script.sh
  • [x] ./test/cmd/node-management.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/old-print.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/proxy.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/rbac.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/request-timeout.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/run.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/save-config.sh
  • [x] ./test/cmd/storage.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/template-output.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/cmd/version.sh (endereçado em: https://github.com/kubernetes/kubernetes/pull/74279)
  • [x] ./test/e2e_node/conformance/run_test.sh
  • [x] ./test/e2e_node/environment/setup_host.sh
  • [x] ./test/e2e_node/gubernator.sh
  • [x] ./test/e2e_node/jenkins/conformance/conformance-jenkins.sh
  • [x] ./test/e2e_node/jenkins/copy-e2e-image.sh
  • [x] ./test/e2e_node/jenkins/e2e-node-jenkins.sh
  • [x] ./test/e2e_node/jenkins/ubuntu-14.04-nvidia-install.sh
  • [x] ./test/images/image-util.sh
  • [x] ./test/images/pets/redis-installer/on-start.sh
  • [x] ./test/images/pets/zookeeper-installer/install.sh
  • [x] ./test/images/pets/zookeeper-installer/on-start.sh
  • [x] ./test/images/volume/gluster/run_gluster.sh
  • [x] ./test/images/volume/iscsi/create_block.sh
  • [x] ./test/images/volume/nfs/run_nfs.sh
  • [x] ./test/images/volume/rbd/bootstrap.sh
  • [x] ./test/images/volume/rbd/create_block.sh
  • [x] ./test/images/volume/rbd/mon.sh
  • [x] ./test/images/volume/rbd/osd.sh
  • [x] ./test/integration/ipamperf/test-performance.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74193)
  • [x] ./test/integration/scheduler_perf/test-performance.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74193)
  • [x] ./test/kubemark/common/util.sh
  • [x] ./test/kubemark/configure-kubectl.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74193)
  • [x] ./test/kubemark/gce/util.sh
  • [x] ./test/kubemark/iks/shutdown.sh
  • [x] ./test/kubemark/iks/startup.sh
  • [x] ./test/kubemark/iks/util.sh
  • [x] ./test/kubemark/master-log-dump.sh
  • [x] ./test/kubemark/pre-existing/util.sh (Feito em: https://github.com/kubernetes/kubernetes/pull/74193)
  • [x] ./test/kubemark/resources/start-kubemark-master.sh
  • [x] ./test/kubemark/run-e2e-tests.sh
  • [x] ./test/kubemark/start-kubemark.sh
  • [x] ./test/kubemark/stop-kubemark.sh
  • [x] ./third_party/forked/shell2junit/sh2ju.sh
  • [x] ./third_party/intemp/intemp.sh
  • [x] ./third_party/multiarch/qemu-user-static/register/qemu-binfmt-conf.sh
  • [x] ./third_party/multiarch/qemu-user-static/register/register.sh
help wanted kincleanup prioritimportant-longterm sitesting

Comentários muito úteis

Um conselho para quem está trabalhando nisso: facilite a tarefa para seus revisores. Explique por que você está mudando o que está mudando. "Corrigindo falhas de verificação de shell" não é contexto suficiente.

Alguns detalhes que podem ajudar seu revisor:

  • que falha do shellcheck você está corrigindo?
  • por que você está fazendo assim?
  • por que isso é mais fácil de manter do que antes?
  • se você precisa ignorar uma falha de verificação de shell, por quê? Existe alguma outra maneira de fazer as coisas que realizaria os mesmos objetivos e _não_ acionaria o shellcheck?

Todos 120 comentários

@spiffxp Estou lidando com isso com otimismo para 1,14, mas posso ver caindo para 1,15 dependendo do tempo de revisão, largura de banda, etc.

/ prioridade importante a longo prazo

/ milestone v1.14

@BenTheElder, estaria tudo bem se eu ajudasse com isso? Eu estava pensando que poderia percorrer e cobrir os scripts em ./cluster/centos/ para começar

Isso seria bom! há um _muito_ para resolver aqui: sweat_smile:

Se você pudesse fazer referência a esse problema (apenas inclua um link em algum lugar, não marque-o como "conserta #": upside_down_face :) isso seria útil para tentar evitar sobreposição :-)
Vou continuar a fazer o mesmo conforme obtenho mais desses.

@BenTheElder
Posso ajudar com ./build/common.sh

Vá em frente! 🙂
Podemos até querer vários PRs para aquele - Se bem me lembro,
havia muito para consertar naquele 😛

No domingo, 20 de janeiro de 2019, 15:07 Kartik Yadav < [email protected] escreveu:

@BenTheElder https://github.com/BenTheElder
Posso ajudar com ./build/common.sh

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956#issuecomment-455911313 ,
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/AA4BqwY2Ny9UJxZ9htOYQYpUBnuy9XNnks5vFPakgaJpZM4aClvY
.

vou arrumar
./hack/dev-build-and-push.sh
./hack/dev-build-and-up.sh
./hack/dev-push-conformance.sh
./hack/dev-push-hyperkube.sh

Vou trabalhar nos scripts em ./cluster/gce/

vou arrumar
./cluster/addons/addon-manager/kube-addons.sh
./cluster/addons/fluentd-elasticsearch/es-image/run.sh
./cluster/addons/fluentd-elasticsearch/fluentd-es-image/run.sh

Vou consertar ./staging/

/Socorro
[temos várias pessoas contribuindo, mas há muitos deles para resolver!]

@BenTheElder :
Este pedido foi marcado como precisando da ajuda de um colaborador.

Certifique-se de que a solicitação atenda aos requisitos listados aqui .

Se esta solicitação não atender mais a esses requisitos, o rótulo pode ser removido
comentando com o comando /remove-help .

Em resposta a isso :

/Socorro
[temos várias pessoas contribuindo, mas há muitos deles para resolver!]

Instruções para interagir comigo usando comentários de RP estão disponíveis aqui . Se você tiver dúvidas ou sugestões relacionadas ao meu comportamento, registre um problema no repositório kubernetes / test-infra .

Vou arrumar:

./hack/print-workspace-status.sh
./hack/run-in-gopath.sh
./hack/test-go.sh
./hack/test-integration.sh

Vou arrumar:

./hack/print-workspace-status.sh
./hack/run-in-gopath.sh
./hack/test-go.sh
./hack/test-integration.sh

@ ds-ms eu corrigi /hack/test-go.sh e /hack/test-integration.sh em # 73199

vou consertar /cluster/common.sh

@ mysunshine92 Eu

Vou consertar ./cluster/juju e ./cluster/local

Eu gostaria de consertar arquivos em ./test/cmd/ .

Estou corrigindo hack/verify-generated-files-remake.sh em https://github.com/kubernetes/kubernetes/pull/73464.

Eu gostaria de consertar arquivos em /test/e2e_node .
Para sua informação: ./hack/lib/test.sh foram incluídos em https://github.com/kubernetes/kubernetes/pull/73518

Os scripts kubemark devem ser corrigidos em # 73746

./cluster/kubemark/*
./test/kubemark/*

Vou trabalhar para consertar tudo em ./test/cmd/* seguir.

Consertar

./pkg/kubectl/cmd/edit/testdata/record_testcase.sh
./pkg/util/verify-util-pkg.sh
./plugin/pkg/admission/imagepolicy/gencerts.sh

em https://github.com/kubernetes/kubernetes/pull/73914

Woops, de alguma forma esqueci que ./test/cmd/* .

Eu atualizei a lista de rastreamento no post original com cada PR mesclado até https://github.com/kubernetes/kubernetes/pull/73213 (o mais recente atualmente) AFAICT. 45/283 mesclado atualmente: sweat_smile:

Vou enfrentar test/images/* seguir.

./third_party/* deve ser corrigido em # 74180

fix cluster / addons / fluentd-elasticsearch / fl …… uentd-es-image / run.sh em # 74187

./cluster/gce/delete-stranded-load-balancers.sh
./cluster/images/etcd-empty-dir-cleanup/etcd-empty-dir-cleanup.sh

em # 74192

./test/kubemark/pre-existing/util.sh
./test/kubemark/configure-kubectl.sh
./test/integration/ipamperf/test-performance.sh
./test/integration/scheduler_perf/test-performance.sh

em # 74193

./hack/verify-test-images.sh
./hack/verify-test-owners.sh
./hack/verify-typecheck.sh

em # 74451

Farei hack/lib/util.sh seguir.

./cluster/kube-down.sh
./cluster/kube-up.sh
./cluster/kube-util.sh
./cluster/kubeadm.sh
./cluster/kubectl.sh

em # 74228

./hack/e2e-internal/e2e-cluster-size.sh
./hack/e2e-internal/e2e-down.sh
./hack/e2e-internal/e2e-grow-cluster.sh
./hack/e2e-internal/e2e-shrink-cluster.sh
./hack/e2e-internal/e2e-status.sh
./hack/e2e-internal/e2e-up.sh
./hack/e2e-node-test.sh

em # 74230

./hack/make-rules/build.sh
./hack/make-rules/cross.sh
./hack/make-rules/make-help.sh
./hack/make-rules/test-cmd.sh
./hack/make-rules/test-integration.sh
./hack/make-rules/test-kubeadm-cmd.sh

em # 74242

Lançamento de um diff (https://github.com/kubernetes/kubernetes/pull/74259) com correções para o seguinte :)

hack/verify-bazel.sh
hack/verify-godep-licenses.sh
hack/verify-godeps.sh
hack/verify-gofmt.sh
hack/verify-imports.sh

Obrigado por liderar os esforços para lidar com este @BenTheElder :)

/reabrir
fechado involuntariamente por # 74193

@mrbobbytables : Reabriu este problema.

Em resposta a isso :

/reabrir
fechado involuntariamente por # 74193

Instruções para interagir comigo usando comentários de RP estão disponíveis aqui . Se você tiver dúvidas ou sugestões relacionadas ao meu comportamento, registre um problema no repositório kubernetes / test-infra .

Para aqueles que estão trabalhando nos problemas de verificação de shell, não coloque "correções # 72956" em seu PR. Esse é um problema geral para rastreamento que não deve ser fechado até que todos os problemas de verificação de shell sejam resolvidos.

Sinto muito @mrbobbytables , vou cuidar disso

@BenTheElder Pode haver muitos arquivos que foram reparados ou estão sob aprovação. Você poderia atualizar os arquivos a serem reparados em questão? Isso será mais conveniente! obrigado

Os seguintes scripts corrigidos por # 74299

hack/benchmark-go.sh
hack/verify-all.sh
hack/build-cross.sh
hack/update-all.sh
hack/test-go.sh
hack/build-go.sh

Tentei atualizar algumas das entradas, mas é algo relativamente manual para verificar e tive que dispensar eventualmente para outro trabalho

Eu já havia chamado ./cluster/gce/ , mas não tive muito tempo para trabalhar nisso. Há muitos lá, então não me deixe impedir ninguém de levá-los

Colocando um diff (https://github.com/kubernetes/kubernetes/pull/74332) para o seguinte rn:

hack/godep-restore.sh
hack/godep-save.sh

Os seguintes scripts corrigidos por # 74333

* hack/list-feature-tests.sh
* hack/verify-govet.sh
* hack/verify-staging-meta-files.sh

Obrigado @spiffxp e obrigado a todos! Eu atualizei a lista de verificação no post original novamente. 89/284 abaixo (ganhamos um novo arquivo desde que começamos)

Abordei hack/verify-generated-* em um PR esta manhã :) Pode ser encontrado aqui: https://github.com/kubernetes/kubernetes/pull/74349.

Corrigido por # 74385

  • hack / generate-bindata.sh
  • hack / generate-docs.sh
  • hack / install-etcd.sh
  • hack / update-bazel.sh

Eu fiz um diff esta manhã para abordar o seguinte no hack: https://github.com/kubernetes/kubernetes/pull/74420

hack/verify-linkcheck.sh
hack/verify-pkg-names.sh
hack/verify-spelling.sh
hack/verify-staging-godeps.sh
hack/verify-symbols.sh
hack/verify-test-images.sh
hack/verify-test-owners.sh
hack/verify-typecheck.sh

Corrigido por # 74455

cluster/gce/gci/node-helper.sh
cluster/update-storage-objects.sh
hack/update-generated-runtime.sh

Se você está tentando corrigir um item da lista, lembre-se de avançar até a metade deste problema e expandir os itens ocultos . Há muitos comentários e RP de itens nos quais as pessoas estão trabalhando ativamente que não serão exibidos durante a pesquisa, a menos que sejam expandidos.
screen shot 2019-02-23 at 10 45 35 am

Um conselho para quem está trabalhando nisso: facilite a tarefa para seus revisores. Explique por que você está mudando o que está mudando. "Corrigindo falhas de verificação de shell" não é contexto suficiente.

Alguns detalhes que podem ajudar seu revisor:

  • que falha do shellcheck você está corrigindo?
  • por que você está fazendo assim?
  • por que isso é mais fácil de manter do que antes?
  • se você precisa ignorar uma falha de verificação de shell, por quê? Existe alguma outra maneira de fazer as coisas que realizaria os mesmos objetivos e _não_ acionaria o shellcheck?

test/images/* deve ser resolvido em # 74601

74645

cluster/gce/gci/shutdown.sh
cluster/test-e2e.sh
cluster/test-network.sh

Olá! O congelamento do código para 1.14 ocorrerá em 2 dias. O trabalho nesta edição será implementado na próxima semana?

Corrigido por # 75158

hack/lib/golang.sh

/ milestone v1.15
apenas movê-lo para fins de rastreamento, pois é muito grande - o trabalho pode continuar normalmente

obrigado por mover isto, +1 para 1,15, podemos realmente terminar até então: sweat_smile:

/reabrir

@xichengliudui : Reabriu esta edição.

Em resposta a isso :

/reabrir

Instruções para interagir comigo usando comentários de RP estão disponíveis aqui . Se você tiver dúvidas ou sugestões relacionadas ao meu comportamento, registre um problema no repositório kubernetes / test-infra .

@ s-ito-ts por favor não use Fixes , use Ref .

Eu listo os arquivos de script cujas falhas de lint não são corrigidas nos PRs. Por favor, consulte isso.

./build/lib/release.sh
./cluster/clientbin.sh (conserta uma parte da falha em # 76152)
./cluster/gce/config-common.sh
./cluster/gce/config-default.sh
./cluster/gce/config-test.sh
./cluster/gce/gci/configure-helper.sh
./cluster/gce/gci/configure.sh
./cluster/gce/gci/flexvolume_node_setup.sh
./cluster/gce/gci/health-monitor.sh
./cluster/gce/gci/master-helper.sh
./cluster/gce/list-resources.sh
./cluster/gce/upgrade-aliases.sh
./cluster/gce/upgrade.sh
./cluster/gce/util.sh
./cluster/restore-from-backup.sh
./hack/cherry_pick_pull.sh
./hack/ginkgo-e2e.sh
./hack/grab-profiles.sh
./hack/lib/init.sh
./hack/lib/swagger.sh
./hack/lib/test.sh
./hack/lib/version.sh
./hack/update-vendor.sh
./hack/verify-import-boss.sh
./hack/verify-no-vendor-cycles.sh
./hack/verify-openapi-spec.sh
./hack/verify-readonly-packages.sh
./hack/verify-test-featuregates.sh
./test/cmd/batch.sh
./test/cmd/certificate.sh
./test/cmd/core.sh
./test/cmd/crd.sh
./test/cmd/create.sh
./test/cmd/diff.sh
./test/cmd/discovery.sh
./test/cmd/generic-resources.sh
./test/cmd/get.sh
./test/cmd/legacy-script.sh
./test/cmd/node-management.sh
./test/cmd/old-print.sh
./test/cmd/proxy.sh
./test/cmd/rbac.sh
./test/cmd/request-timeout.sh
./test/cmd/run.sh
./test/cmd/save-config.sh
./test/e2e_node/conformance/run_test.sh
./test/e2e_node/environment/setup_host.sh
./test/e2e_node/gubernator.sh

Os restantes em test/images/volumes devem ser corrigidos em # 74601

test/images/volume/gluster/run_gluster.sh
test/images/volume/iscsi/create_block.sh
test/images/volume/nfs/run_nfs.sh

alguns arquivos em / hack / make-rules / foram corrigidos em 2 PRs. você poderia ajudar a revisar?

76420

  • /hack/make-rules/update.sh
  • /hack/make-rules/verify.sh

75859

  • /hack/make-rules/clean.sh
  • /hack/make-rules/test.sh
  • /hack/make-rules/vet.sh

estamos chegando lá! desculpe, eu não tive tempo para continuar editando essa lista de verificação, https://github.com/kubernetes/kubernetes/blob/master/hack/.shellcheck_failures está diminuindo!

Olá! Estamos iniciando o Bug Freeze para 1.15. Esse problema ainda está planejado para ser incorporado ao 1.15?

Acho que, de maneira otimista, vamos terminar isso em 1.16. houve
progresso considerável, mas eu duvido que o restante será concluído antes do código
congelar...

Na quarta-feira, 29 de maio de 2019, 20:51 Dave Strebel [email protected] escreveu:

Olá! Estamos iniciando o Bug Freeze para 1.15. Este problema ainda está planejado
sendo incorporado em 1.15?

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK7BDIWR5EKZA3HIU23PX5FLVA5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWRIXIA#issuecomment-497191840 ,
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/AAHADK2TA2OSNWG52PXYXSDPX5FLVANCNFSM4GQKLPMA
.

/ milestone v1.16

estamos chegando perto! alguns dos restantes são bastante nodosos :-)

Na terça-feira, 25 de junho de 2019 às 10:23:48 PM -0700, Benjamin Elder escreveu:

estamos chegando perto! alguns dos restantes são bastante nodosos :-)

-
Você está recebendo isso porque comentou.
Responda a este e-mail diretamente ou visualize-o no GitHub:
https://github.com/kubernetes/kubernetes/issues/72956#issuecomment -505723046

Excelente trabalho conduzindo isso :) Obrigado pelo seu trabalho !!

Eu atualizei a lista de verificação, uma vez que me senti bem, mas parei de tentar rastrear cada RP mapeado para ela 😅

Super perto :-)

Não acho que faremos o 1.16, não tive muito tempo para trabalhar neles e alguns dos outros são ... desafiadores. No entanto, estamos reduzidos a apenas 24 arquivos!

Na quinta-feira, 25 de julho de 2019 às 12:30:04 - 0700, Benjamin Elder escreveu:

Não acho que faremos o 1.16, não tive muito tempo para trabalhar neles e alguns dos outros são ... desafiadores. No entanto, estamos reduzidos a apenas 24 arquivos!

-
Você está recebendo isso porque comentou.
Responda a este e-mail diretamente ou visualize-o no GitHub:
https://github.com/kubernetes/kubernetes/issues/72956#issuecomment -514931006

Uau, trabalho fantástico, só faltam 24 arquivos :)

Criei PRs para corrigir os seguintes arquivos de script. Alguém poderia revisar os PRs?

  • ./cluster/gce/gci/flexvolume_node_setup.sh # 81061
  • ./cluster/test-e2e.sh # 80971
  • ./test/images/image-util.sh # 80742

Tentando voltar a isso.

PR # 81882 enviado para hack/lib/test.sh

Você poderia revisar os PRs?

  • validate-cluster.sh # 81345
  • image-util.sh # 80742

Olá, estou passando esta tarde atualizando a revisão de RP e minha caixa de entrada
atualmente .... vou chegar a estes.

Na segunda-feira, 26 de agosto de 2019 às 4:33 PM Kohei Toyoda [email protected]
escrevi:

Você poderia revisar os PRs?

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK7I6N5BJPMAG2XHOTLQGRR5NA5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXWOR5WOR5WOR5NVREX63WOR5G5D5NVRWWWK3TUL52HS4DFVREX6WOR5WOR5NZ65WOR5G250DZ65WOR5NZ65WOR5DZ650D5WOR50DZ650DZ50D5WOR50D5DZ50D5WR5D5WHYYYYYY3PNVWW-C650D5WH5D5NVREX5WOR5G5D5DVREXWOR5WR5WR5D5NVRWRWR5WRD5D5DZ6WR5WRD5WR5D5WR5WR5D5NVReq .
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/AAHADK4BZSJHYSQ7ZCXNUITQGRR5NANCNFSM4GQKLPMA
.

No futuro, por favor me adicione ao PR (/cc @BenTheElder para revisão, /assign @BenTheElder para aprovação se for revisado e eu sou um PROPRIETÁRIO dos arquivos relevantes).

Eu uso https://gubernator.k8s.io/pr para monitorar PRs que chegam, esses PRs parecem ser atribuídos a outras pessoas (que não estão respondendo :()

Olá! Faço parte da equipe de triagem de bug para o ciclo de lançamento 1.16 e, considerando que este problema está marcado para 1.16, mas não é atualizado por um longo tempo, gostaria de verificar seu status. O congelamento do código está começando em 29 de agosto, o que significa que deve haver um PR pronto (e mesclado) até então.

Ainda pretendemos que esse problema seja corrigido para 1.16? Caso contrário, marque novamente o problema para o marco planejado.

Está muito perto, mas vai ser 1,17.
Não acredito que tenho permissão para mudar marcos, ou teria
já.

Na quarta-feira, 28 de agosto de 2019, 06:44 Tarun Pothulapati [email protected]
escrevi:

Olá! Faço parte da equipe de triagem de bugs para o ciclo de lançamento 1.16 e
considerando que este problema está marcado para 1.16, mas não é atualizado por um longo tempo,
Eu gostaria de verificar seu status. O congelamento do código começa em 29 de agosto
o que significa que deve haver um PR pronto (e mesclado) até então.

Ainda pretendemos que esse problema seja corrigido para 1.16? Caso contrário, marque novamente
a questão para o marco planejado.

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK6I5MGAGUOGVSQLBITQGZ6L5A5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5LFCCQ#issuecomment-525750538 ,
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/AAHADKZ2UVXOFVEG35CKFTLQGZ6L5ANCNFSM4GQKLPMA
.

/ milestone v1.17

@BenTheElder :
Eu quero ajuda.
deixe-me trabalhar nestes arquivos:

./build/lib/release.sh
./cluster/common.sh
./cluster/gce/config-common.sh

vou arrumar
./cluster/gce/gci/configure.sh

Olá @BenTheElder

A equipe de triagem de bug está verificando. O congelamento do código para o lançamento 1.17 é em 14 de novembro. Obrigado!

Provavelmente, isso irá para 1.18. Temos alguns novos problemas para corrigir com o
próxima versão do shellcheck WIP.

No sábado, 26 de outubro de 2019, 11h56, Kirsten [email protected] escreveu:

Olá @BenTheElder https://github.com/BenTheElder

A equipe de triagem de bug está verificando. O congelamento do código para a versão 1.17 é
em 18 de novembro. Ainda está previsto para 1,17? Obrigado!

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK65MQNWSORQIGTS2G3QQSHFBA5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKOQUQ#issuecomment-546629714 ,
ou cancelar
https://github.com/notifications/unsubscribe-auth/AAHADK2KHAWZDEJCKZP2LCTQQSHFBANCNFSM4GQKLPMA
.

@josiahbjorgaard : O marco fornecido não é válido para este repositório. Marcos neste repositório: [ next-candidate , v1.13 , v1.14 , v1.15 , v1.16 , v1.17 , v1.18 , v1.19 , v2.0 ]

Use /milestone clear para limpar o marco.

Em resposta a isso :

/ milestone próximo

Instruções para interagir comigo usando comentários de RP estão disponíveis aqui . Se você tiver dúvidas ou sugestões relacionadas ao meu comportamento, registre um problema no repositório kubernetes / test-infra .

/ milestone v1.18

@BenTheElder :
Eu quero ajuda.
deixe-me trabalhar nestes arquivos:

./build/lib/release.sh
./cluster/common.sh
./cluster/gce/config-common.sh

Qual é o status atual deles? Os dois primeiros desses arquivos ainda são encontrados em hack / .shellcheck_failures, mas talvez haja correções chegando? Ou você gostaria de um ala para isso?

Sugestão para corrigir ./build/lib/release.sh encontrada em https://github.com/kubernetes/kubernetes/pull/87285

Olá, a maioria deles tem algumas tentativas de consertá-los durante o voo, mas têm sido difíceis de pousar (entre algumas falhas estranhas e a obtenção de revisão: /) e tendem a ficar um tanto parados ...: desapontado:
Ainda não posso aprovar algumas das mudanças ( cluster/ ), mas ficarei feliz em encontrar tempo para revisar os PRs.

Olá @BenTheElder !
Equipe de triagem de bug aqui para a versão 1.18. Este é um lembrete amigável de que o congelamento do código está programado para 5 de março. Esse problema ainda está relacionado ao marco 1.18 ou deve se estender ao próximo? Desde já, obrigado!

vamos ter que punt este aqui novamente, fizemos um pequeno progresso, mas está ficando mais difícil obter aprovações nos arquivos restantes (não posso aprovar estes :(). Acho que precisamos de pelo menos 1,19, senão 1,20 para terminar.

Olá @BenTheElder, obrigado pela sua resposta. Vou mover isso para 1,19 por enquanto.
Tenha um bom dia

/ milestone v1.19

atualizou a lista de verificação, 9 para ir! faça um PR para um deles https://github.com/kubernetes/kubernetes/pull/88349

atualizou a lista de verificação, 9 para ir! tenha um PR de um deles # 88349

@ k-toyoda-pi tem https://github.com/kubernetes/kubernetes/pull/82059 para cluster / pre-existing / util.sh que parece estar bloqueado desde novembro?

Ganhamos um novo cluster / aprovador ou dois desde então, vou deixar uma nota
e ver se conseguimos aprovar finalmente ...

Na segunda-feira, 24 de fevereiro de 2020, 01:41 Joakim Roubert [email protected] escreveu:

atualizou a lista de verificação, 9 para ir! tenha um PR de um deles # 88349
https://github.com/kubernetes/kubernetes/pull/88349

@ k-toyoda-pi https://github.com/k-toyoda-pi has # 82059
https://github.com/kubernetes/kubernetes/pull/82059 para
cluster / pre-existing / util.sh que parece estar bloqueado desde novembro?

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK2AGTVIVK72R2UVJ33REOI3VA5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMXE36I#issuecomment-590237177 ,
ou cancelar
https://github.com/notifications/unsubscribe-auth/AAHADK7SVYUCF4PVFA4CKYTREOI3VANCNFSM4GQKLPMA
.

Ganhamos um novo cluster / aprovador ou dois desde então, vou deixar uma nota e ver se podemos finalmente aprová-lo ...

Impressionante!

Pesquisando os comentários aqui e a seção de PRs abertos no momento, parece que esses arquivos estão remanescentes como não tendo um PR ou alguém que o reivindicou?

./cluster/gce/config-test.sh
./cluster/gce/gci/configure-helper.sh
./cluster/gce/gci/master-helper.sh
./cluster/gce/upgrade.sh

Criado um stab em cluster / gce / config-test.sh em https://github.com/kubernetes/kubernetes/pull/88459.

@ eixo-joakimr
Posso levar alguns deles?
./cluster/gce/gci/master-helper.sh
./cluster/gce/upgrade.sh

@ eixo-joakimr
Posso levar alguns deles?
./cluster/gce/gci/master-helper.sh
./cluster/gce/upgrade.sh

Isso seria uma pergunta para os administradores, eu acho, mas quanto a mim, não estou trabalhando nisso.

qualquer um é livre para trabalhar em qualquer coisa, RPs bem-vindos 😅

a questão será apenas colocá-los dentro. Peço desculpas por minha própria revisão
latência atualmente, tem um pouco demais acontecendo.

Na quarta-feira, 26 de fevereiro de 2020 às 12h59 Joakim Roubert [email protected]
escrevi:

@ joakimr-axis https://github.com/joakimr-axis
Posso levar alguns deles?
./cluster/gce/gci/master-helper.sh
./cluster/gce/upgrade.sh

Isso seria uma pergunta para os administradores, eu acho, mas quanto a mim, eu não sou
trabalhando nisso.

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/kubernetes/kubernetes/issues/72956?email_source=notifications&email_token=AAHADK2QWEULE6K73OCIHWDREYVPHA5CNFSM4GQKLPMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM7LTVQ#issuecomment-591313366 ,
ou cancelar
https://github.com/notifications/unsubscribe-auth/AAHADKZNXX6HV4E7PLQSF7DREYVPHANCNFSM4GQKLPMA
.

Criado um stab em cluster / gce / config-test.sh em # 88459.

... onde agora todos os testes foram aprovados.

Sugestão para correção de cluster/gce/util.sh em PR # 90405.

Acredito que agora haja ações em andamento para todos os arquivos restantes em hack / .shellcheck_failures:

./cluster/gce/gci/configure-helper.sh # 90433
./cluster/gce/gci/configure.sh # 90442
./cluster/gce/gci/master-helper.sh # 88582 (precisa de rebase, proprietário @gavinfish)
./cluster/gce/util.sh # 90405
./cluster/log-dump/log-dump.sh # 88349

/ milestone v1.20
Eu não acho que vamos atingir 1,19 e está tudo bem.

Desculpe por ter me tornado ~ inativo aqui, ganhei mais funções e estamos lutando contra a saúde dos testes agora, não espero muito tempo para isso neste lançamento.

Atualizei a lista de verificação, rebatizando um PR que abri para log-dump.sh

Atualizei a lista de verificação, rebatizando um PR que abri para log-dump.sh

Com certeza está bem perto agora!

Fechando agora, log-dump.sh parece estar quase pronto. Status atual dos arquivos restantes:

./cluster/gce/gci/configure.sh # 90442
./cluster/gce/gci/master-helper.sh # 88582 ( precisa de rebase , proprietário @gavinfish , _agora obsoleto devido à inatividade_)
./cluster/gce/util.sh # 90405
./cluster/log-dump/log-dump.sh # 88349

log-dump.sh agora está integrado. Remanescente:

./cluster/gce/gci/configure.sh # 90442
./cluster/gce/gci/master-helper.sh # 88582 ( precisa de rebase , proprietário @gavinfish , agora obsoleto devido à inatividade)
./cluster/gce/util.sh # 90405

O PR para cluster / gce / util.sh # 90405 é muito grande (já que há muitas coisas que precisam ser corrigidas nele).
Quando o (grande) tamanho dele foi mencionado, sugeri que talvez pudesse ser dividido em alguns sub-PRs, mas não houve nenhuma entrada sobre isso ainda. O que vocês acham disso?

1, vamos tentar dividir um pouco. Desculpe pela falta de informações (a "fila" às vezes fica longa ) e obrigado por trabalhar nisso!

1, vamos tentar dividir um pouco.

Muito bom, vou tentar e ver se consigo agrupá-lo de uma maneira boa.

(a "fila" às vezes fica longa )

:-D

O sub-PR # 94458 de 90405 foi aprovado.

Ei: wave: Bug Triage aqui. Desejo acompanhar o status desse problema, pois estamos nos aproximando do congelamento de código em 12.11.2020. Este problema está marcado para 1.20, ainda está planejado para este lançamento?

O status atual é (ainda) 3 restantes (embora um punhado de sub-PRs para cluster / gce / util.sh tenha sido mesclado):

./cluster/gce/gci/configure.sh # 90442 (todos os testes foram aprovados, pronto para jay ou não)
./cluster/gce/gci/master-helper.sh # 88582 _ (fechado devido à inatividade!) _
./cluster/gce/util.sh # 90405 (tem sub-PRs # 94456 # 94459 # 94463 dividindo todas as alterações em pedaços menores mais fáceis de manusear)

Eu criei agora o # 95865 que está trazendo o @gavinfish 's closed-due-to-inactivity # 88582 de volta à vida com algumas atualizações. Com aquele, a linha de atividades ativas ficaria assim:

./cluster/gce/gci/configure.sh # 90442 (todos os testes foram aprovados, pronto para Jay ou não)
./cluster/gce/gci/master-helper.sh # 95865
./cluster/gce/util.sh # 90405 (tem sub-PRs # 94456 # 94459 # 94463 dividindo todas as alterações em pedaços menores mais fáceis de manusear)

👋 Olá, Triagem de Bug aqui, estamos bem atrasados ​​no congelamento do código para 1.20 - vou colocar isso no marco 1.21

/ milestone v1.21

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

ttripp picture ttripp  ·  3Comentários

montanaflynn picture montanaflynn  ·  3Comentários

alexferl picture alexferl  ·  3Comentários

arun-gupta picture arun-gupta  ·  3Comentários

theothermike picture theothermike  ·  3Comentários