Skip to content

Commit

Permalink
rbd: use vaultAuthPath variable name in error msg
Browse files Browse the repository at this point in the history
Before the change, the error msg was the following:
```
failed to set VAULT_AUTH_MOUNT_PATH in Vault config: path is empty
```
`vaultAuthPath` is the actual variable name set by the
user. The error message will now be the following:
```
failed to set "vaultAuthPath" in vault config: path is empty
```

Signed-off-by: Rakshith R <rar@redhat.com>
  • Loading branch information
Rakshith-R authored and mergify[bot] committed May 5, 2022
1 parent 9d7faf8 commit bd57feb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/kms/vault.go
Original file line number Diff line number Diff line change
Expand Up @@ -352,7 +352,7 @@ func initVaultKMS(args ProviderInitArgs) (EncryptionKMS, error) {

kms.vaultConfig[vault.AuthMountPath], err = detectAuthMountPath(vaultAuthPath)
if err != nil {
return nil, fmt.Errorf("failed to set %s in Vault config: %w", vault.AuthMountPath, err)
return nil, fmt.Errorf("failed to set \"vaultAuthPath\" in Vault config: %w", err)
}

vaultRole := vaultDefaultRole
Expand Down

0 comments on commit bd57feb

Please sign in to comment.