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

feat: Use GuCDK's log shipping policy construct #607

Merged
merged 1 commit into from
Jun 21, 2021

Conversation

akash1810
Copy link
Member

@akash1810 akash1810 commented Jun 21, 2021

Follows #600 and #598.

What does this change?

This change replaces the YAML defined log shipping policy with the GuCDK's GuLogShippingPolicy construct.

GuLogShippingPolicy also brings a Parameter Store driven parameter into the stack, which replaces KinesisStreamName.

How to test

  • Perform a full deploy of this branch
  • Witness logs continuing to be shipped into Central ELK

How can we measure success?

We move closer to a CDK only template.

@akash1810 akash1810 requested a review from a team June 21, 2021 09:27
@akash1810 akash1810 force-pushed the aa-replace-ssm-policy branch from 443187b to 342d83a Compare June 21, 2021 09:53
@akash1810 akash1810 force-pushed the aa-replace-log-shipping-policy branch from 4bcaccd to f1bf258 Compare June 21, 2021 09:53
@akash1810 akash1810 force-pushed the aa-replace-ssm-policy branch from 342d83a to e8e02da Compare June 21, 2021 12:03
This change replaces the YAML defined log shipping policy with the GuCDK's `GuLogShippingPolicy` construct.

`GuLogShippingPolicy` also brings a Parameter Store driven parameter into the stack, which replaces `KinesisStreamName`.
@akash1810 akash1810 force-pushed the aa-replace-log-shipping-policy branch from f1bf258 to c695e0b Compare June 21, 2021 12:05
Base automatically changed from aa-replace-ssm-policy to main June 21, 2021 12:07
@akash1810 akash1810 merged commit 6eaf638 into main Jun 21, 2021
@akash1810 akash1810 deleted the aa-replace-log-shipping-policy branch June 21, 2021 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants