Tuesday, 2018-08-14

*** rcernin_ has joined #openstack-nova00:02
*** rcernin has quit IRC00:03
*** takashin has joined #openstack-nova00:06
*** namnh has quit IRC00:17
*** eharney has quit IRC00:18
*** brinzhang has joined #openstack-nova00:22
*** tbachman_ has joined #openstack-nova00:27
*** rcernin has joined #openstack-nova00:29
*** tbachman has quit IRC00:29
*** tbachman_ is now known as tbachman00:29
*** rcernin has quit IRC00:29
*** rcernin has joined #openstack-nova00:30
*** rcernin_ has quit IRC00:32
*** slaweq_ has joined #openstack-nova00:32
*** mriedem has joined #openstack-nova00:33
*** zhurong has joined #openstack-nova00:33
mriedemo/00:33
*** slaweq_ has quit IRC00:37
Kevin_Zhengtoo early00:40
*** nicolasbock has quit IRC00:45
Kevin_Zhengmriedem, got something interesting, as we can provide metadata with this API, https://developer.openstack.org/api-ref/compute/#create-image-createimage-action it might override the system generated metadata: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L287700:45
Kevin_Zhengand we did it again in https://github.com/openstack/nova/blob/master/nova/compute/api.py#L291100:45
Kevin_Zhengseems redundant, and should we limit the key?00:46
Kevin_Zhenglike don't allow set user_id00:46
Kevin_Zhengimage_type00:46
mriedemmy network needs to catch up...00:47
Kevin_ZhengOK00:47
*** yedongcan has joined #openstack-nova00:47
*** slaweq_ has joined #openstack-nova00:53
mriedemKevin_Zheng: hmm, yeah, not sure00:55
mriedemlooks like this could be used to change the owner of the snapshot to another user00:55
*** jpenag has joined #openstack-nova00:56
Kevin_Zhengyeah00:56
*** vivsoni has quit IRC00:57
*** gmann has quit IRC00:57
*** jpena|off has quit IRC00:57
*** lennyb has quit IRC00:57
*** DinaBelova has quit IRC00:57
*** yankcrime has quit IRC00:57
Kevin_ZhengYeah00:57
Kevin_ZhengAnd the uuid might not be important00:57
Kevin_ZhengInstance uuid00:57
*** slaweq_ has quit IRC00:57
*** nicolasbock has joined #openstack-nova00:58
*** panda|ruck has quit IRC00:59
*** namnh has joined #openstack-nova00:59
*** johnthetubaguy has quit IRC01:00
mriedemi would think if this were a major issue, someone would have reported/fixed it by now after 10 years01:01
*** johnthetubaguy has joined #openstack-nova01:02
*** panda has joined #openstack-nova01:02
*** vivsoni has joined #openstack-nova01:03
*** 18WAA8G2G has joined #openstack-nova01:04
*** gmann has joined #openstack-nova01:04
*** DinaBelova has joined #openstack-nova01:04
*** yankcrime has joined #openstack-nova01:04
*** 18WAA8G2G has quit IRC01:04
*** DinaBelova has quit IRC01:04
*** namnh has quit IRC01:04
Kevin_Zhengmriedem, got it01:04
*** DinaBelova has joined #openstack-nova01:05
*** mriedem has quit IRC01:05
*** panda has quit IRC01:06
*** openstackgerrit has quit IRC01:06
*** panda has joined #openstack-nova01:07
*** slaweq_ has joined #openstack-nova01:11
*** Nel1x has joined #openstack-nova01:13
*** slaweq_ has quit IRC01:16
*** hemna_ has quit IRC01:17
*** mhen has quit IRC01:18
*** mhen has joined #openstack-nova01:22
*** mriedem has joined #openstack-nova01:24
*** tbachman_ has joined #openstack-nova01:26
*** tbachman has quit IRC01:28
*** tbachman_ is now known as tbachman01:28
*** mrsoul has quit IRC01:31
*** slaweq_ has joined #openstack-nova01:32
*** slaweq_ has quit IRC01:37
*** zhurong has quit IRC01:39
*** namnh has joined #openstack-nova01:44
*** mriedem1 has joined #openstack-nova01:47
mriedem1blarg!01:47
*** lei-zh has joined #openstack-nova01:52
*** hongbin has joined #openstack-nova02:01
mriedem1are we waiting for anything on this? https://review.openstack.org/#/c/590445/ it's marked for RC202:05
mriedem1but hasn't yet been merged on master02:05
*** openstackgerrit has joined #openstack-nova02:09
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix service list for disabled compute using MC driver  https://review.openstack.org/58393602:09
*** slaweq_ has joined #openstack-nova02:11
*** mriedem has quit IRC02:11
*** mriedem1 is now known as mriedem02:11
*** zhurong has joined #openstack-nova02:15
*** slaweq_ has quit IRC02:16
*** lei-zh has quit IRC02:31
*** lei-zh has joined #openstack-nova02:31
*** slaweq_ has joined #openstack-nova02:32
*** psachin has joined #openstack-nova02:36
*** slaweq_ has quit IRC02:36
takashin02:43
*** tbachman_ has joined #openstack-nova02:48
*** tbachman has quit IRC02:52
*** tbachman_ is now known as tbachman02:52
*** vivsoni has quit IRC03:01
*** takashin has left #openstack-nova03:02
naichuansmriedem: Hi, Matt, because Rocky feather has been freezed while bp: vgpu_rocky hasn't finished, I reprose a new bp here: https://blueprints.launchpad.net/nova/+spec/vgpu-stein. Please check if it is OK. Thank you very much.03:03
*** Bhujay has joined #openstack-nova03:04
openstackgerritliuyamin proposed openstack/python-novaclient master: Replace os-client-config to openstacksdk  https://review.openstack.org/59014103:05
*** slaweq_ has joined #openstack-nova03:11
*** Nel1x has quit IRC03:12
*** zhurong has quit IRC03:14
*** slaweq_ has quit IRC03:16
*** slaweq_ has joined #openstack-nova03:32
*** slaweq_ has quit IRC03:37
*** hongbin has quit IRC03:39
mriedemnaichuans: ok, i guess we'll just obsolete the rocky bp since none of the changes merged03:40
*** Bhujay has quit IRC03:40
*** namnh has quit IRC03:40
*** slaweq_ has joined #openstack-nova03:53
*** udesale has joined #openstack-nova03:53
*** slaweq_ has quit IRC03:57
*** vivsoni has joined #openstack-nova04:00
naichuansmriedem: Got it, thanks04:04
*** dave-mccowan has quit IRC04:12
*** lei-zh has quit IRC04:16
*** Bhujay has joined #openstack-nova04:21
*** janki has joined #openstack-nova04:28
*** slaweq_ has joined #openstack-nova04:32
*** slaweq_ has quit IRC04:36
*** links has joined #openstack-nova04:43
*** abhishekk has joined #openstack-nova04:55
*** gyee has quit IRC05:05
*** slaweq_ has joined #openstack-nova05:11
*** slaweq_ has quit IRC05:16
*** slaweq_ has joined #openstack-nova05:32
*** slaweq_ has quit IRC05:37
openstackgerritMatt Riedemann proposed openstack/nova master: Delete instance_id_mappings record in instance_destroy  https://review.openstack.org/59155805:45
mriedemKevin_Zheng: yikun: bingo ^05:46
*** ratailor has joined #openstack-nova05:47
*** slaweq_ has joined #openstack-nova05:53
*** lei-zh has joined #openstack-nova05:56
*** slaweq_ has quit IRC05:57
mriedemdo you guys use pypi mirrors in china? otherwise tox times out05:57
Kevin_ZhengHmm, it should automatically connect to the fastest one, no?06:00
fanzhangmriedem Kevin_Zheng I am using index-url = https://pypi.tuna.tsinghua.edu.cn/simple ;-)06:04
*** vivsoni has quit IRC06:13
*** odyssey4me has quit IRC06:14
*** odyssey4me has joined #openstack-nova06:14
openstackgerritZhenyu Zheng proposed openstack/nova master: Fix service list for disabled compute using MC driver  https://review.openstack.org/58393606:15
Kevin_Zhengfanzhang Are you doing it in Linux or windows06:21
fanzhangKevin_Zheng Linux and MacOS06:22
Kevin_ZhengThanks, we will try latter06:24
fanzhangnp06:24
*** lpetrut has joined #openstack-nova06:26
*** slaweq_ has joined #openstack-nova06:32
*** lei-zh1 has joined #openstack-nova06:33
*** lei-zh has quit IRC06:36
*** slaweq_ has quit IRC06:36
*** pcaruana has joined #openstack-nova06:44
*** adrianc has joined #openstack-nova06:44
*** slaweq_ has joined #openstack-nova06:53
*** slaweq_ has quit IRC06:57
*** slaweq_ has joined #openstack-nova07:00
*** ccamacho has joined #openstack-nova07:01
*** rcernin has quit IRC07:02
gibimriedem: hi! I'm not planning to have a notification meeting also I assume your timezone is not friendly for that meeting07:20
*** lbragstad has joined #openstack-nova07:23
*** sahid has joined #openstack-nova07:29
sean-k-mooneymriedem: o/ i have been doing some livemigration testing between different backends and different OSs and have been finding issues. im going to keep refining my testing  but while it is posible to move the vms i dont think its working correctly07:39
sean-k-mooneymriedem: live migration from cetos to ubuntu is broken. that used to work but the emltor path is nolonger the same and we use the emulator path of the src for the dest when checnking compatablity07:41
sean-k-mooneylive migration from lb to ovs appears to work but ovs to lb times out waiting for the plug event07:41
sean-k-mooneyfrom neutron07:41
*** sapcc-bot1 has quit IRC07:42
sean-k-mooneyim going to add a few more nodes to my testing to confirm all the behavior. ill send an email to the list with a summary when done07:42
*** ygk_12345 has joined #openstack-nova07:47
*** holser_ has joined #openstack-nova07:47
*** sapcc-bot2 has quit IRC07:51
*** janki is now known as janki|lunch07:52
*** tssurya has joined #openstack-nova07:54
*** alexchadin has joined #openstack-nova07:57
gryfsean-k-mooney, I've answered your review.08:01
gryferm. comments.08:01
ygk_12345hi all08:02
gryfjohnthetubaguy, could you please also look at https://review.openstack.org/#/c/536351 ?08:02
ygk_12345what does negative number indicate for RAM in hypervisor stats ?08:03
ygk_12345does it mean no more memory available for vms on the computes ?08:03
*** Bhujay has quit IRC08:10
*** mchlumsky has quit IRC08:12
*** openstackstatus has quit IRC08:12
*** mchlumsky has joined #openstack-nova08:13
*** dtantsur|afk is now known as dtantsur08:15
sean-k-mooneygryf: thanks, you are correct that the disk filter is deprcated rather than removed.08:15
sean-k-mooneythe disk fileter is calulating the avialble space its self form the host_state.free_diks_mb and host_state.total_usable_disk_gb08:17
sean-k-mooneyi am not sure how host_state.total_usable_disk_gb is related to the migrtion_data disk available_gb08:18
sean-k-mooneythey apepar to be independent08:19
*** mvkr has quit IRC08:20
mriedemgibi: yeah i won't be around for the notification meeting08:21
sean-k-mooneyygk_12345: it means that the total memory used is grater than total ram - reserved ram08:21
gibimriedem: no problem08:21
mriedemsean-k-mooney: ok i think lb plug events is a known issue isn't it?08:22
ygk_12345sean-k-mooney: also I am getting warnings in the nova logs that enough memory cant be allocated . what does this mean ?08:22
sean-k-mooneymriedem: well live migration between 2 lb nodes seams to work.  i thikn libvirt is using the souce xml without the updated network interfaces in the ovs case and that is breaking things. i have not really had time to dig into it yet08:24
sean-k-mooneyygk_12345: can you post the error message to paste.openstack.org08:24
sean-k-mooneyygk_12345: also on the node with the error can you tell me how much ram do you have, what is the reseved  memory set in the config  and how much is used by instance on that node?08:26
*** jpenag is now known as jpena08:27
*** ygk_12345 has quit IRC08:27
*** derekh has joined #openstack-nova08:29
*** Bhujay has joined #openstack-nova08:34
mriedemdims: http://docs.sqlalchemy.org/en/latest/core/metadata.html?highlight=class%20column#sqlalchemy.schema.Column.params.onupdate08:39
*** tbachman has quit IRC08:40
*** kaisers has joined #openstack-nova08:40
mriedeme.g. Column('last_modified', DateTime, onupdate=func.utc_timestamp())08:41
*** janki|lunch is now known as janki08:42
*** cdent has joined #openstack-nova08:43
*** mvkr has joined #openstack-nova08:52
*** jaosorior has quit IRC08:53
sean-k-mooneymdbooth: regarding https://review.openstack.org/#/c/536351 can you confirm that this is never used in the disk filter. looking at https://github.com/openstack/nova/blob/master/nova/scheduler/filters/disk_filter.py#L38-L59 we appear to purely calulate the availe disk from the host_state08:58
mdboothsean-k-mooney: Don't they have the same source?09:01
sean-k-mooneymdbooth: i can not find any use of the migrate_data.disk_available_mb in the condoctor before the downcall to the compute09:02
sean-k-mooneymdbooth: which the host_state object and where we are constuction the migration data09:03
mdboothsean-k-mooney: Can you restate your original question. Can I confirm that *what*? is never used...09:03
sean-k-mooneymdbooth: so looking at https://review.openstack.org/#/c/536351/3/nova/virt/libvirt/driver.py the migrate_data object constructed in this fuction is not used before the downcall to the compute node.09:04
sean-k-mooneybut there is also a comment about this change relating to the disk filter however the disk filter appears to calulate the availabel disk space independly via the info in the host state object09:05
sean-k-mooneyso im trying to figure out at what point is the value store in the migarte_data.disk_available_mb actully used09:06
*** alexchadin has quit IRC09:07
*** huanxie has joined #openstack-nova09:09
mdboothsean-k-mooney: It's used in check_can_live_migrate_source09:12
mdboothBut only if disk_over_commit is set09:12
gryfits verified in _assert_dest_node_has_enough_disk method and only for block_migration09:13
sean-k-mooneymdbooth: where is mdbooth gryf i am not seeing either of those methods in the condoctor live_migrate.py are they performed on the compute node via rpc09:15
mdboothiirc check_can_live_migrate_source is called from check_can_live_migrate_dest09:16
gryfsean-k-mooney, they are performed in driver09:16
mdboothNah, they're called from compute manager09:16
gryfand called from compute manager09:16
mdboothThe code's in the driver09:16
gryfyeah09:16
*** ratailor has quit IRC09:17
mdboothOh, _dest is called from conductor09:17
gryfcompute.manager.check_can_live_migrate_destination09:18
mdboothWhich then calls _source09:18
openstackgerritMerged openstack/nova master: placement: ignore policy scope check failures if not enforcing scope  https://review.openstack.org/59044509:19
*** ratailor has joined #openstack-nova09:19
openstackgerritChris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next  https://review.openstack.org/59136709:20
mriedemmdbooth: https://docs.openstack.org/nova/latest/reference/live-migration.html :)09:21
mdboothmriedem: It's so simple they even drew a picture about it.09:21
sean-k-mooneymdbooth: gryf so yes looking at https://github.com/openstack/nova/blob/0502b0ffaf20ef3735e633715ba0f9dda976cb7a/nova/virt/libvirt/driver.py#L677409:22
sean-k-mooneyi do not see the disk allocation ratio being used in that check09:23
sean-k-mooneyso would that not imply we have to take account of it when we first calulate disk_avaiable_mb09:24
mdboothsean-k-mooney: So this code isn't related to DiskFilter09:25
gryfsean-k-mooney, possibly, but that's another thing, isn't it?09:25
sean-k-mooneymdbooth: yes i know09:25
sean-k-mooneygryf: well the bug is fixing how we calulate the available diskspace. if it does so by not allowing over allocation of disk when its turned on that jsut creates another bug09:26
gryfhm.09:26
mdboothI would hope that disk_available_mb would be calculated as the amount of available disk, in mb.09:26
mdboothIf anybody makes it anything else, I want to know where they live.09:27
mdboothIf we need to take disk allocation ratio into account, we should do that at the point of use.09:27
mdboothOr call it something else.09:27
sean-k-mooneyit should be eitehr available disk = ((total diskspcae - reserved disk space) * disk allocation ratio) - used disk space  or available disk = ((total diskspcae - reserved disk space)) - used disk space09:27
sean-k-mooneyif its the later then we never take account of disk overcomit when live migrating09:28
mdboothI'd argue that overcommit has always been a job for the scheduler, anyway.09:28
gryfsean-k-mooney, i'd move out reserved disk space out of equation09:28
openstackgerritChris Dent proposed openstack/nova stable/pike: [placement] Retry allocation writes server side  https://review.openstack.org/59074509:28
mdboothI think live migration should fail only if there aren't physically enough bits on the destination to hold the source bits.09:29
gryfwe don;t want to change that by allocation ratio if it's something else from 109:29
sean-k-mooneygryf: why not09:29
sean-k-mooneythe allocation ratio should only be applied to the portion of the disk that is not reserved09:30
gryfso, isn't it be (total disk space - used disk space - reserved disk space) * allocation ratio?09:32
cdent"If we need to take disk allocation ratio into account, we should do that at the point of use." yes09:33
gryfor even (total disk space - used disk space) * allocation ratio - reserved disk space09:33
*** markvoelker has joined #openstack-nova09:33
sean-k-mooneyactully gryf no we dont want to multiple the used disk space by the allocation ratio09:33
sean-k-mooneybut its not needed in either case09:34
sean-k-mooneyi just realised https://github.com/openstack/nova/blob/0502b0ffaf20ef3735e633715ba0f9dda976cb7a/nova/virt/libvirt/driver.py#L676809:34
mdboothThere are 2 ways to get a live migration destination:09:34
mdbooth1. From the scheduler, which will take this stuff into account09:34
sean-k-mooneyis using the actul size of the disk not the virtual size09:34
mdbooth2. From the admin, who doesn't care about your pesky artificial resource constraints.09:34
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen support for delete instance allocations  https://review.openstack.org/59159709:35
sean-k-mooneyin the overcomit case that is so the driver only compares bytes need to copy the disk to actul bytes free on the dest and the schduler/placement operates on teh virtual disk size with allocation ratios taken into account09:35
*** alexchadin has joined #openstack-nova09:36
mdboothsean-k-mooney: That's the only thing it should ever compare. Anything else is legacy.09:36
sean-k-mooneymdbooth: yes just trying to get the full end to end effect of this change right in my head to make sure there was no edge case missed09:37
mdboothRight. If an admin wants to move a workload to compute X, the only reason we should say no is if it physically won't fit. Any future problems caused by that are the admin's to deal with.09:37
gryfmdbooth, by "phisically fit" do you mean current src image size, or the maximum one?09:39
mdboothBut hopefully the admin only does that if they really, really need to, and usually they ask the scheduler.09:39
mdboothgryf: I mean current, allocated disk.09:39
gryfack09:39
sean-k-mooneymdbooth: gryf cool im much happier this this change then. its sutle but makes sense in context09:39
*** openstackstatus has joined #openstack-nova09:41
*** ChanServ sets mode: +v openstackstatus09:41
*** hoonetorg has quit IRC09:43
openstackgerritSurya Seetharaman proposed openstack/python-novaclient master: [WIP/POC] Client changes for down cell  https://review.openstack.org/57956309:44
*** ratailor has quit IRC09:48
*** lpetrut has quit IRC09:48
*** ratailor has joined #openstack-nova09:49
*** lei-zh1 has quit IRC09:51
gryfsean-k-mooney, do you want me to rebase and correct typo on commit message?09:54
gryftypos*09:54
sean-k-mooneygryf: they are just nits so no its fine09:54
gryfcool. thanks09:55
sean-k-mooneygryf: if you have to respin the patch for some other reason then sure but no need to respin just for that. that said my spelling is terrible so other may have a different view09:55
gryfsean-k-mooney, I'd just like to have it merged :)09:56
*** hoonetorg has joined #openstack-nova09:57
sean-k-mooneygryf: by the way im not sure if johnthetubaguy  is around at the moment to review this or not09:57
gryfI guess he is not09:57
sean-k-mooneygryf: his time upstream has been limited this cycle i belive given his current positon09:58
gryfRight.09:59
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Force refresh instance info_cache during heal  https://review.openstack.org/59160710:02
sean-k-mooneymriedem: i dont know if your still about but your pretty familar with the livemigration code perhaps you could take a look at https://review.openstack.org/#/c/536351/3 in johnthetubaguy absence. that said not sure this is high enough prority to make it into RC2 so maybe it will have to wait till stein in anycase10:02
mriedemKevin_Zheng: see https://review.openstack.org/59160710:03
mriedemsean-k-mooney: just going to dinner10:03
*** dpawlik has quit IRC10:03
mriedemsean-k-mooney: oh that one, idk10:03
sean-k-mooneymriedem: no worries i was just trying to think who was about to take a look at it.10:04
*** dpawlik has joined #openstack-nova10:05
openstackgerritJay Pipes proposed openstack/nova master: placement: order multi-row INSERT by PK  https://review.openstack.org/59160910:05
*** markvoelker has quit IRC10:07
*** mdbooth has quit IRC10:10
*** lbragstad has quit IRC10:12
openstackgerritJay Pipes proposed openstack/nova master: placement: order multi-row INSERT by PK  https://review.openstack.org/59160910:15
*** mvkr has quit IRC10:22
*** panda is now known as panda|ruck10:24
*** adrianc has quit IRC10:25
*** adrianc has joined #openstack-nova10:26
*** abhishekk has quit IRC10:50
*** jaosorior has joined #openstack-nova10:51
openstackgerritChen proposed openstack/nova master: Typo fixes on notifications doc  https://review.openstack.org/59162110:58
*** udesale has quit IRC10:59
*** markvoelker has joined #openstack-nova11:04
*** jpena is now known as jpena|lunch11:15
*** jaosorior has quit IRC11:17
*** holser_ has quit IRC11:17
*** jaosorior has joined #openstack-nova11:17
*** s10 has joined #openstack-nova11:27
*** josecastroleon has quit IRC11:28
*** josecastroleon has joined #openstack-nova11:28
*** markvoelker has quit IRC11:37
*** yedongcan has left #openstack-nova11:44
*** nicolasbock has quit IRC11:49
*** slagle has joined #openstack-nova11:53
*** alexchadin has quit IRC11:55
*** ratailor has quit IRC12:09
sean-k-mooneymriedem: so far i have found 3 neutron bugs and 1 nova bug in my livemigration testing :( im taking notes and will open them later. ovs-ovs and lb-lb migration seam to be fine however12:14
tssuryagmann: around ?12:15
kosamaraefried: Hi! You say that pci subsys_vendor_id etc should appear in libvirt. I checked what libvirt.Host.device_lookup_by_name returns (calls libvirt) and it doesn't report those properties.12:16
*** holser_ has joined #openstack-nova12:17
sean-k-mooneykosamara: if you do virsh nodedev-list and then virsh nodedev-dumpxml pci_0000_00_00_0 you will see the xml fragment that nova parses form libvirt12:19
*** dpawlik has quit IRC12:20
sean-k-mooneykosamara: if you do a nodedev-dumpxml on a network device it will also return the feature supported by that nic http://paste.openstack.org/show/728005/12:21
*** jpena|lunch is now known as jpena12:21
sean-k-mooneykosamara: we currently store does as netwrok_capablites in the pci_devcice table and they will need to be translated to traits in the future12:21
*** erlon has joined #openstack-nova12:26
sean-k-mooneykosamara: this is where we currently parse all the pcidevice info in the libvirt driver https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5855-L593512:28
*** tbachman has joined #openstack-nova12:28
sean-k-mooneykosamara: we invoke that in _get_pcidev_info which is called in get_available_resource12:29
*** tbachman has quit IRC12:29
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen support for put allocations  https://review.openstack.org/59164712:33
*** udesale has joined #openstack-nova12:33
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen support for put allocations  https://review.openstack.org/59164712:35
*** josecastroleon has quit IRC12:39
*** jaosorior has quit IRC12:39
*** rnoriega has quit IRC12:39
*** dpawlik has joined #openstack-nova12:42
*** dave-mccowan has joined #openstack-nova12:43
*** josecastroleon has joined #openstack-nova12:46
kosamarasean-k-mooney: Still, it doesn't show the subsystem vendor/prod, rev id or class.12:49
kosamarasean-k-mooney but maybe this is just missing from libvirt and should be there, because it makes sense.12:50
sean-k-mooneykosamara: the vendor id is in the parent pci device12:52
sean-k-mooneykosamara: http://paste.openstack.org/show/728011/12:53
*** sahid has quit IRC12:54
kosamarasean-k-mooney: subsystem vendor (SVendor in lspci) is a separate property. Eg an Evga-made GPU might have a different SVendor than a Gigabyte one.12:55
kosamarabut the same "vendor_id" (nvidia)12:56
*** awaugama has joined #openstack-nova12:56
sean-k-mooneyhum perhaps. we dont track that difference in nova12:56
sean-k-mooneyfrom a driver/guest perspective we would not be able to tell the differecne really provided its the same model number.12:57
sean-k-mooneyyes i know evga may have the same part running at a different clockrate then gagabyte or asusu but we are not that detailed in our tracking12:58
openstackgerritSurya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down  https://review.openstack.org/56778513:00
openstackgerritSurya Seetharaman proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down  https://review.openstack.org/58482913:00
openstackgerritSurya Seetharaman proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList  https://review.openstack.org/59165613:00
openstackgerritSurya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell  https://review.openstack.org/59165713:00
openstackgerritSurya Seetharaman proposed openstack/nova master: Return a minimal construct for nova show when a cell is down  https://review.openstack.org/59165813:00
kosamarasean-k-mooney efried: so in the end, does it make sense to leave that open in the inventory yaml spec? maybe for a future use?13:02
*** gbarros has joined #openstack-nova13:04
*** psachin has quit IRC13:06
*** Bhujay has quit IRC13:07
*** Bhujay has joined #openstack-nova13:07
sean-k-mooneykosamara: i dont think so. i think that might be leaking too much host specific info to the enduser. we could leave a comment but i dont think we should whoudl consider 2 GTX1080s form differend ODMs to be different13:08
*** alexchadin has joined #openstack-nova13:09
sean-k-mooneykosamara: your other option is to modify libvirt. the reaosn we get the nic feature flags from libvirt instead of ethtool is because we were not allow to bypass libvirt to look this up form ethool or sysfs in the past so going to sysfs to get the subvendor will likely not be allow nor is lspci likely to be accepted but you could document those approches.13:11
*** gbarros has quit IRC13:11
gmanntssurya: hi13:12
sean-k-mooneyanyway im really goint to lunch now.13:12
*** eharney has joined #openstack-nova13:13
kosamarasean-k-mooney: I agree, just mentioning the possibilities13:14
*** sahid has joined #openstack-nova13:16
*** gbarros has joined #openstack-nova13:20
*** _ix has joined #openstack-nova13:26
tssuryagmann: nothing urgent, was just wondering if https://review.openstack.org/#/c/502859/ - this series could be rebased13:29
tssuryaalso thanks for https://review.openstack.org/#/q/topic:bp/api-extensions-merge-rocky+(status:open) in general, for the handling-down-cell, I guess the two more dependencies are that of extended status and extended volumes part13:30
*** hvvcben has joined #openstack-nova13:30
gmanntssurya: ack, ll do. i am waiting to do policy removal first. anyways will check it tomorrow.13:30
tssuryagmann: np, thanks!13:30
*** _ix has quit IRC13:34
*** hvvcben has quit IRC13:35
*** Bhujay has quit IRC13:40
*** josecastroleon has quit IRC13:40
*** jaosorior has joined #openstack-nova13:44
*** burt has joined #openstack-nova13:46
openstackgerritSurya Seetharaman proposed openstack/python-novaclient master: Client changes for handling-down-cell with microversion bump  https://review.openstack.org/57956313:51
*** _ix has joined #openstack-nova13:51
*** _ix has quit IRC13:57
*** brinzhang has quit IRC13:57
efriedkosamara: I think I would like to converge on an approach where the schema allows the union of what's recognized by all the drivers, for identification. And then each driver documents which fields it pays attention to.13:58
efriedkosamara: I haven't caught up yet, in case you've responded to the comments I left last night, but how do you feel about making the spec address both libvirt and powervm?13:58
sean-k-mooneystill eating lunch but +1 on a single schema that could be used for any virt dirver13:59
sean-k-mooneyit would also be nice the schema wa genreic ennough to support device beyond pci also but that is a streach14:01
efriedsean-k-mooney: That's the point. The schema is restrictive to whatever we support right now, but extensible to incorporate other things in the future. And the design statement is explicitly that drivers pay attention to whichever fields they support, and ignore the rest.14:05
*** brtknr has quit IRC14:05
efriedIt's less about the schema being generic and more about it being inclusive.14:05
sean-k-mooneywell when i said generic i kind of was infering that we can assume what type of device it is and what bus it is attached to.14:07
sean-k-mooneywe have to declar that in the file14:07
*** josecastroleon has joined #openstack-nova14:08
efriedYou would determine the "bus" by parsing the PCI address, right?14:08
sean-k-mooneyno14:08
sean-k-mooneythat wont work for power where we dont have an adress correct14:08
sean-k-mooneywe would know the bus by saying bus: pcie14:08
efriedWe have an address. It just doesn't look like domain:bus:device.function14:08
efriedanyway, determining "device type" (whatever that means) and "bus" (whatever that is) would be the purview of the specific driver.14:10
efriedIf we can identify common fields, like prod and vendor IDs, great.14:10
cdent(specific driver)++14:10
efriedBut something like PCI address is going to be ignored by PowerVM; and something like DRC name is going to be ignored by libvirt.14:10
sean-k-mooneyefried: im mocking somthin up one sec14:11
sean-k-mooneyyou know what i might as well jsut use etherpad for this14:15
sean-k-mooneyhttps://etherpad.openstack.org/p/generic-device-schema14:16
*** alexchadin has quit IRC14:20
*** hongbin has joined #openstack-nova14:21
*** eharney has quit IRC14:24
*** dpawlik has quit IRC14:25
kosamaraefried: I haven't replied yet, I'm halfway through them :) I do think it's best to have a general YAML spec that caters to the union of the drivers' needs as you say.14:27
kosamaraThis is the first part, then implementing the "update_provider_tree" algo including understanding the YAML should be driver-specific.14:28
sean-k-mooneykosamara: efried cdent what to you think of somthing like this https://etherpad.openstack.org/p/generic-device-schema14:28
kosamaraFor the implementation, I don't have any experience on powervm, but I see your reasoning and it makes sense to address both, since it would basically copy over your work on powervm. What I don't know is how much extra attention will have to be paid to the powervm-specific parts, and if this would hold back the rest14:30
efriedkosamara: I am developing the powervm bits anyway, so would be delighted to propose them in tree as well as in nova-powervm.14:32
kosamaraSorry, I'll be back in 1h and check it out14:32
efriedkosamara: If you're talking about holding up the spec, I think we absolutely need to hold up the spec if we can't agree on an approach that's going to work for powervm as well as libvirt.14:33
*** ccamacho has quit IRC14:34
kosamaraefried: of course, it should pave the way for both, I'm only considering the part of the powervm implementation. Let me fix the YAML part first with your comments and we'll have a clearer picture.14:34
*** eharney has joined #openstack-nova14:37
*** _ix has joined #openstack-nova14:40
openstackgerritChris Dent proposed openstack/nova master: Add placement perf info gathering hook to end of nova-next  https://review.openstack.org/59136714:43
openstackgerritChris Dent proposed openstack/nova master: placement: order multi-row INSERT by PK  https://review.openstack.org/59160914:46
*** Bhujay has joined #openstack-nova14:52
*** ccamacho has joined #openstack-nova14:56
*** dpawlik has joined #openstack-nova15:03
*** dpawlik has quit IRC15:07
prometheanfiredansmith: mriedem: not that I'm rushing or anything, but is there an ETA for backports to pike/queens for https://bugs.launchpad.net/nova/+bug/178634615:08
openstackLaunchpad bug 1786346 in OpenStack Compute (nova) rocky "live migrations slow" [High,In progress] - Assigned to Matt Riedemann (mriedem)15:08
dansmithprometheanfire: I think we should be good to go on backporting that now15:08
dansmithwe need to commit the one to rocky too for rc215:08
prometheanfireyarp15:09
prometheanfireiirc the backport isn't going to be clean15:09
dansmithmelwitt: I assume we just need another stabler to hit it15:09
dansmithprometheanfire: likely not, the revert was messy :/15:09
dansmithlyarwood: can you hit this patch we need for rc2? https://review.openstack.org/#/c/591275/15:09
openstackgerritMatthew Booth proposed openstack/nova master: Add regression test for bug 1550919  https://review.openstack.org/59173315:10
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)15:10
*** mdbooth has joined #openstack-nova15:11
mdboothlyarwood mriedem: https://review.openstack.org/#/c/591733/ Regression test for evacuate on shared storage deleting data15:13
mdboothIt's funky because it actually runs the libvirt driver15:13
mdbooth(but not libvirt)15:13
dansmithmdbooth: just fyi, mriedem is in china this week15:14
mdboothdansmith: Yeah, I noticed he's been around disturbingly early.15:14
mdboothNeed to be careful what I say about him in the morning ;)15:14
* dansmith nods15:15
mdboothdansmith: You may also be interested in ^^^ btw15:15
dansmithyou may be surprised15:15
mdboothHehe15:15
openstackgerritDan Smith proposed openstack/nova stable/queens: WIP: Wait for network-vif-plugged before starting live migration  https://review.openstack.org/59173515:17
mdboothdansmith: Is ^^^ a continuation of the problem originally addressed by sahid's patch?15:18
*** ccamacho has quit IRC15:19
dansmithmdbooth: it's a test backport of a more generic solution that already landed in master, but with a non-backportable rpc change that I don't think we actually needed to make it work15:19
dansmithso I'm floating that to see if it's possible15:19
dansmithsince we have to revert his, I'm hoping we can replace it with that15:19
mdboothAh, ok.15:19
mdboothI didn't follow the whole discussion, but I was under the impression that perhaps linux bridge remained a problem?15:20
mdboothOr are we good there, too?15:20
dansmithwe should be good there to as far as all the evidence we've collected shows15:20
mdboothCool15:20
openstackgerritMatthew Booth proposed openstack/nova master: Move static _get_power_off_values to compute_utils  https://review.openstack.org/57882215:21
openstackgerritMatthew Booth proposed openstack/nova master: Don't delete disks on shared storage during evacuate  https://review.openstack.org/57884615:21
dansmithprometheanfire: yeah the backport of the revert to queens is even worse than the original revert15:22
dansmithto the point that I wonder if it's even worth trying to do a normal backport15:23
lyarwooda normal revert you mean, of the stable/queens patch15:24
dansmithwhat I mean is, a backport of the revert patch from master and rocky might not be worth it,15:25
dansmithbut we could do a straight revert of the queens patch on queens yeah15:25
dansmithlyarwood: do you know of any precedent for that?15:25
lyarwoodkk, in the past we've just done a straight revert of the stable changes, not backporting anything from branch to branch15:25
dansmithreally?15:25
prometheanfireya, that's kinda what I was thinking15:25
dansmithI'm confident that we've backported reverts, and that's what mriedem did for stable/rocky15:26
prometheanfireiirc, a clean revert isn't possible either for pike at least15:26
*** psachin has joined #openstack-nova15:26
dansmithbut if that's kosher, that's likely way easier15:26
lyarwooddansmith: I assume it was clean?15:26
dansmithwhich? I haven't tried the direct revert yet15:26
lyarwooddansmith: once things start to conflict the revert backports get awkward15:26
dansmithyup15:26
lyarwooddansmith: the master to rocky revert sorry, and I assume it was as rocky was only just cut15:27
dansmithyeah, I assume it was, matt did it15:27
dansmithlyarwood: so, different change-id for the revert on queens? or force it to be the same so gerrit shows it properly?15:28
lyarwooddansmith: it has to be different doesn't it?15:29
*** adrianc has quit IRC15:29
dansmithno, I can force it to be the same15:29
lyarwooddansmith: I didn't think gerrit allowed the same change-id twice on the same branch15:29
*** adrianc has joined #openstack-nova15:29
dansmithI mean the same change-id as the reverts on master and rocky15:29
dansmithmeaning, use that change id on queens too, even though it's not a direct cherry-pick15:30
lyarwooddansmith: ah right, yeah the same change id for the reverts15:30
openstackgerritDan Smith proposed openstack/nova stable/queens: Revert "libvirt: slow live-migration to ensure network is ready"  https://review.openstack.org/59176115:30
dansmithokay, queens was clean ^15:30
dansmithas was pike15:31
openstackgerritDan Smith proposed openstack/nova stable/pike: Revert "libvirt: slow live-migration to ensure network is ready"  https://review.openstack.org/59176215:32
prometheanfirenice15:33
openstackgerritEric Fried proposed openstack/nova master: [placement] split gigantor SQL query, add logging  https://review.openstack.org/59151015:35
*** janki has quit IRC15:36
openstackgerritEric Fried proposed openstack/nova master: [placement] split gigantor SQL query, add logging  https://review.openstack.org/59004115:37
*** links has quit IRC15:41
*** Sundar_ has joined #openstack-nova15:43
Sundar_efried: Please ping me when you have the time. Thanks.15:45
openstackgerritxhzhf proposed openstack/nova master: [vmware] add missing os types in vSphere sdk 6.5  https://review.openstack.org/59177115:51
*** itlinux has joined #openstack-nova15:54
efriedSundar_: Yo, what's up?15:54
*** tssurya has quit IRC15:54
*** ccamacho has joined #openstack-nova15:56
Sundar_Hi efried, please look at my email to you on merge conflict.15:57
efriedSundar_: Roger that, looking.15:57
efriedSundar_: You probably just need to rebase to master.15:57
efriedSundar_: Right, so I see you tried that, let me see what I can do.15:58
*** cdent has quit IRC15:58
*** gyee has joined #openstack-nova16:02
*** pcaruana has quit IRC16:02
efriedSundar_: What problem did you have rebasing this? It's working out okay for me.16:02
efriedSundar_: Fixed, uploaded PS816:05
*** sahid has quit IRC16:05
*** rpittau has quit IRC16:09
melwitt.16:11
*** jpena is now known as jpena|off16:13
Sundar_Thanks, efried! I explained the problem in the email.16:14
efriedSundar_: I responded to the email. Hopefully I understood the problem correctly. Let me know if PS8 looks correct.16:15
efriedSundar_: Looks like you uploaded a no-op PS9 :)16:16
efried(no op other than to change the committer back to yourself)16:16
Sundar_efried: Yes. I retried my steps to see where the issue is, and there were no issues. The 'git review' succeeded too. Not sure if the no-op patch is an issue. Should we revert that?16:18
sean-k-mooneySundar_: your noop patch just changed the commiter field in the reivew. it wont have any real effect16:19
efriedSundar_: Right, leave it as is.16:19
openstackgerritDan Smith proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting  https://review.openstack.org/59177516:19
Sundar_sean-k-mooney, efried: Thanks. Hopefully I have included all previous input from both of you and others. But there were tons of them, so I will do another review myself. Please LMK if I missed anything.16:20
efriedack16:20
sean-k-mooneydansmith:  regardin the live migration waiting is there any testing i can help with. im currently testing migraitng between different backend / configuration but i have that flag set also16:22
dansmithsean-k-mooney: confirming the new thing works for LB would be great16:23
sean-k-mooneydansmith: lb->lb seams to work fine on master16:23
dansmithas expected, cool16:23
sean-k-mooneywell master as of yesterday16:23
sean-k-mooneyis there a partcalar patch i should check i have16:23
sean-k-mooneyi have the Merge "Revert "libvirt: slow live-migration to ensure network is ready"" patch16:25
dansmiththat16:25
dansmithand did you turn on the manager waiter?16:25
dansmiththe thing I'm forcing to true in this test patch:  https://review.openstack.org/59177516:26
sean-k-mooneylive_migration_wait_for_vif_plug = True16:26
dansmithyep16:26
dansmithcool16:26
sean-k-mooneyin the compute section. ya i copied the stuff form the gate job16:26
dansmithsweet16:27
dansmiththat's super helpful thanks :)16:27
kosamarasean-k-mooney: I didn't get your affinity/anti-affinity comment. Affinity has meaning per physical device?16:27
*** Sundar_ has quit IRC16:27
sean-k-mooneylivemigration between ovs and linux bridge is not workigk. looks like we missed a few thing on the neutron side. im goint to test live migration between host with and without iptable firewall driver next16:27
*** s10 has quit IRC16:28
sean-k-mooneykosamara: in the granualar resouce provider spec we expcitly say if different resouce request groups are allowed to come form teh same resouce provier or may not come form the same resource provider16:29
sean-k-mooneykosamara: this is so we can model affinity or anti afinity for different usecases16:29
sean-k-mooneykosamara: i dont think you have to worry about that in your spec. its more on the consumtion side then whitelisting16:30
kosamarasean-k-mooney: right. The question is between modelling each physical device as an RP vs aggregating inventories with same traits into 1 RP. Does affinity affect this?16:30
sean-k-mooneyif each device is a seperate RP we can say i want 2 devices from teh same resouce provider. e.g. 2 dispaly heads from the same gpu16:32
sean-k-mooneywell you might be able to i would have to think about it.16:33
openstackgerritEric Fried proposed openstack/nova master: Remove blacklisted py3 xen tests  https://review.openstack.org/59141916:34
kosamaraYou have a good point there. Also, I don't know if anyone has a use for sound devices, but some GPUs provide 2 pci devices (functions), a VGA and a sound, that can't be given to separate VMs.16:34
kosamaraAlso relevant for supporting NVlink, which appears as many individual pci devices16:35
sean-k-mooneykosamara: ya i would hope this should effect  how we model the config file but will effect how the device are consumed from placement.16:36
sean-k-mooneyi mean i could but i guess it comes down to how granular things are16:37
kosamarasean-k-mooney for sure if there is a use case for requesting 2 display heads from the *same physical device*, aggregating would break it.16:39
*** ccamacho has quit IRC16:39
sean-k-mooneykosamara: you also have the inverse. i want to dispaly heads from different devices. which not aggregating make much more complicated :)16:40
sean-k-mooneya better example of the anti afinity is sriov VF for nics. i want to ensure that they comre form 2 differnet phyical cards for HA16:42
kosamaraagain aggregating would break this16:42
sean-k-mooneynot how aggreting is propsed to work the aggreation point for VFs is the PR so one RP per pf and each PF RP would have an inventory of VFs16:46
sean-k-mooney/PR/PF/16:46
mdboothmelwitt: Incidentally, I'm not actually convinced that adding destroy_disks_on_failure to spawn is right either. We already have a functional entry point to the driver for this case: rebuild.16:47
kosamaraaggregating: I meant many PFs -> 1 RP16:47
mdboothmelwitt: Still undecided.16:47
sean-k-mooneykosamara: if the PF have no VF the you can safly aggreate them at the numa node level assume the PF are other wise identical16:48
sean-k-mooneykosamara: anyway we dont need to cover all the edgecases on irc :)16:48
kosamarasean-k-mooney: ok, etherpad :)16:49
*** gbarros has quit IRC16:51
melwittmdbooth: okay16:54
*** udesale has quit IRC16:54
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen: remove_provider_from_instance_allocation  https://review.openstack.org/59178416:57
efriedjroll, jaypipes: Can this test be deleted at this point? https://github.com/openstack/nova/blob/61f854ff6445dd7bc797fdde7361e47f3dc3b1bd/nova/tests/functional/compute/test_resource_tracker.py#L21517:02
mdboothmelwitt: I'm not keen on anything involving task state, though: it's a wild west. It's basically impossible to verify, and almost guaranteed to suffer bitrot.17:02
jaypipesefried: should be able to, yes.17:03
jaypipesefried: double check with mriedem and dansmith17:03
efriedjaypipes: ight, will do so via review I guess.17:03
jrollI thought we had a patch out to drop those things17:03
mdboothI think it's robust to verify task state set vs not set, anything else is asking for trouble.17:03
melwittmdbooth: yeah. that would only be for the sake of the backport. I agree it's not ideal, but the copying of code from manager to driver is more prone to bitrot IMHO17:03
dansmithefried: I'm sure we can because we don't technically support that mix of versions anyway17:03
mdboothmelwitt: Yeah. I also don't love that.17:04
*** derekh has quit IRC17:04
efrieddansmith: ack, proposing...17:04
mdboothmelwitt: I think the behaviour being tested by the regression test, namely relying on instance_on_disk() is ok.17:04
mdboothThe principal being: if we created it, we should clean it up. If we didn't, we shouldn't.17:05
openstackgerritEric Fried proposed openstack/nova master: Remove obsolete func test_ironic_ocata_to_pike  https://review.openstack.org/59178517:05
mdboothI think that's moderately robust, although as noted in the comment I think we can also do better with a bit more work.17:05
efrieddansmith, jaypipes, jroll, mriedem: ^17:05
*** adrianc has quit IRC17:06
jrollefried: thanks. you reminded me I need to pick this one back up too https://review.openstack.org/#/c/565841/17:07
jaypipesefried: terrible code. -3.17:07
sean-k-mooneymriedem: just an FYI livemigratin between an ovs host with ip tables firewall and openvswtich firewall  and back appear to work correctly.17:07
efriedjroll: I was just looking at that a few minutes ago.17:07
sean-k-mooneymriedem: im going to do a bit more testing but the xml is corerctly updated so i just going to check that the firewall actully works during the live migration17:08
jrollheh17:08
* jroll does the thing17:08
*** Swami has joined #openstack-nova17:10
*** Bhujay has quit IRC17:22
*** psachin has quit IRC17:22
mdboothdansmith melwitt: Rather than changing rebuild specifically, how about if I changed spawn() in the libvirt driver such that, unconditionally, it only deletes the instance directory if it created it, and it only deletes disks if it created them.17:23
*** dpawlik has joined #openstack-nova17:24
mdboothspawn() is called by compute manager for: boot, unshelve, rebuild.17:24
mdboothFor the first 2 nothing should exist already, but the current behaviour is to ignore that.17:24
mdboothSo the only case where it might legitimately exist already is rebuild on shared storage.17:25
*** dpawlik has quit IRC17:29
*** hongbin has quit IRC17:35
*** gbarros has joined #openstack-nova17:37
sean-k-mooneymdbooth: for rebuild on shared stroage the disk might exists and the instance directory but we have to recreate teh disk from the new iamage anyway right17:37
*** janki has joined #openstack-nova17:45
*** holser_ has quit IRC17:45
openstackgerritJim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources  https://review.openstack.org/56584117:47
jrollefried: I think that does it ^17:47
melwittmdbooth: my initial thought is, I'm not sure that sounds good for a backport, but rather a possible refactor. I'm not sure what all would be involved there or any gotchas17:48
openstackgerritDan Smith proposed openstack/nova stable/queens: DNM: Debug patch to test live migration waiting  https://review.openstack.org/59177517:48
*** holser_ has joined #openstack-nova17:49
*** holser_ has quit IRC17:49
*** holser_ has joined #openstack-nova17:50
*** holser_ has quit IRC18:01
*** eharney has quit IRC18:12
*** dtantsur is now known as dtantsur|afk18:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen support for put allocations  https://review.openstack.org/59164718:15
openstackgerritBalazs Gibizer proposed openstack/nova master: Consumer gen: remove_provider_from_instance_allocation  https://review.openstack.org/59178418:16
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: support claim_resources  https://review.openstack.org/58366718:16
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: move_allocations  https://review.openstack.org/59181018:16
openstackgerritBalazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases  https://review.openstack.org/59181118:16
openstackgerritMerged openstack/nova stable/rocky: Revert "libvirt: slow live-migration to ensure network is ready"  https://review.openstack.org/59127518:19
*** hemna_ has joined #openstack-nova18:35
efriedjroll: Soft -1 on the reno (as well as pep8)18:36
*** sapd1 has quit IRC18:38
jrollughhhhhhh18:41
jrollthanks efried18:43
openstackgerritJim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources  https://review.openstack.org/56584118:44
efriedmuch clearer, thanks jroll18:51
jroll:)18:51
*** eharney has joined #openstack-nova19:17
*** jaosorior has quit IRC19:45
*** cdent has joined #openstack-nova19:53
*** jaosorior has joined #openstack-nova19:58
*** awaugama has quit IRC20:07
*** slaweq_ has quit IRC20:09
*** slaweq_ has joined #openstack-nova20:11
*** gbarros has quit IRC20:11
*** slaweq_ has quit IRC20:15
cfriesenmaybe odd question, but if I boot an instance from a flavor with trait:HW_CPU_X86_AVX2=required, the exact CPU model chosen seems to depend on the nova-compute config.  Suppose I then try to live-migrate...I'm guaranteed that the dest node will support AVX2, but Is there somewhere that we verify that the dest node can support the same CPU model as is currently being used by the running VM?20:19
*** gbarros has joined #openstack-nova20:21
cfriesennever mind, found the code.  the scheduler filters don't check, but check_can_live_migrate_destination() should catch it.20:31
*** slaweq_ has joined #openstack-nova20:32
*** slaweq_ has quit IRC20:36
openstackgerritJay Pipes proposed openstack/nova master: DNM: test possible deadlock cause  https://review.openstack.org/59184520:42
*** hemna_ has quit IRC20:47
*** gbarros has quit IRC20:56
*** jaosorior has quit IRC20:56
*** hemna_ has joined #openstack-nova20:56
*** slaweq_ has joined #openstack-nova21:11
*** slaweq_ has quit IRC21:15
openstackgerritChris Dent proposed openstack/nova master: Add explanatory prefix to post_test_perf output  https://review.openstack.org/59185021:15
openstackgerritChris Dent proposed openstack/nova master: DNM: test possible deadlock cause  https://review.openstack.org/59184521:19
efriedcdent: Love it.21:21
* cdent is glad21:21
cdentI suppose once we've all memorized those words we can read right over them21:22
*** slagle has quit IRC21:23
efriedcdent: One concern is tying that behavior to a specific version of placeload.21:25
efriedPerhaps a TODO to vet that periodically.21:25
cdentbut where will we put the todo to remind us to look for todos?21:26
openstackgerritDmitry Sutyagin proposed openstack/nova master: Implement disabling KSM via flavor's extra specs  https://review.openstack.org/59185421:27
openstackgerritEric Fried proposed openstack/nova master: PowerVM: update_provider_tree() (compatible)  https://review.openstack.org/46856021:30
efriededmondsw, jaypipes: ^21:30
*** s10 has joined #openstack-nova21:30
openstackgerritJim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources  https://review.openstack.org/56584121:32
*** slaweq_ has joined #openstack-nova21:32
jrollefried: jaypipes: ^ had a syntax error in release note :(21:32
efriedjroll: Wish somebody had been paying attention to your release note21:33
jrollheh21:33
jrollI did your suggestion too :)21:33
efriedd'oh, I should have caught that.21:33
efriedthanks.21:33
efriedjroll: +A21:34
jrollthanks21:35
*** slaweq_ has quit IRC21:36
*** slaweq_ has joined #openstack-nova21:53
*** itlinux has quit IRC21:54
*** slaweq_ has quit IRC21:57
*** cdent has quit IRC21:59
*** janki has quit IRC22:01
*** slaweq_ has joined #openstack-nova22:11
*** slaweq_ has quit IRC22:15
*** burt has quit IRC22:19
*** erlon has quit IRC22:21
*** s10 has quit IRC22:23
*** s10 has joined #openstack-nova22:23
*** s10 has quit IRC22:24
*** s10 has joined #openstack-nova22:24
*** s10 has quit IRC22:24
*** s10 has joined #openstack-nova22:25
*** s10 has quit IRC22:25
*** s10 has joined #openstack-nova22:26
*** s10 has quit IRC22:26
*** s10 has joined #openstack-nova22:26
*** s10 has quit IRC22:27
*** slaweq_ has joined #openstack-nova22:32
melwittmriedem: fyi added a question to the RC todos etherpad about translations22:35
*** slaweq_ has quit IRC22:36
*** imacdonn has quit IRC22:38
*** imacdonn has joined #openstack-nova22:38
openstackgerritEric Fried proposed openstack/nova master: [placement] Regex consts for placement schema  https://review.openstack.org/59186322:40
*** slaweq_ has joined #openstack-nova22:53
*** _ix has quit IRC22:57
*** slaweq_ has quit IRC22:57
*** mvkr has joined #openstack-nova23:05
*** slaweq_ has joined #openstack-nova23:32
*** slaweq_ has quit IRC23:36
*** Swami has quit IRC23:45
*** slaweq_ has joined #openstack-nova23:53
*** slaweq_ has quit IRC23:57

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