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

Update docker setup example #3695

Merged
merged 1 commit into from
Jan 23, 2024
Merged

Conversation

Michal-Leszczynski
Copy link
Collaborator

@Michal-Leszczynski Michal-Leszczynski commented Jan 23, 2024

Previously I wasn't able to follow our docker setup example because of two reasons:

  • too little memory assigned to SM DB
  • incorrect echo behavior when writing config files

Now it should work fine.
Fixes #3672

@Michal-Leszczynski
Copy link
Collaborator Author

@karol-kokoszka Could you take a look and check if fixed example works for you as well?

Copy link
Collaborator

@karol-kokoszka karol-kokoszka left a comment

Choose a reason for hiding this comment

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

nice catch !

It may be that scylla-university uses same examples to setup manager in docket env.

👍 from my side

@Michal-Leszczynski
Copy link
Collaborator Author

From what I see in example used in Scylla University, it actually doesn't use echo -e and it uses older Scylla version, that should work fine with less memory, so I don't think that there is a need to update that.

@Michal-Leszczynski Michal-Leszczynski merged commit d96e5d8 into master Jan 23, 2024
78 of 82 checks passed
@Michal-Leszczynski Michal-Leszczynski deleted the ml/docker-doc-example branch January 23, 2024 21:56
This was referenced Feb 22, 2024
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.

Unable to configure Scylladb-manager and agent [using containers]
2 participants