Skip to content

Merge feature/configure-ssh to master #6361

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

Merged

Conversation

BengangY
Copy link
Contributor

@BengangY BengangY commented Mar 17, 2025

  1. Merge feature/configure-ssh to master.
  2. Update datamodel_lifecycle.ml to 25.12.0-next.

BengangY and others added 11 commits January 10, 2025 06:06
Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
)

This PR is to add API and CLI which is used to enable and disable SSH on
all hosts in a pool or on specified hosts within a pool.
Resolved onflicts:
	ocaml/idl/datamodel_errors.ml
	ocaml/xapi-consts/api_errors.ml

Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
Resolved onflicts:
	ocaml/idl/datamodel_errors.ml
	ocaml/xapi-consts/api_errors.ml
Change call_slave_... functions for new added ssh feature code to resolve build
failure.

Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
1. Sync `master` to `feature/configure-ssh`
2. Resolve build failure in `message_forwarding.ml` in the commit
`0b79d88`.
Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>
@BengangY BengangY marked this pull request as ready for review March 17, 2025 02:54
Copy link
Contributor

@gangj gangj left a comment

Choose a reason for hiding this comment

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

$ git show 18ba7e724fbd0d9454a2c62cf9684b7e10af5448
commit 18ba7e724fbd0d9454a2c62cf9684b7e10af5448 (bengang_github/private/bengangy/merge-ssh-to-master, private/bengangy/merge-ssh-to-master)
Author: Bengang Yuan <bengang.yuan@cloud.com>
Date:   Mon Mar 17 01:25:33 2025 +0000

    Update datamodel_lifecycle.ml
    
    Signed-off-by: Bengang Yuan <bengang.yuan@cloud.com>

diff --git a/ocaml/idl/datamodel_lifecycle.ml b/ocaml/idl/datamodel_lifecycle.ml
index 38ed231cd..d21cad211 100644
--- a/ocaml/idl/datamodel_lifecycle.ml
+++ b/ocaml/idl/datamodel_lifecycle.ml
@@ -205,6 +205,10 @@ let prototyped_of_message = function
       Some "22.26.0"
   | "VTPM", "create" ->
       Some "22.26.0"
+  | "host", "disable_ssh" ->
+      Some "25.12.0-next"
+  | "host", "enable_ssh" ->
+      Some "25.12.0-next"
   | "host", "emergency_clear_mandatory_guidance" ->
       Some "24.10.0"
   | "host", "apply_recommended_guidances" ->
@@ -223,6 +227,10 @@ let prototyped_of_message = function
       Some "23.30.0"
   | "VM", "set_groups" ->
       Some "24.19.1"
+  | "pool", "disable_ssh" ->
+      Some "25.12.0-next"
+  | "pool", "enable_ssh" ->
+      Some "25.12.0-next"
   | "pool", "get_guest_secureboot_readiness" ->
       Some "24.17.0"
   | "pool", "set_ext_auth_cache_expiry" ->

@gangj
Copy link
Contributor

gangj commented Mar 17, 2025

I think the commit message of 20d6e34 should be something like: "Merge feature/configure-ssh to master"?

@BengangY BengangY changed the title Merge ssh to master Merge feature/configure-ssh to master Mar 17, 2025
@BengangY BengangY added this pull request to the merge queue Mar 17, 2025
Merged via the queue into xapi-project:master with commit 18e93ff Mar 17, 2025
17 checks passed
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.

3 participants