Skip to content

Update PostgreSQL user permission in Usage Guide #593

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

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

mcserep
Copy link
Collaborator

@mcserep mcserep commented Feb 23, 2023

CodeCompass does not require SUPERUSER privileges in PostgreSQL, only CREATEDB.

Also updating documentation links to more up to date version (9.5 -> 12).

@intjftw
Copy link
Collaborator

intjftw commented Feb 23, 2023

LGTM!
Related question: should we also update the Postgres version in the CI and the tarball as well? It's kind of inconsistent. For Ubuntu 18.04 we install v. 10 (I know we'll stop supporting it soon but still), and also we build v.10.x in the tarball.

@mcserep
Copy link
Collaborator Author

mcserep commented Feb 23, 2023

LGTM! Related question: should we also update the Postgres version in the CI and the tarball as well? It's kind of inconsistent. For Ubuntu 18.04 we install v. 10 (I know we'll stop supporting it soon but still), and also we build v.10.x in the tarball.

I don't think we should spend any effort on updating the Ubuntu 18.04 scripts, as we will drop support for it soon anyways (see #580). Also, in the default package repository of Ubuntu 18.04, PostgreSQL version 10 is available, 12 is not.

@mcserep mcserep force-pushed the postgresql-no-superuser branch from 9064453 to ae74ab9 Compare February 23, 2023 17:11
@mcserep mcserep merged commit a4cad1a into Ericsson:master Mar 10, 2023
@mcserep mcserep deleted the postgresql-no-superuser branch March 10, 2023 08:52
@mcserep mcserep added this to the Release Gershwin milestone Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants