[AWS] Add Route 53 Public Zone Datastream#2316
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
Outdated
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? |
kaiyan-sheng
left a comment
There was a problem hiding this comment.
@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.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots