Skip to content

Add configurable API Timeouts <JIRA: OSPRH-10958>#510

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
mumesan:httpd_timeout_configurable
Feb 17, 2025
Merged

Add configurable API Timeouts <JIRA: OSPRH-10958>#510
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
mumesan:httpd_timeout_configurable

Conversation

@mumesan
Copy link
Copy Markdown
Contributor

@mumesan mumesan commented Jan 15, 2025 •
edited by hjensas
Loading

  • Added apiTimeout property to ironic api with default set to 60 seconds

Jira: OSPRH-10958

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Jan 15, 2025

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/eda06e39d32f415d934e1c76bd6e9fc4

✔️ noop SUCCESS in 0s
❌ openstack-k8s-operators-content-provider FAILURE in 11m 42s
⚠️ podified-multinode-ironic-deployment SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@mumesan mumesan force-pushed the httpd_timeout_configurable branch from 7b34f61 to 5e157a9 Compare January 15, 2025 15:51
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Jan 15, 2025

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0d5e275f442340ecbcf2b760bf28eaa9

✔️ noop SUCCESS in 0s
❌ openstack-k8s-operators-content-provider FAILURE in 9m 35s
⚠️ podified-multinode-ironic-deployment SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@mumesan mumesan force-pushed the httpd_timeout_configurable branch from 5e157a9 to 484955d Compare January 17, 2025 17:02
hjensas
hjensas reviewed Jan 18, 2025
View reviewed changes
hjensas
hjensas reviewed Jan 18, 2025
View reviewed changes
@hjensas
Copy link
Copy Markdown
Contributor

hjensas commented Jan 20, 2025

Putting this on hold until CIX : 638 is fixed.

@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 21, 2025

recheck with depends-on openstack-k8s-operators/ci-framework#2671

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Jan 21, 2025

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/92f610c402254ad994d32ecfc280b3fe

✔️ noop SUCCESS in 0s
✔️ openstack-k8s-operators-content-provider SUCCESS in 48m 41s
❌ podified-multinode-ironic-deployment FAILURE in 31m 08s

@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 21, 2025

recheck trying with openstack-k8s-operators/ci-framework#2672

@raukadah raukadah mentioned this pull request Jan 21, 2025
Closed
@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 23, 2025

recheck after ci-framework patch update

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Jan 23, 2025

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#2672 is needed.

@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 23, 2025

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul bot commented Jan 23, 2025

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/71e8f7a955e347fd8c82b9ec9e2f418c

✔️ noop SUCCESS in 0s
✔️ openstack-k8s-operators-content-provider SUCCESS in 47m 35s
❌ podified-multinode-ironic-deployment FAILURE in 29m 41s

@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 23, 2025

recheck

@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Jan 28, 2025

recheck after ci-framework patch update

@hjensas
Copy link
Copy Markdown
Contributor

hjensas commented Jan 29, 2025

recheck

@mumesan mumesan force-pushed the httpd_timeout_configurable branch from 484955d to e7927bc Compare January 30, 2025 12:04
hjensas
hjensas requested changes Jan 31, 2025
View reviewed changes
@mumesan mumesan force-pushed the httpd_timeout_configurable branch from e7927bc to 235a50d Compare January 31, 2025 12:26
@mumesan
Copy link
Copy Markdown
Contributor Author

mumesan commented Jan 31, 2025

/retest

hjensas
hjensas reviewed Feb 4, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@hjensas hjensas left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Feb 4, 2025
@openshift-ci openshift-ci bot added the approved label Feb 4, 2025
@hjensas hjensas requested a review from steveb February 4, 2025 19:29
@openshift-ci openshift-ci bot removed the lgtm label Feb 14, 2025
@mumesan mumesan force-pushed the httpd_timeout_configurable branch from 3f7fbbd to 131ed8b Compare February 14, 2025 15:50
@openshift-ci openshift-ci bot added the lgtm label Feb 17, 2025
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Feb 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hjensas, mumesan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 5bf975e into openstack-k8s-operators:main Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hjensas hjensas hjensas approved these changes

@frenzyfriday frenzyfriday Awaiting requested review from frenzyfriday

@steveb steveb Awaiting requested review from steveb

Assignees

@hjensas hjensas

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mumesan @hjensas @raukadah