-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
(docdb): incorrect arn generation DatabaseInstance #19554
Labels
@aws-cdk/aws-docdb
Related to Amazon DocumentDB
bug
This issue is a bug.
effort/small
Small work item – less than a day of effort
p1
Comments
peterwoodworth
added
bug
This issue is a bug.
needs-triage
This issue or PR still needs to be triaged.
labels
Mar 25, 2022
Tracking: P61209042 |
peterwoodworth
added
p1
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 25, 2022
4 tasks
mergify bot
pushed a commit
that referenced
this issue
Mar 29, 2022
…tead of 'rds' (#19555) Fixes #19554 ---- ### All Submissions: * [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md) ### Adding new Unconventional Dependencies: * [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md/#adding-new-unconventional-dependencies) ### New Features * [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/master/INTEGRATION_TESTS.md)? * [ ] Did you use `cdk-integ` to deploy the infrastructure and generate the snapshot (i.e. `cdk-integ` without `--dry-run`)? *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
StevePotter
pushed a commit
to StevePotter/aws-cdk
that referenced
this issue
Apr 27, 2022
…tead of 'rds' (aws#19555) Fixes aws#19554 ---- ### All Submissions: * [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md) ### Adding new Unconventional Dependencies: * [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/master/CONTRIBUTING.md/#adding-new-unconventional-dependencies) ### New Features * [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/master/INTEGRATION_TESTS.md)? * [ ] Did you use `cdk-integ` to deploy the infrastructure and generate the snapshot (i.e. `cdk-integ` without `--dry-run`)? *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
@aws-cdk/aws-docdb
Related to Amazon DocumentDB
bug
This issue is a bug.
effort/small
Small work item – less than a day of effort
p1
What is the problem?
docdb arns require using
rds
as the service according to docdb docsReproduction Steps
reference
DatabaseInstance.InstanceArn
What did you expect to happen?
arn with rds as service
What actually happened?
arn with docdb as service
CDK CLI Version
2.17.0
Framework Version
No response
Node.js Version
16.3
OS
mac
Language
Typescript
Language Version
No response
Other information
No response
The text was updated successfully, but these errors were encountered: