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

[Feature]: Profile resources + loading improvements #1598

Closed
ikreymer opened this issue Mar 15, 2024 · 0 comments · Fixed by #1604
Closed

[Feature]: Profile resources + loading improvements #1598

ikreymer opened this issue Mar 15, 2024 · 0 comments · Fixed by #1604
Assignees
Labels
enhancement New feature or request

Comments

@ikreymer
Copy link
Member

Context

Currently, profile browser pods have same resource requirements as crawlers. However, profiles only use one tab, while crawlers may have multiple worker tabs. This is slightly wasteful, instead profiles should use equivalent resources to one browser, and also allow custom profile resource overrides, if needed.

Additionally, loading a profile sometimes shows the nginx error page, due to some race condition. To avoid the, frontend can instead retry until the profile page from the container is reached.

What change would you like to see?

  • Improved resource usage for profiles
  • Retry if profile page doesn't load

Requirements

No response

Todo

No response

@ikreymer ikreymer added the enhancement New feature or request label Mar 15, 2024
@ikreymer ikreymer self-assigned this Mar 15, 2024
ikreymer added a commit that referenced this issue Mar 15, 2024
- default profile browser resources to either 'profile_browser_cpu' / 'profile_browser_memory' or single browser 'crawler_memory_base' / 'crawler_cpu_base',
instead of scaled to the number of browser workers
- frontend: check that profile html page is loading, retry if getting nginx error
- fixes #1598
ikreymer added a commit that referenced this issue Mar 15, 2024
- default profile browser resources to either 'profile_browser_cpu' / 'profile_browser_memory' or single browser 'crawler_memory_base' / 'crawler_cpu_base',
instead of scaled to the number of browser workers
- frontend: check that profile html page is loading, retry if getting nginx error
- fixes #1598
ikreymer added a commit that referenced this issue Mar 15, 2024
- Backend: Use separate resource constraints for profiles: default
profile browser resources to either 'profile_browser_cpu' /
'profile_browser_memory' or single browser 'crawler_memory_base' /
'crawler_cpu_base', instead of scaled to the number of browser workers

- Frontend: check that profile html page is loading, keep retrying if
still getting nginx error instead of loading an iframe with the error.

- Fixes #1598
(Prepared for 1.9.4 release)
ikreymer added a commit that referenced this issue Mar 16, 2024
- Backend: Use separate resource constraints for profiles: default
profile browser resources to either 'profile_browser_cpu' /
'profile_browser_memory' or single browser 'crawler_memory_base' /
'crawler_cpu_base', instead of scaled to the number of browser workers

- Frontend: check that profile html page is loading, keep retrying if
still getting nginx error instead of loading an iframe with the error.

Fixes #1598 (Copy of #1599 from 1.9.4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done!
Development

Successfully merging a pull request may close this issue.

1 participant