From e27b755598e72bc3c3ecd83f434245234a76ac46 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Alvarez=20Pi=C3=B1eiro?= <95703246+emilioalvap@users.noreply.github.com> Date: Fri, 12 May 2023 11:15:20 +0200 Subject: [PATCH] [Heartbeat] remove Synthetics beta label (#35424) * HB remove beta label * Add changelog --- CHANGELOG.next.asciidoc | 1 + heartbeat/docs/monitors/monitor-browser.asciidoc | 2 +- x-pack/heartbeat/monitors/browser/browser.go | 8 -------- 3 files changed, 2 insertions(+), 9 deletions(-) diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc index 9473211369db..7fcbbd647dee 100644 --- a/CHANGELOG.next.asciidoc +++ b/CHANGELOG.next.asciidoc @@ -284,6 +284,7 @@ automatic splitting at root level, if root level element is an array. {pull}3415 *Heartbeat* - Users can now configure max scheduler job limits per monitor type via env var. {pull}34307[34307] - Added status to monitor run log report. +- Removed beta label for browser monitors. {pull}35424[35424]. *Metricbeat* diff --git a/heartbeat/docs/monitors/monitor-browser.asciidoc b/heartbeat/docs/monitors/monitor-browser.asciidoc index aedb2c59a223..2da7059b088a 100644 --- a/heartbeat/docs/monitors/monitor-browser.asciidoc +++ b/heartbeat/docs/monitors/monitor-browser.asciidoc @@ -6,7 +6,7 @@ See the {observability-guide}/synthetics-quickstart.html[quick start guide]. NOTE: While it is possible to configure browser monitors via the heartbeat YAML config files, we highly recommend using project monitors instead! See the https://www.elastic.co/guide/en/observability/current/synthetic-run-tests.html#synthetic-monitor-choose-project[Elastic synthetics docs]. -beta[] The options described here configure {beatname_uc} to run the synthetic +The options described here configure {beatname_uc} to run the synthetic monitoring test projects via Synthetic Agent on the Chromium browser. Additional shared options are defined in <>. diff --git a/x-pack/heartbeat/monitors/browser/browser.go b/x-pack/heartbeat/monitors/browser/browser.go index 4444950a6201..3ef9f80873eb 100644 --- a/x-pack/heartbeat/monitors/browser/browser.go +++ b/x-pack/heartbeat/monitors/browser/browser.go @@ -9,11 +9,9 @@ package browser import ( "fmt" "os" - "sync" "syscall" "github.com/elastic/elastic-agent-libs/config" - "github.com/elastic/elastic-agent-libs/logp" "github.com/elastic/beats/v7/heartbeat/ecserr" "github.com/elastic/beats/v7/heartbeat/monitors/plugin" @@ -24,8 +22,6 @@ func init() { plugin.Register("browser", create, "synthetic", "synthetics/synthetic") } -var showExperimentalOnce = sync.Once{} - func create(name string, cfg *config.C) (p plugin.Plugin, err error) { // We don't want users running synthetics in environments that don't have the required GUI libraries etc, so we check // this flag. When we're ready to support the many possible configurations of systems outside the docker environment @@ -34,10 +30,6 @@ func create(name string, cfg *config.C) (p plugin.Plugin, err error) { return plugin.Plugin{}, ecserr.NewNotSyntheticsCapableError() } - showExperimentalOnce.Do(func() { - logp.Info("Synthetic browser monitor detected! Please note synthetic monitors are a beta feature!") - }) - // We do not use user.Current() which does not reflect setuid changes! if syscall.Geteuid() == 0 && security.NodeChildProcCred == nil { return plugin.Plugin{}, fmt.Errorf("script monitors cannot be run as root")