Hey, <https://github.com/flyteorg/flyte/pull/5944>...
# contribute
c
Hey, https://github.com/flyteorg/flyte/pull/5944 Could I pls get help with a review on this PR? Thank you 🙇
Thanks for the response @average-finland-92144, will resolve the conflicts tomorrow morning 🙂
Resolved the conflicts @average-finland-92144
Is there a list collecting release notes somewhere?
a
well not really as a list, but we could start a Discussion?
c
Where?
a
c
@average-finland-92144 I just added a sentence to the 1.14 release notes 🙂
a
Oh thanks, we're looking to get better at categorizing changes (including breaking) early on by using labels. So for this one:
The helm chart does not set the appProtocol in service manifests by default anymore which is required in some deployments using istio.
What about ?
The
flyte-core
Helm chart doesn't set anymore the
appProtocol
in the Service manifests. This might affect deployments that use Istio as ingress controller.
c
Will change accordingly 👍