Skip to content

Conversation

cvxluo
Copy link
Contributor

@cvxluo cvxluo commented Aug 28, 2025

Forgot to add timestamp_ms to the issue_platform event schema in #98322, will likely need to go in after getsentry/snuba#7373.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 28, 2025
Copy link

codecov bot commented Aug 28, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
27555 1 27554 597
View the top 1 failed test(s) by shortest run time
tests.sentry.replays.endpoints.test_organization_replay_events_meta.OrganizationEventsMetaTest::test_rage_clicks
Stack Traces | 5.14s run time
#x1B[1m#x1B[.../replays/endpoints/test_organization_replay_events_meta.py#x1B[0m:131: in test_rage_clicks
    assert sorted(response.data["data"], key=lambda v: v["id"]) == expected
#x1B[1m#x1B[31mE   AssertionError: assert [{'error.type...d': 189, ...}] == [{'error.type...d': 189, ...}]#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     At index 0 diff: {'issue.id': 189, 'error.type': '', 'level': 'error', 'timestamp': '2025-09-02T23:10:35+00:00', 'id': 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa', 'title': 'Rage Click', 'project.name': 'bar', 'issue': 'BAR-1'} != {'error.type': '', 'id': 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa', 'issue.id': 189, 'issue': 'BAR-1', 'level': 'error', 'project.name': 'bar', 'timestamp': '2025-09-02T23:10:35.123000+00:00', 'title': 'Rage Click'}#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     Full diff:#x1B[0m
#x1B[1m#x1B[31mE       [#x1B[0m
#x1B[1m#x1B[31mE           {#x1B[0m
#x1B[1m#x1B[31mE               'error.type': '',#x1B[0m
#x1B[1m#x1B[31mE               'id': 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa',#x1B[0m
#x1B[1m#x1B[31mE               'issue': 'BAR-1',#x1B[0m
#x1B[1m#x1B[31mE               'issue.id': 189,#x1B[0m
#x1B[1m#x1B[31mE               'level': 'error',#x1B[0m
#x1B[1m#x1B[31mE               'project.name': 'bar',#x1B[0m
#x1B[1m#x1B[31mE     -         'timestamp': '2025-09-02T23:10:35.123000+00:00',#x1B[0m
#x1B[1m#x1B[31mE     ?                                          -------#x1B[0m
#x1B[1m#x1B[31mE     +         'timestamp': '2025-09-02T23:10:35+00:00',#x1B[0m
#x1B[1m#x1B[31mE               'title': 'Rage Click',#x1B[0m
#x1B[1m#x1B[31mE           },#x1B[0m
#x1B[1m#x1B[31mE       ]#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@cvxluo cvxluo marked this pull request as ready for review September 2, 2025 23:08
@cvxluo cvxluo requested a review from a team as a code owner September 2, 2025 23:08
@cvxluo cvxluo merged commit 032370d into master Sep 2, 2025
64 checks passed
@cvxluo cvxluo deleted the cvxluo/add-timestamp_ms-snuba-event-columns branch September 2, 2025 23:54
andrewshie-sentry pushed a commit that referenced this pull request Sep 9, 2025
…umns (#98484)

Forgot to add `timestamp_ms` to the issue_platform event schema in
#98322, will likely need to go
in after getsentry/snuba#7373.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants