JENKINS-28030 I was using the preCheckout method to write the .chef f…

…older when I should have been using setUp. Now .chef folder won't be erased by SCM checkout on first job run.
jenkinsci · May 8, 2015 · a19a226 · a19a226
1 parent 0ce7d00
commit a19a226
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/main/java/io/chef/jenkins/ChefIdentityBuildWrapper.java b/src/main/java/io/chef/jenkins/ChefIdentityBuildWrapper.java
@@ -79,7 +79,7 @@ public String getJobIdentity() {
 	}
 
 	@Override
-	public void preCheckout(AbstractBuild build, Launcher launcher, BuildListener listener) throws IOException, InterruptedException {
+	public Environment setUp(AbstractBuild build, Launcher launcher, BuildListener listener) throws IOException, InterruptedException {
 		// This is where you 'build' the project.
 		listener.getLogger().println("Running build with Chef Identity of " + this.jobIdentity);
 		ChefIdentity chefIdentity = new ChefIdentity();
@@ -116,10 +116,7 @@ public void preCheckout(AbstractBuild build, Launcher launcher, BuildListener li
 				new FilePath(ws, ".chef/knife.rb").write(chefIdentity.getKnifeRb(), "UTF-8");
 			}
 		}
-	}
-
-	@Override
-	public Environment setUp(AbstractBuild build, Launcher launcher, BuildListener listener) throws IOException, InterruptedException {
+
 		return new NoopEnv();
 	}