Wednesday, 2020-01-15

*** macz has quit IRC00:32
*** TxGirlGeek has quit IRC00:33
*** zhanglong has joined #openstack-nova00:49
*** Liang__ has joined #openstack-nova01:13
*** Sundar has quit IRC01:13
*** Liang__ is now known as LiangFang01:14
*** mlavalle has quit IRC01:21
*** gentoorax has quit IRC01:30
*** gentoorax has joined #openstack-nova01:30
brinzhangSunda: just use "reno new < some-short-name-of-thing>" under the project also can create the releasenote file, that depend-on reno packages, so you should installed in your env.01:34
brinzhangSundar01:35
*** adriant has quit IRC01:36
brinzhangSundar: creating releasenote guide:https://docs.openstack.org/reno/latest/user/usage.html#creating-new-release-notes01:41
*** adriant has joined #openstack-nova01:48
*** TxGirlGeek has joined #openstack-nova01:52
openstackgerritMerged openstack/nova master: Print help if nova-manage subcommand is not specified  https://review.opendev.org/69480601:55
openstackgerritMerged openstack/nova master: nova-net: Copy shared utils from nova-net module  https://review.opendev.org/69651501:55
openstackgerritMerged openstack/nova master: nova-net: Remove final references to nova-network  https://review.opendev.org/69651601:55
openstackgerritMerged openstack/nova master: nova-net: Remove 'MetadataManager'  https://review.opendev.org/69674801:55
*** hongbin has joined #openstack-nova02:02
*** hongbin has quit IRC02:02
*** hongbin has joined #openstack-nova02:03
*** hongbin has quit IRC02:03
*** hongbin has joined #openstack-nova02:04
*** hongbin has quit IRC02:04
*** hongbin has joined #openstack-nova02:06
openstackgerritMerged openstack/nova master: nova-net: Add TODOs to remove security group-related objects  https://review.opendev.org/69651702:12
openstackgerritMerged openstack/nova master: nova-net: Remove dependency on nova-net from fake cache  https://review.opendev.org/69674902:12
openstackgerritMerged openstack/nova master: nova-net: Kill it  https://review.opendev.org/69651802:12
*** hamzy_ has joined #openstack-nova02:15
openstackgerritMerged openstack/nova stable/pike: Skip checking of target_dev for vhostuser  https://review.opendev.org/70223102:19
openstackgerritMerged openstack/nova stable/pike: cleanup evacuated instances not on hypervisor  https://review.opendev.org/68791202:25
*** macz has joined #openstack-nova02:30
*** macz has quit IRC02:35
*** awalende has joined #openstack-nova02:38
*** awalende has quit IRC02:43
*** TxGirlGeek has quit IRC03:06
*** factor has quit IRC03:27
*** dave-mccowan has quit IRC03:29
openstackgerritMerged openstack/nova stable/pike: Move restart_compute_service to a common place  https://review.opendev.org/69948803:30
openstackgerritMerged openstack/nova stable/pike: Add functional test for resize crash compute restart revert  https://review.opendev.org/68791303:30
openstackgerritMerged openstack/nova stable/pike: Prevent init_host test to interfere with other tests  https://review.opendev.org/68791603:30
openstackgerritMerged openstack/nova stable/pike: Functional reproduce for bug 1833581  https://review.opendev.org/68791703:30
openstackbug 1833581 in OpenStack Compute (nova) train "instance stuck in BUILD state if nova-compute is restarted" [Low,Fix committed] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)03:30
*** rmk has quit IRC03:32
*** rmk has joined #openstack-nova03:35
*** tkajinam has quit IRC03:48
*** gyee has quit IRC03:56
*** hongbin_ has joined #openstack-nova04:12
*** hongbin has quit IRC04:15
*** tkajinam has joined #openstack-nova04:28
*** macz has joined #openstack-nova04:30
openstackgerritMerged openstack/nova master: pre-commit: Use Python 3 to run checks  https://review.opendev.org/70245304:34
*** macz has quit IRC04:34
*** hongbin_ has quit IRC04:37
*** sapd1_x has joined #openstack-nova04:40
*** evrardjp has quit IRC05:34
*** evrardjp has joined #openstack-nova05:34
*** sapd1_x has quit IRC05:44
*** davee___ has joined #openstack-nova06:04
*** HagunKim has joined #openstack-nova06:05
*** davee_ has quit IRC06:06
*** mugsie has quit IRC06:11
*** mugsie has joined #openstack-nova06:14
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs  https://review.opendev.org/67602906:25
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree  https://review.opendev.org/67652206:25
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346006:25
*** iurygregory has quit IRC07:13
*** KeithMnemonic1 has joined #openstack-nova07:15
*** davee__ has quit IRC07:16
*** davee_ has joined #openstack-nova07:16
*** KeithMnemonic has quit IRC07:19
*** ralonsoh has joined #openstack-nova07:27
*** belmoreira has joined #openstack-nova07:57
*** tesseract has joined #openstack-nova07:58
*** tesseract has quit IRC07:58
*** tesseract has joined #openstack-nova07:59
*** awalende has joined #openstack-nova08:04
*** maciejjozefczyk has joined #openstack-nova08:06
*** slaweq has joined #openstack-nova08:09
*** Luzi has joined #openstack-nova08:14
*** tosky has joined #openstack-nova08:19
*** arxcruz|off is now known as arxcruz08:20
*** iurygregory has joined #openstack-nova08:21
*** rpittau|afk is now known as rpittau08:22
*** dpawlik has joined #openstack-nova08:25
*** damien_r has joined #openstack-nova08:48
*** tkajinam has quit IRC08:48
*** jcosmao has joined #openstack-nova09:10
openstackgerritArnaud Morin proposed openstack/nova master: Force refresh of instance info cache on event  https://review.opendev.org/70239409:14
*** ccamacho has joined #openstack-nova09:15
openstackgerritRiccardo Pittau proposed openstack/nova master: Fix typos for update_available_resource reference  https://review.opendev.org/70260209:22
*** martinkennelly has joined #openstack-nova09:23
*** xek has joined #openstack-nova09:25
*** zhanglong has quit IRC09:36
*** derekh has joined #openstack-nova09:36
*** tonyb has joined #openstack-nova09:38
*** pcaruana has joined #openstack-nova09:42
*** iurygregory is now known as iury_mtg09:50
*** LiangFang has quit IRC09:58
openstackgerritwaleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support  https://review.opendev.org/66596510:00
*** jaosorior has joined #openstack-nova10:12
openstackgerritStephen Finucane proposed openstack/nova stable/train: Print help if nova-manage subcommand is not specified  https://review.opendev.org/70263710:29
brinzhangefried, alex_xu: https://review.opendev.org/#/c/699669/ this spec was +2 by gibi, and sean-k-mooney +1, could you please review this, determinate if it does to go.10:29
brinzhangstephenfin: could you please review this bug fix https://review.opendev.org/#/c/580271/, it is ready.10:31
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add unified '_build_server' helper function  https://review.opendev.org/69753710:31
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add unified '_(build|create)_flavor' helper functions  https://review.opendev.org/69753810:31
openstackgerritStephen Finucane proposed openstack/nova master: functional: Add '_delete_server' to 'InstanceHelperMixin'  https://review.opendev.org/69753910:31
openstackgerritStephen Finucane proposed openstack/nova master: WIP: functional: Add '_create_server' helper  https://review.opendev.org/69754010:31
*** dtantsur|afk is now known as dtantsur10:32
stephenfinbrinzhang: sure10:32
brinzhangstephenfin: cool, thanks ^^10:32
luyaostephenfin: Do you have time look at spec 'support live migration with vpmem' today? https://review.opendev.org/#/c/695863'10:44
stephenfinyup, will do (y)10:44
luyaostephenfin: thanks. :D10:44
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove unused exceptions  https://review.opendev.org/69714910:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes  https://review.opendev.org/69677610:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network'  https://review.opendev.org/69674510:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module  https://review.opendev.org/69674610:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas  https://review.opendev.org/69715210:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object  https://review.opendev.org/69715310:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object  https://review.opendev.org/69715410:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object  https://review.opendev.org/69715510:54
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround  https://review.opendev.org/70244010:55
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module  https://review.opendev.org/70245110:55
openstackgerritStephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715610:55
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network'  https://review.opendev.org/69674510:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module  https://review.opendev.org/69674610:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas  https://review.opendev.org/69715210:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object  https://review.opendev.org/69715310:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object  https://review.opendev.org/69715410:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object  https://review.opendev.org/69715510:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround  https://review.opendev.org/70244010:58
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module  https://review.opendev.org/70245110:58
openstackgerritStephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715610:58
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module  https://review.opendev.org/70245111:08
openstackgerritStephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715611:08
stephenfingibi: Could you have a look at https://review.opendev.org/#/c/695733/, when you have a chance?11:09
*** melwitt has quit IRC11:09
*** pcaruana has quit IRC11:12
brinzhangstephenfin: about https://review.opendev.org/#/c/580271/6/nova/compute/api.py@5258 what do you mean, add "aggregate.updated_at = timeutils.utcnow()" to that api? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/aggregates.py#L17911:24
brinzhangstephenfin: under line17911:25
stephenfinbrinzhang: No, I'm referring to the 'Aggregate.remove_host' method in 'nova.objects.aggregate'11:25
stephenfinbrinzhang: So in here https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L394-L40111:26
gibistephenfin: ack, added to my queue11:26
stephenfinta11:26
kashyapPotentially stupid question, what's invalid about this syntax here:  https://paste.centos.org/view/5d8c9d9711:27
stephenfinkashyap: it's only invalid on Python 211:28
stephenfinyou need to use python3 now11:28
stephenfinkashyap: Look at https://github.com/openstack/nova/blob/master/nova/compute/utils.py#L1374-L1375 then look at https://www.python.org/dev/peps/pep-3102/11:30
kashyapstephenfin: Ah, darn, when I modified my single test-run I forgot to pass py3611:31
kashyapThanks for the references :-)11:31
stephenfinnp11:31
brinzhangstephenfin: I will check, thanks11:32
*** tbachman has quit IRC11:39
*** shilpasd has joined #openstack-nova11:43
*** jaosorior has quit IRC11:46
*** dave-mccowan has joined #openstack-nova11:48
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Add a default Virtio RNG device to guests  https://review.opendev.org/61660311:54
*** mgoddard has quit IRC11:55
*** dviroel has joined #openstack-nova11:58
*** iury_mtg has quit IRC12:00
*** mgoddard has joined #openstack-nova12:01
*** iurygregory has joined #openstack-nova12:01
*** mgoddard has quit IRC12:05
openstackgerritMerged openstack/os-traits master: Add COMPUTE_RESCUE_BFV trait  https://review.opendev.org/69403312:07
*** mgoddard has joined #openstack-nova12:15
openstackgerritBrin Zhang proposed openstack/nova master: Populate 'updated_at' when aggregate is updated  https://review.opendev.org/58027112:24
*** ociuhandu has joined #openstack-nova12:26
brinzhangstephenfin: I removed the duplicate handle in compute api. In the object.aggregate was done it, updated.12:26
*** damien_r has quit IRC12:27
*** HagunKim has quit IRC12:35
*** rchurch has quit IRC12:44
*** udesale has joined #openstack-nova12:45
*** rchurch has joined #openstack-nova12:46
*** pcaruana has joined #openstack-nova12:49
*** cgoncalves has quit IRC12:58
*** ociuhandu has quit IRC12:59
*** cgoncalves has joined #openstack-nova13:00
*** mriedem has joined #openstack-nova13:00
*** cgoncalves has quit IRC13:04
*** cgoncalves has joined #openstack-nova13:10
*** damien_r has joined #openstack-nova13:10
*** eharney has quit IRC13:20
*** dpawlik has quit IRC13:23
*** ociuhandu has joined #openstack-nova13:27
*** bhagyashris has joined #openstack-nova13:31
*** dave-mccowan has quit IRC13:33
*** dave-mccowan has joined #openstack-nova13:36
*** jmlowe has joined #openstack-nova13:38
*** jmlowe has quit IRC13:39
*** nweinber__ has joined #openstack-nova13:39
*** liuyulong has joined #openstack-nova13:41
*** Liang__ has joined #openstack-nova13:45
*** mnaser has quit IRC13:54
*** mnaser has joined #openstack-nova13:55
*** Liang__ is now known as LiangFang14:01
*** eharney has joined #openstack-nova14:01
*** Sundar has joined #openstack-nova14:03
kashyapsean-k-mooney: Hey, finally got around to this RNG thing, mind having a gander when you can - https://review.opendev.org/#/c/616603/14:03
*** rcernin has quit IRC14:04
*** bhagyashris has quit IRC14:06
*** yan0s has joined #openstack-nova14:10
Sundardansmith, efried: I am here if there are any questions on https://review.opendev.org/#/c/631244/ or related stuff.14:16
*** jungleboyj has quit IRC14:20
*** jungleboyj has joined #openstack-nova14:21
dansmithSundar: I just got up, so I haven't seen, but I'm assuming you pushed a new rev?14:37
SundarI published comments, will push patches after you agree.14:38
Sundardansmith: Thanks, no rush14:40
Sundarbrinzhang: Thanks for the release notes pointers14:40
*** bhagyashris has joined #openstack-nova14:40
stephenfinkashyap: I hit that14:41
* kashyap goes to check14:41
kashyapstephenfin: Thank you14:41
*** artom has joined #openstack-nova14:42
kashyapOkay, read your remarks, will respond there.14:43
*** dpawlik has joined #openstack-nova14:45
sean-k-mooneykashyap: sure ill take a look14:47
kashyapThanks14:48
sean-k-mooneySundar: when you push the new version be sure to rebase all the patches in the chain against master14:48
SundarYes, already done internally14:49
sean-k-mooneySundar: i deployed https://review.opendev.org/#/c/699554/2 yesterday but it does not work because it is missing depencies14:49
Sundarsean-k-mooney: I think you mean the root_required patch, which has been merged now?14:51
sean-k-mooneyyes14:51
sean-k-mooneyon the cyborg side i notice the device v2 api has merged on monday/tuseday14:52
sean-k-mooneyare the arq v2 https://review.opendev.org/#/c/696089/ and deployable v2 https://review.opendev.org/#/c/698190/ changes required for this work?14:52
sean-k-mooneyi deployed with cyborg master which had the device v2 changes when i was testing14:54
Sundarsean-k-mooney: Yes, I am updating the patch sets with these dependencies14:54
Sundarsean-k-mooney: Yes, device v2 API merged, but that should be tangential to Nova. The main thing that Nova patches needs is https://review.opendev.org/69884614:55
Sundar*need14:55
*** awalende_ has joined #openstack-nova14:57
*** macz has joined #openstack-nova14:57
sean-k-mooneyok well i was trying to use the cyborg placemtn client so haveing the api to validate the existing of the fake device and compare with placment was useful14:58
Sundarsean-k-mooney: You don;t need https://review.opendev.org/#/c/698190/. Looking at the other one14:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove layer of indirection in 'nova.network'  https://review.opendev.org/69674514:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module  https://review.opendev.org/69674614:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas  https://review.opendev.org/69715214:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object  https://review.opendev.org/69715314:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object  https://review.opendev.org/69715414:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object  https://review.opendev.org/69715514:58
openstackgerritStephen Finucane proposed openstack/nova master: nova-net: Remove now unnecessary nova-net workaround  https://review.opendev.org/70244014:58
openstackgerritStephen Finucane proposed openstack/nova master: Remove 'nova.image.api' module  https://review.opendev.org/70245114:58
openstackgerritStephen Finucane proposed openstack/nova master: WIP: nova-net: Remove unused nova-network objects  https://review.opendev.org/69715614:58
SundarSure, that has been there since Train release14:58
sean-k-mooneywell the device api has but the support in the client only seams to work with v215:00
sean-k-mooneySundar: by the way this is what i found http://paste.openstack.org/show/788372/15:00
Sundarsean-k-mooney: You don't need https://review.opendev.org/#/c/696089/ either15:00
sean-k-mooneySundar: cool i did not have either applied so ill only use the first one you linked15:00
*** awalende has quit IRC15:00
sean-k-mooneyhttps://review.opendev.org/#/c/698846/15:01
*** awalende_ has quit IRC15:01
*** ociuhandu has quit IRC15:01
dansmithSundar: replying to those comments is a lot harder 1-2 weeks after I make them, vs sooner15:02
Sundardansmith: Got it, I had to fix the Cyborg side event nptification patch and its dependencies before I got to this15:02
dansmithSundar: I'm trying to make an effort to drop whatever I'm doing to review those patches as soon as you put them up to keep things moving, so it'd be helpful if you could at least reply to them quicker, even if the code isn't coming immediately15:02
dansmithI think I've replied to all the ones you're expecting to hear about, but if I missed one let me know15:03
*** bhagyashris has quit IRC15:03
*** Luzi has quit IRC15:04
Sundardansmith: OK, but without trying the code changes, the best I could say is 'will consider/try it out'. Presumably you want a firmer reply than that.15:04
Sundardansmith: Thanks, looking now15:04
dansmithSundar: well, some of the comments were not needing code changes. anyway, just saying, faster responses would be nice15:05
Sundarsena-k-mooney: Not sure if 'openstack accelerator device profile' is ready for prime time yet. But here it seems to be fine. Do you have the cyborg API logs?15:05
Sundarsean-k-mooney: ^15:05
sean-k-mooneyam i still have it deploy it seamed to create the profile fine as the list shows15:10
sean-k-mooneyits a littel weird to just dump the raw json on the command line15:11
sean-k-mooneywhen defining the groups15:11
sean-k-mooneyas far as i can tell the issue was the root_required patch not being present on the branch i tested15:12
sean-k-mooneySundar: next time i try goign though the work flow of deploying a vm ill see if i can capture the api logs too15:14
gibistephenfin, efried: the qos live migration patches are ready for review, if you have some time  https://review.opendev.org/#/q/status:open++topic:bp/support-move-ops-with-qos-ports-ussuri15:14
Sundarsean-k-mooney: Thanks15:16
Sundardansmith: Re. https://review.opendev.org/#/c/631244/53/nova/tests/unit/compute/test_compute_mgr.py@6098 , I stepped through the code and noticed that the return value was a dict, not a list. In fact, the first dict from the list.15:18
Sundardansmith: Your concerns about clarity are probably addressed by having mock_get_arqs.assert_has_calls() and clarifying the comments, right?15:19
dansmithSundar: then I think something else is going on: https://pastebin.com/7Rr1CVfd15:19
dansmithSundar: side_effect is the one that behaves the way you're describing (as you know): https://pastebin.com/PHiKTtew15:21
Sundardansmith: let me take another look and get back15:22
dansmiththanks15:22
*** awalende has joined #openstack-nova15:24
*** ociuhandu has joined #openstack-nova15:28
*** awalende has quit IRC15:28
*** dpawlik has quit IRC15:33
*** shilpasd has quit IRC15:33
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs  https://review.opendev.org/55292415:39
bauzashuzzah ^15:39
* bauzas is eventually back upstream15:39
*** ociuhandu has quit IRC15:39
*** ociuhandu has joined #openstack-nova15:40
sean-k-mooneywell it nice that you rejoin us with such a simple and non contorviral proposal :P15:42
*** LiangFang has quit IRC15:46
*** eharney has quit IRC15:48
artomControviral?15:49
artomI kinda like that. Becomes viral because it's so controversial15:50
sean-k-mooneyhehe15:50
sean-k-mooneyyou figured it out so its close enough15:50
sean-k-mooneybut yes i think the decieion we have been making related to mixed cpus makes modeing numa in plamcent more controversial15:51
stephenfingibi: Done (the QoS live migration series)15:52
gibistephenfin: thanks a lot!15:52
*** eharney has joined #openstack-nova15:52
*** mlavalle has joined #openstack-nova15:52
sean-k-mooneyi was less concerned when we used the hw:* syntax but if we go with the placementese syntax then i have deep concerns with granuarl request groups and the group_policy as they exist today15:52
*** udesale_ has joined #openstack-nova15:54
Sundardansmith: Please see http://paste.openstack.org/show/788413/. The main change I made is http://paste.openstack.org/show/788415/. The first call to get_arqs_for_instance is mocked properly, but the second call hits a StopIteration error.15:55
*** bhagyashris has joined #openstack-nova15:55
*** efried is now known as efried_afk15:56
* efried_afk doc appt15:56
dansmithSundar: no, return_value will work as many times as you want: https://pastebin.com/2Wdgb64J15:57
*** udesale has quit IRC15:57
sean-k-mooneyif you want to have a limited number of retruns or different retrun i think you use sideffect with an iterable15:58
SundarAh, my bad, I still have it as side_effect. Will fix that15:58
dansmith...15:58
*** melwitt has joined #openstack-nova16:01
SundarI think it should work with these changes. This was the only pending point, AFAICS. Are you good with updating the patch set once this is resolved?16:01
Sundardansmith: ^16:01
dansmithSundar: unless you've replied to other comments I had, I think so16:01
SundarI pretty much agreed to other changes, even if it is a matter of preference.16:02
*** gyee has joined #openstack-nova16:03
aarentslyarwood:  I replied to your remark https://review.opendev.org/#/c/696084 maybe kashyap can confirm that implemeting finish_unshelved as you suggest is the way to go16:03
*** mrch_ has joined #openstack-nova16:08
*** mrch_ has quit IRC16:08
*** mrch_ has joined #openstack-nova16:08
*** damien_r has quit IRC16:10
*** damien_r has joined #openstack-nova16:11
*** TxGirlGeek has joined #openstack-nova16:14
openstackgerritSundar Nadathur proposed openstack/os-traits master: Add a trait that compute manager can handle accelerator requests.  https://review.opendev.org/69896116:23
*** damien_r has quit IRC16:26
*** bhagyashris has quit IRC16:37
*** ociuhandu has quit IRC16:39
*** ociuhandu has joined #openstack-nova16:39
*** Sundar has quit IRC16:40
*** openstackgerrit has quit IRC16:43
*** openstackgerrit has joined #openstack-nova16:43
openstackgerritBalazs Gibizer proposed openstack/nova stable/stein: DNM: try to trace bug 1859766  https://review.opendev.org/70270016:43
openstackbug 1859766 in OpenStack Compute (nova) stein "functional tests intermittently fails with "ReadOnlyFieldError: Cannot modify readonly field uuid"" [Undecided,New] https://launchpad.net/bugs/1859766 - Assigned to Balazs Gibizer (balazs-gibizer)16:43
*** pcaruana has quit IRC16:45
*** udesale_ has quit IRC16:45
*** bhagyashris has joined #openstack-nova16:46
*** Sundar has joined #openstack-nova16:51
*** ociuhandu has quit IRC16:59
*** maciejjozefczyk has quit IRC17:04
*** iurygregory has quit IRC17:04
*** maciejjozefczyk has joined #openstack-nova17:04
*** ociuhandu has joined #openstack-nova17:08
*** nweinber__ has quit IRC17:09
*** gmann has quit IRC17:09
*** gmann has joined #openstack-nova17:09
*** bhagyashris has quit IRC17:10
*** bhagyashris has joined #openstack-nova17:11
*** iurygregory has joined #openstack-nova17:14
*** rpittau is now known as rpittau|afk17:15
*** bhagyashris has quit IRC17:17
*** tesseract has quit IRC17:26
*** iurygregory has quit IRC17:26
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Ussuri  https://review.opendev.org/69384417:27
kashyapaarents: Sorry, I wasn't ignoring that change, just been buried.17:28
*** arxcruz is now known as arxcruz|off17:29
*** guilhermesp has quit IRC17:31
*** guilhermesp has joined #openstack-nova17:32
*** awalende has joined #openstack-nova17:33
*** ociuhandu has quit IRC17:33
*** evrardjp has quit IRC17:34
*** evrardjp has joined #openstack-nova17:34
*** dtantsur is now known as dtantsur|afk17:35
stephenfinefried_afk: not sure if you're gone for the day, but https://review.opendev.org/#/c/697537/ could probably be revisited now that x-cell resize has merged17:46
stephenfinand melwitt, I think I've clarified my (new) intent in https://review.opendev.org/#/c/696745/, which should assuage some of your concerns hopefully17:47
stephenfinwith that17:47
* stephenfin bails 🪂17:47
melwittstephenfin: ack17:47
*** awalende has quit IRC17:54
*** TxGirlGeek has quit IRC18:01
openstackgerritArtom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146  https://review.opendev.org/68740418:02
openstackbug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms)18:02
artomdansmith, could I ask you to look at that 2-patch series ^^ when you get a chance? stephenfin's the other reviewer, but your +1 or +W'less +2 can still carry weight and help the evential non-RH approver (efried, I'm guessing)18:03
*** jcosmao has left #openstack-nova18:06
*** TxGirlGeek has joined #openstack-nova18:07
*** derekh has quit IRC18:08
dansmithartom: yeah, although I'm a couple layers deep on the stack and have a call in 18 minutes18:12
dansmithartom: meaning, don't let me forget18:12
fungianybody remember how far back nova started providing configdrive by default without needing to manually enable it in the config?18:14
fungi(i mean providing it if the client requests it at boot)18:14
dansmitha long time18:14
*** ociuhandu has joined #openstack-nova18:14
fungicool, config reference only goes back as far as pike but seemed to indicate it was available by default at least that far back18:15
fungithanks dansmith!18:15
dansmithyeah I'm pretty sure it's been around for a real (real) long time18:15
sean-k-mooneyfungi: its been availabel via the api18:15
sean-k-mooneyits not on by defualt unless you request it or set the force config drive options in the config18:15
sean-k-mooneyit might predate micorverions18:16
fungiyep, thanks. trying to help a user in #zuul figure out whether they have to do anything fancy in nova's config to make configdrive available to nodepool18:16
dansmithit definitely predates microversions18:16
fungisounds like no, just make sure nodepool asks for a configdrive18:16
dansmithwell, isn't there a way to configure it to *not* honor the user's configdrive request?18:17
dansmithah, nevermind, looks like they can always ask for it18:17
dansmithso yeah I think you're good18:17
*** xek_ has joined #openstack-nova18:18
sean-k-mooneyfungi: the only thing they shoudl have to do is make sure the iso vfat tools are installed so nova can actully create the config drive18:18
sean-k-mooneybut they will be on any normal install18:18
fungiright, already suggested that to them18:19
fungieither genisofs or mkisofs depending on the vintage18:19
sean-k-mooneyyep18:19
*** ociuhandu has quit IRC18:19
fungii have a feeling they just need to adjust their nodepool launcher config18:19
*** xek has quit IRC18:20
fungithey provided a `openstack show server` paste which indicated config_drive was not True and i expect that comes down to the boot parameters it provided18:20
sean-k-mooneyright. it could be forced in there config but nodepool has a setting for that too right18:22
sean-k-mooneyyou can use https://zuul-ci.org/docs/nodepool/configuration.html#attr-providers.[openstack].diskimages.config-drive18:23
sean-k-mooneyactully it looks like node pool does not use the boot parmater since that is on the image and instead is using the image metadata right?18:24
sean-k-mooneythere are 3 ways to trun on cofig drive, the force_config_drive config option, the api boot request or the image metadata18:24
*** eharney has quit IRC18:26
*** TheJulia has quit IRC18:29
*** rpittau|afk has quit IRC18:30
*** vdrok has quit IRC18:30
*** masayukig has quit IRC18:30
*** _erlon_ has quit IRC18:30
*** coreycb has quit IRC18:30
*** donnyd has quit IRC18:30
*** csatari has quit IRC18:30
*** cz3 has quit IRC18:30
openstackgerritSundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access  https://review.opendev.org/63124218:31
openstackgerritSundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec.  https://review.opendev.org/63124318:31
openstackgerritSundar Nadathur proposed openstack/nova master: Define Cyborg ARQ binding notification event.  https://review.opendev.org/69270718:31
openstackgerritSundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs.  https://review.opendev.org/63124418:31
openstackgerritSundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager.  https://review.opendev.org/69858118:31
openstackgerritSundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver.  https://review.opendev.org/63124518:31
openstackgerritSundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted.  https://review.opendev.org/67373518:31
openstackgerritSundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators.  https://review.opendev.org/69794018:31
openstackgerritSundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators.  https://review.opendev.org/69955318:31
openstackgerritSundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait.  https://review.opendev.org/69955418:31
openstackgerritSundar Nadathur proposed openstack/nova master: Add cyborg tempest job.  https://review.opendev.org/67099918:31
*** ildikov has quit IRC18:31
*** arne_wiebalck_ has joined #openstack-nova18:31
*** mnaser has quit IRC18:31
*** awestin1 has quit IRC18:31
*** donnyd has joined #openstack-nova18:31
*** tonyb[m] has quit IRC18:31
*** arne_wiebalck has quit IRC18:31
*** arne_wiebalck_ is now known as arne_wiebalck18:31
*** TheJulia has joined #openstack-nova18:32
*** openstackstatus has quit IRC18:32
*** coreycb has joined #openstack-nova18:32
*** ildikov has joined #openstack-nova18:33
*** vdrok has joined #openstack-nova18:33
*** csatari has joined #openstack-nova18:33
*** cz3 has joined #openstack-nova18:33
*** mnaser has joined #openstack-nova18:33
*** johnsom has quit IRC18:33
*** tosky has quit IRC18:33
*** awestin1 has joined #openstack-nova18:33
*** awestin1 has quit IRC18:34
*** tonyb[m] has joined #openstack-nova18:35
*** johnsom has joined #openstack-nova18:35
*** awestin1 has joined #openstack-nova18:36
*** masayukig has joined #openstack-nova18:36
*** csatari has quit IRC18:38
*** eharney has joined #openstack-nova18:38
openstackgerritwaleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support  https://review.opendev.org/66596518:39
*** rcernin has joined #openstack-nova18:40
*** cz3 has quit IRC18:40
*** TheJulia has quit IRC18:40
*** donnyd has quit IRC18:40
*** sorrison has quit IRC18:42
*** donnyd has joined #openstack-nova18:42
artomdansmith, sorry, was on a 1:118:43
artomdansmith, thanks! I'll be sure to hound your ass then18:43
*** sorrison has joined #openstack-nova18:46
*** slaweq has quit IRC18:47
*** efried_afk is now known as efried18:55
efried#success With the merging of https://review.opendev.org/#/c/696518/ mox is finally purged from nova \o/19:01
sean-k-mooneycool19:02
efriedtakashin, stephenfin: ^ I closed out the bp19:02
efriedhm, no response from successbot19:02
efriedshrug19:02
sean-k-mooneyit was so amazed that its finally done it need a minute19:02
sean-k-mooneyor its borked19:03
dansmithmaybe it is opinionated now and doesn't agree it's a successbot-worthy moment?19:06
sean-k-mooneyin that case can we get it to do code reviews for us too19:06
sean-k-mooneyits tallens are clearly being wasted19:07
*** Sundar has quit IRC19:08
*** TxGirlGeek has quit IRC19:10
openstackgerritMerged openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION  https://review.opendev.org/69590119:11
*** martinkennelly has quit IRC19:11
*** TxGirlGeek has joined #openstack-nova19:15
efriedstephenfin: done, and the one a couple further up the series.19:16
fungiopenstackstatus is not present in here. checking to see if it's fallen victim to a netsplit or something, hold on19:19
*** openstackstatus has joined #openstack-nova19:22
*** ChanServ sets mode: +v openstackstatus19:22
fungiefried: okay, try again!19:22
efried#success With the merging of https://review.opendev.org/#/c/696518/ mox is finally purged from nova \o/19:22
openstackstatusefried: Added success to Success page (https://wiki.openstack.org/wiki/Successes)19:22
efriednice, thanks fungi!19:22
fungiit was disconnected for the past 50 minutes due to some shakeup in freenode19:23
fungibut should be fine now19:23
efriedyou da person19:23
fungiefried: thanks for communicating the moxtermination!19:23
fungii think we've all been looking forward to it for a *very* long time19:23
* sean-k-mooney waits for dansmith and mriedem19:24
fungiheh19:24
sean-k-mooneyfungi: while you are here. do you know what the plan will be regarding ubuntu 20.04 i assume we will move to it sometime in V19:25
sean-k-mooneyi guess that decsion wont be made untill closer to the start of V but just said i would ask19:26
fungisean-k-mooney: it's slated to be released around ussuri rc1 last i looked, so we'll likely have images available before ussuri releases but maybe not as soon as master reopens for victoria commits19:26
sean-k-mooneyis it actully victoria. i did not see a poll for the name although i might have missed it19:27
fungiaiui, the tc is ready to declare ubuntu 20.04 a release platform for victoria though: https://review.opendev.org/69374319:28
sean-k-mooneyok cool we were discussing a kerenl 5.4 feature downstream earlier and the reason i asked is i think 20.04 will be based on 5.419:29
fungisean-k-mooney: yeah, voting on the v cycle name happened in early december, collecting ideas started in mid-november19:30
fungihttp://lists.openstack.org/pipermail/openstack-discuss/2019-November/010880.html19:30
fungihttp://lists.openstack.org/pipermail/openstack-discuss/2020-January/011947.html19:30
sean-k-mooneyah ok i saw the tread on the proposal but i obviously missed the vote likely when i went on vaction19:31
fungiannouncement was ^ last week after legal was done with trademark searches and risk eval19:31
fungiand the w cycle naming process is about to get underway: https://review.opendev.org/70241419:32
*** iurygregory has joined #openstack-nova19:36
*** dklyle has quit IRC19:36
*** ociuhandu has joined #openstack-nova19:36
sean-k-mooneycool thanks for the update :)19:36
*** dklyle has joined #openstack-nova19:37
*** dklyle has quit IRC19:37
*** dklyle has joined #openstack-nova19:38
sean-k-mooneyefried: dansmith i currently have sundars latest update stacking. ill likely test it out tomorow however since im about to call it a day19:38
dansmithcool19:38
openstackgerritMerged openstack/os-traits master: Add a trait that compute manager can handle accelerator requests.  https://review.opendev.org/69896120:02
*** TheJulia has joined #openstack-nova20:08
*** _erlon_ has joined #openstack-nova20:11
*** mrch_ has quit IRC20:14
*** ccamacho has quit IRC20:16
openstackgerritMerged openstack/nova master: nova-net: Remove unused exceptions  https://review.opendev.org/69714920:33
openstackgerritMerged openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes  https://review.opendev.org/69677620:34
*** eharney has quit IRC20:36
*** slaweq has joined #openstack-nova20:36
*** iurygregory has quit IRC20:37
*** gentoorax is now known as gentoorax_away20:41
*** gentoorax_away is now known as gentoorax20:41
openstackgerritwaleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support  https://review.opendev.org/66596520:42
*** vishalmanchanda has quit IRC20:44
*** mrch_ has joined #openstack-nova20:45
efriedlyarwood: I updated your os-traits release patch to pick up COMPUTE_ACCELERATORS which just merged.20:46
sean-k-mooneyya i had to use libs_from_git to pull that in for testing the cyborg changes so that would be useful20:46
*** ralonsoh has quit IRC20:57
*** mrch_ has quit IRC21:00
*** dviroel has quit IRC21:10
*** cz3 has joined #openstack-nova21:14
*** slaweq has quit IRC21:19
*** luyao has quit IRC21:23
openstackgerritMerged openstack/nova stable/rocky: Improve metadata server performance with large security groups  https://review.opendev.org/69751721:27
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree  https://review.opendev.org/67652221:28
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346021:28
*** igordc has joined #openstack-nova21:29
*** CeeMac has quit IRC21:40
*** rcernin has quit IRC21:42
efriedmsg diablo_rojo21:45
efriedsigh21:45
openstackgerritMerged openstack/nova stable/stein: block_device: Copy original volume_type when missing for snapshot based volumes  https://review.opendev.org/69668621:46
*** eharney has joined #openstack-nova21:48
*** ociuhandu has quit IRC21:54
openstackgerritDustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs  https://review.opendev.org/69346022:18
*** xek_ has quit IRC22:24
*** ganso has quit IRC22:35
*** maciejjozefczyk has quit IRC22:35
*** mriedem has left #openstack-nova22:36
*** dosaboy has quit IRC22:36
*** dklyle has quit IRC22:40
*** nweinber__ has joined #openstack-nova22:44
efrieddustinc: ready for re-review?22:49
*** dklyle has joined #openstack-nova22:49
*** ganso has joined #openstack-nova22:53
*** dosaboy has joined #openstack-nova22:53
*** rcernin has joined #openstack-nova22:55
*** nweinber__ has quit IRC22:56
*** tkajinam has joined #openstack-nova23:03
*** igordc has quit IRC23:09
*** igordc has joined #openstack-nova23:10
*** mlavalle has quit IRC23:15
*** tobiash has quit IRC23:38
*** tobiash has joined #openstack-nova23:39

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