Skip to content

Halyna R.#5

Open
halyna1995 wants to merge 1 commit into
HackYourAssignment:mainfrom
halyna1995:week6/halyna1995
Open

Halyna R.#5
halyna1995 wants to merge 1 commit into
HackYourAssignment:mainfrom
halyna1995:week6/halyna1995

Conversation

@halyna1995

Copy link
Copy Markdown

Complete week 6 Azure assignment

@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: 85 / 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)
  ✓ PASS  pipeline.py uses %s placeholders in execute() (parameterised SQL)
  ✓ PASS  Level 5: idempotent upsert (15/15 pts)
  ✗ FAIL  no mention of sslmode=require in the repo — Azure Postgres rejects connections without it
  ✓ PASS  pipeline.py uses the azure-storage-blob SDK (BlobServiceClient)
  ✓ PASS  Level 6: connection + SDK (5/10 pts)
  ✗ FAIL  AI_ASSIST.md has section headers but is not filled in (1148 chars, 0 TODO line(s); target 1800+ chars, 0 TODOs)
  ✓ PASS  Level 7: AI report (0/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 (73508 bytes)
  ✓ PASS  Level 9: execution screenshot (10/10 pts)
  ✓ PASS  .gitignore correctly excludes __pycache__/, *.pyc, and .env

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

Comment thread src/pipeline.py
logger = logging.getLogger(__name__)
logging.getLogger("azure").setLevel(logging.WARNING)

# TASK 3 hint: quiet the Azure SDK so its DEBUG output does not drown your own

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The Postgres connection should include or enforce sslmode=require. Azure Postgres often rejects non-SSL connections, and the assignment/grader expects this to be handled

Comment thread src/pipeline.py

postqres_url = os.environ.get("POSTGRES_URL")
storage_connection = os.environ.get("AZURE_STORAGE_CONNECTION_STRING")
source_name = os.environ.get("SOURCE_NAME", "weather")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This currently writes to a weather table in the default schema. The assignment required creating a personal schema such as dev_halyna1995, setting search_path, and writing to weather_readings there

Comment thread README.md
```bash
az containerapp job execution list --name halyna1995-job --resource-group rg-hyf-data --output table

az storage blob list --account-name hyfstoragedev --container-name raw --auth-mode login --output table

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The bash code block under verification is not closed, so the rest of the README renders as code

@AgneseG AgneseG left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Core pipeline exists, but misses important assignment requirements: no personal schema/search_path, wrong table name, no sslmode=require, README formatting issue.

For next submissinos, pls write a description of the PR too, thanks!

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