test(outputs.instrumental): Allow setting custom port (#13576)
This commit is contained in:
parent
908c6a551f
commit
00b0ae68e0
|
|
@ -28,6 +28,7 @@ var (
|
||||||
|
|
||||||
type Instrumental struct {
|
type Instrumental struct {
|
||||||
Host string `toml:"host"`
|
Host string `toml:"host"`
|
||||||
|
Port int `toml:"port"`
|
||||||
APIToken config.Secret `toml:"api_token"`
|
APIToken config.Secret `toml:"api_token"`
|
||||||
Prefix string `toml:"prefix"`
|
Prefix string `toml:"prefix"`
|
||||||
DataFormat string `toml:"data_format"`
|
DataFormat string `toml:"data_format"`
|
||||||
|
|
@ -44,6 +45,7 @@ type Instrumental struct {
|
||||||
|
|
||||||
const (
|
const (
|
||||||
DefaultHost = "collector.instrumentalapp.com"
|
DefaultHost = "collector.instrumentalapp.com"
|
||||||
|
DefaultPort = 8000
|
||||||
HelloMessage = "hello version go/telegraf/1.1\n"
|
HelloMessage = "hello version go/telegraf/1.1\n"
|
||||||
AuthFormat = "authenticate %s\n"
|
AuthFormat = "authenticate %s\n"
|
||||||
HandshakeFormat = HelloMessage + AuthFormat
|
HandshakeFormat = HelloMessage + AuthFormat
|
||||||
|
|
@ -70,7 +72,8 @@ func (i *Instrumental) Init() error {
|
||||||
}
|
}
|
||||||
|
|
||||||
func (i *Instrumental) Connect() error {
|
func (i *Instrumental) Connect() error {
|
||||||
connection, err := net.DialTimeout("tcp", i.Host+":8000", time.Duration(i.Timeout))
|
addr := fmt.Sprintf("%s:%d", i.Host, i.Port)
|
||||||
|
connection, err := net.DialTimeout("tcp", addr, time.Duration(i.Timeout))
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
i.conn = nil
|
i.conn = nil
|
||||||
|
|
@ -203,6 +206,7 @@ func init() {
|
||||||
outputs.Add("instrumental", func() telegraf.Output {
|
outputs.Add("instrumental", func() telegraf.Output {
|
||||||
return &Instrumental{
|
return &Instrumental{
|
||||||
Host: DefaultHost,
|
Host: DefaultHost,
|
||||||
|
Port: DefaultPort,
|
||||||
Template: graphite.DefaultTemplate,
|
Template: graphite.DefaultTemplate,
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
|
|
|
||||||
|
|
@ -18,10 +18,11 @@ import (
|
||||||
func TestWrite(t *testing.T) {
|
func TestWrite(t *testing.T) {
|
||||||
var wg sync.WaitGroup
|
var wg sync.WaitGroup
|
||||||
wg.Add(1)
|
wg.Add(1)
|
||||||
TCPServer(t, &wg)
|
port := TCPServer(t, &wg)
|
||||||
|
|
||||||
i := Instrumental{
|
i := Instrumental{
|
||||||
Host: "127.0.0.1",
|
Host: "127.0.0.1",
|
||||||
|
Port: port,
|
||||||
APIToken: config.NewSecret([]byte("abc123token")),
|
APIToken: config.NewSecret([]byte("abc123token")),
|
||||||
Prefix: "my.prefix",
|
Prefix: "my.prefix",
|
||||||
}
|
}
|
||||||
|
|
@ -80,8 +81,8 @@ func TestWrite(t *testing.T) {
|
||||||
wg.Wait()
|
wg.Wait()
|
||||||
}
|
}
|
||||||
|
|
||||||
func TCPServer(t *testing.T, wg *sync.WaitGroup) {
|
func TCPServer(t *testing.T, wg *sync.WaitGroup) int {
|
||||||
tcpServer, err := net.Listen("tcp", "127.0.0.1:8000")
|
tcpServer, err := net.Listen("tcp", "127.0.0.1:0")
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
|
|
||||||
go func() {
|
go func() {
|
||||||
|
|
@ -132,4 +133,6 @@ func TCPServer(t *testing.T, wg *sync.WaitGroup) {
|
||||||
err = conn.Close()
|
err = conn.Close()
|
||||||
require.NoError(t, err)
|
require.NoError(t, err)
|
||||||
}()
|
}()
|
||||||
|
|
||||||
|
return tcpServer.Addr().(*net.TCPAddr).Port
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue