Closed Bug 847534 Opened 8 years ago Closed 8 years ago

please update treestatus stage to 41fb79d69f1e362ba008fc32d48255fcbc36e8ff

Categories

(Infrastructure & Operations Graveyard :: WebOps: Other, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: catlee, Assigned: cturra)

Details

(Whiteboard: [triaged 20130304][push interrupt])

please update treestatus stage to 41fb79d69f1e362ba008fc32d48255fcbc36e8ff (which is the current tip of master right now)

there may be some DB schema upgrades that need to happen as part of this. let's iron these out before pushing to prod.
:catlee - i just pushed origin/master to stage. for the outstanding db schema changes, can you explain what needs to be run? my guess is something with `python manage-db.py`
Assignee: server-ops-webops → cturra
Whiteboard: [triaged 20130304][push interrupt]
yes, you need to run manage-db.py like this:

python manage-db.py version_control --version 1 mysql://user:pass@dbhost/dbname /path/to/treestatus/schema
python manage-db.py upgrade mysql://user:pass@dbhost/dbname /path/to/treestatus/schema

what's the URL for the staging instance?
(In reply to Chris AtLee [:catlee] from comment #2)
> yes, you need to run manage-db.py like this:
> 
> python manage-db.py version_control --version 1
> mysql://user:pass@dbhost/dbname /path/to/treestatus/schema

we're seeing the following traceback:

[root@genericadm.private.phx1 tree-status]# python manage-db.py version_control --version 1 mysql://treestatus_stage:<REMOVED>@dev-zeus-rw.db.phx1.mozilla.com/treestatus_allizom_org schema
Traceback (most recent call last):
  File "manage-db.py", line 9, in <module>
    main()
  File "/data/genericrhel6-stage/src/treestatus.allizom.org/tree-status/vendor/lib/python/migrate/versioning/shell.py", line 207, in main
    ret = command_func(**kwargs)
  File "<string>", line 2, in version_control
  File "/data/genericrhel6-stage/src/treestatus.allizom.org/tree-status/vendor/lib/python/migrate/versioning/util/__init__.py", line 159, in with_engine
    return f(*a, **kw)
  File "/data/genericrhel6-stage/src/treestatus.allizom.org/tree-status/vendor/lib/python/migrate/versioning/api.py", line 250, in version_control
    ControlledSchema.create(engine, repository, version)
  File "/data/genericrhel6-stage/src/treestatus.allizom.org/tree-status/vendor/lib/python/migrate/versioning/schema.py", line 139, in create
    table = cls._create_table_version(engine, repository, version)
  File "/data/genericrhel6-stage/src/treestatus.allizom.org/tree-status/vendor/lib/python/migrate/versioning/schema.py", line 187, in _create_table_version
    raise exceptions.DatabaseAlreadyControlledError
migrate.exceptions.DatabaseAlreadyControlledError


> what's the URL for the staging instance?

  https://treestatus.allizom.org
Can you double-check what the audience is set to in who.ini? I'm having problems logging in. It should be treestatus.allizom.org.

Also, can you verify that catlee@mozilla.com and emorley@mozilla.com are in the users table with is_admin=1 and is_sheriff=1
(In reply to Chris AtLee [:catlee] from comment #4)
> Can you double-check what the audience is set to in who.ini? I'm having
> problems logging in. It should be treestatus.allizom.org.

looks like the who.ini file was overwritten during the last pull from github :( the audiences was set to '127.0.0.1:5000'. i have now updated this to:

  audiences = treestatus.allizom.org


> Also, can you verify that catlee@mozilla.com and emorley@mozilla.com are in
> the users table with is_admin=1 and is_sheriff=1

mysql> select * from users;
+----+---------------------+----------+------------+
| id | name                | is_admin | is_sheriff |
+----+---------------------+----------+------------+
|  1 | catlee@mozilla.com  |        1 |          1 |
|  2 | emorley@mozilla.com |        1 |          1 |
+----+---------------------+----------+------------+
2 rows in set (0.02 sec)
Looks like it's working:
https://treestatus.allizom.org/mozilla-central

Ed, if this looks ok, can we schedule a push to production?
Flags: needinfo?(emorley)
Looks good to me - ready for push :-)
Flags: needinfo?(emorley)
push to production complete.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
For posterity, these were the changes pushed:
https://github.com/catlee/treestatus/compare/257a811e8ef1e05e97eda144247b94cb62403015...41fb79d69f1e362ba008fc32d48255fcbc36e8ff

Notably:
* Add "Message of the day" field to trees (click through to the individual tree)
* Make "approval required" orange
* Pre-fill reason in per-tree page

Ryan, Phil: I've migrated most of the MOTDs - which will hopefully reduce the number of times they get overwritten due to forgetting the "save state". In fact, the save/restore state feature is almost redundant now (or at least not worth the tradeoff, given it's a hard UX to solve for changing subsets of trees). Thoughts?
Component: Server Operations: Web Operations → WebOps: Other
Product: mozilla.org → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.