Skip to main content

Notice: this Wiki will be going read only early in 2024 and edits will no longer be possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.

Jump to: navigation, search

Difference between revisions of "DSDP/TM/Committer Phone Meeting 21-May-2008"

< DSDP‎ | TM
 
Line 17: Line 17:
 
Skype dial-in: '''martin.oberhuber''', ddykstal (or david_dykstal), david-k-mcknight, kevin.j.doyle, xuan.chen886, rupen.mardirossian, javier.montalvoorus, eugenetarassov, michael_scharf, uwe.stieber, radoslav.gerganov, wrsfburton.<br/>
 
Skype dial-in: '''martin.oberhuber''', ddykstal (or david_dykstal), david-k-mcknight, kevin.j.doyle, xuan.chen886, rupen.mardirossian, javier.montalvoorus, eugenetarassov, michael_scharf, uwe.stieber, radoslav.gerganov, wrsfburton.<br/>
  
== Invited Attendees ==
+
== Attendees ==
 
* IBM - Xuan Chen, Dave McKnight, Dave Dykstal, Rupen Mardirossian, Kevin Doyle
 
* IBM - Xuan Chen, Dave McKnight, Dave Dykstal, Rupen Mardirossian, Kevin Doyle
* Wind River - Martin Oberhuber, Michael Scharf, Uwe Stieber, Eugene Tarassov, Felix Burton
+
* Wind River - Martin Oberhuber, Uwe Stieber
* Symbian - Javier Montalvo Orus
+
 
* ProSyst - Radoslav Gerganov
 
* ProSyst - Radoslav Gerganov
 +
* '''Regrets this time''': Michael Scharf, Eugene Tarassov, Felix Burton, Javier Montalvo Orus
  
This is an Open call, so anyone else can join (though we expect the talk to be interesting for committers only).
+
== Notes ==
 
+
== Agenda ==
+
 
* Last meeting: [[DSDP/TM/Committer Phone Meeting 30-Apr-2008]]
 
* Last meeting: [[DSDP/TM/Committer Phone Meeting 30-Apr-2008]]
 
* '''Skype Call Quality'''
 
* '''Skype Call Quality'''
 +
** Skypecast quality was OK in the beginning with up to 4 participants, but became unbearable shortly later. Rado was unable to dial into the Skypecast (sitting behind an HTTP proxy).
 +
** Normal Skype conference worked very fine for all 8 participants this time, but 8 really is the limit here (9th person trying to accept invitation cannot join).
  
 
=== '''New Stuff''' ===
 
=== '''New Stuff''' ===
Line 33: Line 33:
 
* Late API Change Requests
 
* Late API Change Requests
 
** We really need to draw a finishing line now!
 
** We really need to draw a finishing line now!
** {{Bug|233068}} - [api][breaking] Remove tm.discovery.edit reexport of EMF
+
** {{Bug|233068}} - [api][breaking] Remove tm.discovery.edit reexport of EMF -- '''accepted, Martin'''
** {{Bug|231209}} - [api][breaking] IRemoteFile.getSystemConnection() should be changed to IRemoteFile.getHost()
+
** {{Bug|231209}} - [api][breaking] IRemoteFile.getSystemConnection() should be changed to IRemoteFile.getHost() -- '''accepted, DaveM'''
** {{Bug|230821}} - [api][breaking] IRemoteFileSubsystem is inconsistent with IFileService
+
** {{Bug|230821}} - [api][breaking] IRemoteFileSubsystem is inconsistent with IFileService -- '''DaveD to investigate impact''', committers in favor of the change if doable.
** {{Bug|230919}} - [api][breaking][files] IFileService.delete() should not return a boolean
+
** {{Bug|230919}} - [api][breaking][files] IFileService.delete() should not return a boolean -- Rado: can handle all cases with exceptions, inconvenient to do both. Martin: boolean value would hardly ever be used. Rado: Not checking return values is bad practice. Martin: It boils down to the question whether it is an error to try and delete a file that's not there. -- '''accepted, Rado to make the change'''
** {{Bug|212742}} - [api] Need a Utility to send commands and receive output without prompt
+
** {{Bug|212742}} - [api] Need a Utility to send commands and receive output without prompt -- DaveM: We have RemoteCommandShellOperation with BEGIN-END-TAGS (but how reliable is it?) -- '''denied''', due to lack of time
 +
** Changes to go into RC2
  
 
=== '''Steps Towards Ganymede''' ===
 
=== '''Steps Towards Ganymede''' ===
 
* [[DSDP/TM/3.0 Ramp down Plan for Ganymede]]
 
* [[DSDP/TM/3.0 Ramp down Plan for Ganymede]]
** We have RC1, so from now on EVERY checkin must be tracked in bugzilla, even simple cleanups (except documentation-only or unittest-only things)
+
** We have RC1, so from now on EVERY checkin must be tracked in bugzilla, even simple cleanups (except documentation-only or unittest-only things). Also applies to "incubation". Rules do not apply to TCF
 
** there MUST be a +1 vote on the bug (though the vote can happen after the fact)
 
** there MUST be a +1 vote on the bug (though the vote can happen after the fact)
 
** Please do NOT only ask me for reviewing, it doesn't scale
 
** Please do NOT only ask me for reviewing, it doesn't scale
Line 47: Line 48:
 
* '''Testing'''
 
* '''Testing'''
 
** We should do another round of coordinated testing with TM 3.0 RC2 in order to find bugs important enough to get fixed
 
** We should do another round of coordinated testing with TM 3.0 RC2 in order to find bugs important enough to get fixed
** Who can organize it?
+
** Who can organize it? - '''DaveD''' write up wiki page, '''Martin''' E-mail testers
 +
 
  
 
* '''Bug Triage'''
 
* '''Bug Triage'''
Line 77: Line 79:
 
*** Run Findbugs -- Update site on http://findbugs.cs.umd.edu/eclipse, Quickdocs by Rado in [http://dev.eclipse.org/mhonarc/lists/dsdp-tm-dev/msg01869.html tm-dev E-Mail]
 
*** Run Findbugs -- Update site on http://findbugs.cs.umd.edu/eclipse, Quickdocs by Rado in [http://dev.eclipse.org/mhonarc/lists/dsdp-tm-dev/msg01869.html tm-dev E-Mail]
 
**** Question about filtering false positives posted on fb-discuss [https://mailman.cs.umd.edu/pipermail/findbugs-discuss/2008-May/002374.html link]
 
**** Question about filtering false positives posted on fb-discuss [https://mailman.cs.umd.edu/pipermail/findbugs-discuss/2008-May/002374.html link]
** '''TM Website:''' revamp should really be complete for Ganymede! '''AI Kevin'''?
+
** '''TM Website:''' revamp should really be complete for Ganymede! Kevin and Martin won't have time. DaveD will try to get to it, but cannot promise.
  
 
=== What will happen after Ganymede? ===
 
=== What will happen after Ganymede? ===
Line 88: Line 90:
 
** Bring TCF to Maturity
 
** Bring TCF to Maturity
 
** Quality: Add unittest coverage, reduce bloat, improve performance, streamline APIs
 
** Quality: Add unittest coverage, reduce bloat, improve performance, streamline APIs
* Shoot for TM 3.1 or 4.0?
+
* Shoot for TM 3.1 or 4.0? - DaveD: Attempt 3.1; Xuan: lots of classes moved to "internal" e.g. SystemRemoteCommand; Martin: discuss such issues on bugzilla
 +
** Martin: Agree, a non-breaking release seems better; Dave: Will increase community confidence
 +
** Related: Move to Java5? - DaveM: Keep Java 1.4 on the Server, allow 1.5 on the client
 +
** Martin: Parallel 4.0 and 3.1 development? - DaveD: Try not to do that!
  
 +
=== Questions ===
 +
* '''DaveD''' context menu on SystemView / TeamView: something going on with the way how items are being added. Bringing up the context menu immediately after startup shows some debugger related stuff. Martin: {{Bug|208062}}
  
=== Committer Status and Report ===
 
* '''Javier''' - New Employer in Barcelona
 
* '''DaveD''' -
 
** {{bug|189274}} Import/export connections
 
* '''DaveM:''' -
 
** JUnit should autotest dstore against old servers as well - an IBM thing
 
* '''Kevin:''' -
 
* '''Xuan:''' -
 
* '''Martin:''' - {{bug|215301}} New project plan format -- Need to put Project Plan on the Web; '''AI Martin''' Update Releng scripts to automatically run unit tests at night
 
** Commons.Net 1.5.0 not yet released - '''AI Martin''' create placeholder CQ
 
** Next tasks - Move SubsystemConfiguration implementation into non-UI; Lazy loading of SubsystemConfiguration (use the Proxy more); Lazy loading of UI adapters; some exceptions observed by Uwe (Wind River) in our nightly wheels
 
** {{Bug|221190}} EFS getChild() with relative path - perhaps need to check whether our impl still works properly
 
** GSoC student Takuya Miyamoto for {{Bug|185925}}
 
* '''Uwe:''' -
 
* '''Rupen:''' -
 
* '''Michael:''' -
 
* '''Eugene:''' -
 
* '''Rado:''' -
 
* '''Felix:''' -
 
* '''Questions'''
 
 
=== '''Old Stuff''' ===
 
* '''Unit Tests''' - next priority after API
 
* Migrating to new Releng on dsdp.eclipse.org (adopt P2, nightly tests, signing etc)
 
* {{Bug|221211}} MultiStatus for IFileService batch operations - exception on first failure for modifying operations. No exceptions/multistatus on non-modifying operations. '''AI DaveD/DaveM''' update Javadocs when it's clear what we want
 
* {{Bug|220379}} Make DStoreFileService API - encodingHandler - wait on reporter - is this fixed now that IService is IAdaptable as per {{bug|226262}}? '''AI DaveM''' ask reporters
 
* {{Bug|220547}} SimpleSystemMessage message ID - globally unique message id vs. plugin-local status - Dialog Title: MessageID if globally unique / generic title based on severity if plugin-local - '''AI DaveM''' think about
 
 
* '''Bugzilla''':
 
** [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bug_severity=blocker&bug_severity=critical&bug_severity=major&cmdtype=doit Severity Major], [https://bugs.eclipse.org/bugs/buglist.cgi?query_format=advanced&classification=DSDP&product=Target+Management&bug_status=UNCONFIRMED&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&cmdtype=doit&field0-0-0=priority&type0-0-0=regexp&value0-0-0=P%5B12%5D&field0-0-1=bug_severity&type0-0-1=regexp&value0-0-1=blocker%7Ccritical%7Cmajor hi-priority]
 
 
* '''Quality, Backlog and Unit Tests'''
 
** Remember our 2-fix-per-week / 3 unittests-per-milestone plan
 
** Since our [[DSDP/TM/Face-to-face Meeting Toronto 17-Sep-2007]] it's now 31 weeks / 5 milestones, so each committer is due 62 fixes / 10 unittests
 
** Current situation is on [https://bugs.eclipse.org/bugs/report.cgi?x_axis_field=&y_axis_field=assigned_to&z_axis_field=&query_format=report-table&classification=DSDP&product=Target+Management&bug_status=RESOLVED&bug_status=VERIFIED&bug_status=CLOSED&chfieldfrom=2007-09-17&chfieldto=Now&chfield=bug_status&chfieldvalue=RESOLVED&format=table&action=wrap&negate0=1&field0-0-0=resolution&type0-0-0=equals&value0-0-0=DUPLICATE this bugzilla report]
 
** Unittests: when adding a test, please add the tag '''//-test-author:YourName''' in front of it
 
*** DaveM - 5
 
*** DaveD - 22
 
*** KevinD - 11
 
*** MartinO - 3
 
*** TobiasS - 3
 
*** UweS - 7
 
*** XuanC - 51
 
*** '''Total - 101'''
 
*** Javier - 0
 
  
 
== Vacation, away ==  
 
== Vacation, away ==  
* Rupen vacation May 8-15
+
* Martin public holiday Thursday May 22
* Martin public holidays Monday May 12, Thursday May 22
+
* Martin NOT in Ottawa for [[E4/Summit]] May 22-23
* Martin NOT in Ottawa for [[E4/Summit]] May 22-23 .. want to meet in Toronto?
+
* DaveD Monday May 26 public holiday in US
* Martin vacation June 11 - 22
+
* Martin vacation June 11 - 22 -- '''AI Martin''' finish and test the new Build scripts on dsdp.eclipse.org till then
 
* DaveM vacation June 16 - 20
 
* DaveM vacation June 16 - 20
* DaveD in July -- '''AI Martin''' finish and test the new Build scripts on dsdp.eclipse.org till then
+
* DaveD in July
  
 
== Action Items ==
 
== Action Items ==
 
* [[DSDP/TM/Committer Phone Meeting 30-Apr-2008#Action_Items|Last Meeting]] Action Items
 
* [[DSDP/TM/Committer Phone Meeting 30-Apr-2008#Action_Items|Last Meeting]] Action Items
* '''Everyone''': '''Fix PII Issues'''; '''Add @noextend etc''' according to [[DSDP/TM/Code Ownership]] table; Review [[DSDP/TM/3.0 Ramp down Plan for Ganymede]]
+
* '''Everyone''': '''Add @noextend etc''' according to [[DSDP/TM/Code Ownership]] table;
* '''DaveD''': Merge Rupens patches; Update {{bug|221211}} IFileService multi-commands Javadoc
+
** Triage 3.0 and earlier assigned bugs and move to 3.0.1 what we can
* '''DaveM''': create a new "Future" bug for dstore protocol handshake, cloned from {{bug|220892}}
+
** Bug fixes, cleanup, unittests
* '''Kevin''': Website Updates
+
* '''DaveD''': Investigate {{Bug|230821}} exceptions in IRemoteFileSubSystem; write up coordinated testing Wiki page until Thurs May 22 evening
* '''Rupen''': Attach prototype patch for {{bug|227213}}
+
* '''DaveM''': Fix {{Bug|231209}} IRemoteFile.getSystemConnection()
* '''Xuan''': Upgrade Skype to fix quality issues; Use Kevin's Properties for Unit Tests
+
* '''Kevin''': -
* '''Martin''': New Project Plan; Commons Net Placeholder CQ; UI/Non-UI Splitting; finish new releng; Look at PropertyDescriptor issues; unit tests
+
* '''Rupen''': -
* '''Javier''': add unit tests; document the Symbian internal test setup similar to [https://bugs.eclipse.org/bugs/show_bug.cgi?id=204138#c20 CVS] -- see also [[CVS_Development#Testing]]
+
* '''Xuan''': -
 +
* '''Martin''': Fix {{Bug|233068}} Discovery reexport; Prepare Release Review material; New Project Plan; Critical EFS bugs; UI/Non-UI Splitting; Finish new Releng and tell DaveD; E-Mail potential participants of Coordinated Testing; Commons Net Placeholder CQ
 +
* '''Javier''': Hi-PRI FTP BUGS
 
* '''Michael''': Terminal improvements
 
* '''Michael''': Terminal improvements
* '''Uwe''':  
+
* '''Uwe''': -
* '''Rado''':
+
* '''Rado''': Fix {{Bug|230919}} IFileService.delete()
* '''Felix''':
+
* '''Felix''': -
 
+
* '''Eugene''': -
  
 
== Next Meeting ==
 
== Next Meeting ==

Latest revision as of 12:56, 21 May 2008

Meeting Title: TM Committer Phone Meeting
Date & Time: Wednesday May 21, 2008 at 1500 UTC / 0800 SFO / 1000 Rochester / 1100 Toronto / 1600 London / 1700 Salzburg
Dial-in: Join Skypecast

Backup dial-in: International +44 (0)1452 567588 / Freephone +1 (866) 6161738 / UK 08712460713 / Passcode: 0587322148 #

MartinO to start conference call - please dial in using the numbers above.
Please be available for Skype Chat in parallel to the call. MartinO will start Skype chat just prior to call.
Skype dial-in: martin.oberhuber, ddykstal (or david_dykstal), david-k-mcknight, kevin.j.doyle, xuan.chen886, rupen.mardirossian, javier.montalvoorus, eugenetarassov, michael_scharf, uwe.stieber, radoslav.gerganov, wrsfburton.

Attendees

  • IBM - Xuan Chen, Dave McKnight, Dave Dykstal, Rupen Mardirossian, Kevin Doyle
  • Wind River - Martin Oberhuber, Uwe Stieber
  • ProSyst - Radoslav Gerganov
  • Regrets this time: Michael Scharf, Eugene Tarassov, Felix Burton, Javier Montalvo Orus

Notes

  • Last meeting: DSDP/TM/Committer Phone Meeting 30-Apr-2008
  • Skype Call Quality
    • Skypecast quality was OK in the beginning with up to 4 participants, but became unbearable shortly later. Rado was unable to dial into the Skypecast (sitting behind an HTTP proxy).
    • Normal Skype conference worked very fine for all 8 participants this time, but 8 really is the limit here (9th person trying to accept invitation cannot join).

New Stuff

  • Late API Change Requests
    • We really need to draw a finishing line now!
    • bug 233068 - [api][breaking] Remove tm.discovery.edit reexport of EMF -- accepted, Martin
    • bug 231209 - [api][breaking] IRemoteFile.getSystemConnection() should be changed to IRemoteFile.getHost() -- accepted, DaveM
    • bug 230821 - [api][breaking] IRemoteFileSubsystem is inconsistent with IFileService -- DaveD to investigate impact, committers in favor of the change if doable.
    • bug 230919 - [api][breaking][files] IFileService.delete() should not return a boolean -- Rado: can handle all cases with exceptions, inconvenient to do both. Martin: boolean value would hardly ever be used. Rado: Not checking return values is bad practice. Martin: It boils down to the question whether it is an error to try and delete a file that's not there. -- accepted, Rado to make the change
    • bug 212742 - [api] Need a Utility to send commands and receive output without prompt -- DaveM: We have RemoteCommandShellOperation with BEGIN-END-TAGS (but how reliable is it?) -- denied, due to lack of time
    • Changes to go into RC2

Steps Towards Ganymede

  • DSDP/TM/3.0 Ramp down Plan for Ganymede
    • We have RC1, so from now on EVERY checkin must be tracked in bugzilla, even simple cleanups (except documentation-only or unittest-only things). Also applies to "incubation". Rules do not apply to TCF
    • there MUST be a +1 vote on the bug (though the vote can happen after the fact)
    • Please do NOT only ask me for reviewing, it doesn't scale
  • Testing
    • We should do another round of coordinated testing with TM 3.0 RC2 in order to find bugs important enough to get fixed
    • Who can organize it? - DaveD write up wiki page, Martin E-mail testers


  • Bug Triage
    • Please take the time and triage your still 3.0 assigned bugs to see which of them can be deferred to 3.0.1
    • All "3.0" assigned bugs must be triaged now and either assigned a proper RC milestone or 3.0.1 (with the exception of Documentation-only fixes)
  • New and Noteworthy
    • I'll need a list of new features for a New&Noteworthy, and for the Release Review. Release Review material is due next week.
    • See CDT/User/NewIn50 for example; for TM, I put most news into our Build Notes already, so it should be possible to compile the N&N out of the build notes
    • For the N&N, please make screenshots and work on "your" new features, e.g.
      • Useractions
  • Hi-priority-bugs
    • Severity Major, Hi-Priority
    • Current Statistics: 1 blocker (Martin), 1 critical (Martin), 10 major (DaveD, DaveM, Kevin, Martin, Michael), 20 other P2's
    • Please fix the known hi-priority bugs as soon as possible -- best before the coordinated testing on RC2, to ensure we get good test coverage
    • Community contributions: Only few unapplied patches right now, should apply now if possible (see also the Bug Process Page for a query)
  • Cleanup Work
    • Besides the hi-priority issues, there is a LOT of cleanup to do:
      • New releng scripts on dsdp.eclipse.org (Martin: must-have before his vacation)
      • Copyright Year Updates (Run the releng Copyrights Tool)
      • API Tooling Javadocs(@noextend and friends) - bug 227368, bug 225529 comment 4 and bug 225529 comment 6 - DSDP/TM/Code Ownership
      • Review and improve Userdocs; we have several open bugs here
      • Review and improve Javadocs; lots still missing
      • Fix broken hyperlinks in the docs, fix HTML/XML errors (which can make the indexer not work)
      • Get rid of compiler warnings wherever possible
      • Add more Unittests -- ideally write an accompanying unittest for every bugfix
      • Run Findbugs -- Update site on http://findbugs.cs.umd.edu/eclipse, Quickdocs by Rado in tm-dev E-Mail
        • Question about filtering false positives posted on fb-discuss link
    • TM Website: revamp should really be complete for Ganymede! Kevin and Martin won't have time. DaveD will try to get to it, but cannot promise.

What will happen after Ganymede?

  • Propose working on TM 3.0.1 (bugfixes only) in HEAD until fall
  • Start coming up with "Big Rocks" to address in the next cycle; re-use the TM Future Planning page? Some examples:
    • Connection Grouping - be Multicore aware
    • Rewrite the Tableview to be really aware of Properties
    • More dynamic subsystem enablement and configuration (e.g. detect UNIX vs Windows automatically at runtime)
    • Hi-Performance, caching EFS provider
    • Bring TCF to Maturity
    • Quality: Add unittest coverage, reduce bloat, improve performance, streamline APIs
  • Shoot for TM 3.1 or 4.0? - DaveD: Attempt 3.1; Xuan: lots of classes moved to "internal" e.g. SystemRemoteCommand; Martin: discuss such issues on bugzilla
    • Martin: Agree, a non-breaking release seems better; Dave: Will increase community confidence
    • Related: Move to Java5? - DaveM: Keep Java 1.4 on the Server, allow 1.5 on the client
    • Martin: Parallel 4.0 and 3.1 development? - DaveD: Try not to do that!

Questions

  • DaveD context menu on SystemView / TeamView: something going on with the way how items are being added. Bringing up the context menu immediately after startup shows some debugger related stuff. Martin: bug 208062


Vacation, away

  • Martin public holiday Thursday May 22
  • Martin NOT in Ottawa for E4/Summit May 22-23
  • DaveD Monday May 26 public holiday in US
  • Martin vacation June 11 - 22 -- AI Martin finish and test the new Build scripts on dsdp.eclipse.org till then
  • DaveM vacation June 16 - 20
  • DaveD in July

Action Items

  • Last Meeting Action Items
  • Everyone: Add @noextend etc according to DSDP/TM/Code Ownership table;
    • Triage 3.0 and earlier assigned bugs and move to 3.0.1 what we can
    • Bug fixes, cleanup, unittests
  • DaveD: Investigate bug 230821 exceptions in IRemoteFileSubSystem; write up coordinated testing Wiki page until Thurs May 22 evening
  • DaveM: Fix bug 231209 IRemoteFile.getSystemConnection()
  • Kevin: -
  • Rupen: -
  • Xuan: -
  • Martin: Fix bug 233068 Discovery reexport; Prepare Release Review material; New Project Plan; Critical EFS bugs; UI/Non-UI Splitting; Finish new Releng and tell DaveD; E-Mail potential participants of Coordinated Testing; Commons Net Placeholder CQ
  • Javier: Hi-PRI FTP BUGS
  • Michael: Terminal improvements
  • Uwe: -
  • Rado: Fix bug 230919 IFileService.delete()
  • Felix: -
  • Eugene: -

Next Meeting

Copyright © Eclipse Foundation, Inc. All Rights Reserved.