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

Move journalctl after ntp_client #16209

Merged
merged 1 commit into from Jan 11, 2023
Merged

Move journalctl after ntp_client #16209

merged 1 commit into from Jan 11, 2023

Conversation

grisu48
Copy link
Contributor

@grisu48 grisu48 commented Jan 10, 2023

Move the journalctl tests after ntp_client test to ensure that the system time is synchronized.

@grisu48
Move journalctl after ntp_client
ceec0f2 
Move the journalctl tests after ntp_client test to ensure that the
system time is synchronized.
mloviska
mloviska approved these changes Jan 10, 2023
View reviewed changes
Copy link
Contributor

@mloviska mloviska left a comment

Choose a reason for hiding this comment

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

LGTM

@mloviska mloviska mentioned this pull request Jan 10, 2023
Merged
rfan1
rfan1 approved these changes Jan 11, 2023
View reviewed changes
Copy link
Contributor

@rfan1 rfan1 left a comment

Choose a reason for hiding this comment

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

LGTM

@grisu48 grisu48 merged commit 629eeb1 into os-autoinst:master Jan 11, 2023
@grisu48 grisu48 deleted the ntp_client branch January 11, 2023 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mloviska mloviska mloviska approved these changes

@rfan1 rfan1 rfan1 approved these changes

@foursixnine foursixnine Awaiting requested review from foursixnine

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@grisu48 @mloviska @rfan1