Commit bf33fce
fix(ci): remove faiss metadata_store from auth workflow to fix table creation race
The faiss provider's metadata_store config (using sql_default) caused its
initialize() to trigger _ensure_engine() via a fetch_all before the
responses provider had registered its table metadata. Since all sql_default
stores share a single SqlAlchemySqlStoreImpl, metadata.create_all() ran
without the responses table, and the lazy _ensure_engine() guard
(if self._engine is not None: return) prevented it from ever being created.
Removing metadata_store from faiss (matching the passing branch config)
ensures no provider triggers _ensure_engine() before all tables are
registered. The vector store access control tests still work through the
routing table ABAC layer (vector_store::*).
Verified locally: responses table created successfully, vector store
CRUD operations work.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>1 parent b3016a5 commit bf33fce
1 file changed
Lines changed: 0 additions & 6 deletions
Lines changed: 0 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
132 | 132 | | |
133 | 133 | | |
134 | 134 | | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | 135 | | |
139 | 136 | | |
140 | 137 | | |
| |||
170 | 167 | | |
171 | 168 | | |
172 | 169 | | |
173 | | - | |
174 | | - | |
175 | | - | |
176 | 170 | | |
177 | 171 | | |
178 | 172 | | |
| |||
0 commit comments