feat(cloudfront): add ability to set RealtimeLogConfigARN

[Raffaello]

this fixes #2284

this works adding in the configuration yaml:

defaults: # optional
        realtimeLogConfigArn: 'the realtime log config ARN'

[slsnextbot]

Handler Size Report

No changes to handler sizes.

Base Handler Sizes (kB) (commit 36a1751)

{
    "Lambda": {
        "Default Lambda": {
            "Standard": 1524,
            "Minified": 668
        },
        "Image Lambda": {
            "Standard": 1488,
            "Minified": 800
        }
    },
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1534,
            "Minified": 673
        },
        "Default Lambda V2": {
            "Standard": 1526,
            "Minified": 670
        },
        "API Lambda": {
            "Standard": 634,
            "Minified": 318
        },
        "Image Lambda": {
            "Standard": 1496,
            "Minified": 805
        },
        "Regeneration Lambda": {
            "Standard": 1187,
            "Minified": 546
        },
        "Regeneration Lambda V2": {
            "Standard": 1253,
            "Minified": 573
        }
    }
}

New Handler Sizes (kB) (commit 456125b)

{
    "Lambda": {
        "Default Lambda": {
            "Standard": 1524,
            "Minified": 668
        },
        "Image Lambda": {
            "Standard": 1488,
            "Minified": 800
        }
    },
    "Lambda@Edge": {
        "Default Lambda": {
            "Standard": 1534,
            "Minified": 673
        },
        "Default Lambda V2": {
            "Standard": 1526,
            "Minified": 670
        },
        "API Lambda": {
            "Standard": 634,
            "Minified": 318
        },
        "Image Lambda": {
            "Standard": 1496,
            "Minified": 805
        },
        "Regeneration Lambda": {
            "Standard": 1187,
            "Minified": 546
        },
        "Regeneration Lambda V2": {
            "Standard": 1253,
            "Minified": 573
        }
    }
}

[codecov]

Codecov Report

Merging #2301 (36a1751) into master (36a1751) will not change coverage.
The diff coverage is n/a.

❗ Current head 36a1751 differs from pull request most recent head b052f4a. Consider uploading reports for the commit b052f4a to get more accurate results

@@           Coverage Diff           @@
##           master    #2301   +/-   ##
=======================================
  Coverage   83.55%   83.55%           
=======================================
  Files         102      102           
  Lines        3673     3673           
  Branches     1171     1171           
=======================================
  Hits         3069     3069           
  Misses        592      592           
  Partials       12       12           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36a1751...b052f4a. Read the comment docs.

[Raffaello]

hi @dphang 👋
when you have some time, could you please look into this one?
thanks.

[dphang]

Sure I think it looks fine, though not sure why the CI tests aren't running?

[Raffaello]

Sure I think it looks fine, though not sure why the CI tests aren't running?

@dphang
I don't know, I can't see details.
besides i am not authorized to merge the PR.

anyone that can look into that or merging the PR?

[Raffaello]

maybe just try to re trigger those 2 circle ci jobs.

[dphang]

No worries! I think we can merge and let master CI work