851826 - Add phpMyAdmin to TBPL's Vagrant/puppet environment (local development only)

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Ed Morley [:emorley]]

Several times when developing locally it would have been really useful to actually see what was in the DB inside the Vagrant VM, without having to |vagrant ssh| then play the remember the SQL syntax game.

We should add phpMyAdmin to the puppet script used by Vagrant, to simplify this.

Note: This will only be for the local development environment config, not production (before anyone from opsec has an aneurysm!).

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

Comment 2

[Mauro Doglio [:mdoglio]]

Comment on attachment 725792 [details] [diff] [review]
Patch v1

Review of attachment 725792 [details] [diff] [review]:
-----------------------------------------------------------------

::: puppet/files/etc_apache2_sites-available/tbpl
@@ +3,5 @@
>      ServerName tbpl
>      # FIXME customize this
>      ServerAdmin [e-mail]
>      DocumentRoot /var/www/tbpl
> +    Alias /phpMyAdmin /usr/share/phpmyadmin

This looks like a duplicate line

Comment 3

[Ed Morley [:emorley]]

(In reply to mdoglio from comment #2)
> This looks like a duplicate line

Apache's Alias is case sensitive, so this was to add support for both tbpl/phpmyadmin and tbpl/phpMyAdmin

That said, I think we can perhaps ditch the capitalised version since realistically no one will type it anyway.

Comment 4

[Ed Morley [:emorley]]

Thank you for the review :-)

I removed the mixed case alias and adjusted the readme to reference the lowercase URL, and landed as:
https://hg.mozilla.org/webtools/tbpl/rev/1d630ec305e7