Change Details

# Goal definition ## Specific: What do we want to achieve? Our [[ https://github.com/nicholasray/pixel | visual regression tool ]] (Pixel) is more stable and has less tech debt as defined by the criteria below. ## Measurable: How will we know when we've reached our goal? The following tasks and objectives have been completed preferably before I leave for sabbatical on July 5: [] Pixel is versioned (T307936 in progress) [] The database can easily be reset on each test run by passing a flag (e.g. `--reset-db`) (Planning to include this in T307936 in progress). This should make it easier to add tests that change the state of the database. [] Add visual regression tests for the typeahead search component T306846 . The typeahead search component is one of the most frequently occurring visual regressions and the tests will also be helpful when the wvui implementation is replaced with the codex implementation. [] Revise/fix visual regression checkout logic and increase unit test code coverage T309739 [] Pixel's report on https://pixel.wmcloud.org/ and cron job should survive the Wikimedia Cloud VPS server restarting. Currently, I believe manual intervention is required to start `es-dev-server` and `screen` as described in T305563#7879426 . Can we point the apache server that is already setup at the report's static directory and remove `es-dev-server`? Also, the Cloud VPS instance volume has limited space and /srv/pixel.log file currently grows without limits. Can we install logrotate? [] The regression tests are consistently deterministic. We are getting close to this, but I've noticed one or two of the tests are sometimes flaky. Running **all tests** (desktop, mobile, echo, and forthcoming search tests) 10 times against the same code should consistently pass. [] The [[ https://github.com/nicholasray/pixel/blob/63d5b2947be13e828a293bc7c7f9748101d54904/.github/workflows/push.yml#L32 | CI job that currently only runs the desktop tests for each PR ]] should be changed to run all tests and should block merging into the `main` branch if they fail. [] Github/npm ownership is transferred from `nicholasray` to `wikimedia` (T306731). If this proves too difficult or time consuming, at least ensure that each member of the team has the access they need to the repos. ## Achievable: What support will we need to achieve our goal? Support from other teams is most likely not needed except maybe for the last step (I'm not exactly sure what goes into the wikimedia github ownership process). Apart from that, dedicated time and space will be required to complete the above tasks within a month. ### Longer term outlook Although we won't have time to experimentally add Pixel to Vector's CI pipeline T308194 this month, completing the above tasks and improving the stability of Pixel will help facilitate that work.

# Goal definition ## Specific: What do we want to achieve? Our [[ https://github.com/nicholasray/pixel | visual regression tool ]] (Pixel) is more stable and has less tech debt as defined by the criteria below. ## Measurable: How will we know when we've reached our goal? The following tasks and objectives have been completed preferably before I leave for sabbatical on July 5: [] Pixel is versioned (T307936 in progress) [] The database can easily be reset on each test run by passing a flag (e.g. `--reset-db`) (Planning to include this in T307936 in progress). This should make it easier to add tests that change the state of the database. [] Add visual regression tests for the typeahead search component T306846 . The typeahead search component is one of the most frequently occurring visual regressions and the tests will also be helpful when the wvui implementation is replaced with the codex implementation. [] Revise/fix visual regression checkout logic and increase unit test code coverage T309739 [x] Pixel's report on https://pixel.wmcloud.org/ and cron job should survive the Wikimedia Cloud VPS server restarting. Currently, I believe manual intervention is required to start `es-dev-server` and `screen` as described in T305563#7879426 . Can we point the apache server that is already setup at the report's static directory and remove `es-dev-server`? [] Also, the Cloud VPS instance volume has limited space and /srv/pixel.log file currently grows without limits. Can we install logrotate? [] The regression tests are consistently deterministic. We are getting close to this, but I've noticed one or two of the tests are sometimes flaky. Running **all tests** (desktop, mobile, echo, and forthcoming search tests) 10 times against the same code should consistently pass. [] The [[ https://github.com/nicholasray/pixel/blob/63d5b2947be13e828a293bc7c7f9748101d54904/.github/workflows/push.yml#L32 | CI job that currently only runs the desktop tests for each PR ]] should be changed to run all tests and should block merging into the `main` branch if they fail. [] Github/npm ownership is transferred from `nicholasray` to `wikimedia` (T306731). If this proves too difficult or time consuming, at least ensure that each member of the team has the access they need to the repos. ## Achievable: What support will we need to achieve our goal? Support from other teams is most likely not needed except maybe for the last step (I'm not exactly sure what goes into the wikimedia github ownership process). Apart from that, dedicated time and space will be required to complete the above tasks within a month. ### Longer term outlook Although we won't have time to experimentally add Pixel to Vector's CI pipeline T308194 this month, completing the above tasks and improving the stability of Pixel will help facilitate that work.