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

Wrong handling of the global resourceList in combination with livesim #638

Closed
2 tasks
tanyaveksler opened this issue Dec 3, 2023 · 3 comments
Closed
2 tasks
Labels
bug Something isn't working live cluster parsers Anything related to processing inputs

Comments

@tanyaveksler
Copy link
Member

  • When both the global resourceList and the livesim missing resource contain the same resource kind (e.g., pods), the resources from the global resourceList are ignored.
    See example tests/istio_testcases/example_policies/sidecar_examples_w_onlineboutique/sidecars-and-gateways-test-scheme.yaml:
    when the resource ../online_boutique/new_online_boutique_manifests_istio/all_deployments.yaml appears only in global resourceList (remove it from the network config resourceList), then on the livesim run this resource is ignored, resulting in all the pods missing.

  • When none of global resourceList/podList/namespaceList exist, there is a (time consuming) attempt to load resources from live cluster, even if there is local resourceList/podList/namespaceList in a network config.
    Example - comment out the global resourceList in the test above.

@tanyaveksler tanyaveksler added bug Something isn't working parsers Anything related to processing inputs labels Dec 3, 2023
@zivnevo
Copy link
Member

zivnevo commented Dec 4, 2023

Can you compare this to Issue #593? Is it the same root-cause?

@tanyaveksler
Copy link
Member Author

Indeed, it looks that the current issue relates to the similar problem as described in Issue #593.
However, it seems that the current issue specifies the problem in more details.
Also, the second part of the current issue describes an additional problem related to livesim.

@zivnevo
Copy link
Member

zivnevo commented Aug 14, 2024

Moved to NP-Guard repo. See here

@zivnevo zivnevo closed this as not planned Won't fix, can't repro, duplicate, stale Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working live cluster parsers Anything related to processing inputs
Projects
None yet
Development

No branches or pull requests

2 participants