mirror of
https://github.com/cozystack/cozystack.git
synced 2026-11-12 23:54:01 +09:00
ZverGuy
cd04c3e6c4
vlselect widens the window to step boundaries and returns a zero-filled bucket for each one, so a start off a boundary yields one bucket more than the span divided by the step. Sizing the step from the unaligned span let hits and stats_query_range return MaxBuckets+1 buckets. The minimum step now spans MaxBuckets-1 steps, and the smallest accepted --logs-max-buckets rises to 4 so that this step still fits the third-of-range cap that keeps alignment inside the window. Assisted-by: LLM Signed-off-by: ZverGuy <maximbel2003@gmail.com>
80 lines
2.5 KiB
Go
80 lines
2.5 KiB
Go
/*
|
|
Copyright 2026 The Cozystack Authors.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
package main
|
|
|
|
import (
|
|
"testing"
|
|
"time"
|
|
)
|
|
|
|
func TestParseFlagsDefaults(t *testing.T) {
|
|
o, err := parseFlags(nil)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if o.listenAddress != ":8080" || o.probeAddress != ":8081" || o.clusterDomain != "cozy.local" || o.rootTenant != "tenant-root" {
|
|
t.Errorf("contract defaults changed: %+v", o)
|
|
}
|
|
if o.authCacheTTL != 10*time.Second || o.logs.MaxTailResume != 15*time.Minute || o.logs.MaxRange != 7*24*time.Hour {
|
|
t.Errorf("limit defaults changed: %+v", o)
|
|
}
|
|
}
|
|
|
|
func TestParseFlagsOverrides(t *testing.T) {
|
|
o, err := parseFlags([]string{"--cluster-domain=example.org", "--logs-max-range=24h", "--max-tails=3"})
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if o.clusterDomain != "example.org" || o.logs.MaxRange != 24*time.Hour || o.limiter.MaxTails != 3 {
|
|
t.Errorf("overrides not applied: %+v", o)
|
|
}
|
|
// The smallest bucket count and range the step cap can satisfy.
|
|
if _, err := parseFlags([]string{"--logs-max-buckets=4", "--logs-max-range=3s", "--logs-default-range=1s"}); err != nil {
|
|
t.Errorf("smallest satisfiable step cap refused: %v", err)
|
|
}
|
|
}
|
|
|
|
func TestParseFlagsInvalid(t *testing.T) {
|
|
for _, args := range [][]string{
|
|
{"--root-tenant="},
|
|
{"--cluster-domain="},
|
|
{"--logs-default-range=48h", "--logs-max-range=24h"},
|
|
{"--logs-max-buckets=3"},
|
|
{"--logs-max-range=2s", "--logs-default-range=1s"},
|
|
{"--shutdown-timeout=-1s"},
|
|
{"--logs-default-limit=0"},
|
|
{"--logs-default-limit=100", "--logs-max-limit=10"},
|
|
{"--logs-max-tail-resume=-1m"},
|
|
{"--request-timeout=-1s"},
|
|
{"--tail-idle-timeout=-1s"},
|
|
{"--auth-cache-ttl=-1s"},
|
|
{"--rate-limit-per-user=-1"},
|
|
{"--rate-limit-per-user=NaN"},
|
|
{"--rate-limit-per-user=+Inf"},
|
|
{"--rate-limit-burst=-1"},
|
|
{"--max-concurrent-per-user=-1"},
|
|
{"--max-tails-per-user=-1"},
|
|
{"--max-tails=-1"},
|
|
{"--unknown"},
|
|
{"unexpected", "--root-tenant=tenant-x"},
|
|
} {
|
|
if _, err := parseFlags(args); err == nil {
|
|
t.Errorf("parseFlags(%v) accepted", args)
|
|
}
|
|
}
|
|
}
|