Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to override matcher.period #834

Open
roysjosh opened this issue Sep 5, 2023 · 0 comments
Open

Unable to override matcher.period #834

roysjosh opened this issue Sep 5, 2023 · 0 comments

Comments

@roysjosh
Copy link

roysjosh commented Sep 5, 2023

After installing quay 3.9.0 via the operator, it didn't take long to find my k8s cluster going crazy every 6h. I'm apparently hitting my full 40GbE line speed according to Grafana, and I have sustained 500MB/s disk RW. I run rook-ceph in-cluster, FYI. Also, it looks like each clair-app, of which there are two, does their own update leading to two spikes of activity, back to back.

I tried to turn down the update period from 6h to 10d (golang doesn't support the "d" suffix 😢) but found that the YAML isn't merged. When I set

---
matcher:
  period: 240h

I ended up with a broken config:

{"level":"error","component":"main","error":"service initialization failed: failed to initialize matcher: failed to create ConnPool: failed to connect to `host=/tmp user=nobody database=`: dial error (dial unix /tmp/.s.PGSQL.5432: connect: no such file or directory)","time":"2023-09-05T13:05:08Z","message":"fatal error"}
auth:
    psk:
        iss:
            - quay
            - clairctl
        key: REDACTED
http_listen_addr: :8080
indexer:
    connstring: host=key-clair-postgres port=5432 dbname=postgres user=postgres password=postgres sslmode=disable pool_max_conns=33
    layer_scan_concurrency: 5
    migrations: true
    scanlock_retry: 10
log_level: info
matcher:
    period: 240h
metrics:
    name: prometheus
notifier:
    connstring: host=key-clair-postgres port=5432 dbname=postgres user=postgres password=postgres sslmode=disable pool_max_conns=33
    delivery_interval: 1m0s
    migrations: true
    poll_interval: 5m0s
    webhook: REDACTED
[...]

Attempting to specify the generated fields led to a different error:

  Warning  ConfigInvalid              4m44s (x6 over 5m17s)     quayregistry-controller  clairpostgres component marked as managed, but `configBundleSecret` contains required fields

Specified config:

---
matcher:
  connstring: host=key-clair-postgres port=5432 dbname=postgres user=postgres password=postgres sslmode=disable pool_max_conns=33
  migrations: true
  period: 240h
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant