fix: regression for machines supporting nested virtualization on AWS

adrianriobo · adrianriobo · commit fc3f583b4a65 · 2025-08-07T11:15:51.000+02:00
Due to refactor on compute-request selector the value for nested virtualization was missing when filtering machines. Then even when the flag was used the type of machines returned are not taking it into consideration. This commit ensures the value is pass again in the filter. Fix #563. Signed-off-by: Adrian Riobo <ariobolo@redhat.com>
diff --git a/pkg/provider/aws/data/compute-request.go b/pkg/provider/aws/data/compute-request.go
@@ -58,6 +58,7 @@ func getInstanceTypes(args *computerequest.ComputeRequestArgs) ([]string, error)
 		maxResults := computerequest.MaxResults
 		filters.MaxResults = &maxResults
 	}
+	filters.BareMetal = &args.NestedVirt
 	//nolint:staticcheck // following method is deprecated but no replacement yet
 	instanceTypesSlice, err := instanceSelector.Filter(
 		context.Background(),

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ func getInstanceTypes(args *computerequest.ComputeRequestArgs) ([]string, error)`
`58`	`58`	`maxResults := computerequest.MaxResults`
`59`	`59`	`filters.MaxResults = &maxResults`
`60`	`60`	`}`
	`61`	`+ filters.BareMetal = &args.NestedVirt`
`61`	`62`	`//nolint:staticcheck // following method is deprecated but no replacement yet`
`62`	`63`	`instanceTypesSlice, err := instanceSelector.Filter(`
`63`	`64`	`context.Background(),`