mirror of
https://github.com/hanzoai/deploy.git
synced 2026-10-07 08:44:50 +00:00
* 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]>
196 lines
6.2 KiB
Go
196 lines
6.2 KiB
Go
package path
|
|
|
|
import (
|
|
"errors"
|
|
"fmt"
|
|
"os"
|
|
"path/filepath"
|
|
"strings"
|
|
|
|
"github.com/hanzoai/deploy/pkg/apis/application/v1alpha1"
|
|
"github.com/hanzoai/deploy/util/io/files"
|
|
"github.com/hanzoai/deploy/util/security"
|
|
)
|
|
|
|
const ErrMessageAppPathDoesNotExist = "app path does not exist"
|
|
|
|
// ErrAppPathDoesNotExist is returned when the requested app path does not exist
|
|
// in the repository. It is a user input error rather than a server fault.
|
|
var ErrAppPathDoesNotExist = errors.New(ErrMessageAppPathDoesNotExist)
|
|
|
|
func Path(root, path string) (string, error) {
|
|
if filepath.IsAbs(path) {
|
|
return "", fmt.Errorf("%s: app path is absolute", path)
|
|
}
|
|
appPath := filepath.Join(root, path)
|
|
if !strings.HasPrefix(appPath, filepath.Clean(root)) {
|
|
return "", fmt.Errorf("%s: app path outside root", path)
|
|
}
|
|
info, err := os.Stat(appPath)
|
|
if os.IsNotExist(err) {
|
|
return "", fmt.Errorf("%s: %w", path, ErrAppPathDoesNotExist)
|
|
}
|
|
if err != nil {
|
|
return "", err
|
|
}
|
|
if !info.IsDir() {
|
|
return "", fmt.Errorf("%s: app path is not a directory", path)
|
|
}
|
|
return appPath, nil
|
|
}
|
|
|
|
type OutOfBoundsSymlinkError struct {
|
|
File string
|
|
Err error
|
|
}
|
|
|
|
func (e *OutOfBoundsSymlinkError) Error() string {
|
|
return "out of bounds symlink found"
|
|
}
|
|
|
|
// CheckOutOfBoundsSymlinks determines if basePath contains any symlinks that
|
|
// are absolute or point to a path outside of the basePath. If found, an
|
|
// OutOfBoundsSymlinkError is returned.
|
|
func CheckOutOfBoundsSymlinks(basePath string, skipPaths ...string) error {
|
|
absBasePath, err := filepath.Abs(basePath)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to get absolute path: %w", err)
|
|
}
|
|
skipPathsSet := map[string]bool{}
|
|
for _, p := range skipPaths {
|
|
skipPathsSet[filepath.Join(absBasePath, p)] = true
|
|
}
|
|
|
|
return filepath.Walk(absBasePath, func(path string, info os.FileInfo, err error) error {
|
|
if err != nil {
|
|
// Ignore "no such file or directory" errors that can happen with
|
|
// temporary files such as .git/*.lock
|
|
if errors.Is(err, os.ErrNotExist) {
|
|
return nil
|
|
}
|
|
return fmt.Errorf("failed to walk for symlinks in %s: %w", absBasePath, err)
|
|
}
|
|
if skipPathsSet[path] {
|
|
return filepath.SkipDir
|
|
}
|
|
if files.IsSymlink(info) {
|
|
// We don't use filepath.EvalSymlinks because it fails without returning a path
|
|
// if the target doesn't exist.
|
|
linkTarget, err := os.Readlink(path)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to read link %s: %w", path, err)
|
|
}
|
|
// get the path of the symlink relative to basePath, used for error description
|
|
linkRelPath, err := filepath.Rel(absBasePath, path)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to get relative path for symlink: %w", err)
|
|
}
|
|
// deny all absolute symlinks
|
|
if filepath.IsAbs(linkTarget) {
|
|
return &OutOfBoundsSymlinkError{File: linkRelPath}
|
|
}
|
|
// get the parent directory of the symlink
|
|
currentDir := filepath.Dir(path)
|
|
|
|
// walk each part of the symlink target to make sure it never leaves basePath
|
|
parts := strings.SplitSeq(linkTarget, string(os.PathSeparator))
|
|
for part := range parts {
|
|
newDir := filepath.Join(currentDir, part)
|
|
rel, err := filepath.Rel(absBasePath, newDir)
|
|
if err != nil {
|
|
return fmt.Errorf("failed to get relative path for symlink target: %w", err)
|
|
}
|
|
if rel == ".." || strings.HasPrefix(rel, ".."+string(os.PathSeparator)) {
|
|
// return an error so we don't keep traversing the tree
|
|
return &OutOfBoundsSymlinkError{File: linkRelPath}
|
|
}
|
|
currentDir = newDir
|
|
}
|
|
}
|
|
return nil
|
|
})
|
|
}
|
|
|
|
// GetSourceRefreshPaths returns the list of paths that should trigger a refresh for an application.
|
|
// The source parameter influences the returned refresh paths:
|
|
// - if source hydrator configured AND source is syncSource: use sync source path (ignores annotation)
|
|
// - if source hydrator configured AND source is drySource WITH annotation: use annotation paths with drySource base
|
|
// - if source hydrator not configured: use annotation paths with source base, or empty if no annotation
|
|
func GetSourceRefreshPaths(app *v1alpha1.Application, source v1alpha1.ApplicationSource) []string {
|
|
annotationPaths, hasAnnotation := app.Annotations[v1alpha1.AnnotationKeyManifestGeneratePaths]
|
|
|
|
if app.Spec.SourceHydrator != nil {
|
|
syncSource := app.Spec.SourceHydrator.GetSyncSource()
|
|
|
|
// if source is syncSource use the source path
|
|
if (source).Equals(&syncSource) {
|
|
return []string{source.Path}
|
|
}
|
|
}
|
|
|
|
var paths []string
|
|
if hasAnnotation && annotationPaths != "" {
|
|
for item := range strings.SplitSeq(annotationPaths, ";") {
|
|
// Trim whitespace because annotation values may contain spaces around
|
|
// separators (e.g. ".; /path"). Without trimming, paths like " /path"
|
|
// are not treated as absolute and empty/space-only entries may result
|
|
// in duplicate or incorrect refresh paths.
|
|
item = strings.TrimSpace(item)
|
|
// skip empty paths
|
|
if item == "" {
|
|
continue
|
|
}
|
|
// if absolute path, add as is
|
|
if filepath.IsAbs(item) {
|
|
paths = append(paths, item[1:])
|
|
continue
|
|
}
|
|
|
|
// add the path relative to the source path
|
|
paths = append(paths, filepath.Clean(filepath.Join(source.Path, item)))
|
|
}
|
|
}
|
|
|
|
return paths
|
|
}
|
|
|
|
// AppFilesHaveChanged returns true if any of the changed files are under the given refresh paths
|
|
// If refreshPaths or changedFiles are empty, it will always return true
|
|
func AppFilesHaveChanged(refreshPaths []string, changedFiles []string) bool {
|
|
// an empty slice of changed files means that the payload didn't include a list
|
|
// of changed files and we have to assume that a refresh is required
|
|
if len(changedFiles) == 0 {
|
|
return true
|
|
}
|
|
|
|
if len(refreshPaths) == 0 {
|
|
// Apps without a given refreshed paths always be refreshed, regardless of changed files
|
|
// this is the "default" behavior
|
|
return true
|
|
}
|
|
|
|
// At last one changed file must be under refresh path
|
|
for _, f := range changedFiles {
|
|
f = ensureAbsPath(f)
|
|
for _, item := range refreshPaths {
|
|
item = ensureAbsPath(item)
|
|
if f == item {
|
|
return true
|
|
} else if _, err := security.EnforceToCurrentRoot(item, f); err == nil {
|
|
return true
|
|
} else if matched, err := filepath.Match(item, f); err == nil && matched {
|
|
return true
|
|
}
|
|
}
|
|
}
|
|
|
|
return false
|
|
}
|
|
|
|
func ensureAbsPath(input string) string {
|
|
if !filepath.IsAbs(input) {
|
|
return string(filepath.Separator) + input
|
|
}
|
|
return input
|
|
}
|