| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-15 | |||
| 08:21:56 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Optimize time during PCI alias parsing process https://review.openstack.org/427145 | |
| 08:28:40 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Stop using mox in test_compute_cells.py https://review.openstack.org/389750 | |
| 08:28:44 | openstackgerrit | MarkMielke proposed openstack/nova master: DiskFilter should only apply when storage is requested https://review.openstack.org/430091 | |
| 08:28:46 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Stop using mox in virt/xenapi/image/test_glance.py https://review.openstack.org/327457 | |
| 08:28:50 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Stop using mox in unit/virt/disk/test_api.py https://review.openstack.org/342061 | |
| 08:28:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove mox in nova/tests/unit/compute/test_shelve.py (3) https://review.openstack.org/342648 | |
| 08:29:00 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove mox from unit/api/openstack/compute/test_aggregates.py https://review.openstack.org/321351 | |
| 08:29:07 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove mox from api/openstack/compute/test_deferred_delete.py https://review.openstack.org/320346 | |
| 08:29:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Stop using mox int unit/virt/xenapi/image/test_utils.py https://review.openstack.org/342626 | |
| 08:32:24 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove mox in unit/cells/test_cells_manager.py https://review.openstack.org/399036 | |
| 08:48:54 | openstackgerrit | MarkMielke proposed openstack/nova master: DiskFilter should only apply when storage is requested https://review.openstack.org/430091 | |
| 09:02:54 | openstackgerrit | jichenjc proposed openstack/python-novaclient master: Avoid limit=-1 description & check in server list https://review.openstack.org/434152 | |
| 09:05:20 | lyarwood | mriedem: yeah I also have a series downstream backporting that and I2a2c80a164b9f75d0e7e0503a24194bedfc0e66b , I've also included Ifa1fb061df697f03893171a8c6ba96154ec8a29d & I45ac06eae0b1949f746dae305469718649bfcf23 | |
| 09:49:52 | openstackgerrit | jichenjc proposed openstack/nova master: Allow update_cell param to be optional https://review.openstack.org/434179 | |
| 09:54:28 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: testing live migration back and forth https://review.openstack.org/434186 | |
| 10:06:51 | openstackgerrit | Bence Romsics proposed openstack/nova-specs master: Expose trunk details over metadata API https://review.openstack.org/399076 | |
| 10:23:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/432001 | |
| 10:40:12 | snikitin | mriedem: Hello! As master is opened for Pike, can we merge RPC version aliases for Ocata? https://review.openstack.org/#/c/425958 | |
| 11:22:14 | johnthetubaguy | snikitin: good question, I would talk to dansmith about that | |
| 11:22:50 | johnthetubaguy | snikitin: I think we normally backport them too, if I remember correctly | |
| 11:24:52 | snikitin | johnthetubaguy: thanks for information! | |
| 11:25:47 | johnthetubaguy | snikitin: I think this is where we record what we normally do: https://wiki.openstack.org/wiki/Nova/ReleaseChecklist | |
| 11:28:11 | snikitin | johnthetubaguy: good. But I think I confused you. I meant I need to merge this patch just in master. "RPC version aliases for Ocata" is just a title of the patch. | |
| 11:33:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 11:44:30 | johnthetubaguy | snikitin: no, no, agreed with you, I think | |
| 11:48:51 | gibi | johnthetubaguy: hi! if you have some time could you please look at the patches on https://review.openstack.org/#/q/status:open+project:openstack/nova+topic:bp/versioned-notification-transformation-pike ? | |
| 11:49:51 | johnthetubaguy | gibi: honestly, stuck in planning and spec land pre PTG, but I hope to take a look at those soon-ish | |
| 11:50:10 | gibi | johnthetubaguy: OK, thanks! | |
| 11:55:59 | openstackgerrit | John Garbutt proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | |
| 11:56:07 | openstackgerrit | John Garbutt proposed openstack/nova master: DNM: Test service token in Glance https://review.openstack.org/419666 | |
| 11:57:03 | openstackgerrit | John Garbutt proposed openstack/nova master: DNM: Test service token in Glance https://review.openstack.org/419666 | |
| 12:05:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | |
| 12:05:36 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | |
| 12:15:04 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | |
| 12:45:15 | alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | |
| 12:56:18 | abhishekk | hi all, any input why to use n-api-metadata service? | |
| 13:02:10 | alex_xu | johnthetubaguy: sdague gmann api meeting is running~ | |
| 13:19:23 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: Tolerate jsonschema==2.6.0 https://review.openstack.org/434264 | |
| 13:21:07 | mriedem | snikitin: soon. we can do the rpc aliases after we cut rc2 | |
| 13:21:10 | mriedem | which should be tomorrow | |
| 13:21:25 | snikitin | great, thanks | |
| 13:23:23 | mriedem | lyarwood: do we need this? https://review.openstack.org/#/c/416557/ | |
| 13:24:20 | mriedem | johnthetubaguy: bauzas: for rc2 https://review.openstack.org/#/c/433932/ | |
| 13:29:08 | lyarwood | mriedem: going over this again I don't think we do, sorry about that | |
| 13:30:05 | johnthetubaguy | mriedem: I did still wonder about getting this into rc2: https://review.openstack.org/#/c/430400/ | |
| 13:32:41 | bauzas | mriedem: roger, looking | |
| 13:32:45 | lyarwood | mriedem: actually, that does make https://review.openstack.org/#/c/416558/ possible so we do need it as a dependancy | |
| 13:34:23 | openstack | Launchpad bug 1663317 in OpenStack Compute (nova) "IPv6 neighbor discovery not getting through qbr bridge" [Undecided,New] | |
| 13:34:23 | bauzas | mriedem: https://bugs.launchpad.net/nova/+bug/1663317 seems to me rather a new feature for supporting IPv6 than a bug, agreed ? | |
| 13:35:07 | bauzas | mriedem: we did cut IPv6 for the bridge because of the security issue, but we don't had yet a solution for the above point | |
| 13:35:38 | bauzas | mriedem: either way, I'm about to mark it invalid unless you have other thoughts | |
| 13:39:40 | mriedem | johnthetubaguy: isn't that latent? | |
| 13:40:17 | johnthetubaguy | mriedem: yeah, thats true, I have live-migrate on the brain a bit right now | |
| 13:40:35 | mriedem | johnthetubaguy: yeah i'd rather just backport that after ocata is released | |
| 13:41:40 | mriedem | bauzas: i don't really know anything about that ipv6 bug | |
| 13:46:07 | mriedem | lyarwood: ok +2 | |
| 13:46:11 | mriedem | lyarwood: can you get this in? https://review.openstack.org/#/c/433922/ | |
| 13:48:14 | mriedem | johnthetubaguy: we've got a +2 on https://review.openstack.org/#/c/433257/ if you're happy with it, then i'm going to backport for rc2 | |
| 13:48:26 | bauzas | mriedem: okay, no worries, I have enough knowledge and understanding for at least replying to the reporter | |
| 13:48:39 | mriedem | on top of https://review.openstack.org/#/c/432954/ | |
| 13:48:42 | johnthetubaguy | mriedem: ah, looking | |
| 13:49:19 | lyarwood | mriedem: yup looking, the fix for the CI failure landed already right? | |
| 13:49:27 | bhagyashris | mriedem: Hi, Could please review the patch: https://review.openstack.org/#/c/409644/ ? if you get time | |
| 13:49:31 | mriedem | lyarwood: yeah | |
| 13:50:24 | mriedem | bhagyashris: that's an api change, so probably need alex_xu to also look at that | |
| 13:53:52 | bhagyashris | mriedem: ok. Thanks for review. | |
| 14:02:45 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: Tolerate WebOb===1.7.1 https://review.openstack.org/434285 | |
| 14:07:44 | mriedem | johnthetubaguy: the ci failure on https://review.openstack.org/#/c/433257/ is due to the cells v1 test_stamp_pattern blocking bug from yesterday | |
| 14:07:46 | mriedem | already rechecked | |
| 14:08:26 | mriedem | sorry to push but i've got meetings most of the day so won't have a ton of time for dealing with thi | |
| 14:08:27 | mriedem | *this | |
| 14:08:47 | johnthetubaguy | mriedem: yeah, was just thinking about waiting for the recheck, but happy to +W it if you are happy thats likely to work | |
| 14:09:00 | mriedem | i guarantee it :) | |
| 14:09:41 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/432001 | |
| 14:09:42 | johnthetubaguy | mriedem: sure :) | |
| 14:19:23 | oomichi | johnthetubaguy: can you take a look at https://review.openstack.org/#/c/389338/ ? I replied to your question on PS5 | |
| 14:21:44 | johnthetubaguy | oomichi: sorry, I don't see any emails from gerrit right now, I just get noise from the system, I must attempt to fix that again | |
| 14:23:12 | oomichi | johnthetubaguy: no problem. gerrit emails tend to be just noise in many cases :-) | |
| 14:32:25 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: Tolerate WebOb===1.7.1 https://review.openstack.org/434285 | |
| 14:34:59 | mordred | sdague: the add-flavor-to-server thing we were talking about a few days ago - has there been any discussion of doing the same thing with image? (I'm guessing not, since that's proxy info, but thought I'd ask) | |
| 14:35:38 | sdague | mordred: there has not been | |
| 14:35:58 | sdague | the difference with add-flavor-to-server is we actually do that internally already | |
| 14:36:06 | mordred | nod | |
| 14:36:21 | sdague | to protect you from a class of errors, all the flavor info is copied into the instance on boot | |
| 14:36:34 | sdague | and that's the data that is operated on from there on out | |
| 14:37:04 | mordred | ahhhh. so that makes adding it actually quite easy then :) | |
| 14:37:10 | sdague | right | |
| 14:37:12 | mordred | sdague: it came to mind because of the random email I read and answered on the openstack@ list ... someone was asking if there was a rest api to find the os of a server - and I responded that the best chance was looking at the image, but that it might be gone | |
| 14:37:21 | mordred | and it reminded me of the flavor thing | |
| 14:37:24 | sdague | yeh | |
| 14:38:39 | openstackgerrit | Hironori Shiina proposed openstack/nova master: Remove mox from tests/unit/conductor/tasks/test_live_migrate.py(4) https://review.openstack.org/312589 | |
| 14:40:11 | sdague | mordred: that just gets into the realm where we split up the concerns a while ago | |
| 14:40:42 | johnthetubaguy | mordred: for image stuff, this spec nicely hints on some of the problems we have right now: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/virt-image-props-boot-override.html | |
| 14:40:51 | johnthetubaguy | I wish we had been able to get that one done | |
| 14:41:10 | sdague | if we want to be able to interogate nova for these things, you start asking the question about whether glance really should remain an independent concern | |
| 14:41:14 | mordred | johnthetubaguy: zomg. can I mention how crazy I think it is that image props affect instance booting? | |
| 14:41:37 | sdague | mordred: why? | |
| 14:42:35 | mordred | sdague: because to me an image is a filesystem, not a machine. the machine is the thing I ask nova to create at boot time | |
| 14:42:46 | johnthetubaguy | there was that lib that would help us kill half of them | |
| 14:42:53 | johnthetubaguy | mordred: but that image has requirements in order to work | |