aws / aws/aws-cdk

(redshift-alpha): setLoggingProperties method for Clusters

Open
#22,034 1 comment 0 reactions 0 assignees View on GitHub
@aws-cdk/aws-redshift effort/small feature-request p3
Dominant language
TypeScript
Stars
12.9k
Forks
4.6k
Avg merge
2d 3h
Merged PRs (30d)
83

Description

### Describe the feature

Add a `setLoggingProperties` method for Clusters that enables setting the Cluster logging configuration post construction.

### Use Case

1. I have S3 buckets created later in my application
2. It's easier to conditionally set the logging configuration this way (dev, prod)
3. I have Redshift Clusters that get passed into a construct that must be configured in certain ways (ex. logging)

### Proposed Solution

1. Add `setLoggingProperties` to `Cluster` construct
2. Method takes in an `IBucket` and string prefix as parameters
3. Method throws an error if there is already a logging configuration set, otherwise it adds the logging configuration to the underlying `CfnCluster`

### Other Information

_No response_

### Acknowledgements

- [X] I may be able to implement this feature request
- [ ] This feature might incur a breaking change

### CDK version used

N/A

### Environment details (OS name and version, etc.)

N/A

Contributor guide

Open the contributing guide

Research direction

Locate the redshift-alpha Cluster construct and its underlying CfnCluster, then inspect how existing logging configuration and nearby mutator methods are implemented and tested. Done means adding setLoggingProperties with an IBucket and prefix, rejecting an existing logging configuration, and covering the behavior with tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
aws, typescript
Domain
cloud, databases
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.