Skip to content

[WIP] olmv1 catalog search #233

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ankitathomas
Copy link
Contributor

Search catalog contents.

Search catalogs for installable operators matching parameters

Usage:
  operator olmv1 search catalog [flags]

Flags:
      --catalog string                  catalog to search on. If not provided, all available catalogs are searched.
  -h, --help                                  help for catalog
      --list-versions                      list all versions available for each package
  -o, --output string                  output format. One of: (yaml|json)
      --package string                 search for package by name. If empty, all available packages will be listed
  -l, --selector stringToString   selector (label query) to filter catalogs on, supports '=', '==', and '!=' (default [])
PACKAGE             CATALOG        PROVIDER  CHANNELS
zookeeper-operator  operatorhubio            alpha,beta,stable

--list-versions allows listing multiple versions from one or more package.

kubectl-operator]$ ./bin/kubectl-operator olmv1 search catalog --list-versions --package zookeeper-operator
PACKAGE                     CATALOG            PROVIDER        VERSION
zookeeper-operator  operatorhubio  StreamNative  0.17.10
zookeeper-operator  operatorhubio  StreamNative  0.17.9
zookeeper-operator  operatorhubio  StreamNative  0.17.8
zookeeper-operator  operatorhubio  StreamNative  0.17.6
zookeeper-operator  operatorhubio  StreamNative  0.17.0

rashmigottipati and others added 6 commits April 30, 2025 23:11
Signed-off-by: Rashmi Gottipati <[email protected]>
Signed-off-by: Rashmi Gottipati <[email protected]>
Signed-off-by: Rashmi Gottipati <[email protected]>
Signed-off-by: Ankita Thomas <[email protected]>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 12, 2025
@openshift-ci openshift-ci bot requested review from jmrodri and joelanford May 12, 2025 21:01
return resp.Body, nil
}

func NewK8sClient(cfg *rest.Config, cl client.Client) Client {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we be reasonably assured this access will stay safe from the NetworkPolicy work we're also working on? I guess this counts as ingress? IIRC all egress is open initially but maybe not all ingress?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I doubt this will be unaffected by the NetworkPolicy changes in progress. I plan to hold off on this and revisit it next sprint with the added context.

@joelanford
Copy link
Member

@ankitathomas or @rashmigottipati looks like the vendor directory was accidentally committed. Can you amend the commit(s) to remove that?

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 14, 2025
@openshift-merge-robot
Copy link

PR needs rebase.

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-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants