playground/play-with-dagger/dagger-inter... | Dagger | Page 1

midnight jacinth Jan 15, 2024, 8:17 PM

#

looks like a camel-casing issue thinkspin it has a loadCarIcarFromID field, but something is trying to query loadCarICarFromID.

#

wonder if this is fixed on main with all the DagQL changes

tardy birch Jan 15, 2024, 8:18 PM

#

I tried to use dagger from main, same issue unfortunately 😦

#

┃ error: parse selections: parse field "loadCarICarFromID": Query has no such field: "loadCarICarFromID"

The error is way more verbose tho

#

📎 message.txt

midnight jacinth Jan 15, 2024, 8:21 PM

#

huh, I get this:

❯ with-dev dagger call race
✔ dagger call race [2.42s]
┃ <nil>
• Engine: 61eb86cf77e3 (version devel ())
⧗ 6.51s ✔ 96

tardy birch Jan 15, 2024, 8:32 PM

#

OH

#

hmm

#

what binary version?

#

I'll try to prune everything, compile another dagger binary and retry

#

@midnight jacinth Idk what is your dagger binary version but I pull from main and tried directly from ./hack/dev, same issue

./hack/dev dagger -m /Users/tomchauveau/Documents/playground/play-with-dagger/dagger-interfaces/race call race

and the error still the same

#

How is it possible that it works for you 😮

midnight jacinth Jan 15, 2024, 8:53 PM

#

hmmmmmmmmm maybe try deleting the container + volume?

#

docker rm -f dagger-engine.dev; docker volume rm dagger-engine.dev is my routine these days

tardy birch Jan 15, 2024, 8:53 PM

#

Already did but I'll try one more time

midnight jacinth Jan 15, 2024, 8:54 PM

#

worth noting docker rm -fv never removes the volume for me

#

always have to do a separate docker volume rm

tardy birch Jan 15, 2024, 8:54 PM

#

I do it from the docker desktop haha

#

I'll record a video of the repro

#

Oups video too big to be uploaded...

#

Same issue tho

#

I'm sooo confused

#

I also removed the images

#

➜  dagger git:(main) git pull
Already up to date.
➜  dagger git:(main) docker volume ls   
DRIVER    VOLUME NAME
➜  dagger git:(main) docker container ls
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
➜  dagger git:(main) docker images ls   
REPOSITORY   TAG       IMAGE ID   CREATED   SIZE

midnight jacinth Jan 15, 2024, 9:03 PM

#

oh wait it happens to me now

tardy birch Jan 15, 2024, 9:03 PM

#

Hm

#

Weird

#

So it worked and now it doesn't?

#

Did we break something on main?

midnight jacinth Jan 15, 2024, 9:03 PM

#

nah im guessing my dev image was out of date somehow

tardy birch Jan 15, 2024, 9:04 PM

#

But your dev image was working haha

midnight jacinth Jan 15, 2024, 9:04 PM

#

...true lol

#

forgot it was broken on main/v0.9.6 before

tardy birch Jan 15, 2024, 9:05 PM

#

/opt/homebrew/bin/dagger version  
dagger v0.9.6 (registry.dagger.io/engine) darwin/arm64
➜  race git:(main) ✗ /opt/homebrew/bin/dagger call race
✘ dagger call race ERROR [1.05s]
┃ Error: response from query: input:1: race.race failed to get function output directory: process "/runtime" did not com
┃ plete successfully: exit code: 2                                                                                      
✘ race ERROR [1.05s]
✘ exec /runtime ERROR [1.04s]
┃ input:1: car.withCar failed to get function output directory: process "/runtime" did not complete successfully: exit c
┃ e: 2                                                                                                                  
✘ withCar(car: "moddata:sha256:bacac5f6a8f7da4de335667c3447ba0ceb3ecc4b15365e7486bb7eeae7e9721f:Audi:eyJDb2xvciI6InJlZCIsIk1vZGVsIjoiQTQiLCJTcGVlZCI6MTAwfQ==") ERROR [0.33s]
✘ exec /runtime ERROR [0.32s]
┃ json: error calling MarshalJSON for type main.ICar: input:1: Cannot query field "loadCarICarFromID" on type "Query". D
┃  you mean "loadServiceFromID", "loadCarIcarFromID", "loadCarFromID", "loadSecretFromID", or "loadTypeDefFromID"?      
• Engine: d2ff46c9dbd3 (version v0.9.6)

0.9.6 broken too yeah

#

Error is sexier tho

midnight jacinth Jan 15, 2024, 9:06 PM

#

wiping everything and trying again

#

yeah suggesting other fields like 'did you mean <x>' is a nice touch

#

ok still fails after a reset

tardy birch Jan 15, 2024, 9:08 PM

#

Same for me

tardy birch Jan 15, 2024, 9:08 PM

#

midnight jacinth yeah suggesting other fields like 'did you mean <x>' is a nice touch

It's nice but the field asked is correct xD

#

Cannot query field "loadCarICarFromID" on type "Quer
┃ . Did you mean "loadCarFromID", "loadServiceFromID", "loadCarIcarFromID"

^ loadCarICarFromID exists xD

midnight jacinth Jan 15, 2024, 9:09 PM

#

the c is lowercase tho

tardy birch Jan 15, 2024, 9:09 PM

#

Only

#

Holy*

midnight jacinth Jan 15, 2024, 9:09 PM

#

loadCarICarFromID vs loadCarIcarFromID

tardy birch Jan 15, 2024, 9:09 PM

#

Ok 🚀
1 - I need new glasses
2 - Is it fixed on main? Since the error is different?

midnight jacinth Jan 15, 2024, 9:10 PM

#

not fixed, we just have a new custom-built graphql server implementation so the errors coming out are different

#

(dagql)

tardy birch Jan 15, 2024, 9:10 PM

#

Make sense

#

So something's broken on Go side right?

#

Because I see the tests are passing, and our Iface test are way more complex than my little example

#

So I don't really understand why it's breaking :/

midnight jacinth Jan 15, 2024, 9:11 PM

#

it's probably just that none of the tests specifically cover this type of camel casing issue

tardy birch Jan 15, 2024, 9:12 PM

#

If it's just a naming issue, I can change my example (I just want it to work so I can work on the TypeScript SDK)

midnight jacinth Jan 15, 2024, 9:12 PM

#

yeah that's worth a shot

#

i'm looking into the issue now

tardy birch Jan 15, 2024, 9:14 PM

#

Ok you were right

#

█     │ [0.36s] exec /runtime
┃     │ A blue Bmw model M5 is driving at 120kmh..                                                                 
█◀────╯ CACHED exec /usr/local/bin/codegen --module . --propagate-logs=true --introspection-json-path /schema.json
█ CACHED exec go build -o /runtime .
█ [0.12s] exec /runtime
┃ A red Audi model A4 is driving at 100kmh..                                                                       
┻ 
• Engine: 52abe9205d1a (version v0.9.6)

#

Issue comes from the case

#

Funny lol

midnight jacinth Jan 15, 2024, 9:14 PM

#

nice

#playground/play-with-dagger/dagger-inter...