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
JENKINS-64195 Make plugin work on Jenkins >=2.266 #190
Conversation
723da02
to
f1ddda1
Compare
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.
Haven' tested. Jelly code upgrades look good. Cannot evaluate the rest.
Hi @xuzhang3 Would you be able to take a look at this please? Without this fix the Azure VM agents plugin UI is broken on versions newer than 2.263. This is because of the tables to divs migration in Jenkins core for a better UI experience: All UI widgets were updated in core, but the azure-commons plugin had forked versions of two components that broke. Please let me know if you have any questions, thanks. Note: This is using a pre-release build of azure-commmons, that one needs to be merged and released first |
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.
LGTM. A lot of changes for a single PR, but all of them look right
LGTM |
jenkinsci/azure-commons-plugin#51 Needs to be released first and then updating the version here |
Requires jenkinsci/azure-commons-plugin#51
I've included a couple of other fixes in here, https://issues.jenkins.io/browse/JENKINS-56077 and terminology updates, can be extracted to a different PR if required