| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-01 | |||
| 11:57:04 | yikun | so, if anyone have time, could take a look on it. | |
| 11:58:46 | stephenfin | yikun: Sure, I'll take a look after lunch | |
| 11:59:17 | yikun | stephenfin, thanks :) | |
| 12:27:54 | ameeda | please check this out https://review.openstack.org/#/c/526900/ . please notice the last comment | |
| 12:48:40 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 13:46:05 | bhujay | hi all with suse compute node can we use rbd as backend ? although pike release doc says no , i was checking the ceph-ansible upsteram has some commits related to suse . Hence asking this | |
| 13:46:07 | hrw | you know what is worst in code reviews? | |
| 13:46:54 | hrw | all those moments when you adapt code to everyone's comments to get good acceptable version and then someone arrive and asks for another changes ;d | |
| 13:47:39 | bhujay | sorry , asked in wrong channel | |
| 13:48:01 | bauzas | hrw: is that a problem ? I don't think so | |
| 13:48:12 | bauzas | code reviews FTW | |
| 13:48:27 | hrw | bauzas: it is not. just amusing sometimes | |
| 13:49:17 | Spaz-Work | It's definately humbling.. that's the word i'd choose | |
| 13:49:43 | hrw | stephenfin: replied to your comments | |
| 13:50:09 | hrw | Spaz-Work: I would not | |
| 13:53:29 | efried | edleafe: yt? | |
| 13:59:26 | mriedem | melwitt: i guess https://review.openstack.org/#/c/539739/ fixes the storpool encrypted luks issue | |
| 13:59:39 | lyarwood | https://review.openstack.org/#/c/538103/ - any cores able to review this, logging required to move https://bugs.launchpad.net/neutron/+bug/1744361 forward | |
| 13:59:40 | openstack | Launchpad bug 1744361 in neutron "test_server_security_groups failed to reboot with "Domain not found: no domain with matching uuid" because of missing vif event from linuxbridge agent" [Undecided,Confirmed] | |
| 13:59:57 | lyarwood | mriedem: morning, yeah appears it does, thanks again :) | |
| 14:00:12 | gibi | lyarwood: looking | |
| 14:00:44 | hrw | mriedem: good morning | |
| 14:02:03 | openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482 | |
| 14:02:15 | ameeda | can you please check this out https://review.openstack.org/#/c/526900/ . please notice the last comment | |
| 14:03:10 | hrw | mriedem: 7th version of https://review.openstack.org/#/c/538003/ is closer to perfection than yesterday ones were. please take a look once you have time | |
| 14:03:33 | edleafe | efried: ?? | |
| 14:04:16 | efried | edleafe: Was wondering if you would be willing to jump on a short call this afternoon to give an overview of the alt hosts feature. | |
| 14:04:18 | Roamer` | mriedem, good morning, and thanks and sorry again for bailing out last night three minutes before you got the patch ready | |
| 14:05:03 | edleafe | efried: depends on the time. Meetings and such, y'know | |
| 14:05:12 | cdent | edleafe, efried: would something written be better (or at least a written summary of the call) so that other people can benefit? | |
| 14:05:36 | edleafe | cdent: sure. | |
| 14:05:48 | edleafe | Anything in particular to cover? | |
| 14:05:57 | edleafe | Or just a general overview | |
| 14:06:02 | ameeda | mriedem: good morning :) what do you think about this bug https://bugs.launchpad.net/nova/+bug/1489304 ? | |
| 14:06:03 | openstack | Launchpad bug 1489304 in OpenStack Compute (nova) "Lack of volume status checks when detaching volume in rebuild." [Medium,Confirmed] | |
| 14:06:49 | efried | edleafe: Product building on nova, wants to assess the impact of the feature, if/how it can use it, etc. | |
| 14:09:52 | edleafe | efried: cdent: OK, maybe a blog post on a general overview of alt hosts would be in order | |
| 14:10:16 | efried | Sounds good to me. | |
| 14:22:30 | mriedem | stephenfin: i'm not sure it's good to update release notes for ones that have already been tagged https://review.openstack.org/#/c/539490/ | |
| 14:22:59 | stephenfin | mriedem: I built it locally and it displayed as expected | |
| 14:23:06 | stephenfin | i.e. no "new" release notes | |
| 14:23:09 | mriedem | ok, those used to get duplicated | |
| 14:23:21 | stephenfin | Yeah, I think it's branches that still cause issues | |
| 14:23:27 | stephenfin | although even that may be fixed now | |
| 14:29:56 | bauzas | mriedem: AFAIR, you can do that -ie. relnotes that have been tagged for a milestone | |
| 14:30:26 | bauzas | you can even modify an existing relnote from a stable branch, which was something not working for Newton :p | |
| 14:30:40 | bauzas | (if you remember the prelude issue we had) | |
| 14:35:21 | edleafe | gibi: Responded to your comments on https://review.openstack.org/#/c/539254/ | |
| 14:37:22 | bauzas | artom: mriedem_afk: oh man, just noticed a long chat on https://review.openstack.org/#/c/539266 | |
| 14:37:38 | bauzas | I need to digest all the comments for working on a new rev | |
| 14:42:04 | gibi | edleafe: thanks. you are right, let's fix just the real warnings first. | |
| 14:42:24 | gibi | edleafe: I guess the same reasoning holds for https://review.openstack.org/#/c/531479 as well | |
| 14:42:38 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: test_compute_mgr: fix couple of unit tests https://review.openstack.org/539973 | |
| 14:42:49 | edleafe | gibi: yeah, I just saw the comments on the other one first | |
| 14:42:58 | edleafe | gibi: should I file a TODO bug? | |
| 14:43:10 | edleafe | with the caveat that it won't be done until Rocky? | |
| 14:43:20 | gibi | edleafe: yeah, why not. It can be a low hanging fruit for some newcommer | |
| 14:43:46 | edleafe | ok, will do | |
| 14:45:18 | gibi | edleafe: thanks. I +Wd both patches | |
| 14:46:46 | edleafe | gibi: cool. Writing the bug now | |
| 14:47:47 | artom | bauzas, most were just awkward English nits | |
| 14:48:04 | bauzas | 60 comments, man | |
| 14:48:28 | bauzas | my whole Gerrit web page is needing a long scroll :p | |
| 14:56:39 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have USB Host controller for USB tablet https://review.openstack.org/538003 | |
| 14:56:49 | hrw | pep8 in comments this time | |
| 14:58:26 | mriedem_afk | artom: i hope you're just being extra harsh because it's me and sylvain on https://review.openstack.org/#/c/539266 | |
| 14:58:42 | mriedem | at some point, any docs is better than perfect docs | |
| 14:59:38 | stephenfin | mriedem: I respectfully ``disagree`` :) | |
| 15:00:22 | bauzas | if I can pass a new rev, no problem | |
| 15:00:22 | mriedem | stephenfin: heh you would | |
| 15:00:28 | stephenfin | bauzas: Nice doc. Fancy writing up the real-time doc that I've been meaning to get around to for ages, while you're at it? | |
| 15:00:35 | stephenfin | ;) | |
| 15:00:36 | kashyap | mriedem: That's a good principle: "for documentation, _anything_ (that is not misinfo) is better than nothing" | |
| 15:00:42 | kashyap | But I'm with stephenfin on writing | |
| 15:01:25 | mriedem | https://docs.openstack.org/nova/latest/contributor/documentation.html#review-guidelines | |
| 15:01:42 | kashyap | mriedem: Aside: I hope you saw my comments in the scrollback on your question about: http://logs.openstack.org/66/539266/2/check/build-openstack-sphinx-docs/f7a1712/html/admin/virtual-gpu.html#how-to-know-a-gpu-type | |
| 15:01:47 | kashyap | ? | |
| 15:02:09 | bauzas | mriedem: stephenfin: artom: honestly, I'm fine with comments, given I'm not a English expert | |
| 15:02:24 | bauzas | it's just I don't want to have that change merged after Queens | |
| 15:02:55 | mriedem | kashyap: i just did now - i had just removed the part of the docs about the libvirt api to find types | |
| 15:02:58 | kashyap | bauzas: Yeah, FWIW, I also welcome all nit-picks from native speakers and take no offense. | |
| 15:03:38 | kashyap | mriedem: Cool. The official API is: virNodeDeviceGetXMLDesc() -- have to look up the Python binding name. | |
| 15:03:49 | kashyap | (To use in place of that command in the doc.) | |
| 15:04:11 | mriedem | kashyap: so i guess "Note that the current libvirt API doesn’t provide a way to access that information." was wrong | |
| 15:04:16 | kashyap | Yes | |
| 15:04:58 | edleafe | gibi: Done: https://bugs.launchpad.net/nova/+bug/1746747 | |
| 15:04:59 | openstack | Launchpad bug 1746747 in OpenStack Compute (nova) "Clean up UUID usage in tests" [Low,Confirmed] | |
| 15:05:16 | mriedem | bauzas: maybe you want to see if "virsh nodedev-dumpxml" works for you | |
| 15:05:27 | edleafe | gibi: would be interested in your opinion on a related patch: https://review.openstack.org/#/c/539323/ | |
| 15:06:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: mark the max microversions for queens https://review.openstack.org/539978 | |
| 15:07:42 | Spazmotic | Was hoping to get more code work done before Korea but looks like that's not going to happen | |
| 15:07:44 | gibi | edleafe: thanks for the bug | |
| 15:07:47 | Spazmotic | Current commits will have to sit for a couple of weeks | |
| 15:07:57 | gibi | edleafe: opened the related patch | |
| 15:12:31 | artom | bauzas, mriedem, English is easier to wrap my head around than code, so nits come faster and easier | |
| 15:12:45 | artom | For what it's worth, group projects back in uni were torture for me | |
| 15:12:48 | bauzas | artom: no worries ;) | |
| 15:12:54 | artom | I had to force myself not to rewrite others prose | |
| 15:13:04 | artom | In my defense, some were terrible writers | |
| 15:13:08 | bauzas | once French language will rule the world, I'll bite all of you, folks | |