ros-infrastructure / ros_buildfarm

ROS buildfarm based on Docker
Apache License 2.0
77 stars 95 forks source link

Add !relative_url YAML constructor #1057

Open cottsay opened 3 weeks ago

cottsay commented 3 weeks ago

The purpose of this constructor is to support relative URL resolution in arbitrary locations in the buildfarm's configuration YAML. Some fields in the config already implicitly do this (such as the buildfile references from the index), but this will add support for any field to contain a relative URL.

Staging deployment: https://build.test.ros2.org/job/ci__infra_ubuntu_noble_amd64/

Example use: https://github.com/ros2/ros_buildfarm_config/blob/200fcfd96aceaf2d0791682a6b16e80a59669e52/global/ci-infra.yaml#L43-L44

cottsay commented 2 weeks ago

Yeah, we should probably document this as well as !include somewhere.

I don't currently have a plan to transition the existing implicit URL resolutions to use this, but we could consider a follow-up change to deprecate them at some point.