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

Fix NTP issue that was causing SES5 deployment to fail #108

Merged
merged 4 commits into from
Feb 17, 2020
Merged

Fix NTP issue that was causing SES5 deployment to fail #108

merged 4 commits into from
Feb 17, 2020

Conversation

smithfarm
Copy link
Collaborator

@smithfarm smithfarm commented Feb 17, 2020 •
edited
Loading

With these patches, a functioning NTP server setup is deployed on the admin node before DeepSea starts, and DeepSea is instructed -- via time_init: ntp -- to configure NTPD on all the cluster nodes, pointing at the admin node.

Unfortunately, DeepSea has to be patched for this to work...

Fixes: #107

@smithfarm
ses5_pre_stage_0: force ntpd to sync before starting
5a3f236 
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
ses5_pre_stage_0: do not fail easily on test.ping
0b88af7 
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
ses5_pre_stage_0: remove ntp package on all nodes
73fca71 
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
ses5_pre_stage_0: patch ceph.time
f1fb650 
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm smithfarm added the bug label Feb 17, 2020
@smithfarm smithfarm changed the title Fix SES5 deployment Fix NTP issue that was causing SES5 deployment to fail Feb 17, 2020
@smithfarm smithfarm mentioned this pull request Feb 17, 2020
Merged
4 tasks
ImTheKai
ImTheKai approved these changes Feb 17, 2020
View reviewed changes
Copy link
Contributor

@ImTheKai ImTheKai left a comment

Choose a reason for hiding this comment

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

lgtm

ricardoasmarques
ricardoasmarques approved these changes Feb 17, 2020
View reviewed changes
Copy link
Contributor

@ricardoasmarques ricardoasmarques left a comment

Choose a reason for hiding this comment

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

Tested locally and it fixes the issue, lgtm.

@smithfarm smithfarm merged commit 9d8139a into master Feb 17, 2020
@smithfarm smithfarm deleted the wip-ses5-fixes branch February 17, 2020 16:39
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
9190738 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
af03835 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
a7cd33c 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
a3a926a 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
1ba8c1a 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
822146e 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
f7872f5 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
eeb8ee2 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
47fab99 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
d4a8088 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
ses5: refactor time sync
22c3a14 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
e41f745 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
c66e506 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
0c97016 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
1df907c 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
9616315 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
28561bc 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
547079e 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
6ec21d0 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
37e7424 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/sesdev that referenced this pull request Apr 10, 2020
@smithfarm
New time sync implementation for ses5, ses6/nautilus
aaf5a77 
Even after SUSE#108 we were seeing
time synchronization issues in the ses5 deployment.

nautilus/ses6 deployments were also suffering from clock skews.

Signed-off-by: Nathan Cutler <ncutler@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ImTheKai ImTheKai ImTheKai approved these changes

@ricardoasmarques ricardoasmarques ricardoasmarques approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ses5 deployment fails due to NTP issue
3 participants
@smithfarm @ricardoasmarques @ImTheKai