Community CCB Agenda: April 22 2020

Minutes/decisions from CCB are noted with in-line comments.

New topics:

• Use draft pull request feature as default

Agreed

• Use code owners feature: https://github.blog/2017-07-06-introducing-code-owners/

Please check out this feature for our potential future use.

• Versioning Update:
  • Option 1: tag current version of all repos and move forward with MAJOR.MINOR.FIX-beta-NUMBER_FROM_GIT_DESCRIBE

Jake raised the issue that this only works with git. Discussion. Tabled this for further discussion.

• Option 2: Apply option 1 retroactively based on last tag
• New Labels:
  • deprecate: To identify issues that deprecate features
  • TBL: Any other "services"?
• Notes: Inline vs separate document?
• Meta Agenda: Numbering to count number of items as opposed to reset for each repo.
• Special CCB Extension from 1-2pm EST today for OSAL ticket

Relevant Links

• How are you using cFS?: https://form.jotform.com/200561554287052
• GitHub project board: https://github.com/orgs/nasa/projects/4 (only accessible to cFS team members)

Closed Integration Candidate: 2020-04-08

• cFS Bundle PR #66
• cFE PR #615
• OSAL PR #417
• PSP PR #153

In-process Integration Candidate: 2020-04-15

For the conflicts noted below, Gerardo has tagged the folks who are going to help out with those this week.

• cFS Bundle PR #70

• cFE PR #635

  • Conflicts: PR #578, PR #602

• OSAL PR #433

  • Conflicts: PR #415

• PSP PR #155

• cFS-GroundSystem PR #74

  • Conflicts: PR #73

Review PRs:

cFE

1. PR #606, Fix #503 - better events for CFE_TBL_Load()

Jake: My observation is that this is an improvement, but the complexity has increased. Proposed a new ticket to address the complexity in the load table. Chris agreed, and noted there is an existing ticket - he will link that one to this one as well. CCB decided to go forward with this one.

2. PR #622, Fix #121, Add check for UsedFlag and sets APID to CFE_TBL_NOT_OWNED

Approved

3. PR #492, Unit test macros and example use with SB

Recommendation: House them in ut_support so they can be used in all subsystems. It was also decided that this would benefit from a separate review, targeting next week (so it can support the unit testing for the Gateway certification). Tam asked for some summary slides ahead of the separate review. Gerardo/Chris will schedule.

4. PR #619, Fix #591, Avoid deadlock in CFE_ES_CreatObjects

Approved (also, open an issue for an OSAL debug feature)

5. PR #624, Fix #623, Replace OS_FS_* return codes with common definitions

Approved

6. PR #625, Fix #410, separate SYSLOG configurable defaults for power on

Approved

ADDED JUST BEFORE MEETING: Fix #632 (https://github.com/nasa/cFE/pull/634), Infer OSAL_SYSTEM_BSPTYPE from CFE_SYSTEM_PSP_NAME #634 Merge together with OSAL side (this one, which was closed in the meeting: Fix #629 (https://github.com/nasa/cFE/pull/630), Define BSP type in sample config #630). Both will be addressed in this ticket. Approved.

OSAL

7. PR #403, Fix #346, Adds library build, functional, and coverage test to CI

Approved.

8. PR #428, Fix #425, Correct Test_OS_ConvertToArrayIndex assertion typo

Approved

9. PR #418 Fix #354, Shell related API separated for optional implementation

Approved

10. PR #426, Fix #424, Add looping wait for self exiting task #426

Approved

11. PR #423, Fix #344, Consistent directory entry size limit #423

Approved

12. PR #421 Fix #262, Removes OS_FS_* defines that aren't unique to FS #421

Approved

13. PR #431, unit-tests/osloader-test: make modules always be built for the test Need to format title

Approved conditionally pending test

14. PR #634, Fix #632, Infer OSAL_SYSTEM_BSPTYPE from CFE_SYSTEM_PSP_NAME

Approved

Special Topic

15. PR #427 Fix #285, Refactor OSAL code selection #427

Concepts reviewed and approved as special topic, line-by-line review and official approval off-line

Fast Tracked

16. PR #422 Fix #413, Add reference to osal user's guide from ReadMe

Approved

PSP

No new PRs

ci_lab

No new PRs

elf2cfetbl

17. PR #36, Fix #33, Resolve print format mismatches on 32-bit host

Approved

sample_app

18. PR #57, Fix #56, Update coverage compile/link flag options

Approved

Fast Tracked

19. PR #59, Fix #50, Reference skeleton_app in ReadMe

Approved

sample_lib

20. PR #20, Fix #19, Update coverage compile/link flag options

Approved

Fast Tracked

21. PR #18, Fix #17, Fix Doxygen Warnings

Approved

sch_lab

No new PRs

to_lab

No new PRs

cFS-GroundSystem

No new PRs

cFS Bundle

No new PRs

Issues

cFS Bundle

22. #67, Document deprecation process

Deferred to next week

Invite list

• GSFC

  • Jacob Hageman (cFS Framework Lead)
  • Jonathan Wilmot (cFS Program Manager)
  • Alan Cudmore (cFS Architect)
  • Gerardo Cruz-Ortiz (cFS Management)
  • Dan Knutsen (cFS Developer)
  • Jay Czarnecki (Software Process Imp)
  • Beth Timmons (GSFC Apps)
  • Dwaine Molock (Project Developer)
  • Anhelina Yurkova (cFS Test Automation)
  • Leor Bleier (cFS Developer)

• JSC

  • Steve Duran (PSP, OSAL Developer)
  • Ron Maglothin (Lunar CATALYST Developer)
  • Allen Brown (AES Developer)
  • Thomas Brain (AES Integration)
  • Tam Ngo AES Lead
  • Carrejo, Daniel Systems
  • Schlesinger, Adam Avionics and Software
  • Thad Flemming

• ARC

  • Chris Knight (SBN Developer & Research)
  • Joseph (Pat) Castle
  • Michael Scott
  • Craig Pires

• LaRC

  • Chris Thames (Lead/FSW Management)
  • Noah Ryan (Software Developer)

• MSFC

  • Deanna Whitehead
  • Stefanie Justice
  • Lisa Vanderaar (Software lead)

• GRC

  • Joe Hickey (cFS Developer)

• IV&V

  • Matt Grubb (SmallSat baseline, build testing)

• JHU/APL

  • Chris Monaco

• Guest

  • Dave McComas (GSFC)