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

New profile is allowed to use an already existing database (of another profile) #168

Closed
aiida-bot opened this issue Mar 4, 2016 · 4 comments
Assignees
Labels
topic/repository type/duplicate close issue when applying duplicate label

Comments

@aiida-bot
Copy link

Originally reported by: Spyros Zoupanos (Bitbucket: szoup, GitHub: szoupanos)


Please also see if an already existing repository is possible to be used.

In general each profile should have its own database and repository.

Best,
Spyros


@aiida-bot
Copy link
Author

Original comment by Giovanni Pizzi (Bitbucket: pizzi, GitHub: giovannipizzi):


I think this in general should not happen, but I don't know if it's easy to avoid that two people use the same DB or repo (and probably it's ok to let them do it). Maybe the only check can be with existing DBs and repos in the same config.json. I suggest to change verdi install (and the docs) along the line of what I did (for basically the same reason) earlier today for the repo: now, the default repo name suggested in the installation phase is not only ~/.aiida/repository but ~/.aiida/repository-{PROFILENAME}.

I suggest to change the default database name from aiidadb to aiidadb_{USERNAME}_{PROFILENAME} (in all three supported cases with django) [and maybe the default DB username to the linux username]. Underscores should be supported by all backends, right? Also the docs on how to create a new DB should be updated so that users start from the very beginning creating a DB with the right name (e.g. aiidadb_{USERNAME}_default if they don't provide a profile name).

@giovannipizzi
Copy link
Member

I think the default name changed (but probably not as suggested here - @DropD can you confirm?

Also, the new work in the object store will take care of making sure that the file storage and the DB match.

@giovannipizzi giovannipizzi added the type/duplicate close issue when applying duplicate label label Jan 19, 2017
@giovannipizzi
Copy link
Member

Actually, this is a duplicate of #182 - after checking/fixing that the default name contains the linux username, I would close this issue.

@DropD
Copy link
Contributor

DropD commented Feb 27, 2017

default name contains getpass.getuser()

@DropD DropD closed this as completed Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/repository type/duplicate close issue when applying duplicate label
Projects
None yet
Development

No branches or pull requests

5 participants