Skip to content

Commit 05013cc

Browse files
committed
fix incorrect cpu function type name
The last commit was tested and pushed without realizing that the "application" CPU function type was still present in the host defaults and the generated host profiles from the "deploy" tool. The constants reflecting the API values needed to be updated. Signed-off-by: Allain Legacy <[email protected]>
1 parent c3321da commit 05013cc

3 files changed

Lines changed: 9 additions & 8 deletions

File tree

Gopkg.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/apis/starlingx/v1/constructors.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ func parseProcessorInfo(profile *HostProfileSpec, host v1info.HostInfo) error {
161161
}
162162

163163
function := strings.ToLower(c.Function)
164-
if function == cpus.CPUFunctionApplications {
164+
if function == cpus.CPUFunctionApplication {
165165
// These cannot be configured. They are simply a placeholder
166166
// for those CPUs that are not allocated for any other function.
167167
continue

vendor/github.com/gophercloud/gophercloud/starlingx/inventory/v1/cpus/requests.go

Lines changed: 6 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)