From 4efcce53103b3ea7419c88fabd8b9ef12b79ca8f Mon Sep 17 00:00:00 2001 From: Julia Kreger Date: Thu, 24 Aug 2023 08:29:07 -0700 Subject: [PATCH] Extend the lookup timeout to 600 seconds Changes the default lookup timeout to be 600 seconds which reduces the risk of lookup failing as a write operation to the backing database is performed upon lookup thanks to generation of an agent token. Overall, this is fairly harmless since by default ramdisks restart the agent if they were not able to successfully start. Change-Id: I35c64c0b4f9b3b607df1bc0c4c2a852aa3595cbd --- ironic_python_agent/config.py | 2 +- .../extend-ipa-lookup-timeout-05ee5b1372792dbe.yaml | 9 +++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/extend-ipa-lookup-timeout-05ee5b1372792dbe.yaml diff --git a/ironic_python_agent/config.py b/ironic_python_agent/config.py index 1c4359c2a..260c9fa3f 100644 --- a/ironic_python_agent/config.py +++ b/ironic_python_agent/config.py @@ -121,7 +121,7 @@ cli_opts = [ cfg.IntOpt('lookup_timeout', min=0, - default=int(APARAMS.get('ipa-lookup-timeout', 300)), + default=int(APARAMS.get('ipa-lookup-timeout', 600)), help='The amount of time to retry the initial lookup ' 'call to Ironic. After the timeout, the agent ' 'will exit with a non-zero exit code. ' diff --git a/releasenotes/notes/extend-ipa-lookup-timeout-05ee5b1372792dbe.yaml b/releasenotes/notes/extend-ipa-lookup-timeout-05ee5b1372792dbe.yaml new file mode 100644 index 000000000..51df9c958 --- /dev/null +++ b/releasenotes/notes/extend-ipa-lookup-timeout-05ee5b1372792dbe.yaml @@ -0,0 +1,9 @@ +--- +fixes: + - | + The default timeout value for the agent to lookup itself in an Ironic + deployment has been extended to 600 seconds from 300 seconds. This is + to provide better stability for Ironic deployments under heavy load + which may be unable to service new requests. This is particularly true + when the backing database is SQLite for Ironic due to the limited write + concurrency of the database.