Skip to content
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

Use plugin description in search command #26

Closed
ahmetb opened this issue Aug 15, 2018 · 17 comments
Closed

Use plugin description in search command #26

ahmetb opened this issue Aug 15, 2018 · 17 comments
Labels
lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/backlog Higher priority than priority/awaiting-more-evidence. priority/P2 P2 issues or PRs

Comments

@ahmetb
Copy link
Member

ahmetb commented Aug 15, 2018

kubectl plugin search certificate is not bringing up ca-cert which has the word in its description.

/priority P2
/priority backlog

@lbb
Copy link
Contributor

lbb commented Aug 16, 2018

Yeah, we are currently not using the description for the search. We are only searching plugin names.

@ahmetb
Copy link
Member Author

ahmetb commented Oct 17, 2018

@orthros is taking a stab at this.

@nammn
Copy link
Contributor

nammn commented Jan 14, 2019

If its still free, i would like to take a look at this @ahmetb

@ahmetb
Copy link
Member Author

ahmetb commented Jan 14, 2019

@nammn Feel free to work on a patch.

@nammn
Copy link
Contributor

nammn commented Jan 15, 2019

great, i'll work on this patch 👍

@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 28, 2019
@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels May 28, 2019
@fabiand
Copy link

fabiand commented Jun 13, 2019

Any update on this one?

The new virt plugin is also affected by this (not found if searched for kubevirt).

@ahmetb
Copy link
Member Author

ahmetb commented Jun 13, 2019

/remove-lifecycle rotten

We'll probably get to it at some point. It's not a super high priority since most users probably find plugins through some github repo so they know its installation command.

@k8s-ci-robot k8s-ci-robot removed the lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. label Jun 13, 2019
@fabiand
Copy link

fabiand commented Jun 13, 2019 via email

@ahmetb
Copy link
Member Author

ahmetb commented Jun 13, 2019

We already had #150 but happy to give it another try. This might require some experimentation. Text search isn't a simple problem, and we might need to sanitize the input a little bit (e.g. to remove line breaks, figure out how to treat urls in descriptions and so on).

@danielpacak
Copy link

The same applies to the who-can plugin which does some reverse lookup in RBAC permissions. It won't show up when I type kubectl krew search rbac.

Did you consider adding the keywords property to the manifest spec? Something similar to what npmers are doing? https://docs.npmjs.com/files/package.json#keywords

name: "who-can"
keywords: ["rbac", "authorization"]

@k8s-ci-robot k8s-ci-robot added priority/P2 P2 issues or PRs priority/backlog Higher priority than priority/awaiting-more-evidence. labels Jul 23, 2019
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 21, 2019
@corneliusweig
Copy link
Contributor

I'm unsure what to do with this issue. On the one hand #150 was closed long ago, but a search in description seems to be something that many users miss.
So I'd like to gather more input as to how an improved search functionality would be used. Let's put the technical details aside and just create a wish-list of workflows.

@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Nov 20, 2019
@fejta-bot
Copy link

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

@k8s-ci-robot
Copy link
Contributor

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/backlog Higher priority than priority/awaiting-more-evidence. priority/P2 P2 issues or PRs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants