Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(batch): add patch to keep the old name of the Volumes, and EfsVolumeConfiguration types #1545

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

moelasmar
Copy link
Contributor

@moelasmar moelasmar commented Jan 6, 2025

add patch to keep the old name of the Volumes, and EfsVolumeConfiguration types.

This PR deletes the Volume, and EFSVolumeConfiguration types.

Copy link

github-actions bot commented Jan 6, 2025

@aws-cdk/aws-service-spec: Model database diff detected

└[~] service aws-batch
  └ resources
     └[~]  resource AWS::Batch::JobDefinition
        └ types
           ├[~] type ContainerProperties
           │ └ properties
           │    └ Volumes: - Array<Volume> ⇐ Array<Volumes>
           │               + Array<Volumes>
           ├[~] type EcsTaskProperties
           │ └ properties
           │    └ Volumes: - Array<Volume>
           │               + Array<Volumes>
           ├[~] type EfsVolumeConfiguration
           │ └ properties
           │    └ AuthorizationConfig: - AuthorizationConfig
           │                           + EFSAuthorizationConfig ⇐ AuthorizationConfig
           ├[-]  type EFSVolumeConfiguration
           │  ├      documentation: This is used when you're using an Amazon Elastic File System file system for job storage. For more information, see [Amazon EFS Volumes](https://docs.aws.amazon.com/batch/latest/userguide/efs-volumes.html) in the *AWS Batch User Guide* .
           │  │      name: EFSVolumeConfiguration
           │  └ properties
           │     ├ TransitEncryption: string
           │     ├ AuthorizationConfig: EFSAuthorizationConfig
           │     ├ FileSystemId: string (required)
           │     ├ RootDirectory: string
           │     └ TransitEncryptionPort: integer
           ├[~] type MultiNodeContainerProperties
           │ └ properties
           │    └ Volumes: - Array<Volume>
           │               + Array<Volumes>
           ├[~] type MultiNodeEcsTaskProperties
           │ └ properties
           │    └ Volumes: - Array<Volume>
           │               + Array<Volumes>
           ├[-]  type Volume
           │  ├      documentation: A data volume that's used in a job's container properties.
           │  │      name: Volume
           │  └ properties
           │     ├ Host: Host
           │     ├ EfsVolumeConfiguration: EFSVolumeConfiguration
           │     └ Name: string
           └[~] type Volumes
             └ properties
                └ Host: - VolumesHost
                        + Host ⇐ VolumesHost

@aws-cdk-automation aws-cdk-automation added this pull request to the merge queue Jan 6, 2025
Merged via the queue into main with commit db39f6f Jan 6, 2025
11 checks passed
@aws-cdk-automation aws-cdk-automation deleted the melasmar/aws_batch_newpatch branch January 6, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants