-
Notifications
You must be signed in to change notification settings - Fork 3
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
Enable Azure Pipelines integration #21
Conversation
attn @danielballan, I encountered another issue with SIX's profile files. |
The problem should be resolved at the IOC level, reported via https://controlsweb.nsls2.bnl.gov/trac/ticket/3906#ticket. |
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? |
The issue #22 will handle the too-long-PV. This PR is good to go. |
Interestingly, I can caget this PV, despite the excessive characters, but I'll see if there is anything I can do with the IOC |
Thank you, @johnsinsheimer! Epics CLI tools ( |
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. |
@ambarb, do you have any concerns regarding this update? It does not affect any code, just wanted to bring it to your attention. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mrakitin thanks for the information. I didn't do much with this part of the profile so @awalter-bnl would be the best
This PR does not touch any startup files, self-merging. |
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):
xref NSLS-II/profile-collection-ci#2