diff --git a/pkg/controller/backup_controller.go b/pkg/controller/backup_controller.go index ae4cbb9c9..0e53121f0 100644 --- a/pkg/controller/backup_controller.go +++ b/pkg/controller/backup_controller.go @@ -540,8 +540,8 @@ func (b *backupReconciler) validateAndGetSnapshotLocations(backup *velerov1api.B if len(errors) > 0 { return nil, errors } - vsls := &velerov1api.VolumeSnapshotLocationList{} - err := b.kbClient.List(context.Background(), vsls, &kbclient.ListOptions{Namespace: backup.Namespace, LabelSelector: labels.Everything()}) + volumeSnapshotLocations := &velerov1api.VolumeSnapshotLocationList{} + err := b.kbClient.List(context.Background(), volumeSnapshotLocations, &kbclient.ListOptions{Namespace: backup.Namespace, LabelSelector: labels.Everything()}) if err != nil { errors = append(errors, fmt.Sprintf("error listing volume snapshot locations: %v", err)) return nil, errors @@ -549,8 +549,8 @@ func (b *backupReconciler) validateAndGetSnapshotLocations(backup *velerov1api.B // build a map of provider->list of all locations for the provider allProviderLocations := make(map[string][]*velerov1api.VolumeSnapshotLocation) - for i := range vsls.Items { - loc := vsls.Items[i] + for i := range volumeSnapshotLocations.Items { + loc := volumeSnapshotLocations.Items[i] allProviderLocations[loc.Spec.Provider] = append(allProviderLocations[loc.Spec.Provider], &loc) } diff --git a/site/content/docs/main/code-standards.md b/site/content/docs/main/code-standards.md index 732389cf4..36984612f 100644 --- a/site/content/docs/main/code-standards.md +++ b/site/content/docs/main/code-standards.md @@ -8,6 +8,24 @@ toc: "true" When opening a pull request, please fill out the checklist supplied the template. This will help others properly categorize and review your pull request. +### PR title + +Make sure that the pull request title summarizes the change made (and not just "fixes issue #xxxx"): + +Example PR titles: + + - "Check for nil when validating foo" + - "Issue #1234: Check for nil when validating foo" + +### Cherry-pick PRs + +When a PR to main needs to be cherry-picked to a release branch, please wait until the main PR is merged first before creating the CP PR. If the CP PR is made before the main PR is merged, there is a risk that PR modifications in response to review comments will not make it into the CP PR. + +The Cherry-pick PR title should reference the branch it's cherry-picked to and the fact that it's a CP of a commit to main: + + - "[release-1.13 CP] Issue #1234: Check for nil when validating foo" + + ## Adding a changelog Authors are expected to include a changelog file with their pull requests. The changelog file diff --git a/site/content/docs/main/development.md b/site/content/docs/main/development.md index 82ace894d..37ed1e700 100644 --- a/site/content/docs/main/development.md +++ b/site/content/docs/main/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -53,4 +49,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. -[11]: vendoring-dependencies.md diff --git a/site/content/docs/main/troubleshooting.md b/site/content/docs/main/troubleshooting.md index a3bd3caa3..68d2663f1 100644 --- a/site/content/docs/main/troubleshooting.md +++ b/site/content/docs/main/troubleshooting.md @@ -46,6 +46,8 @@ kubectl edit deployment/velero -n velero ... ``` +**Note:** Velero plugins are started as separate processes and once the Velero operation is done (either successfully or not), they exit. So, if you see **received EOF, stopping recv loop** messages in debug logs, that does not mean an error occurred, just that a plugin finished executing. + ## Known issue with restoring LoadBalancer Service Because of how Kubernetes handles Service objects of `type=LoadBalancer`, when you restore these objects you might encounter an issue with changed values for Service UIDs. Kubernetes automatically generates the name of the cloud resource based on the Service UID, which is different when restored, resulting in a different name for the cloud load balancer. If the DNS CNAME for your application points to the DNS name of your cloud load balancer, you'll need to update the CNAME pointer when you perform a Velero restore. diff --git a/site/content/docs/main/vendoring-dependencies.md b/site/content/docs/main/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/main/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.10/development.md b/site/content/docs/v1.10/development.md index 82ace894d..37ed1e700 100644 --- a/site/content/docs/v1.10/development.md +++ b/site/content/docs/v1.10/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -53,4 +49,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.10/vendoring-dependencies.md b/site/content/docs/v1.10/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.10/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.11/development.md b/site/content/docs/v1.11/development.md index 82ace894d..37ed1e700 100644 --- a/site/content/docs/v1.11/development.md +++ b/site/content/docs/v1.11/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -53,4 +49,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.11/vendoring-dependencies.md b/site/content/docs/v1.11/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.11/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.12/development.md b/site/content/docs/v1.12/development.md index 82ace894d..37ed1e700 100644 --- a/site/content/docs/v1.12/development.md +++ b/site/content/docs/v1.12/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -53,4 +49,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.12/vendoring-dependencies.md b/site/content/docs/v1.12/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.12/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.4/development.md b/site/content/docs/v1.4/development.md index c805a27da..de93441cb 100644 --- a/site/content/docs/v1.4/development.md +++ b/site/content/docs/v1.4/development.md @@ -25,9 +25,3 @@ You can run `make verify` to ensure that all generated files (clientset, listers ## Test To run unit tests, use `make test`. - -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.4/vendoring-dependencies.md b/site/content/docs/v1.4/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.4/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.5/development.md b/site/content/docs/v1.5/development.md index 0a9596188..0fd3996a4 100644 --- a/site/content/docs/v1.5/development.md +++ b/site/content/docs/v1.5/development.md @@ -39,8 +39,3 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.5/vendoring-dependencies.md b/site/content/docs/v1.5/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.5/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.6/development.md b/site/content/docs/v1.6/development.md index 0a9596188..0fd3996a4 100644 --- a/site/content/docs/v1.6/development.md +++ b/site/content/docs/v1.6/development.md @@ -39,8 +39,3 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.6/vendoring-dependencies.md b/site/content/docs/v1.6/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.6/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.7/development.md b/site/content/docs/v1.7/development.md index 82ace894d..625bbd6fe 100644 --- a/site/content/docs/v1.7/development.md +++ b/site/content/docs/v1.7/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -52,5 +48,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - ``` **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.7/vendoring-dependencies.md b/site/content/docs/v1.7/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.7/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.8/development.md b/site/content/docs/v1.8/development.md index 82ace894d..625bbd6fe 100644 --- a/site/content/docs/v1.8/development.md +++ b/site/content/docs/v1.8/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -52,5 +48,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - ``` **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.8/vendoring-dependencies.md b/site/content/docs/v1.8/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.8/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html diff --git a/site/content/docs/v1.9/development.md b/site/content/docs/v1.9/development.md index 82ace894d..625bbd6fe 100644 --- a/site/content/docs/v1.9/development.md +++ b/site/content/docs/v1.9/development.md @@ -39,10 +39,6 @@ You can also override the default list of linters by running the command To run unit tests, use `make test`. -## Vendor dependencies - -If you need to add or update the vendored dependencies, see [Vendoring dependencies][11]. - ## Using the main branch If you are developing or using the main branch, note that you may need to update the Velero CRDs to get new changes as other development work is completed. @@ -52,5 +48,3 @@ velero install --crds-only --dry-run -o yaml | kubectl apply -f - ``` **NOTE:** You could change the default CRD API version (v1beta1 _or_ v1) if Velero CLI can't discover the Kubernetes preferred CRD API version. The Kubernetes version < 1.16 preferred CRD API version is v1beta1; the Kubernetes version >= 1.16 preferred CRD API version is v1. - -[11]: vendoring-dependencies.md diff --git a/site/content/docs/v1.9/vendoring-dependencies.md b/site/content/docs/v1.9/vendoring-dependencies.md deleted file mode 100644 index 9fc1bcac1..000000000 --- a/site/content/docs/v1.9/vendoring-dependencies.md +++ /dev/null @@ -1,21 +0,0 @@ ---- -title: "Vendoring dependencies" -layout: docs ---- - -## Overview - -We are using [dep][0] to manage dependencies. You can install it by following [these -instructions][1]. - -## Adding a new dependency - -Run `dep ensure`. If you want to see verbose output, you can append `-v` as in -`dep ensure -v`. - -## Updating an existing dependency - -Run `dep ensure -update [ ...]` to update one or more dependencies. - -[0]: https://github.com/golang/dep -[1]: https://golang.github.io/dep/docs/installation.html