-
Notifications
You must be signed in to change notification settings - Fork 276
Make edge api optional for OSS deployment #1638
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
Merged
Merged
Changes from 49 commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
a9ad877
Nomad nodes discovery is now using shared logic for job allocations
sitole 4e697c9
Renamed structs used for service discovery
sitole 7a22fdf
Rename structs handling instance info
sitole 9b8f562
Edge pool renaming and cleanup
sitole 32f7006
Renamed orchestrators pool to instances pool
sitole f958441
Unused feature flag removed
sitole 2c9ea21
Renamed execution id to sandbox execution id in sandbox info struct
sitole 293329f
Renamed package storing clusters
sitole ae7a01f
Introduced cluster resources provider
sitole 718f50f
Use cluster resource provider for getting sbx and build resources
sitole beeb0a4
Renamed cluster.ClusterInstance to cluster.Instance
sitole f6222f9
Cluster http client is no longer need to be exposed form cluster
sitole c205b18
Rename and rearrange clusters files
sitole 32cdfee
Renamed `clusterSynchronizationStore` to `instancesSyncStore`
sitole 819b4a0
Each instance now holds its own gRPC connection
sitole b36641f
Removed not needed metadata ctx call with instance connection
sitole 728cac8
Nomad allocations discovery with different query filters
sitole 704be35
Cluster local and remote discovery impls
sitole aa207bc
Speedup clusters sync
sitole 5c48b0a
Fixes
sitole bdf3ab5
Implemented local resource provider
sitole a931983
Client proxy (edge api) is no longer needed for integration tests pip…
sitole 6eee058
Fixed missing connection for gRPC mocks
sitole e66fbc6
Fixed not checked err for build logs
sitole f692fb8
For local and integration tests inject static orch instance
sitole 2dd8221
Implemented builds logs for both local and remote resource providers
sitole 41045e8
Code review fixes
sitole 6053299
Fixed place where local ctx was not used
sitole 423332f
Updated text for instance sync store struct
sitole d03d51d
Unify duplicated logic for both instance create and instance sync
sitole 54b0cd6
Removed sandbox prefix from snadbox info struct
sitole 454d1b0
Field rename
sitole 54f59a7
Fix issue with concurrent access to instance fields
sitole 001e4ed
Removed roles getters and replaced with pre-cached values
sitole 5cfd848
Log close err
sitole b35d2ff
Explicitly check if res body parsed struct is nil
sitole e87b9a9
Close instances during cluster close
sitole b49d6f4
Removed double check of res body struct is nil
sitole 2fcfe46
Merge branch 'main' into chore/make-edge-api-optional
sitole 0cab296
Merge branch 'main' into chore/make-edge-api-optional
sitole eeeb5b6
Remove file after conflict resolved
sitole fa2b8fb
Use callback system for build logs query
sitole 045ef03
When instance was not found, just log err do not return
sitole c3adefb
Merge branch 'main' into chore/make-edge-api-optional
sitole a2c9779
After-merge changes
sitole 2ef3a74
Implemented template builder cpu check from 05ac56bf4a12cf01275c0339b…
sitole 8397a81
Access machine info on copied struct
sitole 1e62925
Make parameter naming more obvious, fix use
sitole 62fc82e
Removed duplicated instance sync
sitole d2fc2af
Fixed double fetch
sitole 05d0c67
Merge branch 'main' into chore/make-edge-api-optional
sitole 857b75b
Fail instance health on too many failed synces
sitole 7e39e0b
Added log line during too many failed sync
sitole e6890ef
Make instance sync call lower timeouted
sitole 714b217
Merge branch 'main' into chore/make-edge-api-optional
sitole fe889cd
Fix after merge conflict
sitole 3259ce1
Merge branch 'main' into chore/make-edge-api-optional
sitole a2fbac6
After conflict deps tidy
sitole 2daf850
Renamed const for orchestrator API port
sitole ce46916
Build cluster api endpoint
sitole 3226d6c
Use client instead of connection
sitole d510fea
Nits
sitole 6422db3
For statis service discovery use list with predefined size
sitole 914d08b
Deduplicate function for fetching build logs
sitole File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.