Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dustin, Assigned: dustin)

Tracking

Details

Attachments

(4 attachments)

This is a roll-up of a bunch of small(ish) changes:

* make inventory pattern a config knob
* inventory sync crontask needs a newline
* use mysql for the frontend, not sqlite
* fix 302 redirects generated by the web.py code - they are adding the full path
(In reply to Dustin J. Mitchell [:dustin] from comment #0)
> * fix 302 redirects generated by the web.py code - they are adding the full
> path

Which redirects?
The redirects I've had problems with are those that add a trailing slash.  They're inserting the full filesystem path of the CGI into the URL.
Ah, that behavior comes from templeton, FYI.
Assignee: server-ops → dustin
Component: Server Operations → Server Operations: RelEng
QA Contact: jdow → arich
OK, mock has me stumped on #3.  It can't find the mozilla-python27 dependency for mozilla-python27-mysql-python.

Will resume hacking on this on Monday.
Created attachment 671090 [details] [diff] [review]
didnt-work.patch

So, this was my first attempt, trying to get MySQL-python installed into mozilla-python27.  Build dependencies didn't work right.
Created attachment 671091 [details] [diff] [review]
bug800986.patch

This only fixes point 2, using mysql.  But I'd like to get that in soon.
Attachment #671091 - Flags: review?(jwatkins)
Comment on attachment 671091 [details] [diff] [review]
bug800986.patch


> # temporary until mysql is running

just remember to update the comment otherwise g2t
Attachment #671091 - Flags: review?(jwatkins) → review+
Comment on attachment 671091 [details] [diff] [review]
bug800986.patch

comment removed - thanks
Attachment #671091 - Flags: checked-in+
* make inventory pattern a config knob
* inventory sync crontask needs a newline
* fix 302 redirects generated by the web.py code - they are adding the full path
* move config.ini to /opt/bmm/config.ini
* add automatic DB reconnection support to bmm (modeled on code in Buildbot)
* inventory sync should use limit=100 to get data from inventory faster (with default, inventory fetch takes >300s)
* change owner/group on /var/lib/tftpboot/pxelinux.cfg to allow apache to write
* make inventory pattern a config knob
  (bmm DONE, puppet DONE)

* inventory sync crontask needs a newline
  (puppet DONE)

* fix 302 redirects generated by the web.py code - they are adding the full path
  https://github.com/markrcote/templeton/pull/1
  and we can suffer in silence until that's fixed..

* move config.ini to /opt/bmm/config.ini
  (puppet DONE)

* add automatic DB reconnection support to bmm (modeled on code in Buildbot)
  SKIPPED - better to just not keep a DB conn open while polling inventory

* inventory sync should use limit=100 to get data from inventory faster
  (bmm DONE)

* change owner/group on /var/lib/tftpboot/pxelinux.cfg to allow apache to write
  (puppet DONE)

That should do it.  Patches in a few.
Created attachment 671554 [details] [diff] [review]
bug800986-bmm.patch
Attachment #671554 - Flags: review?(ted.mielczarek)
Created attachment 671556 [details] [diff] [review]
bug800986-puppet.patch
Attachment #671556 - Flags: review?(jwatkins)
Attachment #671556 - Flags: review?(jwatkins) → review+
Ted, I just merged my repo into http://hg.mozilla.org/build/bmm including these changes, so hopefully they're OK.  Also, we have a 4real repo now!
Attachment #671556 - Flags: checked-in+
Comment on attachment 671554 [details] [diff] [review]
bug800986-bmm.patch

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

This looks fine. Hooray, a real repo!
Attachment #671554 - Flags: review?(ted.mielczarek) → review+
Sweet.  They were already landed, but I bumped the version to 0.1.2 and will install that with puppet in a moment, with dividehex's r+.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Component: Server Operations: RelEng → RelOps
Product: mozilla.org → Infrastructure & Operations
You need to log in before you can comment on or make changes to this bug.