From 87a0d54099c5d3743a0392b3c8e684e8bc869ff7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Steven!=20Ragnar=C3=B6k?= Date: Wed, 4 Aug 2021 11:27:54 -0400 Subject: [PATCH] Switch rolling sync threshold from count to required packages. (#175) * Switch rolling sync threshold from count to required packages. For most ROS distributions we want to periodically ratchet up the package count for sync thresholds. However Rolling is meant to be somewhat volatile as large changes are made. A large enough number of ancillary packages have been introduced to Rolling that the "base" numeric package count is no longer sufficient to guarantee the presence of ROS core packages. So let's instead use the required packages sync criteria and require the presence of ros_desktop which will in turn require the ROS core and a relatively healthy ROS system to be available. * Keep a minimum package count as well as required packages. Co-authored-by: Tully Foote * Add package count requirements for other release configurations. Co-authored-by: Tully Foote --- rolling/release-build.yaml | 3 ++- rolling/release-focal-arm64-build.yaml | 3 ++- rolling/release-rhel-build.yaml | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/rolling/release-build.yaml b/rolling/release-build.yaml index 58239c23..e6440340 100644 --- a/rolling/release-build.yaml +++ b/rolling/release-build.yaml @@ -16,7 +16,8 @@ notifications: - ros2-buildfarm-rolling@googlegroups.com maintainers: true sync: - package_count: 262 + package_count: 500 + packages: [desktop] repositories: keys: - | diff --git a/rolling/release-focal-arm64-build.yaml b/rolling/release-focal-arm64-build.yaml index 7ce4f144..af5ee8d8 100644 --- a/rolling/release-focal-arm64-build.yaml +++ b/rolling/release-focal-arm64-build.yaml @@ -13,7 +13,8 @@ notifications: - ros2-buildfarm-rolling@googlegroups.com maintainers: true sync: - package_count: 262 + package_count: 500 + packages: [desktop] repositories: keys: - | diff --git a/rolling/release-rhel-build.yaml b/rolling/release-rhel-build.yaml index 45652190..21473a09 100644 --- a/rolling/release-rhel-build.yaml +++ b/rolling/release-rhel-build.yaml @@ -89,7 +89,8 @@ package_ignore_list: - rosidl_typesupport_gurumdds_cpp # No RPM package for GurumDDS - rti_connext_dds_cmake_module # No RPM package for Connext sync: - package_count: 262 + package_count: 400 + packages: [desktop] repositories: keys: - |