Skip to content

code refactor: fix go lint and improve graph query strategy.#22

Merged
anandf merged 20 commits into
anandf:mainfrom
Mangaal:fix-refactor
Feb 23, 2026
Merged

code refactor: fix go lint and improve graph query strategy.#22
anandf merged 20 commits into
anandf:mainfrom
Mangaal:fix-refactor

Conversation

@Mangaal

@Mangaal Mangaal commented Feb 10, 2026

Copy link
Copy Markdown
Contributor
  • Default strategy is now Dynamic: Cyphernetes/graph remains available, but dynamic is used by default for more consistent results. Cyphernetes relationship query can fail on some OpenShift cluster resources, which prevents retrieving nested child resources. Dynamic analysis works for those cases.
  • Graph query cleanup: removed the pre-loaded/default resource set for graph traversal to avoid pulling unrelated resources and to reduce noise.
  • OpenShift-specific behaviour: OpenShift relationship rules are added in Cyphernetes query only when the destination cluster is detected as OpenShift.
  • Removed unused custom Argo CD “app ownership” rule for graph query. We rely on repo-server to fetch application child manifests, so the Argo app ownership rule is not needed.

Mangaal added 20 commits March 20, 2025 12:56
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>
Signed-off-by: Mangaal <angommeeteimangaal@gmail.com>

@anandf anandf left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM

@anandf anandf merged commit 337a98e into anandf:main Feb 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants