scottk/vault
1
0
Fork 0
mirror of https://github.com/optim-enterprises-bv/vault.git synced 2025-10-29 09:42:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

disable ldap tests for arm (#23118)

Browse Source
This commit is contained in:
Raymond Ho
2023-09-18 08:48:03 -07:00
committed by GitHub
parent 464aeebddc
commit 6e5bd7abd9
2 changed files with 11 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
changelog/23118.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
ldaputil: Disable tests for ARM64
```

8
helper/testhelpers/ldap/ldaphelper.go
View File

@@ -6,6 +6,8 @@ package ldap
import ( import (
"context" "context"
"fmt" "fmt"
"runtime"
"strings"
"testing" "testing"
"github.com/hashicorp/cap/ldap" "github.com/hashicorp/cap/ldap"
@@ -15,6 +17,12 @@ import (
) )
func PrepareTestContainer(t *testing.T, version string) (cleanup func(), cfg *ldaputil.ConfigEntry) { func PrepareTestContainer(t *testing.T, version string) (cleanup func(), cfg *ldaputil.ConfigEntry) {
// note: this image isn't supported on arm64 architecture in CI.
// but if you're running on Apple Silicon, feel free to comment out the code below locally.
if strings.Contains(runtime.GOARCH, "arm") {
t.Skip("Skipping, as this image is not supported on ARM architectures")
}
runner, err := docker.NewServiceRunner(docker.RunOptions{ runner, err := docker.NewServiceRunner(docker.RunOptions{
// Currently set to "michelvocks" until https://github.com/rroemhild/docker-test-openldap/pull/14 // Currently set to "michelvocks" until https://github.com/rroemhild/docker-test-openldap/pull/14
// has been merged. // has been merged.