Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add keystone configurable API Timeout #1223

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Add keystone configurable API Timeout #1223

merged 1 commit into from
Jan 21, 2025
+8 −0

Conversation

Deydra71
Copy link
Contributor

@Deydra71 Deydra71 commented Dec 6, 2024 •
edited
Loading

@Deydra71 Deydra71 requested review from abays and stuggi December 6, 2024 13:00
@openshift-ci openshift-ci bot requested a review from viroel December 6, 2024 13:00
@Deydra71
Copy link
Contributor Author

Deydra71 commented Jan 15, 2025 •
edited
Loading

/test functional
/test precommit-check
/test openstack-operator-build-deploy-kuttl

@Deydra71
Copy link
Contributor Author

/test all

@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

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/3503146e5aec487cb8d7b13386ec03f4

❌ openstack-k8s-operators-content-provider FAILURE in 11m 03s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ openstack-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@Deydra71 Deydra71 force-pushed the keystone-httpd-config branch from 1ca7913 to ee746c3 Compare January 15, 2025 13:55
@softwarefactory-project-zuul softwarefactory-project-zuul
Copy link

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/24ebb9c9c7d44b6ca3a152009541eeb6

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 04m 15s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 24m 38s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 54m 32s
❌ openstack-operator-tempest-multinode FAILURE in 1h 49m 17s

@Deydra71
Copy link
Contributor Author

recheck

stuggi
stuggi approved these changes Jan 21, 2025
View reviewed changes
Copy link
Contributor

@stuggi stuggi left a comment

Choose a reason for hiding this comment

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

/lgtm

@Deydra71
Add keystone configurable API Timeout
5f4253a 
Depends-On: openstack-k8s-operators/keystone-operator#515

Signed-off-by: Veronika Fisarova <vfisarov@redhat.com>
@Deydra71 Deydra71 force-pushed the keystone-httpd-config branch from ee746c3 to 5f4253a Compare January 21, 2025 09:53
@openshift-ci openshift-ci bot removed the lgtm label Jan 21, 2025
olliewalsh
olliewalsh approved these changes Jan 21, 2025
View reviewed changes
Copy link
Contributor

@olliewalsh olliewalsh 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 Jan 21, 2025
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 21, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Deydra71, olliewalsh, stuggi

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

The pull request process is described here

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 675f136 into openstack-k8s-operators:main Jan 21, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stuggi stuggi stuggi approved these changes

@olliewalsh olliewalsh olliewalsh approved these changes

@abays abays Awaiting requested review from abays

@viroel viroel Awaiting requested review from viroel

Assignees

@olliewalsh olliewalsh

@stuggi stuggi

Labels
approved lgtm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Deydra71 @olliewalsh @stuggi