Skip to content

Updating the cluster search query #770

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.

Sign up for GitHub

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
chamalabey wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updating the cluster search query #770

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions cmd/cluster/from_infra_id.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ package cluster
import (
"encoding/json"
"fmt"
"os"
"sort"
"strings"

sdk "github.com/openshift-online/ocm-sdk-go"
amv1 "github.com/openshift-online/ocm-sdk-go/accountsmgmt/v1"
v1 "github.com/openshift-online/ocm-sdk-go/clustersmgmt/v1"
Expand All @@ -11,9 +15,6 @@ import (
"github.com/spf13/cobra"
"gopkg.in/yaml.v2"
cmdutil "k8s.io/kubectl/pkg/cmd/util"
"os"
"sort"
"strings"
)

type fromInfraIdOptions struct {
Expand Down Expand Up @@ -72,7 +73,7 @@ func (ops *fromInfraIdOptions) run(cmd *cobra.Command, args []string) error {
}
clusters, err := utils.ApplyFilters(
ocmClient,
[]string{fmt.Sprintf("name like '%s%%'", clusterName)},
Copy link
Member

@typeid typeid Jul 9, 2025

Choose a reason for hiding this comment

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

@chamalabey do you know why we're not directly returning the output from this?

Copy link
Contributor Author

@chamalabey chamalabey Jul 9, 2025

Choose a reason for hiding this comment

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

@typeid Is it possible for this call to return more than one result (list of clusters)? Line 80 is handling this scenario in a for loop. Either way we might need to check whether if there is more than one result and return only one. Please let me know your thought ?

[]string{fmt.Sprintf("infra_id like '%s%%'", clusterName)},
)
if err != nil {
return fmt.Errorf("could not retrieve clusters for %s", infraId)
Expand Down