From 46a552c8917c05c0bc59417a9df54d3cae7eae8c Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Thu, 22 Sep 2016 18:32:36 +0000 Subject: [PATCH] Test timeout extension for inspection The default introspection time for shade may be too short for some environments. As such, extending the timeout to be longer to provide nodes an opportunity to boot, check-in, and reply with results. The library default, which I believe is 900 seconds, was also based on basic tests where I determined that _most_ systems should check-in with-in based on: - 300 seconds to POST - 300 seconds to PXE - 300 seconds to Boot and start IPA. Change-Id: If17c9d7c6acc19ec05a33fa57cb4bbbef1fa43d0 --- playbooks/roles/ironic-inspect-node/README.md | 11 +++++++++++ playbooks/roles/ironic-inspect-node/defaults/main.yml | 1 + playbooks/roles/ironic-inspect-node/tasks/main.yml | 1 + ...rospection-timeout-parameter-8e86afffd2f1d947.yaml | 6 ++++++ 4 files changed, 19 insertions(+) create mode 100644 releasenotes/notes/introspection-timeout-parameter-8e86afffd2f1d947.yaml diff --git a/playbooks/roles/ironic-inspect-node/README.md b/playbooks/roles/ironic-inspect-node/README.md index a1bfa5a84..f33bd66c1 100644 --- a/playbooks/roles/ironic-inspect-node/README.md +++ b/playbooks/roles/ironic-inspect-node/README.md @@ -27,6 +27,17 @@ noauth_mode: Controls if the module is called in noauth mode. cloud_name: Optional: String value defining a clouds.yaml entry for the ansible module to leverage. +inspection_wait_timeout: Integer value in seconds, defaults to 1800. + This value may need to be adjusted if the underlying + shade library's default timeout is insufficent for + a node to perform an inspection sequence with. + The timeout assumption in the library was + based upon there being three phases to complete + an inspection sequence, BIOS POST, (i)PXE, + and then booting of the ramdisk and IPA. + In most cases, each phase should be completed + under 300 seconds, although that will vary based + upon the hardware configuration. Dependencies ------------ diff --git a/playbooks/roles/ironic-inspect-node/defaults/main.yml b/playbooks/roles/ironic-inspect-node/defaults/main.yml index 7e350926e..88ffdfd0f 100644 --- a/playbooks/roles/ironic-inspect-node/defaults/main.yml +++ b/playbooks/roles/ironic-inspect-node/defaults/main.yml @@ -1,3 +1,4 @@ --- # defaults file for ironic-inspect-node noauth_mode: true +inspection_wait_timeout: 1800 diff --git a/playbooks/roles/ironic-inspect-node/tasks/main.yml b/playbooks/roles/ironic-inspect-node/tasks/main.yml index b0a2f572c..038acdb27 100644 --- a/playbooks/roles/ironic-inspect-node/tasks/main.yml +++ b/playbooks/roles/ironic-inspect-node/tasks/main.yml @@ -39,4 +39,5 @@ auth: "{{ auth | default(omit) }}" ironic_url: "{{ ironic_url | default(omit) }}" uuid: "{{ uuid }}" + timeout: "{{ inspection_wait_timeout }}" delegate_to: localhost diff --git a/releasenotes/notes/introspection-timeout-parameter-8e86afffd2f1d947.yaml b/releasenotes/notes/introspection-timeout-parameter-8e86afffd2f1d947.yaml new file mode 100644 index 000000000..5dd637ed2 --- /dev/null +++ b/releasenotes/notes/introspection-timeout-parameter-8e86afffd2f1d947.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - Some users have encountered issues with introspection + periodically timing out for systems. As a result, we have + added a new parameter ``inspection_wait_timeout`` that is + now defaulted to ``1800`` seconds.