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

Dolt upgrade seems to have lost some settings #3887

Closed
n0mn0m opened this issue Jul 21, 2022 · 4 comments
Closed

Dolt upgrade seems to have lost some settings #3887

n0mn0m opened this issue Jul 21, 2022 · 4 comments

Comments

@n0mn0m
Copy link

n0mn0m commented Jul 21, 2022

Upgrade from dolt 0.40.5 to 0.40.19 last night seems to have created a couple interesting side effects.

Default branch per database settings where lost. We set the default branch using:

SET PERSIST <db>_default_branch='dev'

Based on the docs I thought this would survive a restart/upgrade, but this morning the variable appeared unset and connections were operating against main by default.

Additionally we have a role setup with SELECT and EXECUTE:

CREATE ROLE foo_role;
GRANT SELECT, EXECUTE on *.* TO foo_role;
FLUSH PRIVILEGES;
GRANT foo_role TO <user>@'%';
FLUSH PRIVILEGES;

Previously users could run SELECT and CALL statements/procedures. Today users were receiving the error [HY000][1105] command denied to user .

@druvv
Copy link
Contributor

druvv commented Jul 21, 2022

Thanks for cutting the issue @n0mn0m, we'll investigate today and provide an update.

@druvv
Copy link
Contributor

druvv commented Jul 21, 2022

Just noticed that there are two bugs here. Just made #3890 to track the second bug about the privileges being dropped.

@druvv
Copy link
Contributor

druvv commented Jul 22, 2022

Hey @n0mn0m, setting that variable in Dolt versions 0.40.5 and 0.40.19 does not persist after restarting the sql-server. On restart, the following error is thrown:

error: failed to load persisted global variables: key: 'repro_default_branch'; Unknown system variable 'repro_default_branch'
Starting server with Config HP="localhost:3306"|T="28800000"|R="false"|L="info"

So you definitely we're using the right commands, the functionality is broken for default_branch. We'll make a fix for this tomorrow.

@druvv
Copy link
Contributor

druvv commented Jul 22, 2022

Getting started on this now

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

No branches or pull requests

2 participants