844600078504951838 #spicedb

Channels

spicedb

test-forum

zanzibar

user

04/18/2022, 9:04 PM

not with an env var

cjs

04/18/2022, 9:04 PM

It's not earth shaking, but surprising. Maybe try to parse the endpoint and if it's not a valid URL, disable it?

user

04/18/2022, 9:05 PM

I think it makes sense to just toggle AllowEmpty in cobrautil, does it not?

cjs

04/18/2022, 9:05 PM

Makes sense to me, not exactly sure what "For backwards compatibility reasons" actually affects.

cjs

04/18/2022, 9:07 PM

@User did a quick search, I think it's ok to set it: https://github.com/spf13/viper/pull/440#issuecomment-436356307

user

04/18/2022, 9:08 PM

https://github.com/jzelinskie/cobrautil/pull/10

user

04/18/2022, 9:10 PM

you're running a build with a pinned version of cobrautil already right?

user

04/18/2022, 9:10 PM

bump it and lmk if that's working well and i'll cut a release with that & the new otel stuff

cjs

04/18/2022, 9:10 PM

Yep. Going to pull it in now.

cjs

04/18/2022, 9:45 PM

I've tested locally and it's all good. I can't get a deploy done for a little while because of reasons but I think it's fine.

user

04/18/2022, 11:09 PM

Cool, will cut a release and open a PR against SpiceDB tomorrow!

romil

04/19/2022, 10:59 AM

I see. there is an update in the K8 file on git hub https://raw.githubusercontent.com/authzed/spicedb/main/k8s/example.yaml

# - name: "SPICEDB_DISPATCH_UPSTREAM_ADDR"
            #   value: "kubernetes:///spicedb:dispatch"

is this env required to enable dispatcher ? or as mentioned in the doc https://authzed.com/blog/consistent-hash-load-balancing-grpc/, one needs to pass the below arg:

containers:
        - name: "spicedb"
          image: "quay.io/authzed/spicedb:v1.6.0"
          imagePullPolicy: "IfNotPresent"
          command: ["spicedb", "serve"]
          args:
            - grpc-no-tls
            - --http-enabled
            - --dispatch-upstream-addr=kubernetes:///spicedb.<namespace>:50053

williamdclt

04/19/2022, 11:05 AM

@romil you can configure spicedb via either env vars or cli flags. It's explained in the readme but not in the docs (would be worth adding imo)

romil

04/19/2022, 11:08 AM

Once the said file https://raw.githubusercontent.com/authzed/spicedb/main/k8s/example.yaml is applied, I enabled port-forward on one of the pod, but unable to get the check api working , all delete , create api works fine. Am I missing something? the request just times out - name: "SPICEDB_DISPATCH_UPSTREAM_ADDR" value: "kubernetes:///spicedb:dispatch" is "dispatch" as per the sample example file here refers to namespace ?

williamdclt

04/19/2022, 12:52 PM

Looks like your dispatch cluster is misconfigured. You can read about it in the docs or maybe the readme or maybe the authzed blog

Jake

04/19/2022, 12:56 PM

the

dispatch

here refers to a named port on the kubernetes service, and

spicedb

refers to the service name

romil

04/19/2022, 4:13 PM

the

dispatch

here refers to a named

cjs

04/20/2022, 2:47 PM

Consistency

is not specified for a

CheckPermission

request, is

fully_consistent

used or

minimize_latency

Joey

04/20/2022, 2:47 PM

Minimize latency

cjs

04/20/2022, 2:49 PM

Ah. The doc at https://docs.authzed.com/reference/zedtokens-and-zookies#using-the-stored-zedtoken is a little confusing.

DefinitelyNotSam

04/21/2022, 3:00 PM

Are there version incompatibilities across releases? I went to upgrade to v1.6.0 from (i believe) v1.2.0 and got this error after attempting to create a relation

ERROR: relation "relationship_estimate_counters" does not exist (SQLSTATE 42P01)

Joey

04/21/2022, 3:02 PM

@DefinitelyNotSam you need to run

spicedb migrate

to add the new tables

DefinitelyNotSam

04/21/2022, 3:02 PM

Thanks! Where can I find the docs on this?

Joey

04/21/2022, 3:03 PM

appears to have gotten lost in the shuffle... I'll get that fixed today

DefinitelyNotSam

04/21/2022, 3:04 PM

Thanks!

DefinitelyNotSam

04/21/2022, 3:06 PM

if the revision for the new version or the old version?

Joey

04/21/2022, 3:07 PM

spicedb migrate head

DefinitelyNotSam

04/21/2022, 3:08 PM

Thanks! BTW, I saw you guys added MySQL. Is that a fully supported version?

Jake

04/21/2022, 3:09 PM

not quite yet, we're still ironing out some migration issues and things

Jake

04/21/2022, 3:10 PM

but it should be soon™