Skip to content

Retry create_additional_port_for_machines on ConnectionTimeout#1379

Open
dosaboy wants to merge 1 commit into
openstack-charmers:masterfrom
dosaboy:allow-configure-networks-retry
Open

Retry create_additional_port_for_machines on ConnectionTimeout#1379
dosaboy wants to merge 1 commit into
openstack-charmers:masterfrom
dosaboy:allow-configure-networks-retry

Conversation

@dosaboy

@dosaboy dosaboy commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

If this is called when keystone services are for whatever reason being restarted a timeout occurs and a retry generally works.

@xtrusia xtrusia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Maybe better to add a stop condition though, e.g. stop=tenacity.stop_after_attempt(10)

@dosaboy

dosaboy commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

LGTM Maybe better to add a stop condition though, e.g. stop=tenacity.stop_after_attempt(10)

ack ill add that

If this is called when keystone services are for whatever reason
being restarted a timeout occurs and a retry generally works.
@dosaboy dosaboy force-pushed the allow-configure-networks-retry branch from b807068 to ca72dd4 Compare June 12, 2026 11:26
@dosaboy

dosaboy commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

im not sure this is fixing the issue it is intended to so needs a bit more testing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants