knative / knative/pkg

Migrate to new K8s generator scripts/go commands

Open
#2,927 2 comments 2 reactions 0 assignees View on GitHub
lifecycle/frozen
Dominant language
Go
Stars
276
Forks
343
Avg merge
1d 1h
Merged PRs (30d)
1

Description

## Context
See https://cloud-native.slack.com/archives/C04LGHDR9K7/p1704710449414709 and https://github.com/knative/pkg/pull/2904

## Possible solutions
Based on https://github.com/knative/pkg/pull/2904#issuecomment-1851613905 and discussions.

a) Use `go run` instead of scripts (downside: this does not clean up old resources)
b) Use the new `k8s.io/code-generator/kube_codegen.sh`
c) Wait until @cardil manages to migrate the scripts in K8s to full golang, then adapt to that solution

## Todo
- [ ] Agree on which way to go
- [ ] Implement in `pkg`
- [ ] Implement on all downstream repos

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the linked knative/pkg pull request and the Kubernetes generator options described here, especially go run and kube_codegen.sh. The issue first needs agreement on an approach, then implementation in pkg and migration of all downstream repositories; done means those decisions and migrations are complete.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, kubernetes
Domain
build-system, tooling
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.