Uh oh!
There was an error while loading. Please reload this page.
Conversation
- Updated workflows to replace references from `tracebloc/` to `client/` for CI and release processes. - Introduced new `client/` directory structure, consolidating Helm chart files and templates. - Added migration documentation for transitioning from legacy charts to the unified `client/` chart. - Enhanced values schema and configuration management for improved deployment flexibility across platforms. - Updated README and NOTES.txt for clarity on new deployment instructions and component overview.
…loc, reflecting a reset in versioning as part of the unified deployment strategy.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| version: 1.0.1 | ||
| appVersion: "1.24.15" | ||
| version: 1.0.0 | ||
| appVersion: "1.0.0" |
There was a problem hiding this comment.
Docker registry secret template deleted without replacement
High Severity
The tracebloc/templates/docker-registry-secret.yaml template was deleted but no equivalent file exists in client/templates/. Multiple deployments (jobs-manager-deployment.yaml, mysql-deployment.yaml, resource-monitor-daemonset.yaml) reference tracebloc.registrySecretName for imagePullSecrets, and the _helpers.tpl defines both imagePullSecret and tracebloc.registrySecretName. Without this secret being created, pods will fail to pull images from the private Docker registry. The test file client/tests/secrets_test.yaml also references this missing template.
| version: 1.0.1 | ||
| appVersion: "1.24.15" | ||
| version: 1.0.0 | ||
| appVersion: "1.0.0" |
There was a problem hiding this comment.
Chart version and appVersion incorrectly downgraded
Medium Severity
The chart version was downgraded from 1.0.1 to 1.0.0 and appVersion from "1.24.15" to "1.0.0". Since this PR is a directory rename, these values likely need to be preserved. The appVersion change from 1.24.15 to 1.0.0 is especially suspicious — it's a dramatic downgrade that would cause helm package to produce a lower-versioned chart, and Helm may refuse to upgrade existing 1.0.1 releases.
…ployment - Introduced `client/values.yaml` to define environment variables, storage configurations, and secrets for the Helm chart. - Added `client/templates/docker-registry-secret.yaml` to manage Docker registry credentials as a Kubernetes secret. - Updated `.gitignore` to include new client-specific files while excluding tracebloc-related configurations.
Uh oh!
There was an error while loading. Please reload this page.
Note
Medium Risk
Medium risk because it changes the Helm chart release/CI pipelines and resets chart
version/appVersion, which can affect packaging, publishing, and upgrade expectations.Overview
Switches Helm chart automation to the
client/directory:helm-ci.yamlnow triggers onclient/**and runs lint/template/schema/unittest against./client, and the release workflow now lints/packages./client.Updates references from
tracebloc/→client/across install/migration/deprecation docs, and adjusts.gitignoreexceptions to keepclient/values*.yamlandclientsecret value files tracked. Also resetsclient/Chart.yamlversion/appVersionto1.0.0.Written by Cursor Bugbot for commit 785db02. This will update automatically on new commits. Configure here.