| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-11 | |||
| 19:07:57 | artom | I'm off to the gym, during the day, for a change, while Zuul runs | |
| 19:09:51 | efried | mriedem: Once that bypass merges, which should hopefully happen pdq (test-only patches don't run tempest/dsvm stuff, right?) you can recheck the world. | |
| 19:13:12 | mriedem | dansmith: would you see a use for a weigher config option for the vgpu/numa affinity weigher in sylvain's spec? https://review.opendev.org/#/c/650963/ | |
| 19:13:21 | mriedem | since we have config options to control all other weighers | |
| 19:13:38 | dansmith | just in general? I mean, I would expect all weighers to be controllable like that | |
| 19:13:48 | mriedem | in general yes | |
| 19:14:02 | dansmith | unless it was something SUPER airtight like the failed computes weigher | |
| 19:15:13 | dansmith | dude | |
| 19:15:20 | dansmith | you should have snipped that last line | |
| 19:15:43 | mriedem | ha | |
| 19:16:08 | mriedem | sorry | |
| 19:16:27 | mriedem | also not sure what he's proposing for the claim stuff, | |
| 19:16:52 | mriedem | normally if we were to link up the scheduler to some RT claim, we'd use the limits so that if you disable the filter we don't claim on those limits in the RT, | |
| 19:17:11 | mriedem | but with this, it sounds like the compute would always just pull the VGPU allocations and pass those onto the claim | |
| 19:19:17 | dansmith | erm | |
| 19:20:40 | dansmith | is this really adding new claims to the RT? | |
| 19:20:45 | mriedem | it says it is | |
| 19:21:05 | mriedem | https://review.opendev.org/#/c/650963/7/specs/train/approved/libvirt-vgpu-numa-affinity.rst@99 | |
| 19:21:21 | dansmith | yeah, that's the line I'm wondering about | |
| 19:29:28 | melwitt | does anyone know if the failed_builds count is exposed out of the REST API at all? | |
| 19:32:01 | mriedem | melwitt: it's not | |
| 19:32:16 | melwitt | danke | |
| 19:32:37 | mriedem | it's shoved into compute_nodes.stats which is a json blob in the db | |
| 19:33:34 | melwitt | ah ok, I was wondering about that too. I found the nova/compute/stats.py file and didn't know how it's used. thanks | |
| 19:34:14 | Nick_A | Is there a way to take online snapshots of lxd containers? | |
| 19:34:44 | mriedem | Nick_A: you should ask the lxd devs in #lxcontainers | |
| 19:34:52 | mriedem | https://github.com/openstack/nova-lxd#support-and-discussions | |
| 19:34:52 | Nick_A | thank you | |
| 20:11:04 | efried | dustinc: gonna do that now, mkay? | |
| 20:11:20 | dustinc | 👍 | |
| 20:23:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |
| 20:23:31 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664 | |
| 20:23:32 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028 | |
| 20:23:32 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027 | |
| 20:23:33 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691 | |
| 20:23:33 | openstackgerrit | Eric Fried proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690 | |
| 20:23:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023 | |
| 20:23:50 | efried | dustinc: ^ | |
| 20:44:56 | efried | f f f f. bug 1823251 hit https://review.opendev.org/#/c/664622/ after sitting in the queue for 5h. | |
| 20:44:57 | openstack | bug 1823251 in OpenStack Compute (nova) "Spike in TestNovaMigrationsMySQL.test_walk_versions/test_innodb_tables failures since April 1 2019 on limestone-regionone" [High,Confirmed] https://launchpad.net/bugs/1823251 | |
| 20:46:22 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/662584 | |
| 20:47:31 | sean-k-mooney | efried: is the 5 hour lag just a general issue with the gate being slow or is it related | |
| 20:48:07 | efried | sean-k-mooney: I'm sure it's just because we had a couple dozen patches in the queue (all of which failed on that issue of course) so that one got pushed. | |
| 20:51:08 | mordred | efried: ooh - and you've got the ironicclient patches too - that's so cool | |
| 20:51:37 | efried | mordred: Yes, dustinc has been working on those. I'm not sure how done he is (percentage-wise) - dustinc? | |
| 20:51:55 | sean-k-mooney | i have a ptach queue in neutron for a little over 2 and a half hours without it starting so i think the gate is slow to start jobs currently in general | |
| 20:52:21 | efried | mordred: I'm pretty sure we're going to need some feature parity for ironic in sdk before we can be all the way complete there, but fortunately dtantsur|afk knows his sdk AND his ironic :) | |
| 20:52:59 | dustinc | I am not even sure how far done I am percentage-wise | |
| 20:53:05 | mordred | efried: yah - and has already been working through feature parity for other usage | |
| 20:53:07 | dustinc | but it is coming along | |
| 20:55:24 | mordred | (we don't have one of those right now, because it wouldn't be valuable - but with nova using it to talk to services, it's an additinoal useful thing to check) | |
| 20:56:56 | openstackgerrit | Eric Fried proposed openstack/nova master: get_ksa_adapter: nix by-service-type confgrp hack https://review.opendev.org/664689 | |
| 20:59:25 | efried | mordred: probably not relevant until some of the above merges, eh? | |
| 20:59:39 | efried | mordred: We don't even have an approved blueprint yet :P | |
| 21:00:31 | redkrieg | does anyone have any troubleshooting tips for tracking down why cloud-init on my guests isn't getting ssh keys or passwords from metadata? it works for me in one of my regions but not another, I don't see anything odd in my nova metadata wsgi logs either. | |
| 21:01:49 | efried | sean-k-mooney: your patch is going to fail unless you rebase it on https://review.opendev.org/#/c/664622/ | |
| 21:02:12 | sean-k-mooney | which one? | |
| 21:02:39 | sean-k-mooney | the one i said is 2 and a half hours in the queue? | |
| 21:02:52 | sean-k-mooney | if so its a neutorn patch so its fine | |
| 21:03:11 | sean-k-mooney | my other nova paths will fail however yes | |
| 21:03:48 | sean-k-mooney | im not going to push anything more today so if that isnt merged tommorw ill rebase things on top of it as needed | |
| 21:04:30 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659246 | |
| 21:08:51 | mriedem | can i get some stable core love on https://review.opendev.org/#/q/topic:bug/1830747+branch:stable/rocky ? | |
| 21:09:06 | mriedem | working on flushing stable/rocky for a release | |
| 21:10:12 | mordred | efried: oh - yeah - definitely not until it's actually going to test something | |
| 21:21:30 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove _ContextAuthPlugin.get_endpoint* workaround https://review.opendev.org/664703 | |
| 21:29:55 | openstackgerrit | Merged openstack/nova master: Replace 'is comprised of' with 'comprises' https://review.opendev.org/663175 | |
| 21:32:19 | sean-k-mooney | can we revert ^ | |
| 21:32:26 | sean-k-mooney | efried: it is not correct | |
| 21:33:35 | sean-k-mooney | ... i guess i dont care enought but "is comprised of" was correct "comprises" as used in that patch is not. | |
| 21:34:17 | sean-k-mooney | "Nova is composed of multiple server processes" would also be correct | |
| 21:35:04 | efried | sean-k-mooney: google it | |
| 21:35:08 | efried | pick any result | |
| 21:35:51 | efried | "is composed of" would be correct. "is comprised of" is not, nor is "comprises of". | |
| 21:36:20 | efried | but it is correct as patched | |
| 21:36:38 | sean-k-mooney | the current wording read quite badly to me | |
| 21:37:04 | sean-k-mooney | the pacing feels wrong | |
| 21:38:59 | efried | I would have accepted "is composed of", but now that it's grammatically correct, I don't see it being worth changing. | |
| 21:39:49 | sean-k-mooney | i keep wanting to read it as if there is a comma after comprises | |
| 21:40:21 | sean-k-mooney | if i force my self t to read it as written the its fine | |
| 21:46:45 | sean-k-mooney | hum i can reluctanlty agree to never use comprises but https://www.grammarly.com/blog/comprise-vs-compose/ makes some sense | |
| 21:53:06 | melwitt | I'm surprised someone opened a doc bug for that. as far as I've known, "is comprised of" makes OK sense | |
| 21:53:33 | sean-k-mooney | melwitt: i was assuming it was a us vs uk thing | |
| 21:53:54 | melwitt | yeah, I assumed must be a regional thing | |
| 22:05:11 | efried | no, just such a common mistake it's almost become accepted | |
| 22:05:41 | efried | Ask asettle :P | |
| 22:07:00 | sean-k-mooney | got distracted reading the wiki article https://en.wikipedia.org/wiki/Comprised_of | |
| 22:07:20 | sean-k-mooney | apparently its is commonly used in us patents and dates forn the 18 century | |
| 22:08:03 | melwitt | ¯\_(ツ)_/¯ | |
| 22:08:17 | sean-k-mooney | so i think its on better footing then performant | |
| 22:08:54 | melwitt | haha right? | |
| 22:09:21 | efried | oh, please do not base real english on what's in patents | |
| 22:09:26 | melwitt | of all the other non-words we accept, "is comprised of" is the one to go after | |
| 22:10:16 | sean-k-mooney | efried: the stonger argument is dont belive everything you read on wikipedia :P | |
| 22:50:27 | openstackgerrit | Merged openstack/nova stable/rocky: libvirt: Avoid using os-brick encryptors when device_path isn't provided https://review.opendev.org/656463 | |
| 22:57:38 | openstackgerrit | Merged openstack/nova stable/rocky: Stop logging traceback when skipping quiesce https://review.opendev.org/660327 | |
| 23:16:58 | openstackgerrit | Merged openstack/nova stable/rocky: Restore connection_info after live migration rollback https://review.opendev.org/660375 | |
| 23:17:05 | openstackgerrit | Merged openstack/nova stable/rocky: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/660362 | |
| 23:33:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Make get_provider_by_name public and remove safe_connect https://review.opendev.org/664062 | |
| 23:37:16 | openstackgerrit | Merged openstack/nova stable/rocky: Replace the invalid index of nova-rocky releasenote https://review.opendev.org/663178 | |