13dab7dd24
* Divest api/ package of deps elsewhere in the nomad repo. This will allow making api/ a module without then pulling in the external repo, leading to a package name conflict. This required some migration of tests to an apitests/ folder (can be moved anywhere as it has no deps on it). It also required some duplication of code, notably some test helpers from api/ -> apitests/ and part (but not all) of testutil/ -> api/testutil/. Once there's more separation and an e.g. sdk/ folder those can be removed in favor of a dep on the sdk/ folder, provided the sdk/ folder doesn't depend on api/ or /. * Also remove consul dep from api/ package * Fix stupid linters * Some restructuring
102 lines
2.4 KiB
Go
102 lines
2.4 KiB
Go
package apitests
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"fmt"
|
|
|
|
"github.com/hashicorp/consul/testutil/retry"
|
|
"github.com/hashicorp/nomad/api"
|
|
"github.com/hashicorp/nomad/testutil"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestAPI_OperatorAutopilotGetSetConfiguration(t *testing.T) {
|
|
t.Parallel()
|
|
require := require.New(t)
|
|
c, s := makeClient(t, nil, nil)
|
|
defer s.Stop()
|
|
|
|
operator := c.Operator()
|
|
var config *api.AutopilotConfiguration
|
|
retry.Run(t, func(r *retry.R) {
|
|
var err error
|
|
config, _, err = operator.AutopilotGetConfiguration(nil)
|
|
r.Check(err)
|
|
})
|
|
require.True(config.CleanupDeadServers)
|
|
|
|
// Change a config setting
|
|
newConf := &api.AutopilotConfiguration{CleanupDeadServers: false}
|
|
_, err := operator.AutopilotSetConfiguration(newConf, nil)
|
|
require.Nil(err)
|
|
|
|
config, _, err = operator.AutopilotGetConfiguration(nil)
|
|
require.Nil(err)
|
|
require.False(config.CleanupDeadServers)
|
|
}
|
|
|
|
func TestAPI_OperatorAutopilotCASConfiguration(t *testing.T) {
|
|
t.Parallel()
|
|
require := require.New(t)
|
|
c, s := makeClient(t, nil, nil)
|
|
defer s.Stop()
|
|
|
|
operator := c.Operator()
|
|
var config *api.AutopilotConfiguration
|
|
retry.Run(t, func(r *retry.R) {
|
|
var err error
|
|
config, _, err = operator.AutopilotGetConfiguration(nil)
|
|
r.Check(err)
|
|
})
|
|
require.True(config.CleanupDeadServers)
|
|
|
|
// Pass an invalid ModifyIndex
|
|
{
|
|
newConf := &api.AutopilotConfiguration{
|
|
CleanupDeadServers: false,
|
|
ModifyIndex: config.ModifyIndex - 1,
|
|
}
|
|
resp, _, err := operator.AutopilotCASConfiguration(newConf, nil)
|
|
require.Nil(err)
|
|
require.False(resp)
|
|
}
|
|
|
|
// Pass a valid ModifyIndex
|
|
{
|
|
newConf := &api.AutopilotConfiguration{
|
|
CleanupDeadServers: false,
|
|
ModifyIndex: config.ModifyIndex,
|
|
}
|
|
resp, _, err := operator.AutopilotCASConfiguration(newConf, nil)
|
|
require.Nil(err)
|
|
require.True(resp)
|
|
}
|
|
}
|
|
|
|
func TestAPI_OperatorAutopilotServerHealth(t *testing.T) {
|
|
t.Parallel()
|
|
c, s := makeClient(t, nil, func(c *testutil.TestServerConfig) {
|
|
c.Server.RaftProtocol = 3
|
|
})
|
|
defer s.Stop()
|
|
|
|
operator := c.Operator()
|
|
testutil.WaitForResult(func() (bool, error) {
|
|
out, _, err := operator.AutopilotServerHealth(nil)
|
|
if err != nil {
|
|
return false, err
|
|
}
|
|
|
|
if len(out.Servers) != 1 ||
|
|
!out.Servers[0].Healthy ||
|
|
out.Servers[0].Name != fmt.Sprintf("%s.global", s.Config.NodeName) {
|
|
return false, fmt.Errorf("%v", out)
|
|
}
|
|
|
|
return true, nil
|
|
}, func(err error) {
|
|
t.Fatalf("err: %v", err)
|
|
})
|
|
}
|