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

common: Upgrade Knative Serving manifests to version 1.8.1 #2335

Conversation

apo-ger
Copy link
Contributor

@apo-ger apo-ger commented Nov 30, 2022

Changes:

  • Update knative serving manifests to v1.8.1 to support K8s-v1.25
  • Update Istio Ingress controller manifests to v1.8.0
  • Update README files (version references, instructions etc)

Refs: #2325

Signed-off-by: Apostolos Gerakaris <apoger@arrikto.com>
* Update instructions
  - No need to explicitly change the `portLevelMtls.8443` keys
    to be string as this is fixed in upstream
    (knative-extensions/net-istio#841)

* Update version references of knative-serving

Signed-off-by: Apostolos Gerakaris <apoger@arrikto.com>
@kimwnasptd
Copy link
Member

@apo-ger looks good!

I also run the e2e script manually and it succeeded https://github.com/kubeflow/manifests/tree/master/tests/e2e. Although we'll need to extend it to also make a request to the ISVC to ensure the networking is working properly.

/lgtm
/approve

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: apo-ger, kimwnasptd

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit ded35ed into kubeflow:master Jan 24, 2023
kevin85421 pushed a commit to juliusvonkohout/manifests that referenced this pull request Feb 28, 2023
…2335)

* common: Upgrade knative-serving manifests to v1.8.1

Signed-off-by: Apostolos Gerakaris <apoger@arrikto.com>

* Update README files

* Update instructions
  - No need to explicitly change the `portLevelMtls.8443` keys
    to be string as this is fixed in upstream
    (knative-extensions/net-istio#841)

* Update version references of knative-serving

Signed-off-by: Apostolos Gerakaris <apoger@arrikto.com>

Signed-off-by: Apostolos Gerakaris <apoger@arrikto.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants