Revision ee96cf71
Added by Ondřej Pražák almost 6 years ago
Added by | app/assets/javascripts/foreman_pipeline/jenkins-instances/details/views/jenkins-instance-details-info.html | ||
|---|---|---|
| 32 | 32 |
</span> |
| 33 | 33 |
</div> |
| 34 | 34 |
|
| 35 |
<div class="detail"> |
|
| 36 |
<span class="info-label" translate>Timeout |
|
| 37 |
<i class="fa fa-question-circle" |
|
| 38 |
tooltip="{{ 'Time in minutes we are willing to wait before build on Jenkins starts' | translate }}"
|
|
| 39 |
tooltip-animation="false" |
|
| 40 |
tooltip-append-to-body="true"> |
|
| 41 |
</i> |
|
| 42 |
</span> |
|
| 43 |
<span class="info-value" |
|
| 44 |
bst-edit-custom="jenkinsInstance.timeout" |
|
| 45 |
on-save="save(jenkinsInstance)"> |
|
| 46 |
<div class="form input"> |
|
| 47 |
<input ng-model="jenkinsInstance.timeout" |
|
| 48 |
name="timeout" |
|
| 49 |
type="number" |
|
| 50 |
min="1"/> |
|
| 51 |
</div> |
|
| 52 |
</span> |
|
| 53 |
</div> |
|
| 54 |
|
|
| 35 | 55 |
</section> |
| 36 | 56 |
</div> |
| 37 |
|
|
| 38 | 57 |
<div class="fr"> |
| 39 | 58 |
<button class="btn btn-default" ng-click="checkJenkins()" ng-disabled="working"> |
| 40 | 59 |
<i class="fa fa-wrench" ng-hide="working"></i> |
| app/assets/javascripts/foreman_pipeline/jenkins-instances/new/views/new-jenkins-instance-form.html | ||
|---|---|---|
| 15 | 15 |
name="url" |
| 16 | 16 |
ng-model="jenkinsInstance.url" |
| 17 | 17 |
type="text" |
| 18 |
tabindex="2"
|
|
| 18 |
tabindex="2" |
|
| 19 | 19 |
required/> |
| 20 | 20 |
</div> |
| 21 |
<div bst-form-group>
|
|
| 21 |
<div bst-form-group> |
|
| 22 | 22 |
<p class="alert alert-info" translate> |
| 23 | 23 |
Make sure to specify the port Jenkins master is running on. Example: http://jenkins.example.com:8080 |
| 24 |
</p>
|
|
| 24 |
</p> |
|
| 25 | 25 |
</div> |
| 26 | 26 |
|
| 27 | 27 |
<div bst-form-group label="{{ 'Jenkins Home' | translate }}">
|
| ... | ... | |
| 42 | 42 |
required/> |
| 43 | 43 |
</div> |
| 44 | 44 |
|
| 45 |
<div bst-form-group label="{{ 'Timeout' | translate }}">
|
|
| 46 |
<input id="timeout" |
|
| 47 |
name="timeout" |
|
| 48 |
ng-model="jenkinsInstance.timeout" |
|
| 49 |
type="number" |
|
| 50 |
min="1" |
|
| 51 |
tabindex="5" /> |
|
| 52 |
</div> |
|
| 53 |
|
|
| 54 |
<div bst-form-group> |
|
| 55 |
<p class="alert alert-info" translate> |
|
| 56 |
Time in minutes we are willing to wait before build on Jenkins starts. |
|
| 57 |
</p> |
|
| 58 |
</div> |
|
| 59 |
|
|
| 45 | 60 |
<div bst-form-buttons |
| 46 | 61 |
on-cancel="transitionBack()" |
| 47 | 62 |
on-save="createJenkinsInstance(jenkinsInstance)" |
| app/controllers/foreman_pipeline/api/jenkins_instances_controller.rb | ||
|---|---|---|
| 115 | 115 |
end |
| 116 | 116 |
|
| 117 | 117 |
def jenkins_instance_params |
| 118 |
params.require(:jenkins_instance).permit(:name, :url, :jenkins_home, :cert_path) |
|
| 118 |
params.require(:jenkins_instance).permit(:name, :url, :jenkins_home, :cert_path, :timeout)
|
|
| 119 | 119 |
end |
| 120 | 120 |
end |
| 121 | 121 |
end |
| app/lib/actions/foreman_pipeline/jenkins/build.rb | ||
|---|---|---|
| 5 | 5 |
class Build < AbstractJenkinsAction |
| 6 | 6 |
|
| 7 | 7 |
def run |
| 8 |
# are we willing to wait longer than 30 mins until build starts? |
|
| 9 |
output[:build_num] = job.jenkins_instance.client.job.build(jenkins_project.name, params, 'build_start_timeout' => (30 * 60)) |
|
| 8 |
timeout = job.jenkins_instance.timeout_sec |
|
| 10 | 9 |
output[:project_name] = jenkins_project.name |
| 11 | 10 |
output[:build_params] = params |
| 11 |
output[:build_num] = job.jenkins_instance.client.job.build(jenkins_project.name, params, 'build_start_timeout' => timeout) |
|
| 12 |
rescue => e |
|
| 13 |
fail "Jenkins build for #{jenkins_project.name} failed to start in a timely manner."
|
|
| 12 | 14 |
end |
| 13 | 15 |
|
| 14 | 16 |
def params |
| ... | ... | |
| 17 | 19 |
template_binding project_params |
| 18 | 20 |
end |
| 19 | 21 |
|
| 22 |
def rescue_strategy_for_self |
|
| 23 |
Dynflow::Action::Rescue::Skip |
|
| 24 |
end |
|
| 25 |
|
|
| 20 | 26 |
def template_binding(project_params) |
| 21 | 27 |
host = input[:data][:host] |
| 22 | 28 |
activation_key = input[:data][:activation_key] |
| ... | ... | |
| 32 | 38 |
end |
| 33 | 39 |
end |
| 34 | 40 |
end |
| 35 |
end |
|
| 41 |
end |
|
| app/lib/actions/foreman_pipeline/jenkins/build_project.rb | ||
|---|---|---|
| 16 | 16 |
|
| 17 | 17 |
def run |
| 18 | 18 |
output[:status] = input[:build_status] |
| 19 |
output[:name] = input[:name] |
|
| 20 |
fail "Build on Jenkins failed for #{output[:name]}" unless output[:build_status] == "SUCCESS"
|
|
| 19 | 21 |
end |
| 20 | 22 |
|
| 21 | 23 |
def rescue_strategy_for_self |
| app/lib/actions/foreman_pipeline/jenkins/bulk_build.rb | ||
|---|---|---|
| 17 | 17 |
trigger(BuildProject, input[:opts].merge({:project_id => project.id, :project_name => project.name}))
|
| 18 | 18 |
end |
| 19 | 19 |
end |
| 20 |
|
|
| 21 |
def check_for_errors! |
|
| 22 |
# Do nothing. Just wait for all builds to finish and gather results |
|
| 23 |
end |
|
| 20 | 24 |
end |
| 21 | 25 |
end |
| 22 | 26 |
end |
| app/lib/actions/foreman_pipeline/mixins/uri_extension.rb | ||
|---|---|---|
| 6 | 6 |
require 'uri' |
| 7 | 7 |
|
| 8 | 8 |
def jenkins_hostname(job) |
| 9 |
URI(job.jenkins_instance.url).host
|
|
| 9 |
URI(job.jenkins_instance.url).host |
|
| 10 | 10 |
end |
| 11 | 11 |
end |
| 12 | 12 |
end |
| app/models/foreman_pipeline/jenkins_instance.rb | ||
|---|---|---|
| 15 | 15 |
FILEPATH_REGEX = /\A(\/|~)[a-z0-9\-_.\/]*[^\/]\z/i |
| 16 | 16 |
|
| 17 | 17 |
validates :name, :presence => true |
| 18 |
validates :cert_path, :format => {:with => FILEPATH_REGEX }
|
|
| 19 |
validates :url, :uniqueness => true, :format => { :with => /\A(http|https):\/\/\S+:\d{1,4}\z/}
|
|
| 18 |
validates :timeout, :numericality => { :only_integer => true, :greater_than_or_equal_to => 0 }
|
|
| 19 |
validates :cert_path, :format => { :with => FILEPATH_REGEX }
|
|
| 20 |
validates :url, :uniqueness => true, :format => { :with => /\A(http|https):\/\/\S+:\d{1,4}\z/ }
|
|
| 20 | 21 |
validates :organization, :presence => true |
| 21 | 22 |
validates :jenkins_home, :format => { :with => FILEPATH_REGEX }
|
| 22 | 23 |
|
| ... | ... | |
| 35 | 36 |
@client |
| 36 | 37 |
end |
| 37 | 38 |
|
| 39 |
def timeout_sec |
|
| 40 |
timeout * 60 |
|
| 41 |
end |
|
| 42 |
|
|
| 38 | 43 |
private |
| 39 | 44 |
|
| 40 | 45 |
def authenticated_client(username, password, hash_args) |
| ... | ... | |
| 47 | 52 |
return JenkinsApi::Client.new(hash_args) if username.nil? |
| 48 | 53 |
authenticated_client username, password, hash_args |
| 49 | 54 |
end |
| 50 |
|
|
| 51 | 55 |
end |
| 52 |
end |
|
| 56 |
end |
|
| app/views/foreman_pipeline/api/jenkins_instances/show.json.rabl | ||
|---|---|---|
| 1 | 1 |
object @resource |
| 2 | 2 |
|
| 3 |
attributes :id, :name, :url, :organization_id, :server_version, :jenkins_home, :cert_path |
|
| 3 |
attributes :id, :name, :url, :organization_id, :server_version, :jenkins_home, :cert_path, :timeout
|
|
| 4 | 4 |
|
| 5 | 5 |
child :jenkins_user => :jenkins_user do |
| 6 | 6 |
extends "foreman_pipeline/api/jenkins_users/show" |
| db/migrate/20160629085119_add_timeout_to_jenkins_instance.rb | ||
|---|---|---|
| 1 |
class AddTimeoutToJenkinsInstance < ActiveRecord::Migration |
|
| 2 |
def up |
|
| 3 |
add_column :foreman_pipeline_jenkins_instances, :timeout, :integer, :default => 60 |
|
| 4 |
end |
|
| 5 |
|
|
| 6 |
def down |
|
| 7 |
remove_column :foreman_pipeline_jenkins_instances, :timeout |
|
| 8 |
end |
|
| 9 |
end |
|
Also available in: Unified diff
Fixes #15329 - Add custom timeout for jenkins instance