Closed Bug 1922666 Opened 4 months ago Closed 4 months ago

Remove baseline profiles that are committed to version control

Categories

(Fenix :: Performance, task, P1)

All
Android
task

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jonalmeida, Assigned: titouan)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid][group4] [fxdroid])

As part of bug 1807350, we generated profiles and committed them to the codebase to avoid blockers within infrastructure tooling during the prototype phase.

Once the work to automate and package the baseline profiles for each build is done, we should remove the existing profiles and add an ignore that stops anyone from committing them in the future.

Assignee: nobody → tthibaud
Priority: -- → P1
Whiteboard: [fxdroid][group4] [fxdroid]

The generated profile was actually removed from the tree in D221635.

As the profile is now generated automatically as part as the release pipeline, I guess we don't need to add an ignore that avoids commiting new profiles in the future? What do you think?

Status: NEW → RESOLVED
Closed: 4 months ago
Flags: needinfo?(jonalmeida942)
Resolution: --- → FIXED

The bug was marked as fixed by mistake. I'll reopen it and only close it if you confirm that we don't need to add an ignore for the baseline profiles.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

(In reply to Titouan Thibaud [:titouan] from comment #1)

The generated profile was actually removed from the tree in D221635.

As the profile is now generated automatically as part as the release pipeline, I guess we don't need to add an ignore that avoids commiting new profiles in the future? What do you think?

Oh cool - happy to see it was removed. Yeah, I think it's fine to skip the an ignore entry if this is part of our build pipeline now. We can always re-open if we change our mind. Thanks!

Status: REOPENED → RESOLVED
Closed: 4 months ago4 months ago
Flags: needinfo?(jonalmeida942)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.