docs(knative): update to newer Go and Knative versions by ishaanxgupta · Pull Request #438 · urunc-dev/urunc

ishaanxgupta · 2026-02-03T15:44:04Z

Description

This PR updates the Knative integration tutorial to use the latest versions and improves the documentation structure

Update Go version from 1.21.5 to use dynamic [[ versions.go ]] template (currently 1.24.6)
Update Knative version to use [[ versions.knative ]] template (currently 1.17.0)
Restructure installation to prioritize pre-built binaries (Option 1)
Simplify Go and ko installation instructions
Add ko version verification step
Improve clarity of pre-built vs source build options

Related issues

Fixes Fix Knative tutorial #430

How was this tested?

Verified template syntax matches mkdocs.yml configuration (j2_variable_start_string: "[[")
Confirmed versions.go and versions.knative variables exist in docs/variables/versions.yml
Checked consistency with other tutorials (e.g., Running-urunc-with-kind.md uses same template pattern)
Validated pre-built S3 URL format matches original format (with -5220308.yaml suffix)
Ensured source build option still uses nubificus/serving -b feat_urunc to preserve urunc patches
Reviewed all changes for accuracy and completeness

LLM usage

GitHub Copilot assisted with code review.

Checklist

I have read the contribution guide.
The linter passes locally (make lint).
The e2e tests of at least one tool pass locally (make test_ctr, make test_nerdctl, make test_docker, make test_crictl).
If LLMs were used: I have read the llm policy.

collaborated with @hemang1404

netlify · 2026-02-03T15:44:10Z

✅ Deploy Preview for urunc ready!

Name	Link
🔨 Latest commit	`42df884`
🔍 Latest deploy log	https://app.netlify.com/projects/urunc/deploys/6998b194b2fa5e0007373fec
😎 Deploy Preview	https://deploy-preview-438--urunc.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

ishaanxgupta · 2026-02-03T17:50:23Z

@cmainas please have a look and let me know if any changes are required

cmainas · 2026-02-04T11:08:28Z

Hello @ishaanxgupta ,

please try to create a knative environment with your suggested changes and provide enough proof that this works.

cmainas · 2026-02-04T11:50:23Z

Duplicate with #433

ishaanxgupta · 2026-02-06T04:18:32Z

Hi @cmainas
Here are the validation results tested by me and @hemang1404

Test Environment

OS: Ubuntu 22.04 (WSL2)
Kubernetes: Kind v0.32.0

Results

go version go1.24.6 linux/amd64
Template [[ versions.go ]] correctly resolves to 1.24.6 from versions.yml
Simplified installation using /latest/ works correctly
Knative v1.17.0 Deployed
Pre-built Knative v1.17.0 URL works correctly
Template variable [[ versions.knative ]] resolves properly
All Knative components deploy successfully

cmainas

Thank you @ishaanxgupta and @hemang1404 for testing this. I have a left comment for the structure of the tutorial. I have also verified that it works. A few more notes:

We can add an instruction to get the ingress IP. Simply transforming the note that starts with Note: 10.244.9.220 is the IP of the kourier-internal svc. to an instruction before invoking the function
I think we can remove the Wrapping upsection

cmainas · 2026-02-06T11:10:00Z

docs/tutorials/knative.md

 ```

-### Clone urunc-enabled Knative Serving 
+### Option 1: Use Pre-built Knative (Recommended)


This looks good, but if users decide not to build Knative, then there is no reason to installl docker, go and ko. Therefore, it would be better to move this further up before installing anything and adjusting the text.

Also, we will need to adjust the prerequisites, to keep them as a working K8s and urunc installation and move the "Basic git, curl, kubectl, and docker installed" to the option that builds knative from scratch.

ishaanxgupta · 2026-02-09T09:20:17Z

@cmainas made the changes as you suggested, please have a look

7h3-3mp7y-m4n · 2026-02-09T10:12:06Z

Hey @ishaanxgupta did you also check the tutorial, by building Knative from scratch?

docs/tutorials/knative.md

7h3-3mp7y-m4n · 2026-02-18T20:04:40Z

any updates @ishaanxgupta and @hemang1404 on this ?

- Update Go version from 1.21.5 to use dynamic [[ versions.go ]] template (currently 1.24.6) - Update Knative version to use [[ versions.knative ]] template (currently 1.17.0) - Restructure installation to prioritize pre-built binaries (Option 1) - Simplify Go and ko installation instructions - Add ko version verification step - Improve clarity of pre-built vs source build options Fixes: urunc-dev#430 Signed-off-by: hemang1404 <hemangsharrma@gmail.com> Co-authored-by: ishaanxgupta <ishaankone@gmail.com>

Co-authored-by: Hemang Sharrma <hemangsharrma@gmail.com> Signed-off-by: ishaanxgupta <ishaankone@gmail.com>

Signed-off-by: Ishaan Gupta <ishaankone@gmail.com> Signed-off-by: Hemang Sharrma <hemangsharrma@gmail.com> Co-authored-by: Hemang Sharrma <hemangsharrma@gmail.com>

7h3-3mp7y-m4n · 2026-02-21T21:38:26Z

docs/tutorials/knative.md

+#### Environment Setup

-### Install Go 1.21  
+Install [Docker](/quickstart/#install-docker), Go [[ versions.go ]], and `ko`:


We have already menionted about docker in the prerequisites, so why don't we add docker install link directly to it? Also its duplication (A suggestion)

In the prerequisites, we mention a A Docker-compatible registry and docker is required due to ko (AFAIK).

7h3-3mp7y-m4n · 2026-02-21T21:58:29Z

docs/tutorials/knative.md

+##### Install Go (≥ 1.23)

-### Verify Go installation (Should be 1.21.5)
+Please follow the official [Go installation instructions](https://go.dev/doc/install) to install Go 1.23 or newer.
+
+##### Verify Go installation

 ```console
-$ export GOROOT=/usr/local/go1.21/go 
-$ export PATH=$GOROOT/bin:$PATH  
+$ export PATH=/usr/local/go/bin:$PATH  
 $ export GOPATH=$HOME/go 
 $ go version
-go version go1.21.5 linux/amd64
 ```


We should list Go ≥ 1.23 under Prerequisites rather than documenting the full installation and verification steps. Since most developers already have Go installed if they are already using knative, it’s enough to state the required version and link to the official installation guide (https://go.dev/doc/install). This keeps the documentation clean, concise, and avoids duplicating upstream instructions.

That sounds good.

7h3-3mp7y-m4n · 2026-02-21T22:02:44Z

docs/tutorials/knative.md

+##### Install ko
 ```bash
-export OS=Linux
-export ARCH=x86_64
-curl -sSfL "https://github.com/ko-build/ko/releases/download/v${VERSION}/ko_${VERSION}_${OS}_${ARCH}.tar.gz" -o ko.tar.gz
-sudo tar -zxvf ko.tar.gz -C /usr/local/bin` 
+curl -sSfL https://github.com/ko-build/ko/releases/latest/download/ko_Linux_x86_64.tar.gz | sudo tar xzf - -C /usr/local/bin
+ko version
 ```


The first paragraph of the documentation already says that if we are building from source, we would need ko, so I think we should also put ko in prerequisites, and about the installation guide, I am not sure ( also confused)

Yes, we should add ko in the prerequisites. If ko provides an installation method rather than from its releases we can point to there, otherwise we can provide the commands to fetch and install ko

7h3-3mp7y-m4n · 2026-02-21T22:09:44Z

docs/tutorials/knative.md


-> Note: There are cases where due to the large manifests, kubectl fails. Try a second time, or use `kubectl create -f https://s3.nbfc.io/knative/knative-v[[ versions.knative ]]-urunc-5220308.yaml`
-
 ## Setup Networking (Kourier)


I think we should add a link for kourier, people can also reach their website and explore. ( A suggestion)

I am not sure I get this one.

I mean, adding it like this

## Setup Networking ([Kourier](https://github.com/knative-extensions/net-kourier))

7h3-3mp7y-m4n · 2026-02-21T22:12:50Z

docs/tutorials/knative.md

 ```bash
 kubectl get ksvc -A -o wide
 ```

 Should be empty. Create an simple httpreply


We should remove the first step cause the user is already building from scratch, so nothing would be there. Just continue the line -> Create an simple httpreply service, based on a simple C program:

That makes sense. Also, we do not have to mention that "Should be empty"

7h3-3mp7y-m4n · 2026-02-21T22:16:00Z

docs/tutorials/knative.md

 ### Check Knative Service 

 ```bash
 kubectl get ksvc -A -o wide 
 ```


We can change the header to Check Knative Services are running, or we can also add the output for it

We can add the output, the title can stay the same

docs/tutorials/knative.md

7h3-3mp7y-m4n · 2026-02-21T22:25:05Z

docs/tutorials/knative.md

+kubectl get svc -n kourier-system kourier-internal -o jsonpath='{.spec.clusterIP}'
+```
+
+Note the IP address returned (e.g., `10.244.9.220`). You'll use this in the following curl commands.


We can write something like this -> This command returns the internal ClusterIP (e.g., 10.244.9.220). Use this value in the next curl command (A suggestion)

Let;s just replace "You'll use" with "Use"

docs/tutorials/knative.md

7h3-3mp7y-m4n · 2026-02-21T22:38:47Z

I've added more suggestions @ishaanxgupta and @hemang1404 , what are your thoughts about them?

cmainas · 2026-02-23T09:13:38Z

Hello all, thank you for your comments.

I just want to point out that the tutorial is still broken and we need to fix this. We can apply any other changes later, but if something is broken, we need to fix it quickly and then find ways to improve it.

ishaanxgupta · 2026-02-23T09:15:03Z

Hello all, thank you for your comments.

I just want to point out that the tutorial is still broken and we need to fix this. We can apply any other changes later, but if something is broken, we need to fix it quickly and then find ways to improve it.

@cmainas
So i guess what if we merge this and fix the suggestions in the follow-up PR?

7h3-3mp7y-m4n · 2026-02-23T22:03:25Z

docs/tutorials/knative.md

 kubectl apply -f https://github.com/knative/net-kourier/releases/latest/download/kourier.yaml 
 kubectl patch configmap/config-network -n knative-serving --type merge -p \ 
  '{"data":{"ingress.class":"kourier.ingress.networking.knative.dev"}}'
 kubectl patch configmap/config-domain -n knative-serving --type merge -p \ 
  '{"data":{"127.0.0.1.nip.io":""}}'
 ```


after running we can encounter this error
kubectl apply -f https://github.com/knative/net-kourier/releases/latest/download/kourier.yaml kubectl patch configmap/config-network -n knative-serving --type merge -p \ '{"data":{"ingress.class":"kourier.ingress.networking.knative.dev"}}' kubectl patch configmap/config-domain -n knative-serving --type merge -p \ '{"data":{"127.0.0.1.nip.io":""}}' namespace/kourier-system unchanged configmap/kourier-bootstrap unchanged configmap/config-kourier unchanged serviceaccount/net-kourier unchanged clusterrole.rbac.authorization.k8s.io/net-kourier unchanged clusterrolebinding.rbac.authorization.k8s.io/net-kourier unchanged deployment.apps/net-kourier-controller configured service/net-kourier-controller unchanged deployment.apps/3scale-kourier-gateway unchanged service/kourier unchanged service/kourier-internal unchanged horizontalpodautoscaler.autoscaling/3scale-kourier-gateway unchanged poddisruptionbudget.policy/3scale-kourier-gateway-pdb configured Error from server (BadRequest): Invalid JSON Patch {"data":{"ingress.class":"kourier.ingress.networking.knative.dev"}}: command not found Error from server (BadRequest): Invalid JSON Patch

Better we should go for

kubectl apply -f https://github.com/knative-extensions/net-kourier/releases/latest/download/kourier.yaml kubectl patch configmap/config-network -n knative-serving --type merge -p '{"data":{"ingress.class":"kourier.ingress.networking.knative.dev"}}' kubectl patch configmap/config-domain -n knative-serving --type merge -p '{"data":{"127.0.0.1.nip.io":""}}'

copy paste friendly

hemang1404 force-pushed the fix-knative-tutorial-430 branch from f74e0c9 to 70ff35e Compare February 3, 2026 18:25

cmainas added the do-not-merge label Feb 4, 2026

cmainas added the duplicate This issue or pull request already exists label Feb 4, 2026

cmainas added ok-to-test and removed do-not-merge labels Feb 6, 2026

cmainas requested changes Feb 6, 2026

View reviewed changes

hemang1404 force-pushed the fix-knative-tutorial-430 branch 2 times, most recently from e91074f to 70ff35e Compare February 8, 2026 07:20

cmainas mentioned this pull request Feb 9, 2026

Fix Knative tutorial #430

Open

hemang1404 force-pushed the fix-knative-tutorial-430 branch 2 times, most recently from 2435527 to 0ee4b4b Compare February 11, 2026 21:07

7h3-3mp7y-m4n reviewed Feb 11, 2026

View reviewed changes

docs/tutorials/knative.md Outdated Show resolved Hide resolved

7h3-3mp7y-m4n reviewed Feb 11, 2026

View reviewed changes

docs/tutorials/knative.md Outdated Show resolved Hide resolved

hemang1404 and others added 3 commits February 21, 2026 00:38

docs: restructure Knative tutorial per maintainer feedback

c62c41e

Co-authored-by: Hemang Sharrma <hemangsharrma@gmail.com> Signed-off-by: ishaanxgupta <ishaankone@gmail.com>

fixed requested changes

42df884

Signed-off-by: Ishaan Gupta <ishaankone@gmail.com> Signed-off-by: Hemang Sharrma <hemangsharrma@gmail.com> Co-authored-by: Hemang Sharrma <hemangsharrma@gmail.com>

hemang1404 force-pushed the fix-knative-tutorial-430 branch from 382f3cf to 42df884 Compare February 20, 2026 19:10

7h3-3mp7y-m4n reviewed Feb 21, 2026

View reviewed changes

docs/tutorials/knative.md Show resolved Hide resolved

7h3-3mp7y-m4n reviewed Feb 21, 2026

View reviewed changes

docs/tutorials/knative.md Show resolved Hide resolved

7h3-3mp7y-m4n reviewed Feb 23, 2026

View reviewed changes


		> Note: There are cases where due to the large manifests, kubectl fails. Try a second time, or use `kubectl create -f https://s3.nbfc.io/knative/knative-v[[ versions.knative ]]-urunc-5220308.yaml`

		## Setup Networking (Kourier)

Conversation

ishaanxgupta commented Feb 3, 2026

Description

Related issues

How was this tested?

LLM usage

Checklist

Uh oh!

netlify bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for urunc ready!

Uh oh!

ishaanxgupta commented Feb 3, 2026

Uh oh!

cmainas commented Feb 4, 2026

Uh oh!

cmainas commented Feb 4, 2026

Uh oh!

ishaanxgupta commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Environment

Results

Uh oh!

cmainas left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ishaanxgupta commented Feb 9, 2026

Uh oh!

7h3-3mp7y-m4n commented Feb 9, 2026

Uh oh!

Uh oh!

Uh oh!

7h3-3mp7y-m4n commented Feb 18, 2026

Uh oh!

7h3-3mp7y-m4n Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cmainas Feb 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

7h3-3mp7y-m4n commented Feb 21, 2026

Uh oh!

cmainas commented Feb 23, 2026

Uh oh!

netlify bot commented Feb 3, 2026 •

edited

Loading

ishaanxgupta commented Feb 6, 2026 •

edited

Loading

7h3-3mp7y-m4n Feb 21, 2026 •

edited

Loading

cmainas Feb 23, 2026 •

edited

Loading