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.

Sign up for GitHub

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

Jump to bottom

feat: sync check statuses every 30 seconds #571

Merged
merged 19 commits into from
Oct 4, 2023
Merged

feat: sync check statuses every 30 seconds #571

merged 19 commits into from
Oct 4, 2023
+481 −873

Conversation

adityathebe
Copy link
Member

@adityathebe adityathebe commented Sep 22, 2023 •
edited
Loading

Resolves: #541

  • Remove real-time syncing of check statuses to upstream
  • Add a periodic job that pushes check statuses to upstream
  • Remove status endpoints for config-db agents.
  • Use time based config-scraper pulling (Need to fix this)
  • support env vars for upstream config

Merge after:

@adityathebe adityathebe force-pushed the feat/check_statuses_periodic_sync branch from 31c42b0 to a8a3804 Compare September 27, 2023 15:15
@adityathebe adityathebe marked this pull request as draft September 27, 2023 16:35
@adityathebe adityathebe force-pushed the feat/check_statuses_periodic_sync branch from 7a9e21f to d173d59 Compare September 28, 2023 04:26
@adityathebe adityathebe marked this pull request as ready for review September 28, 2023 04:26
@adityathebe adityathebe force-pushed the feat/check_statuses_periodic_sync branch from 26fcc4c to 928910f Compare September 28, 2023 05:22
moshloop
moshloop reviewed Sep 28, 2023
View reviewed changes
jobs/upstream.go Outdated Show resolved Hide resolved
moshloop
moshloop reviewed Sep 28, 2023
View reviewed changes
jobs/upstream.go Outdated Show resolved Hide resolved
adityathebe
adityathebe commented Sep 29, 2023
View reviewed changes
events/adapters.go Outdated Show resolved Hide resolved
@adityathebe adityathebe force-pushed the feat/check_statuses_periodic_sync branch from 6fdb953 to ca3edfb Compare October 2, 2023 09:46
@adityathebe
fix: failing tests
ae9bbeb 
* bump postq
* set default logger for pg consumer

[skip ci]
@adityathebe
chore: bump duty
d3c6fca
moshloop
moshloop reviewed Oct 4, 2023
View reviewed changes
cmd/root.go Outdated Show resolved Hide resolved
@moshloop moshloop merged commit ccadb4f into main Oct 4, 2023
5 checks passed
@moshloop moshloop deleted the feat/check_statuses_periodic_sync branch October 4, 2023 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moshloop moshloop moshloop left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Job history improvements
2 participants
@adityathebe @moshloop