Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip Aerospike test on 32-bits, warn users it's unsupported. #20825

Merged
merged 2 commits into from
May 29, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions changelog/20825.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:change
storage/aerospike: Aerospike storage shouldn't be used on 32-bit architectures and is now unsupported on them.
```
4 changes: 4 additions & 0 deletions physical/aerospike/aerospike_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ package aerospike

import (
"context"
"math/bits"
"testing"
"time"

Expand All @@ -16,6 +17,9 @@ import (
)

func TestAerospikeBackend(t *testing.T) {
if bits.UintSize == 32 {
t.Skip("Aerospike storage is only supported on 64-bit architectures")
}
cleanup, config := prepareAerospikeContainer(t)
defer cleanup()

Expand Down
3 changes: 3 additions & 0 deletions website/content/docs/configuration/storage/aerospike.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ The Aerospike storage backend is used to persist Vault's data in an
be as knowledgeable about the technology. If you encounter problems with them,
you may be referred to the original author.

- **64-bit only** - the Aerospike storage backend is not supported when Vault runs
on a 32-bit architecture, because the Aerospike Go client doesn't support 32-bits.

```hcl
storage "aerospike" {
hostname = "localhost"
Expand Down