CCS changes for Tucson test-stand #464
Conversation
glennmorris
force-pushed
the
ccs-tu-tts
branch
from
fdfa6b4
to
aef0b7a
Compare
|
Force pushed to fix a last minute typo, will close and reopen for clarity. |
|
Seems github doesn't let me replace this pull request, so I will reopen it. |
There was a problem hiding this comment.
@glennmorris Do you want me to merge this and move the tu_production branch forward?
| @@ -0,0 +1,10 @@ | |||
| --- | |||
| ccs_site: "tucson" | |||
There was a problem hiding this comment.
@jhoblitt Yes, if you are ok with it, please merge it, thanks.
| ccs_site: "tucson" | ||
|
|
||
| ## FIXME this should be "comcam-mcm-dds", same as real comcam. | ||
| ccs_sal::dds_interface: "140.252.147.52" |
There was a problem hiding this comment.
The dds interface still hasn't shown up in foreman for some reason. I'll sort this out tomorrow.
| ## Installation of DAQ client: | ||
| ## Download DAQ tarfile, extract it, create /etc/ccs config files. | ||
|
|
||
| class profile::ccs::daq_client ( |
There was a problem hiding this comment.
This appears to be completely stand alone and could be in an external module. There are a bunch of example of things in the tree that should be in an external module as they aren't a composition. I don't object to merging this now but it should probably be moved external at some point in the future in the interest of keeping our profile tree small.
There was a problem hiding this comment.
Thanks, I will try to separate it out at some point in the future.
| @@ -3,6 +3,7 @@ classes: | |||
| - "profile::core::common" | |||
| - "profile::ccs::common" | |||
| - "profile::ccs::tomcat" | |||
| - "profile::ccs::graphical" | |||
There was a problem hiding this comment.
FYI - I usually sort the inclusion list but the linter doesn't care.
Changes related to Tucson test-stand. The only change that applies elsewhere is the one to hieradata/cluster/comcam-ccs.yaml, but it should be a no-op.
These changes should address most of the issues from
https://jira.lsstcorp.org/browse/IT-3277?jql=project%20%3D%20IT%20AND%20resolution%20%3D%20Unresolved%20AND%20labels%20%3D%20tucson-test-stand%20ORDER%20BY%20priority%20DESC%2C%20updated%20DESC