Skip to content

Fix #410, separate SYSLOG configurable defaults for power on and proc…#625

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
dmknutsen:issue_410
Apr 28, 2020
Merged

Fix #410, separate SYSLOG configurable defaults for power on and proc…#625
astrogeco merged 1 commit intonasa:integration-candidatefrom
dmknutsen:issue_410

Conversation

@dmknutsen
Copy link
Contributor

Describe the contribution
Fix #410 .
Separates configurable defaults for power on and processor reset. Default power on set to overwrite and default processor reset to discard.
Added an additional unit test to cover the path added to CFE_ES_TaskInit.

Testing performed:
Ran unit tests

Expected behavior changes
Upon power on reset, default system log mode set to overwrite.
Upon processor reset, default system log mode set to discard.

System(s) tested on
Oracle VM VirtualBox
OS: ubuntu-19.10
Versions: 6.7.12.0, OSAL 5.0.11.0, PSP 1.4.7.0

Contributor Info
Dan Knutsen
NASA/Goddard

@skliper skliper added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Apr 21, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate April 21, 2020 22:10
@astrogeco astrogeco changed the base branch from integration-candidate to master April 21, 2020 22:11
@astrogeco
Copy link
Contributor

CCB 20200422 - APPROVED

@astrogeco astrogeco added CCB - 20200422 CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Apr 23, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate April 27, 2020 23:01
@astrogeco astrogeco merged commit 2890418 into nasa:integration-candidate Apr 28, 2020
@skliper skliper added this to the 6.8.0 milestone Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code review and approval by community CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Initialize system log mode implementation doesn't match requirement cES1522

3 participants