mirror of
https://github.com/hanzoai/cd.git
synced 2026-10-30 14:56:45 +00:00
41536738c8
* feat(api): hard-fork CRD API group argoproj.io -> deploy.hanzo.ai
Hanzo CD hard-fork, phase 1.1. Single source of truth is
pkg/apis/application/register.go's Group constant (Application/AppProject/
ApplicationSet all derive their FullName from it); every other Go call-site
that hardcoded the literal group string is updated to match: the 3 generated
informer GVRs, the app-of-apps nested-Application detector in types.go, the
resource-tree Application-kind cache populator, the 3 NotFound error
constructors in server/application, and the applicationset-controller's
leader-election Lease name/ID (kept in lockstep across Go source + RBAC
resourceNames, or leader election would get RBAC-denied on startup).
CRD YAMLs (application/appproject/applicationset) and every RBAC apiGroups
rule that actually governs our 3 CRDs are repointed to deploy.hanzo.ai, across
both manifests/base/* (namespaced Roles) and manifests/cluster-rbac/* (cluster
scope). Two adjacent things that share the "argoproj.io" string but are NOT
ours are deliberately left untouched: gitops-engine's health-check dispatch
for Argo Workflows' `Workflow` kind, and argocd-server-clusterrole's second
apiGroups block granting read on Argo Workflows' `workflows` resource — both
belong to a separate, unrelated Argoproj-family project we are not forking.
Also untouched, per explicit scope: annotation/label keys (argocd.argoproj.io/*)
and finalizers (*.argoproj.io) that track already-applied live resources, and
argocd-cm.yaml's resource.customizations.ignoreResourceUpdates keys (a narrow
app-of-apps diffing nicety, not build- or RBAC-relevant).
CGO_ENABLED=0 go build ./... verified green.
* ci: wire canonical hanzo.yml + cicd.yml (Hanzo CD image build)
Phase 1.3. Follows the same one-way pattern every hanzo/lux/zoo repo uses:
hanzo.yml declares the image (root Dockerfile -> ghcr.io/hanzoai/deploy) and a
fast go-build sanity test (the multi-stage Dockerfile build is the slow, real
gate); cicd.yml is the ~10-line trigger shim importing hanzoai/ci's reusable
build workflow on the arc pool, secrets from KMS only.
No `deploy:` block yet, matching the hanzoai/cms build-only-first-cutover
pattern: this repo's own live hanzo-cd namespace runs the whole Hanzo fleet's
GitOps, so the prod image/CRD swap is a deliberate separate action (see the
[HELD] universe PR), never an automatic rollout on merge. Every PR/push still
builds and pushes an immutable ghcr.io/hanzoai/deploy:sha-<7>-amd64 image.
* feat(ui): rebrand Argo CD -> Hanzo CD (title, favicon, logo, sidebar)
Phase 1.2. User-facing "Argo CD"/"ArgoCD" text replaced with "Hanzo CD"
across the login page, help modal, settings screens, version-info panel,
sync-options warning, fullscreen-logs title, app.tsx page titles, and
index.html's <title>/noscript text (dropped the now-wrong upstream CLI
doc link rather than invent a hanzo.ai URL that doesn't exist).
Logo/favicon assets replaced with the canonical Hanzo mark (never
hand-drawn, copied from ~/work/hanzo/logo/dist): the two real usages of
Argo's logo collapse onto one Hanzo glyph (mono + white variants) at the
same 5 call sites (login icon + footer, login.scss $logoPath/
$logoNegativePath, sidebar text-logo), preserving the existing CSS
invert() filters so both the light login panel and dark sidebar still
render correctly. Old argologo.svg/argo_o.svg/argo.png deleted (no
longer referenced). favicon-{16x16,32x32}.png added at
ui/src/assets/favicon/ (index.html's links to that path were previously
dead - the directory didn't exist in this checkout). package.json name
argo-cd-ui -> hanzo-cd-ui.
Left alone, deliberately: the two Kubernetes resource-kind icon sets
under ui/src/assets/images/resources/ (unrelated to branding), and
upstream doc/project links (argoproj.io, argo-cd.readthedocs.io) since
we have no equivalent hanzo.ai docs to point to yet - except the login
footer logo's outbound link, which now points to hanzo.ai instead of
argoproj.io (a Hanzo-branded mark linking to the upstream project it
was forked from read as an obvious leftover).
* feat(module): rename Go module github.com/argoproj/argo-cd/v3 -> github.com/hanzoai/deploy
Phase 2. No version suffix on the new path (org policy: never publish our
own Go modules above v1.x.x / with a v2+ suffix) — this fork starts a fresh
v0/v1 module identity rather than carrying the v3 the upstream fork point
had reached.
Mechanical, symmetric rewrite: 704 .go files (import statements only; two
disjoint substitutions applied specific-first so the nested gitops-engine
module path never double-matches), plus go.mod (module line, the
gitops-engine require+replace pair, version reset to v0.0.0-... since a
suffix-less path can't carry a v3 semver), gitops-engine/go.mod, Makefile
(ldflags PACKAGE + 2 TEST_MODULE filters), .golangci.yaml, .goreleaser.yaml,
.mockery.yaml (+ gitops-engine's), Procfile, the codegen/proto generation
scripts (hack/generate-proto.sh, update-codegen.sh, update-openapi.sh), and
20 .proto files' go_package options. Also one OpenTelemetry tracer name
string in gitops-engine/pkg/sync/sync_context.go (a cosmestic label, not an
import, but worth keeping consistent).
CGO_ENABLED=0 go build ./... verified green for both the root module and
the nested gitops-engine module; go vet clean on both; go mod tidy run
(network available) to reconcile go.sum, dependency set unchanged.
Left alone, deliberately: doc-only mentions (docs/snyk/*.html scan reports,
gitops-engine/README.md), a comment-only reference in
.github/workflows/release.yaml, and one Renovate bot config line — none are
build- or codegen-load-bearing. Also left alone: comments citing upstream
GitHub issue URLs (github.com/argoproj/argo-cd/issues/NNNNN) and a handful
of test fixtures using "https://github.com/argoproj/argo-cd.git" as a
representative example repo URL — both are accurate historical/test-data
references, not stale self-references.
---------
Co-authored-by: Hanzo Dev <[email protected]>
261 lines
5.4 KiB
YAML
261 lines
5.4 KiB
YAML
formatters:
|
|
enable:
|
|
- gofumpt
|
|
- goimports
|
|
|
|
settings:
|
|
goimports:
|
|
local-prefixes:
|
|
- github.com/hanzoai/deploy
|
|
|
|
issues:
|
|
max-issues-per-linter: 0
|
|
|
|
max-same-issues: 0
|
|
|
|
linters:
|
|
enable:
|
|
- errorlint
|
|
- exptostd
|
|
- gocritic
|
|
- gomodguard_v2
|
|
- govet
|
|
- importas
|
|
- misspell
|
|
- modernize
|
|
- noctx
|
|
- perfsprint
|
|
- revive
|
|
- staticcheck
|
|
- testifylint
|
|
- thelper
|
|
- tparallel
|
|
- unparam
|
|
- usestdlibvars
|
|
- usetesting
|
|
- whitespace
|
|
|
|
exclusions:
|
|
rules:
|
|
- linters:
|
|
- unparam
|
|
path: (.+)_test\.go
|
|
|
|
presets:
|
|
- comments
|
|
- common-false-positives
|
|
- legacy
|
|
- std-error-handling
|
|
|
|
warn-unused: true
|
|
|
|
settings:
|
|
gocritic:
|
|
enable-all: true
|
|
# Most of these should probably be enabled one-by-one.
|
|
disabled-checks:
|
|
- appendAssign
|
|
- appendCombine # Leave disabled, multi-line assigns can be more readable.
|
|
- assignOp # Leave disabled, assign operations can be more confusing than helpful.
|
|
- commentedOutCode
|
|
- deferInLoop
|
|
- exitAfterDefer
|
|
- hugeParam
|
|
- importShadow
|
|
- paramTypeCombine # Leave disabled, there are too many failures to be worth fixing.
|
|
- rangeValCopy
|
|
- tooManyResultsChecker
|
|
- unnamedResult
|
|
- whyNoLint
|
|
|
|
gomodguard_v2:
|
|
blocked:
|
|
- module: github.com/golang-jwt/jwt/v4
|
|
recommendations:
|
|
- github.com/golang-jwt/jwt/v5
|
|
- module: github.com/imdario/mergo
|
|
recommendations:
|
|
- dario.cat/mergo
|
|
reason: '`github.com/imdario/mergo` has been renamed.'
|
|
- module: github.com/pkg/errors
|
|
recommendations:
|
|
- errors
|
|
|
|
govet:
|
|
disable:
|
|
- fieldalignment
|
|
- shadow
|
|
enable-all: true
|
|
|
|
importas:
|
|
alias:
|
|
- pkg: github.com/golang-jwt/jwt/v5
|
|
alias: jwtgo
|
|
|
|
- pkg: k8s.io/api/apps/v1
|
|
alias: appsv1
|
|
|
|
- pkg: k8s.io/api/core/v1
|
|
alias: corev1
|
|
|
|
- pkg: k8s.io/api/rbac/v1
|
|
alias: rbacv1
|
|
|
|
- pkg: k8s.io/apimachinery/pkg/api/errors
|
|
alias: apierrors
|
|
|
|
- pkg: k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1
|
|
alias: apiextensionsv1
|
|
|
|
- pkg: k8s.io/apimachinery/pkg/apis/meta/v1
|
|
alias: metav1
|
|
|
|
- pkg: k8s.io/client-go/informers/core/v1
|
|
alias: informersv1
|
|
|
|
- pkg: errors
|
|
alias: stderrors
|
|
|
|
- pkg: github.com/hanzoai/deploy/util/io
|
|
alias: utilio
|
|
|
|
modernize:
|
|
disable:
|
|
# Suggest replacing omitempty with omitzero for struct fields.
|
|
- omitzero
|
|
# Simplify code by using go1.26's new(expr). - generates lots of false positives.
|
|
- newexpr
|
|
|
|
nolintlint:
|
|
require-specific: true
|
|
|
|
perfsprint:
|
|
# Optimizes even if it requires an int or uint type cast.
|
|
int-conversion: true
|
|
# Optimizes into `err.Error()` even if it is only equivalent for non-nil errors.
|
|
err-error: true
|
|
# Optimizes `fmt.Errorf`.
|
|
errorf: true
|
|
# Optimizes `fmt.Sprintf` with only one argument.
|
|
sprintf1: true
|
|
# Optimizes into strings concatenation.
|
|
strconcat: true
|
|
|
|
revive:
|
|
enable-all-rules: false
|
|
enable-default-rules: true
|
|
max-open-files: 2048
|
|
# https://github.com/mgechev/revive/blob/master/RULES_DESCRIPTIONS.md
|
|
rules:
|
|
- name: blank-imports
|
|
disabled: true
|
|
|
|
- name: bool-literal-in-expr
|
|
|
|
- name: context-as-argument
|
|
arguments:
|
|
- allow-types-before: '*testing.T,testing.TB'
|
|
|
|
- name: context-keys-type
|
|
disabled: true
|
|
|
|
- name: dot-imports
|
|
disabled: true
|
|
|
|
- name: duplicated-imports
|
|
|
|
- name: early-return
|
|
arguments:
|
|
- preserve-scope
|
|
|
|
- name: empty-block
|
|
disabled: true
|
|
|
|
- name: error-return
|
|
|
|
- name: error-strings
|
|
disabled: true
|
|
|
|
- name: errorf
|
|
|
|
- name: exported
|
|
disabled: true
|
|
|
|
- name: identical-branches
|
|
|
|
- name: if-return
|
|
|
|
- name: increment-decrement
|
|
|
|
- name: indent-error-flow
|
|
arguments:
|
|
- preserve-scope
|
|
|
|
- name: modifies-parameter
|
|
|
|
- name: optimize-operands-order
|
|
|
|
- name: range
|
|
|
|
- name: receiver-naming
|
|
|
|
- name: redefines-builtin-id
|
|
disabled: true
|
|
|
|
- name: redundant-import-alias
|
|
|
|
- name: superfluous-else
|
|
arguments:
|
|
- preserve-scope
|
|
|
|
- name: time-equal
|
|
|
|
- name: time-naming
|
|
disabled: true
|
|
|
|
- name: unexported-return
|
|
disabled: true
|
|
|
|
- name: unnecessary-format
|
|
|
|
- name: unnecessary-stmt
|
|
|
|
- name: unreachable-code
|
|
|
|
- name: unused-parameter
|
|
|
|
- name: use-any
|
|
|
|
- name: useless-break
|
|
|
|
- name: var-declaration
|
|
|
|
- name: var-naming
|
|
arguments:
|
|
- - ID
|
|
- - VM
|
|
- - skip-initialism-name-checks: true
|
|
upper-case-const: true
|
|
|
|
staticcheck:
|
|
checks:
|
|
- all
|
|
- -SA5011
|
|
- -ST1003
|
|
- -ST1016
|
|
|
|
testifylint:
|
|
enable-all: true
|
|
|
|
disable:
|
|
- go-require
|
|
|
|
unused:
|
|
field-writes-are-uses: false
|
|
exported-fields-are-used: false
|
|
|
|
usetesting:
|
|
os-mkdir-temp: false
|
|
|
|
version: "2"
|