Bug #3598

Make timeouts configurable

Added by Greg Sutcliffe almost 4 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Tomáš Strachota
Category:Foreman commands
Target version:Foreman - Sprint 21
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-

Description

Creating Hosts can be slow, particularly for image-based Compute Resources, which block until the image is up and the Finish template has been run. This leads to a timeout from Hammer:

[greg:~/github/hammer-cli]$ bundle exec bin/hammer host create --name test1 --hostgroup-id 11 --compute-resource-id 9 --compute-attributes "flavor_ref=1,image_ref=9c9d4946-3e33-4f6e-92f0-f3527ef42862,network=public"  
Could not create the host:
Error: Request Timeout

The host is actually created fine:

Body: {"host":{"name":"test1.elysium.emeraldreverie.org","id":3871,"ip":"172.20.10.156","environment_id":6, ... <snip> }}
Completed 200 OK in 117643ms (Views: 54.6ms | ActiveRecord: 2454.5ms)

It would be nice to be able to increase the timeout for these actions


Related issues

Related to Foreman - Feature #3566: expose Orchestration tasks via the API Closed 11/04/2013
Blocked by Hammer CLI - Feature #3897: Replace foreman_api with dynamic bindings Closed 12/17/2013

Associated revisions

Revision dfac02f1
Added by Tomas Strachota over 3 years ago

Refs #3598 - bumped apipie-bindings version requirement

apipie-bindings < 0.0.6 didn't support setting timeouts

Revision 01c89046
Added by Tomas Strachota over 3 years ago

Fixes #3598 - request timeout

Revision e29592d8
Added by mbacovsky over 3 years ago

Merge pull request #110 from tstrachota/headers

Fixes #3598 #4476 - setting locale and timeout of api requests

Revision 0a5a5e31
Added by mbacovsky over 3 years ago

Merge pull request #98 from tstrachota/headers

Refs #3598 - bumped apipie-bindings version requirement

History

#1 Updated by Greg Sutcliffe almost 4 years ago

  • Related to Feature #3566: expose Orchestration tasks via the API added

#2 Updated by Martin Bacovsky over 3 years ago

  • Blocked by Feature #3897: Replace foreman_api with dynamic bindings added

#3 Updated by Martin Bacovsky over 3 years ago

  • Assigned To set to Martin Bacovsky

This will be possible to set with dynamic bindings

#4 Updated by Dominic Cleal over 3 years ago

  • Target version set to Sprint 20

#5 Updated by Sam Kottler over 3 years ago

  • Status changed from New to Assigned
  • Assigned To changed from Martin Bacovsky to Sam Kottler

#6 Updated by Sam Kottler over 3 years ago

This is the first step toward making this happen: https://github.com/theforeman/foreman_api/pull/25.

#7 Updated by Dominic Cleal over 3 years ago

  • Status changed from Assigned to New
  • Assigned To deleted (Sam Kottler)

Above PR still stands, but we'll somebody else to pick this up for completion.

#8 Updated by Dmitri Dolguikh over 3 years ago

  • Target version changed from Sprint 20 to Sprint 21

#9 Updated by Tomáš Strachota over 3 years ago

  • Status changed from New to Assigned
  • Assigned To set to Tomáš Strachota

#10 Updated by Tomáš Strachota over 3 years ago

  • Status changed from Assigned to Ready For Testing

#11 Updated by Anonymous over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Applied in changeset foreman:hammer-cli-foreman|commit:01c89046763ebbe4bbe9435174c1ba938f90d24e.

Also available in: Atom PDF