enhancement(agentctl): add inspect flag in test-logs cmd

Signed-off-by: François Gouteroux <[email protected]>

Author: fgouteroux

CHANGELOG.md

@@ -34,6 +34,8 @@ Main (unreleased)
 
 ### 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)

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
 	}

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
 }

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.

internal/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()
 

internal/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
 	}

internal/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()