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.

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

Ntp check v2 #96

Merged
merged 7 commits into from
Jan 10, 2025
Merged

Ntp check v2 #96

merged 7 commits into from
Jan 10, 2025

Conversation

knvk
Copy link
Contributor

@knvk knvk commented Jan 10, 2025

Hello once again,
I've fixed all the problems in #92
Also, I've refactored check function a bit, to perform mock tests.

knvk added 3 commits January 10, 2025 11:47
Signed-off-by: Konstantin Konov <80425051+knvk@users.noreply.github.com>
Signed-off-by: Konstantin Konov <80425051+knvk@users.noreply.github.com>
@knvk
Copy link
Contributor Author

knvk commented Jan 10, 2025

fixed spec unit test and readme

@teran
Copy link
Contributor

teran commented Jan 10, 2025

@knvk please note, one of the failures you have is about expired ca-certificates package in alpine (where certificates are fetched for container image), I just updated the package so this issue should fixed with just a rebase

knvk added 4 commits January 10, 2025 14:26
Signed-off-by: Konstantin Konov <80425051+knvk@users.noreply.github.com>
Signed-off-by: Konstantin Konov <80425051+knvk@users.noreply.github.com>
Signed-off-by: Konstantin Konov <80425051+knvk@users.noreply.github.com>
@teran teran merged commit bea061b into runityru:master Jan 10, 2025
5 checks passed
@teran
Copy link
Contributor

teran commented Jan 10, 2025

@knvk Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants