| *** lei-zh has joined #openstack-placement | 01:52 | |
| *** tetsuro has joined #openstack-placement | 02:45 | |
| *** tetsuro has quit IRC | 03:12 | |
| *** tetsuro has joined #openstack-placement | 03:16 | |
| *** Nel1x has joined #openstack-placement | 03:23 | |
| *** tetsuro has quit IRC | 03:51 | |
| *** Nel1x has quit IRC | 03:52 | |
| *** lei-zh has quit IRC | 05:03 | |
| *** lei-zh has joined #openstack-placement | 05:39 | |
| *** e0ne has joined #openstack-placement | 06:16 | |
| *** e0ne has quit IRC | 06:18 | |
| *** tssurya has joined #openstack-placement | 07:45 | |
| *** lei-zh1 has joined #openstack-placement | 07:59 | |
| *** lei-zh has quit IRC | 08:01 | |
| *** lei-zh1 has quit IRC | 08:08 | |
| *** e0ne has joined #openstack-placement | 08:12 | |
| openstackgerrit | Takashi NATSUME proposed openstack/placement master: Publish api-ref in the placement project https://review.openstack.org/614434 | 08:50 |
|---|---|---|
| *** e0ne has quit IRC | 09:16 | |
| *** e0ne has joined #openstack-placement | 09:25 | |
| *** ttsiouts has joined #openstack-placement | 09:28 | |
| *** ttsiouts has quit IRC | 09:29 | |
| *** ttsiouts has joined #openstack-placement | 09:45 | |
| *** cdent has joined #openstack-placement | 10:38 | |
| *** ChanServ sets mode: +o cdent | 11:08 | |
| *** cdent changes topic to "See https://docs.openstack.org/placement/latest/ and https://developer.openstack.org/api-ref/placement/" | 11:08 | |
| *** ChanServ sets mode: -o cdent | 11:08 | |
| *** e0ne has quit IRC | 11:15 | |
| *** e0ne has joined #openstack-placement | 11:15 | |
| *** e0ne has quit IRC | 11:27 | |
| *** ttsiouts has quit IRC | 12:30 | |
| *** e0ne has joined #openstack-placement | 13:06 | |
| openstackgerrit | Takashi NATSUME proposed openstack/placement master: Remove build-openstack-api-ref jobs https://review.openstack.org/614497 | 13:13 |
| *** mriedem has joined #openstack-placement | 13:14 | |
| *** mriedem is now known as ash_williams | 13:15 | |
| *** ttsiouts has joined #openstack-placement | 13:16 | |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 13:16 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Delete the old migrations https://review.openstack.org/611440 | 13:16 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Rename the database files https://review.openstack.org/614350 | 13:16 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 13:16 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: WIP - Show an alembic migration https://review.openstack.org/614025 | 13:16 |
| ash_williams | edleafe: is there anything setup in tests that run the mysql/pg-specific opportunistic tests against ^? | 14:05 |
| ash_williams | i'm guessing not since placement repo doesn't have https://github.com/openstack/nova/blob/master/tools/test-setup.sh | 14:05 |
| ash_williams | which is used by infra to setup the mysql/pg backends | 14:05 |
| edleafe | ash_williams: Probably not, AFAIK. I did a local setup that added the /etc/placement/placement.conf file, that defined the connection to a local MariaDB server | 14:08 |
| ash_williams | i just confirmed they aren't | 14:08 |
| edleafe | I'll take a look at test-setup.sh | 14:08 |
| ash_williams | http://logs.openstack.org/41/611441/9/check/openstack-tox-functional-py35/fba1eb7/job-output.txt.gz#_2018-10-31_13_53_26_454042 | 14:08 |
| ash_williams | i'm not sure if just throwing that file into a chnage at the bottom of the series will do the trick or not | 14:08 |
| ash_williams | andreas jaeger would probably know | 14:09 |
| edleafe | ash_williams: I'll play with it. Thanks for the pointer | 14:09 |
| ash_williams | aha http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/test-setup/tasks/main.yaml | 14:10 |
| ash_williams | i think just putting that file in the placement repo will do it | 14:10 |
| edleafe | ash_williams: The extraction script to separate placement out of nova looked at the git history to see anything that touched placement files. It missed some things that we had to pull in manually, and test-setup.sh seems to be one of them | 14:14 |
| cdent | ash_williams: are we in the chainsaw phase or the no chainsaw phase? | 14:14 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 14:18 |
| ash_williams | i still have both hands | 14:18 |
| *** e0ne has quit IRC | 14:19 | |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 14:20 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Delete the old migrations https://review.openstack.org/611440 | 14:20 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Rename the database files https://review.openstack.org/614350 | 14:20 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 14:20 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: WIP - Show an alembic migration https://review.openstack.org/614025 | 14:20 |
| edleafe | ash_williams: cdent: ^^ Now with test-setup.sh! | 14:20 |
| cdent | edleafe: you see my comment (now twice) about the TypeError thing? | 14:21 |
| edleafe | cdent: no, I missed that. I'll look at it now (entering meeting hell in 40 minutes) | 14:22 |
| cdent | edleafe: may be as simple as basing off sean-k-mooney 's change | 14:22 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 14:24 |
| sean-k-mooney | i see my name :) | 14:26 |
| *** e0ne has joined #openstack-placement | 14:26 | |
| * cdent sings destiny's child | 14:27 | |
| * sean-k-mooney sighs that song will be stuck in my head while i go to grab lunch | 14:28 | |
| cdent | success | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Added alembic environment https://review.openstack.org/611441 | 14:38 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Delete the old migrations https://review.openstack.org/611440 | 14:38 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Rename the database files https://review.openstack.org/614350 | 14:38 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 14:38 |
| openstackgerrit | Ed Leafe proposed openstack/placement master: WIP - Show an alembic migration https://review.openstack.org/614025 | 14:38 |
| edleafe | cdent: ^^ rebased on top of sean-k-mooney's patch | 14:38 |
| cdent | cool, will keep an eye on it. will be very good to see this stuff in | 14:40 |
| *** e0ne has quit IRC | 14:41 | |
| *** e0ne has joined #openstack-placement | 14:44 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 14:59 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove sqlalchemy-migrate from requirements.txt https://review.openstack.org/614539 | 15:11 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove sqlalchemy-migrate from requirements.txt https://review.openstack.org/614539 | 15:20 |
| *** ash_williams has left #openstack-placement | 15:21 | |
| *** ash_williams has joined #openstack-placement | 15:21 | |
| cdent | thanks for the reminder ash_williams | 15:21 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused packages from lower-constraints.txt https://review.openstack.org/614559 | 15:45 |
| cdent | efried: there's a mess of stuff starting at https://review.openstack.org/#/c/614307/ which, once in, will allow us to have a working db migration setup and a devstack. So if you've got some eyes, that would be skippy. I expect there are some details that might matter | 16:01 |
| efried | cdent: Heard and understood, but don't get your hopes up. | 16:01 |
| efried | a) Lots of distractions, b) material in there ^ that I don't understand (and don't have bandwidth to learn about) | 16:02 |
| cdent | efried: ack. I figured that might be the case, thus the "if" (which maybe I should have made all caps and *emph*) | 16:02 |
| cdent | i know how that can be | 16:02 |
| cdent | jaypipes: you too on https://review.openstack.org/#/c/614307/ and beyond, IF you've got the time/energy | 16:02 |
| cdent | ash_williams is already on it with his chainsaw | 16:03 |
| *** ttsiouts has quit IRC | 16:39 | |
| *** ttsiouts has joined #openstack-placement | 16:39 | |
| *** ttsiouts has quit IRC | 16:44 | |
| *** rubasov has quit IRC | 16:51 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Correct lower-constraints.txt and the related tox job https://review.openstack.org/614559 | 16:52 |
| cdent | ash_williams: that ^ is mcuh different from the original attempt at that. | 16:53 |
| ash_williams | commented | 16:58 |
| ash_williams | i assume this is also busted in nova | 16:58 |
| *** ash_williams is now known as ash_sawing | 16:59 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Correct lower-constraints.txt and the related tox job https://review.openstack.org/614559 | 17:02 |
| cdent | jaypipes: responded on the ensure consumer stuff. It needs to move, do you have a suggestion on where? | 17:04 |
| *** e0ne has quit IRC | 17:08 | |
| cdent | jaypipes: back at ya | 17:17 |
| openstackgerrit | Matt Riedemann proposed openstack/placement master: Correct lower-constraints.txt and the related tox job https://review.openstack.org/614559 | 17:43 |
| edleafe | cdent: Any suggestions on fixing the config issue here? http://logs.openstack.org/41/611441/11/check/openstack-tox-cover/1ef4b28/testr_results.html.gz | 17:52 |
| cdent | edleafe: have you see the comment I left already? | 17:53 |
| edleafe | geez. No, just resurfaced from meetings. My workflow is messed up | 17:53 |
| cdent | meetings make for sad :( | 17:54 |
| cdent | but yeah, lots of comments rolled through there in the past little while but with regard to those failing tests, you basically need to set up the config in the same way the existing functional tests do | 17:54 |
| cdent | edleafe: I'm going to fix the very bottom of that stack so we're going to have some rebasing hell for a bit. Would you prefer that I just change my thing, or more of the stack? | 17:55 |
| edleafe | cdent: I'm going to have split attention for most of the rest of the day, so go for it if you're so inclined | 17:56 |
| cdent | edleafe: okay, I'll do it | 17:56 |
| *** ash_sawing has quit IRC | 18:07 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Correct lower-constraints.txt and the related tox job https://review.openstack.org/614559 | 18:10 |
| *** mriedem has joined #openstack-placement | 18:11 | |
| *** mriedem is now known as ash_williams | 18:11 | |
| cdent | ash_williams: i think i managed to bring that lower-constraints around to some form of sane | 18:23 |
| cdent | it's probably not perfect, but I don't know if we have a way to find that | 18:23 |
| ash_williams | is it possible to just drop what's unused (not in the pip freeze) and only update what is necessary to update b/c of test failures? | 18:26 |
| cdent | I did something very close to that | 18:27 |
| cdent | basically that, plus add things that were missing | 18:27 |
| ash_williams | ok | 18:27 |
| cdent | the missing things...I'm not sure how to determine a right version | 18:27 |
| *** e0ne has joined #openstack-placement | 18:31 | |
| *** e0ne has quit IRC | 18:32 | |
| ash_williams | upper-constraints is probably fine in that case since it's what we test against | 18:33 |
| ash_williams | or maybe whatever is in the pip freeze via transitive deps i suppose | 18:33 |
| *** tssurya has quit IRC | 18:39 | |
| *** ash_williams is now known as mriedem_away | 19:25 | |
| *** e0ne has joined #openstack-placement | 19:44 | |
| *** e0ne has quit IRC | 19:45 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Move ensure_consumer to a new placement.handlers.util https://review.openstack.org/614307 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Harden placement init under wsgi https://review.openstack.org/612444 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Added alembic environment https://review.openstack.org/614350 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Delete the old migrations https://review.openstack.org/611440 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP - Show an alembic migration https://review.openstack.org/614025 | 20:06 |
| cdent | edleafe: I had to do some pretty extensive fiddling around to get 614350 working with the live tests, including reordering the patches and collapsing the file renames into the alembic add. | 20:08 |
| cdent | Some of the tests that come from oslo_db are still not working, but it is a bit closer. I'm travelling tomorrow and most of Friday so I won't be able to get back to it, but I hope that what I've done there is enough to take us a bit further | 20:08 |
| edleafe | cdent: Thanks - I'll follow up with it as best as I can | 20:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 20:15 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 20:18 |
| cdent | edleafe: I'd tried to leave you with some leading TODOs in python | 20:19 |
| cdent | edleafe: some of it might be safe to remove, but the issue with consumer generation migration sync comparisons is probably a problem, but is also probably related to https://bugs.launchpad.net/oslo.db/+bug/1776527 (I've added a comment on the review about that) | 20:23 |
| openstack | Launchpad bug 1776527 in oslo.db "TestNovaAPIMigrationsMySQL.test_models_sync fails with "AssertionError: Models and migration scripts aren't in sync:" due to TextClause comparisons" [Undecided,Fix released] - Assigned to Tony Breeds (o-tony) | 20:23 |
| cdent | mriedem_away: you might have ideas on that stuff ^ | 20:24 |
| *** e0ne has joined #openstack-placement | 20:30 | |
| cdent | edleafe, efried : I'll be away tomorrow and most of friday travelling | 20:47 |
| efried | ack | 20:48 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove sqlalchemy-migrate from requirements.txt https://review.openstack.org/614539 | 20:49 |
| *** e0ne has quit IRC | 20:50 | |
| * cdent waves goodnight | 20:51 | |
| *** cdent has quit IRC | 20:51 | |
| *** mriedem_away has quit IRC | 22:44 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!