Skip to content

Commit

Permalink
Log a deprecation notice as warning for asptnet and dotnet monitors (s…
Browse files Browse the repository at this point in the history
  • Loading branch information
pjanotti committed Jan 31, 2024
1 parent 197ac4f commit 52e3afe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ var logger = logrus.WithField("monitorType", monitorType)
// Configure the monitor and kick off metric syncing
func (m *Monitor) Configure(conf *Config) error {
m.logger = logger.WithField("monitorID", conf.MonitorID)
m.logger.Warn("[NOTICE] The " + monitorType + " monitor is deprecated and will be removed in a future release. For more information visit https://docs.splunk.com/observability/en/gdi/monitors-languages/asp-dot-net.html")
perfcounterConf := &winperfcounters.Config{
CountersRefreshInterval: conf.CountersRefreshInterval,
PrintValid: conf.PrintValid,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ var logger = logrus.WithField("monitorType", monitorType)
// Configure the monitor and kick off metric syncing
func (m *Monitor) Configure(conf *Config) error {
m.logger = logger.WithField("monitorID", conf.MonitorID)
m.logger.Warn("[NOTICE] The " + monitorType + " monitor is deprecated and will be removed in a future release. For more information visit https://docs.splunk.com/observability/en/gdi/monitors-languages/microsoft-dotnet.html")
perfcounterConf := &winperfcounters.Config{
CountersRefreshInterval: conf.CountersRefreshInterval,
PrintValid: conf.PrintValid,
Expand Down

0 comments on commit 52e3afe

Please sign in to comment.