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.0.1 #3144

Merged
merged 5 commits into from
Apr 9, 2024
Merged

release: v1.0.1 #3144

merged 5 commits into from
Apr 9, 2024

Conversation

Xunzhuo
Copy link
Member

@Xunzhuo Xunzhuo commented Apr 9, 2024

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

release: v1.0.1

@Xunzhuo Xunzhuo requested a review from a team as a code owner April 9, 2024 06:58
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
@Xunzhuo Xunzhuo force-pushed the release-v1.0.1 branch 2 times, most recently from 3d1199a to 2a255fe Compare April 9, 2024 07:07
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
release-notes/v1.0.1.yaml Outdated Show resolved Hide resolved
- area: translator
change: |
Fixed nil secret in resourceversiontable
Fixed add missing http filters to the http filter chain
Copy link
Contributor

@arkodg arkodg Apr 9, 2024

Choose a reason for hiding this comment

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

Suggested change
Fixed add missing http filters to the http filter chain
Add missing http filters to the http filter chain when ClientTrafficPolicy and MergeGateways is enabled

@zirain
Copy link
Contributor

zirain commented Apr 9, 2024

can we just reuse doc from v1.0.0?

@arkodg
Copy link
Contributor

arkodg commented Apr 9, 2024

can we just reuse doc from v1.0.0?

@zirain can you elaborate ?

@zirain
Copy link
Contributor

zirain commented Apr 9, 2024

can we just reuse doc from v1.0.0?

@zirain can you elaborate ?

there's little change around doc between v1.0.0 and v1.0.1, so should not create a new version of doc?

@arkodg
Copy link
Contributor

arkodg commented Apr 9, 2024

can we just reuse doc from v1.0.0?

@zirain can you elaborate ?

there's little change around doc between v1.0.0 and v1.0.1, so should not create a new version of doc?

We either need to set docs to v1.0.1 or rename dir to v1.0, I believe former is easier cc @Xunzhuo

@zirain
Copy link
Contributor

zirain commented Apr 9, 2024

can we just reuse doc from v1.0.0?

@zirain can you elaborate ?

there's little change around doc between v1.0.0 and v1.0.1, so should not create a new version of doc?

We either need to set docs to v1.0.1 or rename dir to v1.0, I believe former is easier cc @Xunzhuo

At least, should do this later

@Xunzhuo
Copy link
Member Author

Xunzhuo commented Apr 9, 2024

can we just reuse doc from v1.0.0?

Actually, it is reused from v1.0.0, just changed path and modified the related image tags in quickstart. We do not generate new paths for patch release, if we do so, there will be two paths for v1.0.0 and v1.0.1.

@@ -40,6 +40,7 @@ jobs:

- name: Check links
run: make docs docs-check-links
continue-on-error: true
Copy link
Contributor

Choose a reason for hiding this comment

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

wait #3146

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can remove this now.

@@ -43,7 +43,7 @@ unprivileged port, so that Envoy Gateway doesn't need additional privileges.
It's important to be aware of this mapping, since you may need to take it into
consideration when debugging.

[`quickstart.yaml`]: https://github.com/envoyproxy/gateway/releases/download/v1.0.0/quickstart.yaml
[`quickstart.yaml`]: https://github.com/envoyproxy/gateway/releases/download/v1.0.1/quickstart.yaml
Copy link
Member Author

Choose a reason for hiding this comment

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

Some links are 404 as expected since release is not cut off yet

Copy link
Contributor

Choose a reason for hiding this comment

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

this link will never 404, because github.com is ignored because of rate limiting.

Xunzhuo added 5 commits April 9, 2024 16:48
Signed-off-by: bitliu <bitliu@tencent.com>
Signed-off-by: bitliu <bitliu@tencent.com>
Signed-off-by: bitliu <bitliu@tencent.com>
Signed-off-by: bitliu <bitliu@tencent.com>
Signed-off-by: bitliu <bitliu@tencent.com>
@Xunzhuo Xunzhuo merged commit 0e1d7f4 into envoyproxy:main Apr 9, 2024
20 checks passed
Xunzhuo added a commit to Xunzhuo/gateway that referenced this pull request Apr 9, 2024
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.

3 participants