Skip to content

Migrate to use swxsoc from sdc_aws_utils - #30

Merged
Alrobbertz merged 2 commits into
swxsoc:mainfrom
Alrobbertz:aws_utils_migration
Aug 5, 2026
Merged

Migrate to use swxsoc from sdc_aws_utils#30
Alrobbertz merged 2 commits into
swxsoc:mainfrom
Alrobbertz:aws_utils_migration

Conversation

@Alrobbertz

@Alrobbertz Alrobbertz commented Jul 30, 2026

Copy link
Copy Markdown
Member
  • Migrate to swxsoc library: Replace deprecated sdc_aws_utils dependency with swxsoc across imports, function calls, and tests to consolidate AWS/utility functionality into the core SWxSOC library.
  • Add AI agent documentation: Create AGENTS.md to guide AI coding agents on project structure, build/test commands, testing conventions, and executor function patterns for improved developer productivity.
  • Add linting configuration: Include ruff.toml with project-specific lint rules (ignoring EXE002, BLE001, TRY201, etc.) to maintain code style consistency.

@Alrobbertz
Alrobbertz marked this pull request as ready for review August 5, 2026 14:50
Copilot AI lite review requested due to automatic review settings August 5, 2026 14:50

Copilot AI 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.

Pull request overview

This PR migrates the Lambda executor away from sdc_aws_utils and onto the consolidated swxsoc package APIs, aligning timeseries recording, S3 upload, and Grafana annotation utilities with the swxsoc ecosystem.

Changes:

  • Replace sdc_aws_utils imports/usages in the executor with swxsoc equivalents (record_timeseries, push_science_file, create_annotation, parse_science_filename, reconfigure).
  • Update unit tests’ monkeypatch targets to match the new executor module-level imports.
  • Add a ruff.toml and switch dev/runtime requirements from sdc_aws_utils to swxsoc.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
ruff.toml Adds Ruff lint ignore configuration for the repo.
requirements.dev.txt Replaces sdc_aws_utils with swxsoc in dev dependency set.
lambda_function/requirements.txt Replaces sdc_aws_utils with swxsoc in Lambda runtime dependencies.
lambda_function/src/executor/executor.py Migrates executor logic to use swxsoc modules/functions instead of sdc_aws_utils.
lambda_function/tests/test_executor.py Updates monkeypatch paths to match new executor import locations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lambda_function/requirements.txt
Comment thread requirements.dev.txt
@Alrobbertz
Alrobbertz merged commit dec1bfe into swxsoc:main Aug 5, 2026
2 of 3 checks passed
@Alrobbertz
Alrobbertz deleted the aws_utils_migration branch August 5, 2026 15:37
@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@c35ef1e). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage        ?   62.69%           
=======================================
  Files           ?        2           
  Lines           ?      252           
  Branches        ?        0           
=======================================
  Hits            ?      158           
  Misses          ?       94           
  Partials        ?        0           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants