Hanzo DevandGitHub
fe4bfd3cf4
* debrand: signoz/SigNoz/SIGNOZ -> o11y/O11y/O11Y across the tree Rename all SigNoz *branding* to O11y in file contents, file/dir names, package names, env vars, config keys, comments, docs, SDK code, and the ClickHouse schema identifiers the querier reads. Collector dependency: - Repoint github.com/hanzoai/signoz-otel-collector -> github.com/hanzoai/otel-collector and bump v0.144.6 -> v0.144.7 (go.mod + go.sum reconciled via go mod tidy). Internal package paths also debranded upstream: signozschemamigrator -> o11yschemamigrator, signozlogspipelineprocessor -> o11ylogspipelineprocessor. - Fix: v0.144.7 dropped otelconst.DistributedFieldKeysTable from the collector's public constants; pin FieldKeysTable = "distributed_field_keys" locally in pkg/telemetrymetadata/tables.go. Package/dir renames (package decl + all importers): pkg/apiserver/signozapiserver -> o11yapiserver, plus signozalertmanager, signozauthzapi, signozglobal, signozquerier, signozruler, and pkg/signoz -> pkg/o11y. Schema (read plane): signoz_traces/metrics/logs/metadata/analytics/meter and signoz_index_v3/index_v2/error_index_v2/spans (+ distributed_*) -> o11y_*. Data-preserving cutover migration added: deploy/clickhouse/migrations/0001_rename_signoz_to_o11y.sql (metadata-only RENAME DATABASE/TABLE, no drop/recreate). Preserved (attribution / external / wire contracts): - LICENSE + NOTICE verbatim (incl. "software from SigNoz", "Copyright ... SigNoz Inc."). - Upstream github.com/SigNoz/* repo URLs + @SigNoz/* CODEOWNERS teams. - Billing resource-attribute regex "signoz.workspace.*" (external wire contract). go build ./pkg/... ./cmd/... = 0; gofmt clean; renamed+schema pkg tests pass. * o11y: bump collector to v0.144.8 (o11y_* physical schema writer) + lockstep deploy plan Collector v0.144.8's schema-migrator + exporters now CREATE/WRITE the same o11y_* physical databases/tables the querier reads and the RENAME migration (0001_rename_signoz_to_o11y.sql) targets. Cross-checked byte-identical: writer DB set == migration target DB set (6); writer prefixed-table set == migration target table set (8); o11y querier reads and cloud reads are subsets of that set. Zero table-name mismatches between writer, readers, and migration. Collector go.mod unchanged between v0.144.7 and v0.144.8 (identical go.mod hash) — pure source rename, no module-graph change. Adds deploy/clickhouse/DEPLOY_signoz_to_o11y_cutover.md: the lockstep order (collector v0.144.8 -> RENAME migration -> o11y+cloud readers) with a fresh/scratch-ClickHouse pre-flight (create via migrator, emit trace+metric, query back) and rollback. Ships together or telemetry blackholes. go build ./pkg/... ./cmd/community = 0.
79 lines
2.6 KiB
Go
79 lines
2.6 KiB
Go
package o11y_test
|
|
|
|
import (
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"testing"
|
|
|
|
"github.com/hanzoai/cloud"
|
|
"github.com/hanzoai/o11y"
|
|
"github.com/zap-proto/zip"
|
|
)
|
|
|
|
func TestMountWithoutHandlerReturns503(t *testing.T) {
|
|
app := zip.New(zip.Config{DisableStartupMessage: true})
|
|
if err := o11y.Mount(app, cloud.Deps{}); err != nil {
|
|
t.Fatalf("Mount: %v", err)
|
|
}
|
|
o11y.SetHandler(nil)
|
|
|
|
req := httptest.NewRequest(http.MethodGet, "/v1/o11y/anything", nil)
|
|
resp, err := app.Fiber().Test(req)
|
|
if err != nil {
|
|
t.Fatalf("Test: %v", err)
|
|
}
|
|
if resp.StatusCode != http.StatusServiceUnavailable {
|
|
t.Fatalf("status=%d want 503", resp.StatusCode)
|
|
}
|
|
}
|
|
|
|
// TestMountNormalizesExternalPath proves the one public contract /v1/o11y/<resource>
|
|
// (one /v1/, no nested version, no /api/) is normalized at the mount seam onto o11y's
|
|
// internal /api/ namespace — where a version-less alias (O11y, highest-version) or an
|
|
// llmobs route answers.
|
|
func TestMountNormalizesExternalPath(t *testing.T) {
|
|
app := zip.New(zip.Config{DisableStartupMessage: true})
|
|
if err := o11y.Mount(app, cloud.Deps{}); err != nil {
|
|
t.Fatalf("Mount: %v", err)
|
|
}
|
|
|
|
var sawPath string
|
|
o11y.SetHandler(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
sawPath = r.URL.Path
|
|
w.WriteHeader(http.StatusOK)
|
|
}))
|
|
defer o11y.SetHandler(nil)
|
|
|
|
cases := []struct {
|
|
name, external, internal string
|
|
}{
|
|
// Canonical version-less contract → internal /api/<resource> (no nested version).
|
|
{"versionless health", "/v1/o11y/health", "/api/health"},
|
|
{"versionless query_range", "/v1/o11y/query_range", "/api/query_range"},
|
|
// Hanzo llmobs resources (own their version-less names).
|
|
{"llmobs traces", "/v1/o11y/traces", "/api/traces"},
|
|
{"llmobs observations", "/v1/o11y/observations", "/api/observations"},
|
|
{"llmobs score by id", "/v1/o11y/score/abc123", "/api/score/abc123"},
|
|
// Explicit-version form still resolves to its exact version route.
|
|
{"explicit version", "/v1/o11y/v3/query_range", "/api/v3/query_range"},
|
|
// Leaked /api/ form kept working for the not-yet-migrated O11y SPA.
|
|
{"legacy api alias", "/v1/o11y/api/v1/health", "/api/v1/health"},
|
|
}
|
|
for _, tc := range cases {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
sawPath = ""
|
|
req := httptest.NewRequest(http.MethodGet, tc.external, nil)
|
|
resp, err := app.Fiber().Test(req)
|
|
if err != nil {
|
|
t.Fatalf("Test: %v", err)
|
|
}
|
|
if resp.StatusCode != http.StatusOK {
|
|
t.Fatalf("status=%d want 200", resp.StatusCode)
|
|
}
|
|
if sawPath != tc.internal {
|
|
t.Fatalf("external %s → internal %q, want %q", tc.external, sawPath, tc.internal)
|
|
}
|
|
})
|
|
}
|
|
}
|