Merge branch 'main' into other/azure-functions-httptrigger-dt

newrelic · Nov 5, 2024 · 5e98a66 · 5e98a66
2 parents 858e922 + 95477b9
commit 5e98a66
Show file tree

Hide file tree

Showing 70 changed files with 1,333 additions and 611 deletions.
diff --git a/.github/workflows/all_solutions.yml b/.github/workflows/all_solutions.yml
@@ -34,12 +34,32 @@ jobs:
       contents: read
       pull-requests: read
 
+  shellcheck:
+    name: Validate shell scripts
+    needs: check-modified-files
+    runs-on: ubuntu-latest
+    steps:
+      - name: Harden Runner
+        uses: step-security/harden-runner@91182cccc01eb5e619899d80e4e971d6181294a7 # v2.10.1
+        with:
+          disable-sudo: true
+          egress-policy: audit
+
+      - name: Checkout
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+        with:
+          fetch-depth: 0
+      - name: Run Shellcheck
+        run: |
+          find ${{ github.workspace }} -name "*.sh" -exec shellcheck --severity=error {} +
 
   # This builds both FullAgent and MSIInstaller since MSIInstaller requires FullAgent artifacts.
   build-fullagent-msi:
     name: Build FullAgent and MSIInstaller
     runs-on: windows-2022
-    needs: check-modified-files
+    needs:
+      - check-modified-files
+      - shellcheck
     # don't run this job if triggered by Dependabot, will cause all other jobs to be skipped as well
     # run this job if non-workflow files were modified, or if triggered by a release, a manual execution or schedule
     if: github.actor != 'dependabot[bot]' && (needs.check-modified-files.outputs.non-workflow-files-changed == 'true' || github.event.release || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule')

diff --git a/.github/workflows/multiverse_run.yml b/.github/workflows/multiverse_run.yml
@@ -62,7 +62,7 @@ jobs:
         shell: bash
 
       - name: Setup .NET Core 3.1.100
-        uses: actions/setup-dotnet@6bd8b7f7774af54e05809fcc5431931b3eb1ddee # v4.0.1
+        uses: actions/setup-dotnet@3e891b0cb619bf60e2c25674b222b8940e2c1c25 # v4.1.0
         with:
           dotnet-version: '3.1.100'
 
@@ -107,7 +107,7 @@ jobs:
           egress-policy: audit 
 
       - name: Setup .NET Core 3.1.100
-        uses: actions/setup-dotnet@6bd8b7f7774af54e05809fcc5431931b3eb1ddee # v4.0.1
+        uses: actions/setup-dotnet@3e891b0cb619bf60e2c25674b222b8940e2c1c25 # v4.1.0
         with:
           dotnet-version: '3.1.100'
 

diff --git a/.github/workflows/run_linux_container_tests.yml b/.github/workflows/run_linux_container_tests.yml
@@ -76,6 +76,6 @@ jobs:
           INTEGRATION_TEST_SECRETS: ${{ secrets.TEST_SECRETS }}
         run: |
           echo $INTEGRATION_TEST_SECRETS | dotnet user-secrets set --project ${{ env.integration_tests_shared_project }}
-  
+
       - name: Build & Run Linux Container Integration Tests
         run: dotnet test ./tests/Agent/IntegrationTests/ContainerIntegrationTests/ContainerIntegrationTests.csproj --framework net8.0
diff --git a/.github/workflows/scripts/nugetSlackNotifications/PackageInfo.cs b/.github/workflows/scripts/nugetSlackNotifications/PackageInfo.cs
@@ -1,4 +1,4 @@
-using System.Text.Json.Serialization;
+using System.Text.Json.Serialization;
 
 namespace nugetSlackNotifications
 {
@@ -10,6 +10,8 @@ public class PackageInfo
         public bool IgnorePatch { get; set; }
         [JsonPropertyName("ignoreMinor")]
         public bool IgnoreMinor { get; set; }
+        [JsonPropertyName("ignoreMajor")]
+        public bool IgnoreMajor { get; set; }
         [JsonPropertyName("ignoreReason")]
         public string IgnoreReason {get; set;}
     }

diff --git a/.github/workflows/scripts/nugetSlackNotifications/Program.cs b/.github/workflows/scripts/nugetSlackNotifications/Program.cs
@@ -131,7 +131,7 @@ static async Task CheckPackage(PackageInfo package, PackageMetadataResource meta
             // check publish date
             if (latest.Published >= searchTime)
             {
-                if (previous != null && (package.IgnorePatch || package.IgnoreMinor))
+                if (previous != null && (package.IgnorePatch || package.IgnoreMinor || package.IgnoreMajor))
                 {
                     var previousVersion = previous.Identity.Version;
                     var latestVersion = latest.Identity.Version;
@@ -154,6 +154,14 @@ static async Task CheckPackage(PackageInfo package, PackageMetadataResource meta
                             return;
                         }
                     }
+                    if (package.IgnoreMajor)
+                    {
+                        if (previousVersion.Major != latestVersion.Major)
+                        {
+                            Log.Information($"Package {packageName} ignores Major version updates.");
+                            return;
+                        }
+                    }
                 }
 
                 var previousVersionDescription = previous?.Identity.Version.ToNormalizedString() ?? "Unknown";

diff --git a/.github/workflows/scripts/nugetSlackNotifications/nugetSlackNotifications.csproj b/.github/workflows/scripts/nugetSlackNotifications/nugetSlackNotifications.csproj
@@ -6,7 +6,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="NewRelic.Agent.Api" Version="10.32.0" />
+    <PackageReference Include="NewRelic.Agent.Api" Version="10.33.1" />
     <PackageReference Include="Octokit" Version="13.0.1" />
     <PackageReference Include="Serilog" Version="4.1.0" />
     <PackageReference Include="Serilog.Sinks.Console" Version="6.0.0" />

diff --git a/.github/workflows/scripts/nugetSlackNotifications/packageInfo.json b/.github/workflows/scripts/nugetSlackNotifications/packageInfo.json
@@ -54,7 +54,11 @@
         "ignoreReason": "frequent patch releases create too much noise"
     },
     {
-        "packageName": "log4net"
+        "packageName": "log4net",
+        "ignorePatch": true,
+        "ignoreMinor": true,
+        "ignoreMajor": true,
+        "ignoreReason": "Breaking major update. See https://github.com/newrelic/newrelic-dotnet-agent/issues/2764"
     },
     {
         "packageName": "microsoft.extensions.logging"
@@ -66,7 +70,11 @@
         "packageName": "microsoft.net.http"
     },
     {
-        "packageName": "mongodb.driver"
+        "packageName": "mongodb.driver",
+        "ignorePatch": true,
+        "ignoreMinor": true,
+        "ignoreMajor": true,
+        "ignoreReason": "Breaking since at least 2.25.0. See https://new-relic.atlassian.net/browse/NR-281915"
     },
     {
         "packageName": "mysql.data"

diff --git a/build/Linux/build/common/run.sh b/build/Linux/build/common/run.sh
@@ -2,4 +2,4 @@
 
 # This script can be used to run a dotnet application with New Relic monitoring.
 
-NRHOME=${CORECLR_NEWRELIC_HOME:-${CORECLR_NEW_RELIC_HOME:-/usr/local/newrelic-dotnet-agent}} CORECLR_ENABLE_PROFILING=1 CORECLR_PROFILER={36032161-FFC0-4B61-B559-F6C5D41BAE5A} CORECLR_PROFILER_PATH=$NRHOME/libNewRelicProfiler.so $@
+NRHOME="${CORECLR_NEWRELIC_HOME:-${CORECLR_NEW_RELIC_HOME:-/usr/local/newrelic-dotnet-agent}}" CORECLR_ENABLE_PROFILING=1 CORECLR_PROFILER={36032161-FFC0-4B61-B559-F6C5D41BAE5A} CORECLR_PROFILER_PATH="$NRHOME/libNewRelicProfiler.so" "$@"
diff --git a/build/Linux/build/common/setenv.sh b/build/Linux/build/common/setenv.sh
@@ -1,11 +1,11 @@
 #!/bin/bash
 
-if [ -z "$CORECLR_NEWRELIC_HOME" ]; && [ -z "$CORECLR_NEW_RELIC_HOME" ]; then
+if [ -z "$CORECLR_NEWRELIC_HOME" ] && [ -z "$CORECLR_NEW_RELIC_HOME" ]; then
     echo "CORECLR_NEWRELIC_HOME is undefined"
 else
     NRHOME=${CORECLR_NEWRELIC_HOME:-${CORECLR_NEW_RELIC_HOME}}
 
     export CORECLR_ENABLE_PROFILING=1
     export CORECLR_PROFILER={36032161-FFC0-4B61-B559-F6C5D41BAE5A}
     export CORECLR_PROFILER_PATH=$NRHOME/libNewRelicProfiler.so
-fi
+fi