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.

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

Separate State dir for cloud/e2e #6291

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Separate State dir for cloud/e2e #6291

wants to merge 6 commits into from

Conversation

himadrisingh
Copy link
Contributor

No description provided.

@himadrisingh himadrisingh self-assigned this Dec 17, 2024
Copy link
Contributor

@ericpgreen2 ericpgreen2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm getting this out-of-memory error with ./rill devtool start e2e --reset, but not with rill devtool start e2e --reset:
image

The code generally LGTM, but given it's Go, CC'ing @begelundmuller for his eyes on this too.

.gitignore Outdated Show resolved Hide resolved
cli/cmd/devtool/data/cloud-deps.docker-compose.yml Outdated Show resolved Hide resolved
@begelundmuller
Copy link
Contributor

I'm getting this out-of-memory error with ./rill devtool start e2e --reset, but not with rill devtool start e2e --reset

Weird, could it be the state directory is not created? A Google search indicates it's from SQLite and can be due to a number of reasons about not being able to create/access the file. E.g. see here: https://community.grafana.com/t/problems-with-error-checking-db-error-checking-for-valid-sqlite-file-unable-to-open-database-file-out-of-memory-14/92025

Copy link
Contributor Author

@himadrisingh himadrisingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename RILL_DEVTOOL_STATE_DIRECTORY

cli/cmd/devtool/data/cloud-deps.docker-compose.yml Outdated Show resolved Hide resolved
cli/cmd/devtool/data/cloud-deps.docker-compose.yml Outdated Show resolved Hide resolved
cli/cmd/devtool/start.go Outdated Show resolved Hide resolved
cli/cmd/devtool/start.go Outdated Show resolved Hide resolved
cli/cmd/devtool/start.go Outdated Show resolved Hide resolved
@ericpgreen2
Copy link
Contributor

Weird, could it be the state directory is not created? A Google search indicates it's from SQLite and can be due to a number of reasons about not being able to create/access the file. E.g. see here: https://community.grafana.com/t/problems-with-error-checking-db-error-checking-for-valid-sqlite-file-unable-to-open-database-file-out-of-memory-14/92025

Yeah, that's possible. Working for me now!

@ericpgreen2
Copy link
Contributor

This is tangential and for another PR, but I just discovered that we should also be able to upload .env files for the e2e environment. See:
image

cli/cmd/devtool/start.go Outdated Show resolved Hide resolved
cli/cmd/devtool/start.go Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants