919270 - Fix up Vagrant for balrog

Status: RESOLVED FIXED

(Release Engineering Graveyard :: Applications: Balrog (backend), defect, P3)

Description

[Nick Thomas [:nthomas] (UTC+12)]

Strongly suspect 
 https://github.com/mozilla/balrog/commit/b26421f4f7594c728b5fcf2c96744a294aa17f5f
will have broken it.

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

Also, at some point I upgraded to Vagrant 1.2.4 (older version unknown) and now get this: 

There were warnings and/or errors while loading your Vagrantfile.
Your Vagrantfile was written for an earlier version of Vagrant,
and while Vagrant does the best it can to remain backwards
compatible, there are some cases where things have changed
significantly enough to warrant a message. These messages are
shown below.

Warnings:
* The 'v-root' shared folders have been renamed to 'vagrant-root'.
Assuming you meant 'vagrant-root'...
-----

It's a simple fix:
diff --git a/Vagrantfile b/Vagrantfile
index e1cf1c3..c32ab11 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -12 +12 @@ Vagrant::Config.run do |config|
-    config.vm.share_folder("v-root", MOUNT_POINT, ".")
+    config.vm.share_folder("vagrant-root", MOUNT_POINT, ".")

Comment 2

[[e-mail] (:bhearsum)]

mass priority change after component move

Comment 3

[Nick Thomas [:nthomas] (UTC+12)]

Torched my vagrant vm the other day, have a patch in hand from setting up a new one.

Comment 4

[Nick Thomas [:nthomas] (UTC+12)]

Attachment: Assorted fixes

This WFM with Vagrant 1.5.2 on Mac 10.9.2 (both current latest), with VirtualBox 4.2.6 (which is stale).

* fixes reference to deprecated v-root config variable
* adds balrog.ini to .gitignore, since the ini files live in code dir now
* apache changes
 * log to dirs matching domains used elsewhere
 * rotate logs every 24 hours, rather than every hour
 * serve jss/css/images using Apache to avoid respawning a wsgi for each
* update templates for the two ini files, and put them next to source instead of /etc/aus. Moves any file already present to <foo>.puppet-backup

Comment 5

[[e-mail] (:bhearsum)]

Comment on attachment 8402459 [details] [diff] [review]
Assorted fixes

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

Looks good to me!

Comment 6

[[github robot]]

Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/18c9b71ea4ab530000f8295969e68951e2f91533
Bug 919270, update Vagrant config for code changes, r=bhearsum

Comment 7

[Nick Thomas [:nthomas] (UTC+12)]

Comment on attachment 8402459 [details] [diff] [review]
Assorted fixes

https://github.com/mozilla/balrog/commit/18c9b71ea4ab530000f8295969e68951e2f91533

Comment 8

[[github robot]]

Commit pushed to master at https://github.com/mozilla/balrog

https://github.com/mozilla/balrog/commit/37ec9bd9f3e991c9758b5bd7c68df99dd0e527b5
Bug 919270, followup fix for cef logging

Comment 9

[Nick Thomas [:nthomas] (UTC+12)]

The followup ensures /var/log/auscef.log is writeable by Apache.