Skip to content

Commit

Permalink
Update the snapshotter to support VSL credential and bump up golang
Browse files Browse the repository at this point in the history
1. Update the snapshotter to support VSL credential
2. Bump up golang to v1.18
3. Remove restic related

Fixes vmware-tanzu/velero#5392
Also related to vmware-tanzu/velero#5313 and vmware-tanzu/velero#5274

Signed-off-by: Wenkai Yin(尹文开) <[email protected]>
Signed-off-by: msdolbey <[email protected]>
  • Loading branch information
ywk253100 authored and msdolbey committed Oct 28, 2022
1 parent 8dba2aa commit 9de7891
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,10 @@ jobs:
runs-on: ubuntu-latest
steps:

- name: Set up Go 1.17
- name: Set up Go
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18
id: go

- name: Check out the code
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/push.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- name: Set up Go
uses: actions/setup-go@v2
with:
go-version: 1.17
go-version: 1.18
id: go

- name: Check out code into the Go module directory
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.

FROM --platform=$BUILDPLATFORM golang:1.17-buster AS build
FROM --platform=$BUILDPLATFORM golang:1.18-buster AS build

ARG TARGETOS
ARG TARGETARCH
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -381,7 +381,7 @@ velero install \
--use-volume-snapshots=false
```
Additionally, you can specify `--use-restic` to enable restic support, and `--wait` to wait for the deployment to be ready.
Additionally, you can specify `--use-node-agent` to enable node agent support, and `--wait` to wait for the deployment to be ready.
### Optional installation steps
1. Specify [additional configurable parameters][7] for the `--backup-location-config` flag.
Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/vmware-tanzu/velero-plugin-for-microsoft-azure

go 1.17
go 1.18

require (
github.com/Azure/azure-sdk-for-go v63.4.0+incompatible
Expand Down
7 changes: 6 additions & 1 deletion velero-plugin-for-microsoft-azure/volume_snapshotter.go
Original file line number Diff line number Diff line change
Expand Up @@ -86,11 +86,16 @@ func (b *VolumeSnapshotter) Init(config map[string]string) error {
subscriptionIDConfigKey,
snapsIncrementalConfigKey,
snapsTagsConfigKey,
credentialsFileConfigKey,
); err != nil {
return err
}

if err := loadCredentialsIntoEnv(credentialsFileFromEnv()); err != nil {
credentialsFile, err := selectCredentialsFile(config)
if err != nil {
return err
}
if err := loadCredentialsIntoEnv(credentialsFile); err != nil {
return err
}

Expand Down

0 comments on commit 9de7891

Please sign in to comment.