From e0164be0181fc80632998b5d82df9f96243dbebf Mon Sep 17 00:00:00 2001 From: Travis Brown Date: Thu, 12 Mar 2026 20:37:36 +0000 Subject: [PATCH] [Nexthop] Fixup Distro Image service dependencies Avoid succeeding through retries. fboss_init uses weutil to produce fruid.json. weutil first requires that platform_manager is up so it can access the EEPROMs via the devmap. Thus here we fix the dependency order and make platform_manager a hard dependency of fboss_init. qsfp_service gets a similar treatment with fboss_init so it had fruid.json available at the first start. --- .../root_files/usr/lib/systemd/system/fboss_init.service | 3 ++- .../root_files/usr/lib/systemd/system/platform_manager.service | 2 +- .../root_files/usr/lib/systemd/system/qsfp_service.service | 2 ++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/fboss_init.service b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/fboss_init.service index 74c320771d9fc..cd5c8bba93d67 100644 --- a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/fboss_init.service +++ b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/fboss_init.service @@ -2,7 +2,8 @@ [Unit] Description=FBOSS Initialization DefaultDependencies=no -Before=platform_manager.service +After=platform_manager.service +Requires=platform_manager.service After=local-fs.target [Service] diff --git a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/platform_manager.service b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/platform_manager.service index 2278a5b17f0af..af961c148d013 100644 --- a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/platform_manager.service +++ b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/platform_manager.service @@ -8,7 +8,7 @@ Before=led_service.service Before=qsfp_service.service Before=fruid.service After=rc-local.service -After=fboss_init.service +Before=fboss_init.service Wants=local_rpm_repo.service Wants=fboss_init.service diff --git a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/qsfp_service.service b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/qsfp_service.service index 263e43a9c91ab..6486435bc5bbe 100644 --- a/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/qsfp_service.service +++ b/fboss-image/image_builder/templates/centos-09.0/root_files/usr/lib/systemd/system/qsfp_service.service @@ -3,6 +3,8 @@ Description=FBOSS QSFP Service After=platform_manager.service After=rc-local.service +Wants=fboss_init.service +After=fboss_init.service [Service] Type=simple