kemko/nomad
1
0
Fork 0
mirror of https://github.com/kemko/nomad.git synced 2026-01-01 16:05:42 +03:00
Code Issues Packages Projects Releases Wiki Activity

state: fix eval list by prefix with * namespace (#13551)

Browse Source
This commit is contained in:
Luiz Aoqui
2022-07-07 14:21:51 -04:00
committed by GitHub
parent 52389ff726
commit d9a0d64e77
3 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/13551.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
state: Fix listing evaluations from all namespaces
```

3
nomad/state/state_store.go
View File

@@ -3294,7 +3294,8 @@ func evalNamespaceFilter(namespace string) func(interface{}) bool {
return true
}
return eval.Namespace != namespace
return namespace != structs.AllNamespacesSentinel &&
eval.Namespace != namespace
}
}

4
nomad/state/state_store_test.go
View File

@@ -4636,11 +4636,15 @@ func TestStateStore_EvalsByIDPrefix_Namespaces(t *testing.T) {
require.NoError(t, err)
iter2, err := state.EvalsByIDPrefix(ws, ns2.Name, sharedPrefix, SortDefault)
require.NoError(t, err)
iter3, err := state.EvalsByIDPrefix(ws, structs.AllNamespacesSentinel, sharedPrefix, SortDefault)
require.NoError(t, err)
evalsNs1 := gatherEvals(iter1)
evalsNs2 := gatherEvals(iter2)
evalsNs3 := gatherEvals(iter3)
require.Len(t, evalsNs1, 1)
require.Len(t, evalsNs2, 1)
require.Len(t, evalsNs3, 2)
iter1, err = state.EvalsByIDPrefix(ws, ns1.Name, eval1.ID[:8], SortDefault)
require.NoError(t, err)