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

conf_file/qemu_conf/seccomp: add checkpoint for invalid value #5008

Merged
merged 1 commit into from
Jul 4, 2023
Merged

conf_file/qemu_conf/seccomp: add checkpoint for invalid value #5008

merged 1 commit into from
Jul 4, 2023

Conversation

smitterl
Copy link
Contributor

@smitterl smitterl commented Jul 4, 2023

Check that a syntax error is logged if an invalid value is set.

@smitterl
conf_file/qemu_conf/seccomp: add checkpoint for invalid value
d837b39 
Check that a syntax error is logged if an invalid value is set.

Signed-off-by: Sebastian Mitterle <[email protected]>
@smitterl
Copy link
Contributor Author

smitterl commented Jul 4, 2023 

JOB ID     : 5795ae664b025d17d037243b6703342ff2f81cbc
JOB LOG    : /var/lib/avocado/job-results/job-2023-07-04T03.35-5795ae6/job.log
 (1/1) type_specific.io-github-autotest-libvirt.conf_file.qemu_conf.seccomp_sandbox.negative_test.set_sandbox_invalid: STARTED
 (1/1) type_specific.io-github-autotest-libvirt.conf_file.qemu_conf.seccomp_sandbox.negative_test.set_sandbox_invalid: PASS (73.91 s)
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 0
JOB HTML   : /var/lib/avocado/job-results/job-2023-07-04T03.35-5795ae6/results.html
JOB TIME   : 75.31 s

chunfuwen
chunfuwen approved these changes Jul 4, 2023
View reviewed changes
Copy link
Contributor

@chunfuwen chunfuwen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

dzhengfy
dzhengfy approved these changes Jul 4, 2023
View reviewed changes
Copy link
Contributor

@dzhengfy dzhengfy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dzhengfy dzhengfy merged commit 1528df2 into autotest:master Jul 4, 2023
@smitterl smitterl deleted the invalid_seccomp branch November 20, 2023 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chunfuwen chunfuwen chunfuwen approved these changes

@dzhengfy dzhengfy dzhengfy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@smitterl @dzhengfy @chunfuwen