search

ros2 / ci

ROS 2 CI Infrastructure
http://ci.ros2.org/
Apache License 2.0
48 stars 30 forks source link

Make the RHEL packaging job respect EL_RELEASE. #697

Closed clalancette closed 1 year ago

clalancette commented 1 year ago

It wasn't doing that before, so was always defaulting to RHEL-8.

@Crola1702 FYI, this should in theory fix https://github.com/osrf/buildfarmer/issues/401

clalancette commented 1 year ago

CI:

clalancette commented 1 year ago

Note: This is a change to the packaging job template, so a straight CI wouldn't confirm whether this fixed the problem or not. I manually modified the ci_packaging_linux-rhel job to add in this piece of the template; assuming CI is successful there, I will revert that and redeploy this for real once we merge it in.

clalancette commented 1 year ago

And I've now deployed the change.