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

Cirrus: Double CPUs in default VM setup #8080

Closed
wants to merge 1 commit into from

Conversation

cevich
Copy link
Member

@cevich cevich commented Oct 20, 2020

This should help all tests to execute faster.

Signed-off-by: Chris Evich cevich@redhat.com

This should help all tests to execute faster.

Signed-off-by: Chris Evich <cevich@redhat.com>
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cevich
To complete the pull request process, please assign giuseppe after the PR has been reviewed.
You can assign the PR to them by writing /assign @giuseppe in a comment when ready.

The full list of commands accepted by this bot can be found 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

@edsantiago
Copy link
Member

edsantiago commented Oct 20, 2020

Doesn't look like it makes any difference for the ten /info requests test:

ok 53 [01-basic] Time for ten /info requests (7 seconds) <= 7s

I can imagine that more CPUs could help the int tests, which run three in parallel, but it's not going to help sys or APIv2

@cevich
Copy link
Member Author

cevich commented Oct 21, 2020

I can imagine that more CPUs could help the int tests

Curiously, it doesn't seem to make much/any difference there either.

Closing this then since it would (literally) be a waste of money.

@cevich cevich closed this Oct 21, 2020
@cevich cevich deleted the more_cpu branch June 30, 2021 18:00
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants