Skip to content

Commit

Permalink
Addressed review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
sssash18 committed Jan 22, 2024
1 parent 96afff6 commit 746ecb6
Show file tree
Hide file tree
Showing 8 changed files with 17 additions and 17 deletions.
4 changes: 2 additions & 2 deletions docs/documents/apis.md
Original file line number Diff line number Diff line change
Expand Up @@ -2352,7 +2352,7 @@ string
</tr>
<tr>
<td>
<code>architecture</code>
<code>cpuArchitecture</code>
</td>
<td>
<em>
Expand All @@ -2361,7 +2361,7 @@ string
</td>
<td>
<em>(Optional)</em>
<p>Architecture of the node belonging to nodeGroup</p>
<p>CPU Architecture of the node belonging to nodeGroup</p>
</td>
</tr>
</tbody>
Expand Down
6 changes: 3 additions & 3 deletions kubernetes/crds/machine.sapcloud.io_machineclasses.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,6 @@ spec:
description: NodeTemplate contains subfields to track all node resources
and other node info required to scale nodegroup from zero
properties:
architecture:
description: Architecture of the node belonging to nodeGroup
type: string
capacity:
additionalProperties:
anyOf:
Expand All @@ -68,6 +65,9 @@ spec:
description: Capacity contains subfields to track all node resources
required to scale nodegroup from zero
type: object
cpuArchitecture:
description: CPU Architecture of the node belonging to nodeGroup
type: string
instanceType:
description: Instance type of the node belonging to nodeGroup
type: string
Expand Down
4 changes: 2 additions & 2 deletions pkg/apis/machine/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -681,6 +681,6 @@ type NodeTemplate struct {
Zone string

// +optional
// Architecture of the node belonging to nodeGroup
Architecture *string
// CPU Architecture of the node belonging to nodeGroup
CpuArchitecture *string
}
4 changes: 2 additions & 2 deletions pkg/apis/machine/v1alpha1/machineclass_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,6 @@ type NodeTemplate struct {
Zone string `json:"zone"`

// +optional
// Architecture of the node belonging to nodeGroup
Architecture *string `json:"architecture,omitempty"`
// CPU Architecture of the node belonging to nodeGroup
CpuArchitecture *string `json:"cpuArchitecture,omitempty"`
}
4 changes: 2 additions & 2 deletions pkg/apis/machine/v1alpha1/zz_generated.conversion.go

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

4 changes: 2 additions & 2 deletions pkg/apis/machine/v1alpha1/zz_generated.deepcopy.go

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

4 changes: 2 additions & 2 deletions pkg/apis/machine/zz_generated.deepcopy.go

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

4 changes: 2 additions & 2 deletions pkg/openapi/openapi_generated.go

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

0 comments on commit 746ecb6

Please sign in to comment.