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

Compiler server #446

Merged
merged 32 commits into from
Jun 5, 2024
Merged

Compiler server #446

merged 32 commits into from
Jun 5, 2024

Conversation

smintz
Copy link
Contributor

@smintz smintz commented May 4, 2024

No description provided.

smintz and others added 10 commits April 1, 2024 08:00
…-inserter-failure

[PE-1779] add retry in case of inserter failure
 - This adds a way to specify which namespace the config should be written to.
  If not set, the namespace provided by the command line will be used.

 - Specifying `namespace` inside `RolloutConfig` will make the inserter switch to `configmaps` store engine regardless of what specified by default

Additional fix:

 - normalize config file with `materialized_config/<filepath>.materialized_JSON` prefixes and suffixes. Users can now use any combination of with/without prefix and suffix.
Copy link

codecov bot commented Jun 4, 2024

Codecov Report

Attention: Patch coverage is 49.94670% with 939 lines in your changes missing coverage. Please review.

Project coverage is 44.52%. Comparing base (6032044) to head (5864d3f).

Files Patch % Lines
pb/protoconf/v1/protoconf.pb.go 27.41% 366 Missing and 2 partials ⚠️
server/server.go 75.93% 45 Missing and 19 partials ⚠️
agent/configmaps/configmaps.go 0.00% 61 Missing ⚠️
pb/protoconf/v1/protoconf_grpc.pb.go 43.51% 51 Missing and 10 partials ⚠️
devserver/command.go 0.00% 55 Missing ⚠️
agent/legacy.go 25.45% 38 Missing and 3 partials ⚠️
compiler/lib/module_service.go 79.33% 20 Missing and 11 partials ⚠️
examples/protoconf/src/crawler/crawler.pb.go 0.00% 27 Missing ⚠️
utils/utils.go 75.00% 18 Missing and 4 partials ⚠️
server/api/proto/v1/protoconf_mutation.pb.go 32.25% 20 Missing and 1 partial ⚠️
... and 20 more
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #446       +/-   ##
===========================================
- Coverage   71.63%   44.52%   -27.11%     
===========================================
  Files          15       56       +41     
  Lines        1657     5471     +3814     
===========================================
+ Hits         1187     2436     +1249     
- Misses        337     2783     +2446     
- Partials      133      252      +119     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@smintz smintz merged commit f4f1231 into main Jun 5, 2024
5 of 7 checks passed
@smintz smintz deleted the compiler_server branch June 5, 2024 04:01
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.

None yet

2 participants