Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-19
20:52:03 mriedem nova meeting in 8 minutes
20:53:45 jroll omg I'm so excited
20:54:36 tdurakov mriedem: if you have a minute, could you please a patch above?
20:58:13 mriedem tdurakov: does that not require a change to the samples?
21:00:24 tdurakov mriedem: according to the previous run(see ps1) I'd say it doesn't(surprise!!!)
21:00:30 tdurakov at least for me
21:09:55 slaweq mriedem: hello, can You have a look on https://review.openstack.org/#/c/420176/ ?
21:09:58 slaweq thx in advance
21:23:13 tdurakov mriedem: so, for the that notifications fix... I assume there should be no notification versions bump, as object versions are remain the same, right?
21:27:08 mriedem tdurakov: well, the format on the thing getting sent has changed
21:28:37 tdurakov mriedem: what is the proper way to bump versions then?
21:31:31 mriedem tdurakov: i'm not sure in this case, and that's why i'm surprised the existing samples tests don't fail
21:31:40 mriedem btw there is a pep8 failure on that change
21:31:46 tdurakov already fixed
21:31:55 tdurakov will resubmit in a minute
21:32:02 tdurakov checking py3 too
21:33:23 tdurakov mriedem: tbh, I'm surprised that nova explicitly handle datetime formatting for notifications, I'd expect it to be managed by serializer
21:38:59 tdurakov mriedem: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L103 - is this todo valid?
21:51:18 mriedem tdurakov: yes, but will require a ton of work to get there
21:51:49 tdurakov should I rollback that to instance.get(...) ?
21:52:04 mriedem tdurakov: if that's what the getattr was for then it's fine to leave getattr
21:52:33 tdurakov mriedem: yes, that was for that todo
21:53:54 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911
21:54:11 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436
21:54:26 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738
21:54:36 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444
21:54:47 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
21:55:00 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
21:55:15 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
21:55:42 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
21:56:33 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626
21:56:48 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
21:56:59 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
21:57:11 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707
22:03:21 openstackgerrit Timofey Durakov proposed openstack/nova: Make all datastamps formats equal https://review.openstack.org/421981
22:03:55 tdurakov mriedem: fix for the pep8 is done, I wonder do we have any issues with py35?
22:09:03 mriedem tdurakov: yes, py35 job is broken until https://review.openstack.org/#/c/422709/ is merged
22:09:47 tdurakov mriedem: ok, anything else on the change?
22:11:52 mriedem tdurakov: i'd like gibi to take a look at it, i'm not ready to vote either way right now
22:11:58 mriedem kind of distracted with other things atm
22:14:50 cfriesen does anyone know of a bug open for the fact that running "nova show <instance>" as the admin user gives useless output if the instance was booted with a large user-data file?
22:16:04 cfriesen (the encoded user-data file contents are included in the output, which messes up the formatting)
22:18:52 cfriesen I guess the solution is to explicitly specify the "--wrap" option?
22:51:50 openstackgerrit Steven Webster proposed openstack/nova: Release PCI devices on drop_move_claim() https://review.openstack.org/370374
22:54:49 mriedem mox3 fix is #2 in the gate,
22:54:53 mriedem so we should be unblocked real soon now
22:58:36 mriedem and it's merged
23:10:26 mriedem jaypipes: easy peasy for you https://review.openstack.org/#/c/410737/
23:11:12 jaypipes mwhahaha: sorry, had to run out. but bottom line I think we're closer to each other's viewpoints than either of us realize.
23:11:28 mwhahaha jaypipes: it's all good
23:12:03 jaypipes mwhahaha: and I really do appreciate yours and others feedback. It is extremely valuable. We just need to do more (and earlier) of these types of conversations.
23:13:23 jaypipes mriedem: awesomesauce on the mox3 gate..
23:16:12 jaypipes mriedem: done.
23:26:36 bauzas jaypipes: back there
23:26:46 bauzas jaypipes: I just saw the meeting logs
23:27:02 bauzas jaypipes: are you planning to present something at the Summit for the placement ?
23:27:16 bauzas jaypipes: if so, I'd love to help you if you accept it
23:27:51 jaypipes bauzas: yes, I already submitted two talks.
23:28:25 bauzas jaypipes: meh, okay
23:30:07 bauzas jaypipes: in case you'd like to have a co-speaker, I can be there
23:30:28 bauzas because I was thinking of providing one for me
23:30:46 jaypipes bauzas: cool with me.
23:31:53 bauzas jaypipes: I was also thinking of providing another talk for explaining how to work with Nova btw.
#openstack-nova - 2017-01-20
00:17:09 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
01:18:15 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
01:20:34 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
01:51:38 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323
02:00:20 openstackgerrit Li Yingjun proposed openstack/nova: Code improvement for hypervisors controller https://review.openstack.org/335282
02:04:08 mriedem harlowja: looks like we have a new nova py35 gate buster with new oslo.serialization release http://logs.openstack.org/37/410737/5/check/gate-nova-python35-db/4fec66c/console.html#_2017-01-20_01_56_21_021702
02:04:48 harlowja another one
02:05:39 mriedem although isn't that was you fixed to get us to this point?
02:05:42 mriedem might just need a rebase
02:05:53 harlowja ya
02:05:59 harlowja it looks like the one i changed, ha
02:06:10 mriedem yeah just needs a rebase i think
02:06:12 mriedem i'll do that
02:06:16 harlowja k
02:08:39 openstackgerrit Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061
02:08:40 openstackgerrit Matt Riedemann proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port https://review.openstack.org/410737
02:26:48 openstackgerrit melanie witt proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379
02:27:52 mriedem can we get this in to get some more details on this failure when it happens? https://review.openstack.org/#/c/421221/
02:27:57 mriedem just saw it hit another change
02:28:30 openstackgerrit Hironori Shiina proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745
02:29:41 mriedem melwitt: just a heads up since i'm not sure i'll have time to get to it before i leave for vacation, but on https://review.openstack.org/#/c/420981/
02:30:09 mriedem my plan was to make a call from devstack-gate to run discover_hosts on the primary node (or only node in a single-node setup)
02:30:30 mriedem d-g uses ansible and we should be able to just write a task to ssh onto the host and run the discover_hosts command
02:31:07 mriedem i'll probably try to write a handoff email to dansmith and you too about the cells stuff before i leave
02:31:32 melwitt mriedem: okay, cool. thanks for the heads up
02:31:45 mriedem i'll have my laptop with me, but probably only checking email in the morning
02:33:40 melwitt k
02:45:38 melwitt mriedem: handoff email would be a nice to have, help us focus on specific things you had in mind
03:02:35 mriedem melwitt: actually hoping this might work https://review.openstack.org/423005
03:02:50 mriedem doesn't account for grenade yet
03:13:32 openstackgerrit Matt Riedemann proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
03:13:33 openstackgerrit Matt Riedemann proposed openstack/nova: Ensure we mark baremetal links as phy links https://review.openstack.org/422068
03:13:33 openstackgerrit Matt Riedemann proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756
04:03:03 openstackgerrit Matt Riedemann proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309
04:03:09 mriedem jaypipes: i went ahead and scoped the 1.1 microversion request to just the GET on the RP aggregates in ^
04:03:18 mriedem jaypipes: don't hate me
04:03:45 mriedem i might be overthinking that one but i'm a bit touchy on the rollout of this stuff now

Earlier   Later