Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Investigate CI analytics pollution#12671

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
joe4dev wants to merge18 commits intomain
base:main
Choose a base branch
Loading
frominvestigate-ci-analytics-pollution

Conversation

joe4dev
Copy link
Member

DO NOT MERGE, only for internal testing!

Motivation

Investigate why we receive analytic events in CI.

Changes

Introduce new test counter and raise intentional exception upon trying to publish CI metrics

@joe4devjoe4dev added this to thePlayground milestoneMay 28, 2025
@github-actionsGitHub Actions
Copy link

github-actionsbot commentedMay 28, 2025•
edited
Loading

Test Results - Preflight, Unit

21 613 tests  ±0   19 955 ✅  - 3   6m 10s ⏱️ -14s
     1 suites ±0    1 658 💤 +3 
     1 files   ±0        0 ❌ ±0 

Results for commitfca8e3e. ± Comparison against base commit02ad74e.

This pull requestskips 3 tests.
tests.unit.cli.test_cli ‑ test_publish_analytics_event_on_command_invocation[--debug config show --format plain-localstack config show-expected_params2]tests.unit.cli.test_cli ‑ test_publish_analytics_event_on_command_invocation[config show-localstack config show-expected_params1]tests.unit.cli.test_cli ‑ test_publish_analytics_event_on_command_invocation[stop-localstack stop-expected_params0]

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedMay 28, 2025•
edited
Loading

Test Results - Alternative Providers

987 tests   589 ✅  30m 15s ⏱️
  4 suites  398 💤
  4 files      0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedMay 28, 2025•
edited
Loading

LocalStack Community integration with Pro

    2 files      2 suites   1h 44m 10s ⏱️
4 873 tests 4 096 ✅ 777 💤 0 ❌
4 875 runs  4 096 ✅ 779 💤 0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedMay 28, 2025•
edited
Loading

S3 Image Test Results (AMD64 / ARM64)

  2 files    2 suites   8m 11s ⏱️
495 tests 445 ✅  50 💤 0 ❌
990 runs  890 ✅ 100 💤 0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@joe4devjoe4devforce-pushed theinvestigate-ci-analytics-pollution branch from673729a to78df7eeCompareJune 2, 2025 14:41
@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 2, 2025•
edited
Loading

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 7s ⏱️ ±0s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commitfca8e3e. ± Comparison against base commit02ad74e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 2, 2025•
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 20m 43s ⏱️
5 230 tests 4 300 ✅ 929 💤 1 ❌
5 236 runs  4 300 ✅ 935 💤 1 ❌

For more details on these failures, seethis check.

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@joe4devjoe4devforce-pushed theinvestigate-ci-analytics-pollution branch fromcf5ba72 to38f72f5CompareJune 3, 2025 15:04
@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 3, 2025•
edited
Loading

Test Results (MA/MR) - Preflight, Unit

21 613 tests   19 955 ✅  6m 1s ⏱️
     1 suites   1 658 💤
     1 files         0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 3, 2025•
edited
Loading

Test Results (amd64, MA/MR) - Acceptance

7 tests   5 ✅  3m 8s ⏱️
1 suites  2 💤
1 files    0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 3, 2025•
edited
Loading

Test Results (MA/MR) - Alternative Providers

987 tests   589 ✅  30m 22s ⏱️
  4 suites  398 💤
  4 files      0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

@github-actionsGitHub Actions
Copy link

github-actionsbot commentedJun 3, 2025•
edited
Loading

Test Results (amd64, MA/MR) - Integration, Bootstrap

    5 files      5 suites   2h 23m 3s ⏱️
5 230 tests 4 301 ✅ 929 💤 0 ❌
5 236 runs  4 301 ✅ 935 💤 0 ❌

Results for commitfca8e3e.

♻️ This comment has been updated with latest results.

# Conflicts:#localstack-core/localstack/services/sns/analytics.py#localstack-core/localstack/utils/analytics/metrics.py
The following bootstrap tests fail due to the missing `LOCALSTACK_` prefix required for CLI tests:https://app.circleci.com/pipelines/github/localstack/localstack/32859/workflows/4f30e6c8-ed63-4ef5-8705-fd9aa2bb7777/jobs/297211* tests/bootstrap/test_container_configurators.py::test_container_configurator_no_deprecation_warning_on_prefix* tests/bootstrap/test_container_configurators.py::test_container_configurator_no_deprecation_warning_for_ci_env_var
@joe4devjoe4devforce-pushed theinvestigate-ci-analytics-pollution branch from10cd525 toed883c2CompareJune 13, 2025 13:04
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@bentskubentskuAwaiting requested review from bentskubentsku will be requested when the pull request is marked ready for reviewbentsku is a code owner

@baermatbaermatAwaiting requested review from baermatbaermat will be requested when the pull request is marked ready for reviewbaermat is a code owner

@simonrwsimonrwAwaiting requested review from simonrwsimonrw will be requested when the pull request is marked ready for reviewsimonrw is a code owner

@thrauthrauAwaiting requested review from thrauthrau will be requested when the pull request is marked ready for reviewthrau is a code owner

@alexrashedalexrashedAwaiting requested review from alexrashedalexrashed will be requested when the pull request is marked ready for reviewalexrashed is a code owner

@dfangldfanglAwaiting requested review from dfangldfangl will be requested when the pull request is marked ready for reviewdfangl is a code owner

@dominikschubertdominikschubertAwaiting requested review from dominikschubertdominikschubert will be requested when the pull request is marked ready for reviewdominikschubert is a code owner

@silv-iosilv-ioAwaiting requested review from silv-iosilv-io will be requested when the pull request is marked ready for reviewsilv-io is a code owner

@k-a-ilk-a-ilAwaiting requested review from k-a-ilk-a-il will be requested when the pull request is marked ready for reviewk-a-il is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Playground
Development

Successfully merging this pull request may close these issues.

1 participant
@joe4dev
[8]ページ先頭
©2009-2025 Movatter.jp