Skip to content

test(coderd): enable dbrollup service for insights tests #12673

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 22, 2024
Merged

test(coderd): enable dbrollup service for insights tests #12673

merged 2 commits into from
Mar 22, 2024

Conversation

mafredri
Copy link
Member

No description provided.

This was referenced Mar 19, 2024
Merged
Merged
Merged
Merged
This was referenced Mar 19, 2024
Merged
Merged
Closed
Merged
@mafredri Graphite App
Copy link
Member Author

mafredri commented Mar 19, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mafredri and the rest of your teammates on Graphite Graphite

This was referenced Mar 19, 2024
Merged
Merged
Merged
@mafredri mafredri force-pushed the mafredri/rewrite-getuseractivityinsights branch from d4fa8af to cedf108 Compare March 19, 2024 16:25
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 77ba7dc to 754fdd5 Compare March 19, 2024 16:25
johnstcn
johnstcn reviewed Mar 20, 2024
View reviewed changes
@mafredri mafredri force-pushed the mafredri/rewrite-getuseractivityinsights branch from cedf108 to 77c3115 Compare March 20, 2024 10:25
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 754fdd5 to 7128953 Compare March 20, 2024 10:25
@mafredri mafredri changed the base branch from mafredri/rewrite-getuseractivityinsights to mafredri/add-dbrollup-service-for-insights March 20, 2024 12:37
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 7128953 to b915c4c Compare March 20, 2024 12:37
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 91fb8e4 to 4913c03 Compare March 20, 2024 13:56
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from b915c4c to a083bc9 Compare March 20, 2024 13:57
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 4913c03 to 32a5d2c Compare March 20, 2024 14:52
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from a083bc9 to 97c9a10 Compare March 20, 2024 14:53
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 32a5d2c to e77ca25 Compare March 20, 2024 16:02
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 97c9a10 to a9d5ab0 Compare March 20, 2024 16:02
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from e77ca25 to 2ef2cc0 Compare March 20, 2024 19:03
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from a9d5ab0 to f2200fb Compare March 20, 2024 19:03
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 2ef2cc0 to ad8679e Compare March 20, 2024 19:18
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from f2200fb to 3101fba Compare March 20, 2024 19:19
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch 3 times, most recently from 4374753 to 7145189 Compare March 20, 2024 19:55
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 3101fba to c9bd6af Compare March 20, 2024 19:55
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 7145189 to c2926ef Compare March 21, 2024 15:06
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from c9bd6af to c08f934 Compare March 21, 2024 15:06
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from c2926ef to 8ccae5f Compare March 21, 2024 17:54
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from c08f934 to 22f614e Compare March 21, 2024 17:54
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from 8ccae5f to c8221f3 Compare March 22, 2024 15:40
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 22f614e to 0f423a2 Compare March 22, 2024 15:40
@mafredri mafredri mentioned this pull request Mar 22, 2024
Merged
@mafredri mafredri changed the title test(coderd): fix insights tests test(coderd): enable dbrollup service for insights tests Mar 22, 2024
@mafredri mafredri force-pushed the mafredri/add-dbrollup-service-for-insights branch from c8221f3 to 2040b87 Compare March 22, 2024 16:33
@mafredri mafredri marked this pull request as ready for review March 22, 2024 16:34
Base automatically changed from mafredri/add-dbrollup-service-for-insights to main March 22, 2024 16:42
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from 0f423a2 to 60d812d Compare March 22, 2024 16:43
@mafredri Graphite App
Copy link
Member Author

mafredri commented Mar 22, 2024 •
edited
Loading

Merge activity

  • Mar 22, 12:43 PM EDT: Graphite rebased this pull request as part of a merge.
  • Mar 22, 12:51 PM EDT: Graphite couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'required', 'test-go (windows-2022)').

@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from fd01a69 to da902a8 Compare March 22, 2024 17:12
@mafredri mafredri force-pushed the mafredri/fix-insights-tests-after-refactor branch from da902a8 to 9c2d085 Compare March 22, 2024 17:14
@mafredri mafredri merged commit b4fd819 into main Mar 22, 2024
@mafredri mafredri deleted the mafredri/fix-insights-tests-after-refactor branch March 22, 2024 18:18
@github-actions github-actions bot locked and limited conversation to collaborators Mar 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johnstcn johnstcn johnstcn approved these changes

@mtojek mtojek mtojek approved these changes

Assignees

@mafredri mafredri

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mafredri @johnstcn @mtojek