Closed Bug 1814896 Opened 2 years ago Closed 2 years ago

Release geckodriver 0.32.2 with marionette 0.3.0 ("cargo install geckodriver" is broken for 0.31.0 and 0.32.0 releases)

Categories

(Testing :: geckodriver, defect, P1)

Default
defect
Points:
2

Tracking

(firefox111 fixed)

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [webdriver:m6])

Attachments

(3 files)

As reported on Matrix by khuey geckodriver cannot be build anymore for the 0.31.0 and 0.32.0 releases via cargo install. The following command will fail:

cargo install --version 0.31.0 geckodriver

The reason is the change from bug 1811996 and that we did a semver incompatible version bump for the Marionette crate when releasing geckodriver 0.32.1 via bug 1794560.

error: could not compile `geckodriver` due to 9 previous errors
error: failed to compile `geckodriver v0.32.0`, intermediate artifacts can be found at `/var/folders/sd/hgkqcwv90sz_q1pqjww78nfm0000gn/T/cargo-install55mx5A`

We should decide what to do here. Most likely we should yank the marionette 0.2.1 crate and geckodriver 0.31.1 after releasing a new 0.31.2 release.

Lets discuss on Monday in our triage meeting.

To prevent such regressions we could run cargo semver-checks check-release for each of the dependency crates as part of Update in-tree dependency crates. The tool needs to be installed once via cargo install cargo-semver-checks --locked (website).

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Points: --- → 2
Priority: -- → P1
Summary: "cargo install geckodriver" is broken for 0.31.0 and 0.32.0 releases → Release geckodriver 0.32.2 with marionette 0.3.0 ("cargo install geckodriver" is broken for 0.31.0 and 0.32.0 releases)
Whiteboard: [webdriver:triage] → [webdriver:m6]

The release of version 0.2.1 didn't obey the semver rules and
introduced a backward incompatible bug. As such we have to
release a minor release now.

Depends on D168984

Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f1a9e1ce2ebd [rust-marionette] Relase version 0.3.0. r=webdriver-reviewers,supply-chain-reviewers,jgraham https://hg.mozilla.org/integration/autoland/rev/a0aa366f7f06 [geckodriver] Release version 0.32.2. r=webdriver-reviewers,jgraham
Keywords: leave-open

The merge to mozilla-central was done as part of 602aa16c20d4.

Once all the signed geckodriver builds are ready I will do the release.

Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b545bf471941 [geckodriver] Update changeset for the 0.32.2 release. r=webdriver-reviewers,jgraham DONTBUILD

Upstream import: https://github.com/mozilla/geckodriver/pull/2083

Version v0.32.2 is released: https://github.com/mozilla/geckodriver/releases/tag/v0.32.2

Crates have been published to crates.io:

Uploading marionette v0.3.0
Uploading geckodriver v0.32.2

Next step is to yank the broken crates from crates.io and to check that everything builds fine again.

Yanked the following crates:

Made the announcement on the mailing list:
https://groups.google.com/a/mozilla.org/g/dev-webdriver/c/XXN5L8JptEk

I'm going to leave the bug open until the remaining commit got merged to mozilla-central.

Keywords: leave-open
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: