Skip to content

CP-54384 Update Auto-Mode Configuration for Pool Operations and Default Values #6484

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

Open
wants to merge 2 commits into
base: feature/configure-ssh-phase3
Choose a base branch
from

Conversation

LunfanZhang
Copy link
Contributor

This PR aim to add following change for auto-mode:
Copy the auto-mode setting from the pool coordinator during a pool join. and restore the auto-mode setting to default value when a pool eject occurs.

  • In XS8, the auto-mode is set to false by loading the config file /etc/xapi.conf.d/ssh-auto-mode.conf.
  • In XS9, default value is set to true.

- For XS8, a configuration file (/etc/xapi.conf.d/ssh-auto-mode.conf) will be loaded, and auto_mode will be set to false.
- For XS9, no configuration file exists, so the default value will be set to true.

Signed-off-by: Lunfan Zhang[Lunfan.Zhang] <[email protected]>
- Copy auto mode setting from pool coordinator in pool join
- Restore auto mode setting to default when pool eject

Signed-off-by: Lunfan Zhang[Lunfan.Zhang] <[email protected]>
@LunfanZhang LunfanZhang force-pushed the private/luzhan/CP-54382 branch from 695701b to e114448 Compare May 23, 2025 06:48
let ssh_monitor_enabled =
Fe_systemctl.is_active ~service:!Xapi_globs.ssh_monitor_service
in
(* For xs9 when fresh install, the ssh_monitor service is not enabled by default.
Copy link
Contributor

@BengangY BengangY May 29, 2025

Choose a reason for hiding this comment

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

Why ssh_monitor is not enabled by default on xs9? Do you mean "xs8"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The ssh_monitor service should be enabled by default on xs9, so after we set auto_mode to true in db, e still need a mechanism to actually enable the service.
There is some tiny difference between ssh-auto is not enabled by default and ssh_monitor service is not enabled by default.

@LunfanZhang LunfanZhang requested a review from BengangY May 30, 2025 01:42
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

Successfully merging this pull request may close these issues.

2 participants