Closed data-sync-user closed 1 year ago
➤ Matt Cleinman commented:
Given that the upgrade message is missing from 2.15.x and the welcome message is missing from 2.16.x, it seems like the issue is with addons - and not our code that runs them. (Because 2.15.x presumably successfully had the welcome message)
The hash requested from releng was f36a57db6ba33836fd28d65e3bdbdc0ebcefd437
Looking at our codebase, the addons are pulled from: https://archive.mozilla.org/pub/vpn/addons/releases/latest/ ( https://archive.mozilla.org/pub/vpn/addons/releases/latest/|smart-link )
You can see the new addons in that URL.
Looking at log outputs from 2.16 on iOS.
So looks like it’s pulling older addons, as it doesn’t include the 2.16 what’s new message.
Looking into the addons index in the app - how is that updated?
From TaskAddOnIndex, which in turn gets in from the addon manifest: QString manifestUrl = QString("%1manifest.json").arg(AddonManager::addonServerAddress());
And as expected, this manifest file doesn’t include our new addons: https://archive.mozilla.org/pub/vpn/addons/releases/latest/manifest.json ( https://archive.mozilla.org/pub/vpn/addons/releases/latest/manifest.json )
This manifest file is not in our repo, so it’s built somewhere. Where is it built, and why isn’t it including our new files?
I believe it happens in scripts/addon/index.py, but some chance it’s in taskcluster/ci/beetmover/kind.yml or taskcluster/ci/build.addons.yml. And if for some reason it’s not there, the other places that mention the manifest file but I haven’t really looked into aretaskcluster/mozillavpn_taskgraph/transform/beetmover.py and taskcluster/scripts/addons/build.sh.
➤ Matt Cleinman commented:
➤ Matt Cleinman commented:
Naomi Kirby hopped in for a bit, thanks!
It was correct in the RC (https://archive.mozilla.org/pub/vpn/addons/candidates/20230801121237/manifest.json ( https://archive.mozilla.org/pub/vpn/addons/candidates/20230801121237/manifest.json )), so perhaps it was an issue with the promotion to prod?
Nope, it actually seems like it's correct in https://archive.mozilla.org/pub/vpn/addons/releases/20230801121237/manifest.json ( https://archive.mozilla.org/pub/vpn/addons/releases/20230801121237/manifest.json )
However, it is wrong in https://archive.mozilla.org/pub/vpn/addons/releases/latest/manifest.json ( https://archive.mozilla.org/pub/vpn/addons/releases/latest/manifest.json )
I’d expect those two files to match - Given https://archive.mozilla.org/pub/vpn/addons/releases/, ( https://archive.mozilla.org/pub/vpn/addons/releases/, ) it seems like latest should match 20230801121237.
I can see in beetmover.py where it should be including the latest directory as part of the promotion to prod.
I reached out to releng to assist, as it now seems like it is in their boat.
➤ Matt Cleinman commented:
Looks like that latest directory’s manifest.json matches https://archive.mozilla.org/pub/vpn/addons/releases/20230524074709/manifest.json ( https://archive.mozilla.org/pub/vpn/addons/releases/20230524074709/manifest.json ). which is the last release. Interesting.
➤ Matt Cleinman commented:
Naomi Kirby hopped in, @relduty, could we get some help with VPN addons?
For some reason the 2.16 version of addons index (manifest.json) in the latest directory got over written with the 2.15 version about 20 minutes after it was promoted. Releng re-ran the job, and latest now has the manifest we’d expect. Releng is currently looking into why it was overwritten previously, and I’m keeping an eye on it to make sure it doesn’t revert again. But as of now, this is done.
➤ Matt Cleinman commented:
So, it reverted again, so this is not fixed.
Rel eng pulled in some more folks, seems related to how we sync changes across a couple cloud providers. Hoping it is fixed tomorrow, but this is mostly happening outside the VPN team at this point.
➤ Valentina Virlics commented:
Verified this across platforms and left a short report via Slack. Add-ons seem to have been pushed out with success.
There is still a question regarding the welcome message.
➤ Matt Cleinman commented:
Per Santiago Andrigo on slack, he is comfortable with this change. Moving this to done, feel free to move back if you disagree.
➤ Valentina Virlics commented:
Matt Cleinman Santiago Andrigo So, going fwd, starting with 2.16, we will no loger have the Welcome message anymore at fresh installs? But instead we will be using the “You’ve updated Mozilla VPN“ everywhere? Thank you!
➤ Santiago Andrigo commented:
Yes, one replaces the other. Note that the message should include the changelog, just as much and to the extent as the “Update” / “Download” message does.
Important details:
Mozilla VPN used versions:
Platform tested:
Prerequisites:
Steps to reproduce:
Expected results:
Actual results:
Notes:
┆Issue is synchronized with this Jira Bug ┆Reporter: Valentina Virlics