From 6e83bcf35e99357119212375d53560babbc7f44d Mon Sep 17 00:00:00 2001 From: Jan Safranek Date: Tue, 6 Aug 2024 13:39:55 +0200 Subject: [PATCH 1/5] Update lib-external-provisioner to v10.0.1 Update sigs.k8s.io/sig-storage-lib-external-provisioner to the latest release to fix removal of the PV finalizer. --- go.mod | 2 +- go.sum | 4 ++-- vendor/modules.txt | 2 +- .../v10/controller/controller.go | 23 ++++++++----------- 4 files changed, 14 insertions(+), 17 deletions(-) diff --git a/go.mod b/go.mod index 874d068471..1f5418f9fb 100644 --- a/go.mod +++ b/go.mod @@ -27,7 +27,7 @@ require ( k8s.io/klog/v2 v2.120.1 sigs.k8s.io/controller-runtime v0.18.2 sigs.k8s.io/gateway-api v1.1.0 - sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.0 + sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.1 ) require ( diff --git a/go.sum b/go.sum index 7ecb69aaf3..38d377ded8 100644 --- a/go.sum +++ b/go.sum @@ -346,8 +346,8 @@ sigs.k8s.io/gateway-api v1.1.0 h1:DsLDXCi6jR+Xz8/xd0Z1PYl2Pn0TyaFMOPPZIj4inDM= sigs.k8s.io/gateway-api v1.1.0/go.mod h1:ZH4lHrL2sDi0FHZ9jjneb8kKnGzFWyrTya35sWUTrRs= sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd h1:EDPBXCAspyGV4jQlpZSudPeMmr1bNJefnuqLsRAsHZo= sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd/go.mod h1:B8JuhiUyNFVKdsE8h686QcCxMaH6HrOAZj4vswFpcB0= -sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.0 h1:+OhFU21hL9Gq/sHKHfYxbc7M6RWV3UqTpnk5/wF9cP4= -sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.0/go.mod h1:mfQ2enu5yAHUhpNWsce9NmkqkRQsk70zQT+7KjZ+JMo= +sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.1 h1:uhd7RobUnVmfkRb3gkYQh4tBOiBEBvdwW/nAthG95Rc= +sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.1/go.mod h1:mfQ2enu5yAHUhpNWsce9NmkqkRQsk70zQT+7KjZ+JMo= sigs.k8s.io/structured-merge-diff/v4 v4.4.1 h1:150L+0vs/8DA78h1u02ooW1/fFq/Lwr+sGiqlzvrtq4= sigs.k8s.io/structured-merge-diff/v4 v4.4.1/go.mod h1:N8hJocpFajUSSeSJ9bOZ77VzejKZaXsTtZo4/u7Io08= sigs.k8s.io/yaml v1.4.0 h1:Mk1wCc2gy/F0THH0TAp1QYyJNzRm2KCLy3o5ASXVI5E= diff --git a/vendor/modules.txt b/vendor/modules.txt index 51e1023e44..b6f76ad2e4 100644 --- a/vendor/modules.txt +++ b/vendor/modules.txt @@ -1495,7 +1495,7 @@ sigs.k8s.io/gateway-api/pkg/client/listers/apis/v1beta1 ## explicit; go 1.18 sigs.k8s.io/json sigs.k8s.io/json/internal/golang/encoding/json -# sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.0 +# sigs.k8s.io/sig-storage-lib-external-provisioner/v10 v10.0.1 ## explicit; go 1.22.0 sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/metrics diff --git a/vendor/sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/controller.go b/vendor/sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/controller.go index 194f6dae74..0b33f66181 100644 --- a/vendor/sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/controller.go +++ b/vendor/sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/controller.go @@ -54,7 +54,7 @@ import ( "k8s.io/client-go/tools/record" ref "k8s.io/client-go/tools/reference" "k8s.io/client-go/util/workqueue" - "k8s.io/klog/v2" + klog "k8s.io/klog/v2" "sigs.k8s.io/sig-storage-lib-external-provisioner/v10/controller/metrics" "sigs.k8s.io/sig-storage-lib-external-provisioner/v10/util" ) @@ -1589,7 +1589,6 @@ func (ctrl *ProvisionController) deleteVolumeOperation(ctx context.Context, volu return fmt.Errorf("expected volume but got %+v", volumeObj) } finalizers, modified := removeFinalizer(newVolume.ObjectMeta.Finalizers, finalizerPV) - // Only update the finalizers if we actually removed something if modified { if _, err = ctrl.patchPersistentVolumeWithFinalizers(ctx, newVolume, finalizers); err != nil { @@ -1608,23 +1607,21 @@ func (ctrl *ProvisionController) deleteVolumeOperation(ctx context.Context, volu return nil } -// removeFinalizer removes finalizer from slice, returns slice and whether modified. +// removeFinalizer removes finalizer from slice, returns the new slice and whether modified. +// It does not modify the original slice. func removeFinalizer(finalizers []string, finalizerToRemove string) ([]string, bool) { - for i, finalizer := range finalizers { - if finalizer == finalizerToRemove { - finalizers = append(finalizers[:i], finalizers[i+1:]...) - if len(finalizers) == 0 { - finalizers = nil - } - return finalizers, true + ret := make([]string, 0, len(finalizers)) + for _, finalizer := range finalizers { + if finalizer != finalizerToRemove { + ret = append(ret, finalizer) } } - if len(finalizers) == 0 { - finalizers = nil + if len(ret) == 0 { + ret = nil } - return finalizers, false + return ret, len(ret) != len(finalizers) } // addFinalizer adds finalizer to slice, returns slice and whether modified. From cbc9a286a6e2d825638df1d02b7501cac5636053 Mon Sep 17 00:00:00 2001 From: Jan Safranek Date: Tue, 6 Aug 2024 14:36:37 +0200 Subject: [PATCH 2/5] Fix typos This is a partial cherry pick of 6625a06 that fixed typos in the master branch. These typos block merges in github. --- pkg/controller/controller_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/controller/controller_test.go b/pkg/controller/controller_test.go index 14c583ad51..d84ffae2ca 100644 --- a/pkg/controller/controller_test.go +++ b/pkg/controller/controller_test.go @@ -598,7 +598,7 @@ func fakeClaim(name, namespace, claimUID string, capacity int64, boundToVolume s case "filesystem": claim.Spec.VolumeMode = &volumeModeFileSystem default: - // leave it undefined/nil to maintaint the current defaults for test cases + // leave it undefined/nil to maintain the current defaults for test cases } return &claim } @@ -5637,7 +5637,7 @@ func generatePVCForProvisionFromPVC(srcNamespace, srcName, scName string, reques case "filesystem": provisionRequest.PVC.Spec.VolumeMode = &volumeModeFileSystem default: - // leave it undefined/nil to maintaint the current defaults for test cases + // leave it undefined/nil to maintain the current defaults for test cases } return provisionRequest @@ -5684,7 +5684,7 @@ func generatePVCForProvisionFromXnsdataSource(scName, namespace string, dataSour case "filesystem": provisionRequest.PVC.Spec.VolumeMode = &volumeModeFileSystem default: - // leave it undefined/nil to maintaint the current defaults for test cases + // leave it undefined/nil to maintain the current defaults for test cases } return provisionRequest From 5b3473f5ed3a0a6322a8ffcbe8108aeaaa199246 Mon Sep 17 00:00:00 2001 From: Jan Safranek Date: Tue, 6 Aug 2024 14:39:12 +0200 Subject: [PATCH 3/5] Squashed 'release-tools/' changes from adb3af9df..988496a1f 988496a1f Merge pull request #257 from jakobmoellerdev/csi-prow-sidecar-e2e-path 028f8c698 chore: bump to Go 1.22.5 69bd71e8a chore: add CSI_PROW_SIDECAR_E2E_PATH f40f0ccd4 Merge pull request #256 from solumath/master cfa92106c Instruction update 379a1bb9b Merge pull request #255 from humblec/sidecar-md a5667bbbb fix typo in sidecar release process 49676850e Merge pull request #254 from bells17/add-github-actions d9bd160c2 Update skip list in codespell GitHub Action f5aebfc9f Add GitHub Actions workflows git-subtree-dir: release-tools git-subtree-split: 988496a1fc3849ed793e03012fdd56813d13d46c --- .github/dependabot.yaml | 12 ++++++++++++ .github/workflows/codespell.yml | 15 +++++++++++++++ .github/workflows/trivy.yaml | 29 +++++++++++++++++++++++++++++ SIDECAR_RELEASE_PROCESS.md | 2 +- generate-patch-release-notes.sh | 16 ++++++++++------ prow.sh | 11 +++++++---- 6 files changed, 74 insertions(+), 11 deletions(-) create mode 100644 .github/dependabot.yaml create mode 100644 .github/workflows/codespell.yml create mode 100644 .github/workflows/trivy.yaml diff --git a/.github/dependabot.yaml b/.github/dependabot.yaml new file mode 100644 index 0000000000..814a34493f --- /dev/null +++ b/.github/dependabot.yaml @@ -0,0 +1,12 @@ +version: 2 +enable-beta-ecosystems: true +updates: +- package-ecosystem: "github-actions" + directory: "/" + schedule: + interval: "daily" + labels: + - "area/dependency" + - "release-note-none" + - "ok-to-test" + open-pull-requests-limit: 10 diff --git a/.github/workflows/codespell.yml b/.github/workflows/codespell.yml new file mode 100644 index 0000000000..e74edcef5e --- /dev/null +++ b/.github/workflows/codespell.yml @@ -0,0 +1,15 @@ +# GitHub Action to automate the identification of common misspellings in text files. +# https://github.com/codespell-project/actions-codespell +# https://github.com/codespell-project/codespell +name: codespell +on: [push, pull_request] +jobs: + codespell: + name: Check for spelling errors + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v4 + - uses: codespell-project/actions-codespell@master + with: + check_filenames: true + skip: "*.png,*.jpg,*.svg,*.sum,./.git,./.github/workflows/codespell.yml,./prow.sh" diff --git a/.github/workflows/trivy.yaml b/.github/workflows/trivy.yaml new file mode 100644 index 0000000000..472984780a --- /dev/null +++ b/.github/workflows/trivy.yaml @@ -0,0 +1,29 @@ +name: Run Trivy scanner for Go version vulnerabilities +on: + push: + branches: + - master + pull_request: +jobs: + trivy: + name: Build + runs-on: ubuntu-latest + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Get Go version + id: go-version + run: | + GO_VERSION=$(cat prow.sh | grep "configvar CSI_PROW_GO_VERSION_BUILD" | awk '{print $3}' | sed 's/"//g') + echo "version=$GO_VERSION" >> $GITHUB_OUTPUT + + - name: Run Trivy scanner for Go version vulnerabilities + uses: aquasecurity/trivy-action@master + with: + image-ref: 'golang:${{ steps.go-version.outputs.version }}' + format: 'table' + exit-code: '1' + ignore-unfixed: true + vuln-type: 'library' + severity: 'CRITICAL,HIGH,MEDIUM,LOW,UNKNOWN' diff --git a/SIDECAR_RELEASE_PROCESS.md b/SIDECAR_RELEASE_PROCESS.md index 647d2342ba..aab8d6e215 100644 --- a/SIDECAR_RELEASE_PROCESS.md +++ b/SIDECAR_RELEASE_PROCESS.md @@ -84,7 +84,7 @@ naming convention `-on-`. 1. Compare the generated output to the new commits for the release to check if any notable change missed a release note. 1. Reword release notes as needed, ideally in the original PRs so that the - release notes can be regnerated. Make sure to check notes for breaking + release notes can be regenerated. Make sure to check notes for breaking changes and deprecations. 1. If release is a new major/minor version, create a new `CHANGELOG-..md` file. diff --git a/generate-patch-release-notes.sh b/generate-patch-release-notes.sh index 2b9c13c387..536a149023 100755 --- a/generate-patch-release-notes.sh +++ b/generate-patch-release-notes.sh @@ -23,13 +23,17 @@ # CSI_RELEASE_TOKEN: Github token needed for generating release notes # GITHUB_USER: Github username to create PRs with # +# Required tools: +# - gh +# - release-notes (https://github.com/kubernetes/release/blob/master/cmd/release-notes/README.md) +# # Instructions: -# 1. Login with "gh auth login" -# 2. Copy this script to the kubernetes-csi directory (one directory above the -# repos) -# 3. Update the repos and versions in the $releases array -# 4. Set environment variables -# 5. Run script from the kubernetes-csi directory +# 1. Install the required tools +# 2. Login with "gh auth login" +# 3. Copy this script to the kubernetes-csi directory (one directory above the repos) +# 4. Update the repos and versions in the $releases array +# 5. Set environment variables +# 6. Run script from the kubernetes-csi directory # # Caveats: # - This script doesn't handle regenerating and updating existing PRs yet. diff --git a/prow.sh b/prow.sh index 049fb79afc..df8c61502f 100755 --- a/prow.sh +++ b/prow.sh @@ -86,7 +86,7 @@ configvar CSI_PROW_BUILD_PLATFORMS "linux amd64 amd64; linux ppc64le ppc64le -pp # which is disabled with GOFLAGS=-mod=vendor). configvar GOFLAGS_VENDOR "$( [ -d vendor ] && echo '-mod=vendor' )" "Go flags for using the vendor directory" -configvar CSI_PROW_GO_VERSION_BUILD "1.22.3" "Go version for building the component" # depends on component's source code +configvar CSI_PROW_GO_VERSION_BUILD "1.22.5" "Go version for building the component" # depends on component's source code configvar CSI_PROW_GO_VERSION_E2E "" "override Go version for building the Kubernetes E2E test suite" # normally doesn't need to be set, see install_e2e configvar CSI_PROW_GO_VERSION_SANITY "${CSI_PROW_GO_VERSION_BUILD}" "Go version for building the csi-sanity test suite" # depends on CSI_PROW_SANITY settings below configvar CSI_PROW_GO_VERSION_KIND "${CSI_PROW_GO_VERSION_BUILD}" "Go version for building 'kind'" # depends on CSI_PROW_KIND_VERSION below @@ -231,8 +231,11 @@ configvar CSI_PROW_E2E_VERSION "$(version_to_git "${CSI_PROW_KUBERNETES_VERSION} configvar CSI_PROW_E2E_REPO "https://github.com/kubernetes/kubernetes" "E2E repo" configvar CSI_PROW_E2E_IMPORT_PATH "k8s.io/kubernetes" "E2E package" -# Local path for e2e tests. Set to "none" to disable. -configvar CSI_PROW_SIDECAR_E2E_IMPORT_PATH "none" "CSI Sidecar E2E package" +# Local path & package path for e2e tests. Set to "none" to disable. +# When using versioned go modules, the import path is the module path whereas the path +# should not contain the version and be the directory where the module is checked out. +configvar CSI_PROW_SIDECAR_E2E_IMPORT_PATH "none" "CSI Sidecar E2E package (go import path)" +configvar CSI_PROW_SIDECAR_E2E_PATH "${CSI_PROW_SIDECAR_E2E_IMPORT_PATH}" "CSI Sidecar E2E path (directory)" # csi-sanity testing from the csi-test repo can be run against the installed # CSI driver. For this to work, deploying the driver must expose the Unix domain @@ -1035,7 +1038,7 @@ run_e2e () ( trap move_junit EXIT if [ "${name}" == "local" ]; then - cd "${GOPATH}/src/${CSI_PROW_SIDECAR_E2E_IMPORT_PATH}" && + cd "${GOPATH}/src/${CSI_PROW_SIDECAR_E2E_PATH}" && run_with_loggers env KUBECONFIG="$KUBECONFIG" KUBE_TEST_REPO_LIST="$(if [ -e "${CSI_PROW_WORK}/e2e-repo-list" ]; then echo "${CSI_PROW_WORK}/e2e-repo-list"; fi)" ginkgo --timeout="${CSI_PROW_GINKGO_TIMEOUT}" -v "$@" "${CSI_PROW_WORK}/e2e-local.test" -- -report-dir "${ARTIFACTS}" -report-prefix local else cd "${GOPATH}/src/${CSI_PROW_E2E_IMPORT_PATH}" && From 761a4f0b2bcf9b2ae4329c0deb18887cdca312bb Mon Sep 17 00:00:00 2001 From: Jan Safranek Date: Tue, 6 Aug 2024 15:33:11 +0200 Subject: [PATCH 4/5] Add changelog for v5.0.2 --- CHANGELOG/CHANGELOG-5.0.md | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) diff --git a/CHANGELOG/CHANGELOG-5.0.md b/CHANGELOG/CHANGELOG-5.0.md index 6a2abc8056..bef1a4f6ed 100644 --- a/CHANGELOG/CHANGELOG-5.0.md +++ b/CHANGELOG/CHANGELOG-5.0.md @@ -1,3 +1,26 @@ +# Release notes for v5.0.2 + +## Changes by Kind + +### Bug or Regression + +- Fixed removal of PV protection finalizer. PVs are no longer Terminating forever after PVC deletion. ([#1251](https://github.com/kubernetes-csi/external-provisioner/pull/1251), [@jsafrane](https://github.com/jsafrane)) + +### Uncategorized + +- Updated go to 1.22.5 ([#1251](https://github.com/kubernetes-csi/external-provisioner/pull/1251), [@jsafrane](https://github.com/jsafrane)) + +## Dependencies + +### Added +_Nothing has changed._ + +### Changed +- sigs.k8s.io/sig-storage-lib-external-provisioner/v10: v10.0.0 → v10.0.1 + +### Removed +_Nothing has changed._ + # Release notes for v5.0.1 [Documentation](https://kubernetes-csi.github.io) From 5d1349849b527bfae97bfbbdb259969785447c4c Mon Sep 17 00:00:00 2001 From: Maxim Patlasov Date: Wed, 20 Mar 2024 20:46:53 -0700 Subject: [PATCH 5/5] UPSTREAM: : Add OpenShift files And remove .github --- .ci-operator.yaml | 4 +++ .github/ISSUE_TEMPLATE/bug_report.md | 25 ----------------- .github/ISSUE_TEMPLATE/enhancement.md | 18 ------------ .github/ISSUE_TEMPLATE/support.md | 16 ----------- .github/PULL_REQUEST_TEMPLATE.md | 40 --------------------------- .github/dependabot.yaml | 40 --------------------------- .github/workflows/codespell.yml | 15 ---------- .snyk | 7 +++++ Dockerfile.openshift | 8 ++++++ Dockerfile.openshift.rhel7 | 8 ++++++ OWNERS | 10 ++----- OWNERS_ALIASES | 10 ++++++- 12 files changed, 38 insertions(+), 163 deletions(-) create mode 100644 .ci-operator.yaml delete mode 100644 .github/ISSUE_TEMPLATE/bug_report.md delete mode 100644 .github/ISSUE_TEMPLATE/enhancement.md delete mode 100644 .github/ISSUE_TEMPLATE/support.md delete mode 100644 .github/PULL_REQUEST_TEMPLATE.md delete mode 100644 .github/dependabot.yaml delete mode 100644 .github/workflows/codespell.yml create mode 100644 .snyk create mode 100644 Dockerfile.openshift create mode 100644 Dockerfile.openshift.rhel7 mode change 120000 => 100644 OWNERS_ALIASES diff --git a/.ci-operator.yaml b/.ci-operator.yaml new file mode 100644 index 0000000000..1e59c02c25 --- /dev/null +++ b/.ci-operator.yaml @@ -0,0 +1,4 @@ +build_root_image: + name: release + namespace: openshift + tag: rhel-9-release-golang-1.22-openshift-4.17 diff --git a/.github/ISSUE_TEMPLATE/bug_report.md b/.github/ISSUE_TEMPLATE/bug_report.md deleted file mode 100644 index ad0d03f9f9..0000000000 --- a/.github/ISSUE_TEMPLATE/bug_report.md +++ /dev/null @@ -1,25 +0,0 @@ ---- -name: Bug Report -about: Create a report to help us improve this project - ---- - - - - -**What happened**: - -**What you expected to happen**: - -**How to reproduce it**: - -**Anything else we need to know?**: - -**Environment**: -- Driver version: -- Kubernetes version (use `kubectl version`): -- OS (e.g. from /etc/os-release): -- Kernel (e.g. `uname -a`): -- Install tools: -- Others: diff --git a/.github/ISSUE_TEMPLATE/enhancement.md b/.github/ISSUE_TEMPLATE/enhancement.md deleted file mode 100644 index a9d33a2af0..0000000000 --- a/.github/ISSUE_TEMPLATE/enhancement.md +++ /dev/null @@ -1,18 +0,0 @@ ---- -name: Enhancement Request -about: Suggest an idea for this project - ---- - - -**Is your feature request related to a problem?/Why is this needed** - - -**Describe the solution you'd like in detail** - - -**Describe alternatives you've considered** - - -**Additional context** - diff --git a/.github/ISSUE_TEMPLATE/support.md b/.github/ISSUE_TEMPLATE/support.md deleted file mode 100644 index 6d3b436fb1..0000000000 --- a/.github/ISSUE_TEMPLATE/support.md +++ /dev/null @@ -1,16 +0,0 @@ ---- -name: Support Request -about: Ask questions about this project - ---- - - - diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md deleted file mode 100644 index d70526403f..0000000000 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ /dev/null @@ -1,40 +0,0 @@ - - -**What type of PR is this?** -> Uncomment only one ` /kind <>` line, hit enter to put that in a new line, and remove leading whitespaces from that line: -> -> /kind api-change -> /kind bug -> /kind cleanup -> /kind design -> /kind documentation -> /kind failing-test -> /kind feature -> /kind flake - -**What this PR does / why we need it**: - -**Which issue(s) this PR fixes**: - -Fixes # - -**Special notes for your reviewer**: - -**Does this PR introduce a user-facing change?**: - -```release-note - -``` diff --git a/.github/dependabot.yaml b/.github/dependabot.yaml deleted file mode 100644 index 064cb4270b..0000000000 --- a/.github/dependabot.yaml +++ /dev/null @@ -1,40 +0,0 @@ -version: 2 -enable-beta-ecosystems: true -updates: -- package-ecosystem: gomod - directory: "/" - allow: - - dependency-type: "all" - schedule: - interval: weekly - groups: - golang-dependencies: - patterns: - - "github.com/golang*" - k8s-dependencies: - patterns: - - "k8s.io*" - - "sigs.k8s.io*" - - "github.com/kubernetes-csi*" - github-dependencies: - patterns: - - "*" - exclude-patterns: - - "github.com/golang*" - - "k8s.io*" - - "sigs.k8s.io*" - - "github.com/kubernetes-csi*" - labels: - - "area/dependency" - - "release-note-none" - - "ok-to-test" - open-pull-requests-limit: 10 -- package-ecosystem: "github-actions" - directory: "/" - schedule: - interval: "daily" - labels: - - "area/dependency" - - "release-note-none" - - "ok-to-test" - open-pull-requests-limit: 10 diff --git a/.github/workflows/codespell.yml b/.github/workflows/codespell.yml deleted file mode 100644 index 06d688d212..0000000000 --- a/.github/workflows/codespell.yml +++ /dev/null @@ -1,15 +0,0 @@ -# GitHub Action to automate the identification of common misspellings in text files. -# https://github.com/codespell-project/actions-codespell -# https://github.com/codespell-project/codespell -name: codespell -on: [push, pull_request] -jobs: - codespell: - name: Check for spelling errors - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v4.1.6 - - uses: codespell-project/actions-codespell@master - with: - check_filenames: true - skip: ./.git,./.github/workflows/codespell.yml,.git,*.png,*.jpg,*.svg,*.sum,./vendor,go.sum,./release-tools/prow.sh diff --git a/.snyk b/.snyk new file mode 100644 index 0000000000..186efac005 --- /dev/null +++ b/.snyk @@ -0,0 +1,7 @@ +# References: +# https://docs.snyk.io/scan-applications/snyk-code/using-snyk-code-from-the-cli/excluding-directories-and-files-from-the-snyk-code-cli-test +# https://docs.snyk.io/snyk-cli/commands/ignore +exclude: + global: + - vendor/** + - release-tools/** diff --git a/Dockerfile.openshift b/Dockerfile.openshift new file mode 100644 index 0000000000..6f250325b8 --- /dev/null +++ b/Dockerfile.openshift @@ -0,0 +1,8 @@ +FROM registry.svc.ci.openshift.org/openshift/release:golang-1.15 AS builder +WORKDIR /go/src/github.com/kubernetes-csi/external-provisioner +COPY . . +RUN make build + +FROM registry.svc.ci.openshift.org/openshift/origin-v4.7:base +COPY --from=builder /go/src/github.com/kubernetes-csi/external-provisioner/bin/csi-provisioner /usr/bin/ +ENTRYPOINT ["/usr/bin/csi-provisioner"] diff --git a/Dockerfile.openshift.rhel7 b/Dockerfile.openshift.rhel7 new file mode 100644 index 0000000000..070083fff5 --- /dev/null +++ b/Dockerfile.openshift.rhel7 @@ -0,0 +1,8 @@ +FROM registry.ci.openshift.org/ocp/builder:rhel-9-golang-1.22-openshift-4.17 AS builder +WORKDIR /go/src/github.com/kubernetes-csi/external-provisioner +COPY . . +RUN make build + +FROM registry.ci.openshift.org/ocp/4.17:base-rhel9 +COPY --from=builder /go/src/github.com/kubernetes-csi/external-provisioner/bin/csi-provisioner /usr/bin/ +ENTRYPOINT ["/usr/bin/csi-provisioner"] diff --git a/OWNERS b/OWNERS index ca2de348c1..a3edb31917 100644 --- a/OWNERS +++ b/OWNERS @@ -1,11 +1,5 @@ # See the OWNERS docs at https://go.k8s.io/owners approvers: -- kubernetes-csi-approvers -- pohly - -reviewers: -- kubernetes-csi-reviewers - -# emeritus_reviewers: -# - davidz627 +- openshift-storage-maintainers +component: "Storage / Kubernetes External Components" diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES deleted file mode 120000 index 3f60d46159..0000000000 --- a/OWNERS_ALIASES +++ /dev/null @@ -1 +0,0 @@ -release-tools/KUBERNETES_CSI_OWNERS_ALIASES \ No newline at end of file diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES new file mode 100644 index 0000000000..c448025044 --- /dev/null +++ b/OWNERS_ALIASES @@ -0,0 +1,9 @@ +aliases: + openshift-storage-maintainers: + - jsafrane + - tsmetana + - gnufied + - bertinatto + - dobsonj + - RomanBednar + - mpatlasov