| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a failure to format config sample https://review.openstack.org/597986 | 00:16 |
|---|---|---|
| *** lei-zh has joined #openstack-placement | 02:31 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Need further updates, no need to review https://review.openstack.org/599276 | 02:57 |
| openstackgerrit | Chen proposed openstack/nova master: Fix SOFT_DELETE vm_state string https://review.openstack.org/598482 | 03:33 |
| *** tetsuro has joined #openstack-placement | 04:34 | |
| *** tetsuro has quit IRC | 04:38 | |
| *** tetsuro_ has joined #openstack-placement | 04:39 | |
| *** lei-zh has quit IRC | 04:46 | |
| *** lei-zh has joined #openstack-placement | 05:23 | |
| *** tetsuro_ has quit IRC | 05:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Make scheduling weight more granular https://review.openstack.org/599308 | 07:51 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Make scheduling weight more granular https://review.openstack.org/599308 | 08:02 |
| *** giblet_off has quit IRC | 08:09 | |
| *** cdent has joined #openstack-placement | 08:17 | |
| *** e0ne has joined #openstack-placement | 08:37 | |
| *** ttsiouts has joined #openstack-placement | 08:41 | |
| openstackgerrit | Matthew Thode proposed openstack/nova stable/rocky: Don't use '_TransactionContextManager._async' https://review.openstack.org/597421 | 08:57 |
| openstackgerrit | Chen proposed openstack/nova master: Fix SOFT_DELETE vm_state string https://review.openstack.org/598482 | 09:11 |
| *** tssurya has joined #openstack-placement | 09:18 | |
| *** e0ne has quit IRC | 09:18 | |
| *** e0ne has joined #openstack-placement | 09:18 | |
| *** lei-zh has quit IRC | 09:20 | |
| *** ttsiouts has quit IRC | 09:30 | |
| openstackgerrit | Merged openstack/nova master: libvirt: guest: introduce blockStats instead of domain.blockStats https://review.openstack.org/526833 | 09:30 |
| *** ttsiouts has joined #openstack-placement | 09:46 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Making instance listing skipping down cells configurable https://review.openstack.org/592428 | 09:55 |
| openstackgerrit | Sujeet Banerjee proposed openstack/nova master: WIP: Added a periodic task for network management. https://review.openstack.org/599338 | 10:00 |
| *** ttsiouts has quit IRC | 10:18 | |
| *** ttsiouts has joined #openstack-placement | 10:20 | |
| *** ttsiouts has quit IRC | 10:22 | |
| *** ttsiouts has joined #openstack-placement | 11:13 | |
| *** gibi has joined #openstack-placement | 11:22 | |
| gibi | o/ | 11:23 |
| * cdent waves at gibi | 11:26 | |
| *** tetsuro has joined #openstack-placement | 11:26 | |
| *** tetsuro has quit IRC | 11:28 | |
| openstackgerrit | Merged openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | 11:30 |
| *** ttsiouts has quit IRC | 12:35 | |
| *** ttsiouts has joined #openstack-placement | 12:36 | |
| *** ttsiouts has quit IRC | 12:38 | |
| *** ttsiouts has joined #openstack-placement | 12:38 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Making instance/migration listing skipping down cells configurable https://review.openstack.org/592428 | 13:05 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Inspect and correct tox.ini, .stestr.conf and setup.cfg https://review.openstack.org/599377 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Update i18n handling to be placement oriented. https://review.openstack.org/599378 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Mechanically correct import of functional base class https://review.openstack.org/599379 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Correct several nova.tests and nova.tests.functional imports https://review.openstack.org/599380 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Empty __init__.py files that should be empty https://review.openstack.org/599381 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove some imports from test/fixtures that will not be used https://review.openstack.org/599382 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused db functionality and files https://review.openstack.org/599383 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove more unused imports from fixtures https://review.openstack.org/599384 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove the PlacementFixture from fixtures https://review.openstack.org/599385 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use placement.uuidsentinel https://review.openstack.org/599386 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused CheatingSerializer https://review.openstack.org/599387 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Tidy up use of policy_fixture https://review.openstack.org/599388 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Establish an importable 'conf' package https://review.openstack.org/599389 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Make a basic working DatabaseFixture https://review.openstack.org/599390 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Import placement, not nova, in rp db tests https://review.openstack.org/599391 | 13:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use absolute import in gabbi fixture https://review.openstack.org/599392 | 13:09 |
| cdent | functional test success: https://review.openstack.org/#/c/599392/ | 13:26 |
| *** ttsiouts has quit IRC | 13:26 | |
| *** ttsiouts has joined #openstack-placement | 13:26 | |
| *** ttsiouts has quit IRC | 13:31 | |
| gibi | placement folks, do you have oppinon about this API behavior question from tetsuro? https://review.openstack.org/#/c/565741/7/specs/stein/approved/placement-mixing-required-traits-with-any-traits.rst@156 | 13:32 |
| gibi | cdent: awesome | 13:33 |
| cdent | unit tests coming | 13:34 |
| gibi | I guess US is out today so they will have a nice suprise seeing passing tests tomorrow. | 13:35 |
| gibi | I also have a api-ref job locally that needs to be moved top of the series to pass | 13:35 |
| cdent | do we want to have a placement/scheduler meeting the since the US crowd is out, or should we skip it? My preference would be skip it so I can carry on with this test work | 13:39 |
| gibi | I'm OK to skip it | 13:41 |
| gibi | I've to leave early today to agree with some folks about how they will ruin my bathroom to fix the plumbing in the apartment building while I'm in Denver. | 13:43 |
| cdent | sounds a bit scary | 13:45 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Trim placement/utils.py to the single method used https://review.openstack.org/599397 | 13:47 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix configuration handling in policy unit test https://review.openstack.org/599398 | 13:47 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Turn on logging for the request log test https://review.openstack.org/599399 | 13:47 |
| cdent | that ^ ought to get unit going | 13:47 |
| gibi | cdent: yeah, it will be fun to arrie home after 12 hours of flight for a bathroom that might not exists | 13:49 |
| *** takashin has joined #openstack-placement | 13:56 | |
| *** ttsiouts has joined #openstack-placement | 14:04 | |
| *** ttsiouts has quit IRC | 14:10 | |
| *** takashin has left #openstack-placement | 14:11 | |
| alex_xu | gibi: it sounds like 1.24 break the old microverion | 14:14 |
| alex_xu | it isn't break, just change the old version API behaviour | 14:16 |
| gibi | alex_xu: yeah and I think that should not happen. But it did. But now I don't know which is better having consistency between member_of and required behavior and changing the old microversion behavior OR keeping the old behavior intact but not having a symmetric API | 14:19 |
| alex_xu | gibi: I think the rule is that not changing the old version behavior. whether we fix 1.24 can a separate problem. | 14:22 |
| cdent | gibi: on https://review.openstack.org/#/c/598948/ that syntax error (as you noticed) gets cleaned up further down the stack. All of the changes in that stack from ed are automated and some were expected to be wrong. It's only once my name is on the commits that we should start seeing any real improvements | 14:27 |
| cdent | I think, since it was automated, it would be good to just let it go through as is. | 14:27 |
| cdent | However, if you're not happy about that, the other option is to simply remove the offending line (which is what happens later) | 14:28 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove placement/test.py https://review.openstack.org/599406 | 14:32 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix alpha-ordering of imports for pep8 https://review.openstack.org/599407 | 14:32 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused imports as identified by pep8 https://review.openstack.org/599408 | 14:32 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix line length and whitespace issues https://review.openstack.org/599409 | 14:32 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove placement/db/api.py https://review.openstack.org/599410 | 14:32 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused fixtures in placement/tests/fixtures.py https://review.openstack.org/599411 | 14:32 |
| cdent | pep8 ^ | 14:32 |
| openstackgerrit | sahid proposed openstack/nova stable/rocky: hardware: fix memory check usage for small/large pages https://review.openstack.org/599413 | 14:45 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Trim placement/utils.py to the single method used https://review.openstack.org/599397 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix configuration handling in policy unit test https://review.openstack.org/599398 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Turn on logging for the request log test https://review.openstack.org/599399 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove placement/test.py https://review.openstack.org/599406 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix alpha-ordering of imports for pep8 https://review.openstack.org/599407 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused imports as identified by pep8 https://review.openstack.org/599408 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Fix line length and whitespace issues https://review.openstack.org/599409 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove placement/db/api.py https://review.openstack.org/599410 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove unused fixtures in placement/tests/fixtures.py https://review.openstack.org/599411 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Make functional tests voting and gating https://review.openstack.org/599414 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Make unit tests voting https://review.openstack.org/599415 | 14:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Make pep8 tests voting https://review.openstack.org/599416 | 14:46 |
| openstackgerrit | sahid proposed openstack/nova stable/rocky: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/599419 | 14:51 |
| *** ttsiouts has joined #openstack-placement | 14:52 | |
| gibi | alex_xu: thanks for your feedback here and on the review. This helps me proposing the right thing in the spec | 14:58 |
| gibi | cdent: still trying to find a grip how to judge the automated steps. I | 14:59 |
| gibi | cdent: sometimes those steps introduce a movement in the wrong direction like this syntax error | 15:00 |
| gibi | cdent: but I can accept that these automations help the overall progress | 15:00 |
| cdent | that pretty much the idea: they, in a gross/large fashion, move things in the right direction, with some cost of needing cleanup later | 15:01 |
| gibi | cdent: let me remove the -1 as it is true that the syntax error is fixed in a followup | 15:01 |
| gibi | cdent: I will plug +2 if other had a chance to look at it as well | 15:02 |
| cdent | the patch at the top is now green and voting on all 5 jobs: https://review.openstack.org/#/c/599416/ | 15:02 |
| gibi | cdent: awesome work | 15:02 |
| cdent | i've got the process mostly memorized now after doing a few times, despite doing it slightly differently each time | 15:02 |
| gibi | cdent: I reached up to https://review.openstack.org/#/c/599392/ nice work | 15:28 |
| cdent | great, thanks gibi | 15:29 |
| gibi | cdent: I'm going offline now. see you tomorrow | 15:29 |
| * cdent waves | 15:29 | |
| *** gibi has quit IRC | 15:46 | |
| *** e0ne has quit IRC | 15:48 | |
| *** ttsiouts has quit IRC | 16:02 | |
| *** ttsiouts has joined #openstack-placement | 16:03 | |
| *** ttsiouts has quit IRC | 16:07 | |
| *** purplerbot has quit IRC | 16:14 | |
| *** purplerbot has joined #openstack-placement | 16:15 | |
| *** tssurya has quit IRC | 17:48 | |
| *** e0ne has joined #openstack-placement | 17:48 | |
| *** e0ne_ has joined #openstack-placement | 17:50 | |
| *** e0ne has quit IRC | 17:54 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Removing pip-missing-reqs from default tox jobs https://review.openstack.org/599442 | 18:15 |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Update requirements and test-requirements https://review.openstack.org/599443 | 18:26 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Removing non-existent job from tox envlist https://review.openstack.org/599444 | 18:29 |
| openstackgerrit | Merged openstack/nova master: libvirt: skip setting rx/tx queue sizes for not virto interfaces https://review.openstack.org/595592 | 18:40 |
| *** e0ne_ has quit IRC | 18:43 | |
| *** e0ne has joined #openstack-placement | 18:48 | |
| *** e0ne has quit IRC | 19:33 | |
| *** cdent has quit IRC | 20:28 | |
| *** cdent has joined #openstack-placement | 20:43 | |
| *** cdent has quit IRC | 20:43 | |
| *** fried_rice1 has joined #openstack-placement | 21:19 | |
| *** fried_rice has quit IRC | 21:20 | |
| *** fried_rice1 is now known as fried_rice | 21:20 | |
| *** fried_rice1 has joined #openstack-placement | 21:29 | |
| *** fried_rice has quit IRC | 21:30 | |
| *** fried_rice has joined #openstack-placement | 21:31 | |
| *** fried_rice1 has quit IRC | 21:33 | |
| *** fried_rice has quit IRC | 21:57 | |
| *** fried_rice has joined #openstack-placement | 22:22 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!