Closed Bug 1726251 Opened 4 years ago Closed 23 hours ago

Try builds for Flatpak are broken

Categories

(Firefox Build System :: Task Configuration, defect, P3)

defect

Tracking

(firefox137 fixed)

RESOLVED FIXED
137 Branch
Tracking Status
firefox137 --- fixed

People

(Reporter: rmader, Assigned: jcristau)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

In order to make it easier for contributors to help with Flatpak builds, it would be great if there was an easy way to trigger try builds. There is already some initial support for that by applying the following patch:

diff --git a/taskcluster/ci/release-flatpak-repackage/kind.yml b/taskcluster/ci/release-flatpak-repackage/kind.yml
index 59c4bb2da8b7e..421320ff92591 100644
--- a/taskcluster/ci/release-flatpak-repackage/kind.yml
+++ b/taskcluster/ci/release-flatpak-repackage/kind.yml
@@ -12,17 +12,16 @@ transforms:
 
 kind-dependencies:
     - post-beetmover-dummy
     - post-langpack-dummy
 
 job-defaults:
     description: Generates flatpak by reackaging the existing tar.bz2
     run-on-projects: []  # to make sure this never runs as part of CI
-    run-on-releases: [beta, release, release-rc]
     shipping-phase: promote
     scopes: []
     treeherder:
         platform: linux64-shippable/opt
         kind: build
         tier: 2
     worker-type: b-linux
     worker:

which unlocks the release-flatpak-repackage-firefox task in ./mach try fuzzy --full.

Doing so[1] currently fails with:

+ tar jxf /home/worker/workspace/firefox.tar.bz2
bzip2: (stdin) is not a bzip2 file.

1: https://treeherder.mozilla.org/jobs?repo=try&revision=43223b2fc9eea4937e586b2cb0c66b30b16039e1

The Bugbug bot thinks this bug should belong to the 'Firefox Build System::Task Configuration' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Task Configuration

Just for the record, I'm not sure these ever worked, and we certainly don't make guarantees about them. I agree this would be good to fix though.

I think this might be fixed? At least here's a passing task on a level 1 worker:
https://firefox-ci-tc.services.mozilla.com/tasks/GG6Ty8D1TISuR23MxvRNvw

Please re-open if this is still an issue.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WORKSFORME

I think the issue is that the flatpak-repackage task can currently only run if you can schedule a (staging) release, because it expects to find langpacks on archive.m.o or the staging equivalent. It might be possible to download those artifacts from dependencies instead, and not rely on release promotion?

Severity: -- → S4
Status: RESOLVED → REOPENED
Priority: -- → P3
Resolution: WORKSFORME → ---
Depends on: 1949189
Assignee: nobody → jcristau
Status: REOPENED → ASSIGNED

Copy commands from taskcluster/docker/firefox-flatpak/runme.sh into a
standalone repackage command that can be pointed at a firefox build and
spit out a flatpak repository.

The browser/installer/linux/app/flatpak directory contains templates and
extra files that get injected into the flatpak build process.

Add debian12-flatpak docker image, with the necessary tools for building
a flatpak. This image also runs the dbus service so flatpak as non-root
can work.

Add a repackage-flatpak task for the linux64/opt build and on-change
l10n.

Blocks: 1949464
Blocks: 1925839
Pushed by jcristau@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/96ff89531ad1 add `mach repackage flatpak` command. r=releng-reviewers,gerard-majax,bhearsum https://hg.mozilla.org/integration/autoland/rev/2cf34b3c9e61 add repackage-flatpak CI task. r=taskgraph-reviewers,releng-reviewers,bhearsum
Status: ASSIGNED → RESOLVED
Closed: 2 years ago23 hours ago
Resolution: --- → FIXED
Target Milestone: --- → 137 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: