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

release: v1.1.0-rc.1 #3791

Merged
merged 3 commits into from
Jul 9, 2024
Merged

release: v1.1.0-rc.1 #3791

merged 3 commits into from
Jul 9, 2024

Conversation

guydc
Copy link
Contributor

@guydc guydc commented Jul 8, 2024

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Release v1.1.0-rc.1

guydc added 2 commits July 8, 2024 17:24
Signed-off-by: Guy Daich <[email protected]>
Signed-off-by: Guy Daich <[email protected]>
change: |
Added Performance Benchmarking Document
Added User Guide for Zipkin Tracing
Added User Guide for Customizatio of Filter Order
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Added User Guide for Customizatio of Filter Order
Added User Guide for Customizating for Ordering Filters

Fixed Reconciliation when a referenced Secret or ConfigMap changes
Fixed ReplaceFullPath not working for root path
Fixed Default Application Protocol to TCP
Fixed not using well-known ports (80, 443) in rediret location
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Fixed not using well-known ports (80, 443) in rediret location
Fixed not appending well-known ports (80, 443) in redirect Location header

Fixed Reconciliation when EnvoyProxy backendRefs changes
Fixed Reconciliation when a referenced Secret or ConfigMap changes
Fixed ReplaceFullPath not working for root path
Fixed Default Application Protocol to TCP
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Fixed Default Application Protocol to TCP
Fixed Default Application Protocol to TCP for Zipkin Tracing

Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.02%. Comparing base (07e2852) to head (fdd09af).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3791      +/-   ##
==========================================
+ Coverage   69.00%   69.02%   +0.01%     
==========================================
  Files         176      176              
  Lines       21760    21760              
==========================================
+ Hits        15015    15019       +4     
+ Misses       5663     5661       -2     
+ Partials     1082     1080       -2     

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

- area: installation
change: |
Added new gateway-addons-helm chart for Observability
Added support for global image settings for all images in the EG chart
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Added support for global image settings for all images in the EG chart
Added support for global image settings for all images in the Envoy Gateway helm chart



breaking-change: |
Gateway-API BackendTLSPolicy v1alpha3 is incompatible with previous versions of the CRD
Copy link
Contributor

Choose a reason for hiding this comment

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

we need to add some Upgrade notes in here similar to https://gateway-api.sigs.k8s.io/guides/?h=release+not#backendtlspolicy

Copy link
Contributor Author

@guydc guydc Jul 8, 2024

Choose a reason for hiding this comment

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

I followed the GW-API approach and added this to the "quickstart" guide.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks for adding it to the Quickstart, can we also add the link in here ?

Signed-off-by: Guy Daich <[email protected]>
@guydc guydc marked this pull request as ready for review July 9, 2024 00:55
@guydc guydc requested a review from a team as a code owner July 9, 2024 00:55
Copy link
Contributor

@arkodg arkodg left a comment

Choose a reason for hiding this comment

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

LGTM thanks !

@arkodg arkodg requested review from a team July 9, 2024 01:01
@arkodg arkodg merged commit e63ffa0 into envoyproxy:main Jul 9, 2024
24 of 27 checks passed
Copy link
Member

@Xunzhuo Xunzhuo left a comment

Choose a reason for hiding this comment

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

I think we should generate the release note based the yaml.

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.

4 participants