Skip to content

MohammedA#1

Open
mohammedalfakih-dev wants to merge 1 commit into
HackYourAssignment:mainfrom
mohammedalfakih-dev:week6/mohammed
Open

MohammedA#1
mohammedalfakih-dev wants to merge 1 commit into
HackYourAssignment:mainfrom
mohammedalfakih-dev:week6/mohammed

Conversation

@mohammedalfakih-dev

Copy link
Copy Markdown

Summary

  • Implemented the Week 6 Azure weather pipeline
  • Added Blob Storage upload for raw JSON output
  • Added Postgres schema/table creation and row upsert
  • Built and pushed the Docker image to ACR
  • Deployed and ran the pipeline as an Azure Container App Job
  • Added execution-history verification and AI_ASSIST report

@lassebenni lassebenni closed this Jun 10, 2026
@lassebenni lassebenni reopened this Jun 10, 2026
@github-actions

Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 95 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 60
🧪 The auto grade is experimental and still being improved

Test Details

=== Week 6 Autograder ===
  ✓ PASS  found Dockerfile
  ✓ PASS  found requirements.txt
  ✓ PASS  found src/pipeline.py
  ✓ PASS  found AI_ASSIST.md
  ✓ PASS  found README.md
  ✓ PASS  found docs/ directory
  ✓ PASS  Level 1: required files (10/10 pts)
  ✓ PASS  requirements.txt pins azure-storage-blob
  ✓ PASS  requirements.txt pins psycopg2-binary
  ✓ PASS  Level 2: pinned dependencies (10/10 pts)
  ✓ PASS  Dockerfile uses a python:3.11 base image
  ✓ PASS  Dockerfile copies requirements before src/ (layer cache stays warm)
  ✓ PASS  Dockerfile has a real CMD instruction
  ✓ PASS  Level 3: Dockerfile (10/10 pts)
  ✓ PASS  pipeline.py reads POSTGRES_URL and AZURE_STORAGE_CONNECTION_STRING from env
  ✓ PASS  pipeline.py uses contextlib.closing() to wrap the Postgres connection
  ✓ PASS  pipeline.py silences the azure SDK logger
  ⚠ WARN  pipeline.py still contains 'raise NotImplementedError' — finish the stubs before submitting
  ✓ PASS  Level 4: pipeline shape (15/15 pts)
  ✓ PASS  pipeline.py uses an upsert (ON CONFLICT ... DO UPDATE)
  ✗ FAIL  pipeline.py does not use %s placeholders for parameterised SQL
  ✓ PASS  Level 5: idempotent upsert (10/15 pts)
  ✓ PASS  connection string includes sslmode=require
  ✓ PASS  pipeline.py uses the azure-storage-blob SDK (BlobServiceClient)
  ✓ PASS  Level 6: connection + SDK (10/10 pts)
  ✓ PASS  AI_ASSIST.md has all three sections and is filled in (2136 chars)
  ✓ PASS  Level 7: AI report (10/10 pts)
  ✓ PASS  README.md has '## Verification' heading and embeds a docs/ image
  ✓ PASS  Level 8: README verification (10/10 pts)
  ✓ PASS  execution-history screenshot present at execution_history.png (21240 bytes)
  ✓ PASS  Level 9: execution screenshot (10/10 pts)
  ✓ PASS  .gitignore correctly excludes __pycache__/, *.pyc, and .env

Score: 95 / 100  (passing: 60)  pass=true

Comment thread AI_ASSIST.md
<!-- Paste the exact prompt you gave an LLM (ChatGPT, Claude, Copilot, etc.). -->

TODO: paste your prompt here.
help me complete the Azure deployment for my weather pipeline. I am already done doing the pipeline to upload raw JSON to Azure Blob Storage and insert weather rows into Azure Postgres and both the local Python run and Docker run were working after pushing my Docker image to Azure Container Registry I tried to create and verify an Azure Container App Job but Azure CLI kept failing with this error:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good that you included the AI usage, but this could be more specific. Next time pls include the exact problem you asked AI about, what answer you used, and what you changed manually afterwards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants