Skip to content

Commit

Permalink
[elastic-agent] ensure container is backwards compatible (#25092)
Browse files Browse the repository at this point in the history
only run setup and the elastic agent if configuration is provided
when in cloud mode

closes #25005

(cherry picked from commit cc7fcd1)
  • Loading branch information
simitt authored and mergify-bot committed Apr 14, 2021
1 parent 914c7ac commit 005d362
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions x-pack/elastic-agent/pkg/agent/cmd/container.go
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,9 @@ func containerCmd(streams *cli.IOStreams, cmd *cobra.Command) error {
return err
}

elasticCloud := envBool("ELASTIC_AGENT_CLOUD")
// if not in cloud mode, always run the agent
runAgent := !elasticCloud
// create access configuration from ENV and config files
cfg := defaultAccessConfig()
for _, f := range []string{"fleet-setup.yml", "credentials.yml"} {
Expand All @@ -155,13 +158,14 @@ func containerCmd(streams *cli.IOStreams, cmd *cobra.Command) error {
if err != nil {
return fmt.Errorf("unpacking config file(%s): %s", f, err)
}
// if in elastic cloud mode, only run the agent when configured
runAgent = true
}
}

// start apm-server legacy process when in cloud mode
var wg sync.WaitGroup
var apmProc *process.Info
_, elasticCloud := os.LookupEnv("ELASTIC_AGENT_CLOUD")
apmPath := os.Getenv("APM_SERVER_PATH")
if elasticCloud {
logInfo(streams, "Starting in elastic cloud mode")
Expand Down Expand Up @@ -202,13 +206,18 @@ func containerCmd(streams *cli.IOStreams, cmd *cobra.Command) error {
apmProc.Stop()
logInfo(streams, "Initiate shutdown legacy apm-server.")
}
wg.Wait()
}()
}
}

// run the main elastic-agent container command
return runContainerCmd(streams, cmd, cfg)
var err error
if runAgent {
// run the main elastic-agent container command
err = runContainerCmd(streams, cmd, cfg)
}
// wait until APM Server shut down
wg.Wait()
return err
}

func runContainerCmd(streams *cli.IOStreams, cmd *cobra.Command, cfg setupConfig) error {
Expand Down

0 comments on commit 005d362

Please sign in to comment.