Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-14
21:57:17 melwitt dansmith ^
22:02:54 hrw and sorry for writing 3rd patch to what looks the same
22:07:12 hrw dansmith: after reading https://review.openstack.org/#/c/643458 and https://review.openstack.org/#/c/643455 I think that 643355 is wrong, your patch is good (left comment) and mine https://review.openstack.org/643467 just fixes to comply with capabilities contents
22:07:30 hrw but that's me - random guy from the internet D:
22:13:24 openstackgerrit Merged openstack/nova master: Customize irrelevant-files for nova-live-migration job https://review.openstack.org/643346
22:13:43 openstackgerrit Merged openstack/nova master: Re-enable Ceph in live migration testing https://review.openstack.org/643150
22:21:35 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899
22:26:57 dansmith melwitt: what are you pointing me to?
22:27:25 melwitt dansmith: hrw's patch https://review.openstack.org/643467
22:28:04 melwitt it's related to the unknown arch blowup, sort of
22:28:28 dansmith yeah, I'm not sure which is the official arch name and/or if they're both legit as I'm not an arm specialist
22:28:52 dansmith but my patch is still something we should do as (as hrw said)
22:29:12 hrw dansmith: armv6 armv6l depends on kernel.
22:29:13 melwitt hrw: if you think https://review.openstack.org/643455 is wrong, it would be helpful to put your comment on it about why. I'm curious anyway
22:29:23 dansmith hrw: my concern would be it might also depend on libvirt behavior
22:29:40 melwitt dansmith: yeah, not saying your patch isn't something we should do. just that along that line, if you knew anything about changing that enum, hrw's patch was there
22:29:52 dansmith hrw: because if the l just means long or something, then I would expect both or either could be emulatable
22:30:51 dansmith melwitt: hopefully it's obvious that hrw's patch to change the value of the enum is not okay :)
22:31:29 melwitt dansmith: gah, I thought maybe you could comment on the review
22:31:40 melwitt nevermind me
22:31:55 hrw dansmith: libvirt mentions only armv6l and armv7l.
22:31:59 dansmith melwitt: I just did
22:32:05 melwitt thanks
22:32:14 dansmith hrw: on your system you mean right?
22:32:17 hrw https://gitlab.com/search?utf8=%E2%9C%93&snippets=&scope=&search=VIR_ARCH_ARMV6L&project_id=192693#L30
22:32:23 hrw in libvirt code
22:33:29 dansmith hrw: okay, like I said, I'm not really able to say whether armv6 or armv6l (or both) are right
22:33:49 dansmith but we can't just make the change you've proposed because it has effects to versioned objects and will break communication between services
22:33:55 hrw ok.
22:34:03 hrw perfectly fine for me.
22:34:30 dansmith hrw: I will say, if you're right that it's always armv6l, then we definitely need a variant of your patch over mgoddard's
22:34:55 hrw dansmith: let me discuss it tomorrow with libvirt guys, ok?
22:34:58 hrw 23:34 here
22:35:13 dansmith sure
22:36:05 hrw as nova/virt/libvirt gets capabilities from libvirt so is it armv6 or armv6l is more or less their decision. if they say 'no, it was always armv6z' then we both are wrong
22:36:57 dansmith yeah
22:36:59 hrw the good point is that armv6 is not able to be a host so it is only guest issue
22:38:07 openstackgerrit Kevin Rasmussen proposed openstack/nova master: created AggregateSubnetAvailabilityFilter https://review.openstack.org/633178
22:38:17 dansmith I'm not sure that's relevant to the thing that nova has to do in this case, but.. okay :)
22:38:32 hrw nothing in this case ;D
22:42:11 hrw ok, went, asked, go to sleep. wake up in ~6h
22:42:27 dansmith go :)
22:44:21 hrw and I promised myself to not play with nova for at least a year... ;d
22:44:43 hrw have a nice day
23:03:00 openstackgerrit Eric Fried proposed openstack/nova master: Remove [ironic]api_endpoint option https://review.openstack.org/643483
23:03:27 openstackgerrit Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484
23:03:36 TheJulia mriedem_away: melwitt: ^^^
23:06:11 melwitt TheJulia: commented. I had thought jroll or someone had said we shouldn't do that in _cleanup_deploy unilaterally
23:07:21 TheJulia I really don't know what to do then
23:10:56 melwitt TheJulia: well, I just mean that try-except would move to the destroy() method
23:11:13 TheJulia I'm checking to see if we can do that
23:11:25 melwitt if I'm remembering correctly that it should be done for all clean deploys. I might be wrong
23:11:32 melwitt *shouldn't
23:14:21 TheJulia well, making sure we nuke everything and if we can just tolerate it on a destroy
23:19:13 TheJulia melwitt: looks like we can tolerate a conflict anywhere along there because ironic will do cleanup. Some of the things more recently, but yeah... we should be good to just move it
23:20:04 melwitt TheJulia: ok. maybe we'll double check with jroll first before you change anything. bc I'm not 100% sure I remembered that right and don't remember what date to look for #openstack-ironic logs
23:20:24 TheJulia melwitt: I just double checked the paths and coductor behavior
23:20:56 melwitt TheJulia: oh, sorry, I misunderstood. you confirmed it should be only for destroy() and not all cleanup deploy
23:21:05 TheJulia yeah
23:21:08 melwitt ok, cool
23:27:55 openstackgerrit Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484
23:27:59 TheJulia melwitt: ^^
23:28:39 melwitt coolness
23:29:53 melwitt TheJulia: there's comments on PS1 about whether it's ok to ignore any and all conflict (only one way that conflict can occur)
23:30:33 melwitt if you could reply and confirm that
23:36:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline https://review.openstack.org/627282
#openstack-nova - 2019-03-15
00:36:16 gmann takashin: ^^ let's just say "- more information about sorting order of response results". mentioned particular field even as example can create confusion.
01:01:20 takashin gmann: Thanks. I will fix it.
01:02:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline https://review.openstack.org/627282
01:31:01 openstackgerrit Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272
01:35:52 openstackgerrit Julia Kreger proposed openstack/nova master: [ironic] ignore conflict upon cleaning up node https://review.openstack.org/643484
01:40:45 openstackgerrit zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
01:51:50 openstackgerrit zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
02:45:14 openstackgerrit zhufl proposed openstack/nova master: Pass kwargs to exception to get better format of error message https://review.openstack.org/643501
02:52:51 openstackgerrit Merged openstack/nova stable/queens: Fix WeighedHost logging regression https://review.openstack.org/641359
04:13:24 openstackgerrit Merged openstack/nova master: Add functional test to delete a server while in VERIFY_RESIZE https://review.openstack.org/615644
05:09:56 openstackgerrit Merged openstack/nova stable/pike: Migrate nova v2.0 legacy job to zuulv3 https://review.openstack.org/620579
07:17:45 openstackgerrit zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
08:01:18 kashyap efried: dansmith: Caught up with the scrollback on #openstack-meeting. And this is the Intel's equivalent for AMD's SEV: "MKTME": https://www.redhat.com/archives/libvir-list/2019-February/msg01730.html
08:01:59 kashyap dansmith: efried: The enablement of "MKTME" (Multikey Total Memory Encryption) is not merged in kernel itself yet.
08:02:36 kashyap Then comes the design and implementation part in QEMU and libvirt (which is what the above libvirt mailing list URL points to).
08:17:39 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances https://review.openstack.org/627765
08:45:18 bauzas good morning Nova
08:47:52 hrw morning
10:07:45 kashyap Probably it's OK to not check too. "Distraction is the bane" ;-)
10:18:38 hrw ;D
10:18:48 hrw zuul -> irc gate would be lovely
11:11:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix ProviderUsageBaseTestCase._run_periodics for multi-cell https://review.openstack.org/641179
11:11:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Improve CinderFixtureNewAttachFlow https://review.openstack.org/639382
11:11:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1818914 https://review.openstack.org/641521
11:11:30 openstack bug 1818914 in OpenStack Compute (nova) "Hypervisor resource usage on source still shows old flavor usage after resize confirm until update_available_resource periodic runs" [Low,In progress] https://launchpad.net/bugs/1818914 - Assigned to Matt Riedemann (mriedem)
11:11:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unused context parameter from RT._get_instance_type https://review.openstack.org/641792
11:11:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Update usage in RT.drop_move_claim during confirm resize https://review.openstack.org/641806
11:11:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Migration.cross_cell_move and get_by_uuid https://review.openstack.org/614012
11:11:31 openstackgerrit Matt Riedemann proposed openstack/nova master: Add InstanceAction/Event create() method https://review.openstack.org/614036
11:11:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Add Instance.hidden field https://review.openstack.org/631123
11:11:32 openstackgerrit Matt Riedemann proposed openstack/nova master: Add TargetDBSetupTask https://review.openstack.org/627892
11:11:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Add CrossCellMigrationTask https://review.openstack.org/631581
11:11:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Execute TargetDBSetupTask https://review.openstack.org/633853
11:11:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Add can_connect_volume() compute driver method https://review.openstack.org/621313

Earlier   Later