-
Notifications
You must be signed in to change notification settings - Fork 169
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
kola/tests/ignition: add test for checking journald logs #1329
kola/tests/ignition: add test for checking journald logs #1329
Conversation
f8eac5c
to
303d493
Compare
Waiting for a new release of |
6302f96
to
946202b
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.
It's looking pretty good so far. A few notes added.
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.
Ran the test locally and ran without issue as the code looks good to me.
20ae9cd
to
15b6bd6
Compare
15b6bd6
to
9871951
Compare
|
||
func sendJournaldLog(c cluster.TestCluster) { | ||
m := c.Machines()[0] | ||
// See https://github.com/coreos/ignition/pull/958 |
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.
Perfect 👍
// for the MESSAGE_ID source. It will track the | ||
// journal messages related to an ignition config | ||
// provided by the user. | ||
out := c.MustSSH(m, fmt.Sprintf("journalctl -o json-pretty MESSAGE_ID=%s | jq -s '.[] | select(.IGNITION_CONFIG_TYPE == \"user\")' | wc -l", ignitionJournalMsgId)) |
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.
This is much clearer.
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashcrow, cgwalters, sohankunkerkar The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This test will validate the changes in coreos/ignition#958 to see if we're successfully sending the journald log entry for a
user-provided
config.