Core Commiter Weekly Interlock - March 20th,2017
to Tom Cap last week and he felt there was still I need. I will let him answer.Former user (Deleted)
1 Merge training - need to set up a confluence page that summarize role of a CC and what it takes to be a CC. Have a meeting with each Dev team and ??? to walk through what it takes to be a CC. Idea being that we can train more dev engineers so they can have the power to commit after a PR passes. Avoids having all PR merging go through a small group of CC.
AI Tom Sullivan - Rod to follow up
2 Renaming of tags in Docker - everyone agree with the following
- latest = latest stable (I.e. release) build
- devel = latest DockerHub (I.e. commit triggered) build
- nightly = latest nightly build
Approval passed to follow the above mentioned tags via CC approval.
Leo Zhang: Create new repo "image-service" RI-61 Documentation needed on how to use the different styles of OS installation. AI Leo Zhang to provide document. Need to add this style to FIT tests. AI norton luo to add tests to FIT (Regression and smoke)
Felix: I don't have permission to create repo in RackHD organization. Tim Larson, I see you are the OrgMaintainers, could you please do me a favor?
4. Former user (Deleted): Minimize Environment for CI Test: Minimize Environment For CI Test.pptx
AI Thomas Sullivan Tom Capirchio do we post an OVA anywhere for outside of Dell/EMC? Do we want them to?
Approved to move forward with this plan.
5. Former user (Deleted): Whether enable Code Climate for PR gate check? https://github.com/RackHD/on-core/pull/264
- Example PR: https://github.com/yyscamper/on-core/pull/7
- Example PR Analysis report: https://codeclimate.com/github/yyscamper/on-core/pull/7
- RackHD master report (on-core): https://codeclimate.com/github/RackHD/on-core
- Browser extension: https://codeclimate.com/browser-extension
- Available Analysis Engines: https://docs.codeclimate.com/docs/list-of-engines
- Replace JSHint with ESLint (http://eslint.org/)?
- We need to cleanup all existing static analysis issues before using code climate and ESLint
AI Former user (Deleted) to look at configuration options for code climate to make sure that it can give us some useful information. Are there other options besides hound and code climate?
Felix: Former user (Deleted), From my example code climate configuration (https://github.com/yyscamper/on-core/blob/d647ea9a5cb4df4e384f8f83d1bf8e9e9025b7f7/.codeclimate.yml), the duplicated code check is disabled.
AI Former user (Deleted) to look into if it supports Java and see if it can be used against the mustang code base.
Felix: Former user (Deleted) I checked the code climate doc, it supports Java using PMD engine (see https://docs.codeclimate.com/docs/pmd), but it is still in Beta version.
6. Naming conventions for daily releases - contain the previous release candidate plus date 1.3.0-20170208 How do we handle multiple delivery in a day?
AI Former user (Deleted) and Former user (Deleted) to send out there thoughts on this. To discuss at the next CC meeting
Selenium was verified against the on-webui can this also be used against swagger (AI Former user (Deleted))
(Former user (Deleted): Exchanged information with Muqeeth Kowkab who confirmed that Selenium can also be used with swagger-ui. Was confirmed with Firefox addon to Selenium IDE. Should also work with Chrome WebDriver).