Skip to content

Conversation

Markoutte
Copy link
Collaborator

Description

There's a lot of OutOfMemoryError in logs. We added this limit only with the latest fix, so, it can be removed, I think.

Self-check list

  • I've set the proper labels for my PR (at least, for category and component).
  • PR title and description are clear and intelligible.
  • I've added enough comments to my code, particularly in hard-to-understand areas.
  • The functionality I've repaired, changed or added is covered with automated tests.
  • Manual tests have been provided optionally.
  • The documentation for the functionality I've been working on is up-to-date.

@Markoutte Markoutte added comp-infrastructure Infrastructure issues comp-github-action GitHub action, gradle plugin or maven plugin ctg-bug-fix PR is fixing a bug labels Aug 10, 2023
@Markoutte Markoutte requested a review from bissquit August 10, 2023 04:29
Copy link
Member

@bissquit bissquit left a comment

Choose a reason for hiding this comment

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

LGTM

@Markoutte Markoutte merged commit 5089489 into main Aug 10, 2023
@Markoutte Markoutte deleted the pelevin/remove-memory-limit-for-night-monitoring branch August 10, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-github-action GitHub action, gradle plugin or maven plugin comp-infrastructure Infrastructure issues ctg-bug-fix PR is fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants