Skip to content

chore(release): v0.1.20 - #536

Open
dray92 wants to merge 1 commit into
mainfrom
release/v0.1.20
Open

dray92 wants to merge 1 commit into
mainfrom
release/v0.1.20

Conversation

@dray92

@dray92 dray92 commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Release v0.1.20 of zxporter, published from the internal services monorepo.

The v0.1.20 tag has already been created and points at this branch's tip — ArgoCD and other tag consumers can use it immediately. This PR is only to advance public main to the released commit for browser convenience.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@gitar-bot

gitar-bot Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
CI failed: 1 lint failure in the helm-chart test module from exceeding the maximum line length limit.

Overview

Golangci-lint failed due to a line length violation in the newly added chart topology spread test file.

Failures

Golangci-Lint Line Length Violation (confidence: high)

  • Type: tooling
  • Affected jobs: 112543621659
  • Related to change: yes
  • Root cause: A line containing a JSON string in helm-chart/zxporter/chart_topology_spread_test.go is 162 characters long, exceeding the 120-character limit enforced by the lll linter.
  • Suggested fix: Wrap or split the long line so that all lines adhere to the 120-character limit.

Summary

  • Change-related failures: 1 failure due to a lint check violation in the new test file.
  • Infrastructure/flaky failures: 0 failures.
  • Recommended action: Format the long line in helm-chart/zxporter/chart_topology_spread_test.go to satisfy the linter rules.
Code Review ✅ Approved

🔴 High risk · New RBAC grants broaden cluster-wide resource access and scheduler metrics access.

Release v0.1.20 advance to main with the tag already created and deployed. No issues found.

Auto-approved: No blocking issues found.
Please see Auto-approve Docs for details on setting custom approval criteria.

Review coverage

🤖 Auto-approval Approved

📋 Rules No rules evaluated

🧪 Functional validation Not enabled · Set up

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Counting what did not apply, without listing it.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@gitar-bot gitar-bot Bot added the gitar-approved Added by Gitar label Oct 6, 2026

@gitar-bot gitar-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gitar has auto-approved this PR (configure)

if err == nil && pool.AppendCertsFromPEM(caData) {
tlsConfig.RootCAs = pool
} else {
tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment.
tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment.
}
} else {
tlsConfig.InsecureSkipVerify = true //nolint:gosec // see schedulerMetricsCAFileEnv doc comment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitar-approved Added by Gitar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants