fix: bygg naisdevice når agentkoden endrer seg - #565
Conversation
Path-filteret sa cmd/naisdevice-*. I GitHub sine path-filtre matcher ikke * skilletegnet /, så mønsteret traff bare en fil som lå rett under cmd/. Koden ligger i katalogene cmd/naisdevice-agent, -helper og -systray, så en endring der utløste aldri workflowen på push til main. PR-er ble likevel grønne, fordi pull_request-triggeren ikke har path-filter. Det er derfor dette ikke har vært synlig: bygget kjører i PR-en og uteblir i releasen. #564 endret bare cmd/naisdevice-agent og ble stående ubygget på main. De andre oppføringene i lista er allerede skrevet med /**.
📝 Changelog previewBelow is a preview of the Changelog that will be added to the next release. Only commit messages that follow the Conventional Commits specification will be included in the Changelog. v1.18.0 - 2026-09-15Full Changelog: v1.17.3...v1.18.0 🚀 Features
🐛 Bug Fixes |
Lista nevner build.yaml og templates, men ikke naisdevice.yaml. En endring i release-workflowen utløste derfor ikke release-workflowen, og en rettelse i path-filteret kunne ikke bekrefte seg selv. Den praktiske følgen nå: uten denne linja bygges ikke #564, som ligger ubygget på main, før noen tilfeldigvis rører agentkoden igjen.
|
La til én linje til: Lista nevner Med linja på plass er det å slå sammen denne PR-en selve pushen som bygger Workflowen har ingen |
1 participant
cmd/naisdevice-*matches a file directly undercmd/, never the contents of a directory. GitHub path filters treat*as "any character except/". The agent, helper and systray code all live in directories (cmd/naisdevice-agent/,cmd/naisdevice-helper/,cmd/naisdevice-systray/), so a change confined to one of them never triggered this workflow on push tomain.The rest of the list is already written with
/**, so this looks like a slip rather than a decision.It has been invisible because the
pull_requesttrigger has no path filter. A PR touching only agent code builds and goes green, and then the push tomaindoes nothing. The build passes, the release never happens.#564 is a live example: it changed
cmd/naisdevice-agent/**andREADME.md, went green on the PR, merged asd9bfa45d, and started only the Controlplane workflow. It is sitting onmainunreleased.After this change the same push matches
cmd/naisdevice-*/**and builds.