Thursday, 2019-01-03

*** hongbin has quit IRC00:03
*** macza has quit IRC00:08
*** Jeffrey4l has joined #openstack-nova00:08
*** wolverineav has quit IRC00:33
*** mlavalle has quit IRC00:35
*** amodi has quit IRC00:50
*** wolverineav has joined #openstack-nova00:53
*** wolverineav has quit IRC00:57
*** wolverineav has joined #openstack-nova00:57
*** brault has quit IRC01:08
*** dave-mccowan has quit IRC01:23
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Spec for cross-cell resize  https://review.openstack.org/61603701:23
mriedemmelwitt: dansmith: ^ round deux01:24
*** mriedem has quit IRC01:25
*** markvoelker has quit IRC01:26
*** artom has joined #openstack-nova01:26
*** markvoelker has joined #openstack-nova01:26
*** markvoelker has quit IRC01:31
*** wolverineav has quit IRC01:33
*** wolverineav has joined #openstack-nova01:35
*** macza has joined #openstack-nova01:39
*** ttsiouts has quit IRC01:40
*** efried has joined #openstack-nova01:47
*** efried has quit IRC01:50
*** wolverineav has quit IRC01:55
*** wolverineav has joined #openstack-nova02:06
*** mhen has quit IRC02:06
*** mhen has joined #openstack-nova02:12
*** wolverineav has quit IRC02:18
*** ttsiouts has joined #openstack-nova02:18
*** ttsiouts has quit IRC02:23
*** markvoelker has joined #openstack-nova02:27
*** hongbin has joined #openstack-nova02:29
*** mschuppert has quit IRC02:44
*** dave-mccowan has joined #openstack-nova02:48
*** markvoelker has quit IRC02:50
*** markvoelker has joined #openstack-nova02:50
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225603:08
*** ttsiouts has joined #openstack-nova03:17
*** sapd1 has quit IRC03:20
*** ttsiouts has quit IRC03:22
openstackgerritYongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225603:22
*** owalsh has quit IRC03:25
*** cfriesen has quit IRC03:30
*** sapd1 has joined #openstack-nova03:33
*** psachin has joined #openstack-nova03:37
*** owalsh has joined #openstack-nova03:39
*** igordc has quit IRC04:02
*** wolverineav has joined #openstack-nova04:11
*** tetsuro has joined #openstack-nova04:13
alex_xujaypipes: thanks for the response04:16
*** tetsuro has quit IRC04:16
*** tetsuro has joined #openstack-nova04:33
*** udesale has joined #openstack-nova04:36
*** sridharg has joined #openstack-nova04:36
*** dave-mccowan has quit IRC04:56
*** mkrai has joined #openstack-nova04:57
*** psachin has quit IRC05:00
*** sapd1 has quit IRC05:03
*** hongbin has quit IRC05:05
*** whoami-rajat has joined #openstack-nova05:05
*** tetsuro has quit IRC05:17
*** markvoelker has quit IRC05:17
*** tetsuro has joined #openstack-nova05:17
*** ttsiouts has joined #openstack-nova05:18
*** ratailor has joined #openstack-nova05:22
*** ttsiouts has quit IRC05:23
*** wolverineav has quit IRC05:26
*** macza has quit IRC05:27
*** macza has joined #openstack-nova05:31
*** psachin has joined #openstack-nova05:34
*** macza has quit IRC05:38
*** yedongcan has joined #openstack-nova05:44
*** macza has joined #openstack-nova05:47
*** markvoelker has joined #openstack-nova05:49
*** macza has quit IRC05:56
*** wolverineav has joined #openstack-nova06:07
*** sapd1 has joined #openstack-nova06:11
openstackgerritOpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata  https://review.openstack.org/62810906:17
*** markvoelker has quit IRC06:22
*** markvoelker has joined #openstack-nova06:23
*** macza has joined #openstack-nova06:25
*** markvoelker has quit IRC06:27
*** macza has quit IRC06:30
*** udesale has quit IRC06:33
*** udesale has joined #openstack-nova06:34
*** jbernard has quit IRC06:35
*** jbernard has joined #openstack-nova06:39
*** ccamacho has quit IRC06:41
*** brinzhang has joined #openstack-nova06:41
*** macza has joined #openstack-nova06:41
brinzhangHi, when I boot an instance with a network A, and A has two or more IPv6 subnet, then the instance will be allocated two or more ipv6, I didn't figure this design idea. Can anyone explain it?06:44
brinzhangIf the network A has more IPv4 subnet, there will be allocated one IP(ipv4).06:45
alex_xubrinzhang: that sounds like a question for neutron, not nova06:48
alex_xubrinzhang: since nova only pass the network to neturon, the neutron done all the logic, that is scope of neutron I think06:49
brinzhangalex_xu: Thanks. I was asked in openstack-neutron channel too.06:50
alex_xubrinzhang: np, cool06:51
*** macza has quit IRC06:51
*** wolverineav has quit IRC06:53
*** wolverineav has joined #openstack-nova06:54
*** wolverineav has quit IRC06:58
*** rcernin has quit IRC07:09
*** sapd1 has quit IRC07:10
*** ttsiouts has joined #openstack-nova07:19
*** ccamacho has joined #openstack-nova07:24
*** ttsiouts has quit IRC07:24
*** ccamacho has quit IRC07:24
*** ccamacho has joined #openstack-nova07:25
*** sapd1 has joined #openstack-nova07:47
openstackgerritMerged openstack/nova master: Add functional regression recreate test for bug 1790204  https://review.openstack.org/61912307:53
openstackbug 1790204 in OpenStack Compute (nova) "Allocations are "doubled up" on same host resize even though there is only 1 server on the host" [High,Triaged] https://launchpad.net/bugs/179020407:53
*** dpawlik has joined #openstack-nova08:01
*** luksky has joined #openstack-nova08:02
*** lpetrut has joined #openstack-nova08:03
*** moshele has joined #openstack-nova08:05
*** ondrejme has joined #openstack-nova08:06
*** slaweq has joined #openstack-nova08:06
*** ccamacho has quit IRC08:09
yedongcanbrinzhang: Please refer this: https://specs.openstack.org/openstack/neutron-specs/specs/kilo/multiple-ipv6-prefixes.html08:21
*** markvoelker has joined #openstack-nova08:23
openstackgerritRui Zang proposed openstack/nova-specs master: support virtual persistent memory  https://review.openstack.org/60159608:28
*** maciejjozefczyk has joined #openstack-nova08:28
*** tetsuro has quit IRC08:36
*** skatsaounis has joined #openstack-nova08:38
brinzhangyedongcan: Thank you, I will look at this :P08:44
*** tetsuro has joined #openstack-nova08:45
*** xek has joined #openstack-nova08:51
*** ccamacho has joined #openstack-nova08:54
*** dtantsur|afk is now known as dtantsur08:59
*** whoami-rajat has quit IRC09:13
*** derekh has joined #openstack-nova09:18
*** ttsiouts has joined #openstack-nova09:20
openstackgerritDaniel Pawlik proposed openstack/nova stable/queens: Remove stale pip-missing-reqs tox test  https://review.openstack.org/62812809:23
*** erlon has joined #openstack-nova09:24
*** ttsiouts has quit IRC09:24
*** whoami-rajat has joined #openstack-nova09:25
*** sapd1 has quit IRC09:27
*** amorin has quit IRC09:30
*** ttsiouts has joined #openstack-nova09:30
*** lpetrut has quit IRC09:32
openstackgerritJie Li proposed openstack/nova master: Add compute version 37 to support volume backed server rebuild  https://review.openstack.org/62589309:44
*** sapd1 has joined #openstack-nova09:44
*** sayalilunkad has joined #openstack-nova10:04
*** ttsiouts has quit IRC10:33
*** smcginnis_away is now known as smcginnis10:34
*** tetsuro has quit IRC10:34
*** mkrai has quit IRC10:35
*** luksky has quit IRC10:49
*** sapd1 has quit IRC10:59
*** amorin has joined #openstack-nova10:59
*** janki has joined #openstack-nova11:21
*** janki has quit IRC11:22
*** janki has joined #openstack-nova11:23
*** udesale has quit IRC11:26
*** tbachman has quit IRC11:39
*** cdent has joined #openstack-nova11:41
*** janki has quit IRC11:41
*** whoami-rajat has quit IRC11:43
*** janki has joined #openstack-nova11:44
openstackgerritChris Dent proposed openstack/nova master: Delete the placement code  https://review.openstack.org/61821511:55
*** dosaboy has quit IRC12:02
*** ratailor has quit IRC12:04
*** Kimmo_ has joined #openstack-nova12:05
*** dosaboy has joined #openstack-nova12:07
openstackgerritYikun Jiang proposed openstack/nova master: WIP: Per aggregate scheduling weight  https://review.openstack.org/62816312:21
*** luksky has joined #openstack-nova12:24
*** brinzhang has quit IRC12:38
*** ttsiouts has joined #openstack-nova12:49
*** szaher has joined #openstack-nova12:50
*** dpawlik has quit IRC13:03
*** tbachman has joined #openstack-nova13:06
maciejjozefczykhey!13:08
maciejjozefczykmaybe somebody has quick how-to run grenade jobs under my patch?13:08
*** markvoelker has quit IRC13:18
*** dave-mccowan has joined #openstack-nova13:18
jaypipesalex_xu: see my latest comment on the PMEM spec... do you think it would be better to just use the block mode translation system for PMEM instead of the PMEM namespace system? After reading (most of) the original PMEM spec from Intel, it seems that the block mode setup is the more generic "I'm just a bunch of GB of block storage" resource class versus the PMEM namespace mode...13:20
*** markvoelker has joined #openstack-nova13:20
cdentjaypipes: I thought the reason for not doing that was because of fragmentation? I haven't, however, read the original spec, so I may have no idea what I'm talking about13:23
*** derekh has quit IRC13:24
jaypipescdent: well, unless I'm badly mistaken, the difference is kind of like the difference between a bunch of GB of block storage and a filesystem that might go on a partition of said block storage.13:27
jaypipescdent: one needs to be wiped/cleaned/formatted whereas the other is just the bunch of blocks for storing stuff. it's up to the user to format it how they need.13:27
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416713:28
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160713:28
cdentit's all a bit bewildering. So often we these things I think "is a vm really what you want for this?"13:28
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Rename method _gather_port_ids_and_networks to _get_cached_network_info  https://review.openstack.org/62817513:28
*** whoami-rajat has joined #openstack-nova13:30
jaypipesmaciejjozefczyk: reviewing your patches now.13:32
jaypipesmaciejjozefczyk: I am sincerely sorry. Please see my comment on the bottom patch :(13:44
*** nehaalhat has quit IRC13:54
openstackgerritChris Dent proposed openstack/nova master: WIP: Update nova docs front page for placement removal  https://review.openstack.org/62818313:56
openstackgerritChris Dent proposed openstack/nova master: Remove placement from nova install docs  https://review.openstack.org/62818413:56
openstackgerritChris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs  https://review.openstack.org/62818513:56
openstackgerritChris Dent proposed openstack/nova master: Remove placement from contributor doc  https://review.openstack.org/62818613:56
maciejjozefczykjaypipes:  :) easy13:57
maciejjozefczykI'm thinking now how previous patch failed on this: http://logs.openstack.org/67/614167/9/check/neutron-grenade/105121b/logs/grenade.sh.txt.gz13:57
*** lbragstad has joined #openstack-nova13:58
maciejjozefczykjaypipes: maybe its related to the mixed context that you found in online_data_migrations script13:58
alex_xujaypipes: the PMEM namespace is the way suggested by the qemu, it is best performance way.14:02
alex_xujaypipes: block mode need a driver layer in the kernel, that isn't real direct access. I asked Rui reply on the spec, I think he can answer this better14:03
*** mmethot has joined #openstack-nova14:03
*** jistr is now known as jistr|mtg14:04
*** derekh has joined #openstack-nova14:04
alex_xujaypipes: also in the beginning, we take a look at another way, build a pmem aware filesystem on the pmem namespace, but after talk with virtualization guys, they told me that is a way kernel push forward in the before, but there is bug can't be fixed in the kernel, so pmem namespace is only way for now14:05
alex_xuthen the  dream of no fragmentation is broken14:06
jaypipesalex_xu: ack14:12
*** mriedem has joined #openstack-nova14:14
*** munimeha1 has joined #openstack-nova14:18
*** mriedem has quit IRC14:19
*** dpawlik has joined #openstack-nova14:20
*** mriedem has joined #openstack-nova14:21
*** udesale has joined #openstack-nova14:22
*** dpawlik has quit IRC14:24
*** dpawlik has joined #openstack-nova14:24
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Exclude build request marker from server listing  https://review.openstack.org/62658514:27
*** dpawlik has quit IRC14:28
*** rui-zang has joined #openstack-nova14:28
*** dpawlik has joined #openstack-nova14:28
rui-zangjaypipes, I just replied in the pmem spec on why "block mode" is not recommended14:30
rui-zangIf you still have concerns or I do not express it clearly, I am here :)14:32
alex_xurui-zang: being here whole night :)14:33
rui-zangalex_xu, ->< only have a battery less than 2 hours14:33
*** burt has joined #openstack-nova14:39
maciejjozefczykjaypipes: ok, found what's wrong with gates :)14:41
HW-Peterhey14:48
HW-PeterIs there any documentation for setting up a Multi Cell CellsV2 Environment14:48
HW-PeterWere running around 500 nodes per location, and are wanting to create a 2nd cell to help with scale14:49
*** macza has joined #openstack-nova14:49
*** awaugama has joined #openstack-nova14:50
HW-PeterThe amount of cells v1/v2 overlap in current documentation makes it difficult to see if the information presented relates to cells v1 or cellsv214:50
*** lbragstad has quit IRC14:50
HW-PeterThe amount of cells v1/v2 overlap in current documentation makes it difficult to see if the information presented relates to cellsv1 or cellsv214:50
*** macza has quit IRC14:51
*** lbragstad has joined #openstack-nova14:53
mriedemHW-Peter: i'm assuming you've seen https://docs.openstack.org/nova/latest/user/cells.html and https://docs.openstack.org/nova/latest/user/cellsv2-layout.html ?14:54
mriedemhttps://docs.openstack.org/nova/latest/user/cells.html#adding-a-new-cell-to-an-existing-deployment14:54
*** jistr|mtg is now known as jistr14:55
mriedemHW-Peter: you setup the new db and mq for the new cell and then create the cell mapping pointing at the new db/mq14:55
mriedemnew computes in that cell will be configured to point at that cell mq/db14:56
mriedemand then you run discover_hosts to populate the new cell db with the compute_nodes records14:56
mriedemif you're trying to shard an existing db to a new cell, that is a bit harder14:57
mriedem"The amount of cells v1/v2 overlap in current documentation" - if you can point me at what you're referring to i can see if the docs can be made more clear14:57
mriedemi know the ops guide is way out of date14:57
dansmithWe probably should be removing cellsv1 words across the board anyway14:58
mriedemi had reorganized the bug user/cells.html doc at one point to move the cells v1 stuff to the bottom so it's not the first thing you see, but abandoned it15:00
mriedem*big15:00
*** hongbin has joined #openstack-nova15:01
dansmithack15:01
dansmithwe should just remove it I think15:01
dpawlikmriedem: Hi, I reply you https://review.openstack.org/#/c/628128/ . If you have time please check :)15:01
mriedemdpawlik: replied15:03
mriedemdansmith: yeah, could - if a cells v1 user needs it to upgrade they can get the rocky docs15:03
dpawlikmriedem: how fast. Thanks15:03
dansmithyeah I mean, we're not going to be able to support upgrade forever, so I think just removing the docs about that probably helps to make it clear15:03
*** rui-zang has quit IRC15:04
mriedemwhen can we just drop cells v1 again?15:05
mriedemi don't see any specific mention of it in the stein ptg etherpad15:05
*** amodi has joined #openstack-nova15:05
*** moshele has quit IRC15:06
mriedemrocky ptg etherpad says we'd drop nova-net before cells v1,15:06
mriedembut stein ptg etherpad says we can't drop nova-net until cells v1 since nova-net only runs if cells v1 is enabled :/15:06
dansmithwell, I think we lost steam because (a) removing n-net is hard and (b) cern is still using n-net15:06
dpawlikmriedem: I can listen in my CI job which tox env should be executed, but if I am alone with that problem, we don;'t need to backport the patch.15:07
mriedemsurely cern just comments out this line for nova-net: https://github.com/openstack/nova/blob/master/nova/cmd/network.py#L4315:07
dansmithmriedem: that's what they're doing, yes.. I mean if we remove it more than that, it'll be an issue15:07
mriedemyeah i know the service itself15:08
*** udesale has quit IRC15:08
mriedemsorrison is on pike now i think he said15:08
mriedemgodaddy is... ?15:08
dansmiththey were neutron anyway15:09
mriedemyeah just wondering if/when/why we can't remove cells v115:10
*** yedongcan has left #openstack-nova15:12
mriedemanywho, we moved the job to experimental queue, maybe next step is dropping docs, and just continue to whittle away at removing it15:13
mriedemmake it less discoverable15:13
*** cfriesen has joined #openstack-nova15:14
*** ivve has joined #openstack-nova15:15
*** wolverineav has joined #openstack-nova15:15
*** Bhujay has joined #openstack-nova15:15
mriedemguess i can take a shot at https://bugs.launchpad.net/openstack-manuals/+bug/180425315:16
openstackLaunchpad bug 1804253 in openstack-manuals "Capacity planning and scaling in Operations Guide - cells information is out of date" [Undecided,Confirmed] - Assigned to lei zhang (zhang-lei)15:16
*** Bhujay has quit IRC15:16
*** ttsiouts has quit IRC15:18
*** wolverineav has quit IRC15:20
*** efried has joined #openstack-nova15:25
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416715:29
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160715:29
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup soft (anti)affinity weight multiplier options  https://review.openstack.org/62820515:35
*** mlavalle has joined #openstack-nova15:37
*** whoami-rajat has quit IRC15:39
*** wwriverrat has joined #openstack-nova15:41
*** xek_ has joined #openstack-nova15:46
*** xek has quit IRC15:48
*** ade_lee has joined #openstack-nova15:51
ade_leehi -- has anyone seen this error before? http://logs.openstack.org/20/621920/1/gate/barbican-simple-crypto-devstack-tempest/eb68a52/logs/screen-n-cpu.txt.gz#_Jan_02_18_35_13_49893115:52
HW-PeterThanks for that15:56
HW-PeterGoing to start testing this today15:57
ade_leeHW-Peter, sorry - was that directed to me?15:59
ade_leemriedem, is the error I posted above related to https://bugs.launchpad.net/nova/+bug/1808247  ?15:59
openstackLaunchpad bug 1808247 in OpenStack Compute (nova) "nova-lvm job failing with ArgsAlreadyParsedError (privsep)" [Medium,Confirmed]15:59
ade_leeI'm seeing .. "ImportError: cannot import name api"16:00
*** mdbooth has joined #openstack-nova16:03
openstackgerritMerged openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances  https://review.openstack.org/60977916:07
*** igordc has joined #openstack-nova16:07
mriedemade_lee: yeah since you're using lvm16:11
mriedemJan 02 18:31:41.561505 ubuntu-xenial-inap-mtl01-0001459779 nova-compute[1225]: DEBUG oslo_service.service [None req-bd4be651-b5b9-4638-86a8-36108a0b464f None None] libvirt.images_type            = lvm {{(pid=1225) log_opt_values /usr/local/lib/python2.7/dist-packages/oslo_config/cfg.py:2575}}16:11
*** tbachman has quit IRC16:15
ade_leemriedem, any workaround/fix for that?  Its blocking some gate tests16:15
mriedemade_lee: no idea,16:15
mriedemit looks like something failing in privsep but i'm not expert on that code,16:15
mriedemmikal is probably the best person to ask,16:16
mriedembut he's in australia16:16
mriedemi also don't know why it's only hitting the lvm configuration16:16
mriedemshould maybe try running with this set to DEBUG https://github.com/openstack/nova/blob/master/nova/config.py#L4616:17
mriedemand see if that helps show where privsep is blowing up16:17
ShilpaSDhi All, doing restacking, but failing with error "neutronclient.Unauthorized", any suggestions?16:18
ade_leemriedem, ok -- will try doing that16:18
ade_leethanks16:18
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: debugging lvm privsep failures  https://review.openstack.org/62821316:24
mriedemade_lee: ^16:24
ade_leemriedem, cool - I'll piggyback off that16:24
*** macza has joined #openstack-nova16:26
*** itlinux has joined #openstack-nova16:40
*** mrjk has joined #openstack-nova16:44
*** erlon has quit IRC16:45
*** erlon has joined #openstack-nova16:46
*** tbachman has joined #openstack-nova16:54
*** ccamacho has quit IRC16:55
openstackgerritChris Dent proposed openstack/nova master: Update nova docs front page for placement removal  https://review.openstack.org/62818316:57
openstackgerritChris Dent proposed openstack/nova master: Remove placement from nova install docs  https://review.openstack.org/62818416:57
openstackgerritChris Dent proposed openstack/nova master: Remove link to placement configuration from nova config docs  https://review.openstack.org/62818516:57
openstackgerritChris Dent proposed openstack/nova master: Remove placement from contributor doc  https://review.openstack.org/62818616:57
*** dpawlik has quit IRC17:00
*** dpawlik has joined #openstack-nova17:00
openstackgerritMatt Riedemann proposed openstack/nova master: Add microversion 2.68 to support for volume-backed server rebuild  https://review.openstack.org/62751817:01
*** dpawlik has quit IRC17:05
openstackgerritMatt Riedemann proposed openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225617:09
*** janki has quit IRC17:13
*** gyee has joined #openstack-nova17:25
*** panda is now known as panda|off17:27
*** luksky has quit IRC17:38
*** panda|off has quit IRC17:44
*** panda has joined #openstack-nova17:48
dansmithmriedem: while we're waiting for confirm, we still have the original instance root disk still on the source, right?17:51
mriedemper v2 of the spec i said we destroy the guest17:52
mriedemon revert we re-spawn it17:52
dansmithdestroy, but does that mean delete too?17:52
mriedemi went the driver.destroy route because that unplugs vifs and disconnects volumes on the source host17:52
dansmithright, saw that17:53
dansmithwas not sure that would delete the image though as I thought there's a flag for that and was thinking maybe you were planning to not17:53
mriedemthat driver.destroy method has a destroy_disks flag which defaults to True,17:54
mriedemwhich deletes the instance path,17:54
mriedemwhich is where the root disk image is right?17:55
dansmithokay let me finish my comment on v2 and you can see what I'm asking17:55
dansmithyes17:55
dansmithmriedem: look at my last two comments just now17:55
mriedemi think the main difference between destroy and migrate_disk_and_power_off is that migrate_disk_and_power_off powers off the instance rather than destroy it (and copies the disks obviously), it also disconnects volumes like destroy, but doesn't unplug vifs for some weird reason17:56
* dansmith nods17:56
*** imacdonn has quit IRC17:59
*** imacdonn has joined #openstack-nova17:59
*** derekh has quit IRC18:03
*** ttsiouts has joined #openstack-nova18:04
dansmithmriedem: added a few more comments, but I think if you fix the image question I'll be ready to +218:04
mriedemreplying to the first set18:04
openstackgerritIfat Afek proposed openstack/nova master: The field instance_name was added to InstanceCreatePayload  https://review.openstack.org/62796718:06
openstackgerritAdrian Chiris proposed openstack/nova master: Add free for claimed, allocated devices  https://review.openstack.org/61612018:09
openstackgerritAdrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile  https://review.openstack.org/60736518:09
openstackgerritAdrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif  https://review.openstack.org/61992918:09
openstackgerritAdrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support  https://review.openstack.org/62011518:09
*** tbachman has quit IRC18:09
*** bnemec has quit IRC18:13
ade_leemriedem, hey is the log with the privsep logging on (I think) -- http://logs.openstack.org/20/621920/2/check/barbican-simple-crypto-devstack-tempest/f013bd9/logs/screen-n-cpu.txt.gz18:17
ade_leemriedem, I do see this -- http://logs.openstack.org/20/621920/2/check/barbican-simple-crypto-devstack-tempest/f013bd9/logs/screen-n-cpu.txt.gz#_Jan_03_17_17_25_53223718:18
ade_leemriedem, and also this -- http://logs.openstack.org/20/621920/2/check/barbican-simple-crypto-devstack-tempest/f013bd9/logs/screen-n-cpu.txt.gz#_Jan_03_17_17_27_63585818:18
openstackgerritAdam Spiers proposed openstack/nova-specs master: AMD SEV: address final feedback received prior to merge  https://review.openstack.org/62825218:21
openstackgerritAdam Spiers proposed openstack/nova-specs master: AMD SEV: address final feedback received prior to merge  https://review.openstack.org/62825218:22
*** whoami-rajat has joined #openstack-nova18:25
*** bnemec has joined #openstack-nova18:26
*** bnemec has quit IRC18:30
*** tbachman has joined #openstack-nova18:31
*** bnemec has joined #openstack-nova18:34
mriedemade_lee: yeah this http://logs.openstack.org/20/621920/2/check/barbican-simple-crypto-devstack-tempest/f013bd9/logs/screen-n-cpu.txt.gz#_Jan_03_17_17_25_52573418:37
mriedemlooks like we have a classic circular import18:38
*** bnemec has quit IRC18:39
mriedemade_lee: yeah it's this change https://github.com/openstack/nova/commit/26521718bdba3bccbf6270e26b76754c26304658#diff-31323cc5d9a493e21871761c5051db7c18:40
mriedemmerged on dec 8 which is when the failures started18:40
mriedemi need to wrap up replying to comments on a spec and then i can push a change to see if i can fix that18:42
dansmithwhy is tripleo the only thing hitting that I wonder?18:42
*** dtantsur is now known as dtantsur|afk18:43
mriedemthat lvm thing? it's not18:44
mriedemthe nova-lvm job is busted by it18:44
dansmithoh okay18:50
dansmithfirst saw it come up in #tripleo so I just assumed18:51
*** wolverineav has joined #openstack-nova18:52
*** bnemec has joined #openstack-nova18:52
*** bnemec has quit IRC18:58
*** wolverineav has quit IRC18:59
mriedemdansmith: replies inline https://review.openstack.org/#/c/616037/ thanks - seems like non-drastic changes19:03
mriedemmaybe i'll change that 'open questions' section to 'known issues' to denote things we know about but don't really care about right now and could be dealt with later if necesary19:04
mriedemjaypipes: so in yonglihe's show server topology spec, your alternative is the GET /servers/{server_id}/topology which would just be NUMA info (not pci devices) and admin-only by default?19:08
jaypipesmriedem: right19:08
jaypipesmriedem: with work on a /devices endpoint done in a separate spec (with efried's help)19:09
mriedemi believe i said in the ML thread that i was ok with /diagnostics or a sub-resource, as long as it wasn't something shoe-horned into the normal response body (GET /servers/{server_id}) or list (GET /servers or GET /servers/detail)19:10
mriedemso whatever19:10
mriedemreplied to the ML19:12
dansmithmriedem: yep, just need to fix the fact that you're deleting all the original copies of the instance in case of rollback and then I'm good19:12
mriedemi eagerly await the spec proposal in 12 months to include GET /servers/{server_id}/topology info in GET /servers/{server_id} and GET /servers/detail output19:12
mriedembecause 2 api calls is hard19:12
jaypipesmriedem: well, by that point, we'll have changed everything to GraphQL so we're all good.19:13
*** bnemec has joined #openstack-nova19:14
*** sridharg has quit IRC19:14
ade_leemriedem, thanks!19:15
*** wolverineav has joined #openstack-nova19:20
*** wolverineav has quit IRC19:21
*** wolverineav has joined #openstack-nova19:21
*** wolverineav has quit IRC19:21
cdentsomething something pabst blue ribbon19:21
*** wolverineav has joined #openstack-nova19:21
*** mriedem has quit IRC19:22
*** mriedem has joined #openstack-nova19:24
*** sean-k-mooney1 has joined #openstack-nova19:25
*** awaugama has quit IRC19:32
openstackgerritsean mooney proposed openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511619:35
openstackgerritChris Dent proposed openstack/nova master: Switch to using os-resource-classes  https://review.openstack.org/62827819:40
*** tbachman has quit IRC19:43
*** awaugama has joined #openstack-nova19:45
*** awaugama has quit IRC19:46
*** awaugama has joined #openstack-nova19:46
*** erlon has quit IRC19:48
*** awaugama has quit IRC19:51
*** awaugama has joined #openstack-nova19:52
*** ttsiouts has quit IRC20:10
*** ttsiouts has joined #openstack-nova20:11
*** ttsiouts has quit IRC20:16
*** ttsiouts has joined #openstack-nova20:17
*** ccamacho has joined #openstack-nova20:20
*** ccamacho has quit IRC20:22
*** tbachman has joined #openstack-nova20:28
*** whoami-rajat has quit IRC20:35
*** takashin has joined #openstack-nova20:37
*** awaugama has quit IRC20:38
cfriesenwhen advertising new traits (specifically the COMPUTE_SECURITY_TPM_2_0 trait), does the trait need to be added to some standard list first?20:38
cdentcfriesen: you mean placement related traits or other kinds of traits?20:40
cfriesenplacement20:41
cfriesendoes it need to get added to os_traits first or something?20:41
cdentto create a new standard trait it gets added to the os-traits lib: http://git.openstack.org/cgit/openstack/os-traits20:41
cdentand then the code which uses the added trait depends on the release of the lib that includes it20:41
cfriesenokay, makes sense.  for the one I mentioned above, would that go in os_traits/compute/security.py then?  with TRAITS=['TPM_2_0"] as the content?20:42
cdentlemme confirm, one sec20:43
sean-k-mooney1cfriesen: any trait not defiend in os-traitns must start with CUSTOM_20:43
melwitto/20:43
cfriesensean-k-mooney1: yeah...the intent is for this to be a standard trait, so I think we need to add it to os_traits20:43
cdentcfriesen: simple answer to you where and content question is "yes"20:43
sean-k-mooney1yes you do i had a patch for that i abandons over a year ago20:44
cfriesencdent: sweet, thanks20:44
cdentyou're welcome20:44
sean-k-mooney1cfriesen: https://review.openstack.org/#/c/514712/3/os_traits/hw/platform/security.py20:45
sean-k-mooney1melwitt: o/ happy new year20:46
melwittsean-k-mooney1: happy new year20:46
cdentsean-k-mooney1, cfriesen is it COMPUTE or HW?20:47
cfriesenso in the spec we have it as COMPUTE_SECURITY_TPM_2_020:47
cfriesenat the time of sean's change I don't think the "compute" subdirectory existed yet20:47
sean-k-mooney1cdent: my patch was from 2017 so the example is in the wong folder but as cfriesen said the spec says COMPUTE_SECURITY_TPM_2_020:48
*** psachin has quit IRC20:48
mriedemmelwitt: i'm out next week so if you want me to review the counting quotas from placement spec after updates, you probably need to make those soonish https://review.openstack.org/#/c/509042/20:50
mriedemi made a lot of comments but i'm not sure there is anything earth shattering in there20:51
sean-k-mooney1cdent: i tink where we left the disucstion was compute traits did not require hardware an hypervior that can emulate a tpm can advertise COMPUTE_SECURITY_TPM_2_0 but HW_SECURITY_TPM_2_0 would be resrved for futur use to signify that the host has a tpm that supports v220:51
melwittmriedem: thank you for the reminder. I'll go through and reply/update today20:51
melwittnova meeting in 4 minutes20:56
*** moshele has joined #openstack-nova21:01
mriedemdansmith: forgot to ask earlier but was hoping you could reply to http://lists.openstack.org/pipermail/openstack-discuss/2018-December/001112.html and if there is compromise we can update the spec and move forward before the freeze21:02
dansmithmriedem: ugh21:07
*** xek_ has quit IRC21:07
dansmithmriedem: can we discuss tomorrow? I'm guessing I'm not really going to be happy with any of the options, but I need to page all that in before I will know I guess21:08
mriedemsure21:09
openstackgerritMatt Riedemann proposed openstack/nova master: Fix circular import in nova.privsep.utils  https://review.openstack.org/62830221:21
openstackgerritMatt Riedemann proposed openstack/nova master: Run nova-lvm job on nova/privsep/* changes  https://review.openstack.org/62830621:43
mriedemade_lee: ^21:46
cfriesenI'm looking for input on https://review.openstack.org/#/c/618542...for the "quota:" stuff that is defined in https://docs.openstack.org/nova/latest/user/flavors.html but might only apply to one or two hypervisors, does it make sense to enforce the rules even if the instance might end up on a hypervisor that ignores those extra-specs?21:47
sean-k-mooney1i was going to mention https://review.openstack.org/#/c/591607/ for open discussion. melwitt: looks like the ordering was corrected21:47
sean-k-mooney1ah the parent patch has a -1 while they work on functional tests21:48
sean-k-mooney1nevermind21:49
melwittsean-k-mooney1: ah, ok21:50
melwittI saw some discussion in gerrit21:51
*** erlon has joined #openstack-nova21:51
ade_leemriedem, thanks!21:52
*** cdent has quit IRC21:56
*** ttsiouts has quit IRC21:56
dansmithmriedem: there, I replied21:57
mriedem<321:58
*** slaweq has quit IRC22:11
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3)  https://review.openstack.org/57410422:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4)  https://review.openstack.org/57410622:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5)  https://review.openstack.org/57411022:20
openstackgerritMatt Riedemann proposed openstack/nova master: Fix circular import in nova.privsep.utils  https://review.openstack.org/62830222:21
openstackgerritMatt Riedemann proposed openstack/nova master: Run nova-lvm job on nova/privsep/* changes  https://review.openstack.org/62830622:21
*** itlinux has quit IRC22:24
*** tbachman has quit IRC22:26
*** efried has quit IRC22:35
*** efried has joined #openstack-nova22:35
*** efried has quit IRC22:39
*** rcernin has joined #openstack-nova22:42
*** moshele has quit IRC23:17
*** mmethot has quit IRC23:22
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Spec for cross-cell resize  https://review.openstack.org/61603723:26
mriedemround trois23:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6)  https://review.openstack.org/57411323:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7)  https://review.openstack.org/57497423:29
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8)  https://review.openstack.org/57531123:29
*** owalsh_ has joined #openstack-nova23:38
*** owalsh has quit IRC23:40
*** owalsh_ has quit IRC23:42
*** owalsh has joined #openstack-nova23:43
*** munimeha1 has quit IRC23:46
openstackgerritMerged openstack/nova-specs master: Remove doc about metadata from 'share-pci-between-numa-nodes'  https://review.openstack.org/61980723:48
*** hongbin has quit IRC23:53
*** macza has quit IRC23:54
*** mriedem has quit IRC23:58

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