From 0531e2ce484878155d15b362ea5349ffa27e9272 Mon Sep 17 00:00:00 2001
From: runzexia <runzexia@yunify.com>
Date: Wed, 15 Jan 2020 11:11:04 +0800
Subject: [PATCH] add more comments

Signed-off-by: runzexia <runzexia@yunify.com>
---
 pkg/models/devops/project_pipeline.go | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/pkg/models/devops/project_pipeline.go b/pkg/models/devops/project_pipeline.go
index cf7e1eccf..495aba7da 100644
--- a/pkg/models/devops/project_pipeline.go
+++ b/pkg/models/devops/project_pipeline.go
@@ -380,7 +380,11 @@ func (s *Parameters) fromEtree(properties *etree.Element) *Parameters {
 					Type:         ParameterTypeMap["hudson.model.PasswordParameterDefinition"],
 				})
 			case "hudson.model.ChoiceParameterDefinition":
-				/* case1
+				/*
+				In Jenkins, different configuration methods will lead to different serialization results.
+				We need to be compatible with serialization results.
+
+				case1: Configured by KubeSphere console / Jenkins console
 				<hudson.model.ChoiceParameterDefinition>
 				<name>1</name>
 				<description>x</description>
@@ -392,8 +396,7 @@ func (s *Parameters) fromEtree(properties *etree.Element) *Parameters {
 					</a>
 				</choices>
 				</hudson.model.ChoiceParameterDefinition>
-				*/
-				/* case2
+				case2: Configured by pipeline syntax, sample:  parameters { choice(name: 'CHOICES', choices: ['one', 'two', 'three'], description: '') }
 				<hudson.model.ChoiceParameterDefinition>
 				<name>1</name>
 				<description>x</description>