Agile Code Reviews: Supporting collaboration and improving production uptime for agile development teams
1.
2. Agile Code Reviews Supporting collaboration and improving production uptime for agile development teams Sean deBardelaben Sr. Software Developer
3. What is AWS? How many folks here have come to expect and rely on wireless connectively to add value to some aspect of their lives?
4. What is AWS? Nintendo 3DS owners get Wi-Fi freebie, CNET, 6/2/11 Nintendo 3DS owners will get free Wi-Fi access at 25,000 locations around the U.S., starting Tuesday. The wireless access will be available at nearly 200 malls owned by Simon Property Group, Nintendo said today, as well as in more than 1,000 Best Buy locations. Nintendo has also inked a deal that will allow 3DS owners to connect to AT&T Wi-Fi hot spots at no charge.
7. What are our business challenges? Network growth Security Custom requirements Bandwidth utilization IPv6
8. Agile at AWS Build projects around motivated individuals. Give them the environment and support they need, and trust them to get the job done. Continuous attention to technical excellence and good design enhances agility. The better we are at what we do, the less frequently we have to cut corners in order to go fast. Ron Jeffries - http://xprogramming.com/articles/quality-speed-tradeoff-or-skill-deficit/
9. Challenges for the development team Scale infrastructure Provide Cross Training Collaborate to improve the reliability of our releases
10. Show of hands: How many folks have ever done a code review? How many folks are thinking about asking their developers to add code reviews to their current activities?
21. Tools Selected FishEye, Crucible and Crowd Benefits: interfaces with JIRA ease of use support online review all comments automatically archived tracks time spent, amount of code reviewed powerful and flexible (eg. RSS feed of code updates) 13
22. Tool Versions Crowd Version 2.1.0 Build #431 FishEye and Crucible Version:2.4.4 Build:20110111045701 Crowd used for user mapping FishEye used for source code browsing Crucible used for code review process 14
23. AT&T Proprietary & Confidential Light weight Process Author / Developer Driven Review for standards Review for business process adherence Review for training Reviewers on scrum team Page 15
24. Overview of the process cvscommit –c “SAR-1025 Updating makefile” Makefile author creates Review from Jira Ticket author addresses concerns review session is marked complete author moves JIRA task to the DONE state ~10% overhead per reviewer AT&T Proprietary & Confidential Page 16
25.
26.
27. Pilot Technical Challenges: Index a very large cvs repository Index a growing svn repository Personnel Challenges: Get engineers involved in process Collect meaningful feedback
41. Release Reports select revisions from dir "/" where ((comment matches "TZ-1420" or comment matches "TZ-1635" or comment matches "TZ-1623" or comment matches "TZ-1621" or comment matches "TZ-1619") and date >= 2011-03-01T06:00:00.00Z) order by date desc group by changeset return path, revision, author, date, csid, totalLines, linesAdded, linesRemoved, reviews 27
42. Scrum Team Reports select revisions from dir "/" where ((comment matches "DOGS" or author in(bs9162,dr213u,ri950d,rk9847,sd9393,sg002j,sr170u,tm214k,wwiggins)) and date in [now-P21D , now ] ) order by date desc group by changeset return csid, author ,count(distinct reviews)