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

Set up optional redis container #105

Closed
dvvanessastoiber opened this issue Oct 23, 2024 · 0 comments · Fixed by datavisyn/visyn_core#610 · May be fixed by #110
Closed

Set up optional redis container #105

dvvanessastoiber opened this issue Oct 23, 2024 · 0 comments · Fixed by datavisyn/visyn_core#610 · May be fixed by #110
Assignees
@dvviktordelev
Labels
type: feature New feature or request

Comments

@dvvanessastoiber
Copy link
Contributor

dvvanessastoiber commented Oct 23, 2024 •
edited by thinkh
Loading

We are currently starting a postgres container in the build-node-python workflow (https://github.com/datavisyn/github-workflows/blob/main/.github/workflows/build-node-python.yml#L203-L217). For Ordino (superset in particular), an additional redis container is needed. Please add this container - it should be disabled by default (it should be possible to enable it here). Please make sure that the container IDs etc. do not conflict with each other when running multiple tests in parallel on the self-hosted runner.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dvviktordelev dvviktordelev

Labels
type: feature New feature or request
Projects
None yet
Milestone
No milestone
3 participants
@thinkh @dvvanessastoiber @dvviktordelev