Skip to content

Add lastCrawlExecSeconds to workflow #2612

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

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

SuaYoo
Copy link
Member

@SuaYoo SuaYoo commented May 19, 2025

Related to #2566, allows showing more accurate run duration in workflow list and details.

ikreymer and others added 12 commits May 14, 2025 10:24
- add 'pause' crawl state
- turns off crawler pods, and then redis pod when paused
- add 'paused' on crawl spec to indicate when crawl is paused
- /crawl/<id>/{un}pause apis to toggle 'paused' on crawl spec
- ui: add pause/resume button, paused state
- ui: add pausing/unpausing derivative states when crawl is running and pausing, or paused and not pausing
- Hide "Pause" button if it's not relevant, instead of disabling without
a displayed reason
- Make "Resume" button primary
- Use circular icons to match other status icons
- Show toast message on successful pause/unpause
- set <crawlid>:paused key when a crawl is paused and at least one crawler pod exists
- clear <crawlid>:paused when crawl is paused and more pods running
ensure flag is cleared before redis is shutdown, already cleared when a crawl is unpaused
- stop crawls that have been paused for too long
- add 'paused_crawl_limit_minutes' to Helm chart
- add paused time and expiry to crawlconfig API response
- set to 'stopped_pause_expired' state
- ui: add support for 'Stopped: Paused Too Long' for stopped_pause_expired
- use 'paused_at' in CrawlJob to indicate crawl is paused and when
…vious versions of crawler:

- set :stopping key to true
- when crawler pod exits, immediately reset done -> interrupted
- clear :stopping key when all crawler pods have exited OR crawl not longer paused (to allow resume)
@ikreymer ikreymer force-pushed the pause-resume branch 2 times, most recently from 971531e to d14945f Compare May 21, 2025 16:09
Base automatically changed from pause-resume to main May 21, 2025 21:05
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.

3 participants