Skip to content

[JENKINS-31801] Add Pipeline throttle(category) step #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
oleg-nenashev merged 20 commits into from
May 19, 2017
Merged

[JENKINS-31801] Add Pipeline throttle(category) step #46

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
af877cd
[JENKINS-31801] Initial work on throttle(category) step - needs tests
abayer Mar 8, 2017
c000c30
Reworked to no longer rely on StepExecutions.
abayer Mar 10, 2017
91cdb4b
Add a trailing newline to messages.
abayer Mar 10, 2017
d011b40
Make findbugs happy.
abayer Mar 10, 2017
666ff55
Whoops, this needs to take a block
abayer Mar 11, 2017
42f4da1
Initial test, actually working
abayer Mar 13, 2017
8378722
Cleanup, commenting, javadoc
abayer Mar 13, 2017
492c500
Test across all nodes
abayer Mar 13, 2017
89f7bce
Add interop with freestyle test
abayer Mar 13, 2017
33b8e0f
Add snippet generator support.
abayer Mar 13, 2017
f09aea8
Add snippetizer support and test
abayer Mar 13, 2017
aca9d87
Review comments
abayer Mar 20, 2017
bb0fb3e
Check for pending PlaceholderTasks as well.
abayer Mar 21, 2017
13cafdf
Allow multiple comma-separated categories
abayer Apr 7, 2017
fdd9102
Minor review responses, moving step UI to correct dir
abayer Apr 25, 2017
4ec2057
Go away, empty category names!
abayer Apr 25, 2017
968d77c
Fixed up snippetizer, switched to a list of strings for the throttle …
abayer May 18, 2017
0b222d3
Check for and respond to duplicate or non-existent category names
abayer May 18, 2017
7d83b60
Adding help for ThrottleJobProperty pointing out it doesn't work for …
abayer May 18, 2017
ee1e959
unmodifiableList
abayer May 18, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
85 changes: 68 additions & 17 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,13 @@ THE SOFTWARE.
<parent>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>plugin</artifactId>
<version>2.6</version>
<version>2.22</version>
Copy link
Copy Markdown
Member Author

@abayer abayer Mar 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2.23 led to errors due to a TestExtension usage in workflow-support's tests, so I bumped back down to 2.22 for now at least.

Copy link
Copy Markdown
Member

@jglick jglick Apr 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harmless, just stack traces. workflow-support itself should be bumped up.

</parent>

<artifactId>throttle-concurrents</artifactId>
<packaging>hpi</packaging>
<name>Jenkins Throttle Concurrent Builds Plug-in</name>
<version>1.9.1-SNAPSHOT</version>
<version>2.0-beta-SNAPSHOT</version>
<url>http://wiki.jenkins-ci.org/display/JENKINS/Throttle+Concurrent+Builds+Plugin</url>
<description>Plugin to throttle the number of concurrent builds of a single job per node.</description>

Expand All @@ -44,12 +44,11 @@ THE SOFTWARE.
</licenses>

<properties>
<jenkins.version>1.609.3</jenkins.version>
<jenkins.version>1.642.3</jenkins.version>
Copy link
Copy Markdown
Member Author

@abayer abayer Mar 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As mentioned in the commit message, we needed to bump dependency versions to get the latest durable-task-step for PlaceholderTask.getNode().

<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
Copy link
Copy Markdown
Member

@jglick jglick Mar 8, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

Copy link
Copy Markdown
Member Author

@abayer abayer Mar 9, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup yup - at one point, I was experimenting with stuff that did need newer versions of other dependencies and bumped things, but I'll roll this back now.

<compileSource>1.6</compileSource>
<compileTarget>1.6</compileTarget>
<!--TODO: do not fail on errors-->
<findbugs.failOnError>false</findbugs.failOnError>
<java.level>7</java.level>
</properties>

<developers>
Expand Down Expand Up @@ -105,29 +104,87 @@ THE SOFTWARE.
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>matrix-project</artifactId>
<version>1.4.1</version>
<version>1.8</version>
</dependency>

<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-api</artifactId>
<version>2.12</version>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-step-api</artifactId>
<version>2.7</version>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-durable-task-step</artifactId>
<version>2.8</version>
Copy link
Copy Markdown
Member

@oleg-nenashev oleg-nenashev Mar 17, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a need to have it as required dependency? It pulls a significant number of Pipeline plugins into (like Pipeline Support & Co)

Copy link
Copy Markdown
Member Author

@abayer abayer Mar 17, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes - we need ExecutorStepExecution.class for checking against, etc.

Copy link
Copy Markdown
Member

@oleg-nenashev oleg-nenashev Apr 25, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do not use this class from what I see

Copy link
Copy Markdown
Member Author

@abayer abayer Apr 25, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, that may have gone away. Fixing.

Copy link
Copy Markdown
Member Author

@abayer abayer Apr 25, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, it's still there - specifically ExecutorStepExecution.PlaceholderTask in ThrottleQueueTaskDispatcher.

</dependency>

<!-- Dependencies for test -->
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>cloudbees-folder</artifactId>
<version>4.0</version>
<version>6.0.2</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>matrix-auth</artifactId>
<version>1.2</version>
<version>1.4</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>credentials</artifactId>
<version>1.9.4</version>
<version>2.1.13</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-cps</artifactId>
<version>2.28</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-cps</artifactId>
<version>2.28</version>
<classifier>tests</classifier>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-job</artifactId>
<version>2.9</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-basic-steps</artifactId>
<version>2.3</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins</groupId>
<artifactId>junit</artifactId>
<version>1.15</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-support</artifactId>
<version>2.13</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.jenkins-ci.plugins.workflow</groupId>
<artifactId>workflow-support</artifactId>
<version>2.13</version>
<classifier>tests</classifier>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.assertj</groupId>
<artifactId>assertj-core</artifactId>
Expand All @@ -140,12 +197,6 @@ THE SOFTWARE.
<version>0.7.2</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
<version>18.0</version>
<scope>test</scope>
</dependency>
</dependencies>
</project>

Loading