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

Why disable defaults of ntpsec configuration when migration file not exists? #20

Open
zLie-a opened this issue Mar 1, 2024 · 1 comment
Assignees

Comments

@zLie-a
Copy link

zLie-a commented Mar 1, 2024

Issue-template

Summary

When integrating this dm-ntp into our system, we found that the ntp-server in our system is in abnormal state after onboared to Industrail Edge Management system.

Version of the used components

Which versions of the Industrial Edge components did you use?
e.g.

  • dm-ntp: 1.17.1-1
  • ...

Steps to reproduce

NA

What is the current bug behavior?

No default workable ntp-server configured.

What is the expected correct behavior?

Default configuration should take effect rather than commented

Relevant logs and/or screenshots

image
image
image

Line of code

https://github.com/industrial-edge/device-kit-ntp-service/blob/main/migration/ntpclassic_to_ntpsec.go#https://github.com/industrial-edge/device-kit-ntp-service/blob/main/migration/ntpclassic_to_ntpsec.go#L64-L73

@saffetturkoglu saffetturkoglu self-assigned this Mar 1, 2024
@saffetturkoglu
Copy link
Collaborator

Hi @zLie-a ,

Please fill out "Steps to reproduce".

Thanks

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

No branches or pull requests

2 participants