#๐ฒ v0.12.2 - 22nd July 2024
1 messages ยท Page 1 of 1 (latest)
๐ฒ v0.12.2 - 22nd July 2024
cc @young wasp @quasi garden @old stump @plush condor
Yes, this fix is ready: https://github.com/dagger/dagger/pull/8000
Just reading over https://github.com/dagger/dagger/compare/v0.12.1...main it looks like the only other significant change is https://github.com/dagger/dagger/pull/7824 - I guess that's OK?
It might be worth a double check that our docs don't need any updating to account for that one. Seems possible we could somewhere explain the directories created after dagger init which might need to be fixed now?
Unless @old stump @plush condor already covered that, just a drive-by thought
Hm that reminds me of https://github.com/dagger/dagger/pull/7938 (I hadn't considered user modules being able to hit this case, so didn't think of it as particularly important, but I guess it's possible)
๐ Cherry-picked out of #7438, since that one is kinda stalled, not really sure what to do with it.
In the process of the above PR, I'd attempted to add a required enum arg, which the typescript...
Not urgent, it also needs a test case
Yeah good point. I'll go see if I can find any spots that need updating
Just saw this new error on the python sdk provisioning test on main https://github.com/dagger/dagger/actions/runs/10045043161/job/27761780836#step:5:17 @plush condor any idea what's going wrong?
Yeah, that file was removed. Need to check where it's being referenced from.
follow-up to #7824
Went for minimal changes here to maintain correctness. I think each of these makes sense in context but could use a double check.
Gonna move leftovers in the v0.12.2 milestone (https://github.com/dagger/dagger/milestone/53) to v0.12.3 (https://github.com/dagger/dagger/milestone/54)
@quasi garden Did you want to get this in? https://github.com/dagger/dagger/pull/7943
Can anyone help with this? https://github.com/dagger/dagger/blob/2d6390b1d673f8ac7f53a76a61b049017c04b78f/.github/workflows/engine-and-cli-publish.yml#L326-L335
I'm on family time and I'm not familiar enough with GHA that'll require me to research how to do this, so I'll just describe what needs to happen in case I can't get it done, someone else can.
- uses: actions/setup-python@v4
with:
python-version: "3.11"
cache: "pip"
cache-dependency-path: "sdk/python/requirements.txt"
- name: "Test Python SDK"
run: |
cd sdk/python
pip install -r requirements.txt .
pytest -xm provision
Basically, Python is no longer needed to be installed, nor pip install -r requirements.txt. This is what needs to happen:
- name: "Test Python SDK"
run: |
cd sdk/python
uv run pytest -xm provision
Instead of uses: actions/setup-python@v4 we need something with uv installed:
# it's installed on $HOME/.cargo/bin
curl -LsSf https://astral.sh/uv/0.2.27/install.sh | sh
Or from docker image if better (ghcr.io/astral-sh/uv), or homebrew on mac if it's available there.
cache-dependency-path would be sdk/python/uv.lock.
EDIT: Also, sdk/python/requirements.txt should be deleted. It's only referencing requirements-dev.lock which doesn't exist anymore.
That fix is for our test flakes (technically a user could hit it, but it's outrageously obscure and no user has hit it that we know of despite existing since basically forever). The fix is just complicated enough (creates new state files on client hosts to give them a persistent unique id) that I'd say lets just skip it for this patch release and get merged asap after. Not quite worth the risk, would prefer a more thorough review on it.
Will look into this, thanks for the pointers
did find this: https://github.com/marketplace/actions/setup-uv - but maybe overkill?
May be ok! But just needs this part:
- uses: yezz123/setup-uv@v4
with:
uv-version: "0.2.27"
No need for the setup-python above since these latest uv versions can manage Python already.
Opened https://github.com/dagger/dagger/pull/8003 but not sure if this is a verify-after-merging issue or if it'll be tested by the PR
I think it'll only run provision tests on main. Let's merge and iterate if needed.
There's also this one https://github.com/hynek/setup-cached-uv, but seems like in some cases fetching the cache takes longer than just reinstalling on a clean env.
engine lint is unhappy though: https://github.com/dagger/dagger/actions/runs/10046365444/job/27765656933 - not sure what's going on there
hmm. gonna try re-running I guess, it's green on previous commits, no idea what it would have to do with this one
yep, passed on a rerun 
think we're good to go now
https://github.com/dagger/dagger/pull/8005 cc @quasi garden whenever you have a chance
all ๐ฒ (and not just because of the bug) - tagging!
https://github.com/dagger/dagger/releases/tag/v0.12.2 - moving on to SDKs now (...er, some publish stuff still going, waiting a bit more)
@quasi garden looks like the bump PR creation succeeded ๐ https://github.com/dagger/dagger/pull/8006
Docker setup failed on macOS: https://github.com/dagger/dagger/actions/runs/10047948096/job/27770815514?pr=8006 - suppose I'll re-run
Looking good now
Yeah that one is very annoying, issue here fwiw: https://github.com/dagger/dagger/issues/7980
Pushing Go SDK tag now (oh, after waiting for ๐ฒ on main)
TestPythonProjectLayout + TestLegacyExportAbsolutePath flakes, rerunning: https://dagger.cloud/dagger/traces/fa300a99dc706ae97951869415326cbf
Gonna avoid blocking on that and just go ahead and start tagging
Go SDK released, moving on to Python
Huh. Got past the SDKs, but for some reason the Helm version didn't get bumped in helm/dagger/Chart.yaml, not sure where/how that's supposed to happen.
@quasi garden's engine bump commit for v0.12.1 has this change: https://github.com/dagger/dagger/commit/78fd31cf2cfb5c010c7b65ddf27fdc4eedf3450b#diff-5d1b3093eee42b4e50b00a1a1a8cce2d60d6be6d1998976e4320545d4f0ae177
But mine doesn't - only the changelogs: https://github.com/dagger/dagger/commit/1ed939590738c349bcb34afc2620fd5b1a2e5547#diff-6df9c40f2229da7eeb25929e41a720500c42a7ab39829c0006e113a80ac2f389
The only mention of helm before this point in RELEASING.md is the changie stuff, which was done (confirmed in my shell history)
Oh weird, that is supposed to get picked up in the diff generated for the bump engine PR via this: https://github.com/sipsma/dagger/blob/1ed939590738c349bcb34afc2620fd5b1a2e5547/.github/workflows/engine-and-cli-publish.yml#L136-L136
But since that automated PR didn't work the last two releases I'm guessing it's not getting picked up as expected and wasn't noticed until now
Ah it's probably this: https://github.com/sipsma/dagger/blob/1ed939590738c349bcb34afc2620fd5b1a2e5547/.github/workflows/engine-and-cli-publish.yml#L141
Just needs to include ./helm/
I'll add that to https://github.com/dagger/dagger/pull/8007
Tagged the wrong commit (v0.12.2 release, needed my Helm bump followup) for helm/chart/v0.12.2 so pushed a new one, watching https://github.com/dagger/dagger/actions/runs/10049008129/job/27774326909 now
aaaand it finished before I was done typing that
Deploying docs now, and poking @proud patrol @gusty kindle @quasi garden since Playground + Daggerverse + Cloud are bump-able ๐
Also install.sh and install.ps1 - it mentions to ping Gerhard but he's PTO, not sure how to bump the S3 bucket myself
Did install.sh/ps1 change in this release?
If not then nothing should be needed
oh, guess not, I guess it doesn't hardcode a version?
Yeah it's not versioned itself, so should be good this time
PR for the Action: https://github.com/dagger/dagger-for-github/pull/139 cc @shy hearth ๐ (haven't done the tagging step, not 100% sure I understand it yet, seems to propose tagging before the PR is even merged? - https://github.com/dagger/dagger/blob/main/RELEASING.md#-dagger-for-github--10mins)
I'll look. No, I cut a release with the full tag using the auto-release notes that GitHub produces and then I move the major tag to point to that same commit, e.g. v6
Homebrew PR is approved, but looks like it flaked: https://github.com/Homebrew/homebrew-core/pull/178139
Updated the final PR: https://github.com/dagger/dagger/pull/8007
That's a wrap! (Modulo Daggerverse/Playground bump etc., will leave the thread open for now)
Improving these notes on the road to automating them: https://github.com/dagger/dagger/pull/8017
first try