3fcac242c6
This PR introduces the `address` field in the `service` block so that Nomad or Consul services can be registered with a custom `.Address.` to advertise. The address can be an IP address or domain name. If the `address` field is set, the `service.address_mode` must be set in `auto` mode.
178 lines
5.4 KiB
Go
178 lines
5.4 KiB
Go
package command
|
|
|
|
import (
|
|
"fmt"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/hashicorp/nomad/api"
|
|
"github.com/hashicorp/nomad/ci"
|
|
"github.com/hashicorp/nomad/testutil"
|
|
"github.com/mitchellh/cli"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestServiceInfoCommand_Run(t *testing.T) {
|
|
ci.Parallel(t)
|
|
|
|
srv, client, url := testServer(t, true, nil)
|
|
defer srv.Shutdown()
|
|
|
|
// Wait until our test node is ready.
|
|
testutil.WaitForResult(func() (bool, error) {
|
|
nodes, _, err := client.Nodes().List(nil)
|
|
if err != nil {
|
|
return false, err
|
|
}
|
|
if len(nodes) == 0 {
|
|
return false, fmt.Errorf("missing node")
|
|
}
|
|
if _, ok := nodes[0].Drivers["mock_driver"]; !ok {
|
|
return false, fmt.Errorf("mock_driver not ready")
|
|
}
|
|
return true, nil
|
|
}, func(err error) {
|
|
require.NoError(t, err)
|
|
})
|
|
|
|
ui := cli.NewMockUi()
|
|
cmd := &ServiceInfoCommand{
|
|
Meta: Meta{
|
|
Ui: ui,
|
|
flagAddress: url,
|
|
},
|
|
}
|
|
|
|
// Run the command without any arguments to ensure we are performing this
|
|
// check.
|
|
require.Equal(t, 1, cmd.Run([]string{"-address=" + url}))
|
|
require.Contains(t, ui.ErrorWriter.String(),
|
|
"This command takes one argument: <service_name>")
|
|
ui.ErrorWriter.Reset()
|
|
|
|
// Create a test job with a Nomad service.
|
|
testJob := testJob("service-discovery-nomad-info")
|
|
testJob.TaskGroups[0].Services = []*api.Service{
|
|
{Name: "service-discovery-nomad-info", Provider: "nomad", PortLabel: "9999", Tags: []string{"foo", "bar"}}}
|
|
|
|
// Register that job.
|
|
regResp, _, err := client.Jobs().Register(testJob, nil)
|
|
require.NoError(t, err)
|
|
registerCode := waitForSuccess(ui, client, fullId, t, regResp.EvalID)
|
|
require.Equal(t, 0, registerCode)
|
|
|
|
// Reset the output writer, otherwise we will have additional information here.
|
|
ui.OutputWriter.Reset()
|
|
|
|
// Job register doesn't assure the service registration has completed. It
|
|
// therefore needs this wrapper to account for eventual service
|
|
// registration. One this has completed, we can perform lookups without
|
|
// similar wraps.
|
|
require.Eventually(t, func() bool {
|
|
|
|
defer ui.OutputWriter.Reset()
|
|
|
|
// Perform a standard lookup.
|
|
if code := cmd.Run([]string{"-address=" + url, "service-discovery-nomad-info"}); code != 0 {
|
|
return false
|
|
}
|
|
|
|
// Test each header and data entry.
|
|
s := ui.OutputWriter.String()
|
|
if !assert.Contains(t, s, "Job ID") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, "Address") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, "Node ID") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, "Alloc ID") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, "service-discovery-nomad-info") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, ":9999") {
|
|
return false
|
|
}
|
|
if !assert.Contains(t, s, "[foo,bar]") {
|
|
return false
|
|
}
|
|
return true
|
|
}, 5*time.Second, 100*time.Millisecond)
|
|
|
|
// Perform a verbose lookup.
|
|
code := cmd.Run([]string{"-address=" + url, "-verbose", "service-discovery-nomad-info"})
|
|
require.Equal(t, 0, code)
|
|
|
|
// Test KV entries.
|
|
s := ui.OutputWriter.String()
|
|
require.Contains(t, s, "Service Name = service-discovery-nomad-info")
|
|
require.Contains(t, s, "Namespace = default")
|
|
require.Contains(t, s, "Job ID = service-discovery-nomad-info")
|
|
require.Contains(t, s, "Datacenter = dc1")
|
|
require.Contains(t, s, "Address = :9999")
|
|
require.Contains(t, s, "Tags = [foo,bar]")
|
|
|
|
ui.OutputWriter.Reset()
|
|
ui.ErrorWriter.Reset()
|
|
}
|
|
|
|
func Test_argsWithNewPageToken(t *testing.T) {
|
|
ci.Parallel(t)
|
|
|
|
testCases := []struct {
|
|
inputOsArgs []string
|
|
inputNextToken string
|
|
expectedOutput string
|
|
name string
|
|
}{
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-page-token=abcdef", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -page-token=ghijkl example-cache",
|
|
name: "page token with equals sign",
|
|
},
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-page-token", "abcdef", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -page-token ghijkl example-cache",
|
|
name: "page token with whitespace gap",
|
|
},
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-per-page", "3", "-page-token", "abcdef", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -per-page 3 -page-token ghijkl example-cache",
|
|
name: "per page and page token",
|
|
},
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-page-token", "abcdef", "-per-page", "3", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -page-token ghijkl -per-page 3 example-cache",
|
|
name: "page token and per page",
|
|
},
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-page-token", "abcdef", "-per-page=3", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -page-token ghijkl -per-page=3 example-cache",
|
|
name: "page token and per page with equal",
|
|
},
|
|
{
|
|
inputOsArgs: []string{"nomad", "service", "info", "-verbose", "-page-token", "abcdef", "-per-page=3", "example-cache"},
|
|
inputNextToken: "ghijkl",
|
|
expectedOutput: "nomad service info -verbose -page-token ghijkl -per-page=3 example-cache",
|
|
name: "page token per page with verbose",
|
|
},
|
|
}
|
|
|
|
for _, tc := range testCases {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
actualOutput := argsWithNewPageToken(tc.inputOsArgs, tc.inputNextToken)
|
|
require.Equal(t, tc.expectedOutput, actualOutput)
|
|
})
|
|
}
|
|
}
|