Tuesday, 2020-01-14

*** TxGirlGeek has quit IRC00:04
*** vishalmanchanda has joined #openstack-nova00:06
*** TxGirlGeek has joined #openstack-nova00:07
*** macz has quit IRC00:17
*** jmlowe has quit IRC00:25
*** jmlowe has joined #openstack-nova00:26
*** jmlowe has quit IRC00:26
*** brault has quit IRC00:33
*** brault has joined #openstack-nova00:33
*** gentoorax has quit IRC00:46
*** gentoorax has joined #openstack-nova00:50
*** igordc has joined #openstack-nova00:51
*** tkajinam has quit IRC00:56
*** tkajinam has joined #openstack-nova00:59
*** Liang__ has joined #openstack-nova01:10
*** brinzhang has joined #openstack-nova01:13
*** brinzhang_ has quit IRC01:17
*** nicolasbock has quit IRC01:18
*** zhanglong has joined #openstack-nova01:23
*** TxGirlGeek has quit IRC01:28
*** gyee has quit IRC01:34
*** yedongcan has joined #openstack-nova01:36
*** igordc has quit IRC01:45
*** dviroel has quit IRC01:57
*** Liang__ is now known as LiangFang02:26
*** LiangFang has quit IRC02:32
*** Liang__ has joined #openstack-nova02:32
*** tetsuro has joined #openstack-nova02:33
*** Liang__ has quit IRC02:34
*** Liang__ has joined #openstack-nova02:34
*** tkajinam has quit IRC02:36
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree  https://review.opendev.org/67652202:37
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346002:37
*** davee_ has quit IRC02:38
*** davee__ has quit IRC02:38
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree  https://review.opendev.org/67652202:48
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346002:48
*** davee_ has joined #openstack-nova02:55
*** davee__ has joined #openstack-nova02:55
*** tkajinam has joined #openstack-nova03:02
*** awalende has joined #openstack-nova03:24
*** awalende has quit IRC03:29
*** psachin has joined #openstack-nova03:36
*** hongbin has joined #openstack-nova03:50
*** macz has joined #openstack-nova03:56
*** brault has quit IRC03:58
*** psachin has quit IRC04:09
*** bhagyashris has joined #openstack-nova04:18
*** bhagyashris_ has joined #openstack-nova04:20
*** udesale has joined #openstack-nova04:21
*** jhesketh has quit IRC04:23
*** bhagyashris has quit IRC04:23
*** jhesketh has joined #openstack-nova04:24
*** sapd1_x has joined #openstack-nova04:25
*** rchurch_ has quit IRC04:34
*** rchurch has joined #openstack-nova04:35
*** TxGirlGeek has joined #openstack-nova04:52
*** hongbin has quit IRC05:05
*** macz has quit IRC05:06
*** macz has joined #openstack-nova05:07
*** hongbin has joined #openstack-nova05:14
*** hongbin has quit IRC05:19
*** gentoorax has quit IRC05:27
*** gentoorax has joined #openstack-nova05:27
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-nova05:34
*** ratailor has joined #openstack-nova05:38
*** ratailor_ has joined #openstack-nova05:42
*** ratailor has quit IRC05:45
*** lvbin02 has joined #openstack-nova05:50
*** sapd1_x has quit IRC05:53
*** lvbin01 has quit IRC05:53
*** lvbin01 has joined #openstack-nova05:55
*** lvbin02 has quit IRC05:55
*** ociuhandu has joined #openstack-nova05:56
*** ociuhandu has quit IRC06:00
*** macz has quit IRC06:06
*** pcaruana has quit IRC06:17
openstackgerritBrin Zhang proposed openstack/nova master: Populate 'updated_at' when aggregate is updated  https://review.opendev.org/58027106:27
*** yedongcan has quit IRC06:34
*** bhagyashris_ has quit IRC06:38
*** bhagyashris_ has joined #openstack-nova06:42
*** hamzy has quit IRC06:49
*** hamzy has joined #openstack-nova06:50
*** tetsuro has quit IRC06:53
*** tetsuro has joined #openstack-nova06:54
*** ccamacho has quit IRC07:00
*** TxGirlGeek has quit IRC07:05
*** ralonsoh has joined #openstack-nova07:12
*** tesseract has joined #openstack-nova07:41
*** bhagyashris_ is now known as bhagyashris07:45
openstackgerritsean mooney proposed openstack/nova master: Libvirt: report storage bus traits  https://review.opendev.org/66691407:47
*** ociuhandu has joined #openstack-nova07:52
*** ociuhandu has quit IRC07:52
*** rcernin has quit IRC07:52
*** ociuhandu has joined #openstack-nova07:55
*** damien_r has joined #openstack-nova07:59
*** damien_r has quit IRC07:59
brinzhanggibi: are you around?08:00
*** damien_r has joined #openstack-nova08:00
*** slaweq has joined #openstack-nova08:02
*** maciejjozefczyk has joined #openstack-nova08:06
*** awalende has joined #openstack-nova08:06
*** ircuser-1 has joined #openstack-nova08:07
*** tosky has joined #openstack-nova08:08
*** bhagyashris has quit IRC08:11
*** zbr|rover has quit IRC08:14
*** zbr has joined #openstack-nova08:15
*** yedongcan has joined #openstack-nova08:16
openstackgerritAlexandre arents proposed openstack/nova master: Avoid allocation leak when deleting instance stuck in BUILD  https://review.opendev.org/70236808:20
*** ccamacho has joined #openstack-nova08:20
*** dpawlik has quit IRC08:22
*** tkajinam has quit IRC08:24
*** bhagyashris has joined #openstack-nova08:26
*** dpawlik has joined #openstack-nova08:27
*** iurygregory has joined #openstack-nova08:37
*** Liang__ is now known as LiangFang08:42
openstackgerritLee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue  https://review.opendev.org/70081108:46
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue  https://review.opendev.org/70081208:46
openstackgerritLee Yarwood proposed openstack/nova master: WIP compute: Report COMPUTE_BFV_RESCUE and check during rescue  https://review.opendev.org/70142908:46
openstackgerritLee Yarwood proposed openstack/nova master: WIP api: Introduce microverion 2.82 allowing boot from volume rescue  https://review.opendev.org/70143008:46
openstackgerritLee Yarwood proposed openstack/nova master: WIP libvirt: Support boot from volume instance rescue  https://review.opendev.org/70143108:46
lyarwoodalex_xu: https://review.opendev.org/#/c/694063/ - I'm looking for some non-RH nova-specs core review on this if you have time today, ^ PoC code posted above.08:47
openstackgerritBrin Zhang proposed openstack/nova-specs master: Add action event fault details  https://review.opendev.org/69966908:47
*** martinkennelly has joined #openstack-nova08:56
*** bhagyashris has quit IRC08:56
*** pcaruana has joined #openstack-nova08:56
*** ccamacho has quit IRC09:00
openstackgerritwaleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support  https://review.opendev.org/66596509:06
*** xek has joined #openstack-nova09:08
*** avolkov has joined #openstack-nova09:12
*** rpittau|afk is now known as rpittau09:16
*** bhagyashris has joined #openstack-nova09:21
*** zhanglong has quit IRC09:37
*** derekh has joined #openstack-nova09:39
openstackgerritIury Gregory Melo Ferreira proposed openstack/nova master: Add ironic hypervisor doc  https://review.opendev.org/70199809:44
*** Liang__ has joined #openstack-nova09:48
*** LiangFang has quit IRC09:51
*** ccamacho has joined #openstack-nova09:52
*** Liang__ has quit IRC09:55
gibilyarwood, dansmith: I'm ok with https://review.opendev.org/#/c/694063/ but there are precedence in nova (support move ops with qos ports) not to add api microversions. So I want to double check with you about it before +ait09:56
*** ociuhandu has quit IRC09:57
*** ociuhandu has joined #openstack-nova10:02
brinzhanggibi: https://review.opendev.org/#/c/699669/ I was updated this patch, add the decription of the new policy, if you have time please review again :)10:04
gibibrinzhang: ack, will look10:05
brinzhanggibi: thanks~10:09
*** ociuhandu has quit IRC10:22
*** pcaruana has quit IRC10:23
gibibrinzhang: I'm +2 on the instance action spec10:25
gibithanks for the update10:25
brinzhanggibi: cool, thanks ^^10:26
*** CeeMac has joined #openstack-nova10:30
*** ociuhandu has joined #openstack-nova10:35
*** pcaruana has joined #openstack-nova11:02
*** damien_r has quit IRC11:04
*** rpittau is now known as rpittau|bbl11:11
*** ociuhandu has quit IRC11:13
openstackgerritArnaud Morin proposed openstack/nova master: Force refresh of instance info cache on event  https://review.opendev.org/70239411:13
*** yedongcan has left #openstack-nova11:15
*** dviroel has joined #openstack-nova11:16
stephenfinbauzas: Can you send this on its way, please? https://review.opendev.org/#/c/695733/11:18
*** ociuhandu has joined #openstack-nova11:21
alex_xulyarwood: got it11:26
*** udesale has quit IRC11:28
*** lvbin01 has quit IRC11:28
openstackgerritMerged openstack/nova-specs master: Spec: Ussuri: Encrypted Emulated Virtual TPM  https://review.opendev.org/68680411:29
luyaostephenfin: Could you look at spec 'support live migration with vpmem'?  https://review.opendev.org/#/c/69586311:30
stephenfinluyao: I can, but not today. Could you ask me again tomorrow?11:31
* stephenfin is just back from PTO11:31
luyaostephenfin: OK, thanks at advance. :)11:31
alex_xulyarwood: gibi https://review.opendev.org/#/c/694063/7/specs/ussuri/approved/virt-bfv-instance-rescue.rst@7611:42
*** tbachman has quit IRC11:43
*** ratailor_ has quit IRC11:51
*** dtantsur|afk is now known as dtantsur12:03
*** ociuhandu has quit IRC12:03
*** etingof has joined #openstack-nova12:12
etingofo/ do we have a JSON schema for whatever is exposed through Nova metadata service? I am particularly interested in network_data.json12:14
openstackgerritLuyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory  https://review.opendev.org/69586312:15
sean-k-mooneydansmith: efried i deployed https://review.opendev.org/#/c/699554/2 and installed the required version fo the sdk and cyborg client12:23
sean-k-mooneyit looks like there are still issue however http://paste.openstack.org/show/788372/12:24
sean-k-mooneyit looks like everything on the placement side is fine but its failing to boot a vm with "AttributeError: \'RequestSpec\' object has no attribute \'root_required\'\n\n'"12:25
*** Luzi has joined #openstack-nova12:26
*** dpawlik has quit IRC12:26
*** dpawlik has joined #openstack-nova12:28
huaqianghello stephenfin. I hope you enjoyed your vacation12:31
huaqiangI also want to invite your to review  https://review.opendev.org/#/c/668656/12:32
huaqiangwe have had some disscution especially on how to create a mixed instance, and Alex have put those dicussion links to the update of the spec12:34
huaqiangI hope to get your comments.12:34
*** damien_r has joined #openstack-nova12:35
gibialex_xu: responded. If you are OK with the microversion then feel free to +A, I will +A myself if lyarwood or dansmith state that the microversion is intentional12:37
*** jangutter has quit IRC12:37
*** ociuhandu has joined #openstack-nova12:39
lyarwoodgibi / alex_xu ; apologies just back from a long lunch, yeah it's intentional, I'll add a comment now.12:39
gibilyarwood: cool thanks12:39
*** ociuhandu has quit IRC12:43
brinzhangThis bug fix is ready to review, if you have free time, please review https://review.opendev.org/#/c/580271/12:47
brinzhangit's bug 166345612:48
openstackbug 1663456 in OpenStack Compute (nova) "Field 'updated_at' always 'None' when show aggregate" [Low,In progress] https://launchpad.net/bugs/1663456 - Assigned to Brin Zhang (zhangbailin)12:48
openstackgerritLee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue  https://review.opendev.org/69406312:54
lyarwood^ addressed the commit nit btw12:54
sean-k-mooneydansmith: efried: ok so the cyborg series just need to be rebased on https://review.opendev.org/#/c/699050/12:55
sean-k-mooneywell that is merges so rebaseing on master would be enough12:56
gibilyarwood: +Ad the spec13:00
gibilyarwood: thanks for the confirmation13:00
lyarwoodgibi: awesome thanks :)13:03
*** ociuhandu has joined #openstack-nova13:03
*** tbachman has joined #openstack-nova13:05
*** tbachman_ has joined #openstack-nova13:07
*** tbachman has quit IRC13:10
*** tbachman_ is now known as tbachman13:10
*** bbowen has joined #openstack-nova13:12
sean-k-mooneystephenfin: can you review this when you get a chance https://review.opendev.org/#/c/701601/13:19
*** iurygregory has quit IRC13:23
*** iurygregory has joined #openstack-nova13:23
*** rpittau|bbl is now known as rpittau13:26
openstackgerritMerged openstack/nova-specs master: Boot from volume instance rescue  https://review.opendev.org/69406313:34
*** bhagyashris has quit IRC13:41
stephenfinhuaqiang: As with luyao, if you can ask me again tomorrow I'll try get to it. Too much to do today :(13:48
*** nweinber__ has joined #openstack-nova13:48
*** liuyulong has joined #openstack-nova13:53
*** tkajinam has joined #openstack-nova13:56
lyarwoodefried: https://review.opendev.org/#/c/694033/ - The spec for this has now merged if you're able to look again today, thanks in advance.13:57
*** mlavalle has joined #openstack-nova14:01
stephenfinsean-k-mooney: done14:01
*** ociuhandu has quit IRC14:03
huaqiangstephenfin: Understand. Don't worry. I will connect you later :)14:04
stephenfincool, thanks :)14:04
*** ociuhandu has joined #openstack-nova14:05
*** eharney has joined #openstack-nova14:16
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715614:16
openstackgerritStephen Finucane proposed openstack/nova master: Remove now unnecessary nova-network workaround  https://review.opendev.org/70244014:16
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround  https://review.opendev.org/70244014:17
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715614:17
*** iurygregory has quit IRC14:18
stephenfingibi: I addressed your comments in https://review.opendev.org/#/c/696516/ Think you could revisit at some point?14:18
stephenfingibi: Also, I think I addressed mriedem's concerns on https://review.opendev.org/#/c/696745/ but we can have dansmith look at it to make sure (since he was of the same opinion), if that helps14:19
*** ociuhandu has quit IRC14:29
*** udesale has joined #openstack-nova14:30
*** iurygregory has joined #openstack-nova14:33
*** amodi has joined #openstack-nova14:35
*** awalende has quit IRC14:36
openstackgerritsean mooney proposed openstack/os-vif master: [DNM]test composing devstack_local_conf sections  https://review.opendev.org/70244614:40
sean-k-mooneystephenfin: if ^ works ill squash it into the previous patch but im not sure that we can split devstack_local_conf defination across the job inheritance  or if we can if that extends to the post-config: section too14:41
sean-k-mooneyso i expect that might fail14:42
*** liuyulong has quit IRC14:45
stephenfinack14:45
*** READ10 has joined #openstack-nova14:45
gansoHello folks. I have a question about the impact of security vulnerability upgrades to previously existing VMs. I have a customer that after upgrading the kernel to a newer version that supressed cpu flags ended up not being able to turn their VMs back on because qemu wouldn't allow unless the flags are specifically disabled in the XML. I believe this is something we will see happen several times in the future so it will be a recurrent impact. I see14:46
gansothis spec exists https://blueprints.launchpad.net/nova/+spec/allow-disabling-cpu-flags but I don't see it handling previously existing VMs, only newly created VMs so that they get the correct flags set in the instance XML. Is there any work in-progress to minimize the impact around this? or any known workaround besides having to edit every instance XML manually?14:46
stephenfinganso: kashyap might be able to help you with that, if they're around ^14:47
kashyapAlready reading :-)14:47
kashyapganso: Even if you edit every instance XML manually, which we don't recommend, you do know that it will go away once you reboot the guest?14:48
kashyapganso: I haven't gotten around to implementing that BP, got buried in other stuff.  But good news, there's a valid workaround:14:49
gansokashyap: I wasn't aware of that, thanks. It will go away in the sense that every time it the VM is rebooted nova will overwrite the cpu flags in the XML with what it has defined?14:49
kashyapganso: QEMU has added newer variants of CPU models (with affected flags disabled) that you can directly specify with Nova14:49
kashyapganso: So, for the recent "TSX" vulnerability fiasco ...14:50
kashyapganso: ... QEMU / libvirt has added *-noTSX CPU models.14:51
kashyapganso: E.g. on my Fedora host, running qemu-system-x86-4.2.0-2.fc30.x86_64:14:52
*** vesper has quit IRC14:53
kashyap$> qemu-system-x86_64 -cpu help | egrep *.noTSX*14:53
kashyapx86 Broadwell-noTSX       (alias of Broadwell-v2)14:53
kashyapx86 Broadwell-noTSX-IBRS  (alias of Broadwell-v4)14:53
kashyapx86 Cascadelake-Server-noTSX  (alias of Cascadelake-Server-v3)14:53
*** vesper11 has joined #openstack-nova14:53
kashyapx86 Haswell-noTSX         (alias of Haswell-v2)14:53
kashyapx86 Haswell-noTSX-IBRS    (alias of Haswell-v4)14:53
kashyapx86 Icelake-Client-noTSX  (alias of Icelake-Client-v2)14:53
kashyapx86 Icelake-Server-noTSX  (alias of Icelake-Server-v2)14:53
kashyapx86 Skylake-Client-noTSX-IBRS  (alias of Skylake-Client-v3)14:53
kashyapx86 Skylake-Server-noTSX-IBRS  (alias of Skylake-Server-v3)14:53
kashyap---14:53
gansokashyap: cool, so the new name needs to get to the instance XML14:53
kashyapganso: So if you've got libvirt/QEMU versions with noTSX stuff (confirm by running the above version), then you can use those model names in your nova.conf [libvirt] section14:53
kashyapganso: Yes, indeed.14:54
gansokashyap: ok, replacing them in nova.conf [libvirt] section will only affect new VMs or previously existing ones as well?14:54
openstackgerritStephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715614:54
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module  https://review.opendev.org/70245114:54
kashyapganso: Yes, it will affect — once you reboot the Compute node — _all_ the VMs re-started on that node14:55
kashyapYou'd want something like:14:55
kashyap[libvirt]14:55
kashyapcpu_mode = custom14:55
kashyapcpu_models = Haswell-noTSX-IBRS14:55
kashyapcpu_model_extra_flags = pcid,spec-ctrl,ssbd,md-clear14:55
kashyapganso: And also, related reference: https://docs.openstack.org/nova/latest/admin/mitigation-for-Intel-MDS-security-flaws.html14:56
kashyapSpeaking of which ... /me should probably one for the not-so-fresh-off-the-oven "TAA" (TSX Asynchronous Abort) CVE14:57
gansokashyap: thanks for the clarification! :D14:57
gansokashyap: I will talk to the customer about this approach, should get the impact sorted out14:58
kashyapNo problem; this whole space is a barrel of cockroaches14:58
dansmithsean-k-mooney: yep (re: the rebase)14:59
*** Luzi has quit IRC14:59
openstackgerritStephen Finucane proposed openstack/nova master: pre-commit: Use Python 3 to run checks  https://review.opendev.org/70245315:00
sean-k-mooneyit look like some fo the cyborg code is still pending by the way15:01
*** artom has joined #openstack-nova15:02
sean-k-mooneyspecificly https://review.opendev.org/#/c/698190/ for the deploables v2 api although i dont know if that is needed for nova integration15:03
sean-k-mooneythe devices v2 patches merged today https://review.opendev.org/#/c/695648/15:04
stephenfinefried, bauzas: Since you reviewed the original, could you blast this through? It's hurting me when working on some of the newly Python 3-only files we now have https://review.opendev.org/70245315:05
*** awalende has joined #openstack-nova15:06
*** ociuhandu has joined #openstack-nova15:06
efriedsean-k-mooney: would you +1 that ^ and I'll fast approve?15:08
efriedlyarwood: can you please tag the commit message for https://review.opendev.org/#/c/694033/ with the `blueprint xxx-xxx-xxx` magic?15:09
sean-k-mooneythe precommit change yes runnign it with python3 makes sesne give me a sec and ill add a link in a comment to the relevent docs15:10
*** tkajinam has quit IRC15:10
lyarwoodefried: ack, I'll do that now.15:10
efriedlyarwood: when was the blueprint set to Definition:Approved and by whom?15:10
*** awalende has quit IRC15:11
efried(I thought there used to be a History button, but I must be thinking of something else)15:11
*** ociuhandu has quit IRC15:11
efriedsean-k-mooney: yes, cyborg series should be rebased to use root_required15:11
*** maciejjozefczyk has quit IRC15:12
efriedokay, I think I'm caught up15:12
lyarwoodefried: I'm not sure about https://blueprints.launchpad.net/nova/+spec/virt-rescue-stable-disk-devices but https://blueprints.launchpad.net/nova/+spec/virt-bfv-instance-rescue for this change and spec still needs approval15:12
lyarwoodefried: ah found the email, MattR approved virt-rescue-stable-disk-devices15:13
efriedFYI nova, we've merged merged https://review.opendev.org/#/c/701792/ which should get rid of the "multiple possible networks" tempest errors we've been seeing a lot of lately. If you see more of them, let me know, cause the fix should be simple.15:13
efriedstephenfin: ^15:13
efried"we've merged merged"? #uncaffeinated15:14
artomQuestion about bug triage15:14
artom" Close as "invalid" if it is a support request or feature request." from https://wiki.openstack.org/wiki/Nova/BugTriage15:14
artomIs that a thing we do?15:14
efriedlyarwood: would you mind adding a note for that. But... why are there two blueprints? (I haven't actually *read* anything, so maybe it's obvious, but lead me by the nose here)15:14
artomSpecifically about https://bugs.launchpad.net/nova/+bug/185940315:14
openstackLaunchpad bug 1859403 in OpenStack Compute (nova) "The instance needs to supports dongle devices" [Wishlist,New]15:14
efriedartom: this actually came up last week and I didn't consult that wiki page.15:14
openstackgerritLee Yarwood proposed openstack/os-traits master: Add COMPUTE_BFV_RESCUE trait  https://review.opendev.org/69403315:15
artomI set it to wishlist, is that it? Or since we know what it is (wishlist), we can set it to Triaged as well to remove it from filters/lists15:15
efrieddansmith opened a pro forma bug for something, trying to remember...15:15
dansmith"opened a pro forma bug....against his will"15:16
efriedyeah yeah15:18
efrieddansmith: this one rite? https://bugs.launchpad.net/nova/+bug/185887715:18
openstackLaunchpad bug 1858877 in OpenStack Compute (nova) "Silent wasted storage with multiple RBD backends" [Wishlist,Confirmed]15:18
dansmithyar15:18
efriedSince artom points to an actual document saying we should close as Invalid, Ima do that.15:18
efriedand artom, yeah, do same with yours.15:18
dansmithlol15:18
artomefried, well, wikis are fickle things15:18
efriedI was looking for any excuse15:18
artomI could edit that to say every time a wishlist is filed, we need to call POTUS and get his approval15:18
efriedI think if we get to 100 open bugs, the stay-puft marshmallow man shows up or something.15:19
efriedThen we have to cross the streams, marshmallow everywhere, big mess.15:19
*** maciejjozefczyk has joined #openstack-nova15:19
gibistephenfin: I'm +2 on https://review.opendev.org/#/c/696516/ based on your answers15:19
stephenfingibi: thanks15:19
artomefried, follow-up question then - if we close wishlist items as invalid, how do we expect folks to file them?15:19
efriedblueprints, right?15:20
artomYou're asking me?15:20
gibistephenfin: regarding the rename in https://review.opendev.org/#/c/696745/ I can be convinced both ways so I'm not voting now, sorry15:20
dansmithyes, my point about my bug was that "nova does not support $thing" is not a bug, it's a feature request, hence blueprint at some point15:20
dansmithhowever, we don't file blueprints for things we may or may not do in the future15:20
stephenfingibi: All good. Comes down to dansmith in that case15:21
dansmith(nor should we)15:21
efriedRight, there seems to be a process gap there.15:21
dansmithwhy?15:21
artomIOW, if you have a feature request but aren't prepared to work on it yourself or pay someone to do it, your sool?15:21
artom*you're15:21
efriedif there's a closed/wishlist bug, how do we ever "find" it and make a blueprint out of it.15:21
dansmithwe don't need to enumerate everything nova doesn't do, right?15:21
efriedof course, but a way to track "something nova doesn't do, but probably should, but nobody's going to work on it right now, but we don't really want to forget about it and *never* do it"15:22
efrieddansmith: I think your bug is a good example. You said it's something we probably want to do soonish. But who's gonna remember?15:22
artomI mean, given the project/community dynamics, it'd be reasonable to say "sorry, if you want a thing but can't commit resources, it'll realistically never get done."15:22
artomNot very welcoming, but reasonable15:22
dansmithefried: if putting together that list is actually going to form a backlog that we chew through, then sure, but history tells me it will just become a wasteland of every crazy thing anyone ever thought of once15:23
dansmithif it was a cultivated list that was pruned to just the things we do actually want to do then sure, but.. closed wishlist bugs are not that :)15:24
efriedokay, well, I closed https://bugs.launchpad.net/nova/+bug/1858877 as invalid, so artom you have a recent precedent as well as instructions from the wiki. Go forth and close.15:26
openstackLaunchpad bug 1858877 in OpenStack Compute (nova) "Silent wasted storage with multiple RBD backends" [Wishlist,Invalid]15:26
dansmithstephenfin: I saw lots of mentions as my backscroll flowed in this morning.. if you want me to look at something specific, please relink me15:26
stephenfindansmith: https://review.opendev.org/#/c/696745/ (see last comment)15:30
dansmithoh jesus15:31
dansmithI still would rather it not change, mostly for selfish reasons, but I'm sure I'm the only one (left) that feels that way15:32
dansmithso please just don't make me review it15:32
*** udesale_ has joined #openstack-nova15:34
*** udesale has quit IRC15:37
stephenfin:D15:38
stephenfinfair15:38
stephenfinI know the refactoring impact is still present, but does the use of a 'nova.network.neutron' module (vs. 'nova.network.api') resolve your other concerns at least?15:40
stephenfiniiuc that was mriedem's primary concern (that people would see nova.network.api and think we were talking abut nova-network)15:40
*** ociuhandu has joined #openstack-nova15:42
dansmithit's better in terms of the confusion, yeah15:46
efriedlyarwood: removed -2, but left -1 for discrepancy with the spec. Honestly don't know which would be better tho.15:48
lyarwoodefried: ack yeah apologies I totally forgot to update this in the spec after switching here. I really don't mind tbh so I'll just change this back to COMPUTE_RESCUE_BFV to avoid editing the spec.15:56
openstackgerritLee Yarwood proposed openstack/os-traits master: Add COMPUTE_RESCUE_BFV trait  https://review.opendev.org/69403315:58
*** maciejjozefczyk has quit IRC16:02
*** ccamacho has quit IRC16:02
*** TxGirlGeek has joined #openstack-nova16:02
efriedlyarwood: I honestly want thought put into which is better, actually.16:09
efriedWe don't need to belabor it, but avoiding editing one patch or the other shouldn't be the reason.16:10
*** Sundar has joined #openstack-nova16:10
lyarwoodefried: yeah that's fair, well an actual reason to switch back to COMPUTE_RESCUE_BFV would be that it would allow additional traits to be added in the future16:10
lyarwoodsorry I mean additional rescue traits, COMPUTE_RESCUE_FOO etc16:11
efriedthat's my point: do we want to add more COMPUTE_RESCUE_* traits, or do we want to add more COMPUTE_BFV_* traits?16:11
efriedokay, so COMPUTE_RESCUE_BFV is actually preferred. I can dig that.16:12
*** arxcruz is now known as arxcruz|off16:13
efriedlyarwood: +216:14
lyarwoodefried: thanks16:14
openstackgerritDan Smith proposed openstack/nova master: Add NovaEphemeralObject class for non-persistent objects  https://review.opendev.org/70204916:14
efriedstephenfin: +A on the precommit gizmo, sorry for the delay. And thanks for the vTPM spec approval \o/16:15
*** dosaboy has quit IRC16:28
*** ganso has quit IRC16:28
sean-k-mooneyefried: is the blueprint/spec approval deadline m2?16:30
sean-k-mooneyim just looking at my downstream feature backlog and i might file one more blueprint for this cycle to add viommu support to th elibvirt driver. it will just be a small change like the vpmu change but just tring to figure out when i need to decided if im going to file it16:32
*** hamzy has quit IRC16:33
sean-k-mooneyi want to finish the  image metadata prefiltering work first but the viommu feature is tecnically more useful to end customers16:34
*** awalende has joined #openstack-nova16:37
*** dave-mccowan has joined #openstack-nova16:38
*** awalende has quit IRC16:42
*** gyee has joined #openstack-nova16:42
*** udesale_ has quit IRC16:43
efriedsean-k-mooney: spec freeze 2/13 https://wiki.openstack.org/wiki/Nova/Ussuri_Release_Schedule yes. At that point I'll probably ask us to do a scrub to prioritize whatever's been Definition:Approved and drop some things off the bottom.16:49
sean-k-mooneyya i was execpting there to be a scrub16:50
openstackgerritBalazs Gibizer proposed openstack/nova master: Remove extra instance.save() calls related to qos SRIOV ports  https://review.opendev.org/70226116:51
openstackgerritBalazs Gibizer proposed openstack/nova master: Use common server create function for qos func tests  https://review.opendev.org/70135316:51
sean-k-mooneymy main two feature that were important for this cycle have already merged so the rest are less imporant. i want to finish stuff i have started before doing new stuff too so im lean to not filing another blueprint but i also know its will be trivail to do16:52
openstackgerritBalazs Gibizer proposed openstack/nova master: Enable live migration with qos ports  https://review.opendev.org/69906616:53
*** macz has joined #openstack-nova16:54
*** maciejjozefczyk has joined #openstack-nova16:55
*** rpittau is now known as rpittau|afk17:01
*** mriedem has joined #openstack-nova17:01
*** ganso has joined #openstack-nova17:04
*** dpawlik has quit IRC17:14
*** ociuhandu_ has joined #openstack-nova17:20
*** damien_r has quit IRC17:23
*** ociuhandu has quit IRC17:24
*** ociuhandu_ has quit IRC17:24
*** ociuhandu has joined #openstack-nova17:26
*** ociuhandu has quit IRC17:28
*** hamzy has joined #openstack-nova17:33
*** evrardjp has quit IRC17:33
*** evrardjp has joined #openstack-nova17:34
*** martinkennelly has quit IRC17:36
*** brinzhang has quit IRC17:38
*** brinzhang has joined #openstack-nova17:39
*** born2bake has joined #openstack-nova17:52
*** dosaboy has joined #openstack-nova17:59
*** derekh has quit IRC18:00
*** maciejjozefczyk has quit IRC18:02
*** tbachman has quit IRC18:09
*** tbachman has joined #openstack-nova18:10
*** maciejjozefczyk has joined #openstack-nova18:17
openstackgerritMatt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate  https://review.opendev.org/69522018:18
*** dtantsur is now known as dtantsur|afk18:32
*** maciejjozefczyk has quit IRC18:56
*** maciejjozefczyk has joined #openstack-nova18:59
*** brinzhang_ has joined #openstack-nova19:01
*** brinzhang has quit IRC19:04
*** brinzhang has joined #openstack-nova19:10
*** brinzhang_ has quit IRC19:14
*** pcaruana has quit IRC19:16
*** factor has joined #openstack-nova19:18
*** maciejjozefczyk has quit IRC19:26
*** nweinber__ has quit IRC19:35
*** nweinber__ has joined #openstack-nova19:35
*** gmann is now known as gmann_afk19:36
*** macz has quit IRC19:50
*** macz has joined #openstack-nova19:51
*** slaweq has quit IRC20:02
*** ralonsoh has quit IRC20:06
*** hamzy_ has joined #openstack-nova20:08
*** hamzy has quit IRC20:11
efrieddansmith: old, simple, should be an easy +A: https://review.opendev.org/#/c/694806/20:12
*** maciejjozefczyk has joined #openstack-nova20:13
*** born2bake has quit IRC20:15
*** born2bake has joined #openstack-nova20:15
*** READ10 has quit IRC20:17
*** born2bake has quit IRC20:18
*** born2bake has joined #openstack-nova20:18
dustincefried: I was working on moving checks for no-op providers from the merge step to the validation step as you suggested here https://review.opendev.org/#/c/676029/31/nova/compute/provider_config.py@19920:23
dustincI realized that there was an unintended change in functionality and am not sure if it is worth worrying about or not20:23
dustincold behavior: if using uuid=$COMPUTE_NODE you could exempt individual providers from the $COMPUTE_NODE entry during merge step by adding specific no-op entries for them20:23
dustincnew behavior: since the no-op entry gets removed during validation, the $COMPUTE_NODE value gets used when merging20:23
*** brinzhang_ has joined #openstack-nova20:24
dustincthe old behavior was not specifically intended or documented, but might actually be useful to someone20:24
*** maciejjozefczyk has quit IRC20:25
efrieddustinc: interesting, glad we caught it before we landed this rather than later when somebody *was* relying on it and we yanked the rug. We should noodle whether we want to allow that or not.20:25
efriedI'm going to say: for now, we should explicitly document (wherever we're going to document this thing) that no-op stanzas will be ignored completely and have no effect.20:26
efriedwhich means the new way you're doing the check will be the right way.20:26
efried"ignored completely" with a warning (which I guess isn't ignored completely, but you get the idea)20:26
*** brinzhang has quit IRC20:27
*** maciejjozefczyk has joined #openstack-nova20:29
*** N3l1x has joined #openstack-nova20:31
*** Sundar has quit IRC20:33
*** artom has quit IRC20:33
*** slaweq has joined #openstack-nova20:33
*** N3l1x has quit IRC20:34
*** N3l1x has joined #openstack-nova20:34
*** N3l1x has quit IRC20:35
*** N3l1x has joined #openstack-nova20:35
*** damien_r has joined #openstack-nova20:37
*** awalende has joined #openstack-nova20:38
dustincSounds good to me, I want to push what I have so will go that route for now20:39
*** rchurch has quit IRC20:39
*** rchurch has joined #openstack-nova20:40
*** damien_r has quit IRC20:41
*** awalende has quit IRC20:42
*** brinzhang has joined #openstack-nova20:43
*** hamzy__ has joined #openstack-nova20:43
*** bbowen has quit IRC20:45
*** hamzy_ has quit IRC20:46
*** brinzhang_ has quit IRC20:46
*** tesseract has quit IRC20:50
*** maciejjozefczyk has quit IRC20:50
*** hamzy_ has joined #openstack-nova20:51
*** hamzy__ has quit IRC20:53
*** hamzy__ has joined #openstack-nova21:02
*** hamzy_ has quit IRC21:04
*** hamzy_ has joined #openstack-nova21:10
*** avolkov has quit IRC21:12
*** hamzy__ has quit IRC21:13
*** nweinber__ has quit IRC21:17
*** brinzhang_ has joined #openstack-nova21:21
*** hamzy has joined #openstack-nova21:22
*** hamzy_ has quit IRC21:23
*** brinzhang has quit IRC21:24
openstackgerritEric Fried proposed openstack/nova master: nova-net: Kill it  https://review.opendev.org/69651821:25
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation  https://review.opendev.org/67334121:49
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs  https://review.opendev.org/67602921:49
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree  https://review.opendev.org/67652221:49
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346021:49
*** hamzy_ has joined #openstack-nova21:50
*** tbachman has quit IRC21:53
*** hamzy has quit IRC21:53
*** bbowen has joined #openstack-nova21:54
*** hamzy_ has quit IRC21:58
*** hamzy has joined #openstack-nova21:59
*** Sundar has joined #openstack-nova22:05
SundarHello all, in release notes like https://github.com/openstack/nova/blame/master/releasenotes/notes/image-precaching-d46506568fefa1ea.yaml, what is the id that gets added at the end?22:06
*** hamzy_ has joined #openstack-nova22:06
*** rcernin has joined #openstack-nova22:08
*** hamzy has quit IRC22:09
*** tbachman has joined #openstack-nova22:12
*** brinzhang has joined #openstack-nova22:20
*** mvkr has quit IRC22:21
dansmithSundar: don't hand-create them22:23
dansmithSundar: tox -evenv reno new some-short-name-of-thing22:23
*** brinzhang_ has quit IRC22:23
Sundardansmith: do I use that to create a file which I then 'git add'?22:24
dansmithSundar: yes22:25
SundarThanks22:25
*** tbachman has quit IRC22:30
openstackgerritMerged openstack/nova master: Avoid spurious error logging in _get_compute_nodes_in_db  https://review.opendev.org/69545322:31
*** xek has quit IRC22:33
*** slaweq has quit IRC22:34
*** N3l1x has quit IRC22:40
*** gmann_afk is now known as gmann22:53
*** tkajinam has joined #openstack-nova22:55
*** igordc has joined #openstack-nova22:58
*** igordc has quit IRC23:01
*** igordc has joined #openstack-nova23:01
*** igordc has quit IRC23:02
*** igordc has joined #openstack-nova23:03
*** igordc has quit IRC23:06
*** igordc has joined #openstack-nova23:07
*** igordc has quit IRC23:09
*** igordc has joined #openstack-nova23:10
efrieddustinc: finished reviewing the provider config series. Getting close, nice work.23:10
*** igordc has quit IRC23:11
*** igordc has joined #openstack-nova23:12
dustincefried: thanks, checking it out now..23:12
*** igordc has quit IRC23:14
*** igordc has joined #openstack-nova23:15
*** tosky has quit IRC23:15
efriedaaand I'm out23:16
*** igordc has quit IRC23:19
*** awalende has joined #openstack-nova23:21
*** awalende has quit IRC23:25
*** mvkr has joined #openstack-nova23:27
*** ociuhandu has joined #openstack-nova23:30
*** ociuhandu has quit IRC23:35
*** dviroel has quit IRC23:37
*** hamzy_ has quit IRC23:38
*** tbachman has joined #openstack-nova23:38
*** tbachman_ has joined #openstack-nova23:44
*** mriedem has left #openstack-nova23:46
*** tbachman has quit IRC23:46
*** tbachman_ is now known as tbachman23:46
*** born2bake has quit IRC23:55

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