diff --git a/CHANGELOG.md b/CHANGELOG.md index 5806e91cc4bb..38c3a1082393 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -42,6 +42,8 @@ v0.41.0 (2024-05-31) ### Enhancements +- Add `--inspect` flag in ``grafana-agentctl test-logs` cmd. (@fgouteroux) + - Add support for importing folders as single module to `import.file`. (@wildum) - Add support for importing directories as single module to `import.git`. (@wildum) diff --git a/cmd/grafana-agent/entrypoint.go b/cmd/grafana-agent/entrypoint.go index a0a8341952f4..4598916501db 100644 --- a/cmd/grafana-agent/entrypoint.go +++ b/cmd/grafana-agent/entrypoint.go @@ -79,7 +79,7 @@ func NewEntrypoint(logger *server.Logger, cfg *config.Config, reloader Reloader) return nil, err } - ep.lokiLogs, err = logs.New(reg, cfg.Logs, logger, false) + ep.lokiLogs, err = logs.New(reg, cfg.Logs, logger, false, false) if err != nil { return nil, err } diff --git a/cmd/grafana-agentctl/main.go b/cmd/grafana-agentctl/main.go index 127f61d6df26..5367c0ceac0c 100644 --- a/cmd/grafana-agentctl/main.go +++ b/cmd/grafana-agentctl/main.go @@ -434,6 +434,7 @@ func filterAgentOwners(refs []meta_v1.OwnerReference) (filtered []meta_v1.OwnerR } func testLogs() *cobra.Command { + var inspect bool cmd := &cobra.Command{ Use: "test-logs [config file]", Short: "Collect logs but print entries instead of sending them to Loki.", @@ -452,7 +453,7 @@ func testLogs() *cobra.Command { } logger := log.NewLogfmtLogger(log.NewSyncWriter(os.Stderr)) - l, err := logs.New(prometheus.NewRegistry(), cfg.Logs, logger, true) + l, err := logs.New(prometheus.NewRegistry(), cfg.Logs, logger, true, inspect) if err != nil { fmt.Fprintf(os.Stderr, "failed to start log collection: %s\n", err) os.Exit(1) @@ -466,6 +467,7 @@ func testLogs() *cobra.Command { fmt.Fprintf(os.Stderr, "received shutdown %v signal, stopping...", sig) }, } + cmd.Flags().BoolVarP(&inspect, "inspect", "i", false, "Allows for detailed inspection of pipeline stages.") return cmd } diff --git a/internal/component/loki/source/file/legacy_file_test.go b/internal/component/loki/source/file/legacy_file_test.go index 583b911e022d..a3eadeadb025 100644 --- a/internal/component/loki/source/file/legacy_file_test.go +++ b/internal/component/loki/source/file/legacy_file_test.go @@ -84,7 +84,7 @@ configs: require.NoError(t, dec.Decode(&cfg)) require.NoError(t, cfg.ApplyDefaults()) logger := logkit.NewNopLogger() - l, err := logs.New(prometheus.NewRegistry(), &cfg, logger, false) + l, err := logs.New(prometheus.NewRegistry(), &cfg, logger, false, false) require.NoError(t, err) // // Write a log line and wait for it to come through. diff --git a/static/logs/http_test.go b/static/logs/http_test.go index e37110f2058e..9830eb25ddd3 100644 --- a/static/logs/http_test.go +++ b/static/logs/http_test.go @@ -27,7 +27,7 @@ configs: var cfg Config logger := util.TestLogger(t) - l, err := New(prometheus.NewRegistry(), &cfg, logger, false) + l, err := New(prometheus.NewRegistry(), &cfg, logger, false, false) require.NoError(t, err) defer l.Stop() @@ -72,7 +72,7 @@ configs: require.NoError(t, dec.Decode(&cfg)) logger := util.TestLogger(t) - l, err := New(prometheus.NewRegistry(), &cfg, logger, false) + l, err := New(prometheus.NewRegistry(), &cfg, logger, false, false) require.NoError(t, err) defer l.Stop() diff --git a/static/logs/logs.go b/static/logs/logs.go index 2d6c478fe510..667abf3f853a 100644 --- a/static/logs/logs.go +++ b/static/logs/logs.go @@ -14,6 +14,7 @@ import ( "github.com/grafana/agent/internal/agentseed" "github.com/grafana/agent/internal/useragent" "github.com/grafana/agent/internal/util" + "github.com/grafana/loki/clients/pkg/logentry/stages" "github.com/grafana/loki/clients/pkg/promtail" "github.com/grafana/loki/clients/pkg/promtail/api" "github.com/grafana/loki/clients/pkg/promtail/client" @@ -40,12 +41,15 @@ type Logs struct { } // New creates and starts Loki log collection. -func New(reg prometheus.Registerer, c *Config, l log.Logger, dryRun bool) (*Logs, error) { +func New(reg prometheus.Registerer, c *Config, l log.Logger, dryRun, inspect bool) (*Logs, error) { logs := &Logs{ instances: make(map[string]*Instance), reg: reg, l: log.With(l, "component", "logs"), } + + stages.Inspect = inspect + if err := logs.ApplyConfig(c, dryRun); err != nil { return nil, err } diff --git a/static/logs/logs_test.go b/static/logs/logs_test.go index 255c99b55f59..4134a2a02f9c 100644 --- a/static/logs/logs_test.go +++ b/static/logs/logs_test.go @@ -23,7 +23,7 @@ import ( ) func TestLogs_NilConfig(t *testing.T) { - l, err := New(prometheus.NewRegistry(), nil, util.TestLogger(t), false) + l, err := New(prometheus.NewRegistry(), nil, util.TestLogger(t), false, false) require.NoError(t, err) require.NoError(t, l.ApplyConfig(nil, false)) @@ -88,7 +88,7 @@ configs: require.NoError(t, dec.Decode(&cfg)) require.NoError(t, cfg.ApplyDefaults()) logger := log.NewSyncLogger(log.NewNopLogger()) - l, err := New(prometheus.NewRegistry(), &cfg, logger, false) + l, err := New(prometheus.NewRegistry(), &cfg, logger, false, false) require.NoError(t, err) defer l.Stop() @@ -195,7 +195,7 @@ configs: require.NoError(t, dec.Decode(&cfg)) require.NoError(t, cfg.ApplyDefaults()) logger := util.TestLogger(t) - l, err := New(prometheus.NewRegistry(), &cfg, logger, false) + l, err := New(prometheus.NewRegistry(), &cfg, logger, false, false) require.NoError(t, err) defer l.Stop()