ros-infrastructure / buildfarm_deployment

Apache License 2.0
30 stars 39 forks source link

add sections for colcon package output #217

Closed dirk-thomas closed 5 years ago

dirk-thomas commented 5 years ago

Adds three sections for colcon specific output as it is already in use on ci.ros2.org:

nuclearsandwich commented 5 years ago

The optimal way to deploy this today would be to manually place the config file in the Jenkins directory (since re-running puppet on master nodes is not a good idea #160) and restart Jenkins, then verify the config was loaded as expected.

dirk-thomas commented 5 years ago

I just edited the sections through the web interface.

dirk-thomas commented 5 years ago

PS: no need to reboot Jenkins for this. The change is effective immediately for every build started afterwards.

nuclearsandwich commented 5 years ago

no need to reboot Jenkins for this. The change is effective immediately for every build started afterwards.

No the reboot was to make sure that the config loaded as expected from the file on disk. Since you made the change directly placing the config file manually is itself unnecessary.