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
Don't require python to be installed for everything #5245
Conversation
d3131aa
to
93c7867
Compare
We only use python in one place and probably won't start using it more without some kind of policy change. We don't need to require that everyone has it installed, and can instead only require it for people who're running the boilerplate check Signed-off-by: Ashley Davis <ashley.davis@jetstack.io>
93c7867
to
2c9344e
Compare
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io>
## make kind [KIND_CLUSTER_NAME=name] [K8S_VERSION=<kubernetes_version>] | ||
## make [KIND_CLUSTER_NAME=name] [K8S_VERSION=<kubernetes_version>] e2e-setup-kind |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: unrelated change, but I noticed this comment was wrong and wanted to fix it quickly!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jakexks, SgtCoDFish 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 |
We only use python in one place and probably won't start using it more without some kind of policy change.
We don't need to require that everyone has it installed, and can instead only require it for people who're running the boilerplate check.
Kind
/kind cleanup
Release Note