Let's change the db names to proper reference and also check the possibility of db name collision.

[akhil-naidu]

ledokku/ledokku-bootstrap.sh

Lines 228 to 233 in eb8eaf7

	# We create the required databases
	echo "=> ${YELLOW}Creating databases${END}"
	dokku redis:create ledokku
	dokku redis:link ledokku ledokku
	dokku postgres:create ledokku
	dokku postgres:link ledokku ledokku

Issue Showcase

In the above code, we can see, that the databases created are all named as ledokku, which itself is not an error prone variable assignment => but we can be more explicit in our naming conventions.

My Suggestions

Let's make our database name more explicit and add some logic to check to verify the duplication of databases. It would be something like this:

• Changing Redis db name to: ledokku-redis-db
• Changing Postgress db name to: ledokku-pg-db

and finally, during the installation process, let's add some logic to check whether there is any name collision => If there is a name collision we will prompt the user that we are about to delete the old databases and install new ones.

Reason => In case of installation failure(in the middle), not everyone might be familiar with dokk-cli to troubleshoot things.

---

@pradel @Akirtovskis What do you say? Shall I start working on it by creating a template PR?

[Akirtovskis]

@akhil-naidu Thanks for the idea, looks good to me! 🚀