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

Add syslog for CFE_ES_DeleteApp and CFE_ES_ReloadApp ID failures #1775

Closed
skliper opened this issue Aug 9, 2021 · 0 comments · Fixed by #1779 or #1808
Closed

Add syslog for CFE_ES_DeleteApp and CFE_ES_ReloadApp ID failures #1775

skliper opened this issue Aug 9, 2021 · 0 comments · Fixed by #1779 or #1808
Assignees
@skliper
Labels
enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Aug 9, 2021

Is your feature request related to a problem? Please describe.
CFE_ES_DeleteApp is required to log a bad id (ES: Delete Application - Reject Undefined,cES1309.1), and CFE_ES_RestartApp has a requirement and does syslog. CFE_ES_ReloadApp doesn't have an associated requirement, but should be consistent.

Describe the solution you'd like
Add syslog for CFE_ES_DeleteApp and CFE_ES_ReloadApp

Describe alternatives you've considered
Remove syslog write from the requirements and remove from CFE_ES_RestartApp...

Additional context
#1773

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone Aug 9, 2021
@skliper skliper mentioned this issue Aug 9, 2021
Closed
@skliper skliper self-assigned this Aug 9, 2021
skliper added a commit to skliper/cFE that referenced this issue Aug 9, 2021
@skliper
Fix nasa#1775, Add invalid ID Syslog entry for CFE_ES_DeleteApp and C…
10d3bde 
…FE_ES_ReloadApp
@skliper skliper mentioned this issue Aug 9, 2021
Merged
astrogeco added a commit that referenced this issue Aug 13, 2021
@astrogeco
Merge pull request #1779 from skliper/fix1775-app_syslog_entries
471b94d 
Fix #1775, #1773 -  Adds invalid id syslog to for CFE_ES_DeleteApp and CFE_ES_ReloadApp and verifies required reporting
@astrogeco astrogeco mentioned this issue Aug 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
enhancement
Projects
None yet
Milestone
Caelum
1 participant
@skliper