MoM 2016 05 13

Attendees

• Alexandre Montplaisir (EfficiOS)
• Jonathan Rajotte (EfficiOS)
• Bernd Hufmann (Ericsson)
• Patrick Tassé (Ericsson)
• Matthew Khouzam (Ericsson)
• Marc-André Laperle (Ericsson)
• Jean-Christian Kouamé (Ericsson)
• Simon Marchi (Ericsson)
• Geneviève Bastien (Polytechnique)

Minutes

Go over action items of previous meeting

• Custom state value patches
  • One out of 3 patch was merged. The other two should be straightforward to review.
  • The approach taken was to have one implementation and two interfaces, one for reading and one for writing. Similar to StateSystem and ITmfStateSystem.

• State system node cache patch
  • Matthew's patch was merged
  • Alex's patch could be rebased and merged later, post-Neon, no rush.

Status of features to merge before the feature freeze

Feature freeze is next Wednesday, May 18th, at noon!

Loïc's state system patch

• Geneviève wanted it to go in, Bernd and Alex were a bit weary. Note: it was mentioned last meeting that this was discussed with Loïc, and this wasn't planned for Neon:
• It could go in master soon after the stable branch is branched.

Debug-info analysis

• Patch to update to the final LTTng 2.8 event names was merged.
• Only change remaining is to make use of the is_pic field to determine if the binary is PIC or not. This is not a big change, could be merged after the freeze.
• Documentation patch will be updated post-feature-freeze too.

LTTng-Analyses integration

• The review on the "big patch" is advancing, but is not done yet.
• The other important patches in that branch to have merged for the FF are:
  • lttng: Add configuration files to call LTTng-Analyses scripts
  • analysis.lami: Add support for user-defined LAMI analyses
  • analysis.lami: Show one view per report

All the other patches are not as invasive or important, so they could be rebased after the ones above, and merged post-FF.

Custom state values

As mentioned at the beginning, there are still 2 patches left:

• ss: Introduce a safe byte buffer wrapper for use by custom state values
• ss: Add support for custom state values

Callstack changes

All patches are up. The "getQuarks()" patch is actually needed for this branch, so Alex will look at it.

Timestamp formatting patch from Patrick

Patch to be uploaded to Gerrit, Geneviève will review.

Error reporting / messaging API

• One of the patches in the LTTng-Analyses branch triggered a discussion about a generic error report and messaging API.
  • Could be based on the Platform's notifications API:
  • https://wiki.eclipse.org/Platform_UI/Notifications
  • http://wiki.eclipse.org/Platform_UI_Error_Handling
• This is way too late in the cycle to integrate that for Neon though.
• The patch could be fixed post-freeze. But it's not clear at this time what the approach should be.

Stable branch

• A stable Neon branch will be created next week, right after the RC1 release.
• Changes to go for the Neon release should be committed to master first, then cherry-picked to the stable branch, as usual.
• A discussion about the branching strategy (master, 3.0, etc.) should be held at the next meeting.
• No API breaks should be made in master until then.

Action items

Everyone:

• Get all the patches in for the feature freeze!
• Discuss the API/branching strategy for Oxygen at the next meeting