On 10/03/2025 6:38 pm, Roger Pau Monné wrote: > On Mon, Mar 10, 2025 at 06:30:15PM +0000, Andrew Cooper wrote: >> On 10/03/2025 6:16 pm, Roger Pau Monne wrote: >>> Always store xen/.config as an artifact, renamed to xen-config to match >>> the naming used in the Gitlab CI tests. >>> >>> Reported-by: Andrew Cooper <andrew.coop...@citrix.com> >>> Signed-off-by: Roger Pau Monné <roger....@citrix.com> >> Looking at this, I suspect my failure was caused by trying to capture >> ".config" directly. >> >>> --- >>> .cirrus.yml | 12 ++++++++++++ >>> 1 file changed, 12 insertions(+) >>> >>> diff --git a/.cirrus.yml b/.cirrus.yml >>> index e2949d99d73a..1a39f5026f9a 100644 >>> --- a/.cirrus.yml >>> +++ b/.cirrus.yml >>> @@ -15,6 +15,14 @@ freebsd_template: &FREEBSD_ENV >>> APPEND_INCLUDES: /usr/local/include >>> CIRRUS_CLONE_DEPTH: 1 >>> >>> +freebsd_artifacts: &FREEBSD_ARTIFACTS >>> + always: >>> + rename_script: >>> + - cp xen/.config xen-config >>> + config_artifacts: >>> + path: xen-config >>> + type: text/plain >> Can't this be part of freebsd_template directly? >> >> Or is there an ordering problem with the regular build_script ? > Exactly, that was my first attempt (placing it in freebsd_template), > but then the collection would be done before the build, as > freebsd_template sets the env variables ahead of the build, see: > > https://cirrus-ci.com/task/5086615544004608 > > Thanks, Roger.
Ok. Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
