/
TSC Meeting Notes 2019-06-13
TSC Meeting Notes 2019-06-13
Attending:
- Cary Phillips - ILM
- Larry Gritz - SPI
- Rod Bogart - Epic
- Peter Hillman - Weta
- Kimball Thurston - Weta
- Nick Porcino - Oculus
- Christina Tempelaar-Lietz - Epic
- John Mertic - Linux Foundation
Discussion:
- Welcome new TSC members:
- Nick Porcino - Oculus
- Christina Tempelaar-Lietz - Epic
- Kimball Thurston - Weta
- Piotr was invited by hasn’t responded.
- Project goals by SIGGRAPH:
- Reach “adopted” status, seems attainable.
- CII badge progress is at 68%
- Need CI setup
- Need static/dynamic analysis setup
- Need to address security issues/policies
- Put out a new release, version 2.4:
- Address outstanding CVE’s
- Document existing CVE’s
- Compiler warnings
- CMake fixes
- Acknowledge most Issues and PRs; many can be closed as is.
- Proposal/position on the future of Imath; go into the BOF prepared to discuss.
- Reach “adopted” status, seems attainable.
- “Guide to the OpenEXR Project”
- GOVERNANCE.md, CONTRIBUTING.md, etc.
- Clarify project roles and clarify terminology:
- Committer vs. TSC
- Nick Rasmussen is an example of a committer not on the TSC.
- Need to clarify terms of service, how long do TSC members serve.
- OCIO meeting notes designate a PR Reviewer?
- Ad-hoc for that project to ensure fast turn around due to large number of contributions
- May want some form of this to ensure that PRs are reviewed
- Committer vs. TSC
- Legal:
- Who keeps track of CLA’s on file?
- Should just be part of the process (CLA, DCO checks)
- Old PRs, what to do about obtaining CLAs prior to merge
- Need to check w/ ILM legal just to clarify old copyright / CLA
- Does OCIO have a policy of accepting small fixes without a CLA? No, that’s OIIO. All ASWF/Linux Foundation projects require CLA’s for every PR, no matter how small.
- Who keeps track of CLA’s on file?
- CII Badge requires “acknowledgment of bug reports”. What should our policy be?
- Standard use of GitHub labels, documented in CONTRIBUTING.
- Use of GitHub Milestones to indicate priority.
- Status of the CII Best Practices Badge:
- Must ensure timely response to posts on openexr-dev
- What does it take to ensure that the website now uses https?
- Security:
- security@openexr.com, info@openexr.com
- John to arrange setting up the forwards to private tsc
- Outstanding CVE’s:
- One has a PR that needs to be merged.
- Other CVE issues have been resolved but the issues were never closed.
- CVE’s should be documented in CHANGES.md.
- OpenEXR needs a designated security expert
- Cryptographic protocols are not applicable.
- security@openexr.com, info@openexr.com
- Ready to set up Azure?
- Compiler Warning flags
- -Wall with GCC 7/C++17 on Ubuntu gives a handful of warnings, straightforward to fix.
- What about Windows?
- Code coverage, static analysis, dynamic analysis: SonarCloud?
- Just needs to have been done at least prior to release (as opposed to fully integrated to release schedule)
- Test policy adherence?
- Should we update source code copyright notices?
- Cary will check with Lucasfilm Business Affairs.
- What’s the status of moving the github repository?
- TODO’s and action items:
- Label issues and PR’s
- Gives perception of response
- Use milestones instead to mark issues for including in next release
- Need to be more aggressive with closing issues that had a back and forth resulting in a consensus there’s nothing to do
- Standard comment about a house-cleaning event in migration to ASWF
- Merge/act on existing PR’s
- Fix compiler warnings
- SonarCloud: Static and dynamic analysis
- Azure
- Web site/documentation
- JM: Can be moved independently, whenever ready
- CP: wants to re-do web site to make it easier to maintain
- Email addresses for info & security
- Imath white paper
- Christina - will work on SonarCloud
- Kimball - will work on PRs, cmake, compile warnings
- Nick - will review PRs as well
- Label issues and PR’s