fix: 3.13 debug logging, error message typo, remove unhelpful warning log (#1799)

* minor fixes

* flake

Author: hallvictoria

eng/templates/official/jobs/publish-library-release.yml

@@ -57,6 +57,9 @@ jobs:
   dependsOn: ['CheckGitHubRelease']
   displayName: 'PyPI Package'
   steps:
+  - script: |
+      echo "##vso[task.setvariable variable=BranchName]refs/heads/${{ parameters.BRANCH_NAME }}/$(NewLibraryVersion)"
+    displayName: 'Set branch variable'
   - task: DownloadPipelineArtifact@2
     displayName: 'Download Python V2 Library release-*/x.y.z Artifact'
     inputs:
@@ -65,7 +68,7 @@ jobs:
       definition: 652
       specificBuildWithTriggering: true
       buildVersionToDownload: latestFromBranch
-      branchName: 'refs/heads/${{ parameters.BRANCH_NAME }}/$(NewLibraryVersion)'
+      branchName: '$(BranchName)'
       targetPath: PythonRuntimeArtifact
   - task: UsePythonVersion@0
     displayName: 'Use Python 3.13'

eng/templates/official/jobs/publish-release.yml

@@ -127,8 +127,8 @@ jobs:
       ((Get-Content test\WebJobs.Script.Tests\WebJobs.Script.Tests.csproj) -replace "PythonWorker`" Version=`"(\d)+.(\d)+.(\d)+.?(\d)*`"","PythonWorker`" Version=`"$newWorkerVersion`"" -join "`n") + "`n" | Set-Content -NoNewline test\WebJobs.Script.Tests\WebJobs.Script.Tests.csproj
   
       # Modify release_notes.md
-      Write-Host "Adding a new entry in release_note.md"
-      ((Get-Content release_notes.md) -replace "-->","$&`n- Update Python Worker Version to [$newWorkerVersion](https://github.com/Azure/azure-functions-python-worker/releases/tag/$newWorkerVersion)" -join "`n") + "`n" | Set-Content -NoNewline release_notes.md
+      Write-Host "Adding a new entry in release_notes.md"
+      Add-Content -Path release_notes.md -Value "`n- Update Python Worker Version to [$newWorkerVersion](https://github.com/Azure/azure-functions-python-worker/releases/tag/$newWorkerVersion)"
   
       # Commit Python Version
       Write-Host "Pushing $newBranch to host repo"

runtimes/v1/azure_functions_runtime_v1/functions.py

@@ -135,7 +135,7 @@ def validate_function_params(params: dict, bound_params: dict,
             raise FunctionLoadError(
                 func_name,
                 'the following parameters are declared in function.json but '
-                'not in Python: ' + repr(set(params) - set(bound_params)))
+                'not in Python: ' + repr(set(bound_params) - set(params)))
 
         input_types: typing.Dict[str, ParamTypeInfo] = {}
         output_types: typing.Dict[str, ParamTypeInfo] = {}

runtimes/v1/azure_functions_runtime_v1/handle_event.py

@@ -276,7 +276,7 @@ async def function_environment_reload_request(request):
             os.environ[var] = env_vars[var]
 
         if is_envvar_true(PYTHON_ENABLE_DEBUG_LOGGING):
-            root_logger = logging.getLogger("azure.functions")
+            root_logger = logging.getLogger()
             root_logger.setLevel(logging.DEBUG)
 
         # calling load_binding_registry again since the

runtimes/v2/azure_functions_runtime/functions.py

@@ -160,7 +160,7 @@ def validate_function_params(params: dict, bound_params: dict,
                 func_name,
                 'Extra parameters in binding definition — the following parameters '
                 'are declared as bindings but are not '
-                'present in Python: ' + repr(set(params) - set(bound_params)))
+                'present in Python: ' + repr(set(bound_params) - set(params)))
 
         input_types: typing.Dict[str, ParamTypeInfo] = {}
         output_types: typing.Dict[str, ParamTypeInfo] = {}

runtimes/v2/azure_functions_runtime/handle_event.py

@@ -326,7 +326,7 @@ async def function_environment_reload_request(request):
         # TODO: Apply PYTHON_THREADPOOL_THREAD_COUNT
 
         if is_envvar_true(PYTHON_ENABLE_DEBUG_LOGGING):
-            root_logger = logging.getLogger("azure.functions")
+            root_logger = logging.getLogger()
             root_logger.setLevel(logging.DEBUG)
 
         # calling load_binding_registry again since the

workers/azure_functions_worker/utils/dependency.py

@@ -405,7 +405,7 @@ def _remove_module_cache(path: str):
                 if any([p for p in module_paths if p.startswith(path)]):
                     sys.modules.pop(module_name)
             except Exception as e:
-                logger.warning(
+                logger.debug(
                     'Attempt to remove module cache for %s but failed with '
                     '%s. Using the original module cache.',
                     module_name, e)

workers/proxy_worker/utils/dependency.py

@@ -302,7 +302,7 @@ def _remove_module_cache(path: str):
                 if any([p for p in module_paths if p.startswith(path)]):
                     sys.modules.pop(module_name)
             except Exception as e:
-                logger.warning(
+                logger.debug(
                     'Attempt to remove module cache for %s but failed with '
                     '%s. Using the original module cache.',
                     module_name, e)

workers/proxy_worker/version.py

@@ -1,4 +1,4 @@
 # Copyright (c) Microsoft Corporation. All rights reserved.
 # Licensed under the MIT License.
 
-VERSION = "4.40.2"
+VERSION = '4.40.2'