Skip to content

Commit 30cdbfa

Browse files
authored
fix: infinite store reconcile (#140)
1 parent 6bdc4ba commit 30cdbfa

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

internal/subroutine/authorization_model.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ const (
3333

3434
var (
3535
privilegedGroupVersions = []string{"rbac.authorization.k8s.io/v1"}
36-
groupVersions = []string{"authentication.k8s.io/v1", "authorization.k8s.io/v1", "v1"}
36+
groupVersions = []string{"authentication.k8s.io/v1", "authorization.k8s.io/v1", "v1"}
3737

3838
privilegedTemplate = template.Must(template.New("model").Parse(`module internal_core_types_{{ .Name }}
3939
@@ -195,8 +195,8 @@ func (a *authorizationModelSubroutine) Process(ctx context.Context, instance run
195195
return ctrl.Result{}, errors.NewOperatorError(err, true, false)
196196
}
197197

198-
// Compare Proto objects directly instead of DSL strings to avoid ordering issues
199-
// The two models should be semantically equivalent even if DSL ordering differs
198+
res.AuthorizationModel.Id = ""
199+
200200
currentRaw, err := protojson.Marshal(res.AuthorizationModel)
201201
if err != nil { // coverage-ignore
202202
log.Error().Err(err).Msg("unable to marshal current model")

0 commit comments

Comments
 (0)