search

NSLS-II-SIX / profile_collection

Collection ipython profile
BSD 3-Clause "New" or "Revised" License
0 stars 3 forks source link

Enable Azure Pipelines integration #21

Closed mrakitin closed 5 years ago

mrakitin commented 5 years ago

Uses https://github.com/NSLS-II/profile-collection-ci.

Currently fails on caproto client (https://dev.azure.com/nsls2/profile_collections/_build/results?buildId=63):

CaprotoTimeoutError‌: <PV name='XF:02ID1-ES{RIXSCam}:HDF1:PluginType_RBV' priority=0 (searching....)> could not connect within 1.0-second timeout.‌
Exception in thread retry:
Traceback (most recent call last):
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/site-packages/caproto/threading/client.py", line 858, in _retry_unanswered_searches
    SEARCH_MAX_DATAGRAM_BYTES - len(version_req)):
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/site-packages/caproto/_utils.py", line 772, in batch_requests
    for command in request_iter:
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/site-packages/caproto/threading/client.py", line 830, in _construct_search_requests
    ca.DEFAULT_PROTOCOL_VERSION)
  File "/home/vsts/mc/envs/collection-2019C3.0.1/lib/python3.6/site-packages/caproto/_commands.py", line 607, in __init__
    ''.format(MAX_RECORD_LENGTH, name, _len))
caproto._utils.CaprotoValueError: EPICS 3.14 imposes a 59-character limit on record names. The record 'XF:02ID1-ES{RIXSCam}:XIP1:HR_ISOLINEAR_CALIBRATION_THRESHOLD' is 60 characters.

xref https://github.com/NSLS-II/profile-collection-ci/issues/2

mrakitin commented 5 years ago

attn @danielballan, I encountered another issue with SIX's profile files.

mrakitin commented 5 years ago

The problem should be resolved at the IOC level, reported via https://controlsweb.nsls2.bnl.gov/trac/ticket/3906#ticket.

mrakitin commented 5 years ago

For now this PR has nothing to do with the IOC configuration, and can be merged to enable the CI integration.

@danielballan, do you mind to have a look?

mrakitin commented 5 years ago

The issue #22 will handle the too-long-PV. This PR is good to go.

johnsinsheimer commented 5 years ago

Interestingly, I can caget this PV, despite the excessive characters, but I'll see if there is anything I can do with the IOC

mrakitin commented 5 years ago

Thank you, @johnsinsheimer! Epics CLI tools (caget, ...) and pyepics seem to be quite forgiving, so we haven't noticed the issue until now using caproto.

mrakitin commented 5 years ago

I think it's good to go in "as is". The known issues are understood and documented, all other updated will be made via separate PRs.

mrakitin commented 5 years ago

@ambarb, do you have any concerns regarding this update? It does not affect any code, just wanted to bring it to your attention.

mrakitin commented 5 years ago

This PR does not touch any startup files, self-merging.