Closed Bug 1771587 Opened 2 years ago Closed 2 years ago

Collect ESM-ification progress

Categories

(Core :: XPConnect, task)

task

Tracking

()

RESOLVED FIXED
104 Branch
Tracking Status
firefox104 --- fixed

People

(Reporter: arai, Assigned: yulia)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

To track the progress of ESM-ification, count the number of remaining JSM files and already-ESM-ified files

Attached file areweesmifiedyet.py (obsolete) —

This script collects status and prints JSON file.
jsm and esm property are the number of files inside the directory and sub-directories.

{
  "jsm": 1335,
  "esm": 7,
  "subdir": {
    "browser": {
      "jsm": 340,
      "esm": 0,
      "subdir": {
        "actors": {
          "jsm": 56,
          "esm": 0
        },
        "base": {
          "jsm": 3,
          "esm": 0,
...
Attachment #9278601 - Attachment mime type: text/x-python-script → text/plain

I forgot how we've typically wired this type of thing up to CI to make sure it's running automatically so that we can i.e. fetch the value from a given day and build a chart to show progress. I believe this can be done with taskcluster artifacts and running the script on a cron but I'm having trouble finding a good example. Marco or Andrew, do you have any thoughts on how we might do that?

Flags: needinfo?(mcastelluccio)
Flags: needinfo?(ahal)

some TODO for the script:

  • perhaps split the tree/count into test and non-test
  • exclude tests that's not going to be ESM-ified (tests for JSM loader)
Attached file areweesmifiedyet.py

Added excluded files list.

Attachment #9278601 - Attachment is obsolete: true

(In reply to Brian Grinstead [:bgrins] from comment #2)

I forgot how we've typically wired this type of thing up to CI to make sure it's running automatically so that we can i.e. fetch the value from a given day and build a chart to show progress. I believe this can be done with taskcluster artifacts and running the script on a cron but I'm having trouble finding a good example. Marco or Andrew, do you have any thoughts on how we might do that?

I can think of two options:

Flags: needinfo?(mcastelluccio)

I don't know anything about the bugbug approach, but can help with setting up a normal cron task if needed. But as Marco mentioned, you should be able to copy how other cron tasks are implemented.

Flags: needinfo?(ahal)

Thanks Marco and Andrew! I'd say if it's easy enough to run some arbitrary python and save an artifact from a cron in-tree that's probably where I'd start (because then you could i.e. edit the script later on if you want to update it in parallel with changes in m-c). But bugbug's definitely a good option too if cron isn't a good fit for whatever reason.

Assignee: nobody → ystartsev
Status: NEW → ASSIGNED
Attachment #9283037 - Attachment description: Bug 1771587 - Implement are-we-esmified-yet check on ci; r=marco,arai → Bug 1771587 - Implement are-we-esmified-yet check on ci; r=marco!,arai!
Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/7d6ca1460d77
Implement are-we-esmified-yet check on ci; r=marco,jmaher
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch
Regressions: 1778418
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: