jon edvald 0487 or thsig 9024 any
c
calm-family-29323
03/31/2023, 1:32 PM@alert-helicopter-61082 or @swift-garage-61180 any chance for a quick pairing session today? I found a bug with versioning between action resolution stages that's causing problems down the line
The action spec validation[1] actually changes the spec in some cases (applies defaults etc).
It seems that in some cases version string is calculated before and sometimes after, resulting in different versions.
Can be reproduced by running
gdev exec backend 'echo hello'js
BaseAction.getFullVersion {
name: 'backend',
kind: 'Build',
resolved: false,
configVersion: 'v-b3c54e2cba',
versionString: 'v-ef871fb126'
}
BaseAction.getFullVersion {
name: 'backend',
kind: 'Deploy',
resolved: false,
configVersion: 'v-0932abf706',
versionString: 'v-013ea2e12a'
}
ResolveActionTask.process: validateSpec
BaseAction.getFullVersion {
name: 'backend',
kind: 'Build',
resolved: true,
configVersion: 'v-3a23be21fb',
versionString: 'v-343eafe5f7'
}
ResolveActionTask.process: validateSpec
BaseAction.getFullVersion {
name: 'backend',
kind: 'Deploy',
resolved: true,
configVersion: 'v-b7658cb3a4',
versionString: 'v-5ae88504ff'
}
compareDeployedResources { localManifestV: 'v-5ae88504ff', deployedV: 'v-5ae88504ff' }
compareDeployedResources { localManifestV: 'v-5ae88504ff', deployedV: 'v-5ae88504ff' }
compareDeployedResources { localManifestV: 'v-5ae88504ff', deployedV: 'v-5ae88504ff' }
BaseAction.getFullVersion {
name: 'backend',
kind: 'Deploy',
resolved: true,
configVersion: 'v-faf39fe647', // NEW CONFIGVERSION
versionString: 'v-fc876cd88e', // NEW VERSIONSTRING
}
compareDeployedResources { localManifestV: 'v-fc876cd88e', deployedV: 'v-5ae88504ff' }diff
variables: {
},
spec: {
+ buildArgs: {
+ },
+ dockerfile: "Dockerfile",
},
dependencies: [
],slack formatting is awesome
I'm signing off for today but I'd like to figure this out on Monday, will ping you then
a
alert-helicopter-61082
03/31/2023, 3:18 PMIt is expected that validation adds fields in some cases. There are scenarios where a resolved action should have a different version than the unresolved one. It's a little hard to read your log there, not sure when/where some of that is being logged.
Can you point to two cases where the version comes out different? A repro, basically
Then I can more easily dig in over the weekend
c
calm-family-29323
03/31/2023, 4:03 PMMaybe this is a bit more of a clear log
gdev exec backend 'echo hello'Running command echo hello in service undefined runner
ℹ garden → Running in environment local.default
ℹ providers → Getting status...
✔ providers → Cached (in 0.8 sec)
ℹ providers → Run with --force-refresh to force a refresh of provider statuses.
ℹ graph → Resolving actions and modules...
✔ graph → Done (in 0.1 sec)
ℹ build.backend → Build type=container name=backend is ready, nothing to do.
ℹ deploy.backend → Deploying version v-5ae88504ff...
ℹ deploy.backend → Deploying version v-5ae88504ff...
ℹ deploy.backend → Waiting for resources to be ready...
ℹ deploy.backend → Deployment/backend: Started container backend
ℹ deploy.backend → Resources ready
✔ deploy.backend → Done (in 5.2 sec)
ℹ deploy.backend → Ingress: http://demo-project.local.app.garden/hello-backend
trying to run exec in deploy.backend v-2dee648ec6
✖ Deploy type=container name=backend is not running@alert-helicopter-61082 I think ^ should be clear enough
The build.backend version changes causing a version in the deploy down the line in the middle of the exec command execution
a
alert-helicopter-61082
04/03/2023, 11:45 AMYup, that should be enough, thanks
Solved here: https://github.com/garden-io/garden/pull/4040
Turns out the configuration was being modified by the plugin handler