-
Notifications
You must be signed in to change notification settings - Fork 457
[AWS] Add Route 53 Public Zone Datastream #2316
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
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
e02bae2
to
9dbe3c1
Compare
@kaiyan-sheng when u get a chance, could u take a look at this? I will fix the conflicts later today. I plan to create another PR for the private route 53 logging after this is merged. |
9dbe3c1
to
70450fc
Compare
@legoguy1000 Will do! Thanks for adding this and also using |
Tracking the GeoIP stuff. I will swap them out to make full use of the processor. |
packages/aws/data_stream/route53_public_logs/agent/stream/aws-cloudwatch.yml.hbs
Show resolved
Hide resolved
cddc3cf
to
3f0489f
Compare
@kaiyan-sheng should be updated per ur comments. Also if u want to merge #2323 first then I can just update this to match how you did the closest cloud watch input |
3f0489f
to
b2cef6b
Compare
@kaiyan-sheng When u get a chance, could you review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@legoguy1000 Sorry for the delay!! Let's merge your PR first and then I can work on mine later!
/test |
@legoguy1000 Could you rerun the test for test-route53.log please? Thanks!! |
@kaiyan-sheng Should be good now. |
Hmm seems like the expected file is still a little off:
|
Could it be a stack version mismatch? I think I generated the data with a 7.16.x stack but it looks like Jenkins spun up 7.15. I can see if that makes a difference. |
@legoguy1000 I think that's the problem hmm |
7.15.0 stack didn't produce any different data for my locally run test, trying 8.0.0-SNAPSHOT. |
Looks like I just needed to run |
What does this PR do?
Adds a new datastream for Route 53 Public Zone logs
Checklist
changelog.yml
file.Author's Checklist
How to test this PR locally
Related issues
Screenshots