| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-12 | |||
| 15:54:52 | melwitt | yeah... | |
| 15:54:55 | efried | If I know a dep is out of date, I'll enter the venv and pip install/upgrade the sucker manually. | |
| 15:55:02 | melwitt | I'd like to know too but I've no idea how to find out | |
| 15:55:04 | efried | Takes way less time than rebuilding the whole damn venv. | |
| 15:55:29 | melwitt | heh, I've done that in the past but nowadays I'm like meh | |
| 15:56:00 | stephenfin | efried: That. I personally let it rebuild but Real World Devs (TM) often fix this stuff themselves. I think a flag would be a good idea | |
| 15:56:08 | efried | Usually when I'm running tox it's because I actually want to run tests, not because I want to kick off a command and go eat lunch and come back to see if it has finished yet. | |
| 15:56:25 | stephenfin | ...as soon as we figure out what is taking that time when it's not rebuilding (the thing 'touch nova.*' solves) | |
| 15:56:58 | efried | Cool mon, let me know if you want me to try anything out. | |
| 15:57:43 | stephenfin | Ack. I'll try take a look tomorrow once I get somewhere on these functional tests for numa-aware-vswitch | |
| 15:57:59 | efried | That's not using nrp, is it? | |
| 15:58:12 | efried | (I haven't read the spec in yonks, have totally forgotten what it actually does) | |
| 15:58:58 | stephenfin | Nope, nothing to do with placement | |
| 15:59:33 | stephenfin | Not now, anyway. A fancier version of it might use it in the future but that's 2-3 cycles down the road, I'd imagine | |
| 16:03:01 | mriedem | TheJulia: do you know if there are any ironic jobs that can be used to test this rescue patch? https://review.openstack.org/#/c/416487/3 | |
| 16:03:05 | mriedem | https://review.openstack.org/#/c/416487/ | |
| 16:03:20 | mriedem | i know tempest tests rescue, we'd just have to flip the switch in some ironic job config | |
| 16:03:42 | TheJulia | mriedem: we've not merged them yet because of the nova patch not merging, looking it up | |
| 16:03:56 | mriedem | TheJulia: ok but we can see a depends-on patch passing tests right? | |
| 16:03:59 | TheJulia | that being said, the logs show it working in the proposed tempest patch | |
| 16:04:07 | mriedem | ah https://review.openstack.org/#/c/528699/ | |
| 16:04:09 | TheJulia | mriedem: already did if my memory is correct | |
| 16:04:20 | TheJulia | yup, thats the one | |
| 16:05:18 | TheJulia | https://review.openstack.org/#/c/528704/ | |
| 16:06:03 | mriedem | ok that depends on the plugin change which depends on the nova change, i see | |
| 16:06:48 | TheJulia | and I see you already rechecked it | |
| 16:06:49 | TheJulia | thanks! | |
| 16:07:00 | openstackgerrit | MultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418 | |
| 16:12:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Follow up for Ie49d605c66062d2548241d7e04f5a2a6b98c011e https://review.openstack.org/582292 | |
| 16:13:01 | kashyap | mriedem: The above commit message is really odd | |
| 16:13:40 | kashyap | (It's logically correct. But it can be turned into a sentence w/o a commit has :-)) | |
| 16:17:36 | openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource provider examples https://review.openstack.org/553461 | |
| 16:27:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata https://review.openstack.org/564452 | |
| 16:27:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440 | |
| 16:27:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441 | |
| 16:30:42 | anupn | mriedem_away: Yes n-cpu showed the sec group used was wrong, and after changing that instance was launched :) | |
| 16:42:57 | stephenfin | dansmith: PEBCAK http://paste.openstack.org/show/725735/ (line 14) | |
| 16:43:25 | dansmith | stephenfin: heh, good | |
| 16:44:04 | stephenfin | Two days. | |
| 16:46:23 | melwitt | I know the feeling all too well | |
| 16:47:54 | dansmith | pebkac is way better than some obscure bug | |
| 16:48:05 | dansmith | feels stupid, but it's almost always better | |
| 16:49:30 | melwitt | true that | |
| 16:51:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field https://review.openstack.org/564441 | |
| 16:52:01 | stephenfin | Agreed | |
| 16:52:08 | stephenfin | and with that, I'm outta here | |
| 17:15:21 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236 | |
| 17:25:56 | openstackgerrit | melanie witt proposed openstack/nova master: Update process doc to be more generic about point of contact https://review.openstack.org/582312 | |
| 17:34:54 | melwitt | does anyone know if/how we can identify ARM64 platform in the libvirt driver? would it just be fields.Architecture.AARCH64? asking in the context of this bug I'm triaging https://bugs.launchpad.net/nova/+bug/1748280 | |
| 17:34:55 | openstack | Launchpad bug 1748280 in nova (Ubuntu) "'nova reboot' on arm64 is just 'nova reboot --hard' but slower" [Undecided,Confirmed] | |
| 17:39:15 | mriedem | the host capabilities should have the arch | |
| 17:40:17 | melwitt | yeah, wasn't sure if it's ARMV6 or ARMV7 or ARMV7B or AARCH64 or all of them. commenting on the bug | |
| 17:44:17 | melwitt | this is a troubling one from pike https://bugs.launchpad.net/nova/+bug/1781368 | |
| 17:44:18 | openstack | Launchpad bug 1781368 in OpenStack Compute (nova) "KeyError: 'flavor' when nova boot" [Undecided,New] - Assigned to zhaolihui (zhaolh) | |
| 17:44:43 | melwitt | actually, I can check some stable/pike gate runs to see if that's in the log | |
| 17:46:21 | dansmith | that trace shows novaclient code | |
| 17:46:27 | dansmith | not on the server side | |
| 17:46:34 | melwitt | oh, guh | |
| 17:48:16 | melwitt | before that, the server returns a 404 which seems wrong though. hm. | |
| 17:54:53 | dansmith | yeah which might be related, but also, | |
| 17:55:00 | dansmith | isn't 2.60 much later than pike? | |
| 17:55:27 | dansmith | 2.53 is pike | |
| 17:55:54 | melwitt | oh yeah, queens. 17.0.5 is queens, sorry | |
| 17:56:02 | dansmith | okay | |
| 18:03:16 | melwitt | this is a weird one that I've noticed myself when trying to trace requests during debugging, periodic tasks sharing request_id with actual requests. the weird thing is I swear this did not use to happen in the past, but AFAIK nothing has changed around the periodic task context use https://bugs.launchpad.net/nova/+bug/1773102 | |
| 18:03:17 | openstack | Launchpad bug 1773102 in OpenStack Compute (nova) "Abnormal request id in logs" [Undecided,New] | |
| 18:03:49 | melwitt | I don't know why this has started happening in the last couple of cycles (not sure when it started) | |
| 18:04:48 | melwitt | maybe cell targeting is messing with it somehow | |
| 18:05:01 | efried | melwitt: Nothing to do with the global request ID work? | |
| 18:05:30 | melwitt | efried: I don't think so. the global request ID is a separate request ID IIUC | |
| 18:19:36 | artom | *sigh* | |
| 18:19:47 | artom | How do I file a bug in Storyboard? Is it just a Story? | |
| 18:24:36 | melwitt | artom: I don't know, but what are you looking to file a bug for? I didn't think we use storyboard in nova | |
| 18:24:44 | artom | melwitt, osc | |
| 18:24:51 | melwitt | ah | |
| 18:25:14 | artom | Am I an old man now? Grumbling about change for change's sake? | |
| 18:25:43 | arvindn05 | mriedem: thanks for the review on https://review.openstack.org/#/c/581920/ - Update glance documentation for trait support. I've addressed the comments. I asked the glance core folks to take a look as well, they should put in their comments/approvals early next week | |
| 18:29:33 | mriedem | artom: yes it's a story | |
| 18:29:51 | openstackgerrit | melanie witt proposed openstack/nova master: DNM: Don't overwrite greenthread-local context during target_cell https://review.openstack.org/582332 | |
| 18:30:18 | artom | mriedem, thanks! | |
| 18:39:12 | mriedem | oopsydoodle http://logs.openstack.org/04/528704/33/check/ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3/ec5520f/logs/screen-n-cpu.txt.gz#_Jul_12_18_10_59_143344 | |
| 18:41:11 | melwitt | hm, looks like that's coming from ironicclient | |
| 18:43:23 | openstackgerrit | Merged openstack/nova master: Revert "docs: Disable smartquotes" https://review.openstack.org/578841 | |
| 18:48:01 | melwitt | from distutils.version import StrictVersion and the compare operator for StrictVersion is the thing that accesses the 'version' attribute | |
| 18:48:27 | melwitt | so something about statements like this blows up https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L161 | |
| 18:49:07 | openstackgerrit | Merged openstack/nova master: Mention osc-placement for managing traits in docs https://review.openstack.org/582173 | |
| 18:54:43 | mriedem | i can't get hits on that in logstash, not sure if the ironic job logs are indexed | |
| 18:55:30 | efried | Is the gate particularly slow at the moment? The check queue seems unusually deep and not moving very fast. | |
| 18:56:08 | mriedem | http://grafana.openstack.org/dashboard/db/zuul-status | |
| 18:56:35 | mriedem | looks like executors have dropped down | |
| 18:59:19 | efried | you must be seeing something I'm not. That dashboard looks pretty healthy to me. | |
| 19:00:00 | efried | or at least pretty stable for the past 8h. | |
| 19:00:23 | efried | 'cept for the spike in "Node Requests", whatever that is. | |
| 19:09:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid up-call in get_instance_availability_zone https://review.openstack.org/582342 | |
| 19:09:22 | mriedem | fix for the cross_az_attach=False up-call issue with cells v2 ^ | |
| 19:09:38 | mriedem | kashyap: ^ is that commit message more to your liking? | |
| 19:12:08 | melwitt | really wish we still had mirroring of release tags to github *tear* | |
| 19:14:43 | mriedem | we do, but sometimes you have to put them in yourself | |
| 19:15:11 | mriedem | https://github.com/openstack/nova/tree/17.0.5 | |
| 19:15:21 | mriedem | i don't know why they don't show up in the dropdown | |