Skip to content

feat: generate estimated finish time#535

Merged
deryrahman merged 26 commits intomainfrom
feat-estimated-finish-time-calculation
Feb 18, 2026
Merged

feat: generate estimated finish time#535
deryrahman merged 26 commits intomainfrom
feat-estimated-finish-time-calculation

Conversation

@deryrahman
Copy link
Copy Markdown
Member

@deryrahman deryrahman commented Feb 9, 2026

@deryrahman deryrahman marked this pull request as draft February 9, 2026 08:06
@deryrahman deryrahman changed the title feat: initial implementation of estimated finish time feat: generate estimated finish time Feb 9, 2026
@deryrahman deryrahman marked this pull request as ready for review February 10, 2026 07:14
@deryrahman deryrahman force-pushed the feat-estimated-finish-time-calculation branch from ef3c161 to e863789 Compare February 12, 2026 08:08
Comment thread core/scheduler/service/job_expectator_service.go Outdated
Comment thread core/scheduler/handler/v1beta1/job_run.go Outdated
Comment thread core/scheduler/service/job_expectator_service.go Outdated
Comment thread core/scheduler/service/job_expectator_service.go
Comment thread core/scheduler/service/job_expectator_service.go Outdated
Comment thread core/scheduler/service/job_expectator_service.go
Comment thread core/scheduler/service/job_expectator_service.go Outdated
Comment thread core/scheduler/handler/v1beta1/job_run.go Outdated
Comment thread server/optimus.go Outdated
@deryrahman deryrahman force-pushed the feat-estimated-finish-time-calculation branch from 6f64e63 to 7193fc9 Compare February 18, 2026 07:25
@deryrahman deryrahman merged commit 0a09f65 into main Feb 18, 2026
3 checks passed
@deryrahman deryrahman deleted the feat-estimated-finish-time-calculation branch February 18, 2026 08:29
deryrahman added a commit that referenced this pull request Feb 19, 2026
* feat: initial implementation of estimated finish time

* refactor: use existing function from sla predictor service

* feat: store estimated duration to db

* fix: lint

* feat: update proto

* feat: add handler to generate estimated finish time

* feat: integrate job finish time estimator service

* feat: safety check for nil variables

* test: add test cases for job estimator finish time

* test: add test cases for generate estimation on handler

* fix: linter

* fix: test case

* feat: skip disabled job

* refactor: estimated -> estimator + enrich api response

* refactor: remove unecessary vars

* feat: create table if not exist

* refactor: update the log using fmt.Sprintf

* feat: when job is not started yet use buffer time as well

* feat: when job doesn't have sufficient run to estimate duration

* feat: when job already finished, it should be checked first

* refactor: adjust the calculation + fix test cases

* feat: use different config for job expectator detail

* feat: integrate buffer duration config

* feat: use different key to distinguished actual finish time

* fix: new server config

* feat: update proto commit
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