hwp-supervisor: make IBootState compatible with synaccess. - Githubissues

simonsobs / socs

Simons Observatory specific OCS agents.

BSD 2-Clause "Simplified" License

12 stars 12 forks source link

hwp-supervisor: make IBootState compatible with synaccess. #693

Closed ykyohei closed 3 months ago

ykyohei commented 3 months ago

Make IBootState compatible with synaccess agent originally it was only compatible with ibootbar agent.

Description

Make IBootState compatible with synaccess agent and add encoder timestamp This PR addresses https://github.com/simonsobs/socs/issues/691 and https://github.com/simonsobs/socs/issues/681

Motivation and Context

https://github.com/simonsobs/socs/issues/691 and https://github.com/simonsobs/socs/issues/681

How Has This Been Tested?

Tested in daq-dev for satp3

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

ykyohei commented 3 months ago

Right, the configuration is not changed, and I'm confused why it was working before. supervisor agent skips IBootState monitoring if synaccess agent is down when supervisor agent is brought up, so we might have been in such an edge case. But I couldn't completely reproduce the previous state.

And I do not understand why the docker build test is failed..