as to your error - that is a very weird one. Can y...
# spicedbv
vroldanbet
11/29/2022, 3:05 PMas to your error - that is a very weird one. Can you share the ENVs of your SpiceDB
Deploymente
ensonic
11/29/2022, 3:12 PMv
vroldanbet
11/29/2022, 3:13 PMlike I tested the PR locally, and check requests seem to work
vroldanbet
11/29/2022, 3:13 PMlet me have a look
vroldanbet
11/29/2022, 3:14 PMdispatch is still set, that's not what the PR does
vroldanbet
11/29/2022, 3:14 PM Copy code
SPICEDB_DISPATCH_UPSTREAM_ADDR: kubernetes:///dev.default:dispatchvroldanbet
11/29/2022, 3:14 PMthe PR fixes making sure that ENV is not set at all
e
ensonic
11/29/2022, 3:22 PMI've just rebuild from your PR, repushed the image and deleted dev-spicedb deployment and recreated it and the env is set
ensonic
11/29/2022, 3:23 PMI'll add some logging to your change to double check that I am running the changed code
ensonic
11/29/2022, 3:39 PMIt works now!
ensonic
11/29/2022, 3:39 PM"kubectl set image" only restarts the pod if the image name changed :/
ensonic
11/29/2022, 3:40 PMsince iI first build from head and then from your PR the 2nd round had no effect
v
vroldanbet
11/29/2022, 3:45 PMglad to hear! there is usually a lot of moving parts in a kube setup and is easy to miss those
e
ensonic
11/29/2022, 3:46 PMsorry for the noise. Hope this can be merged soon. But at least I can test now
v
vroldanbet
11/29/2022, 3:49 PMnot at all! you are helping the projects get better for the whole community, so we appreciate you bringing those rough edges up ❤️
2 Views