Work in Progress
Lesson Learned
Retrospective
• Identification of issues - earlier than later by tracking Risks
• Pair-wise testing performed by the project teams
• 72h stability test run
Need for continuous integration testing:
- Some experience is based on HPA testing on vCPE use case. vCPE use case, even though, it was tested on Amsterdam successfully, it took a while to make vCPE work on Beijing components. It could be that there was some regression. Since HPA depends on vCPE use case, it could not be tested till the end. This can be avoided if previously tested use cases are verified continuously. Automation of use case testing & continuous integration verifications are key to add new functionalities in future releases.
Lesson Learned Areas
• Communication – Wiki, Mailing Lists, IRC
- When you are a mailing list moderator do not approve subscription requests from accounts with obviously pornographic domain names. (yes, this really happened. recently.)
- Lack of transparency on LF IT ticket raised by the community. The LF RT tools has poor capability to email properly the whole set of information. The ONAP community has no visibility on what LF IT is currently working on.
- Feedback compiled by Kenny Paulfrom ONS-NA: (See TSC 2018-04-05 Agenda slides)
Onboarding an engineer is very hard; The wiki isn’t laid out well and is confusing, needs to be cleaned up
Responsiveness from existing Community members answering new contributor questions can be a challenge
- All approved projects should be required to post meeting minutes to the wiki
The volume of untagged emails makes filtering almost impossible
- Need absolutely clear and unambiguous validation that RocketChat can be used via https from China and most companies without requiring a VPN, or the use of an alternative device/network -kenny:. Must be secured if setup by LF (simply b/c of well known pwd).
- Would like the Community to decide whether the use of IRC as the "official" minute mechanism is a practice worth continuing or not; very few people are using it, many can't use it and unlike other communities, virtually no one contributes to the minutes. Compare that with the fact that everyone can use zoom chat and there is always a great deal of contribution being performed there. -kenny
- WIKI Help: things are hard to find unless you type the right keywords or have the proper URL. Some information no more accurate. Matter of organization, not matter of quantity of information. Recommendation to use ReadTheDocs to start versus the wiki?
- WIKI: Onboarding new community members is the challenge.
- Is there a good enough flow of information-communication between sub-committees and projects? Overall: yes. Sub-committee feedback of to the TSC is not systematic(as we wish).
- Expectation from sub-sub-committee back to the team and then driving the execution. Use-case owner is missing. Use-case team to fill a checklist? We may defined what that role is.
• Labs & Test Environment
- After M4 code freeze, the community is spending a lot of time to get the Health Check sanity test to pass. HealthCheck is the kind of automated test that MUST pass everyday all along the release lifecycle.
- Despite the effort made by some ONAP partners in providing Labs, we have reached the limit on current labs infrastructure. This is preventing further needed testing (like test job during the verify phase).
- Need to develop a full Agile CI-CD pipeline. Full Chain to run automatically the sanity HC, CSIT, E2E. Everything running continuously.
- How to make better usage of XCI-CD environment (OPNFV,...)
- CSIT tests under integration repo must branch early to support the other component branches and their test automation (at least must branch on code freeze deadline to provide enough time to fix the CSIT tests in the release branches).
- Supporting HEAT and OOM based deployments is getting harder with duplicate maintenance of the config values (it may help if we can somehow abstract such duplication of config values).
More ONS-NA Feedback: The amount of time to setup the dev environment is a barrier to engagement - Developers don’t want to invest a lot of time on that.
Improve CSIT coverage to cover all features that the project delivers and to reduce manual testing (if any)
• Release Management
- Progress check between M2 and M4 of intermediate development by release manager might help to improve the quality of the final product (for example scheduling the demos of each component to show the current progress at M3 deadline might be right choice too). This avoids any misunderstanding of the features or requirements by the dev team and can receive feedback from the usecase or architecture teams to improve them when there is still time until code freeze.
- Lack of formality to freeze model and scope increasing risks to put any milestone in jeopardy
- Reset expectation on Milestone meaning (enforcement?).
- Grouping of Projects (staggered): to address Release dependency.
- Idea of 3 months project (push back at TSC)
- Defining another milestone (no that good idea?)
- Start early pair-wise testing?
• JIRA Management
- Often lack of updated information in "Status" field, that prevent to know if someone is working on a defect. This issue varies depending on project and people.
- Adopt a singe JIRA workflow. Currently 2 are in place (1 from openecomp, 1 simple from Open-o).
- PTLs are owner of the scope of iteration.
- Review the 2 JIRA workflows and define 1 workflow for ALL.
• Code Review
- Requires active committers and more active code reviewers (right now only few committers or reviewers are always reviewing and merging the code).
• Development Infrastructure
- The LF toolchain that is currently in place, do allow to merge in master code that has not been thoroughly tested. This leads to massive disruption in the testing.
- Nexus 3 slowness. This has been impacting Integration Team tremendously as it tooks 3-4 more times just to download Docker images.
- Current 1/3 party scan security tool (Nexus IQ) do not work for Python and JavaScript packages and thus prevent the community to have an holistic view on vulnerabilities.
- For the record, it was decided that the community would not account for JavaScript in using Sonar (code coverage) in Beijing Release.
- Slowness of the full IT chain (jenkins, nexus, wiki,..)
- Local testing: how easy to setup an env for a developer to perform its own testing before submitting code. Need reference VNF, AAI, ... (too much time spend to setup environment). Idea of lab reference to be used as a model for configuration. (currently SB-07 serves that purpose).
- Pair wise testing for a great value added in Beijing release.
- 72 hours helps to identify defects. Docker Upgrades went fine.
- Backup and restore capacity for SB 04-07 in Windriver? Have we ever asked Windriver?
- Feature parity on LABS (do not over taxe Windriver).
- Nexus IQ scan performed during the verify. If error then block the build.
- Idea of parallel on a single job. Currently atomic at build level. To investigate-optimize Jenkins Jobs. (time to build the VMs, ...).
• PTL Role
- More ONS-NA feedback:
“Either you commit or you forfeit. There really isn’t any middle ground here."
If you can’t attend a meeting, assign a delegate. If you send a delegate more often than you attend, you shouldn’t be a PTL.
PTL-cross PTL discussion: feeling of not involvement in some decision. Having a place for PTLs to speak up.
• Architecture
- Need for simplifying ONAP Micro-Service architecture by leveraging best practices from Industry :
- Certificate enrollment (for Mutual TLS communication among services) in Beijing was manual, time consuming and error prone. Need automation of certificate enrollment is needed.
- Scale-out of ONAP is removed from the scope in Beijing release. Services that attempt do it has some learnings. Scale-Out/Load balancing/Circuit-breaking/DB-sync related functionality should be removed from actual micro services to sidecars/side-kicks to avoid each developer spending time and getting it right.
- Need share as much as possible the building blocks with regarding to the implementation of new ONAP Micro-services
- While the community focus on more about the micro-service arch. the diversity of building blocks for each micro-services complicates the debug/maintain effort for everyone willing to exercise/develop existing/new use cases. e.g. While java and python language are prevalent used for each most of micro-services, some other language binding(e.g. golang) was introduced as well. so whoever want to debug those microservices will have to either seek the help from community (with no promise to get timely support) or have to learn those language/framework,etc. This apparently hinder the adoption of some ONAP components, even they complies to micro-services architecture. I suggest each team should evaluate/review carefully what kind of building blocks are utilized instead of the choice to the contributors only. The principle is : whenever it is possible, the existing building blocks should be considered as preferred choices.
- New project introduction
- New projects that are in "incubation" stage should not be a dependency for the rest of the mature projects. This will avoid introducing new complexities to the process of deploying and running ONAP.
- Only when a project graduates from their incubation stage should other projects create such dependencies.
• Use cases:
- Increase the scope of vCPE use case in deploying multiple virtual Gateways (Multi-Customer support)
• Open Source Values
- One critical aspect of Open Source is transparency (but opposition to Silos). We still see some corporation submitting huge pile of code days prior to major milestones. This approach is preventing the whole community to collaborate effectively.
- More ONS-NA feedback:
- internal company staff meetings/decisions != project team meetings/decisions
- Maturity of the community to understand the release is time-boxed and thus we have to adjust on scope and quality. Process adherence.
• Others
- Commit process: code review. Dialog necessary.
- 1) Concerns on big code merge. 2) That comes late. In case big code should come in, define an upper timeframe limit.
- Use Case: more details functional requirements. more focus on cross system integration (architecture). Lack of defined type, flow and content of messages (need additional details).
Process Improvement
Goals:
- Don’t merge code that has not been tested
- Speed up the verify and the merge
- More automated testing
High Level Categories of Process Improvements
- CI-CD: tools and processes
- Self-commit:
- automate Docker release version
- When time is critical (after RC0), allow PTLs to merge their change. This should allow be applicable to LF IT (CI-Management).
- Option: Follow OpenStack process loosely, I.E. no one merges code except Jenkins. When a Gerrit patch gets at least a +2 from a committer and a +1 from someone else we could have Jenkins merge the patch or do some variation of this to ensure faster patch merges but still ensure code review.
- Increase CI-Management committers list
- CD: use of OPNFV Clover. Helen/Integration
- Packaging: Improve Docker images build: incremental versus rebuild everything, optimize docker package (size, time to download) (LF+Gary-Michael). Fact: 13 docker templates in Beijing. Focus first on the most downloaded docker images (sdc, so aai,...)
- Standardize the build jobs (benefit: ease to add dependency): LF.
- Daily Build: get back to daily build. Build only image on code changes. (once a week one full build).
- Partner with OPNFV: Clover project.
- Self-commit:
- Testing
- Improve time to get the full ONAP installed and started (currently 12 hours to debug): 1 hour for HC + 1 another hour for instantiate => 3 hours. Every project team effort. Better knowledge (training) of OOM + K8S + Helm
- Look at how to architect for testability, and debugability : Brian
- the idea is how could we modify the helm chart setup so that we can enable a project to pull a new merge or verify docker without the risk that they will lead to a broken helm install from failed terminatinig pods, pv,pvc, port conflicts etc.
- helm upgrade --set enabled=true/false has led to broken installs and the need to purge
- Would be nice to be able to have projects in testing labs segmented so that project A's make project/make onap; docker stop/enabled=false/enabled=true doesnt cause other projects to be affected
- Right now the teams use HEAT base for preliminary testing because that isolation maintains a more stable debugging environment particulary since developers dont run into the read only file system issues for various configuration files
- While it is not the right solution - in some sense the Amsterdam per project namespace was actually more forgiving - although in fairness we didnt use the Amsterdam OOM as much as we did the Beijing OOM
- Love to plug a vFW-vDNS automated testing in the verify jobs: Not realistic for Casablanca. ( Gary to try)
- Manual way to build docker off on Verify job
- Release milestones
- Enforcement (respect and act upon automated results): (naming and shaming if criteria not met) on TLabs starting at M2 for Casablanca.(increase lab coverage over time, over release)
- HealthCheck: must not be broken for more than 48 hours
- Use case (VFW vDNS are automated): must not be broken for more than 48 hours
- CSIT: must not be broken for more than 48 hours
- Daily Build: must not be broken for more than 48 hours
- What for new projects?: HealthCheck after M4
- Perform testing Staging and Prod env: Gary.
- Enforcement (respect and act upon automated results): (naming and shaming if criteria not met) on TLabs starting at M2 for Casablanca.(increase lab coverage over time, over release)
- Labs and Infrastructure
- Resource (hardware): Get data from Stephen Gooch : Michael - OPENLABS-291Getting issue details... STATUS . WindRiver current config: 500 VMs, 2.8 TB RAM, 9.8TB Disk.
- Increase uplink speed on windriver network (slicing 10 GB):
- jenkins + nexus (currently 8GB is minimum): make it faster: LF. Jenkins fails cost minimum 1/2 days (for US). To bring this to the TSC→G Board.
- Packaging
- :
- Code Coverage and License scan
- Security
- SLA with LF: Idea Use JIRA to prioritize tasks.
LF to fix Staging Nexus.
H