search

simonsobs / scheduler

Scheduler for SO
0 stars 1 forks source link

update hwp preamble #65

Closed ykyohei closed 4 months ago

ykyohei commented 4 months ago

Once both satp1 and satp3 deploy the hwp-supervisor agent, we can simplify the hwp preamble. This PR will adress https://github.com/simonsobs/scheduler/issues/52

bbixler500 commented 4 months ago

Looks good to me

kmharrington commented 4 months ago

We need to coordinate implementing this change and deploying the supervisor agents on both machines

ykyohei commented 4 months ago

Bryce and I talked about the supervisor agent status yesterday. It is deployed in both satp1 and satp3 For satp1, we should merge https://github.com/simonsobs/ocs-deployment-configs/pull/198, but docker agents are already running in daq-satp1 node. @bbixler500 Correct me if something is wrong.

bbixler500 commented 4 months ago

Yes that is correct Kyohei. The current satp1 schedule is already running using the supervisor agent with https://github.com/simonsobs/ocs-deployment-configs/pull/198 deployed to the daq-satp1 node (I manually changed the hwp function calls in the existing schedule).

ykyohei commented 4 months ago

Now the ocs-deployment-config https://github.com/simonsobs/ocs-deployment-configs/pull/198 is officially merged. Let me proceed to merge this PR.