For what it's worth, the  Templating Engine Plugin  had almost the exact same issue where objects created and injected into the binding were throwing CPS Mismatch warnings.

we're refactoring anything placed in the binding that's going to be invoking methods to be CPS transformed before being placed injected.  

After doing this - the only CpsMismatch warnings we got were around metaprogramming examples such as using InvokerHelper to invoke methods, or by taking advantage of groovy's methodMissing functionality.

those use cases are being addressed in https://github.com/cloudbees/groovy-cps/pull/99

Thanks you sterrana - we're not sure we can implement a similar solution in the Artifactory Plugin.

dnusbaum - In our case, the method name used in the pipeline script (for example, "upload"), is different from the name returned by DescriptorImpl.getFunctionName() ("artifactoryUpload").  Can you please recommend a solution?

Thanks

eyalbe Hmm, looks like your code is rather unique, you directly call CpsScript.invokeMethod inside of ArtifactoryServer.upload. This kind of metaprogramming is opaque to the CPS method mismatch detector because ArtifactoryServer.upload is in a Plugin rather than directly in a shared library or the Pipeline itself, so what we did for sterrana's case doesn't apply here.

CC jglick, any ideas on a fix or workaround? The only thing I can think of would be to ignore mismatch warnings in workflow-cps whenever the receiver is a plugin using something like this:

diff --git a/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsVmExecutorService.java b/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsVmExecutorService.java
index 7034810e..e96757cb 100644
--- a/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsVmExecutorService.java
+++ b/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsVmExecutorService.java
@@ -12,4 +12,5 @@ import java.util.logging.Level;
 import java.util.logging.Logger;
 import javax.annotation.CheckForNull;
+import jenkins.model.Jenkins;
 import jenkins.util.InterceptingExecutorService;
 
@@ -113,4 +114,9 @@ class CpsVmExecutorService extends InterceptingExecutorService {
 
     private void handleMismatch(Object expectedReceiver, String expectedMethodName, Object actualReceiver, String actualMethodName) {
+        Class receiverClass = expectedReceiver.getClass();
+        if (Jenkins.get().getPluginManager().whichPlugin(receiverClass) != null) {
+            // Plugin code is opaque to the mismatch detector.
+            return;
+        }
         String mismatchMessage = mismatchMessage(className(expectedReceiver), expectedMethodName, className(actualReceiver), actualMethodName);
         if (FAIL_ON_MISMATCH) {

I haven't tested that to see if it even works, and I think that might also mask real problems in DSLs like pipeline-model-definition and docker-workflow, so it doesn't seem like a great solution.

It does not seem like a great workaround either.

We already explicitly discourage domain-specific plugins from using GlobalVariable and implementing DSLs, but what ArtifactoryServer is doing is an additional complication: the returned object, or one of its resultant objects, is implemented in Java rather than CPS-transformed Groovy. I would consider this to be an unsupported and unsupportable trick.

Although I should say that since docker-workflow and pipeline-model-definition use the less problematic approach of exposing CPS-transformed Groovy, the receiverClass in those cases should not be loaded by a Jenkins plugin class loader as far as I know, so I do not believe such a patch would mask mistakes in those plugins, though it is worth confirming this.

(To confuse matters, pipeline-model-definition-plugin/pipeline-model-definition/src/main/groovy/ exists, but as far as Jenkins is concerned these are just plain old classes in the plugin. The CPS-transformed code that would be involved here is in pipeline-model-definition-plugin/pipeline-model-definition/src/main/groovy/**/*.groovy.

jglick, so what should we do to resolve this issue?

Can we perhaps allow specific steps to be excluded and avoid this message?