Skip to content

Commit

Permalink
Merge pull request #934 from willkutler/OCM-6715
Browse files Browse the repository at this point in the history
OCM-6715 | add user defined tags to machinepool model
  • Loading branch information
gdbranco authored Apr 24, 2024
2 parents b7dd9a3 + 8212c44 commit 25aac61
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 0 deletions.
10 changes: 10 additions & 0 deletions model/clusters_mgmt/v1/aws_machine_pool_type.model
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,14 @@ class AWSMachinePool {

// Associates nodepool availability zones with zone types (e.g. wavelength, local).
AvailabilityZoneTypes [String]String

// Optional keys and values that the machine pool provisioner will add as AWS tags to all AWS resources it creates.
//
// AWS tags must conform to the following standards:
// - Each resource may have a maximum of 25 tags
// - Tags beginning with "aws:" are reserved for system use and may not be set
// - Tag keys may be between 1 and 128 characters in length
// - Tag values may be between 0 and 256 characters in length
// - Tags may only contain letters, numbers, spaces, and the following characters: [_ . : / = + - @]
Tags [String]String
}
7 changes: 7 additions & 0 deletions model/clusters_mgmt/v1/aws_node_pool_type.model
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,13 @@ class AWSNodePool {
InstanceProfile String

// Optional keys and values that the installer will add as tags to all AWS resources it creates.
//
// AWS tags must conform to the following standards:
// - Each resource may have a maximum of 25 tags
// - Tags beginning with "aws:" are reserved for system use and may not be set
// - Tag keys may be between 1 and 128 characters in length
// - Tag values may be between 0 and 256 characters in length
// - Tags may only contain letters, numbers, spaces, and the following characters: [_ . : / = + - @]
Tags [String]String

// Associates nodepool subnets with AWS Outposts.
Expand Down

0 comments on commit 25aac61

Please sign in to comment.