search

ocf / ocfweb

The main ocf website
https://www.ocf.berkeley.edu
Other
38 stars 90 forks source link

Add docs for CLI printer removal #754

Closed Boomaa23 closed 2 years ago

Boomaa23 commented 2 years ago

Fix docs according to https://github.com/ocf/utils/issues/101.

dkess commented 2 years ago

The reason why my issue was filed in ocf/utils is because I was suggesting writing a script to do it. When taking a printer out of service, you always want to take it out of service for double and single at the same time. These docs don't make that clear. It's better to have a script where you just select a printer and it automatically runs the command twice (once for single and once for double).

Boomaa23 commented 2 years ago

I see. Okay in that case I will work on it some more, add a script, and modify these docs.

dkess commented 2 years ago

Please explain that the script has to be run on whiteout.

Also, I get a command not found error when I try it on whiteout now.

Boomaa23 commented 2 years ago

I will add that it has to be run on whiteout. I'm fairly confident "command not found" is because someone with root has to run update-symlinks. An additional problem is that required is an invalid argument for a subparser with Python 3.5, so that will have to be fixed first.

Boomaa23 commented 2 years ago

See https://github.com/ocf/utils/pull/175 for script fix. I don't have root so I'll try to ask someone to run update-symlinks for me.

Boomaa23 commented 2 years ago

update-symlinks done, see https://github.com/ocf/utils/pull/176. Should be good to go!