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

Cdash reporting timeout #44213

Merged
merged 3 commits into from
May 15, 2024
Merged

Conversation

johnwparent
Copy link
Contributor

Add basic timeout to requests to CDash

Add cdash timeout everywhere
Correct mock responder api
@spackbot-app spackbot-app bot added core PR affects Spack core functionality tests General test capability(ies) labels May 15, 2024
@johnwparent
Copy link
Contributor Author

attn @alecbcs

@johnwparent johnwparent requested a review from alecbcs May 15, 2024 17:29
zackgalbreath
zackgalbreath previously approved these changes May 15, 2024
Copy link
Member

@alecbcs alecbcs left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks @johnwparent!

@alecbcs alecbcs enabled auto-merge (squash) May 15, 2024 17:51
@alecbcs alecbcs merged commit 43291aa into spack:develop May 15, 2024
33 of 35 checks passed
@johnwparent johnwparent added this to the v0.22.1 milestone May 17, 2024
@alecbcs alecbcs self-assigned this May 17, 2024
@johnwparent johnwparent mentioned this pull request Jun 3, 2024
38 tasks
alalazo pushed a commit that referenced this pull request Jun 5, 2024
* Add timeout to cdash reporter PUT request

Add cdash timeout everywhere
Correct mock responder api

* Style

* brief doc
teaguesterling pushed a commit to teaguesterling/spack that referenced this pull request Jun 15, 2024
* Add timeout to cdash reporter PUT request

Add cdash timeout everywhere
Correct mock responder api

* Style

* brief doc
haampie pushed a commit that referenced this pull request Jun 28, 2024
* Add timeout to cdash reporter PUT request

Add cdash timeout everywhere
Correct mock responder api

* Style

* brief doc
haampie pushed a commit that referenced this pull request Jul 1, 2024
* Add timeout to cdash reporter PUT request

Add cdash timeout everywhere
Correct mock responder api

* Style

* brief doc
haampie pushed a commit that referenced this pull request Jul 1, 2024
* Add timeout to cdash reporter PUT request

Add cdash timeout everywhere
Correct mock responder api

* Style

* brief doc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core PR affects Spack core functionality tests General test capability(ies)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants