Skip to content

Add option to reset multiple iocs for both afc and rffe.#191

Open
gustavosr8 wants to merge 1 commit intomasterfrom
all-iocs-restart
Open

Add option to reset multiple iocs for both afc and rffe.#191
gustavosr8 wants to merge 1 commit intomasterfrom
all-iocs-restart

Conversation

@gustavosr8
Copy link
Copy Markdown
Contributor

@gustavosr8 gustavosr8 commented Apr 2, 2026

Since we don't know beforehand which IOC the user wants to restart, we cannot define a default device list as required in ValidateDeviceListAction. To solve this, create a child class that waits for ´ioc´ to be defined in the parser namespace and then define the device list when called.

@gustavosr8 gustavosr8 requested a review from ericonr April 2, 2026 18:09
@gustavosr8 gustavosr8 linked an issue Apr 2, 2026 that may be closed by this pull request
@gustavosr8
Copy link
Copy Markdown
Contributor Author

@GuPecz

@ericonr
Copy link
Copy Markdown
Contributor

ericonr commented Apr 6, 2026

./sirius-script-rabpm-mngr.py ioc-restart rffe -s 13 -r 14
Enter passphrase for /home/sirius/.ssh/id_ed25519_rabpm:
Identity added: /home/sirius/.ssh/id_ed25519_rabpm (sirius@lnls451-linux)
Traceback (most recent call last):
  File "./sirius-script-rabpm-mngr.py", line 234, in <module>
    main()
  File "./sirius-script-rabpm-mngr.py", line 226, in main
    mngr.run_ioc_restart(rack, args.ioc, args.slots)
  File "./sirius-script-rabpm-mngr.py", line 78, in run_ioc_restart
    self._launch_ssh(self.verbose, hostname, "ioc-restart", ioc, slot)
  File "./sirius-script-rabpm-mngr.py", line 45, in _launch_ssh
    stdout=output, stderr=output))
  File "/usr/local/lib/python3.6/subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "/usr/local/lib/python3.6/subprocess.py", line 1295, in _execute_child
    restore_signals, start_new_session, preexec_fn)
TypeError: expected str, bytes or os.PathLike object, not int

am I holding it wrong?

Since we don't know beforehand which IOC the user wants to restart, we
cannot define a default device list as required in
ValidateDeviceListAction. To solve this, create a child class that
waits for ´ioc´ to be defined in the parser namespace and then define
the device list when called.
@gustavosr8
Copy link
Copy Markdown
Contributor Author

am I holding it wrong?

ops

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.

RaBPMMgr doesn't support -s all flag for ioc-restart command

2 participants