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

Noise reduced run start #1023

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Noise reduced run start #1023

merged 3 commits into from
Dec 17, 2024

Conversation

ArneTR
Copy link
Member

@ArneTR ArneTR commented Dec 17, 2024

In some occurences a machine_id = NULL entry could happen if the runner failed with an unexpected git repo.

This PR reworks this functionality by only saving to the DB once the init checks are all done and the machine verified.

Also adding more short circuits to remove noise when run_id not set

Greptile Summary

Reordered database operations in the runner to prevent NULL machine_id entries and reduce log noise by validating machine configuration before creating database entries.

  • Moved register_machine_id() before import_metric_providers() to validate machine configuration early
  • Relocated initialize_run() to execute after all initialization checks are complete
  • Added short-circuit returns for DB operations when run_id is not set
  • Consolidated update_and_insert_specs() functionality into initialize_run()
  • Reordered operations in run() method to ensure proper machine validation sequence

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

2 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

runner.py Show resolved Hide resolved
@ArneTR ArneTR merged commit 8620e6f into main Dec 17, 2024
3 of 4 checks passed
@ArneTR ArneTR deleted the noise-reduced-run-start branch December 17, 2024 23:37
ArneTR added a commit that referenced this pull request Dec 19, 2024
* main: (46 commits)
  Updated cloud energy
  (fix): Unselect button broke repositories view
  Noise reduced run start (#1023)
  (improvement): More strict check
  Added unselect button [skip ci] (#1022)
  (fix): tests
  Clickable Comparsion Charts + Display Missing Values (#1017)
  Bump uvicorn[standard] from 0.32.1 to 0.34.0 (#1019)
  Bump playwright/python in /docker/auxiliary-containers/gcb_playwright (#1018)
  Updated EE
  Adding not implemented error
  Kill script for GMT must use full commandline when killing reporters
  +x for tcpdump
  Tcp dump (#919)
  Hash must be decoded to understand spaces [skip ci]
  Allowing Deeplinks to specific phases [skip ci] (#1016)
  Bump python from 3.13.0-slim-bookworm to 3.13.1-slim-bookworm in /docker (#1015)
  Bump pydantic from 2.10.2 to 2.10.3 (#1010)
  Bump fastapi[standard] from 0.115.5 to 0.115.6 (#1011)
  Bump aiohttp from 3.11.9 to 3.11.10 (#1013)
  ...
ArneTR added a commit that referenced this pull request Dec 22, 2024
* main:
  pytest.skip() [skip ci]
  Adding saving for image and volume sizes (#1027)
  Added average function to badges (#1029)
  Bump psutil from 6.1.0 to 6.1.1 (#1028)
  Removing href for non actual links [skip ci] (#1026)
  Badges for CI now include carbon and also Totals (#998)
  Removes dead link
  Bump aiohttp from 3.11.10 to 3.11.11 (#1025)
  Bump pydantic from 2.10.3 to 2.10.4 (#1024)
  Bump deepdiff from 8.0.1 to 8.1.1 (#1020)
  Updated cloud energy
  (fix): Unselect button broke repositories view
  Noise reduced run start (#1023)
  (improvement): More strict check
  Added unselect button [skip ci] (#1022)
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.

1 participant