Skip to content
Permalink
Browse files

Merge pull request #14 from Dohbedoh/JENKINS-48049

[JENKINS-48049] Switch to SimpleBuildWrapper for pipeline compatibility

Closes JENKINS-48049
  • Loading branch information
ptierno committed Apr 10, 2018
2 parents fa47d4c + 7144cf9 commit 3d74f83ddb4e567dc39c2f430083890d7d93d068
Showing with 3 additions and 4 deletions.
  1. +3 −4 src/main/java/com/datapipe/jenkins/vault/VaultBuildWrapper.java
@@ -33,6 +33,7 @@
import javax.annotation.CheckForNull;
import javax.annotation.Nonnull;

import hudson.tasks.BuildWrapperDescriptor;
import org.apache.commons.lang.StringUtils;
import org.kohsuke.stapler.DataBoundConstructor;
import org.kohsuke.stapler.DataBoundSetter;
@@ -60,11 +61,9 @@
import hudson.Launcher;
import hudson.console.ConsoleLogFilter;
import hudson.model.AbstractProject;
import hudson.model.Descriptor;
import hudson.model.Run;
import hudson.model.TaskListener;
import hudson.security.ACL;
import hudson.tasks.BuildWrapper;
import jenkins.tasks.SimpleBuildWrapper;

public class VaultBuildWrapper extends SimpleBuildWrapper {
@@ -177,14 +176,14 @@ public ConsoleLogFilter createLoggerDecorator(
* that it can be accessed from views.
*/
@Extension
public static final class DescriptorImpl extends Descriptor<BuildWrapper> {
public static final class DescriptorImpl extends BuildWrapperDescriptor {
public DescriptorImpl() {
super(VaultBuildWrapper.class);
load();
}

@Override
public boolean isApplicable(AbstractProject<?, ?> item) {
// Indicates that this builder can be used with all kinds of project types
return true;
}

0 comments on commit 3d74f83

Please sign in to comment.
You can’t perform that action at this time.