So I've hand removed those lines from beta2 and the templates are happy. What's not so happy is the "Use Rest API" - all I get when I enter a username password and try to test is "Connection error : peer not authenticated" I even tried a user in the Gerrit Administrators group but I still get the same response. The https cert is signed by our internal certificate - but that won't be in the Jenkins java trustsore. Is that a problem - or is it something else? I've tried both the users ldap password and the Gerrit http password.
/James From: [email protected] [mailto:[email protected]] On Behalf Of Mathieu Wang Sent: 12 December 2013 15:19 To: [email protected] Cc: 'Mathieu Wang' Subject: Re: Beta release of Gerrit Trigger plugin 2.11.0 Yes, this variable is not used anywhere, so we can remove the tag safely. On Thursday, December 12, 2013 4:41:15 AM UTC-5, Robert Sandell wrote: Actually the GerritTrigger class is quite explicit in its declaration to Jenkins that it asserts AbstractProject to be the item it is added to. Whatever kind of trickery the template plugin does after that is somewhat beyond its control. On the other hand I did a quick usage search for the trigger variable that is being declared on the line where it blows up and couldn't find any. @Mathew do you think it's safe to remove the <j:invokeStatic var="trigger"... tag on line 5 in config.jelly? Robert Sandell Software Tools Engineer - SW Environment and Product Configuration Sony Mobile Communications From: [email protected]<javascript:> [mailto:[email protected]<javascript:>] On Behalf Of teilo Sent: den 11 december 2013 17:45 To: [email protected]<javascript:> Subject: Re: Beta release of Gerrit Trigger plugin 2.11.0 sorry trigger happy that should read "from an earlier post 2.10.1 release (aka 2.10.2-snapshot)" On Wednesday, 11 December 2013 16:43:32 UTC, teilo wrote: Not sure this is still the case (as its from an early post but the plugins assumptions about project types appears incorrect and results in the following error javax.servlet.ServletException: org.apache.commons.jelly.JellyTagException: file:/opt/jenkins/data/plugins/gerrit-trigger/WEB-INF/classes/com/sonyericsson/hudson/plugins/gerrit/trigger/hudsontrigger/GerritTrigger/config.jelly:6:68: <j:arg> Can't convert com.cloudbees.hudson.plugins.modeling.controls.DescriptorListControl@3e0dfca4<javascript:> to class hudson.model.AbstractProject I don't think this is a regression in the Cloudbees templates (although they may have been upgraded at similar times). On Monday, 9 December 2013 18:45:07 UTC, Robert Sandell wrote: Did some fixes in this area and released 2.11-beta-2 Robert Sandell Software Tools Engineer - SW Environment and Product Configuration Sony Mobile Communications From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of Sandell, Robert Sent: den 9 december 2013 16:48 To: [email protected]<mailto:[email protected]> Subject: RE: Beta release of Gerrit Trigger plugin 2.11.0 Ah, It's because the server config is retrieved via the serverName field of the trigger and not from the provider of the event. I noticed some other places where I've missed that particular issue. I'll start a 2.11-beta branch to work on these. Robert Sandell Software Tools Engineer - SW Environment and Product Configuration Sony Mobile Communications From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of Jyrki Puttonen Sent: den 7 december 2013 18:00 To: [email protected]<mailto:[email protected]> Subject: Re: Beta release of Gerrit Trigger plugin 2.11.0 I found out one problem, using "Any Server" prevents Jenkins from reporting result to Gerrit. This happens from retriggering, pushing and from the search and trigger -page. Log has following line in every case: Caused by: java.lang.NullPointerException at com.sonyericsson.hudson.plugins.gerrit.trigger.hudsontrigger.GerritTrigger.schedule(GerritTrigger.java:531) At that line, there's PluginImpl.getInstance().getServer(serverName).getConfig().isGerritBuildCurrentPatchesOnly() I would guess that getServer return null when "Any Server" is selected. I'm not sure what should be done when the returned server is null, so I couldn't fix this myself :) On Sat, Dec 7, 2013 at 3:55 PM, Jyrki Puttonen <[email protected]<mailto:[email protected]>> wrote: Hi, I just installed the beta version, and had few problems. 1. Gerrit configuration was lost, and I had to add a new server. 2. Manual triggering from search didn't work. After following steps and testing after everyone it started to work - I had $GERRIT_REFSPEC as Branch Specifier, changed to $GERRIT_BRANCH - Changed server from Any Server to specific one - Touched every field in Project configuration (pattern, branch) Then it started to work. After it started to work, I changed server back to Any Server and Branch Specifier back to $GERRIT_REFSPEC and it seems to be working. Pushing to gerrit seems to be working, at least after I did those previous steps. I'll keep the beta version running and report if there's other problems. On Thu, Nov 28, 2013 at 5:40 PM, Sandell, Robert <[email protected]<mailto:[email protected]>> wrote: The notify parameter is not something you can set (yet), there is probably still much that can be done to enhance the usage of the REST API. The only thing that is used from the command settings is the contents of the message. Robert Sandell Software Tools Engineer - SW Environment and Product Configuration Sony Mobile Communications From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of Øyvind Harboe Sent: den 28 november 2013 10:24 To: [email protected]<mailto:[email protected]> Cc: [email protected]<mailto:[email protected]> Subject: Re: Beta release of Gerrit Trigger plugin 2.11.0 Hi Robert, I installed the beta because I would like to switch from the ssh commands to REST API commands in order to more carefully manage which email notifications are sent out. Is there any documentation or pointers on how I can change the line below, which adds a comment that a build was started, to the REST API command to do the same, but set notify=NONE? gerrit approve <CHANGE>,<PATCHSET> --message 'Build Started <BUILDURL> <STARTED_STATS>' --verified <VERIFIED> --code-review <CODE_REVIEW> REST API feature that I would like to use: https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#review-input On Tuesday, November 26, 2013 4:15:19 PM UTC+1, Robert Sandell wrote: I just released version 2.11.0-beta-1 of the Gerrit Trigger plugin. It contains some great new features by some great contributors, the release notes are on the wiki [1]. The support for multiple Gerrit servers required quite a lot of refactoring in central parts of the plugin that potentially could affect everything, so I released a beta that should be available in the experimental update center [2] within the normal waiting period. And I am looking for some brave people who can help us test it and provide me with a hint if it is good enough for a "normal" release or if there is anything major in need of fixing before. Thanks, Bobby [1] https://wiki.jenkins-ci.org/display/JENKINS/Gerrit+Trigger [2] Set your update center to http://updates.jenkins-ci.org/experimental/update-center.json Robert Sandell Software Tools Engineer SW Environment and Product Configuration Development Environment Sony Mobile Communications Tel: +46 (0)10 80 12721 sonymobile.com<http://sonymobile.com/> -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<javascript:>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/groups/opt_out. -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
