Skip to content

Commit

Permalink
🧹 Remove old resolver (#1516)
Browse files Browse the repository at this point in the history
* 🧹 Remove old resolver

This removes the old resolver and uses the rewrite everywhere.

Should fix #1515

* fix tests
  • Loading branch information
jaym authored Jan 10, 2025
1 parent cca68a9 commit f5aaceb
Show file tree
Hide file tree
Showing 6 changed files with 77 additions and 3,269 deletions.
8 changes: 2 additions & 6 deletions policy/bundle.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,11 +102,7 @@ func BundleExecutionChecksum(ctx context.Context, policy *Policy, framework *Fra
// So far the checksum only includes the policy and the framework
// It does not change if any of the jobs changes, only if the policy or the framework changes
// To update the resolved policy, when we change how it is generated, change the incoporated version of the resolver
if IsNextGenResolver(ctx) {
res = res.Add(RESOLVER_VERSION_NG)
} else {
res = res.Add(RESOLVER_VERSION)
}
res = res.Add(RESOLVER_VERSION)

return res.String()
}
Expand Down Expand Up @@ -624,7 +620,7 @@ func (c *bundleCache) removeFailing(res *Bundle) {
group := policy.Groups[j]
group.Queries = explorer.FilterQueryMRNs(c.removeQueries, group.Queries)
group.Checks = explorer.FilterQueryMRNs(c.removeQueries, group.Checks)
if len(group.Queries)+len(group.Checks) > 0 {
if len(group.Policies)+len(group.Queries)+len(group.Checks) > 0 {
groups = append(groups, group)
}
}
Expand Down
Loading

0 comments on commit f5aaceb

Please sign in to comment.