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

chore: right size some circleci jobs #29448

Merged
merged 4 commits into from
May 6, 2024

Conversation

jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Apr 30, 2024

Additional details

Updating our Circle job resources to be better right sized. Some were using 100% resources, so those have been bumped up. Some were not using near 100% resources, so those have been bumped down.

The timing of the run is the same as before, but I would expect to have less situations of flake that would be caused by a situation where resources are strained.

Increasing resource sizes does increase cost.

How has the user experience changed?

N/A, but for internal:

Before

Screenshot 2024-05-01 at 1 32 30 PM

After

Screenshot 2024-05-01 at 10 02 47 AM

Before

Screenshot 2024-05-01 at 1 35 41 PM

After

Screenshot 2024-05-01 at 1 36 10 PM

PR Tasks

@jennifer-shehane jennifer-shehane self-assigned this Apr 30, 2024
Copy link

cypress bot commented Apr 30, 2024

1 flaky test on run #55311 ↗︎

0 6036 142 0 Flakiness 1

Details:

Merge branch 'develop' into rightsize-jobs-resources
Project: cypress Commit: 242f9d7203
Status: Passed Duration: 14:26 💡
Started: May 3, 2024 11:57 AM Ended: May 3, 2024 12:11 PM
Flakiness  cypress/e2e/e2e/origin/cookie_login.cy.ts • 1 flaky test • 5x-driver-firefox

View Output

Test Artifacts
... > past Max-Age, before Expires -> not logged in
    </td>
  </tr></table>

Review all test suite changes for PR #29448 ↗︎

@jennifer-shehane jennifer-shehane merged commit 84b6bf2 into develop May 6, 2024
80 of 82 checks passed
@jennifer-shehane jennifer-shehane deleted the rightsize-jobs-resources branch May 6, 2024 20:55
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.

None yet

3 participants