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

Sidecar Front I/O QSFP LEDs stay on while system is in A2 #1995

Closed
rmustacc opened this issue Jan 29, 2025 · 0 comments · Fixed by #1998
Closed

Sidecar Front I/O QSFP LEDs stay on while system is in A2 #1995

rmustacc opened this issue Jan 29, 2025 · 0 comments · Fixed by #1998
Assignees
Labels
hardware Related to hardware components.

Comments

@rmustacc
Copy link
Contributor

@leftwo was working on a Sidecar and had commanded the system go to A2. While it did successfully do so, it left all the front I/O LEDs in the last state that they were in when dendrite had operated it. Given that the switch zone and dendrite only generally expect to operate while Sidecar is in A0 (though this may be something we need to consider and change), we likely should be defaulting to turning off LEDs when in A2.

This definitely requires some deeper thought about how we want the service and control points to operate while we're doing troubleshooting and in-field service, but given that this can happen independent of dendrite existing, they likely shouldn't remain on. I expect the flow of control and the ordering of different requests via the broader control plane and dendrite is something that will still want to continue to evolve, but at the moment what we have for behavior is rather confusing.

@rmustacc rmustacc added the hardware Related to hardware components. label Jan 29, 2025
Aaron-Hartwig added a commit that referenced this issue Feb 4, 2025
While Sidecar is not in A0 the transceivers-server will now keep the
LEDs off. The current behavior is that the LEDs only change state when
Dendrite tells them to, and while the Sidecar is not in Dendrite isn't
doing much at all! An A0->A2 transition left the LEDs in their current
state which is confusing for folks who may have eyes on the switch as it
would indicate Dendrite is running when it is in fact not. The system
attention LED is unaffected by this commit.

fixes #1995
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hardware Related to hardware components.
Projects
None yet
2 participants