-
Notifications
You must be signed in to change notification settings - Fork 5
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
RFC 1001: Explicit installation method in edgedb server install
#9
Comments
Can we default to |
I also think that defaulting to |
Okay. That's fine for linux if we don't fallback to docker or tarball implicitly, right? For windows both docker or WSL(2) aren't good defaults, right? For macos: for me it's unclear if we want |
Yes, for Windows I would always make
Definitely not brew. I don't even think we should wrap brew or macports with our tooling. We can make |
There are no native packages for Windows, so erroring out with
Definitely. |
Basically it looks like we want to make |
That's not what I meant. I meant that the default IOW, I agree with you. |
And another related question: if there is an already installed edgedb-server in non-default method, say docker,
Same applies when method is applied explicitly. Should we allow a parallel install by default or have any kind of |
I think we should not allow parallel install of the same major version regardless of the installation method, so 2. |
So |
Yes. |
Done. |
Or @elprans do you need to update RFC? |
Please open an issue in the rfc repo with a reminder. Just one issue with pointers to relevant implementation discussions is OK. Thanks! |
This is the RFC repo :) So just reopening. |
Proposal
edgedb install --method=package
oredgedb install --method=docker
for installation in non-interactive modeWe can simplify parameters like
Motivation
If we choose "the best available method" automatically, we can have unexpected switch between method in certain situations:
sudo
when runningedgedb server
Even if we don't consider
edgedb server install
be used in production, there still a point of confusion:Docs and Tutorials
We're going to recommend `edgedb server install --interactive'
The text was updated successfully, but these errors were encountered: