-
Notifications
You must be signed in to change notification settings - Fork 107
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
Update YUI JavaScript library to YUI3 #10597
Comments
This is not as simple as a library update. Most of the code under |
This issue introduces a very bad hold off of an obsolete library, which on the other hand was blocking this issue : #8797 from progressing. This situation led to a really unpleasant hack implemented at: dmwm/CMSKubernetes#1364, which was basically - to manually download from cmsrpo and unpack at the destination, the obsolete What we need to do here is not only upgrade to a newer version of this package, but completely re-estimate the need of it and refactor the code if possible. Because So currently we need to get rid of it completely if possible. |
@todor-ivanov , could you please clarify where YUI library is used. |
Impact of the new feature
T0, WMAgent and WorkQueue
Is your feature request related to a problem? Please describe.
As I investigate another problem with pushing couchapps, it turns out that the YUI javascript/css library that some of our DMWM services use has been deprecated since 2011 (!!!), see:
https://yui.github.io/yui2/
The recommended library is YUI3, and we seem to already have the spec and some COMP services depending on it. So we should update the other specs to the newest library as soon as possible.
Describe the solution you'd like
Update our WMCore-related spec files depending on
yui
toyui3
. We should also profit from this opportunity and try to update the yui3 version, if possible.Describe alternatives you've considered
This must be done for yesterday! But we better wait for the python3 services deployment first, to avoid having too many changes all in at the same time.
Additional context
Somehow related: #10596
The text was updated successfully, but these errors were encountered: