Improve Vacation Planner Implementation (#93)

* Improve Vacation Planner Implementation

* Add gunicorn.conf.py to zip file

Author: paolosalvatori

samples/web-app-cosmosdb-mongodb-api/python/bicep/deploy.sh

@@ -134,7 +134,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py mongodb.py static templates requirements.txt
+zip -r "$ZIPFILE" app.py mongodb.py gunicorn.conf.py static templates requirements.txt
 
 # Deploy the web app
 # Deploy the web app

samples/web-app-cosmosdb-mongodb-api/python/scripts/deploy.sh

@@ -919,7 +919,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py mongodb.py static templates requirements.txt
+zip -r "$ZIPFILE" app.py mongodb.py gunicorn.conf.py static templates requirements.txt
 
 # List the contents of the zip package
 echo "Contents of the zip package [$ZIPFILE]:"

samples/web-app-cosmosdb-mongodb-api/python/src/gunicorn.conf.py

@@ -0,0 +1,18 @@
+import os
+
+
+def worker_int(worker):
+    # SIGINT (Ctrl+C) default path raises SystemExit inside the worker's recv()
+    # loop, dumping a traceback through gunicorn's HTTP parser frames. os._exit
+    # short-circuits the unwind for a clean foreground stop. SIGTERM (graceful)
+    # is unaffected — it goes through a different code path.
+    os._exit(0)
+
+
+def worker_abort(worker):
+    # SIGABRT is what the arbiter sends when a worker misses its heartbeat
+    # ([CRITICAL] WORKER TIMEOUT). The default handler does sys.exit(1), which
+    # unwinds through the same recv() stack as SIGINT and prints a misleading
+    # traceback. The WORKER TIMEOUT log line above it is the real diagnostic;
+    # exit at the C level to suppress the spurious trace.
+    os._exit(1)

samples/web-app-cosmosdb-mongodb-api/python/src/requirements.txt

@@ -1,5 +1,5 @@
 Flask==3.1.3
-azure-identity==1.25.1
-pymongo==4.15.3
-gunicorn==23.0.0
-python-dotenv==1.2.2
+azure-identity==1.25.3
+pymongo==4.17.0
+gunicorn==26.0.0
+python-dotenv==1.2.2

samples/web-app-cosmosdb-mongodb-api/python/terraform/deploy.sh

@@ -61,7 +61,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py mongodb.py static templates requirements.txt
+zip -r "$ZIPFILE" app.py mongodb.py gunicorn.conf.py static templates requirements.txt
 
 # Deploy the web app
 # Deploy the web app

samples/web-app-cosmosdb-nosql-api/python/scripts/deploy.sh

@@ -108,7 +108,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py cosmosdb_client.py static templates requirements.txt
+zip -r "$ZIPFILE" app.py cosmosdb_client.py gunicorn.conf.py static templates requirements.txt
 
 # Deploy the web app
 echo "Deploying web app [$WEB_APP_NAME] with zip file [$ZIPFILE]..."

samples/web-app-cosmosdb-nosql-api/python/src/gunicorn.conf.py

@@ -0,0 +1,18 @@
+import os
+
+
+def worker_int(worker):
+    # SIGINT (Ctrl+C) default path raises SystemExit inside the worker's recv()
+    # loop, dumping a traceback through gunicorn's HTTP parser frames. os._exit
+    # short-circuits the unwind for a clean foreground stop. SIGTERM (graceful)
+    # is unaffected — it goes through a different code path.
+    os._exit(0)
+
+
+def worker_abort(worker):
+    # SIGABRT is what the arbiter sends when a worker misses its heartbeat
+    # ([CRITICAL] WORKER TIMEOUT). The default handler does sys.exit(1), which
+    # unwinds through the same recv() stack as SIGINT and prints a misleading
+    # traceback. The WORKER TIMEOUT log line above it is the real diagnostic;
+    # exit at the C level to suppress the spurious trace.
+    os._exit(1)

samples/web-app-cosmosdb-nosql-api/python/src/requirements.txt

@@ -1,6 +1,6 @@
 Flask==3.1.3
-azure-mgmt-cosmosdb==9.8.0
-azure-cosmos==4.7.0
-gunicorn==22.0.0
+azure-mgmt-cosmosdb==9.9.0
+azure-cosmos==4.15.0
+gunicorn==26.0.0
 python-dotenv==1.2.2
-setuptools==79.0.1
+setuptools==82.0.1

samples/web-app-managed-identity/python/bicep/deploy.sh

@@ -129,7 +129,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py requirements.txt static templates
+zip -r "$ZIPFILE" app.py gunicorn.conf.py requirements.txt static templates
 
 # Deploy the web app
 echo "Deploying web app [$WEB_APP_NAME] with zip file [$ZIPFILE]..."

samples/web-app-managed-identity/python/scripts/system-assigned.sh

@@ -248,7 +248,7 @@ fi
 
 # Create the zip package of the web app
 echo "Creating zip package of the web app..."
-zip -r "$ZIPFILE" app.py requirements.txt static templates
+zip -r "$ZIPFILE" app.py gunicorn.conf.py requirements.txt static templates
 
 # Deploy the web app
 echo "Deploying web app [$WEB_APP_NAME] with zip file [$ZIPFILE]..."