palantir / gerrit-ci

Plugin for Gerrit enabling self-service continuous integration workflows with Jenkins.
Apache License 2.0
18 stars 9 forks source link

Force all jobs to have a build timeout and clean some unrelated code. #4

Closed mass closed 9 years ago

mass commented 9 years ago

Job parsing could break for existing jobs if they do not have the build-timeout plugin installed. Before deploying this to production, we should manually enable the build-timeout plugin for all jobs on the Jenkins server.

Also make some small style changes to the Gerrit-CI UI semi-related to the removal of the timeoutEnabled checkbox.

newren commented 9 years ago

Looks great other than the one orthogonal change in the first commit that should be pulled out into a separate commit.