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

Updated grover config #1932

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Updated grover config #1932

wants to merge 8 commits into from

Conversation

gowsik-ragunath
Copy link
Contributor

@gowsik-ragunath gowsik-ragunath commented Nov 7, 2024

  • Added swap to worker machine this will fix the memory issue.

  • Grover config changes:

    • prefer_css_page_size set to false as we don't have any page css property.
    • Set timeout to 30 seconds. This will avoid Grover to waiting indefinitively which will also increase the memory usage and cause the server to restart.
    • Added --disable-gpu as launch_args to disable the GPU acceleration.
    • set javascript_enabled: false as the PDF pages are static.
  • Removed networkidle0 and networkidle2 as load should be enough to get the images from s3 and it will reduce the 500 millisecond wait.

  • Fixed the address issue in organization edit form.

@gowsik-ragunath gowsik-ragunath changed the title [WIP] Updated grover config Updated grover config Nov 8, 2024
@gowsik-ragunath gowsik-ragunath marked this pull request as ready for review November 8, 2024 07:44
Copy link
Contributor

@keshavbiswa keshavbiswa left a comment

Choose a reason for hiding this comment

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

LGTM! Haven't tested it yet. Can @apoorv1316 test this on staging and confirm it's working as expected?

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.

2 participants