#๐Ÿš€ v0.11.7 - 11th June 2024

1 messages ยท Page 1 of 1 (latest)

dreamy linden
#

Hallo all ๐Ÿ™‚ just going to start a release thread, since we've not done one this week.

I'd like to aim to release something next week, though not sure what. It feels like we're getting close to a v0.12.0, with some larger changes in the pipeline (lots of api breaking things we'd like to do, and maybe the telemetry changes that @brazen wave, depends on how breaky that is) is working on.

That said, not sure what's "ready". I think maybe we aim for at least one more release in the v0.11.X series, and then, aim to do a bunch of breaking changes/etc for v0.12.0?

For context:

#

cc @brazen wave @bitter seal @winged oak @frail idol @formal dust @left willow @sick dragon

#

If we're going for v0.11.7 next week, I'd probably say we should aim for Tuesday/Wednesday.

brazen wave
#

@bitter seal @dreamy linden Done addressing all feedback on https://github.com/dagger/dagger/pull/7452! thanks again ๐Ÿ™

As @dreamy linden mentioned it would be good to have https://github.com/dagger/dagger/pull/7031 in the same release since the OTel changes will make an outdated client hang. (Looked into a fix for backwards compat. and it's a bit complicated, wouldn't want to keep it for long, so kind of just delaying the inevitable.) Taking a look at that now but getting near EOD

bitter seal
#

I'd really like to get https://github.com/dagger/dagger/pull/7315 in too. Needs to come after OTEL and is another doozy of a PR but will be good to review once OTEL is in main (currently has all the commits from Alex's PR).

It also creates incompatibilities with old CLIs (due to changes to the engine's API it serves, just pure http now rather than insane grpc tunneling), so same story of ideally having 7031

dreamy linden
#

โœจ v0.11.7 - 11th June 2024

#

cool! let's go for a v0.11.7 release, and intentionally break our client-server incompat ๐Ÿ˜„ i'll revive 7031, and add some tests ๐ŸŽ‰

dreamy linden
#

๐Ÿ—’๏ธ note! please add your issues/prs to the milestone! if they're on the milestone, they'll make it into the release, and the release will block until we can get them fixed/merged ๐Ÿ™‚
also please add any release notes if applicable (it saves me so much time)

dreamy linden
#

woo ๐ŸŽ‰ finally rebased https://github.com/dagger/dagger/pull/7031 (and added some automated tests) - from manual testing as well, this works all the way back to v0.10.X, which is kinda neat ๐Ÿ˜„
(potentially we might also be able to deprecate+remove checkVersionCompatibility for v0.12 as well ๐ŸŽ‰)

GitHub

See the rationale in #7013, #7001 (comment). TL;DR, during the release of v0.11.0, we broke compatibility with older clients, but didn't really have the tooling to make sure that this errored c...

bitter seal
#

Just FYI that afaict right now all TS module users (including those on any previous release) are broken atm unless they happen to have cache for the npm install -g tsx step in the TS SDK: #maintainers message

#

Options are either A) open an issue on the tsx repo and hope they fix it or B) do an engine release ASAP

bitter seal
#

Thought of an option C, which is a wrapper of the TS SDK that works as a git ref and can be used in the meantime before we do the release: #maintainers message So unless there's any objections I say we go with that (I'll post to the TS channel and such) and just do the engine release ASAP early next week

brazen wave
bitter seal
bitter seal
#

Also, we can postpone https://github.com/dagger/dagger/pull/7315 for the subsequent release in the interest of getting it all out faster, was nice to have but not worth blocking on given it will take some time to review

bitter seal
dreamy linden
#

i keep thinking i will get to it, and it never seems to happen

dreamy linden
#

going to cut the release soon

hot rivet
dreamy linden
#

awesome, will pull this fix into the release

dreamy linden
#

tagged v0.11.7

dreamy linden
#

going to keep going, and get all the sdks out

#

retriggered the jobs, it could be a flake (doesn't seem to be)

#

fyi @brazen wave @bitter seal

bitter seal
#

Will be online in a few, does this happen at all locally when provisioning the new engine version the first time?

dreamy linden
#

we seem to hit it in the test/lints, looks like it's at the end of a job related to that

#

we bumped to the latest buildkit version - could maybe be related to that

#

or maybe something service-y again

#

@hot rivet @pine oyster we should be good to update the playground

dreamy linden
#

we seem to not be getting the end of the initialize span somehow

bitter seal
#

Seeing if I can repro locally, pulled the improve releasing PR locally and running those sdk lint jobs, no hangs so far

dreamy linden
#

turned on the cloud token for those "waiting steps", kinda curious what the traces look like for them

bitter seal
#

No idea if related, but certainly odd

dreamy linden
#

ah we're linting the typescript runtime

bitter seal
#

ohh okay, that makes sense

#

Actually I just hit an attempt locally of running call sdk typescript lint that seems to hang, looking at it

pine oyster
dreamy linden
frail idol
#

Done!

dreamy linden
#

again, will take an action to work out why our provisioning tests didn't catch this

#

am i good to retag the go and php onto that merged pr? requires a force-tag push, but since there are no changes to those sdks it should be "alright" ah you mentioned you'd do it, nvm

frail idol
dreamy linden
#

wonderful ๐ŸŽ‰

frail idol
dreamy linden
frail idol
dreamy linden
frail idol
#

We now have the right tag in place

dreamy linden
#

if it doesn't - we could tag v0.11.7+1

#

or something like that

frail idol
#

Trying to pull it explicitly in case that fixes it

frail idol
#

A separate issue which I have noticed is that PHP SDK v0.11.7 did not get published. Publishing now manually:

frail idol
#

TL;DR we will not be able to fix the v0.11.7 Go SDK release. Publishing v0.11.8 as soon as we address some of the edge cases that we discovered sounds most reasonable to me.

dreamy linden
#

if we publish v0.11.8 - we will be making an intentionally broken release - are we alright with that? cc @brazen wave @bitter seal

#

there is a legitimate timeout of testdev on main

frail idol
#

Where does the breakage come from?

dreamy linden
#

Probably an old version, v0.11.5 or so

brazen wave
#

could we re-release just Go "0.11.7" as 0.11.8? or are we keen to keep the versions in lock step

dreamy linden
#

We could release v0.11.8-1 or something (v0.11.8-rc1 would also work)

#

That's a prerelease

#

Then we don't do a whole new release, just one new tag

brazen wave
#

might not get picked up by Go versioning (not sure)

dreamy linden
frail idol
#

Also, I was not able to get the PHP SDK to force update to v0.11.7 . Still not sure what is happening with that one, but both SDKs are affected. The Go one is definitely far more important to get fixed.

dreamy linden
#

The other issue with doing an engine released now, is we just merged a large pr that refactored a lot - I was hoping to give it more time to settle to avoid just hitting similar issues in several days again

frail idol
#

Yes, we could try tagging v0.11.8-rc1 for Go only. Even if it will use Engine v0.11.7, that should be OK.

dreamy linden
#

It should upgrade automatically in the same way, we don't have a version conflict when we do v0.11.8, it sounds good to me

#

We could push it directly to the go-sdk repo as well (same for php)

frail idol
#

OK, how about this?

  • let's retract v0.11.7 explicitly since it's broken
  • once this is merged & sync'ed to dagger-go-sdk, we can tag sdk/go/v0.11.8-rc.1 on dagger/dagger and let the workflow do the rest

WDYT?

#

OK, let me put together a PR that implements this.

dreamy linden
#

Fyi, I'm going to be at a tech meetup in a few minutes, so will have slower response times

frail idol
dreamy linden
#

working on the mixed up tags/branches thing now

dreamy linden