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

Fix Coverity issues for acl_platform_test.cpp #262

Merged
merged 1 commit into from
Jan 31, 2023
Merged

Fix Coverity issues for acl_platform_test.cpp #262

merged 1 commit into from
Jan 31, 2023

Conversation

IlanTruanovsky
Copy link
Contributor

Fixes a bunch of PRINTF_ARGS Coverity issues.

A macro had a Coverity issue, causing it to be reported multiple times.

Because of this, there were too many issues so I didn't include them in the commit message to not clutter it.

@IlanTruanovsky @pcolberg
Fix Coverity issues for acl_platform_test.cpp
73c843b 
A macro had a PRINTF_ARGS Coverity issue, and so whenever the macro was used, Coverity detected an error.
This meant that many errors were actually caused by the same problem.
@pcolberg
Copy link
Contributor

Looks like GitHub is having an issue with their container infrastructure:

2023-01-30T21:54:51.4521017Z ##[group]Starting job container
2023-01-30T21:54:51.4545846Z ##[command]/usr/bin/docker --config /home/runner/work/_temp/.docker_d3833483-31ed-43f8-8e4f-f81bce1ccc5b login ghcr.io -u IlanTruanovsky --password-stdin
2023-01-30T21:54:51.7918284Z ##[error]Docker login for 'ghcr.io' failed with exit code 1
2023-01-30T21:54:51.8248612Z ##[error]Value cannot be null. (Parameter 'ContainerId')

@pcolberg pcolberg added the bug Something isn't working label Jan 30, 2023
@pcolberg pcolberg added this to the 2023.2 milestone Jan 30, 2023
@pcolberg pcolberg merged commit 2256e22 into intel:main Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcolberg pcolberg pcolberg approved these changes

Assignees

@IlanTruanovsky IlanTruanovsky

Labels
bug Something isn't working
Projects
None yet
Milestone
2023.2
Development

Successfully merging this pull request may close these issues.
2 participants
@IlanTruanovsky @pcolberg