| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-15 | |||
| 15:15:07 | mordred | fwiw - this conversation has helped me understand why having a property on an image that relates to hypervisor behavior is not crazy | |
| 15:15:12 | mordred | so thanks to you all | |
| 15:17:12 | openstackgerrit | Jordan Pittier proposed openstack/nova master: api/metadata/vendordata_dynamic: don't import Requests for its constants https://review.openstack.org/434339 | |
| 15:21:44 | hongbin | alex_xu: ping | |
| 15:37:10 | mriedem1 | lyarwood: the ocata change for this is merged now https://review.openstack.org/#/c/429416/ | |
| 15:40:52 | lyarwood | mriedem: ack thanks | |
| 15:44:32 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Optimize time during PCI alias parsing process https://review.openstack.org/427145 | |
| 15:44:58 | BobBall | dansmith: Any chance you can re-review https://review.openstack.org/#/c/424428/ ? I understand a backport is v.tricky so we'd dearly love to get the fix in Ocata. | |
| 15:55:25 | dansmith | BobBall: yesterday I said I had looked at the object stuff and was okay with it, but didn't have time to really do the rest... I was kinda punting to mriedem at that point.. did he say yay or nay about it? | |
| 15:56:38 | BobBall | Ah, no, mriedem hasn't looked at it. | |
| 15:57:21 | dansmith | mriedem: for your benefit: | |
| 15:57:23 | BobBall | I guess you don't expect time to look at the rest of the patch today? | |
| 15:57:26 | dansmith | 07:55:24] <dansmith>BobBall: yesterday I said I had looked at the object stuff and was okay with it, but didn't have time to really do the rest... I was kinda punting to mriedem at that point.. did he say yay or nay about it? | |
| 15:57:57 | dansmith | BobBall: I've got something going on for the next couple hours but could probably look at it after that if mriedem agrees that we should get it into rc2 | |
| 15:58:05 | mriedem | BobBall: i planned on reviewing it today | |
| 15:58:22 | mriedem | i'm in meetings for the next couple of hours | |
| 15:58:24 | BobBall | Ah - awesome. Thanks both. | |
| 15:58:38 | mriedem | it's the last thing i'm trying to get into rc2 before we cut it | |
| 15:59:19 | BobBall | Cool. I'll keep my eyes open for any changes or feedback that's needed. | |
| 16:09:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 16:12:59 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | |
| 16:13:00 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | |
| 16:13:01 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | |
| 16:13:02 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Add RPC version aliases for Ocata https://review.openstack.org/425958 | |
| 16:24:48 | openstackgerrit | Roman Podoliaka proposed openstack/nova master: Make eventlet hub use a monotonic clock https://review.openstack.org/434327 | |
| 16:42:31 | mordred | sdague, mriedem did anything change in the last couple of days related to nova usage? | |
| 16:42:47 | mordred | specifically, did total_hours go away? | |
| 16:44:00 | mriedem | simple tenant usage? | |
| 16:44:18 | sdague | still looks like it's in master to me | |
| 16:44:30 | mriedem | which microversion? | |
| 16:52:09 | mordred | latest | |
| 16:52:34 | mordred | so there was a change to remove that in latest microversion? /me goes to look at microversion docs | |
| 16:53:51 | mordred | the only change I see is the 2.40 one which adds pagination ... does the text there https://docs.openstack.org/developer/nova/api_microversion_history.html#id36 mean that the total fields are just gone now? | |
| 16:54:32 | diana_clarke | mordred: totals should still be there, but they only reflect a "page" of data and may need to be stitched back together | |
| 16:54:57 | diana_clarke | mordred: pastebin an api response, so I can see what you're seeing? | |
| 16:54:59 | mordred | diana_clarke: hrm. well - I'm getting this: http://logs.openstack.org/44/434344/1/check/gate-shade-dsvm-functional-libs-nv/495b14a/console.html#_2017-02-15_16_10_01_704583 | |
| 16:55:29 | mordred | diana_clarke: but unfortunately there is no nova api in the log there ... I'm making a patch now to try to capture more things | |
| 17:05:32 | jroll | hey nova, this refactor has been sitting for a while, if you'd like an easy-ish review :) https://review.openstack.org/#/c/399022/ | |
| 17:05:41 | jroll | +13, -285 btw :D | |
| 17:06:53 | mriedem | jroll: that reminds me, how do we ci test the hashring stuff? weren't there ideas on that with a multinode job? | |
| 17:07:26 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: remove *_topic config opts https://review.openstack.org/433924 | |
| 17:07:41 | jroll | mriedem: yeah, we've been working on a multinode job, which I believe has two computes | |
| 17:07:54 | jroll | idk if we want to do anything fancy like bring on compute host down | |
| 17:08:18 | mriedem | probably not if yo'ure running tempest on it at the same time | |
| 17:08:24 | jroll | right | |
| 17:08:28 | mriedem | jroll: the ironic networking one | |
| 17:08:32 | openstackgerrit | Sujitha proposed openstack/nova master: Adding tags field to InstancePayload https://review.openstack.org/407228 | |
| 17:08:38 | jroll | we could do something in setup, though | |
| 17:08:42 | mriedem | jroll: you could write your own integration tests for it i guess... | |
| 17:08:53 | jroll | or that yeah | |
| 17:08:57 | mriedem | ironic-ha-dsvm-integration | |
| 17:09:12 | jroll | ya | |
| 17:10:28 | mriedem | sdague: i've got the backport on top of https://review.openstack.org/#/c/432954/ for the CONF.placement.os_interface fixes so can you change that -1 to a +2? | |
| 17:10:32 | mordred | morgan, mriedem: either of you know what, if anyhting, I need to do with the novaclient Client constructor or python logging to get http debug tracing from novaclient commands? I don't seem to be getting them just from setting keystoneauth logger to debug | |
| 17:11:33 | mriedem | mordred: python logging is always greek to me, i did this in nova for enabling debug logging with glanceclient https://github.com/openstack/nova/blob/master/nova/config.py#L38 | |
| 17:11:54 | jroll | mriedem: I don't believe that ironic networking job is multinode, fwiw, unless you think this might affect it some other way | |
| 17:12:00 | mriedem | i assume there is a logging.conf or something somewhere where you should be able to just set this stuff | |
| 17:12:29 | mriedem | jroll: i'm just buying time brothaman. i'm in an auditorium at work right now hearing about the new hotness | |
| 17:12:33 | mriedem | so can't review things atm | |
| 17:12:37 | mriedem | but bs is free | |
| 17:12:38 | openstackgerrit | Eric Brown proposed openstack/nova master: Add RPC version aliases for Ocata https://review.openstack.org/425958 | |
| 17:12:49 | jroll | mriedem: heh, right on | |
| 17:15:42 | mordred | mriedem: so - looking further - I _think_ the problem is that novaclient passes its own logger to keystoneauth - so in this case my config of "hey keystoneauth, please log at debug" is not matching the logger named "novaclient" | |
| 17:15:54 | mordred | mriedem: I'm now trying also adding novaclient to the list of things I log at debug level | |
| 17:16:02 | mordred | and we'll see where that gets me | |
| 18:01:47 | openstackgerrit | Anusha Unnam proposed openstack/nova master: [WIP]Remove mox from nova/tests/unit/test_service.py https://review.openstack.org/434437 | |
| 18:07:16 | openstackgerrit | Sujitha proposed openstack/nova master: Change tags to default field in Instance object. https://review.openstack.org/415298 | |
| 18:10:20 | dims | jaypipes : oomichi : can you please take a look at this https://review.openstack.org/#/c/434264/ - will help get a bunch of upper constraints changes into requirements (https://review.openstack.org/#/c/434434/) | |
| 18:11:02 | dims | jaypipes : oomichi : basically checks if the exception message is from the old library or new library and asserts on both formats | |
| 18:11:06 | jaypipes | dims: on it. | |
| 18:11:21 | jaypipes | finally done with conf calls for the day. | |
| 18:12:50 | dims | :) | |
| 18:13:29 | jaypipes | dims: got a link to some output that shows the problem with this? | |
| 18:13:35 | jaypipes | dims: in the absense of a bug report... | |
| 18:13:55 | dims | one sec | |
| 18:16:54 | dims | jaypipes : test_validate_patternProperties_fails in http://logs.openstack.org/84/433484/2/check/gate-cross-nova-python27-ubuntu-xenial/dcdc886/testr_results.html.gz | |
| 18:18:07 | jaypipes | dims: got it. thanks! | |
| 18:21:25 | jaypipes | dims: done | |
| 18:21:32 | dims | thanks jay | |
| 18:39:25 | oomichi | dims: Thanks for doing that :-) This kind of message change is I imagined when implementing it at the first time. Maybe we will need this kind of change again and again. | |
| 18:40:05 | dims | oomichi : cross my fingers, it has held up for 2 years since your last change :) so maybe in 2 more years we revisit | |
| 18:41:15 | oomichi | dims: Haha, then I need to take care of them 25 times before the end of my life | |
| 18:41:26 | dims | LOL oomichi | |
| 18:46:39 | mordred | dims: Patch Set 1: Verified-1 Build failed (check pipeline). For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing - gate-shade-docs-ubuntu-xenial http://docs-draft.openstack.org/28/434428/1/check/gate-shade-docs-ubuntu-xenial/0d76e6f//doc/build/html/ : SUCCESS in 1m 55s - gate-shade-pep8-ubuntu-xenial | |
| 18:46:40 | mordred | http://logs.openstack.org/28/434428/1/check/gate-shade-pep8-ubuntu-xenial/56ea0fa/ : FAILURE in 1m 54s - gate-shade-python27-ubuntu-xenial http://logs.openstack.org/28/434428/1/check/gate-shade-python27-ubuntu-xenial/f4e4e9f/ : SUCCESS in 2m 55s - gate-shade-python35 http://logs.openstack.org/28/434428/1/check/gate-shade-python35/0792130/ : SUCCESS in 1m 24s - shade-coverage-ubuntu-xenial | |
| 18:46:42 | mordred | http://logs.openstack.org/28/434428/1/check/shade-coverage-ubuntu-xenial/e55e969/ : SUCCESS in 2m 21s - gate-shade-dsvm-functional-legacy http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-functional-legacy/9297a44/ : POST_FAILURE in 26m 36s - gate-shade-dsvm-functional-legacy-libs-nv http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-functional-legacy-libs-nv/aac8988/ : POST_FAILURE | |
| 18:46:44 | mordred | in 27m 46s (non-voting) - gate-shade-dsvm-functional-libs-nv http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-functional-libs-nv/9a98b97/ : FAILURE in 33m 18s (non-voting) - gate-shade-dsvm-functional-magnum-nv http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-functional-magnum-nv/1ecf057/ : POST_FAILURE in 25m 20s (non-voting) - gate-shade-dsvm-functional-neutron | |
| 18:46:46 | mordred | http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-functional-neutron/3a5b115/ : POST_FAILURE in 32m 26s - gate-shade-dsvm-ansible http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-ansible/cc31729/ : POST_FAILURE in 28m 11s - gate-shade-dsvm-ansible-dev-nv http://logs.openstack.org/28/434428/1/check/gate-shade-dsvm-ansible-dev-nv/99f5d9e/ : POST_FAILURE in 25m 11s (non-voting) - | |
| 18:46:48 | mordred | gate-dsvm-nodepool-src http://logs.openstack.org/28/434428/1/check/gate-dsvm-nodepool-src/54b5fd4/ : SUCCESS in 36m 54s12:31 | |
| 18:46:50 | mordred | gah | |
| 18:46:52 | mordred | sorry everyone | |
| 18:47:02 | dims | oops :) | |
| 18:47:10 | mordred | diana_clarke: http://logs.openstack.org/04/434404/1/check/gate-shade-dsvm-functional-neutron/2e8773b/console.html#_2017-02-15_17_39_38_245587 | |
| 18:47:30 | mordred | diana_clarke: I'm getting back an empty tenant_usage with novaclient latest microversion | |
| 18:47:45 | mordred | diana_clarke: setting microversion to 2.0 seems to at least fix that test | |
| 18:48:10 | mordred | in any case - not sure if that's useful information or if it's just a thing I should know and deal with as I add in more explicit microversion support | |
| 18:48:13 | mordred | but thought I'd report back | |
| 18:50:01 | mordred | mriedem: is there anything in the new hotness we'd be excited about? | |
| 18:50:55 | mriedem | mordred: new hotness = ocata? we = ? | |
| 18:51:45 | mordred | mriedem: oh - you're learning about ocata in an auditorium at work? I was hoping you were learning what was going to be cool after containers or something | |