ROS Quality Assurance Working Group meeting minutes Kick Off Meeting
Time: 9 a.m. UTC and 5 p.m. UTC
Participants:
9 a.m. UTC Group
- Adam Alami
- Akshay Jain
- Andrzej Wasowski
- Geoffrey Biggs
- Kei Okada
5 p.m. UTC Group
- Adam Alami
- Aaditya Saraiya
- David Bensoussan
- Dirk Thomas
- Gijs van der Hoorn
- Ian McMahon
- Luca Marchionni
- Matt Droter
- Shaun Edwards
- Victor Lopez
Notes:
ROSIN quality assurance (QA) initiatives were discussed. Below is a summary of the discussion. The following problems and solutions were discussed:
ROS Quality Assurance Working Group – Kick Off Meeting Agenda.pdf (31.0 KB)
ROS Working Group Mission Statement and Scope of Work.pdf (15.3 KB)
ROS Quality Assurance Initiatives.pdf (70.1 KB)
-
Problem: There is a lack of a centralized source for community quality assurance practices, knowledge, and collaboration.
- Solutions:
- Quality Hub: Would inform about existing practices and would be a central “go-to” place for QA knowledge sharing (documentation of QA practices)
- Discussion:
- Make the content of the website educational and easy to digest.
- The content should capture the knowledge most engineers do not already have.
- The website should be incorporated into the existing infrastructure (i.e., Wiki, ROS Answers).
- Discussion:
- Quality Discourse: A dedicated QA forum
- Discussion:
- A chapter was created for Quality Assurance.
- Discussion:
- Quality Hub: Would inform about existing practices and would be a central “go-to” place for QA knowledge sharing (documentation of QA practices)
- Solutions:
-
Problem: The quality of packages is not visible.
- Solution:
- Make ROS packages’ quality visible.
1. Discussion:- A “Quality Stamp” was suggested. We can use a script (leverage existing Github feature) to generate the stamp.
- Enforce the stamp creation in the distribution process.
- Make ROS packages’ quality visible.
- Solution:
-
Problem: Inconsistent practice of code review
- Solution:
- Energize the code review process.
- Discussion:
- It was recommended to use the combination of a tool and peer review.
- It was suggested to create a website (i.e., similar to answers.org) dedicated to code review.
- Motivation was discussed. What would motivate community members to do code review? A reward system similar to the “Karma” system was discussed.
- Review and update the current standards.
- Possibly provide tutorials on how to review a pull request.
- Discussion:
- Energize the code review process.
- Solution:
-
Problem: Recruiting maintainers is a “real problem” for ROS and ROS-I. This has led to an increasing number of orphan packages. This is a capacity issue within the core team. The team is struggling to attract new maintainers. The team capacity does not reflect the maintenance effort required. This is also applicable to non-core packages. There is a lack of willingness to contribute to packages’ maintenance. It is a challenge to attract and retain new maintainers.
- Solutions:
- Propose and implement a funding model for the maintenance activities.
- Organize periodic campaigns to recruit new maintainers for both core and non-core packages.
- Define an onboarding process for both core and non-core community members.
- Document the onboarding process, including online educational materials (i.e., tutorials).
- Implement the onboarding process.
- Formalize the code ownership process.
1. Discussion:- Reward maintainers with Github Bounty.
- Identify a sustainability strategy.
- The possibility of using ROSIN FTPs to finance maintenance was discussed.
Links:
- Solutions:
On maintaining orphaned packages
- https://github.com/ros-infrastructure/rep/pull/150
- https://roscon.ros.org/2017/presentations/ROSCon%202017%20Lightning%20113.pdf
Recruiting more maintainers