-
Notifications
You must be signed in to change notification settings - Fork 93
Fix: bootstrap: Detect cluster service on init node before saving the canonical hostname (bsc#1222714) #1386
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
Fix: bootstrap: Detect cluster service on init node before saving the canonical hostname (bsc#1222714) #1386
Conversation
liangxin1300
commented
Apr 12, 2024
•
edited
Loading
edited
@@ -1625,8 +1625,6 @@ def join_ssh_impl(local_user, seed_host, seed_user, ssh_public_keys: typing.List | |||
user_by_host.add(local_user, utils.this_node()) | |||
user_by_host.set_no_generating_ssh_key(bool(ssh_public_keys)) | |||
user_by_host.save_local() | |||
user_by_host.add(seed_user, get_node_canonical_hostname(seed_host)) |
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 required by line 1633 when a hostname alias is used.
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.
Then we can try to check init node's cluster service status just after the line 1620
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.
Then the process/order for this part will be:
- call ping_node to check if the init node is up
- call join_ssh to setup passwordless with init node
- inside join_ssh, after swap the key for the first user('hacluster' is the second), and before using utils.HostUserConfig to save the info of user, check the status of cluster service on init node
- then get the canonical hostname from init node
d68e99f
to
4b0dd66
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1386 +/- ##
==========================================
- Coverage 53.32% 53.27% -0.05%
==========================================
Files 80 80
Lines 23941 23942 +1
==========================================
- Hits 12766 12755 -11
- Misses 11175 11187 +12 ☔ View full report in Codecov by Sentry. |
… canonical hostname (bsc#1222714)
c3800b2
to
5036bc6
Compare