Skip to content

[release-1.19] pilot: set istio.alpn_override: false in subset clusters#46529

Merged
istio-testing merged 1 commit intoistio:release-1.19from
jewertow:46504
Aug 15, 2023
Merged

[release-1.19] pilot: set istio.alpn_override: false in subset clusters#46529
istio-testing merged 1 commit intoistio:release-1.19from
jewertow:46504

Conversation

@jewertow
Copy link
Copy Markdown
Member

@jewertow jewertow commented Aug 14, 2023

Please provide a description of this PR:

#46504

* Add tests for destination rules with port level settings

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Add test cases for subsets

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Refactor AddALPNOverrideToMetadata

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Update comment for AddALPNOverrideToMetadata

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Fix lint error

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Add release note

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Fix release note

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

---------

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
@jewertow jewertow requested a review from a team August 14, 2023 23:48
@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 14, 2023
@istio-testing istio-testing merged commit 91f1a5f into istio:release-1.19 Aug 15, 2023
luksa pushed a commit to luksa/istio that referenced this pull request Apr 11, 2024
…e"` for SIMPLE and MUTUAL TLS (istio#833)

* Set alpnOverride depending on TLS Mode (istio#44918)

* Set alpnOverride

Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>

* remove test logs

Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>

---------

Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>

* pilot: set `istio.alpn_override: false` in subset clusters (istio#46496) (istio#46529)

* Add tests for destination rules with port level settings

* Add test cases for subsets

* Refactor AddALPNOverrideToMetadata

* Update comment for AddALPNOverrideToMetadata

* Fix lint error

* Add release note

* Fix release note

---------

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

---------

Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>
Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
Co-authored-by: Kalya Subramanian <42158129+ksubrmnn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants