Closed Bug 1622282 Opened 1 year ago Closed 1 year ago

[snap] added cleanup part to remove duplicate files

Categories

(Release Engineering :: Release Automation: Snap, enhancement)

enhancement
Not set
normal

Tracking

(firefox76 fixed)

RESOLVED FIXED
Tracking Status
firefox76 --- fixed

People

(Reporter: ken.vandine, Assigned: ken.vandine)

References

Details

Attachments

(2 files)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:75.0) Gecko/20100101 Firefox/75.0

Steps to reproduce:

The current snap includes some files that are also provided by the base snap and the gnome-3-28-1804 content snap.

Assignee: nobody → ken.vandine
Pushed by jlorenzo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1a0776f93e88
Find files provided by the base and platform snap and ensure they aren't duplicated in this snap r=jlorenzo
Status: UNCONFIRMED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

Comment on attachment 9133208 [details]
Bug 1622282 - Find files provided by the base and platform snap and ensure they aren't duplicated in this snap r=jlorenzo

Beta/Release Uplift Approval Request

  • User impact if declined: Unable to get official builds with this patch, because Snaps are only repackaged in mozilla-beta and above.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This patch is just cleaning up the Snap repackage which contained duplicated files. We're at the beginning of the beta cycle, backing it out in case something goes South is less risky at this stage.
  • String changes made/needed: None
Attachment #9133208 - Flags: approval-mozilla-beta?

Comment on attachment 9133208 [details]
Bug 1622282 - Find files provided by the base and platform snap and ensure they aren't duplicated in this snap r=jlorenzo

That looks safe and reasonnable to me, uplift approved for 75 beta 6, thanks!

Attachment #9133208 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Crap, this patch broke the snap repackage task[1]:

Priming xdg-open 
Priming shared-mime-info 
Priming cleanup 
+ cd /snap/core18/current
+ find . -type f,l -exec rm -f /home/worker/workspace/prime/{} ;
+ cd /snap/gnome-3-28-1804/current
/bin/sh: 38: cd: can't cd to /snap/gnome-3-28-1804/current
Failed to run 'override-prime': Exit code was 2.

This could have been avoided with bug 1451694, for instance. That said, I also see a good opportunity to grant Ken level 1 access to try patches on our infra (bug 1623696). I confess I didn't have enough bandwidth get this running on our infra, and I thought the patch was low-risk enough. Moreover, Ken knows Snap inside out and he's been a great help to me in setting the Snap packages up.

[1] https://firefox-ci-tc.services.mozilla.com/tasks/PUYx2nfdRxegJzUh_QkcRA/runs/0/logs/https%3A%2F%2Ffirefox-ci-tc.services.mozilla.com%2Fapi%2Fqueue%2Fv1%2Ftask%2FPUYx2nfdRxegJzUh_QkcRA%2Fruns%2F0%2Fartifacts%2Fpublic%2Flogs%2Flive.log#L8170

Status: RESOLVED → REOPENED
Ever confirmed: true
Resolution: FIXED → ---
See Also: → 1451694, 1623696

Backed out on autoland: https://hg.mozilla.org/integration/autoland/rev/414dba231f7f89576dae6727b91390d751cb90dc will be included in the next merge to m-c. Will back out from beta soon.

Attachment #9133208 - Flags: approval-mozilla-beta+

There is an additional patch attached that fixes the build issue that caused this to be reverted.

Ken and I both tested both patches. Latest try is at https://treeherder.mozilla.org/#/jobs?repo=try&revision=f9340198f4b18322ba7970bbbe24ccb920ec436b.

I think we're good to land this in central. No need to land this on beta anymore, we start a new beta cycle next week.

Pushed by jlorenzo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/800b27ffb354
Find files provided by the base and platform snap and ensure they aren't duplicated in this snap r=jlorenzo
Pushed by jlorenzo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/738219cbc8c1
extract gnome-3-28-1804 platform snap to be used in the cleanup part r=jlorenzo
Status: REOPENED → RESOLVED
Closed: 1 year ago1 year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.