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

(aws_rds): (Add Support for RDS Oracle 19.0.0.0.ru-2024-01.rur-2024-01.r1 and 21.0.0.0.ru-2024-01.rur-2024-01.r1 in OracleEngineVersion) #29360

Closed
2 tasks
ayushrawat007 opened this issue Mar 5, 2024 · 4 comments · Fixed by #29422
Assignees
Labels
@aws-cdk/aws-rds Related to Amazon Relational Database effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2

Comments

@ayushrawat007
Copy link

Describe the feature

As per the RDS Doc: both RDS Oracle 19.0.0.0.ru-2024-01.rur-2024-01.r1 and 21.0.0.0.ru-2024-01.rur-2024-01.r1 are supported.

https://docs.aws.amazon.com/AmazonRDS/latest/OracleReleaseNotes/oracle-version-21-0.html#oracle-version-RU-RUR.21.0.0.0.ru-2024-01.rur-2024-01.r1
https://docs.aws.amazon.com/AmazonRDS/latest/OracleReleaseNotes/oracle-version-19-0.html#oracle-version-RU-RUR.19.0.0.0.ru-2024-01.rur-2024-01.r1

However the same is not reflected in the OracleEngineVersion class.

Hence requesting to add the support for the same.

Use Case

To be able to define RDS Oracle 19.0.0.0.ru-2024-01.rur-2024-01.r1 and 21.0.0.0.ru-2024-01.rur-2024-01.r1 via Static property value like the rest of the RDS Oracle versions.

Workaround: static of(oracleFullVersion, oracleMajorVersion) is know to us.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.131.0

Environment details (OS name and version, etc.)

Mac

@ayushrawat007 ayushrawat007 added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Mar 5, 2024
@github-actions github-actions bot added the @aws-cdk/aws-rds Related to Amazon Relational Database label Mar 5, 2024
@pahud
Copy link
Contributor

pahud commented Mar 6, 2024

Yes we should add that into the enum-like class.

@pahud pahud added p2 effort/small Small work item – less than a day of effort and removed needs-triage This issue or PR still needs to be triaged. labels Mar 6, 2024
@pahud pahud self-assigned this Mar 6, 2024
@ayush-shah-1501
Copy link
Contributor

@pahud I can take this

@nmussy
Copy link
Contributor

nmussy commented Mar 9, 2024

I'm currently working on a tool to help detect these kinds of drifts (RDS engine versions, EC2 Windows images, etc.), by comparing the CDK code to SDK requests. For RDS, it's also checking whether a deprecated engine has been marked as such in TSDoc:

image

It's just a CLI at the moment, but I'll probably end up turning it into a scheduled task, maybe even automate the opening of issues here if it's something you're interested in?

@mergify mergify bot closed this as completed in #29422 Mar 11, 2024
mergify bot pushed a commit that referenced this issue Mar 11, 2024
### Issue # (if applicable)

Closes #29360, closes #23810

**Note**: #29421 already adds some of the missing Oracle versions, this PR also includes them

### Reason for this change

Update the CDK RDS engine versions to match the current availability, as well as facilitate automated alerts for updates in the future

### Description of changes

* Added missing engine version for all RDS engines
	* This includes engines missing in the CDK that are already deprecated, but still returned by the SDK. The goal is to remove any differences between the two, even if it means a few needless static fields
* Updated missing or un-needed `@deprecated` TSDoc tags

### Description of how you validated changes

Like I mentioned in #29360 (comment), I'm working on a CLI that compares the `aws-cdk-lib` TypeScript declarations to SDK API calls, in this case `DescribeDBEngineVersions`.
I am re-validating these locally edited files as I update them, until I have no differences between the CDK and the live SDK results.

This is currently a mostly manual process. I'll add support for more version fields such as `ec2.WindowsVersion` before adding automation, to bridge the other currently missing versions ASAP.

### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-rds Related to Amazon Relational Database effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2
Projects
None yet
4 participants