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.

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

Fix #1407, Add error report on EVS failure during log header write #2272

Conversation

thnkslprpt
Copy link
Contributor

@thnkslprpt thnkslprpt commented Mar 29, 2023

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully.

Local tests (seeded with bad value for BytesWritten) to check formatting etc.:

Screenshot 2023-03-30 08 48 25

Expected behavior changes
No change to behavior other than event report on error as per above.

System(s) tested on
Intel(R) Celeron(R) N4100 CPU @ 1.10GHz x86_64
Debian GNU/Linux 11 (bullseye)
Current main branch of cFS.

Contributor Info
Avi Weiss @thnkslprpt

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration coding-standard. As part of the setup process, we have scanned this repository and found 1905 existing alerts. Please check the repository Security tab to see all alerts.

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration security. As part of the setup process, we have scanned this repository and found 29 existing alerts. Please check the repository Security tab to see all alerts.

@thnkslprpt thnkslprpt force-pushed the fix-1407-add-event-for-evs-log-header-write-fail branch from cba84a2 to 4eeba08 Compare May 5, 2023 04:10
@dzbaker dzbaker added the CCB:Approved Indicates code review and approval by community CCB label Jun 8, 2023
dzbaker added a commit to nasa/cFS that referenced this pull request Jun 8, 2023
*Combines:*

cFE v7.0.0-rc4+dev342

**Includes:**

*cFE*
- nasa/cFE#2369
- nasa/cFE#2272
- nasa/cFE#2276

Co-authored by: Daniel Knutsen <dmknutsen@users.noreply.github.com>
Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
@dzbaker dzbaker mentioned this pull request Jun 8, 2023
2 tasks
@dzbaker dzbaker merged commit e9d21f2 into nasa:main Jun 8, 2023
dzbaker added a commit to nasa/cFS that referenced this pull request Jun 8, 2023
*Combines:*

cFE v7.0.0-rc4+dev342

**Includes:**

*cFE*
- nasa/cFE#2369
- nasa/cFE#2272
- nasa/cFE#2276

Co-authored by: Daniel Knutsen <dmknutsen@users.noreply.github.com>
Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
@thnkslprpt thnkslprpt deleted the fix-1407-add-event-for-evs-log-header-write-fail branch June 8, 2023 20:16
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.

Report error when writing event log header
2 participants