| *** tetsuro has joined #openstack-placement | 05:11 | |
| *** tetsuro has quit IRC | 07:19 | |
| *** e0ne has joined #openstack-placement | 07:54 | |
| *** cdent has joined #openstack-placement | 07:59 | |
| *** rubasov has joined #openstack-placement | 07:59 | |
| *** tetsuro has joined #openstack-placement | 08:02 | |
| *** e0ne has quit IRC | 08:17 | |
| *** tetsuro has quit IRC | 08:18 | |
| *** tetsuro has joined #openstack-placement | 08:19 | |
| *** e0ne has joined #openstack-placement | 08:39 | |
| *** rubasov has quit IRC | 08:46 | |
| *** tetsuro has quit IRC | 08:47 | |
| *** cdent has quit IRC | 08:48 | |
| *** e0ne has quit IRC | 08:49 | |
| *** e0ne has joined #openstack-placement | 08:50 | |
| *** mriedem has joined #openstack-placement | 08:53 | |
| *** tetsuro has joined #openstack-placement | 08:59 | |
| *** rubasov has joined #openstack-placement | 09:09 | |
| *** rubasov has quit IRC | 09:16 | |
| *** e0ne has quit IRC | 09:20 | |
| *** cdent has joined #openstack-placement | 09:22 | |
| cdent | gibi: the sibling stuff is weird and confusing and hard to trace, but very useful | 09:24 |
|---|---|---|
| gibi | cdent: yeah, sorry for the noise :) Anyhow now I left a question about the test-requirements | 09:29 |
| cdent | I'm changing that. I'm not going to put placement in test-requirements because we don't want to require it for the unit tests | 09:30 |
| cdent | and we don't want "unit" tests doing tox siblings | 09:30 |
| gibi | cdent: that makes sense. Then I guess for the local dev env case we need something in the tox.ini to install placement master for the functional tests | 09:33 |
| gibi | cdent: I think we could live with an extra deps line for each functional env declaration | 09:33 |
| cdent | yeah, that's what I've done | 09:34 |
| *** tetsuro has quit IRC | 09:34 | |
| gibi | cdent: but that might interfere with the tox-siblings on the gate | 09:34 |
| cdent | we'll see :) | 09:34 |
| gibi | :) | 09:34 |
| gibi | as far as I understand sibilings install is the last installation step so it should replace the placement installed from tox.ini | 09:35 |
| cdent | I believe so | 09:36 |
| *** rubasov has joined #openstack-placement | 09:49 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Adapt placement fixtures for external use https://review.openstack.org/617935 | 09:50 |
| *** rubasov has quit IRC | 10:00 | |
| *** e0ne has joined #openstack-placement | 10:00 | |
| *** tetsuro has joined #openstack-placement | 10:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/placement master: Add a document for creating DB revisions https://review.openstack.org/614024 | 10:12 |
| openstackgerrit | Matt Riedemann proposed openstack/placement master: Fix comment in earlier patch https://review.openstack.org/617988 | 10:12 |
| openstackgerrit | Matt Riedemann proposed openstack/placement master: Remove sqlalchemy-migrate from requirements.txt https://review.openstack.org/614539 | 10:12 |
| openstackgerrit | Matt Riedemann proposed openstack/placement master: Add a placement-manage CLI https://review.openstack.org/600161 | 10:12 |
| *** cdent has quit IRC | 10:27 | |
| *** cdent has joined #openstack-placement | 10:31 | |
| *** e0ne has quit IRC | 10:35 | |
| *** rubasov has joined #openstack-placement | 10:54 | |
| *** rubasov has quit IRC | 11:01 | |
| *** mriedem has quit IRC | 11:23 | |
| *** tetsuro has quit IRC | 11:27 | |
| *** cdent has quit IRC | 11:27 | |
| *** tetsuro has joined #openstack-placement | 11:41 | |
| *** mriedem has joined #openstack-placement | 12:06 | |
| *** tetsuro has quit IRC | 12:15 | |
| *** cdent has joined #openstack-placement | 12:22 | |
| *** tetsuro has joined #openstack-placement | 12:26 | |
| *** e0ne has joined #openstack-placement | 12:33 | |
| *** mriedem has quit IRC | 12:37 | |
| *** mriedem has joined #openstack-placement | 12:45 | |
| cdent | gibi: yeah, figured out the delete local thing, just as you said | 12:53 |
| cdent | need to keep the db alive across that | 12:54 |
| *** e0ne has quit IRC | 12:56 | |
| mriedem | tetsuro: can you take a look at my questions in your patch here? https://review.openstack.org/#/c/613305/ | 13:01 |
| tetsuro | mriedem: looking... | 13:03 |
| gibi | cdent: I'm not sure that test really tests what it say it tests. If I keep the placement fixture alive during the whole test and just reject every request with HTTP500 from the fixture when the test deletes the instance then the delete of the server fails | 13:03 |
| gibi | cdent: hm, I can try rejecting the requests by throwing ks_exc.MissingAuthPlugin to get back the old behavior | 13:04 |
| cdent | gibi: It's testing not-configured, not error, as far as I read the thing | 13:04 |
| cdent | "nova-api isn't configured to talk to placement" | 13:05 |
| cdent | I'm not sure we want, at least not right now, to change the semantics of the test | 13:05 |
| gibi | cdent: yeah, you are right. so most probably throwing ks_exc.MissingAuthPlugin would be appropriate during the server delete | 13:05 |
| cdent | i have it nearly working | 13:05 |
| gibi | cool | 13:06 |
| cdent | it just required a couple more tweaks in the placement fixture. useful ones | 13:07 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Adapt placement fixtures for external use https://review.openstack.org/617935 | 13:07 |
| *** e0ne has joined #openstack-placement | 13:24 | |
| *** tetsuro_ has joined #openstack-placement | 13:27 | |
| *** tetsuro_ has quit IRC | 13:27 | |
| *** tetsuro has quit IRC | 13:29 | |
| *** tetsuro_ has joined #openstack-placement | 13:29 | |
| *** rubasov has joined #openstack-placement | 13:31 | |
| *** e0ne has quit IRC | 13:51 | |
| *** e0ne has joined #openstack-placement | 14:00 | |
| *** cdent has quit IRC | 14:03 | |
| *** mriedem has quit IRC | 14:04 | |
| *** tetsuro has joined #openstack-placement | 14:05 | |
| *** cdent has joined #openstack-placement | 14:09 | |
| *** tetsuro_ has quit IRC | 14:09 | |
| *** e0ne has quit IRC | 14:25 | |
| *** s10 has joined #openstack-placement | 14:36 | |
| *** s10_ has joined #openstack-placement | 14:51 | |
| *** s10 has quit IRC | 14:53 | |
| *** tetsuro has quit IRC | 14:57 | |
| *** rubasov has quit IRC | 15:05 | |
| *** cdent has quit IRC | 15:06 | |
| *** s10_ has quit IRC | 15:12 | |
| *** rubasov has joined #openstack-placement | 15:12 | |
| *** s10_ has joined #openstack-placement | 15:16 | |
| *** cdent has joined #openstack-placement | 15:17 | |
| *** cdent has quit IRC | 15:18 | |
| *** cdent has joined #openstack-placement | 15:18 | |
| *** tetsuro has joined #openstack-placement | 15:21 | |
| *** e0ne has joined #openstack-placement | 15:48 | |
| *** s10_ has quit IRC | 16:01 | |
| *** cdent has quit IRC | 16:02 | |
| *** tetsuro has quit IRC | 16:02 | |
| *** s10 has joined #openstack-placement | 16:07 | |
| *** tetsuro has joined #openstack-placement | 16:07 | |
| *** cdent has joined #openstack-placement | 16:11 | |
| *** cdent has quit IRC | 16:12 | |
| *** cdent has joined #openstack-placement | 16:12 | |
| *** tetsuro has quit IRC | 16:13 | |
| *** s10 has quit IRC | 16:14 | |
| *** tetsuro has joined #openstack-placement | 16:20 | |
| *** N3l1x has quit IRC | 16:22 | |
| *** s10 has joined #openstack-placement | 16:25 | |
| *** s10 has quit IRC | 16:27 | |
| *** rubasov has quit IRC | 16:51 | |
| *** rubasov has joined #openstack-placement | 16:52 | |
| *** cdent has quit IRC | 17:00 | |
| *** rubasov has quit IRC | 17:08 | |
| *** tetsuro has quit IRC | 17:16 | |
| *** e0ne has quit IRC | 17:42 | |
| openstackgerrit | zhouxinyong proposed openstack/osc-placement master: Replacing the link in HACKING.rst https://review.openstack.org/618270 | 18:32 |
| *** rubasov has joined #openstack-placement | 19:39 | |
| *** s10_ has joined #openstack-placement | 20:13 | |
| *** rubasov has quit IRC | 20:21 | |
| *** jaypipes has quit IRC | 20:27 | |
| *** e0ne has joined #openstack-placement | 21:00 | |
| *** cdent has joined #openstack-placement | 22:01 | |
| *** cdent has quit IRC | 22:08 | |
| *** e0ne has quit IRC | 22:42 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!