ESC meeting minutes: 2020-10-22

Miklos Vajna vmiklos at collabora.com
Thu Oct 22 14:54:21 UTC 2020


* Present:
    + Stephan, Michael W, Caolan, Cloph, Heiko, Xisco, Olivier, Miklos, Sophie, Thorsten, Michael M, Michael S, Gabriel

* Completed Action Items:
    + Look at why some uitests are not failing but timing out e.g. at https://gerrit.libreoffice.org/c/core/+/103993 (Stephan)
    + conclusion: python code checks for something while the dialog is open, and we don’t close the dialog, so we hang → would need to write the tests to first close the dialogs and then assert
      + auto-closing dialogs are not a good way, would leak to soffice processes

* Pending Action Items:
    + None

* Release Engineering update (Cloph)
    + 7.0 status: 7.0.3 rc1 tagged + branches yesterday
    + 6.4 status: 6.4.7 announced today
    + 7.1 status: 7.1 alpha1 from master, to be tagged tomorrow
      + feature freeze: week 47 (2020-11-16)
    + Remotes
    + Android viewer
    + heads-up for distro branches: if you use submodules, then make sure that .gitmodules uses relative paths

* Would it be possible to release 7.0.3 RC1 as final next week? (Xisco)
	- Fixes a couple of critical issues
	- https://bugs.documentfoundation.org/show_bug.cgi?id=133771 encryption
        + 12 duplicates
	- https://bugs.documentfoundation.org/show_bug.cgi?id=137248 Calc recalc
        + 15 duplicates
      + theoretically possible, but nobody tested RC1 so far (Cloph)
      + otherwise the RC2 would be in 3 weeks from now, right? (Xisco)
        + in 2 weeks
      + still that fresh version, not noticed with 7.0.x so far (Thorsten)
        + trying to not avoid time-based releases
      => let’s ship earlier in this special case (all)

* Documentation (Olivier)
    + New Help
        + No news
    + Helpcontent2
        + Updates and fixes (S. Chaiklin, Fitoshido, A. Gemini)
    + Google seasons of Doc
        + Progressing
    + Guides
        + Community actively working on 7.0 releases.
        + Published Math Guide 7.0
    + Collabora Online split, was that discussed in the doc community? (Thorsten)
      + not discussed (Olivier)

* UX Update (Heiko)
     + Bugzilla (topicUI) statistics
         246(246) (topicUI) bugs open, 258(258) (needsUXEval) needs to be evaluated by the UXteam
     + Updates:
         BZ changes   1 week   1 month    3 months    12 months
              added     11(3)     23(-2)      55(0)      163(1)
          commented     94(9)    394(-38)   1056(-14)   3852(-17)
            removed      0(-2)     2(-4)       7(-3)      53(-2)
           resolved     13(4)     62(-7)     131(-1)     433(6)
     + top 10 contributors:
           Heiko Tietze made 240 changes in 1 month, and 2234 changes in 1 year
           Foote, V Stuart made 78 changes in 1 month, and 622 changes in 1 year
           Kainz, Andreas made 66 changes in 1 month, and 488 changes in 1 year
           Telesto made 63 changes in 1 month, and 554 changes in 1 year
           Seth Chaiklin made 46 changes in 1 month, and 196 changes in 1 year
           Dieter Praas made 38 changes in 1 month, and 491 changes in 1 year
           Roman Kuznetsov made 34 changes in 1 month, and 266 changes in 1 year
           Kaganski, Mike made 30 changes in 1 month, and 132 changes in 1 year
           BogdanB made 25 changes in 1 month, and 182 changes in 1 year
           Thomas Lendo made 22 changes in 1 month, and 134 changes in 1 year

    + 23 new tickets with needsUXEval Oct/08-23

      * Give user specific warning when saving a spreadsheet with
        > 65536 rows as XLS
        + https://bugs.documentfoundation.org/show_bug.cgi?id=137327
        + NEW
        + not warning when converting from csv to xls in headless mode
          + on the UI: CSV save gives a warning, but only on first save
      * UI: Discourage usage of the to page anchor by hiding it from
        toolbar/context menu
        + https://bugs.documentfoundation.org/show_bug.cgi?id=135836
        + FIXED and .uno:SetAnchorToPage removed from menus
        + still there, but the UI/menu hides it
      * Add option to disable splash screen
        + https://bugs.documentfoundation.org/show_bug.cgi?id=137581
        + RESOLVED/WF
        + not true that all apps have no splash screen
        + hurts branding
      * Many places have tight integration of extensions now
        + still not finished tough (tdf#137470)

* Crash testing (Caolan)
     + 2(-6) import failure, 1(+0) export failures
     + 1 coverity issue
     + 7 ossfuzz issues (all timeouts)
     + bring the timeout problems to public TDF bugzilla?
        + see no issues with that (all)

* Crash Reporting (Xisco)
   + https://crashreport.libreoffice.org/stats/version/6.4.5.2
        + (-36) 1474 1510 1604 1618 1744 1814 2208 2552 2315 1761 1162 0
   + https://crashreport.libreoffice.org/stats/version/6.4.6.2
        + (+214) 2000 1778 1564 1306 1053 874 0
   + https://crashreport.libreoffice.org/stats/version/7.0.0.3
        + (-453) 3314 3767 3952 4171 5084 6489 3840 788 0
   + https://crashreport.libreoffice.org/stats/version/7.0.1.2
        + (-1629) 4969 6598 4561 3223 1986 0
   + https://crashreport.libreoffice.org/stats/version/7.0.2.2
        + (+3457) 3457 0
   + 7.0.2.2 have no symbols yet so far


* Mentoring/easyhack update
           committer...   1 week     1 month      3 months    12 months
                   open      65(0)      118(0)       119(-1)      128(-1)
                reviews     354(66)    1482(-28)    4364(-42)   18779(-220)
                 merged     273(102)   1307(-155)   4599(-75)   19460(-93)
              abandoned      18(-9)     103(-1)      252(0)      1058(-7)
            own commits     200(56)     998(-124)   3430(-56)   15546(-216)
         review commits      49(15)     273(-37)    1096(-22)    4688(-43)
         contributor...   1 week     1 month      3 months      12 months
                   open      24(-12)     71(4)         71(3)         73(3)
                reviews     740(96)    3290(-360)   11132(-292)   45015(-156)
                 merged      21(3)       96(1)        367(-4)      1650(4)
              abandoned       4(-3)      17(-6)        81(-13)      563(1)
            own commits      22(6)      104(1)        448(-17)     1728(13)
         review commits       0(0)        0(0)          0(0)          0(0)
     + easyHack statistics:
        needsDevEval 9(9)   needsUXEval 2(2)   cleanup_comments 249(249)
        total 295(295)   assigned 14(14)   open 259(259)
     + top 10 contributors:
           Johnny_M made 15 patches in 1 month, and 307 patches in 1 year
           Attila Szűcs made 8 patches in 1 month, and 19 patches in 1 year
           Gleb Popov made 7 patches in 1 month, and 7 patches in 1 year
           Alain Romedenne made 6 patches in 1 month, and 147 patches in 1 year
           Weblate made 6 patches in 1 month, and 75 patches in 1 year
           Regényi Balázs made 6 patches in 1 month, and 9 patches in 1 year
           Nagy Tibor made 4 patches in 1 month, and 22 patches in 1 year
           Attila Bakos made 4 patches in 1 month, and 34 patches in 1 year
           Ayhan Yalçınsoy made 4 patches in 1 month, and 23 patches in 1 year
           Dr. David Alan Gilbert made 4 patches in 1 month, and 6 patches in 1 year
     + top 10 reviewers:
           Nabet, Julien made 134 review comments in 1 month, and 1424 in 1 year
           Timar, Andras made 134 review comments in 1 month, and 1478 in 1 year
           McNamara, Caolán made 110 review comments in 1 month, and 824 in 1 year
           Michael Stahl made 100 review comments in 1 month, and 1116 in 1 year
           Behrens, Thorsten made 98 review comments in 1 month, and 1344 in 1 year
           Adolfo Jayme Barrientos made 98 review comments in 1 month, and 1282 in 1 year
           Németh, László made 94 review comments in 1 month, and 666 in 1 year
           Holešovský, Jan made 56 review comments in 1 month, and 557 in 1 year
           Xisco Fauli made 56 review comments in 1 month, and 808 in 1 year
           Olivier Hallot made 52 review comments in 1 month, and 766 in 1 year
     + Patches automatically abandoned:
         tdf#84502 Accelerators for sidebar decks ( Jim Raykowski )
             - https://gerrit.libreoffice.org/#/c/100148

     + big CONGRATULATIONS to contributors who have at least 1 merged patch, since last report:
           Mariam Fahmy
           Platon Pronko
           Leo Wang

* Commit Access
  + None

* Developer Certification (Stephan/Kendy/Thorsten)
    + sleep -2 weeks
    + seeking for a new person as a replacement for Bjoern, who is currently in    + reach out to NISZ? (Michael M)
AI:        + reach out to Laszlo (Miklos)

    + now is the time to start building a new proposal (Thorsten)

* Jenkins / CI update (Cloph)
    from:Thu Oct 15 13:46:13 2020
    master linux rel  jobs:  62 ok:  62 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master linux dbg  jobs:  58 ok:  53 ko:   5 fail ratio:  8.62 % break:   3 broken duration:15.84%
    master mac rel    jobs:  97 ok:  94 ko:   3 fail ratio:  3.09 % break:   1 broken duration: 1.01%
    master mac dbg    jobs:  96 ok:  95 ko:   1 fail ratio:  1.04 % break:   1 broken duration: 3.04%
    master win rel    jobs:  78 ok:  78 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master win dbg    jobs:  78 ok:  78 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master win64 dbg  jobs:  77 ok:  77 ko:   0 fail ratio:  0.00 % break:   0 broken duration: 0.00%
    master gerrit lin jobs: 512 ok: 401 ko:  88 fail ratio: 17.19% time for ok: mean:  23 median:  23
    master gerrit plg jobs: 490 ok: 324 ko: 128 fail ratio: 26.12% time for ok: mean:  32 median:  30
    master gerrit win jobs: 528 ok: 373 ko:  98 fail ratio: 18.56% time for ok: mean:  64 median:  57
    master gerrit mac jobs: 529 ok: 380 ko: 117 fail ratio: 22.12% time for ok: mean:  31 median:  29
    master gerrit all jobs: 530 ok: 231 ko: 221 fail ratio: 41.70% time for ok: mean: 142 median: 127
  + https://dev-www.libreoffice.org/tmp/gerrit_201022.html
  + tests that failed more than twice in last seven days:
      3 CppunitTest_sc_subsequent_export_test		gerrit_windows
      3 CppunitTest_sw_odfexport		gerrit_linux_clang_dbgutil
      4 CppunitTest_sw_odfexport		gerrit_windows
      4 CppunitTest_sw_uiwriter		gerrit_windows
      4 CppunitTest_vcl_backend_test		gerrit_mac
      5 Killed by the kill-wrapper		gerrit_linux_clang_dbgutil
      7 CppunitTest_xmlsecurity_signing		gerrit_windows
      8 CppunitTest_dbaccess_tdf119625		gerrit_linux_clang_dbgutil
      9 CppunitTest_dbaccess_hsql_binary_import		gerrit_linux_clang_dbgutil
     22 CppunitTest_vcl_filters_test		gerrit_mac
     23 CppunitTest_vcl_bitmap_render_test		gerrit_mac
  + mac failures: "tdf#137468: Use the colour space of the main display" (reverted)
     https://bugs.documentfoundation.org/show_bug.cgi?id=137468

Higher failure rate:
- mac failures: poor change passed Jenkins once, but then failed regularly
  -  depends on monitor settings
- Android builders: machine had not enough RAM → fixed

* l10n (Sophie)
  + working is ongoing
  + soon have weblate integrated to the dashboard
  + Collabora Online move – not discussed yet, waiting for the board

* discuss – ESC proposal to move on with Online (or not) (Thorsten)
  + reason to bring this up again: ESC members can provide their opinion
  + heard repeatadly: people have no firm opinion
  + largely: 3 options
    1) keep everything as-is
    2) just shut it down
    3) half-way house: mirroring the github repo under the TDF github org
  + personally like a clean cut, prefer to let the board/community to decide (Heiko)
  + LO Online code & docs are an asset of the foundation (Olivier)
    + prefer 1) and then 3)
  + unfortunate for it to stay in a limbo permanently (Caolan)
    + re-direct it to where development is until someone shows up
      to do something there.
  + delivering LibreOffice on tablets / phones (Sophie)
    + are part of our mission, should stay.
  + don’t mind in gerrit or a fork on github (Christian)
    + wouldn’t completely shut it down


* Graphics architecture (Thorsten)
  * https://pad.documentfoundation.org/p/ESC_graphics_architecture
  * skia / vulkan metabug: https://bugs.documentfoundation.org/show_bug.cgi?id=129062
      + 20 (+4) open bugs, 129 total (+5)

* QA update (Xisco)
     + Please help flesh out the monthly reports:
           https://nextcloud.documentfoundation.org/s/2qbepFYXXan4ief

     + UNCONFIRMED: 1301 (+5)
         + enhancements: 132  (-2)
         + needsUXEval: 7 (+2)
         + haveBackTrace: 2 (+1)
         + needsDevAdvice: 15 (+0)
         + documentation:  16 (+1)
         + android:  5 (+0)
         + iOS:  0 (+0)
         + Online:  21 (+0)



     + Most pressing bugs:
      New:
		None

      Old:
       + PDF export: LibreOffice 6.3 option buttons in forms not working in Adobe Reader (OK in other readers)
           + https://bugs.documentfoundation.org/show_bug.cgi?id=127217
           + Thorsten ?

       + CRASH: Chart: Changing properties in different objects
           + https://bugs.documentfoundation.org/show_bug.cgi?id=133630
           + Muhammet ?

       + macOS: Can't Save As after installing language pack on Catalina
           + https://bugs.documentfoundation.org/show_bug.cgi?id=132025
           + more info here: “Subject: Budget 2020 proposals and macOS
Catalina
             support / onboarding” on the dev list (Heiko)

        + Text runs of RTL scripts (e.g. Arabic, Hebrew, Persian) from
imported
          PDF are reversed, PDFIProcessor::mirrorString not behaving
            + https://bugs.documentfoundation.org/show_bug.cgi?id=104597
            + Thorsten looking at it
            + old problem, but many duplicates

    Fixed:
        + When saving an encrypted document, unchecking 'Save with password' has no effect, encryption is kept
            + Thanks to Vasily


     + New high severity bugs of the week:
         + Give user specific warning when saving a spreadsheet with > 65536 rows as XLS
             + https://bugs.documentfoundation.org/show_bug.cgi?id=137327
             + not a regression?

* QA stats
     + bug-stats charts end with 2020-08-13, could the script be updated to include the recent past? (Miklos)
        + AI: will look at it (Xisco)
     + https://bugs.documentfoundation.org/page.cgi?id=weekly-bug-summary.html
       +174  -11 (-162) overall)
       many thanks to the top bug squashers:
        QA Administrators       16
        Heiko Tietze            15
        Aron Budea              12
        Mike Kaganski           12
        Timur                   12
        Caolán McNamara         10
        Buovjaga                9
        sdc.blanco              6
        Michael Stahl (CIB)     5
        Telesto                 4

     + top 10 bugs reporters:
        Telesto                 16
        NISZ LibreOffice Team   12
        Heiko Tietze            6
        Aron Budea              4
        Rizal Muttaqin          4
        sdc.blanco              4
        Xisco Faulí             3
        juergenkohler23         3
        ASHIK ACHARYA           2
        Leroy                   2

     + top 10 bugs fixers:
        McNamara, Caolán        9
        Seth Chaiklin           6
        Budea, Áron             5
        Heiko Tietze            5
        Michael Stahl           5
        Németh, László          3
        Glogowski, Jan-Marek    2
        Kevin Suo               2
        Rathke, Eike            2
        Timur                   2

     + top 10 bugs confirmers:
        Ilmari Lauhakangas      34
        Heiko Tietze            10
        Dieter Praas            9
        Timur                   8
        *UNKNOWN*               7
        BogdanB                 6
        Kaganski, Mike          6
        Foote, V Stuart         5
        Nabet, Julien           5
        Telesto                 5


 * Highest-Priority bugs (aka "MABs"):
    + http://bit.ly/2lJAAJI
      6.0 : 1/52 -     1 %  (+0)
      5.4 : 0/41 -     0 %  (+0)
      5.3 : 0/54 -     0 %  (+0)
      5.2 : 0/43 -     0 %  (+0)
      5.1 : 1/36 -     2 %  (+0)
      5.0 : 0/64 -     0 %  (+0)
      4.4 : 0/76 -     0 %  (+0)
      4.3 : 2/73 -     2 %  (+0)
      4.2 : 2/134 -    1 %  (+1)
      4.1 : 3/85 -     3 %  (+0)
      4.0 : 4/83 -     4 %  (+0)
      old : 6/247 -    2 %  (+0)


 * Bisected bugs open: keyword 'bisected'
    + more accurate - down to a single commit.
    + http://bit.ly/2dyIfDy
    + 891/4819 892/4798 883/4780 881/4768 874/4746 880/4729 862/4687

      done by:
         Budea, Áron             5
         Telesto                 4
         Xisco Fauli             4
         Timur                   2
         Ilmari Lauhakangas      1
         NISZ LibreOffice Team   1
         Kaganski, Mike          1


 * Bibisected bugs open: keyword 'bibisected'
    + http://bit.ly/2cSCXlS
    + 1070/5717 1070/5692 1057/5667 1057/5654 1049/5632 1055/5615 1046/5583

      done by:
         Budea, Áron             5
         Telesto                 4
         Xisco Fauli             4
         Timur                   2
         BogdanB                 1
         Glogowski, Jan-Marek    1
         Ilmari Lauhakangas      1
         Justin Luth             1
         NISZ LibreOffice Team   1
         Kaganski, Mike          1


 * all bugs tagged with 'regression'
    + 1335(-4) bugs open of 10363(+20) total 13(-3) high prio.

      done by:
         Telesto                 7
         Budea, Áron             6
         Ilmari Lauhakangas      3
         BogdanB                 1
         Timur                   1
         Justin Luth             1
         NISZ LibreOffice Team   1
         Kaganski, Mike          1
         Xisco Fauli             1


 * ~Component   count net * high severity regressions
    + http://bit.ly/1HWHb3E
      Writer        - 7(-2)
      Calc          - 2(-1)
      Impress       - 2(+0)
      Chart         - 1(+0)
      LibreOffice   - 1(+0)

    by OS:
      All           - 11(-3)
      Linux         - 1(+0)
      Mac OS X      - 0(+0)
      Windows       - 1(+0)


 * ~Component   count net * all regressions
    + http://bit.ly/1BUdI8i
      Writer: other            - 419(-6)
      Calc                     - 229(+0)
      Impress                  - 154(+0)
      Writer: docx filter      - 57(+2)
      LibreOffice              - 56(+0)
      Crashes                  - 54(-2)
      Draw                     - 47(-1)
      Borders                  - 44(+0)
      UI                       - 43(+1)
      Writer: perf             - 42(+0)
      Base                     - 35(+0)
      Writer: other filter     - 30(+0)
      Writer: doc filter       - 28(+0)
      graphics stack           - 25(+1)
      Chart                    - 23(+0)
      RTL                      - 23(+0)
      Printing and PDF export  - 21(-1)
      filters and storage      - 19(+0)
      BASIC                    - 15(+1)
      framework                - 2(+0)
      Formula Editor           - 1(+0)
      Installation             - 1(+0)
      sdk                      - 1(+0)


More information about the LibreOffice mailing list