Merge pull request #11286 from jwhonce/issues/11227

Update /version endpoint to add components

1 files changed, 31 insertions, 12 deletions

diff --git a/pkg/api/handlers/compat/version.go b/pkg/api/handlers/compat/version.go
index f1cd77a9a..a115cc885 100644
--- a/pkg/api/handlers/compat/version.go
+++ b/pkg/api/handlers/compat/version.go
@@ -13,20 +13,19 @@ import (
 	"github.com/containers/podman/v3/pkg/domain/entities/types"
 	"github.com/containers/podman/v3/version"
 	"github.com/pkg/errors"
+	"github.com/sirupsen/logrus"
 )
 
 func VersionHandler(w http.ResponseWriter, r *http.Request) {
-	// 200 ok
-	// 500 internal
 	runtime := r.Context().Value("runtime").(*libpod.Runtime)
 
-	versionInfo, err := define.GetVersion()
+	running, err := define.GetVersion()
 	if err != nil {
 		utils.Error(w, "Something went wrong.", http.StatusInternalServerError, err)
 		return
 	}
 
-	infoData, err := runtime.Info()
+	info, err := runtime.Info()
 	if err != nil {
 		utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrapf(err, "failed to obtain system memory info"))
 		return
@@ -34,20 +33,40 @@ func VersionHandler(w http.ResponseWriter, r *http.Request) {
 
 	components := []types.ComponentVersion{{
 		Name:    "Podman Engine",
-		Version: versionInfo.Version,
+		Version: running.Version,
 		Details: map[string]string{
 			"APIVersion":    version.APIVersion[version.Libpod][version.CurrentAPI].String(),
 			"Arch":          goRuntime.GOARCH,
-			"BuildTime":     time.Unix(versionInfo.Built, 0).Format(time.RFC3339),
-			"Experimental":  "true",
-			"GitCommit":     versionInfo.GitCommit,
-			"GoVersion":     versionInfo.GoVersion,
-			"KernelVersion": infoData.Host.Kernel,
+			"BuildTime":     time.Unix(running.Built, 0).Format(time.RFC3339),
+			"Experimental":  "false",
+			"GitCommit":     running.GitCommit,
+			"GoVersion":     running.GoVersion,
+			"KernelVersion": info.Host.Kernel,
 			"MinAPIVersion": version.APIVersion[version.Libpod][version.MinimalAPI].String(),
 			"Os":            goRuntime.GOOS,
 		},
 	}}
 
+	if conmon, oci, err := runtime.DefaultOCIRuntime().RuntimeInfo(); err != nil {
+		logrus.Warnf("Failed to retrieve Conmon and OCI Information: %q", err.Error())
+	} else {
+		additional := []types.ComponentVersion{
+			{
+				Name:    "Conmon",
+				Version: conmon.Version,
+				Details: map[string]string{
+					"Package": conmon.Package,
+				}},
+			{
+				Name:    fmt.Sprintf("OCI Runtime (%s)", oci.Name),
+				Version: oci.Version,
+				Details: map[string]string{
+					"Package": oci.Package,
+				}},
+		}
+		components = append(components, additional...)
+	}
+
 	apiVersion := version.APIVersion[version.Compat][version.CurrentAPI]
 	minVersion := version.APIVersion[version.Compat][version.MinimalAPI]
 
@@ -56,13 +75,13 @@ func VersionHandler(w http.ResponseWriter, r *http.Request) {
 			Platform: struct {
 				Name string
 			}{
-				Name: fmt.Sprintf("%s/%s/%s-%s", goRuntime.GOOS, goRuntime.GOARCH, infoData.Host.Distribution.Distribution, infoData.Host.Distribution.Version),
+				Name: fmt.Sprintf("%s/%s/%s-%s", goRuntime.GOOS, goRuntime.GOARCH, info.Host.Distribution.Distribution, info.Host.Distribution.Version),
 			},
 			APIVersion:    fmt.Sprintf("%d.%d", apiVersion.Major, apiVersion.Minor),
 			Arch:          components[0].Details["Arch"],
 			BuildTime:     components[0].Details["BuildTime"],
 			Components:    components,
-			Experimental:  true,
+			Experimental:  false,
 			GitCommit:     components[0].Details["GitCommit"],
 			GoVersion:     components[0].Details["GoVersion"],
 			KernelVersion: components[0].Details["KernelVersion"],