Skip to content

fix: agentless mode, process drift reported by clusters in pull mode - #2074

Merged
gianlucam76 merged 1 commit into
projectsveltos:mainfrom
gianlucam76:agentless-pull-mode-resourcesummary
Oct 29, 2026
Merged

gianlucam76 merged 1 commit into
projectsveltos:mainfrom
gianlucam76:agentless-pull-mode-resourcesummary

Conversation

@gianlucam76

Copy link
Copy Markdown
Member

In pull mode, sveltos-applier creates in the management cluster the ResourceSummary used to report the drift detected in the managed cluster. It only sets the ClusterSummary annotations. In agentless mode, collectAndProcessAllResourceSummaries ignored any changed ResourceSummary without the cluster name and type labels, so the drift of a cluster in pull mode was reported but never remediated.

When the labels are missing, get the cluster from the ClusterSummary referenced by the annotations, and process the ResourceSummary only if the cluster is in pull mode. Any other ResourceSummary without labels is still ignored.

In pull mode, sveltos-applier creates in the management cluster the ResourceSummary used to
report the drift detected in the managed cluster. It only sets the ClusterSummary
annotations. In agentless mode, collectAndProcessAllResourceSummaries ignored any changed
ResourceSummary without the cluster name and type labels, so the drift of a cluster in pull
mode was reported but never remediated.

When the labels are missing, get the cluster from the ClusterSummary referenced by the
annotations, and process the ResourceSummary only if the cluster is in pull mode. Any other
ResourceSummary without labels is still ignored.
@gianlucam76
gianlucam76 force-pushed the agentless-pull-mode-resourcesummary branch from 6aad628 to 781af9f Compare October 29, 2026 19:23
@gianlucam76
gianlucam76 merged commit 7a79765 into projectsveltos:main Oct 29, 2026
12 checks passed
@gianlucam76
gianlucam76 deleted the agentless-pull-mode-resourcesummary branch October 29, 2026 20:03
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.

1 participant