Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove system log elements from cFE1521, ES: Processor Reset Preservation List requirement #1029

Closed
olamoo13 opened this issue Nov 23, 2020 · 1 comment · Fixed by #1609
Closed

Remove system log elements from cFE1521, ES: Processor Reset Preservation List requirement #1029

olamoo13 opened this issue Nov 23, 2020 · 1 comment · Fixed by #1609
Assignees
@skliper
Labels
docs This change only affects documentation. requirements
Milestone
Caelum

Comments

@olamoo13
Copy link

olamoo13 commented Nov 23, 2020 •
edited by skliper
Loading

Description
Remove System log elements from cES1521 (CFE-183) requirement. Over-specified, out-of-family, and somewhat redundant with cES1511 requirement to preserve the system log.

cES1521:
Upon a Processor Reset, the cFE shall preserve the following:

  • Boot Source
  • Reset Type
  • Reset Subtype
  • Reset Reason
  • Number of Processor Resets
  • Maximum Processor Resets
  • Number of entries in System Log
  • Size of System Log
  • Number of bytes used in the System Log
@skliper skliper added this to the 7.0.0 milestone Nov 30, 2020
@skliper skliper added the docs This change only affects documentation. label Nov 30, 2020
@skliper
Copy link
Contributor

skliper commented Jan 22, 2021

If I recall correctly, this was due to an overlap with cES1511: "Upon a Processor Reset, the cFE shall preserve the Executive Services System Log." and the language of "preserve the following" in relation to these items is confusing since it's really preserving but then appending, overwriting, or discarding all the new startup messages. It really seems like overkill in this requirement, and over-specifying w/ respect to design/implementation and out of family with the rest of the items in the list.

@skliper skliper changed the title Remove - Number of entries in System Log - Size of System Log - Number of bytes used in the System from CFE-183 ES: Processor Reset Preservation List Remove system log elements from cFE1521, ES: Processor Reset Preservation List requirement Jan 22, 2021
@skliper skliper self-assigned this Feb 1, 2021
skliper added a commit to skliper/cFE that referenced this issue Jun 7, 2021
@skliper
Fix nasa#1205 nasa#1131 nasa#1127 nasa#1029 nasa#942, Requirements Up…
9b08eec 
…dates

 nasa#1205: Added task and CDS file write default filenames
 nasa#1131: Removed optional from EVS log
 nasa#1127: Non-parameter reload/restart doesn't increment cmd error
 nasa#1029: Removed system log elements from reset preservation list
 nasa#942: Simplifed table partial load file requirement
skliper added a commit to skliper/cFE that referenced this issue Jun 7, 2021
@skliper
Fix nasa#1205 nasa#1131 nasa#1127 nasa#1029 nasa#942, Requirements Up…
f9e4117 
…dates

 nasa#1205: Added task and CDS file write default filenames
 nasa#1131: Removed optional from EVS log
 nasa#1127: Non-parameter reload/restart doesn't increment cmd error
 nasa#1029: Removed system log elements from reset preservation list
 nasa#942: Simplifed table partial load file requirement
@skliper skliper mentioned this issue Jun 7, 2021
Merged
astrogeco added a commit that referenced this issue Jun 15, 2021
@astrogeco
Merge pull request #1609 from skliper/fix1205-1131-1127-1029-942-requ…
7522ce6 
…irements

Fix #1205 #1131 #1127 #1029 #942 - Requirements updates
@astrogeco astrogeco mentioned this issue Jun 15, 2021
Merged
@skliper skliper closed this as completed Jun 29, 2021
@skliper skliper mentioned this issue Jan 20, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
docs This change only affects documentation. requirements
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #1205 #1131 #1127 #1029 #942 - Requirements updates skliper/cFE
2 participants
@skliper @olamoo13