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

chore: clean up and decouple API Server bootstrapping #5949

Merged
merged 55 commits into from
Oct 22, 2024
Merged

chore: clean up and decouple API Server bootstrapping #5949

merged 55 commits into from
Oct 22, 2024

Conversation

rangoo94
Copy link
Member

@rangoo94 rangoo94 commented Oct 18, 2024 •
edited
Loading

Pull request description

Checklist (choose whats happened)

  • breaking change! (describe)
  • tested locally
  • tested on cluster
  • added new dependencies
  • updated the docs
  • added a test

Breaking changes

Changes

Fixes

@rangoo94
chore: delete very old migrations
21919ed
@rangoo94
fixup
a23890c
@rangoo94
fixup delete migrations
4ac164f
@rangoo94
chore: first batch of simplifying API Server bootstrap
b12898e
@rangoo94
chore: continue simplifying API Server bootstrap
6bfb9de
@rangoo94
fixup delete migrations
8f8ba3e
@rangoo94
chore: delete unused cronjob package
49eff74
@rangoo94
fix: List()
7f29f26
@rangoo94
chore: continue clean up
cff3f6b
@rangoo94
chore: continue clean up
36c0e09
@rangoo94
chore: reorder a bit
af23da1
@rangoo94
fixup
ea63627
@rangoo94
chore: rename tests/test suites clients to omit version
d4d4e3a
@rangoo94
chore: wrap deprecated clients and repositories in more recognizable …
9526e14 
…structures
@rangoo94
chore: extract connecting to Minio and MongoDB
29e7e49
@rangoo94
chore: distribute DeprecatedRepositories and DeprecatedClients as a w…
ad05b4e 
…hole
@rangoo94
chore: extract event handlers initialization outside of API
467ea6a
@rangoo94
fixup
8d6c8fe
@rangoo94
fixup
4b98f0d
@rangoo94
fixup
605db50
@rangoo94
fixup delete unused
2b88b2f
@rangoo94
chore: delete unused code
ae9170c
@rangoo94
chore: extract function for streaming Test Workflow logs via Agent ou…
e92c0e9 
…tside of API
@rangoo94
chore: extract ExecutionWorker factory
56918f2
@rangoo94
fix: typo in error message
bbdfcc5
@rangoo94
chore: decouple synchronising default executors out of internal images
9de3056
@rangoo94
fixup
15cf87f
@rangoo94
chore: use client for TestTriggers
ecad5a7
@rangoo94
chore: extract building API's oauth/storage params outside of API
716e567
@rangoo94
fixup
cbd6d11
@rangoo94
chore: move httpConfig to be argument of NewTestkubeAPI
c6738d9
@rangoo94
chore: delete unused parts of server.Config
f52813a
@rangoo94
chore: extract HTTPServer out of TestkubeAPI
28522d4
@rangoo94
chore: delete proxying / on cluster to dashboard
d65c63d
@rangoo94
chore: split Testkube API to smaller modules
4e923fa
@rangoo94
fixup
20360dd
@rangoo94
chore: clean up API dependencies
88f2ddb
@rangoo94
chore: clean up DeprecatedAPI dependencies
ff5ce75
@rangoo94
chore: delete unused HTTP config computation
f91c315
@rangoo94
chore: reorder code a bit
a5be472
@rangoo94
chore: use interface for WebhooksClient
2acb5ac
@rangoo94
fix: server telemetry + telemetry heartbeat
052eada
@rangoo94
chore: reorder API bootstrap a bit
a1c7c1f
@rangoo94
chore: extract telemetry heartbeat to separate function
7914241
@rangoo94
chore: allow running regular API with no deprecated clients
4bac524
@rangoo94
chore: simplify SubscriptionChecker, include migrations in DB factory
ed21707
@rangoo94
chore: delete unnecessary gRPC connection close
8d95976
@rangoo94
chore: pass context to ProContext factory
e5534a1
@rangoo94
chore: extract telemetry heartbeat handler
f84453f
@rangoo94
chore: extract utility to read default executors
46bb1b1
@rangoo94
chore: extract utility to read ProContext
34d7103
@rangoo94
chore: extract utility to create Slack Loader and connect to NATS
eaf2fb4
@rangoo94
fixup linting
6413126
@rangoo94 rangoo94 marked this pull request as ready for review October 21, 2024 08:41
@rangoo94 rangoo94 requested a review from a team as a code owner October 21, 2024 08:41
vsukhin
vsukhin approved these changes Oct 21, 2024
View reviewed changes
Copy link
Collaborator

@vsukhin vsukhin left a comment

Choose a reason for hiding this comment

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

lgtm

@rangoo94 rangoo94 merged commit 068d044 into main Oct 22, 2024
7 checks passed
@rangoo94 rangoo94 deleted the dawid/feat/decouple-api-bootstrap branch October 22, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vsukhin vsukhin vsukhin approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@rangoo94 @vsukhin