Skip to content

Updating Memory and CPU Usage Error Bounds for CI Tests#921

Open
SabrinaBrickner wants to merge 1 commit intodevelopfrom
ci_memory_updates
Open

Updating Memory and CPU Usage Error Bounds for CI Tests#921
SabrinaBrickner wants to merge 1 commit intodevelopfrom
ci_memory_updates

Conversation

@SabrinaBrickner
Copy link

Description

Please provide a detailed description of the changes this pull request introduces.

$${\color{blue}!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!}$$

Updates the bounds on memory and CPU usage errors within the CI system to reduce the number of unnecessary memory based errors in the CI.

$${\color{blue}!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!}$$

Checklist

  • [ x] Added at least 1 label from available labels.
  • [x ] Assigned at least 1 reviewer under Reviewers,
  • [ x] Assigned all contributers including yourself under Assignees
  • [x ] Linked any relevant issues under Developement
  • Does this PR affect CAF data format? If so, please assign a CAF maintainer (PetrilloAtWork or JosiePaton) as additional reviewer.
  • Does this affect the standard workflow?
  • Is this PR a patch for the ongoing production? If so, separate PR must also be made for production/v10_06_00 branch!

Relevant PR links (optional)

Does this PR require merging another PR in a different repository (such as sbnanobj/sbnobj etc.)?

Link(s) to docdb describing changes (optional)

Is there a docdb describing the issue this solves or the feature added?

Copy link
Contributor

@kjplows kjplows left a comment

Choose a reason for hiding this comment

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

Thanks! Could round up/down the limits perhaps...
I wonder what causes the usage to fluctuate down...

@nathanielerowe nathanielerowe mentioned this pull request Mar 9, 2026
@nathanielerowe
Copy link
Contributor

Would it be possible to also change to fcls to match the ones used in production? This was a request I was getting from Linyan. There was an open PR that did this, but closed it in favor of this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants