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

Modify snapshotRetentionLimit to 0 for cluster creations. #66

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

kyriechen96
Copy link
Contributor

Issue #, if available:
Snapshot window sometimes causes additional snapshotting process during cluster creation which takes additional time.

Description of changes:
Modify snapshotRetentionLimit to 0 for cluster creations.

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

@ack-prow ack-prow bot added the approved label Feb 20, 2023
@kyriechen96 kyriechen96 requested review from nmvk and removed request for vijtrip2 and RedbackThomson March 2, 2023 09:12
@nmvk
Copy link
Member

nmvk commented Mar 3, 2023

/lgtm

@ack-prow ack-prow bot assigned nmvk Mar 3, 2023
@ack-prow ack-prow bot added the lgtm Indicates that a PR is ready to be merged. label Mar 3, 2023
@ack-prow
Copy link

ack-prow bot commented Mar 3, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kyriechen96, nmvk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kyriechen96
Copy link
Contributor Author

/retest

@ack-prow ack-prow bot merged commit 23e6d49 into aws-controllers-k8s:main Mar 3, 2023
kyriechen96 added a commit to kyriechen96/memorydb-controller that referenced this pull request Mar 3, 2023
…llers-k8s#66)

Issue #, if available:
Snapshot window sometimes causes additional snapshotting process during cluster creation which takes additional time.

Description of changes:
Modify snapshotRetentionLimit to 0 for cluster creations.

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
approved lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants