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.

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

Jump to bottom

Merge Polling functions #28

Open
h3llix opened this issue Jul 17, 2021 · 0 comments
Open

Merge Polling functions #28

h3llix opened this issue Jul 17, 2021 · 0 comments
Labels
help wanted Extra attention is needed
Projects
Katana v1 v1

Comments

@h3llix
Copy link
Contributor

h3llix commented Jul 17, 2021

katana/lib/deployment/polling.go

Line 19 in 5107528

func PingStatefulSets(ctx context.Context, kubeclientset *kubernetes.Clientset, opts map[string]string) ([]*ResourceStatus, bool, error) {

katana/lib/deployment/polling.go

Line 42 in 5107528

func PingDeployments(ctx context.Context, kubeclientset *kubernetes.Clientset, opts map[string]string) ([]*ResourceStatus, bool, error) {

Discussion over merging these functions.
@h3llix h3llix added the help wanted Extra attention is needed label Jul 17, 2021
@cypherean cypherean added this to To do in Katana v1 Dec 21, 2021
@cypherean cypherean added this to To do in v1 Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
v1
To do
Katana v1
To do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@h3llix