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

stable-2.11.3 release #8779

Closed
12 of 13 tasks
kleimkuhler opened this issue Jun 30, 2022 · 3 comments
Closed
12 of 13 tasks

stable-2.11.3 release #8779

kleimkuhler opened this issue Jun 30, 2022 · 3 comments
Assignees
Milestone

Comments

@hawkw
Copy link
Member

hawkw commented Jul 1, 2022

proxy bump: #8789

kleimkuhler added a commit that referenced this issue Jul 1, 2022
#8322 

Closes #8719 Part of #8779 

2.11 only has the Server and ServerAuthorization policy CRDs. This fixes the
Helm label for Server and adds the label for ServerAuthorization.

Signed-off-by: Kevin Leimkuhler <kleimkuhler@icloud.com>
@kleimkuhler kleimkuhler changed the title stable-2.11.3 checklist stable-2.11.3 release Jul 1, 2022
kleimkuhler added a commit that referenced this issue Jul 5, 2022
Cherry pick change #8474 which fixed #8469.

This was originally missed by #8779 because the issue was opened as a
discussion.

---

When labeling a headless service for exporting, the service-mirror controller
was ignoring the change because `isExportedEndpoints()` was checked on the _old_
version of the EP update (which doesn't have the label).

This change checks on both the old and new version of the updated EP (and so I
had to move some of the stuff out of `isExportedEndpoints()` into the calling
sites, to avoid dupped log entries when calling it).

In order to test this, I removed the `mirror.linkerd.io/exported` label from the
nginx SS and instead label it in the test after the SS persists.

* Consolidated isExportedEndpoints() and isExportedService() into isExported(),
  which uses the label configured in the Link

Signed-off-by: Kevin Leimkuhler <kleimkuhler@icloud.com> Co-authored-by:
Alejandro Pedraza <alejandro@buoyant.io>
@hawkw
Copy link
Member

hawkw commented Jul 6, 2022

I believe this version has been released!

@hawkw hawkw closed this as completed Jul 6, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants