Skip to content

config: fix compatibility issue#44

Closed
matttbe wants to merge 1 commit intojenkinsci:masterfrom
matttbe:patch-1
Closed

config: fix compatibility issue#44
matttbe wants to merge 1 commit intojenkinsci:masterfrom
matttbe:patch-1

Conversation

@matttbe
Copy link

@matttbe matttbe commented Aug 6, 2018

It seems that the configuration name has changed and this break the
compatibility:

Before (v1.22 or before):

<org.jenkinsci.plugins.cppcheck.CppcheckPublisher>
  <cppcheckConfig>
    <pattern>**/cppcheck.xml</pattern>

After (v1.23):

<org.jenkinsci.plugins.cppcheck.CppcheckPublisher plugin="cppcheck@1.23">
  <config> <!-- Not cppcheckConfig anymore -->
    <pattern>**/cppcheck.xml</pattern>

You can check the generated config.xml file by looking at this address:

https://<JENKINS_URL>/job/<JOBNAME>/config.xml

I am not sure how the XML scheme is generated but I guess it inspects
the name of the variables. Then this change should fix the
compatibility issue.

Linked to this issue: JENKINS-52854
Signed-off-by: Matthieu Baerts matthieu.baerts@tessares.net

@matttbe
Copy link
Author

matttbe commented Aug 6, 2018

@marcosteffan : I didn't try this but I hope it should fix JENKINS-52854 issue !

It seems that the configuration name has changed and this break the
compatibility:

Before (v1.22 or before):

    <org.jenkinsci.plugins.cppcheck.CppcheckPublisher>
      <cppcheckConfig>
        <pattern>**/cppcheck.xml</pattern>

After (v1.23):

    <org.jenkinsci.plugins.cppcheck.CppcheckPublisher plugin="cppcheck@1.23">
      <config> <!-- Not cppcheckConfig anymore -->
        <pattern>**/cppcheck.xml</pattern>

You can check the generated config.xml file by looking at this address:

    https://<JENKINS_URL>/job/<JOBNAME>/config.xml

I am not sure how the XML scheme is generated but I guess it inspects
the name of the variables. Then this change should fix the
compatibility issue.

Issue: JENKINS-52854
Fixes: 32710c4 (Fixed storing plugin-settings)
Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
@matttbe
Copy link
Author

matttbe commented Aug 20, 2018

@marcosteffan : I didn't try this but I hope it should fix JENKINS-52854 issue !

I mentioned it on the Jenkins ticket but I forgot to mention it here as well. I did test it and it does fix the Jenkins issue ;-)

@eckschi
Copy link

eckschi commented Sep 5, 2018

the change in 32710c4 also broke all projects made by https://docs.openstack.org/infra/jenkins-job-builder/ so merging would fix issues there

@matttbe
Copy link
Author

matttbe commented Sep 5, 2018

@eckschi Yes, I confirm that. Note that you can temporary use the .hpi from https://ci.jenkins.io/blue/organizations/jenkins/Plugins%2Fcppcheck-plugin/detail/PR-44/2/artifacts/

@janitza-thbe
Copy link

@marcosteffan
Please be so kind and merge this PR and release a new version afterwards since my company is badly awaiting for both pipeline support and backward compatibility to our dozens of build jobs w/ old v1.21 configurations that are lost on plugin upgrade.
Thx.

@marcosteffan
Copy link
Contributor

Hi,
that issue should be fixed with https://issues.jenkins-ci.org/browse/JENKINS-52854 which has been released just now.

Best regards

@matttbe
Copy link
Author

matttbe commented Sep 13, 2018

@marcosteffan : thx you for the fix. I guess we can abandon this one

@matttbe matttbe closed this Sep 13, 2018
@matttbe matttbe deleted the patch-1 branch September 13, 2018 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants