*** tosky has quit IRC | 00:01 | |
*** gibi has joined #openstack-nova | 00:19 | |
openstackgerrit | Merged openstack/nova master: Add service version check for live migrate with qos https://review.opendev.org/713679 | 00:28 |
---|---|---|
openstackgerrit | Merged openstack/nova master: Add service version check for evacuate with qos https://review.opendev.org/713680 | 00:29 |
openstackgerrit | Merged openstack/nova stable/queens: Stop setting glance_api_version in cinder.conf in nova-live-migration https://review.opendev.org/713843 | 00:29 |
openstackgerrit | Merged openstack/nova stable/queens: Replace ansible --sudo with --become in live_migration/hooks scripts https://review.opendev.org/713844 | 00:29 |
openstackgerrit | Merged openstack/nova stable/queens: nova-live-migration: Wait for n-cpu services to come up after configuring Ceph https://review.opendev.org/713845 | 00:29 |
*** TxGirlGeek has quit IRC | 00:33 | |
*** hongbin has quit IRC | 00:37 | |
*** hongbin has joined #openstack-nova | 00:41 | |
openstackgerrit | melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478 | 00:49 |
openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | 00:49 |
*** mlavalle has quit IRC | 00:54 | |
*** brinzhang_ has joined #openstack-nova | 00:54 | |
*** brinzhang has quit IRC | 00:58 | |
*** brinzhang_ has quit IRC | 01:00 | |
*** brinzhang_ has joined #openstack-nova | 01:00 | |
*** macz_ has quit IRC | 01:01 | |
*** brinzhang_ has quit IRC | 01:02 | |
*** brinzhang_ has joined #openstack-nova | 01:02 | |
*** Liang__ has joined #openstack-nova | 01:17 | |
*** zhanglong has joined #openstack-nova | 01:23 | |
*** lbragstad__ has joined #openstack-nova | 01:33 | |
*** lbragstad_ has quit IRC | 01:36 | |
*** spatel has joined #openstack-nova | 01:39 | |
*** spatel has quit IRC | 01:44 | |
*** zhanglong has quit IRC | 01:50 | |
*** zhanglong has joined #openstack-nova | 01:56 | |
*** gyee has quit IRC | 02:02 | |
*** zhanglong has quit IRC | 02:11 | |
*** lbragstad__ has quit IRC | 02:13 | |
*** zhanglong has joined #openstack-nova | 02:14 | |
*** tetsuro_ has joined #openstack-nova | 02:46 | |
*** tetsuro has quit IRC | 02:49 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] fixing some nits in instance actions policy tests https://review.opendev.org/714560 | 03:02 |
*** mkrai has joined #openstack-nova | 03:05 | |
*** mmethot has joined #openstack-nova | 03:09 | |
*** mmethot_ has quit IRC | 03:11 | |
openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: WIP Microversion 2.83 - action event fault details https://review.opendev.org/714561 | 03:16 |
*** tetsuro has joined #openstack-nova | 03:18 | |
openstackgerrit | melanie witt proposed openstack/nova stable/train: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/713116 | 03:21 |
openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | 03:21 |
*** tetsuro_ has quit IRC | 03:22 | |
*** mkrai has quit IRC | 03:23 | |
*** mkrai_ has joined #openstack-nova | 03:23 | |
*** hongbin has quit IRC | 03:29 | |
*** brinzhang has joined #openstack-nova | 03:33 | |
*** tetsuro has quit IRC | 03:35 | |
*** zhanglong has quit IRC | 03:36 | |
*** brinzhang_ has quit IRC | 03:36 | |
*** brinzhang has quit IRC | 03:37 | |
*** brinzhang has joined #openstack-nova | 03:38 | |
openstackgerrit | Kevin Zhao proposed openstack/nova master: [DNM] Run tempest jobs on Arm64 needed fix https://review.opendev.org/714564 | 03:40 |
*** brinzhang_ has joined #openstack-nova | 03:54 | |
*** brinzhang has quit IRC | 03:57 | |
*** brinzhang has joined #openstack-nova | 03:58 | |
*** brinzhang_ has quit IRC | 03:59 | |
*** brinzhang_ has joined #openstack-nova | 04:01 | |
*** larainema has joined #openstack-nova | 04:03 | |
*** brinzhang has quit IRC | 04:03 | |
*** brinzhang has joined #openstack-nova | 04:04 | |
*** brinzhang_ has quit IRC | 04:07 | |
*** tetsuro has joined #openstack-nova | 04:11 | |
*** jamesdenton has quit IRC | 04:30 | |
*** mkrai_ has quit IRC | 04:48 | |
*** mkrai has joined #openstack-nova | 04:49 | |
*** brinzhang has quit IRC | 04:55 | |
*** brinzhang has joined #openstack-nova | 04:55 | |
*** ratailor has joined #openstack-nova | 04:58 | |
*** jamesdenton has joined #openstack-nova | 05:03 | |
*** brinzhang_ has joined #openstack-nova | 05:04 | |
*** brinzhang_ has joined #openstack-nova | 05:06 | |
*** brinzhang has quit IRC | 05:07 | |
*** brinzhang has joined #openstack-nova | 05:09 | |
*** brinzhang_ has quit IRC | 05:12 | |
*** brinzhang_ has joined #openstack-nova | 05:12 | |
*** brinzhang has quit IRC | 05:15 | |
*** udesale has joined #openstack-nova | 05:19 | |
*** evrardjp has quit IRC | 05:36 | |
*** evrardjp has joined #openstack-nova | 05:36 | |
*** links has joined #openstack-nova | 05:37 | |
*** zhanglong has joined #openstack-nova | 05:38 | |
*** spatel has joined #openstack-nova | 05:40 | |
*** spatel has quit IRC | 05:46 | |
*** mkrai has quit IRC | 05:50 | |
*** mkrai_ has joined #openstack-nova | 05:50 | |
*** brinzhang_ has quit IRC | 05:54 | |
*** tetsuro_ has joined #openstack-nova | 06:27 | |
*** tetsuro has quit IRC | 06:31 | |
*** vishalmanchanda has joined #openstack-nova | 06:32 | |
*** dklyle has quit IRC | 06:34 | |
*** dpawlik has joined #openstack-nova | 07:16 | |
*** mkrai_ has quit IRC | 07:24 | |
*** mkrai_ has joined #openstack-nova | 07:24 | |
*** ircuser-1 has joined #openstack-nova | 07:30 | |
*** nightmare_unreal has joined #openstack-nova | 07:33 | |
*** mkrai_ has quit IRC | 07:37 | |
*** jawad_axd has joined #openstack-nova | 07:39 | |
*** links has quit IRC | 07:47 | |
*** links has joined #openstack-nova | 07:54 | |
*** belmoreira has joined #openstack-nova | 07:56 | |
*** tkajinam has quit IRC | 08:05 | |
*** slaweq has joined #openstack-nova | 08:05 | |
*** ralonsoh has joined #openstack-nova | 08:13 | |
*** zhanglong has quit IRC | 08:14 | |
*** xek has joined #openstack-nova | 08:15 | |
*** tesseract has joined #openstack-nova | 08:16 | |
*** zhanglong has joined #openstack-nova | 08:23 | |
*** rpittau|afk is now known as rpittau | 08:26 | |
*** zhanglong has quit IRC | 08:27 | |
*** zhanglong has joined #openstack-nova | 08:29 | |
*** Luzi has joined #openstack-nova | 08:29 | |
*** mkrai has joined #openstack-nova | 08:34 | |
*** amoralej|off is now known as amoralej | 08:40 | |
*** mkrai has quit IRC | 08:40 | |
*** mkrai has joined #openstack-nova | 08:40 | |
*** tosky has joined #openstack-nova | 08:41 | |
*** zhanglong has quit IRC | 08:42 | |
*** zhanglong has joined #openstack-nova | 08:46 | |
*** dtantsur|afk is now known as dtantsur | 08:47 | |
*** ccamacho has joined #openstack-nova | 08:48 | |
*** priteau has joined #openstack-nova | 08:56 | |
*** maciejjozefczyk has joined #openstack-nova | 08:56 | |
*** brinzhang has joined #openstack-nova | 09:04 | |
*** brinzhang has quit IRC | 09:05 | |
*** brinzhang_ has joined #openstack-nova | 09:06 | |
*** brinzhang_ has quit IRC | 09:07 | |
*** brinzhang_ has joined #openstack-nova | 09:07 | |
brinzhang_ | sean-k-mooney: https://review.opendev.org/#/c/674726/3/nova/compute/api.py@3389, do we need to block the rebuild? | 09:08 |
*** brinzhang_ has quit IRC | 09:09 | |
*** brinzhang_ has joined #openstack-nova | 09:09 | |
brinzhang_ | In https://review.opendev.org/#/c/631245/62/nova/virt/libvirt/driver.py@3484, when we rebuild a server with accerators, but I think the accel_info always None, so I think we also should block the rebuild compute API | 09:10 |
*** brinzhang_ has quit IRC | 09:10 | |
*** brinzhang_ has joined #openstack-nova | 09:11 | |
*** ccamacho has quit IRC | 09:15 | |
*** zhanglong has quit IRC | 09:25 | |
*** brinzhang has joined #openstack-nova | 09:31 | |
*** Liang__ has quit IRC | 09:32 | |
*** brinzhang_ has quit IRC | 09:33 | |
openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83 - action event fault details https://review.opendev.org/714561 | 09:36 |
*** ociuhandu has joined #openstack-nova | 09:39 | |
*** tbachman has quit IRC | 09:39 | |
*** tbachman has joined #openstack-nova | 09:39 | |
*** ociuhandu has quit IRC | 09:41 | |
*** spatel has joined #openstack-nova | 09:42 | |
*** tetsuro_ has quit IRC | 09:43 | |
*** spatel has quit IRC | 09:47 | |
*** ganso has quit IRC | 09:47 | |
*** priteau has quit IRC | 09:48 | |
openstackgerrit | Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856 | 09:53 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593 | 09:53 |
*** lbragstad__ has joined #openstack-nova | 10:14 | |
*** mkrai has quit IRC | 10:24 | |
*** mkrai_ has joined #openstack-nova | 10:24 | |
openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459 | 10:30 |
stephenfin | gibi: could you revisit https://review.opendev.org/#/c/666915/ today, please? | 10:30 |
gibi | stephenfin: sure | 10:30 |
stephenfin | ta | 10:30 |
nightmare_unreal | hey can someone look at my patch . I am stuck. | 10:31 |
nightmare_unreal | a comment in right direction can be helpful. Thanks | 10:31 |
openstackgerrit | John Garbutt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/712137 | 10:32 |
openstackgerrit | John Garbutt proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/712140 | 10:32 |
openstackgerrit | John Garbutt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/712141 | 10:32 |
*** ganso has joined #openstack-nova | 10:35 | |
*** hrw has quit IRC | 10:35 | |
*** udesale_ has joined #openstack-nova | 10:36 | |
*** udesale has quit IRC | 10:38 | |
gibi | stephenfin: responded. | 10:40 |
gibi | nightmare_unreal: link the patch please | 10:41 |
nightmare_unreal | https://review.opendev.org/714459 | 10:42 |
nightmare_unreal | bug : https://bugs.launchpad.net/nova/+bug/1868531 | 10:42 |
openstack | Launchpad bug 1868531 in OpenStack Compute (nova) "nova manage placement doesn't support registration per Cell." [Undecided,New] - Assigned to jayaditya gupta (jayssj11) | 10:42 |
gibi | looking | 10:44 |
nightmare_unreal | Thanks | 10:46 |
openstackgerrit | Merged openstack/nova master: [Trivial] fixing some nits in instance actions policy tests https://review.opendev.org/714560 | 10:52 |
brinzhang | stephenfin: can you review this patch https://review.opendev.org/#/c/694430/? | 10:52 |
gibi | nightmare_unreal: I left some comment I hope they help | 10:54 |
nightmare_unreal | thanks | 10:55 |
*** martinkennelly has joined #openstack-nova | 10:58 | |
* gibi lunches | 11:01 | |
*** tbachman_ has joined #openstack-nova | 11:04 | |
*** tbachman has quit IRC | 11:06 | |
*** tbachman_ is now known as tbachman | 11:06 | |
*** tesseract has quit IRC | 11:11 | |
*** tesseract has joined #openstack-nova | 11:20 | |
* lyarwood pokes his head in for 10mins before going AFK again to deal with the UK lockdown. | 11:22 | |
lyarwood | kashyap: https://review.opendev.org/#/c/696834/ - would you mind hitting this again? | 11:22 |
lyarwood | elod: https://review.opendev.org/#/c/713036/ - should be ready for stable/pike now, hopefully unblocking the branch. | 11:23 |
kashyap | lyarwood: Yep, a sec | 11:23 |
elod | lyarwood: thanks, looking! | 11:24 |
elod | lyarwood: just a minor thing: it's not backkported from the latest commit | 11:26 |
elod | from queens | 11:26 |
lyarwood | hmm it should be | 11:26 |
* lyarwood checks | 11:26 | |
lyarwood | elod: ah it's PS2, I pushed the stable/pike change to the stable/queens change in PS3 by accident | 11:27 |
kashyap | lyarwood: "+1", FWIW. I don't see anything else that is blocking there. | 11:27 |
lyarwood | elod: https://review.opendev.org/#/c/713845/2..4 | 11:27 |
* lyarwood was trying to do about 5 things at once yesterday while doing that, apologies. | 11:28 | |
lyarwood | elod: FWIW I've never really enforced the commit SHA's being correct | 11:28 |
lyarwood | elod: we end up in an endless game up updates if the earlier changes are rebased etc | 11:29 |
lyarwood | elod: if the content is the same I typically ACK | 11:29 |
elod | lyarwood: well, yes, other teams don't even bother with conflict lines either. and it's not a big thing, i know. maybe this forces people to backport things when the patch on newer branch has merged o:) | 11:33 |
*** dpawlik has quit IRC | 11:33 | |
*** dpawlik has joined #openstack-nova | 11:34 | |
elod | fot me it seemed mriedem kept the rules quite strictly so I try to do the same :D | 11:34 |
lyarwood | yup that's cool | 11:34 |
lyarwood | I wanted to revisit the SHA issue at some point but until then feel free to enforce if you want to | 11:34 |
*** Luzi has quit IRC | 11:35 | |
elod | lyarwood: ok, no problem, I'll edit the SHA as that is the only thing that's needed :) | 11:37 |
*** psachin has joined #openstack-nova | 11:43 | |
openstackgerrit | Elod Illes proposed openstack/nova stable/pike: nova-live-migration: Wait for n-cpu services to come up after configuring Ceph https://review.opendev.org/713036 | 11:44 |
lyarwood | elod: many thanks | 11:51 |
elod | lyarwood: thanks, too :) | 11:52 |
*** derekh has joined #openstack-nova | 11:56 | |
*** ccstone has quit IRC | 11:56 | |
*** ccstone has joined #openstack-nova | 11:57 | |
lyarwood | gibi: https://review.opendev.org/#/c/696834/ - would you mind taking a look at this today please? | 12:01 |
*** sapd1_x has joined #openstack-nova | 12:07 | |
kevinz | stephenfin: lyarwood: would you mind to re-visit this one: https://review.opendev.org/#/c/709494/? Thanks | 12:11 |
*** Luzi has joined #openstack-nova | 12:13 | |
*** mkrai_ has quit IRC | 12:14 | |
alex_xu | dansmith: query another thing, https://review.opendev.org/#/c/713354/1/nova/objects/fields.py@277, as I understand, we can't change a field. But we won't support mixed policy in the middle of upgrade, we only enable it after upgrade finished. so can we change a field like that? just do a version object bump. | 12:17 |
* alex_xu looking for a service GET /dansmith?question=blahblahblah | 12:18 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915 | 12:19 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | 12:19 |
stephenfin | gibi: Tests added ^ | 12:20 |
luyao | alex_xu: https://review.opendev.org/#/c/687856 comments addressed, thanks for review | 12:28 |
luyao | stephenfin: could you review vpmem live migration https://review.opendev.org/#/c/687856 if you get time? | 12:29 |
stephenfin | kevinz: replied. one more thing needed, I think | 12:30 |
stephenfin | luyao: I can, but I was waiting until alex_xu had most of his things addressed before :) | 12:31 |
*** nweinber has joined #openstack-nova | 12:33 | |
luyao | stephenfin: OK, thanks | 12:35 |
openstackgerrit | Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494 | 12:36 |
alex_xu | stephenfin: I think most of my things addressed... | 12:53 |
*** ociuhandu has joined #openstack-nova | 12:55 | |
*** jawad_axd has quit IRC | 13:02 | |
*** jawad_axd has joined #openstack-nova | 13:03 | |
*** psachin has quit IRC | 13:05 | |
*** ociuhandu has quit IRC | 13:07 | |
*** jawad_axd has quit IRC | 13:08 | |
*** mgariepy has quit IRC | 13:09 | |
*** amoralej is now known as amoralej|lunch | 13:20 | |
*** mgariepy has joined #openstack-nova | 13:23 | |
*** jawad_axd has joined #openstack-nova | 13:24 | |
*** derekh has quit IRC | 13:32 | |
*** jawad_axd has quit IRC | 13:32 | |
*** jawad_axd has joined #openstack-nova | 13:33 | |
gibi | lyarwood, stephenfin: ack on both, I'm on a call will check it after | 13:33 |
openstackgerrit | John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139 | 13:35 |
*** dklyle has joined #openstack-nova | 13:50 | |
*** ratailor has quit IRC | 13:52 | |
*** liuyulong has joined #openstack-nova | 13:54 | |
*** tkajinam has joined #openstack-nova | 13:57 | |
*** ociuhandu has joined #openstack-nova | 14:02 | |
stephenfin | lyarwood: do my comments on https://review.opendev.org/#/c/712607/ make sense? | 14:04 |
openstackgerrit | John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142 | 14:04 |
dansmith | artom: sorry, just got to your base test patch.. fix that thing right quick and I can +2 | 14:05 |
*** mkrai has joined #openstack-nova | 14:05 | |
* artom panics and hax stuff real quix | 14:05 | |
artom | (Dammit, missed the panix possibility) | 14:05 |
dansmith | alex_xu: we can but you have to do it in a specific way.. I commented | 14:07 |
dansmith | alex_xu: the hint is the test_objects hash change with no version bump | 14:08 |
dansmith | alex_xu: good job catching that | 14:08 |
openstackgerrit | John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139 | 14:10 |
openstackgerrit | John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142 | 14:10 |
*** brinzhang has quit IRC | 14:10 | |
*** brinzhang has joined #openstack-nova | 14:11 | |
*** derekh has joined #openstack-nova | 14:11 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into base class https://review.opendev.org/714228 | 14:11 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546 | 14:11 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | 14:11 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | 14:11 |
openstack | bug 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) | 14:11 |
artom | dansmith, ^^ | 14:11 |
*** ociuhandu has quit IRC | 14:14 | |
artom | \o/ | 14:14 |
*** ociuhandu has joined #openstack-nova | 14:14 | |
*** amoralej|lunch is now known as amoralej | 14:15 | |
*** mgariepy has quit IRC | 14:16 | |
nightmare_unreal | gibi: what if user pass both instance-id and cell-id | 14:20 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593 | 14:20 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856 | 14:20 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/714653 | 14:20 |
*** mgariepy has joined #openstack-nova | 14:23 | |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Take the instance dedicated CPU list from 'cpu_pinning' if possible https://review.opendev.org/713352 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: 'cpu_pinning_requested' property could be derived directly from cpu_policy https://review.opendev.org/713353 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Introduce 'MIXED' CPU allocation policy for instance https://review.opendev.org/713354 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Introduce the interface of creating 'MIXED' policy instance through 'PCPU' and 'VCPU' https://review.opendev.org/713355 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: libvirt: support to create instance with dedicated and shared CPUs https://review.opendev.org/714655 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Remove 'InstanceNUMACell.cpu_pinning_requested' field https://review.opendev.org/714656 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: Refactor the code in checking available host CPUs https://review.opendev.org/714657 | 14:30 |
openstackgerrit | Huaqiang Wang proposed openstack/nova master: [WIP] An alternative way for keeping instance dedicated CPUs https://review.opendev.org/714658 | 14:30 |
*** jawad_axd has quit IRC | 14:31 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915 | 14:37 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | 14:37 |
openstackgerrit | Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494 | 14:40 |
*** spatel has joined #openstack-nova | 14:41 | |
openstackgerrit | Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494 | 14:47 |
gibi | nightmare_unreal: good point. You can make the two filters (instance_uuid and cell_id) mutually exclusive | 14:51 |
*** Luzi has quit IRC | 14:52 | |
gibi | I don't see any reason why the user want to filter for both at the same time | 14:52 |
nightmare_unreal | yup I was thinking something like that in for loop | 14:52 |
nightmare_unreal | gibi: I am not sure but can we add/append to cells ? for e.g. if instance_id is provided cells variable will have cell_id of that instance and along with cell_id is also passed, can we just append it in cells. it can make code more simple | 14:53 |
huaqiang | hi stephenfin,in my new patches, I haven't add a 'cpu_policy' property to 'InstanceNUMATopology' object. | 14:56 |
huaqiang | If I do, I have to replace every 'topology.cpu_pinning_requested' line with 'if topology.cpu_policy in "DEDICATED or MIXED"' | 14:57 |
huaqiang | I think 'topology.cpu_pinning_requested' and 'topology.cpu_sharing_requested' are more concise. | 14:58 |
gibi | nightmare_unreal: there is not much reason to append to the cells variable if both instance_uuid and cell_id is provided. Because today if the instance_uuid is passed the it means that heal _only_ that instance. | 14:59 |
gibi | so if you just append the extra cell specified by the cell_id then you still only heal one instance | 14:59 |
gibi | not one instance + a whole other cell | 15:00 |
nightmare_unreal | ah yes | 15:00 |
*** tkajinam has quit IRC | 15:04 | |
stephenfin | huaqiang: I don't think that replacement is that much work (I've it done locally and it's quite concise). However, adding 'cpu_policy' to the 'InstanceNUMATopology' object is a lot of work though that can wait, yes. | 15:08 |
*** ociuhandu has quit IRC | 15:08 | |
*** ociuhandu has joined #openstack-nova | 15:09 | |
sean-k-mooney | we can infer the policy from the cpu pinning info in anycase | 15:10 |
sean-k-mooney | the cpu_policy is also somewhat incorrectly stored in the insance numa cell https://github.com/openstack/nova/blob/master/nova/objects/instance_numa.py#L59-L60 | 15:11 |
*** macz_ has joined #openstack-nova | 15:11 | |
gibi | stephenfin: I'm +2 on the image-metadata-prefiltering | 15:12 |
sean-k-mooney | it should be in the InstanceNUMATopology instead bug you could add a property that would return the value form the first cell and in a later version add a cpu_policy field and remove it form teh instance numa cell | 15:13 |
stephenfin | sean-k-mooney: yeah, I'd a suggestion to duplicate it to the topology and slowly deprecate use of the cell field, but it's not really related to this | 15:13 |
huaqiang | stephenfin: nice! | 15:13 |
sean-k-mooney | right its not related to this but we shoudl eventually move it | 15:14 |
huaqiang | stephenfin, can you share some detail? or you can submit the changes | 15:15 |
huaqiang | then I do the work under your changes | 15:16 |
stephenfin | for the replacement? | 15:16 |
huaqiang | yes | 15:16 |
stephenfin | sure | 15:16 |
sean-k-mooney | by the way if you want to start using the value form the isntanceNUMATopology without bumping the verion fo the object you can add this | 15:17 |
sean-k-mooney | @property | 15:17 |
sean-k-mooney | def cpu_policy(self): | 15:17 |
sean-k-mooney | return self.cells[0].cpu_policy if self.cells else "shared" | 15:17 |
stephenfin | sean-k-mooney: that's what I have | 15:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675 | 15:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676 | 15:17 |
stephenfin | albeit with a check to make sure all the cells have the same value and error out if not (it shouldn't ever be triggered) | 15:17 |
huaqiang | sean-k-mooney,stephenfin: thanks | 15:18 |
sean-k-mooney | cool | 15:18 |
sean-k-mooney | stephenfin: ya so that cant happen today unless you messed with the db | 15:18 |
sean-k-mooney | so i did not bother with the check | 15:18 |
openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459 | 15:22 |
sean-k-mooney | ok ill be afk for about an hour or so. bbl | 15:22 |
*** links has quit IRC | 15:24 | |
*** ociuhandu has quit IRC | 15:24 | |
*** gyee has joined #openstack-nova | 15:26 | |
gibi | lyarwood, stephenfin: is this a bugfix or a feature? https://review.opendev.org/#/c/696834 | 15:31 |
stephenfin | gibi: It's probably closer to a feature. We couldn't do this before and now we can | 15:32 |
stephenfin | Should there be a specless blueprint? I didn't think of that | 15:32 |
stephenfin | alex_xu: Think you could look at this? https://review.opendev.org/#/c/666915/ | 15:34 |
*** mriedem has joined #openstack-nova | 15:35 | |
gibi | stephenfin: would be nice to track it somehow. I'm OK to have a specless bp as the impl basically done and only affects the libvirt driver | 15:37 |
*** tesseract has quit IRC | 15:37 | |
gibi | this is a bit of paperwork I know | 15:37 |
openstackgerrit | Luigi Toscano proposed openstack/nova stable/pike: Remove exp legacy-tempest-dsvm-full-devstack-plugin-nfs https://review.opendev.org/702061 | 15:39 |
*** tesseract has joined #openstack-nova | 15:40 | |
*** lbragstad__ has quit IRC | 15:40 | |
*** dtruong has quit IRC | 15:47 | |
*** dtruong has joined #openstack-nova | 15:48 | |
*** spatel has quit IRC | 15:50 | |
openstackgerrit | Merged openstack/nova-specs master: Move 'live migration with vpmems' spec https://review.opendev.org/708229 | 15:50 |
*** ociuhandu has joined #openstack-nova | 16:02 | |
tosky | oh, the last nova-live-migration fix is finally fine on stable/pike: it just needs another +2 (and a +w)! https://review.opendev.org/#/c/713036/ | 16:08 |
*** mkrai has quit IRC | 16:08 | |
*** sapd1_x has quit IRC | 16:08 | |
*** mkrai has joined #openstack-nova | 16:11 | |
*** TxGirlGeek has joined #openstack-nova | 16:11 | |
openstackgerrit | Merged openstack/nova master: Enforce os-traits/SUPPORTED_STORAGE_BUSES sync https://review.opendev.org/709233 | 16:20 |
*** belmoreira has quit IRC | 16:22 | |
*** dtantsur is now known as dtantsur|afk | 16:25 | |
openstackgerrit | John Garbutt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/712143 | 16:27 |
dansmith | kinda seems like the openstack sdk test is failing a lot | 16:36 |
dansmith | I didn't see that mentioned on the recent gate bug thread | 16:36 |
dansmith | it has failed a couple times in a row with a bunch of missing packages | 16:37 |
dansmith | ah, known mirror failure which might be fixed now | 16:38 |
*** artom has quit IRC | 16:40 | |
*** artom has joined #openstack-nova | 16:40 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Add validation for 'cpu_realtime_mask' https://review.opendev.org/468203 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Update and correct typing information https://review.opendev.org/714694 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add typing information https://review.opendev.org/714695 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Split instance NUMA object tests https://review.opendev.org/714696 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Replace 'cpu_pinning_requested' helper https://review.opendev.org/714697 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Don't consider overhead CPUs for unpinned instances https://review.opendev.org/714698 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Remove handling of pre-Train compute nodes https://review.opendev.org/714699 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Rework 'get_realtime_constraint' https://review.opendev.org/714700 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: 'numa_fit_instance_to_host' requires 'limits' argument https://review.opendev.org/714702 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Remove '_numa_fit_instance_cell_with_pinning' https://review.opendev.org/714703 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! objects: Add 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/714704 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! objects: Add 'InstanceNUMACell.pcpuset' field https://review.opendev.org/714705 | 16:43 |
stephenfin | huaqiang: ^ | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! hardware: Add support for 'hw:cpu_dedicated_mask' extra spec https://review.opendev.org/714706 | 16:43 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP! hardware: Start configuring 'InstanceNUMACell.pcpuset' https://review.opendev.org/714707 | 16:43 |
openstackgerrit | John Garbutt proposed openstack/nova master: Update limit APIs https://review.opendev.org/712707 | 16:53 |
openstackgerrit | John Garbutt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/712749 | 16:53 |
*** udesale_ has quit IRC | 16:53 | |
*** lbragstad__ has joined #openstack-nova | 17:02 | |
*** lbragstad__ is now known as lbragstad | 17:04 | |
*** mlavalle has joined #openstack-nova | 17:06 | |
sean-k-mooney | stephenfin: im going to rebase https://review.opendev.org/#/c/666915/24 since its in merge conflict | 17:07 |
stephenfin | sean-k-mooney: go for it | 17:07 |
*** mkrai has quit IRC | 17:16 | |
*** rpittau is now known as rpittau|afk | 17:18 | |
*** maciejjozefczyk has quit IRC | 17:21 | |
*** maciejjozefczyk has joined #openstack-nova | 17:21 | |
*** ociuhandu has quit IRC | 17:24 | |
*** ociuhandu has joined #openstack-nova | 17:25 | |
*** maciejjozefczyk has quit IRC | 17:26 | |
openstackgerrit | sean mooney proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915 | 17:30 |
openstackgerrit | sean mooney proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | 17:30 |
*** ociuhandu has quit IRC | 17:32 | |
*** evrardjp has quit IRC | 17:36 | |
*** evrardjp has joined #openstack-nova | 17:36 | |
*** vesper11 has quit IRC | 17:45 | |
*** sean-k-mooney has quit IRC | 17:45 | |
*** mgoddard has quit IRC | 17:45 | |
*** openstackstatus has quit IRC | 17:45 | |
*** iurygregory has quit IRC | 17:45 | |
*** alex_xu has quit IRC | 17:45 | |
*** vesper11 has joined #openstack-nova | 17:47 | |
*** openstackstatus has joined #openstack-nova | 17:47 | |
*** ChanServ sets mode: +v openstackstatus | 17:47 | |
*** irclogbot_2 has quit IRC | 17:47 | |
*** irclogbot_2 has joined #openstack-nova | 17:48 | |
*** mgoddard has joined #openstack-nova | 17:49 | |
openstackgerrit | Merged openstack/nova stable/pike: nova-live-migration: Wait for n-cpu services to come up after configuring Ceph https://review.opendev.org/713036 | 17:49 |
openstackgerrit | John Garbutt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/713301 | 17:53 |
*** derekh has quit IRC | 18:01 | |
*** xek_ has joined #openstack-nova | 18:08 | |
*** xek has quit IRC | 18:11 | |
*** ociuhandu has joined #openstack-nova | 18:12 | |
openstackgerrit | Elod Illes proposed openstack/nova stable/pike: pike-only: remove broken non-voting ceph jobs https://review.opendev.org/700072 | 18:18 |
*** iurygregory has joined #openstack-nova | 18:19 | |
*** sean-k-mooney has joined #openstack-nova | 18:21 | |
*** nightmare_unreal has quit IRC | 18:22 | |
*** ociuhandu has quit IRC | 18:22 | |
*** larainema has quit IRC | 18:23 | |
*** ociuhandu has joined #openstack-nova | 18:23 | |
*** ociuhandu has quit IRC | 18:28 | |
*** amoralej is now known as amoralej|off | 18:31 | |
openstackgerrit | Elod Illes proposed openstack/nova stable/pike: Init HostState.failed_builds https://review.opendev.org/669538 | 18:33 |
*** macz_ has quit IRC | 18:47 | |
*** tesseract has quit IRC | 18:48 | |
*** macz_ has joined #openstack-nova | 18:48 | |
*** ociuhandu has joined #openstack-nova | 18:51 | |
*** dpawlik has quit IRC | 18:53 | |
*** martinkennelly has quit IRC | 19:03 | |
*** ralonsoh has quit IRC | 19:07 | |
*** maciejjozefczyk has joined #openstack-nova | 19:32 | |
*** seba has quit IRC | 19:44 | |
*** seba has joined #openstack-nova | 19:44 | |
*** rcernin|brb has quit IRC | 19:57 | |
*** vishalmanchanda has quit IRC | 19:59 | |
*** spatel has joined #openstack-nova | 20:01 | |
*** ociuhandu has quit IRC | 20:04 | |
*** amodi has joined #openstack-nova | 20:06 | |
*** ociuhandu has joined #openstack-nova | 20:06 | |
openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Improve metadata server performance with large security groups https://review.opendev.org/697523 | 20:11 |
*** ociuhandu has quit IRC | 20:11 | |
lyarwood | melwitt: https://review.opendev.org/#/q/I77ebada015f6522a300be4fa043fb8676458402b - could you hit this if you have time today? | 20:14 |
melwitt | lyarwood: sure | 20:14 |
lyarwood | melwitt: https://review.opendev.org/#/q/I8af2ad741ca08c3d88efb9aa817c4d1470491a23 - and this if you wouldn't mind | 20:15 |
melwitt | sure | 20:15 |
lyarwood | thanks :) | 20:16 |
melwitt | np | 20:17 |
*** maciejjozefczyk has quit IRC | 20:20 | |
*** nweinber has quit IRC | 20:48 | |
*** amodi has quit IRC | 21:01 | |
*** ociuhandu has joined #openstack-nova | 21:18 | |
*** mriedem has left #openstack-nova | 21:24 | |
*** ociuhandu has quit IRC | 21:29 | |
*** ociuhandu has joined #openstack-nova | 21:30 | |
*** ociuhandu has quit IRC | 21:35 | |
*** ociuhandu has joined #openstack-nova | 21:36 | |
*** xek_ has quit IRC | 21:43 | |
*** brinzhang_ has joined #openstack-nova | 21:56 | |
*** brinzhang has quit IRC | 22:00 | |
*** ociuhandu has quit IRC | 22:05 | |
*** gregwork has quit IRC | 22:21 | |
*** slaweq has quit IRC | 22:23 | |
*** lbragstad has quit IRC | 22:38 | |
*** rcernin has joined #openstack-nova | 22:45 | |
*** spatel has quit IRC | 22:49 | |
*** tkajinam has joined #openstack-nova | 22:53 | |
*** macz_ has quit IRC | 23:12 | |
*** alex_xu has joined #openstack-nova | 23:13 | |
*** bbowen has quit IRC | 23:14 | |
alex_xu | dansmith: thanks | 23:15 |
*** macz_ has joined #openstack-nova | 23:27 | |
*** macz_ has quit IRC | 23:32 | |
*** lbragstad has joined #openstack-nova | 23:46 | |
*** mlavalle has quit IRC | 23:54 | |
*** ociuhandu has joined #openstack-nova | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!