| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-07 | |||
| 19:12:21 | artom | Cheers! | |
| 19:27:12 | mriedem | super critical bug that needs fixing asap https://bugs.launchpad.net/nova/+bug/1802167 | |
| 19:27:12 | openstack | Launchpad bug 1802167 in OpenStack Compute (nova) "nova-manage db purge -h has weird help for --before option" [Low,Triaged] | |
| 19:27:44 | artom | oh shiite im on that | |
| 19:27:59 | sean-k-mooney | artom: feels more like a stephenfin thing :) | |
| 19:29:10 | mriedem | let's not get stephen started on the cliff thing again | |
| 19:29:29 | openstackgerrit | Jack Ding proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902 | |
| 19:29:33 | mriedem | or clint or clive or whatever manly naming library it is | |
| 19:30:17 | mriedem | efried_rollin: you can drop the -2 on https://review.openstack.org/#/c/605902/ - the hpet blueprint was approved | |
| 19:30:22 | sean-k-mooney | haha didnt he end up porting the stuff he wanted form the one we are not useing to the one we are upstream in the end | |
| 19:30:48 | mriedem | jackding: make sure to throw that in the runways queue when it's ready | |
| 19:31:04 | mriedem | runways reviews are just burning down the house right now | |
| 19:37:21 | openstackgerrit | sean mooney proposed openstack/os-vif master: always create ovs port during plug https://review.openstack.org/602384 | |
| 19:37:22 | openstackgerrit | sean mooney proposed openstack/os-vif master: add isolate_vif config option https://review.openstack.org/612534 | |
| 19:42:15 | openstackgerrit | sean mooney proposed openstack/os-vif stable/rocky: always create ovs port during plug https://review.openstack.org/609850 | |
| 19:45:23 | jackding | mriedem: sure | |
| 19:49:10 | artom | mriedem, keith. I want novaclient to use python-keith | |
| 19:50:29 | mriedem | i had a shift manager at my part time high school job named keith, | |
| 19:50:39 | mriedem | oakleys, jorts, nascar shirts, tight mullet | |
| 19:50:41 | mriedem | that was a man | |
| 19:57:10 | openstackgerrit | sean mooney proposed openstack/os-vif stable/rocky: add isolate_vif config option https://review.openstack.org/616285 | |
| 20:00:40 | openstackgerrit | sean mooney proposed openstack/os-vif stable/queens: always create ovs port during plug https://review.openstack.org/609851 | |
| 20:17:45 | efried_rollin | mriedem_afk: done | |
| 20:33:13 | efried | mriedem_afk: That nova-next thing worked this time. And the job passed, but as expected, no significant difference in the size of the placement log. | |
| 22:19:09 | openstackgerrit | Merged openstack/nova stable/pike: Refactor _build_device_metadata https://review.openstack.org/587248 | |
| 22:19:15 | openstackgerrit | Merged openstack/nova stable/pike: Consider hostdev devices when building metadata https://review.openstack.org/587249 | |
| 22:31:29 | openstackgerrit | Merged openstack/nova master: Allow resource_provider_association_refresh=0 https://review.openstack.org/615606 | |
| 22:38:52 | openstackgerrit | Merged openstack/nova master: Remove LazyLoad of Scheduler Clients https://review.openstack.org/615641 | |
| 22:45:48 | efried | melwitt: That fixture merged. Are you going to propose a release or you want me to do it? | |
| 22:46:41 | efried | melwitt: Never mind, it seems to have already been done. | |
| 23:57:00 | vishwana_ | Hi, I was wondering if a single VM instance can have multiple interfaces from a single subnet? If yes, how would I enable it? Thanks in advance for your guidance | |
| #openstack-nova - 2018-11-08 | |||
| 03:05:15 | openstackgerrit | Fan Zhang proposed openstack/nova master: Retry after hitting libvirt error VIR_ERR_OPERATION_INVALID in live migration. https://review.openstack.org/612272 | |
| 04:20:05 | openstackgerrit | Merged openstack/nova master: Improve formats of the Compute API guide https://review.openstack.org/615809 | |
| 08:03:44 | openstackgerrit | liuyamin proposed openstack/nova master: Fix version details API does not return 200 OK https://review.openstack.org/616444 | |
| 08:34:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 08:40:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | |
| 08:40:41 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | |
| 08:50:40 | openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort https://review.openstack.org/616461 | |
| 09:06:23 | openstackgerrit | Jeffrey Zhang proposed openstack/nova master: Add feature to flatten the volume from glance image snapshort https://review.openstack.org/616461 | |
| 09:58:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add extra unit test coverage for @run_once https://review.openstack.org/616485 | |
| 10:01:05 | gibi | sean-k-mooney: I left some comments in https://review.openstack.org/#/c/610034 and proposed a followup ^^ | |
| 10:35:59 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send rp_uuid in the port binding https://review.openstack.org/569459 | |
| 10:36:00 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 11:56:20 | sean-k-mooney | gibi: hi | |
| 11:56:38 | gibi | sean-k-mooney: hi | |
| 11:56:58 | sean-k-mooney | i was under the impression that try finally was not legal in python but reading the reference docs on the try statement i apears to be | |
| 11:57:08 | sean-k-mooney | i can remove teh except blocks | |
| 11:57:28 | gibi | sean-k-mooney: yeah, try:finally: without except is totally valid | |
| 11:57:48 | gibi | sean-k-mooney: also if you want, you can squash my followup into your patch | |
| 11:58:29 | sean-k-mooney | i was just going to ask if i could add you as a co-author and do that to reduce the number of patches i need to backport | |
| 11:59:07 | gibi | sean-k-mooney: sure, that totally works for me | |
| 12:01:16 | sean-k-mooney | thanks ill make those change and resubmit once i finish makeing a cup of coffee. im alway happy to learn new things and til try finally is ok in python :) | |
| 12:11:39 | gibi | sean-k-mooney: :) | |
| 12:33:15 | openstackgerrit | sean mooney proposed openstack/nova master: Harden placement init under wsgi https://review.openstack.org/610034 | |
| 13:13:58 | sean-k-mooney | gibi: i have cross ported the squashed patch to placement. can you abandon https://review.openstack.org/#/c/616485 when you get a chance | |
| 13:19:43 | sean-k-mooney | bauzas: if you have time today can you rereview https://review.openstack.org/#/c/610034/ i adressed your feedback for extra comments + some other nits. i would like to try and get this merged today if possible so i can backport it before everyone goes to the summit | |
| 13:25:05 | bauzas | sean-k-mooney: I can try :) | |
| 13:25:10 | bauzas | but still on a customer issue | |
| 13:27:45 | sean-k-mooney | bauzas: for the live migration case | |
| 13:27:52 | bauzas | indeed | |
| 13:27:59 | bauzas | pbd'ing | |
| 13:29:30 | sean-k-mooney | bauzas: my understanding was we relied on the filters to check the host did not already have a member of the server group on the host in that case so the late check is not need however i belive there is a race when doing multiple live migration or booting more instnaces into the server group that was not handeled correct | |
| 13:32:13 | bauzas | sean-k-mooney: no, it's not a concurrent issue | |
| 13:32:31 | bauzas | sean-k-mooney: I guess the problem is about the persisted RequestGroup | |
| 13:32:44 | bauzas | oops s/RequestGroup/RequestSpec | |
| 13:33:01 | bauzas | AFAICS, the instancegroup.members is wrong | |
| 13:33:07 | bauzas | anyway | |
| 13:33:13 | bauzas | I'll continue to look | |
| 13:33:14 | sean-k-mooney | bauzas: do you have a functional test that can reporduce? | |
| 13:34:05 | sean-k-mooney | i have a couple of small patches i need to work on but i can try and help later once i get them written/pushed | |
| 13:34:51 | sean-k-mooney | i have other stuff i need to work on but i could also do with a break/context switch for a few hours so if i can help let me know. | |
| 13:35:27 | bauzas | I also have to go in a garage | |
| 13:35:31 | bauzas | see you in 1h | |
| 13:35:59 | sean-k-mooney | o/ | |
| 13:47:55 | jroll | kashyap: when you have a few minutes, curious your thoughts on this: https://review.openstack.org/#/c/616405/ | |
| 13:48:02 | kashyap | jroll: Hi | |
| 13:48:16 | jroll | hi :) | |
| 13:48:51 | kashyap | Let me check w/ the libvirt upstream folks | |
| 13:50:21 | kashyap | jroll: I'd expect the reporter to also note what precise failure they're seeing | |
| 13:50:24 | kashyap | Let me ask on the change | |
| 13:51:38 | jroll | kashyap: good point | |
| 13:52:11 | kashyap | Asked on the change. | |
| 14:18:01 | openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/616551 | |
| 14:19:21 | dansmith | mriedem: good for me to just double-ack this series from two stable cores? https://review.openstack.org/#/c/612496 | |
| 14:20:49 | mriedem | yeah | |
| 14:22:20 | maciejjozefczyk | mriedem: stephenfin johnthetubaguy PTAL https://review.openstack.org/#/c/591607 | |
| 14:23:51 | mriedem | s10 (vlad) should probably look at that again to make sure he's happy with it | |
| 14:26:57 | mriedem | dansmith: if you're in that stable kind of mood https://review.openstack.org/#/q/topic:bug/1797580+(status:open+OR+status:merged) | |
| 14:27:41 | mriedem | this has probably hung around too long as well https://review.openstack.org/#/q/topic:bug/1746863+status:open | |
| 14:28:11 | dansmith | no good deed, eh? | |
| 14:30:53 | maciejjozefczyk | mriedem: right, I don't know his nick or he's simply off (s10) | |
| 14:33:00 | mriedem | must be off | |
| 14:33:11 | mriedem | dansmith: you are rewarded with more reviews | |
| 14:34:00 | kashyap | jroll: BTW, I agree with you there; added a comment in the change | |
| 14:34:08 | mriedem | bauzas: you were +2 on this https://review.openstack.org/#/c/610034/ - if you still are I guess we should get that in so we can move the mirrored placement change forward | |
| 14:34:40 | kashyap | mriedem: About the 'virsh capabilities' ping you made last night, I see that's resolved as to what tssurya was looking for. | |
| 14:35:31 | tssurya | kashyap: hi, yes thanks we found out the code and as suspected it was adding the offline cpus as well | |
| 14:36:34 | kashyap | Hi; cool. (My "waking hours" are in the CET timezone) | |
| 14:48:36 | frickler | aarents: we are affected by https://bugs.launchpad.net/nova/+bug/1801702 and your patch works fine for us, too. just needed to find the correct location for that function in pike. do you want to submit it yourself to upstream? otherwise I'd push it and add you as co-author | |
| 14:48:36 | openstack | Launchpad bug 1801702 in OpenStack Compute (nova) "Spawn may fail when cache=none on block device with logical block size > 512" [Undecided,New] | |