Skip to content
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

ci: call lightwalletd wallet queries #4168

Closed
conradoplg opened this issue Apr 21, 2022 · 1 comment · Fixed by #4267 or #4453
Closed

ci: call lightwalletd wallet queries #4168

conradoplg opened this issue Apr 21, 2022 · 1 comment · Fixed by #4267 or #4453
Assignees
@gustavovalverde @oxarbitrage
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-enhancement Category: This is an improvement lightwalletd any work associated with lightwalletd

Comments

@conradoplg
Copy link
Collaborator

conradoplg commented Apr 21, 2022 •
edited by teor2345
Loading

Motivation

We need to call the lightwalletd_wallet_grpc_tests tests from #3655 in CI, using the lightwalletd cached state from #4165.

(PR #4267 already added the tests from #3512.)
@conradoplg conradoplg added A-devops Area: Pipelines, CI/CD and Dockerfiles C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage P-Medium ⚡ lightwalletd any work associated with lightwalletd labels Apr 21, 2022
@oxarbitrage oxarbitrage self-assigned this Apr 29, 2022
@gustavovalverde gustavovalverde mentioned this issue May 2, 2022
Merged
2 tasks
@ftm1000 ftm1000 added P-High 🔥 and removed S-needs-triage Status: A bug report needs triage P-Medium ⚡ labels May 3, 2022
@mergify mergify bot closed this as completed in #4267 May 6, 2022
@teor2345 teor2345 reopened this May 18, 2022
@teor2345
Copy link
Contributor

@gustavovalverde I re-opened this ticket, because we still need to call the lightwalletd_wallet_grpc_tests in CI.

@oxarbitrage oxarbitrage mentioned this issue May 18, 2022
Merged
3 tasks
@gustavovalverde gustavovalverde self-assigned this May 20, 2022
@oxarbitrage oxarbitrage mentioned this issue May 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gustavovalverde gustavovalverde

@oxarbitrage oxarbitrage

Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles C-enhancement Category: This is an improvement lightwalletd any work associated with lightwalletd
Projects
None yet
Milestone
No milestone
5 participants
@conradoplg @teor2345 @gustavovalverde @oxarbitrage @ftm1000