Justin Chadwell
Justin Chadwell
As pointed out by @charjr and @vito, we probably should also add a `Container.exitCode` API alongside this, since now an exec has multiple possible exit codes, and it can be...
@helderco any ideas as to why python is failing here? ``` try: result = await resolver.get_result(self._converter, root, inputs) except Exception as e: > raise FunctionError(e) from e E dagger.mod._exceptions.FunctionError: Function...
In discord, I've had separate conversations with both @vito ([here](https://discord.com/channels/707636530424053791/1275216033862647890/1285283637599797268)) and @sipsma ([here](https://discord.com/channels/707636530424053791/911305510882513037/1286004869127213068)) about whether operations with an `exitCode` of non zero should cache or not. We should probably bikeshed...
> The array feels unnecessary. I previously put some points in https://github.com/dagger/dagger/pull/8466#issuecomment-2355244959 about this. Here's some reasons why it makes sense to me: - I have a step that will...
> Having to figure out what exit codes to catch in the array, by a painful process of trial and error, won't make the caching situation any better, will it?...
> I think we need a reference test case. How will you implement a go test which catches failed tests, to report them in a "report" object rather than a...
`Container.exitCode` is actually in here already - no need for a follow-up. --- Having had the weekend to think about this a bit more, I think I like your original...
Is there no value in having token caps as part of the permissions granted? I get that we have `maxApiCalls`/`maxTokens`, but that's still controlled by the caller of `dag.llm()`. If...
This can be done today using something like the following syntax: dagger -m github.com/vikram-dagger/daggerverse/fileutils call tree --dir=https://github.com/dagger/dagger#main:cmd/dagger This is the same syntax as found in dockerfiles - see https://docs.docker.com/build/building/context/#url-fragments. ---...
Potential proposal - could we "borrow" the go method of working out where the split is, and then also add our own version in as well (essentially `s/go/dagger`) for anyone...