David B. comments

Results 30 comments of


                                            David B.

runtime error: invalid memory address or nil pointer dereference

We also faced this issue a few days ago with 2.13.3. We could enable server side diff but the documentation states that [the feature is stil in beta](https://argo-cd.readthedocs.io/en/stable/user-guide/diff-strategies/#server-side-diff). Also, it...

[ENHANCE] Set log level in helm chart

Implemented in #687

feat(log-level): add option to set logLevel in chart

Tested with: ``` spec: containers: - args: - --log-level=error ``` Yet in the actual container logs I can see: ``` time="2024-06-13T12:21:17Z" level=info msg="Environment: Kubernetes" ``` Feedback welcome.

feat(log-level): add option to set logLevel in chart

cc @MuneebAijaz

feat(log-level): add option to set logLevel in chart

Yes that's what I was trying to check and figure out but I don't immediately see any issues with the flag in the codebase. I can take another look anyway.

feat(log-level): add option to set logLevel in chart

@martin-gutwin thanks for the insights, didn't think about that. So I can update this PR to resolve the conflicts then.

feat(log-level): add option to set logLevel in chart

Can this PR be re-reviewed and merged if it satisfy requirements?

feat(log-level): add option to set logLevel in chart

> hi @dabcoder we have to do some changes to chart workflows, till then this sadly cant be merged. I will merge this when workflows are fixed. Any updates on...

feat(log-level): add option to set logLevel in chart

cc @MuneebAijaz

Add Prometheus Metric for number of jobs waiting for self-hosted runners

I was looking into this as well, but isn't what the [github_workflow_jobs_queued_total](https://github.com/actions/actions-runner-controller/blob/a07dce28bbd819428c787594ee693596ac1e56f0/pkg/actionsmetrics/metrics.go#L103) is supposed to do?