Skip to content

Commit

Permalink
Merge pull request #4014 from tonistiigi/fix-policy-rule-order
Browse files Browse the repository at this point in the history
llbsolver: fix policy rule ordering
  • Loading branch information
tonistiigi committed Jul 12, 2023
2 parents 6c3bdd4 + 22d8446 commit 18fc875
Showing 1 changed file with 5 additions and 11 deletions.
16 changes: 5 additions & 11 deletions solver/llbsolver/solver.go
Original file line number Diff line number Diff line change
Expand Up @@ -981,27 +981,21 @@ func loadEntitlements(b solver.Builder) (entitlements.Set, error) {
}

func loadSourcePolicy(b solver.Builder) (*spb.Policy, error) {
set := make(map[spb.Rule]struct{}, 0)
var srcPol spb.Policy
err := b.EachValue(context.TODO(), keySourcePolicy, func(v interface{}) error {
x, ok := v.(spb.Policy)
if !ok {
return errors.Errorf("invalid source policy %T", v)
}
for _, f := range x.Rules {
set[*f] = struct{}{}
r := *f
srcPol.Rules = append(srcPol.Rules, &r)
}
srcPol.Version = x.Version
return nil
})
if err != nil {
return nil, err
}
var srcPol *spb.Policy
if len(set) > 0 {
srcPol = &spb.Policy{}
for k := range set {
k := k
srcPol.Rules = append(srcPol.Rules, &k)
}
}
return srcPol, nil
return &srcPol, nil
}

0 comments on commit 18fc875

Please sign in to comment.