Added description of issue

rundeck · Nov 4, 2024 · fe790d0 · fe790d0
1 parent 72b7c2e
commit fe790d0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/rundeck/job.go b/rundeck/job.go
@@ -207,7 +207,7 @@ type JobOption struct {
 	ValueChoicesURL string `xml:"valuesUrl,attr,omitempty"`
 
 	// URL value choices can be configured
-	JobValueChoicesURL JobValueChoicesURL `xml:"valuesUrlConfig,omitempty"`
+	JobValueChoicesURL JobValueChoicesURL `xml:"configRemoteUrl,omitempty"`
 
 	// Description of the value to be shown in the Rundeck UI.
 	Description string `xml:"description,omitempty"`

diff --git a/rundeck/resource_job.go b/rundeck/resource_job.go
@@ -844,7 +844,8 @@ func jobFromResourceData(d *schema.ResourceData) (*JobDetail, error) {
 				ValueChoices:    JobValueChoices([]string{}),
 				ValueChoicesURL: optionMap["value_choices_url"].(string),
 				// Mark: This should be a map[string]string of sorts to be able to parse the subset of URL options
-				// and should reference
+				// and should reference. Right now this compiles, but fails on runtime because the schema
+				// expects this to be a map, not an object (I think)
 				JobValueChoicesURL:      JobValueChoicesURL{},
 				RequirePredefinedChoice: optionMap["require_predefined_choice"].(bool),
 				ValidationRegex:         optionMap["validation_regex"].(string),