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

Remove static base runtime layer from bootstrap #6063

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

lubronzhan
Copy link
Contributor

Right now dynamic runtime layer is available, they add the same config, so the static layer could be removed
Fix #5845

@lubronzhan lubronzhan requested a review from a team as a code owner January 10, 2024 02:32
@lubronzhan lubronzhan requested review from skriss and sunjayBhatia and removed request for a team January 10, 2024 02:32
@sunjayBhatia sunjayBhatia requested review from a team and davinci26 and removed request for a team January 10, 2024 02:32
Signed-off-by: lubronzhan <lzhan@vmware.com>
@lubronzhan lubronzhan force-pushed the topic/lubron/fix-5845 branch from 7e32f23 to 0290f8b Compare January 10, 2024 02:33
Copy link

Hi @lubronzhan! Welcome to our community and thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Contour better. You can also join us on our mailing list and in our channel in the Kubernetes Slack Workspace

Signed-off-by: lubronzhan <lzhan@vmware.com>
@lubronzhan lubronzhan force-pushed the topic/lubron/fix-5845 branch from 0d71e6f to f9c40b7 Compare January 10, 2024 02:34
Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (95f3eab) 78.51% compared to head (f9c40b7) 78.78%.
Report is 207 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6063      +/-   ##
==========================================
+ Coverage   78.51%   78.78%   +0.27%     
==========================================
  Files         138      138              
  Lines       18930    19744     +814     
==========================================
+ Hits        14862    15556     +694     
- Misses       3787     3885      +98     
- Partials      281      303      +22     
Files Coverage Δ
internal/envoy/v3/bootstrap.go 92.24% <ø> (-0.14%) ⬇️

... and 62 files with indirect coverage changes

@sunjayBhatia sunjayBhatia added the release-note/small A small change that needs one line of explanation in the release notes. label Jan 10, 2024
@sunjayBhatia
Copy link
Member

Can ignore the test-osx failure, it is a race related to: #6055

@sunjayBhatia sunjayBhatia enabled auto-merge (squash) January 10, 2024 16:48
@sunjayBhatia sunjayBhatia merged commit d8931d4 into projectcontour:main Jan 10, 2024
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/small A small change that needs one line of explanation in the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider removing static "base" Runtime layer from bootstrap
2 participants