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

Refactor jobstats pkg #18

Merged
merged 9 commits into from
Dec 26, 2023
Merged

Refactor jobstats pkg #18

merged 9 commits into from
Dec 26, 2023

Conversation

mahendrapaipuri
Copy link
Owner

@mahendrapaipuri mahendrapaipuri commented Dec 26, 2023

  • Split jobstats pkg into logical sub pkgs
  • Adopt standard go project layout
  • Use a coherent style for creating stats server and DB
  • Upgrade RTE emission source to use v2 API
  • Misc fixes and improvements

* Move configs from etc to configs folder

* Move systemd unit files to init folder

Signed-off-by: Mahendra Paipuri <[email protected]>
* Use a sub pkg for each component db, server, cli, etc

* Rename interface Batch to Fetcher inline with go recommendations

* Define and use a config struct for DB to be coherent with Server

* Improvements in naming of structs and fields

* Use base.go instead of types.go for definining base structs

Signed-off-by: Mahendra Paipuri <[email protected]>
* We dont want one source to block entire scrape.

* Timeout used in exporter should be smaller than prometheus scrape timeout

Signed-off-by: Mahendra Paipuri <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>
@mahendrapaipuri mahendrapaipuri merged commit 887e31a into main Dec 26, 2023
5 checks passed
@mahendrapaipuri mahendrapaipuri deleted the refactor_jobstats branch December 26, 2023 14:13
@mahendrapaipuri mahendrapaipuri added enhancement New feature or request maintenance General maintenance labels Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request maintenance General maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant