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.

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

Increase request timeout from 5s to 30s #145

Merged
merged 3 commits into from
Aug 25, 2023

Conversation

chayleaf
Copy link

@chayleaf chayleaf commented Aug 15, 2023

Fixes #144

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify queue

@mergify
Copy link

mergify bot commented Aug 22, 2023

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge train check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@chayleaf
Copy link
Author

"Failing after 1s"...?

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

Looks a garnix issue

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify requeue

@mergify
Copy link

mergify bot commented Aug 22, 2023

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify queue

@mergify
Copy link

mergify bot commented Aug 22, 2023

queue

🛑 The pull request has been removed from the queue default

Pull request #145 has been dequeued due to failing checks or checks timeout.

You can take a look at Queue: Embarked in merge train check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify rebase

@mergify
Copy link

mergify bot commented Aug 22, 2023

rebase

✅ Branch has been successfully rebased

@Mic92 Mic92 force-pushed the increase-request-timeout branch from 87f72eb to 1fef5ae Compare August 22, 2023 18:21
@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify requeue

@mergify
Copy link

mergify bot commented Aug 22, 2023

requeue

❌ This pull request head commit has not been previously disembarked from queue.

@Mic92
Copy link
Member

Mic92 commented Aug 22, 2023

@mergify queue

@mergify
Copy link

mergify bot commented Aug 22, 2023

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge train check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

Mic92 added 2 commits August 25, 2023 06:49
Too many dependencies are broken on this platform and probably no one uses this anyway
@Mic92 Mic92 merged commit a410b93 into nix-community:master Aug 25, 2023
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.

A lot of 408s
2 participants