Wednesday, 2018-07-04

*** tetsuro has joined #openstack-placement00:00
*** takashin has joined #openstack-placement00:06
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5)  https://review.openstack.org/57084200:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6)  https://review.openstack.org/57133000:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7)  https://review.openstack.org/57199200:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8)  https://review.openstack.org/57199300:49
openstackgerritTetsuro Nakamura proposed openstack/nova master: Regression test for bug 1779818  https://review.openstack.org/58001101:15
openstackbug 1779818 in OpenStack Compute (nova) "child's root provider is not updated." [Medium,Confirmed] https://launchpad.net/bugs/177981801:15
*** edmondsw has joined #openstack-placement01:42
*** edmondsw has quit IRC01:46
*** takashin has left #openstack-placement02:05
openstackgerritEric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation  https://review.openstack.org/55666902:31
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension  https://review.openstack.org/58001702:47
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension  https://review.openstack.org/57906703:07
*** edmondsw has joined #openstack-placement03:30
*** edmondsw has quit IRC03:35
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension  https://review.openstack.org/58001703:58
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object  https://review.openstack.org/57362804:07
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Refactor policies to policy in InstanceGroup DB model  https://review.openstack.org/57911304:07
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension  https://review.openstack.org/58001704:15
*** alex_xu has quit IRC04:49
*** alex_xu has joined #openstack-placement04:57
*** alex_xu has quit IRC05:13
*** alex_xu has joined #openstack-placement05:15
*** edmondsw has joined #openstack-placement05:19
openstackgerritMerged openstack/nova master: Add action initiator attribute to the instance payload  https://review.openstack.org/53624305:19
openstackgerritMerged openstack/nova master: Prevent updating an RP's parent to form a loop  https://review.openstack.org/57961805:20
*** edmondsw has quit IRC05:24
openstackgerritMerged openstack/nova master: Remove unused DB API instance_group_member* methods  https://review.openstack.org/57848805:37
*** e0ne has joined #openstack-placement05:42
*** e0ne has quit IRC05:53
openstackgerritMerged openstack/nova master: Merge server create schema for security group extension  https://review.openstack.org/57834506:18
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Refactor policies to policy in InstanceGroup DB model  https://review.openstack.org/57911306:25
*** nicolasbock has joined #openstack-placement06:35
openstackgerritDeepak Mourya proposed openstack/nova master: Resource_provider API handler does not return specific error codes  https://review.openstack.org/57942307:00
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337507:05
*** edmondsw has joined #openstack-placement07:07
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801907:12
*** edmondsw has quit IRC07:12
*** e0ne has joined #openstack-placement07:20
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337507:27
*** tssurya has joined #openstack-placement07:36
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Refactor policies to policy in InstanceGroup DB model  https://review.openstack.org/57911307:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337507:54
*** tetsuro has quit IRC08:21
openstackgerritTuan Do Anh proposed openstack/nova-specs master: fix tox python3 overrides  https://review.openstack.org/57979308:25
*** tssurya has quit IRC08:33
*** ttsiouts has joined #openstack-placement08:36
*** tssurya has joined #openstack-placement08:36
*** ttsiouts has quit IRC08:43
*** tetsuro has joined #openstack-placement08:46
*** ttsiouts has joined #openstack-placement08:51
*** edmondsw has joined #openstack-placement08:56
*** tetsuro has quit IRC08:59
*** edmondsw has quit IRC09:01
openstackgerritSilvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver  https://review.openstack.org/55419509:10
*** ttsiouts has quit IRC09:10
*** tetsuro has joined #openstack-placement09:12
*** tetsuro has quit IRC09:12
openstackgerritSilvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver  https://review.openstack.org/55419509:12
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337509:16
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340109:16
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116609:16
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146509:16
*** ttsiouts has joined #openstack-placement09:18
*** ttsiouts has quit IRC09:23
openstackgerritGhanshyam Mann proposed openstack/nova master: Complete the api-ref of security group rule  https://review.openstack.org/58010909:35
*** ttsiouts has joined #openstack-placement09:37
*** ttsiouts has quit IRC10:01
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753410:10
*** cdent has joined #openstack-placement10:31
*** edmondsw has joined #openstack-placement10:45
*** edmondsw has quit IRC10:50
openstackgerritMultipleCrashes proposed openstack/nova master: Retry decorator fix for autoscale delete  https://review.openstack.org/56341811:29
*** ttsiouts has joined #openstack-placement11:39
*** peereb has joined #openstack-placement11:52
*** edmondsw has joined #openstack-placement12:33
*** edmondsw has quit IRC12:38
openstackgerritMerged openstack/nova stable/queens: libvirt: Fix the rescue race for vGPU instances  https://review.openstack.org/57950314:01
*** edmondsw has joined #openstack-placement14:22
*** edmondsw has quit IRC14:27
cdentnot holidaying efried ?14:35
openstackgerritMerged openstack/nova stable/pike: Use instance project/user when creating RequestSpec during resize reschedule  https://review.openstack.org/57792614:54
openstackgerritMerged openstack/nova stable/ocata: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56371914:54
openstackgerritMerged openstack/nova master: libvirt: add qemu version check when configuring mtu for network  https://review.openstack.org/57961114:54
gibicdent: is it just me or we really use an invalid UUID as default missing project_id and user_id in pacement? https://github.com/openstack/nova/blob/07d3844123115b4b0e47d202a339aaab942d1a92/nova/conf/placement.py#L2014:59
gibicdent: hint, the last block has 13 digits if my counting is correct15:00
cdentugh15:00
cdentwell that's a mighty bug15:01
gibiIs it safe to fix this with a simple bugfix removing the extra 0 or we need to do some db magic as well to remove the wrong values from there as well?15:01
cdenthow is it that's not hit in tests anywhere?15:01
gibiwe only have warning for bad UUIDs15:02
gibicdent: this patch tries to escalate that to error in test https://review.openstack.org/#/c/540386/15:02
cdentgibi: we'll probably want to check with jaypipes but I would guess that will need some db fixing magic15:02
gibicdent: and it caught it15:02
gibicdent: OK, then I file a bug and ask around tomorrow when we have more brains15:03
cdentgibi: I meant in terms of the db schema, but apparently that's not set to String(36)15:03
cdentlooks like an Integer?15:04
cdenthow does that even work?15:04
gibiI don't know either15:05
cdentthe reason it was a long varchar in the first place was because we had thought that project and user id could be something other than a uuid15:06
cdentI guess note that in the bug too15:06
gibiI will do15:06
cdentsomething has gone very weird and confused with the creation of the consumer stuff. neither review nor tests seemed to have been as good as they needed to be15:07
cdentI assume this is because we've been rushing...15:07
gibistill better to caught it now than after the release15:10
cdentyes, definitely15:10
* cdent spins up a devstack15:10
gibibut I agree that it is due to the rush15:10
gibireported https://bugs.launchpad.net/nova/+bug/178010715:12
openstackLaunchpad bug 1780107 in OpenStack Compute (nova) "The default of the missing project_id and user_id in placement is an invalid UUID" [Undecided,New]15:12
*** tssurya has quit IRC15:14
openstackgerritsahid proposed openstack/nova stable/queens: add mtu to libvirt xml for ethernet and bridge types  https://review.openstack.org/57801015:17
openstackgerritsahid proposed openstack/nova stable/queens: libvirt: add qemu version check when configuring mtu for network  https://review.openstack.org/58018815:17
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029715:20
openstackgerritBalazs Gibizer proposed openstack/nova master: Send soft_delete from context manager  https://review.openstack.org/47645915:20
openstackgerritMerged openstack/nova master: Fix unbound local when saving an unchanged RequestSpec  https://review.openstack.org/57376815:28
openstackgerritMerged openstack/nova stable/ocata: Ensure resource class cache when listing usages  https://review.openstack.org/57427915:28
openstackgerritBalazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object  https://review.openstack.org/56884015:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec  https://review.openstack.org/56726715:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes  https://review.openstack.org/57084715:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler  https://review.openstack.org/56726815:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945915:32
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331715:32
*** e0ne has quit IRC15:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec  https://review.openstack.org/56726715:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes  https://review.openstack.org/57084715:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler  https://review.openstack.org/56726815:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945915:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331715:44
*** tssurya has joined #openstack-placement15:55
*** edmondsw has joined #openstack-placement16:10
*** edmondsw has quit IRC16:15
cdentgibi: turns out using int for project id and user id is broken16:18
cdentwill make a bug for that oo16:18
*** e0ne has joined #openstack-placement16:21
*** peereb has quit IRC16:21
cdentoh wait, no16:22
cdentsigh16:22
cdentthere's a projects table16:22
cdentso we've normalized out from an allocations table with consumer, project and user uuids in it, to allocations, consumer, project and user tables16:23
*** e0ne has quit IRC16:25
cdentgibi: i added a comment to the existing bug16:27
jaypipescdent: yes, that is correct.16:37
jaypipesgibi: how can I help?16:38
cdentjaypipes: yeah, we got confused because we couldn't figure out why the badly formed not-a-uuid DEFAULT was still working16:38
cdentjaypipes: the questions are on the bug: https://bugs.launchpad.net/nova/+bug/178010716:39
openstackLaunchpad bug 1780107 in OpenStack Compute (nova) "The default of the missing project_id and user_id in placement is an invalid UUID" [Undecided,New]16:39
cdentbasically: what's the best way to fix it (if at all)16:39
* jaypipes looks16:41
*** ttsiouts has quit IRC16:43
openstackgerritMerged openstack/nova stable/pike: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/55903216:46
openstackgerritMerged openstack/nova stable/ocata: Use instance project/user when creating RequestSpec during resize reschedule  https://review.openstack.org/57792916:55
*** e0ne has joined #openstack-placement17:08
openstackgerritMerged openstack/nova master: Remove remaining legacy DB API instance_group* methods  https://review.openstack.org/57849717:36
*** edmondsw has joined #openstack-placement17:59
*** tssurya has quit IRC17:59
*** tssurya has joined #openstack-placement18:00
*** edmondsw has quit IRC18:03
*** e0ne has quit IRC18:36
*** edmondsw has joined #openstack-placement19:48
*** edmondsw has quit IRC19:52
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:53
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:55
openstackgerritAndreas Karis proposed openstack/nova master: Add documentation for emulator threads policy  https://review.openstack.org/57995619:58
*** cdent has quit IRC21:13
*** nicolasbock has quit IRC21:16
*** edmondsw has joined #openstack-placement21:36
*** edmondsw has quit IRC21:41
*** tssurya has quit IRC22:34
*** tssurya has joined #openstack-placement22:36
*** tssurya has quit IRC22:41
*** edmondsw has joined #openstack-placement23:25
*** edmondsw has quit IRC23:30
*** takashin has joined #openstack-placement23:44

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!