HarmenandClaude Opus 5
b2e9c8fc7a
De gebruiker koos ervoor het pakket meteen te maken en een installatie te proberen, met de image lokaal gebouwd en het recept in de repo. Dit is dat pakket. Er is nog niets gebouwd en niets geinstalleerd; "gebouwd" is hier nadrukkelijk niet "werkend". De zwaarste ontwerpvraag is met een precedent beslecht en niet met een gok. Trezor Suite is geen browser met een sessiecookie en kan dus niet achter de inlog van umbrelOS; het was onduidelijk of PROXY_AUTH_ADD "false" dan verantwoord is of een omweg. De eigen nostr-relay-app van Umbrel doet exact hetzelfde, om precies dezelfde reden, en heeft ook geen eigen ports:. De prijs staat in de compose en in het plan: wie die poort bereikt, bereikt de relay. Wat de schade beperkt is dat de relay elke eigenaar zonder limietenrij weigert. Daarom gaat de quota-manager mee, en dat is geen restje van Trezor's betaalde hosting: hij is wat die rijen aanmaakt. Relay en quota-manager komen uit dezelfde image met een ander command, want bovenstrooms is het een codebase met meerdere startscripts. Het command staat expliciet en leunt niet op de CMD van de Dockerfile, waar yarn start staat met bovenstrooms zelf een twijfel erbij. Het bouwrecept staat in tools/ en niet in de app-map. Dat is geen netheid: een Dockerfile staat niet in de update-whitelist, dus bouwen-in-de-app zou elke nieuwe versie een deinstallatie plus herinstallatie kosten. Onder tools/ en niet onder build/, want dat laatste staat in .gitignore als bouwselmap en het recept zou stilzwijgend buiten de repo zijn gebleven. Dat kwam pas bij git status aan het licht. De poort is 3851 en niet 4000. 4000 is de eigen poort van de relay maar ook een veelgebruikte poort, en een botsing op de host merk je pas als de app niet start. Die les komt van 50002 tegen Fulcrum. Nieuw testbestand test_appstore_vorm.py, en het gaat over de store en niet over een app: id gelijk aan mapnaam, store-voorvoegsel, veldvolgorde, app_proxy die naar een bestaande service wijst, en elke gemounte map die in de repo bestaat. Het vindt zijn apps zelf, dus een derde app valt er automatisch onder. Digests toetst het expres niet: geen van de twee apps haalt die regel vandaag en een suite die altijd rood staat wordt niet gelezen. Mutatie-getest met drie ingrepen: het app-id laten afwijken van de mapnaam, APP_HOST naar een niet-bestaande service laten wijzen, en de .gitkeep weghalen. Alle drie vielen om bij de juiste toets, en git diff was daarna leeg. Umbrelapp is gepromoveerd naar Actief als 008, tussen Proefopstelling en Appstore, en het masterplan is naar het archief. Wat er in de plannen als open blijft staan is niet klein: of Trezor Suite dit adres accepteert, of het databaseschema zichzelf aanmaakt, en hoe je een eigenaar registreert. Tests: 32 goed 0 fout, 39 goed 0 fout en 54 goed 0 fout, niets overgeslagen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
72 lines
3.8 KiB
Bash
72 lines
3.8 KiB
Bash
#!/bin/sh
|
|
# ═══════════════════════════════════════════════════════════════════════════════
|
|
# Bouwt de image voor de app whatsnext-evolu-relay.
|
|
#
|
|
# Waarom dit bestand bestaat: Trezor publiceert geen image. Hun werkproces duwt
|
|
# naar een eigen Amazon ECR en op Docker Hub staat niets, dus er is geen
|
|
# `docker pull` die iets oplevert. De broncode is wel publiek, met een Dockerfile
|
|
# erin. Dit script is de brug: het haalt de broncode op een vastgezette commit en
|
|
# bouwt hún Dockerfile.
|
|
#
|
|
# Waarom het NIET in de app-map staat: umbreld kopieert de hele app-map naar het
|
|
# apparaat, en bij een update wordt alleen een whitelist ververst waar een
|
|
# Dockerfile niet in zit. Bouwen tijdens het starten van de app zou de installatie
|
|
# bovendien minuten laten hangen. Het recept hoort dus in de repo en het resultaat
|
|
# in een register of in de lokale Docker-opslag; de app verwijst alleen naar de tag.
|
|
#
|
|
# Waarom we hun Dockerfile gebruiken en geen eigen: dan hoeven we hun bouwstappen
|
|
# niet na te bouwen en niet bij te houden. Wat wij toevoegen is uitsluitend de pin.
|
|
#
|
|
# Draaien: sh build.sh
|
|
# Vereist: docker en git op de machine waar je bouwt.
|
|
# ═══════════════════════════════════════════════════════════════════════════════
|
|
|
|
set -eu
|
|
|
|
# ── De pin ────────────────────────────────────────────────────────────────────
|
|
# Dit is het enige wat je aanpast als je een nieuwere versie wil. Verhoog daarna
|
|
# ook `version` in whatsnext-evolu-relay/umbrel-app.yml, anders rolt umbrelOS de
|
|
# wijziging niet uit.
|
|
UPSTREAM_REPO="https://github.com/trezor/trezor-suite-sync.git"
|
|
UPSTREAM_COMMIT="c03a2043acec48d4a1bbafca65e94da33dce1edd"
|
|
|
|
# De tag waar docker-compose.yml van de app naar verwijst. De korte commit zit
|
|
# erin, zodat je op het apparaat kunt zien wat er draait.
|
|
IMAGE="whatsnext/evolu-relay:c03a204"
|
|
|
|
# ── Bouwen ────────────────────────────────────────────────────────────────────
|
|
WORKDIR="$(mktemp -d)"
|
|
cleanup() { rm -rf "$WORKDIR"; }
|
|
trap cleanup EXIT INT TERM
|
|
|
|
echo "Broncode ophalen op ${UPSTREAM_COMMIT}"
|
|
|
|
# Niet `clone --branch`: dat pint op een naam die meebeweegt. Een fetch van één
|
|
# commit haalt precies deze toestand op en niets anders. GitHub staat het ophalen
|
|
# van een losse commit-sha toe; een Git-server die dat niet doet, geeft hier een
|
|
# foutmelding in plaats van stil een andere versie te bouwen.
|
|
git init --quiet "$WORKDIR"
|
|
git -C "$WORKDIR" remote add origin "$UPSTREAM_REPO"
|
|
git -C "$WORKDIR" fetch --quiet --depth 1 origin "$UPSTREAM_COMMIT"
|
|
git -C "$WORKDIR" checkout --quiet FETCH_HEAD
|
|
|
|
# Controle dat we hebben wat we dachten. Zonder deze regel bouwt het script bij
|
|
# een gewijzigde afspraak over sha's stil de verkeerde toestand.
|
|
GEVONDEN="$(git -C "$WORKDIR" rev-parse HEAD)"
|
|
if [ "$GEVONDEN" != "$UPSTREAM_COMMIT" ]; then
|
|
echo "FOUT: opgehaald ${GEVONDEN}, verwacht ${UPSTREAM_COMMIT}" >&2
|
|
exit 1
|
|
fi
|
|
|
|
echo "Bouwen als ${IMAGE}"
|
|
docker build --tag "$IMAGE" "$WORKDIR"
|
|
|
|
echo
|
|
echo "Klaar. De app verwijst naar deze tag:"
|
|
docker image inspect --format '{{.RepoTags}} {{.Id}}' "$IMAGE"
|
|
echo
|
|
echo "Wil je hem later uit een register halen in plaats van lokaal, dan is dit"
|
|
echo "de plek: tag hem naar dat register, push, en zet de digest uit"
|
|
echo "\`docker buildx imagetools inspect\` in docker-compose.yml. Zie"
|
|
echo "Docs/Referenties/Images-pinnen.md."
|