unkin-agent
8c796f4087
forgebot applied its schema by executing an inline DDL string on every boot, with no version tracking and no lock, so two API replicas starting together raced and the SQL had nowhere to grow. golib/pg already owns that mechanic for the estate; take it and keep owning the SQL. - Move the schema into migrations/0001_init.sql, embedded via migrations.FS. The DDL is verbatim. - The four legacy-status UPDATEs move into 0001 unchanged. Each reads only retired statuses (pending/failed/running/succeeded/cancelled) and writes only current ones, and no current status is a source, so replaying 0001 once against the live database is a no-op. A test pins that property. - Build the pool with pg.NewMigrated, LockName "forgebot-migrations": the advisory lock serializes replicas, schema_migrations records what ran, and a migration failure fails startup instead of half-migrating. database.New and apiserver.New take a context and logger for it. - Render the DSN with pg.DSN, which percent-escapes the credentials the fmt.Sprintf builder pasted in raw. LoadConfig still reads the environment itself: pg.DSNFromEnv has no defaults for user and database name, where forgebot defaults both to "forgebot", and would newly honour DATABASE_URL and PG*. The deployed DBHOST/DBPORT/DBUSER/DBPASS/DBNAME/DBSSL contract and its defaults are unchanged, and pinned by a test. - Plumb GOPRIVATE=git.unkin.net for the first cross-repo Go dependency: exported by the Makefile, set in both Dockerfiles and the woodpecker Go steps, documented in the README. - gofmt the four files that were already unformatted on main, so the pre-commit step can pass.
61 lines
1.9 KiB
Makefile
61 lines
1.9 KiB
Makefile
.PHONY: build test lint fmt generate docker-api docker-operator clean tidy
|
|
|
|
BINARY_API := bin/forgebot-api
|
|
BINARY_OP := bin/forgebot-operator
|
|
BINARY_TUI := bin/forgebot-tui
|
|
VERSION ?= $(shell git describe --tags --always --dirty 2>/dev/null || echo "0.0.0-dev")
|
|
|
|
# git.unkin.net modules (golib) are fetched straight from Gitea, never via the
|
|
# public proxy or sum db, which have no view of them. Exported here rather than
|
|
# written with `go env -w`, so a fresh checkout needs no machine-local setup.
|
|
export GOPRIVATE := git.unkin.net
|
|
|
|
build: tidy
|
|
go build -ldflags="-s -w" -o $(BINARY_API) ./cmd/api
|
|
go build -ldflags="-s -w" -o $(BINARY_OP) ./cmd/operator
|
|
go build -ldflags="-s -w" -o $(BINARY_TUI) ./cmd/tui
|
|
|
|
test:
|
|
go test -race -count=1 ./pkg/... ./internal/... ./api/...
|
|
|
|
lint:
|
|
go vet ./...
|
|
|
|
fmt:
|
|
gofmt -w .
|
|
|
|
generate:
|
|
controller-gen object paths="./api/..."
|
|
controller-gen crd paths="./api/..." output:crd:artifacts:config=config/crd/bases
|
|
controller-gen rbac:roleName=forgebot-operator paths="./internal/controller/..." output:rbac:dir=config/rbac
|
|
|
|
docker-api:
|
|
docker build -t forgebot-api:$(VERSION) -f Dockerfile.api .
|
|
|
|
docker-operator:
|
|
docker build -t forgebot-operator:$(VERSION) -f Dockerfile.operator .
|
|
|
|
clean:
|
|
rm -rf bin/
|
|
|
|
tidy:
|
|
go mod tidy
|
|
|
|
_LATEST := $(shell git tag --sort=-v:refname | grep -E '^v[0-9]+\.[0-9]+\.[0-9]+$$' | head -1)
|
|
_BASE := $(if $(_LATEST),$(_LATEST),v0.0.0)
|
|
_MAJ := $(shell echo $(_BASE) | sed 's/^v//' | cut -d. -f1)
|
|
_MIN := $(shell echo $(_BASE) | sed 's/^v//' | cut -d. -f2)
|
|
_PAT := $(shell echo $(_BASE) | sed 's/^v//' | cut -d. -f3)
|
|
|
|
patch:
|
|
@NEW=v$(_MAJ).$(_MIN).$(shell expr $(_PAT) + 1); \
|
|
git tag $$NEW && echo "Tagged $$NEW" && git push origin $$NEW
|
|
|
|
minor:
|
|
@NEW=v$(_MAJ).$(shell expr $(_MIN) + 1).0; \
|
|
git tag $$NEW && echo "Tagged $$NEW" && git push origin $$NEW
|
|
|
|
major:
|
|
@NEW=v$(shell expr $(_MAJ) + 1).0.0; \
|
|
git tag $$NEW && echo "Tagged $$NEW" && git push origin $$NEW
|