`make deploy` should not require `go`
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 88
- Forks
- 66
- Avg merge
- 53m
- Merged PRs (30d)
- 1
Description
Version
N/A
Describe the bug
Invoking make deploy as guided in Getting Started for Pulp Operator requires go binary because deploy refers manifests and manifests requires controller-gen which uses go.
It's ok for devs since current operator is written in Go, but for users, it's like an unnecessary dependency. I think this dependency should be removed, or Getting Started should be updated that the command requires go (or provide alternative handy way without make deploy to deploy Operaotr).
To Reproduce
- Clone this repository
- Invoke
make deploy
Expected behavior
Pulp Operator can be deployed without go binary.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the repository's Makefile and the make deploy target, then trace its manifests dependency and the controller-gen step. Compare this flow with the Getting Started for Pulp Operator quickstart. Done means users can deploy without a go binary, or the quickstart clearly documents the requirement and provides a practical alternative.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, kubernetes
- Domain
- devops, infrastructure
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100