Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Meru800biab: Fix invalid led_manager config #304

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aalamsi22
Copy link
Contributor

Description

Recently there have been changes to revamp the config structure. This PR updates the config to match the new revamped structure.

NOTE: This config is identical to that of meru800bia config.

Test

[root@vpr402 ~]# systemctl status data_corral_service
● data_corral_service.service - Start data_corral_service
     Loaded: loaded (/etc/systemd/system/data_corral_service.service; enabled; preset: disabled)
     Active: active (running) since Wed 2024-12-04 01:08:48 UTC; 2s ago
   Main PID: 6680 (run_data_corral)
      Tasks: 50 (limit: 405268)
     Memory: 6.2M
        CPU: 10ms
     CGroup: /system.slice/data_corral_service.service
             ├─6680 /bin/bash /opt/fboss/bin/run_data_corral_service.sh
             └─6681 /opt/fboss/bin/data_corral_service -config_file /opt/fboss/share/platform_configs/led_manager.json

Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544359  6682 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/psu_led:red:status/brightness
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544364  6682 LedManager.cpp:55] Programmed PSU LED with presence true
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544369  6682 LedManager.cpp:45] Programming FAN LED with true
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544387  6682 LedManager.cpp:80] Wrote 1 to file /sys/class/leds/fan_led:green:status/brightness
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544404  6682 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/fan_led:red:status/brightness
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544409  6682 LedManager.cpp:55] Programmed FAN LED with presence true
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544414  6682 LedManager.cpp:25] Programming system LED with true
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544432  6682 LedManager.cpp:80] Wrote 1 to file /sys/class/leds/sys_led:green:status/brightness
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544451  6682 LedManager.cpp:80] Wrote 0 to file /sys/class/leds/sys_led:red:status/brightness
Dec 04 01:08:48 vpr402.sjc.aristanetworks.com run_data_corral_service.sh[6681]: I1204 01:08:48.544456  6682 LedManager.cpp:34] Programmed system LED with true

@facebook-github-bot
Copy link
Contributor

@joancaneus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot pushed a commit that referenced this pull request Dec 5, 2024
Summary:
# Description
Recently there have been changes to revamp the config structure. This PR updates the config to match the new revamped structure.

NOTE: This config is identical to that of meru800bia config.

# Test
```
[root@vpr402 ~]# systemctl status data_corral_service
● data_corral_service.service - Start data_corral_service
     Loaded: loaded (/etc/systemd/system/data_corral_service.service; enabled; preset: disabled)
     Active: active (running) since Wed 2024-12-04 01:08:48 UTC; 2s ago
   Main PID: 6680 (run_data_corral)

Reviewed By: kimdo8736

Differential Revision: D66798665

Pulled By: joancaneus

fbshipit-source-id: ee2c1853672494ad1afa39edcb156935d02fda83
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants