-
Notifications
You must be signed in to change notification settings - Fork 871
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
test: add get_syslog_or_console for minimal images without syslog #5793
Conversation
a9276c0
to
57efc02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One inline comment, but otherwise LGTM
f"Image {CURRENT_RELEASE.os} doesn't contain 'file' util" | ||
) | ||
|
||
class_client.execute("apt-get update") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it worth updating this test to use something other than file
? That particular command doesn't seem to be integral to the test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, we don't need to use file
for this verification/validation. Simplifying this to just check md5sums of the output file.
Minimal images do not have rsyslog installed. As a result, no /var/log/syslog exists. Add helper function get_syslog_or_console to allow minimal images to use pycloudlib.Instance.get_console instead of reading /var/log/syslog.
57efc02
to
6ca01d3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Proposed Commit Message
Additional Context
Sample jenkins test failures for minimal.
https://jenkins.canonical.com/server-team/job/zzz-chad-jammy-lxd-minimal/lastSuccessfulBuild
Test Steps
Merge type