Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-11
09:32:45 gibi Kevin_Zheng: thanks
09:32:57 gibi Kevin_Zheng: have a nice dinner!
09:33:04 Kevin_Zheng THX
09:33:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: WIP Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092
09:39:07 openstackgerrit sahid proposed openstack/nova master: libvirt: remove unused get_ovs_interfaceid() https://review.openstack.org/572699
09:43:24 jichen hi,sahid, can you take a look at https://review.openstack.org/#/c/534240/52 and its follow up patches since you -1 on previous patch? Eric mentioned he hope you to take a look before he +2, thanks
09:53:33 tetsuro gibi: hi, could you have a look at https://review.openstack.org/#/c/573113/ if you have time?
09:56:18 sahid jichen: sure
09:57:53 jichen sahid: thanks a lot
10:23:53 pvc anyone
10:23:54 pvc : rm_init_adapter failed for device bearing minor number 0
10:49:26 gibi tetsuro: I've put it in my queue
10:49:37 tetsuro thx!
10:55:49 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
10:55:51 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
10:55:52 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
10:55:53 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
10:55:54 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
11:09:27 openstackgerrit Lee Yarwood proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533
11:46:25 openstackgerrit Alexandre arents proposed openstack/nova master: Preserve images_type of instance during live migration https://review.openstack.org/570528
12:09:56 gibi tetsuro: left some question inline in https://review.openstack.org/#/c/573113
12:33:23 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240
12:35:22 gmann johnthetubaguy: any opinion on this - https://review.openstack.org/#/c/571335/
12:37:11 bauzas pvc: you had concerns last week with vGPUs ?
13:08:26 gibi Kevin_Zheng: I added some suggestion regarding sample testing in https://review.openstack.org/#/c/564092
13:08:36 gibi Kevin_Zheng: let me know if it is unclear
13:31:45 openstackgerrit ymadhavi@in.ibm.com proposed openstack/nova stable/queens: mock utils.execute() in qemu-img unit test https://review.openstack.org/574255
13:37:28 cdent johnthetubaguy: if you're around could you drop your -2 on https://review.openstack.org/#/c/270116/ , the spec is approved. Thanks.
14:05:26 openstackgerrit Dan Smith proposed openstack/nova master: Document the internal online_migrations function behaviors https://review.openstack.org/574268
14:05:48 dansmith mriedem: ^
14:07:11 mriedem ack, also, scheduler meeting talking about the migrator perf
14:07:14 mriedem #openstack-meeting-alt
14:13:58 openstackgerrit Merged openstack/nova stable/queens: Ensure resource class cache when listing usages https://review.openstack.org/573811
14:27:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018
14:28:31 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840
14:28:32 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267
14:28:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847
14:28:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268
14:28:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459
14:28:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317
14:30:12 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkInfo object https://review.openstack.org/564439
14:30:13 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440
14:30:14 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field https://review.openstack.org/564441
14:30:15 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add InstanceNUMANetworkInfo object https://review.openstack.org/564442
14:30:16 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info https://review.openstack.org/564443
14:30:17 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add 'create_resource_requests' to network API https://review.openstack.org/564444
14:30:18 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_network_info' https://review.openstack.org/564445
14:30:19 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests https://review.openstack.org/564448
14:31:54 openstackgerrit Chris Dent proposed openstack/nova stable/ocata: Ensure resource class cache when listing usages https://review.openstack.org/574279
14:44:47 gibi mriedem: Is it OK for you to skip the tomorrow's notification meeting? I don't see anything I need to talk about there
14:45:23 mriedem yup
14:45:43 gibi cool
14:53:30 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
14:53:31 openstackgerrit Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435
14:53:32 openstackgerrit Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857
14:57:12 mriedem dansmith: think there is a typo in there
14:57:26 dansmith not possible
14:59:01 openstackgerrit Dan Smith proposed openstack/nova master: Document the internal online_migrations function behaviors https://review.openstack.org/574268
15:00:20 efried mriedem, dansmith: Is `found` ever used?
15:01:28 efried mriedem, dansmith: Seems like, if we were enforcing the meaning of those return values, we could quit one iteration earlier, the first time found == done. But if there exist migrations out there that are doing it "wrong" like the consumer gen one that prompted this discussion, they would bail on the first iteration and not finish their migrations.
15:03:24 efried Okay, I just read the documentation Dan wrote - the above is totally wrong, ignore.
15:05:09 dansmith I guess that means the documentation is useful? :)
15:06:42 dansmith mriedem: melwitt: I'd also like to get this in so it has soak time: https://review.openstack.org/#/c/566696/
15:07:03 dansmith it got caught by the zuul stuff late last week, but hasn't been having trouble passing
15:07:08 dansmith so it's about to go green again
15:09:11 efried dansmith: I foudn anothre tyop.
15:09:22 efried dansmith: But also content-wise, left some questions.
15:09:59 tssurya mriedem: thanks (#CERN totally appreciates) for the initiative regarding the handling the cell down spec on the mailing list and the discussion during the nova-team meeting, I will update the spec soon (just got back from Greece this morning).
15:11:46 mriedem tssurya: yw
15:12:13 mriedem unfortunately no user/ops feedback in the ML or spec
15:12:25 openstackgerrit Dan Smith proposed openstack/nova master: Document the internal online_migrations function behaviors https://review.openstack.org/574268
15:15:14 dansmith mriedem: I think you should just go +W that patch of jay's.. my only remaining comment was about functional tests which I'm sure you're on top of
15:16:40 mriedem smcginnis: dims: should we be also bumping lower-constraints in a project when requirements.txt is updated?
15:16:44 mriedem https://review.openstack.org/#/c/486475/ for example
15:17:22 dansmith oh did I forget to do that?
15:17:26 smcginnis mriedem: Yeah, I believe that should be done in lower-constraints.
15:17:31 mriedem dansmith: you did, but that's not your patch
15:17:35 dansmith oh
15:17:40 dansmith I was going to say
15:18:05 dansmith surely the requirements job should check that?
15:18:25 mriedem there is a lower-constraints job
15:18:39 mriedem but i didn't see it run on https://review.openstack.org/#/c/566696/
15:19:05 smcginnis I'm honestly not sure what the job checks now with different versions set between requirements and lower-constraints.
15:19:35 smcginnis It may just check that requirements isn't less than lower-constraints, but seems like it should also check the other way as well.
15:19:45 smcginnis Or versions should just be removed from requirements.
15:19:46 mriedem lower-constraints just runs unit tests with the lower constraints installed, right? whereas py27 runs with upper-constraints
15:19:55 smcginnis Yeah
15:20:00 mriedem we need more requirements files
15:20:04 mriedem that's all i know
15:20:06 openstackgerrit Curt Moore proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.openstack.org/574301
15:20:12 smcginnis Yeah, half a dozen more and we should be solid.
15:22:30 mriedem dansmith: nvm you have that in htere
15:22:35 mriedem the job didn't run on your patch though for some reason
15:22:48 dansmith it trusts me?
15:22:55 mriedem no, shit failed hard
15:23:05 mriedem openstack-tox-lower-constraints finger://ze02.openstack.org/d6e860451b074239a28aae41f39381c8 : POST_FAILURE in 9m 56s
15:23:38 mriedem the xenapi patch last ran jobs in may before we had the lower-constraints job so that's probably why it wasn't caught there
15:23:46 dansmith ah and doesn't get reported up top I guess?
15:23:57 mriedem i would have expected those POST_FAILURES to get reported up top
15:24:02 mriedem they used to with lots of red

Earlier   Later