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

Hostmanager + jackhammer compatibility #451

Merged
merged 4 commits into from
Feb 25, 2025
Merged

Conversation

jlashner
Copy link
Collaborator

In the near future, we want to get hostmanager operating on smurf-servers to automatically start ocs agents.

Once this is the case, we will not want to touch ocs dockers using jackhammer... it should all be managed by the hostmanager.

With this PR, if sys_config["use_hostmanager"] is set to True, it will exclude ocs dockers from any hammer operations.

When setting this up, we will need to move ocs-related docker services to an independent docker-compose file to keep host-manager from interacting with smurf-streamer dockers, as is done here: https://github.com/simonsobs/ocs-deployment-configs/pull/332.

@jlashner jlashner requested a review from msilvafe January 22, 2025 21:31
@jlashner
Copy link
Collaborator Author

I've tested this with satp3 and its good to merge!

One thing is that there was an issue (simonsobs/ocs#425) that required us to change all docker-compose commands to docker compose.

The old docker-compose v1 has been deprecated for several years so I think this is ok. I believe should only effect servers running ubuntu 18.04, so nothing in operation, and any lab-setup running 18.04 should probably upgrade anyway.

Copy link
Contributor

@msilvafe msilvafe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to send out some sort of slack blast to smurf users to let them know that docker-compose is no longer used but I think its time. This change happened a while ago now

@jlashner jlashner merged commit 2ab9d08 into master Feb 25, 2025
1 check passed
@jlashner jlashner deleted the hostmanager-hammer-support branch February 25, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants