-
Notifications
You must be signed in to change notification settings - Fork 465
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
feat(kubectl-validate): add validate plugin #3730
feat(kubectl-validate): add validate plugin #3730
Conversation
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
🤖 Beep beep! I’m a robot speaking on behalf of @ahmetb. 🤖 Thanks for submitting your kubectl plugin to Krew! In the meanwhile, here are a few tips to make your plugin manifest better:
Thanks for your patience! |
plugins/validate.yaml
Outdated
name: validate | ||
spec: | ||
version: v0.0.3 | ||
homepage: https://github.com/microsoft/retina |
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.
seems incorrect
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.
homepage: https://github.com/microsoft/retina | |
homepage: https://github.com/kubernetes-sigs/kubectl-validate |
plugins/validate.yaml
Outdated
spec: | ||
version: v0.0.3 | ||
homepage: https://github.com/microsoft/retina | ||
shortDescription: a SIG-CLI subproject to support the local validation of resources for native Kubernetes types and CRDs. |
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.
keep it <50 chars (i.e. cut to the chase)
plugins/validate.yaml
Outdated
kubectl-validate distinguishes itself among other OpenAPI-based validators by to its deep integration | ||
with Kubernetes upstream validation. kubectl-validate is written by Kubernetes apiserver authors | ||
using the same code as the server-side. | ||
This allows kubectl-validate to give the most accurate error messages and support the Kubernetes-specific | ||
validations often ignored by other tools. |
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.
wrap at 80 chars wide please
plugins/validate.yaml
Outdated
|
||
|
||
|
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.
please configure your editor to trim empty newlines. there should be only one \n at the end
plugins/validate.yaml
Outdated
|
||
|
||
|
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.
@nlamirault gentle ping. it seems like the build is failing due to a checksum error. |
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
Seems like it's green now. 🎉 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahmetb, nlamirault 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 |
Add a new plugin for kubectl-validate CLI
Ref: kubernetes-sigs/kubectl-validate#70