Skip to content

Step Functions: Add Telemetry for SFN_MOCK_CONFIG Usage #12584

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 6, 2025

Conversation

MEPalma
Copy link
Contributor

@MEPalma MEPalma commented May 6, 2025

Motivation

The mocked service integrations feature for Step Functions testing was recently introduced, but no telemetry was added to track its usage. These changes address that gap by recording usage of the associated environment variable, SFN_MOCK_CONFIG, without capturing the file path it points to.

Changes

  • added SFN_MOCK_CONFIG to analytics' PRESENCE_ENV_VAR list

@MEPalma MEPalma added this to the 4.4 milestone May 6, 2025
@MEPalma MEPalma requested a review from joe4dev May 6, 2025 09:48
@MEPalma MEPalma self-assigned this May 6, 2025
@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label May 6, 2025
@MEPalma MEPalma requested a review from thrau as a code owner May 6, 2025 09:48
Copy link

github-actions bot commented May 6, 2025

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   8m 35s ⏱️
488 tests 438 ✅  50 💤 0 ❌
976 runs  876 ✅ 100 💤 0 ❌

Results for commit 31ca86d.

Copy link

github-actions bot commented May 6, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 41m 47s ⏱️ - 1m 17s
4 399 tests ±0  4 037 ✅ ±0  362 💤 ±0  0 ❌ ±0 
4 401 runs  ±0  4 037 ✅ ±0  364 💤 ±0  0 ❌ ±0 

Results for commit 31ca86d. ± Comparison against base commit 7a2f96f.

@MEPalma MEPalma merged commit 1b1d7cd into master May 6, 2025
42 checks passed
@MEPalma MEPalma deleted the MEP-SFN-mock_config_flag branch May 6, 2025 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants