Merge pull request #729 from juanfont/fix-reuse-of-ns

Minor fix to linting issue introduced when fixing excludeCorrectlyTaggedNodes (#707)
This commit is contained in:
Juan Font 2022-08-11 15:01:53 +02:00 committed by GitHub
commit 73cd428ed2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -421,9 +421,9 @@ func excludeCorrectlyTaggedNodes(
) []Machine { ) []Machine {
out := []Machine{} out := []Machine{}
tags := []string{} tags := []string{}
for tag, ns := range aclPolicy.TagOwners { for tag := range aclPolicy.TagOwners {
owners, _ := expandTagOwners(aclPolicy, namespace, stripEmailDomain) owners, _ := expandTagOwners(aclPolicy, namespace, stripEmailDomain)
ns = append(owners, namespace) ns := append(owners, namespace)
if contains(ns, namespace) { if contains(ns, namespace) {
tags = append(tags, tag) tags = append(tags, tag)
} }