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

Support new 'includeIdleEdges` option. False by default. #3522

Merged
merged 2 commits into from Dec 16, 2020

Conversation

jshaughn
Copy link
Collaborator

@jshaughn jshaughn commented Dec 11, 2020

Server-side support in the graph code, to support the new includeIdleEdges option.

UI PR: kiali/kiali-ui#2037

Part of #3519

@jshaughn jshaughn added requires UI PR A PR sent to the backend kiali/kiali requires changes on frontend kiali/kiali-ui team/saturn labels Dec 11, 2020
@jshaughn jshaughn added this to In Review in Sprint 49 via automation Dec 11, 2020
@jshaughn jshaughn self-assigned this Dec 11, 2020
@jmazzitelli
Copy link
Collaborator

You should run the "api-test" molecule test - I suspect the graph test might be affected by this - consider adding to that test as well. See: https://github.com/kiali/kiali-operator/blob/master/molecule/api-test/graph.test.yaml

@lucasponce lucasponce added this to In Review in Sprint 50 via automation Dec 11, 2020
@lucasponce lucasponce removed this from In Review in Sprint 49 Dec 11, 2020
@jshaughn
Copy link
Collaborator Author

You should run the "api-test" molecule test

@jmazzitelli OK, I finally ran the api-test molecule test and it passed because the test graph does have traffic on its edge, so the new default did not break the test.

jmazzitelli
jmazzitelli previously approved these changes Dec 14, 2020
Copy link
Collaborator

@jmazzitelli jmazzitelli left a comment

Choose a reason for hiding this comment

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

lgtm

pbajjuri20
pbajjuri20 previously approved these changes Dec 14, 2020
@pbajjuri20
Copy link
Contributor

pbajjuri20 commented Dec 15, 2020

Deployed and tested the new changes on "Idle Nodes" and "Idle Edges"

Idell_node

@jshaughn jshaughn merged commit 23e82cd into kiali:master Dec 16, 2020
Sprint 50 automation moved this from In Review to Done Dec 16, 2020
@ghost ghost added this to the v1.29.0 milestone Dec 16, 2020
@jshaughn jshaughn deleted the kiali#3519 branch December 16, 2020 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires UI PR A PR sent to the backend kiali/kiali requires changes on frontend kiali/kiali-ui
Projects
No open projects
Sprint 50
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants