Skip to content

Commit

Permalink
Add username password back from es to kibana config (#10553)
Browse files Browse the repository at this point in the history
  • Loading branch information
kaiyan-sheng authored Feb 5, 2019
1 parent 2091dc4 commit f5ffaa8
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
6 changes: 5 additions & 1 deletion libbeat/cmd/instance/beat.go
Original file line number Diff line number Diff line change
Expand Up @@ -681,6 +681,10 @@ func (b *Beat) loadDashboards(ctx context.Context, force bool) error {
}

if b.Config.Dashboards.Enabled() {
var esConfig *common.Config
if b.Config.Output.Name() == "elasticsearch" {
esConfig = b.Config.Output.Config()
}

var withMigration bool
if b.RawConfig.HasField("migration") {
Expand Down Expand Up @@ -716,7 +720,7 @@ func (b *Beat) loadDashboards(ctx context.Context, force bool) error {
}

err = dashboards.ImportDashboards(ctx, b.Info, paths.Resolve(paths.Home, ""),
kibanaConfig, b.Config.Dashboards, nil, pattern)
kibanaConfig, esConfig, b.Config.Dashboards, nil, pattern)
if err != nil {
return errw.Wrap(err, "Error importing Kibana dashboards")
}
Expand Down
14 changes: 13 additions & 1 deletion libbeat/dashboards/dashboards.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import (
func ImportDashboards(
ctx context.Context,
beatInfo beat.Info, homePath string,
kibanaConfig, dashboardsConfig *common.Config,
kibanaConfig, esConfig, dashboardsConfig *common.Config,
msgOutputter MessageOutputter,
pattern common.MapStr,
) error {
Expand All @@ -50,6 +50,18 @@ func ImportDashboards(
return err
}

if esConfig.Enabled() {
username, _ := esConfig.String("username", -1)
password, _ := esConfig.String("password", -1)

if !kibanaConfig.HasField("username") && username != "" {
kibanaConfig.SetString("username", -1, username)
}
if !kibanaConfig.HasField("password") && password != "" {
kibanaConfig.SetString("password", -1, password)
}
}

if !kibanaConfig.Enabled() {
return errors.New("kibana configuration missing for loading dashboards.")
}
Expand Down

0 comments on commit f5ffaa8

Please sign in to comment.