Skip to content

Adds logging messages for IpynbDiff

Eduardo Bonet requested to merge 344676-investigate-diff-issues-in-production into master

What does this MR do and why?

Describe in detail what your merge request does and why.

This MR doesn't add any changes to the UI or functionality, just new log messages to improve debugging.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

  1. Enable jupyter_clean_diffs 
    Feature.enable(:jupyter_clean_diffs)
  2. Tail the log file 
    tail -f log/application_json.log
  3. Clone https://gitlab.com/gitlab-org/incubation-engineering/mlops/ipynb-test-projects/diff-sample
  4. Open the diff for New Commit
  5. On log file you should see one entry for "IPYNB_DIFF_GENERATED" and two entries for "IPYNBDIFF_BLOB_GENERATED"

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #344676 (closed)

Edited by Eduardo Bonet

Merge request reports