Merge pull request #122 from AkhilReddyS-NR/NR-303954

Set default value of `NRLoggingEnabled` Parameter to True
newrelic · Sep 9, 2024 · 020955f · 020955f
2 parents e0f511a + 136559b
commit 020955f
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 10 deletions.
diff --git a/README.md b/README.md
@@ -30,9 +30,7 @@ to deploy the log ingestion function manually.
 4. Build the SAM application (if on Linux `-u` can be omitted): `sam build -u --parameter-overrides 'ParameterKey=NRLicenseKey,ParameterValue=your-license-key-here'`
 5. Deploy the SAM application: `sam deploy --guided`
 
-Additional notes:
 
-* To set `LOGGING_ENABLED`: `sam build ... --parameter-overrides 'ParameterKey=NRLoggingEnabled,ParameterValue=True'`
 
 ### Serverless
 
@@ -44,9 +42,7 @@ Additional notes:
 6. Set the LICENSE_KEY environment variable: `export LICENSE_KEY=your-license-key-here`
 7. Deploy the function: `sls deploy`
 
-Additional notes:
 
-* To set `LOGGING_ENABLED`: `export LOGGING_ENABLED=True` (prior to deploy)
 
 ### Terraform
 

diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "aws-log-ingestion"
-version = "2.9.3"
+version = "2.9.4"
 description = ""
 authors = ["New Relic <[email protected]>"]
 license = "Apache 2.0"

diff --git a/serverless.yml b/serverless.yml
@@ -15,7 +15,7 @@ provider:
     # Your NewRelic license key
     LICENSE_KEY: ${env:LICENSE_KEY}
     # Determines if logs are forwarded to New Relic Logging
-    LOGGING_ENABLED: ${env:LOGGING_ENABLED, "False"}
+    LOGGING_ENABLED: ${env:LOGGING_ENABLED, "True"}
     # A boolean to determine if you want to output debug messages in the CloudWatch console
     DEBUG_LOGGING_ENABLED: ${env:DEBUG_LOGGING_ENABLED, "False"}
 

diff --git a/src/function.py b/src/function.py
@@ -118,7 +118,7 @@ class EntryType(Enum):
     r"(?P<request_id>[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12})"
 )
 
-LOGGING_LAMBDA_VERSION = "2.9.3"
+LOGGING_LAMBDA_VERSION = "2.9.4"
 LOGGING_PLUGIN_METADATA = {"type": "lambda", "version": LOGGING_LAMBDA_VERSION}
 
 

diff --git a/template.yaml b/template.yaml
@@ -9,7 +9,7 @@ Parameters:
   NRLoggingEnabled:
     Type: String
     Description: Determines if logs are forwarded to New Relic Logging
-    Default: 'False'
+    Default: 'True'
     AllowedValues:
       - 'True'
       - 'False'
@@ -62,7 +62,7 @@ Metadata:
     LicenseUrl: LICENSE
     ReadmeUrl: README.md
     HomePageUrl: https://github.com/newrelic/aws-log-ingestion
-    SemanticVersion: 2.9.3
+    SemanticVersion: 2.9.4
     SourceCodeUrl: https://github.com/newrelic/aws-log-ingestion
 
 Resources:

diff --git a/terraform.tf b/terraform.tf
@@ -17,7 +17,7 @@ variable "nr_license_key" {
 variable "nr_logging_enabled" {
   type        = bool
   description = "Determines if logs are forwarded to New Relic Logging"
-  default     = false
+  default     = true
 }
 
 variable "nr_infra_logging" {