| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-25 | |||
| 09:22:53 | kashyap | Yeah, noted. | |
| 09:22:57 | mdbooth | A code comment from the distant past | |
| 09:23:56 | kashyap | Yep | |
| 09:25:33 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842 | |
| 09:31:47 | openstackgerrit | Rikimaru Honjo proposed openstack/nova master: compute: Sweep duplicated bdm records https://review.openstack.org/290793 | |
| 09:32:31 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/482557 | |
| 09:36:02 | bauzas | gabor_antal: gibi: about https://review.openstack.org/#/c/467514/7/nova/compute/api.py@4977 honestly my only concern was that I think "import" is probably not a great RESTful action | |
| 09:37:10 | bauzas | gabor_antal: gibi: cdent reminded me previously that a PUT can create or update with a body so in theory there should only be one notification of creation | |
| 09:37:39 | bauzas | but anyway, seems I'm nitpicking | |
| 09:45:44 | bauzas | gibi: btw. any notifications patches I could help to land before FF ? | |
| 10:18:30 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386 | |
| 10:29:33 | gibi | bauzas: hi! thanks for the feedback. I don't have direct user feedback so I cannot say for sure that we need import in the notification but it feels useful for me | |
| 10:29:43 | bauzas | cools | |
| 10:31:14 | gibi | bauzas: for patches to help in, I see on patch that needs a second +2 https://review.openstack.org/#/c/480984/ | |
| 10:31:51 | gibi | bauzas: besides that we have a list of patches that looks good from subteam perspective (and has +1) here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/versioned-notification-transformation-pike+label:Code-Review%253E%253D%252B1+label:Verified%253E%253D1+AND+NOT+label:Verified%253C0+AND+NOT+label:Code-Review%253C0 | |
| 10:34:12 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622 | |
| 10:34:15 | gibi | bauzas: any help is appreciated | |
| 10:38:28 | bauzas | gibi: going out for lunch but will review those afterwards | |
| 10:40:50 | openstackgerrit | Merged openstack/nova master: Remove check_detach https://review.openstack.org/446671 | |
| 10:43:18 | gibi | bauzas: enjoy for lunch, I just brought mine in the meantime | |
| 10:49:05 | gibi | s/for/your/ | |
| 10:50:24 | openstackgerrit | Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 11:03:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 11:09:27 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 11:14:50 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 11:21:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 11:25:36 | ildikov | stephenfin: hi | |
| 11:26:57 | ildikov | stephenfin: I just wanted to ask whether you got a chance to check stvnoyes's answers here: https://review.openstack.org/#/c/463987/ to see whether they address your concerns? | |
| 11:28:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 11:38:25 | mamandle | bauzas: https://review.openstack.org/#/c/483911/ needs a second +2, can you please take a look? | |
| 11:43:15 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | |
| 11:43:17 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | |
| 12:12:51 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:14:06 | mamandle | stephenfin: can you please check https://review.openstack.org/#/c/483911/ , its waiting on another +2. | |
| 12:19:13 | openstackgerrit | Merged openstack/nova master: [placement] Flush RC_CACHE after each gabbit sequence https://review.openstack.org/486805 | |
| 12:25:51 | mriedem | ildikov: stephenfin is on vacation all week | |
| 12:30:08 | s-dean | Hi, im a at a loss trying to get this working, this is a RPC problem right ? <class 'oslo_messaging.exceptions.MessagingTimeout'> | |
| 12:30:26 | sdague | cdent: so it looks like you got to the bottom of the fixture issue? | |
| 12:30:36 | sdague | though we seem to be bouncing off a keystone issue | |
| 12:30:44 | cdent | sdague: I _think_ so | |
| 12:32:24 | openstackgerrit | Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | |
| 12:32:26 | cdent | I started working on the OSAPIFixture too, but something has introduced some painful slowness. not clear if it is just wsgi intercept itself, or some other weirdness | |
| 12:35:43 | mriedem | jaypipes: i think i'm going to mark this done for pike https://blueprints.launchpad.net/nova/+spec/placement-allocation-requests | |
| 12:39:11 | jaypipes | mriedem: ack | |
| 12:39:39 | jaypipes | mriedem: I'll update the spec | |
| 12:41:38 | mriedem | thank | |
| 12:41:39 | mriedem | s | |
| 12:51:59 | openstackgerrit | Gábor Antal proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | |
| 12:55:00 | s-dean | Hi, could someone take a look at this log this is really weird. compute01 connects to rabbit, but throws this error. https://pastebin.com/ZbpJtBq8 | |
| 12:55:19 | bauzas | jaypipes: I just thought about a possible problem with https://review.openstack.org/#/c/483566/10 | |
| 12:55:39 | bauzas | jaypipes: tl;dr how can we be sure that the first allocation request node is accepted by the filters ? | |
| 12:56:19 | openstackgerrit | Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 12:56:25 | bauzas | jaypipes: when we're getting a list of hosts after we run filters, we should possibly subset the alloc_reqs list to be only for accepted nodes, nope ? | |
| 12:56:43 | bauzas | jaypipes: unless you're changing it somewhere and I'm blind | |
| 13:02:03 | mriedem | TheJulia: looks like CI for https://review.openstack.org/#/c/215385/ has passed using https://review.openstack.org/#/c/485812/ yes? | |
| 13:02:13 | mriedem | although maybe not on the latest change? | |
| 13:03:43 | cdent | sdague: is this nutbar or crazy pants or hmmm? https://review.openstack.org/#/c/486829/ | |
| 13:03:53 | openstackgerrit | Chris Dent proposed openstack/nova master: Use wsgi-intercept in OSAPIFixture https://review.openstack.org/486825 | |
| 13:03:53 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Use wsgi_intercept in PlacementFixture https://review.openstack.org/486237 | |
| 13:04:18 | mriedem | TheJulia: or are you testing here now? https://review.openstack.org/#/c/485349/ | |
| 13:04:34 | bauzas | cdent: I'd be okay with wsgi-intercept but my concern goes with how many people are working for it | |
| 13:05:15 | cdent | bauzas: the crazy pants question was about modify MiniDNS not wsgi-intercept. | |
| 13:05:31 | cdent | on wsgi-intercept: we are already committed to using it because we are committed to using gabbi | |
| 13:05:40 | bauzas | cdent: I'm looking at its github page, and it means nova would have a dependency for a small package | |
| 13:05:55 | cdent | it’s had that dependency for over a year | |
| 13:05:57 | bauzas | cdent: sure, but gabbi looks pretty vibrant and I'm hoping it to have more contributors | |
| 13:06:13 | bauzas | cdent: do we use it elsewhere but in that fixture ? | |
| 13:06:18 | bauzas | if so, nevermind my concern | |
| 13:06:18 | cdent | in gabbi itself | |
| 13:06:30 | sdague | cdent: that's just optimizing for the opens? | |
| 13:06:31 | bauzas | a-ha, so a transitive dependency anyway | |
| 13:07:04 | cdent | sdague: that, and for avoiding on disk files when CONF.log_dir is not set. I was doing an strace and so many lines of open and closing dnstest.txt | |
| 13:07:24 | openstackgerrit | Alexandra Settle proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | |
| 13:07:24 | sdague | yeh, that seems reasonable | |
| 13:07:32 | bauzas | cdent: any plan to push wsgi-intercept to for example openstack ? | |
| 13:07:51 | sdague | cdent: I kind of wonder whether the write to disk path is needed at all | |
| 13:07:51 | TheJulia | mriedem: not ignoring you, presently debating a related grenade issue in another channel | |
| 13:08:12 | cdent | bauzas: neither wsgi-intercept nor gabbi will come to openstack because a few of the other contributors _really_ do not want that | |
| 13:08:19 | bauzas | cdent: that's understandable | |
| 13:08:39 | bauzas | well, if we already have that transitive dependency for gabbi, anyway... | |
| 13:08:51 | sdague | cdent: I remember seeing minidns spew a bunch in the past, and was always curious about getting it to stop that | |
| 13:09:00 | cdent | sdague: I tried that, and what I found was there are few different tests where there are more than one dns manager (instance/floating ip) that are sharing the same data file | |
| 13:09:03 | mriedem | TheJulia: np, take your time | |
| 13:09:29 | cdent | sdague: and I didn’t have the horsepower to go digging to see if that could be changed | |
| 13:10:38 | cdent | sdague: for most tests the in-memory thing is used | |
| 13:13:57 | mriedem | TheJulia: found what i was looking for http://logs.openstack.org/49/485349/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/376d4dd/console.html#_2017-07-25_04_37_15_094371 | |
| 13:14:26 | TheJulia | mriedem: I do love how much faster test runs :) | |
| 13:14:49 | mriedem | 4 minutes is faster? | |
| 13:15:42 | TheJulia | this can cut a reboot out of the process of handing a ready machine off, so naturally much faster :) | |
| 13:16:05 | mriedem | TheJulia: final questions when you get a moment are, i see some patches in flight on the ironic side, should we hold the nova change for those? or make the nova change depends-on them? | |
| 13:16:12 | mriedem | like https://review.openstack.org/#/c/484032/5 | |
| 13:19:50 | dtantsur | if jlvillal agrees, we can merge it now, and address the tests issues in a follow-up | |
| 13:20:03 | dtantsur | I suspect this is the only in-flight patch that matters | |
| 13:20:27 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850 | |
| 13:21:55 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from ``ip`` commands to ``pyroute2`` https://review.openstack.org/484386 | |
| 13:22:04 | TheJulia | dtantsur: I already got agreement from him to do that last night :) | |
| 13:23:30 | TheJulia | dtantsur: well, agreement on the ironic patch :) | |
| 13:23:53 | dtantsur | okie, let's wait for Sam's review, and Just Do It (tm) | |
| 13:26:00 | TheJulia | Excellent! | |
| 13:26:13 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |