Skip to content

Conversation

@ZLLentz
Copy link
Member

@ZLLentz ZLLentz commented Dec 9, 2025

Description

Do not merge: most team members can't use sdfconfig yet

Remove all references to netconfig and include suitable replacement calls to sdfconfig or host

  • restartdaq: Replace the netconfig search *-fez sort of call with the equivalent sdfconfig search for fez subnets (can't do the same search because the -fez etc. are part of the same entry as the main interface)
  • serverStat: All calls that used netconfig to search for hostname existence have been replaced by the host utility, which is faster and wont disappear. (Incidentally: serverStat status is now much faster). The netconfig call that was used to view the netconfig output has been replaced by the equivalent sdfconfig call.

Motivation and Context

https://jira.slac.stanford.edu/browse/ECS-9486

Netconfig is going away
The replacement tool is sdfconfig
In some cases, we can get away with using neither, so that is preferred when possible for forward compatibility.

How Has This Been Tested?

Interactively

  • restartdaq I've forced the bash script into this error state and made sure it gave reasonable results
  • serverStat was run through the paces with the status and expert commands on servers with/without fez to get some output variety

Where Has This Been Documented?

Here only

@ZLLentz ZLLentz requested a review from a team as a code owner December 9, 2025 21:45
@ZLLentz ZLLentz marked this pull request as draft December 9, 2025 21:46
@ZLLentz
Copy link
Member Author

ZLLentz commented Dec 15, 2025

I think what's left for me to do here is:

  • run serverStat in all the combinations that cover the lines I changed, making sure it works as intended
  • run restartDaq in a hutch where it will hit the error message I changed

@ZLLentz ZLLentz marked this pull request as ready for review December 15, 2025 22:41
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.

1 participant