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

ENH: pkg mirror, static xar configs #21

Merged
merged 4 commits into from
Mar 28, 2024
Merged

Conversation

ZLLentz
Copy link
Member

@ZLLentz ZLLentz commented Mar 26, 2024

Description

Main changes:

  • Optionally/default configure PLCs to load their packages from an internal pcds mirror of Beckhoff's repositories
  • Allow you to specify a static TwinCAT runtime (XAR) version in your config, rather than whichever random version your PLC started with (or in some cases, whichever was most recent online at time of setup). This required the internal mirror to implement.
  • The XAR defaults to 4026.3.55 because it's what I've been testing with, but is defined statically in each host's vars file instead of in the group vars files. This prevents us from accidentally updating any XARs automatically.
  • Allow us to enable/disable the FreeBSD, TCBSD, and SLAC package repositories independently and reversibly.

Smaller changes:

  • make the psproxy and psntp configs reversible
  • new default: no psproxy pkg config (not needed with the internal mirror)
  • Remove plc-tmo-tmp-vac, it's been renamed and will need a full re-boostrapping later.
  • Fix a small typo on the ecs-user's bash shell path. I had fixed this previously but it must have not been properly committed.

Motivation and Context

How Has This Been Tested?

Interactively on PLCs that run BSD13 and BSD14

Where Has This Been Documented?

Added the steps to https://confluence.slac.stanford.edu/display/PCDS/TcBSD+Ansible+Workflows
Looking for other places to make relevant callouts

Pre-merge checklist

@ZLLentz ZLLentz marked this pull request as ready for review March 26, 2024 23:48
@ZLLentz ZLLentz requested review from ghalym and nrwslac March 26, 2024 23:53
@ZLLentz
Copy link
Member Author

ZLLentz commented Mar 27, 2024

It'd be neat if this made it in before Divya tried out the scripts tomorrow, but it isn't fully necessary. If not merged by EOD, then I'm going to cherrypick the small bugfix with the bash path into a separate PR and deploy just that change.

Copy link

@slactjohnson slactjohnson left a comment

Choose a reason for hiding this comment

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

I don't know enough about ansible or BSD (yet) to comment to evaluate the playbooks for accuracy, but I don't see anything that stands out as incorrect. All the steps makes sense to me as things that should be done.

@ZLLentz ZLLentz merged commit ac1e707 into pcdshub:master Mar 28, 2024
1 check passed
@ZLLentz ZLLentz deleted the enh_pkg_mirror branch March 28, 2024 01:23
@ZLLentz ZLLentz mentioned this pull request Mar 28, 2024
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