Skip to content

Commit

Permalink
fix incorrect cpu function type name
Browse files Browse the repository at this point in the history
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 <allain.legacy@windriver.com>
  • Loading branch information
alegacy committed Oct 30, 2019
1 parent c3321da commit 05013cc
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
4 changes: 2 additions & 2 deletions Gopkg.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pkg/apis/starlingx/v1/constructors.go
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ func parseProcessorInfo(profile *HostProfileSpec, host v1info.HostInfo) error {
}

function := strings.ToLower(c.Function)
if function == cpus.CPUFunctionApplications {
if function == cpus.CPUFunctionApplication {
// These cannot be configured. They are simply a placeholder
// for those CPUs that are not allocated for any other function.
continue
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 05013cc

Please sign in to comment.