| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-08 | |||
| 11:21:41 | mikal | So yeah, I think I just wasted your time | |
| 11:21:42 | mdbooth | cdent: Too late: https://goo.gl/photos/31jQijVTf5wsAuFi6 :) | |
| 11:21:55 | cdent | +many | |
| 11:22:30 | mdbooth | mikal: Hehe, np. | |
| 11:28:22 | openstackgerrit | Merged openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676 | |
| 11:54:08 | openstackgerrit | Merged openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | |
| 12:08:57 | openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowing console logs in libvirt https://review.openstack.org/472229 | |
| 12:08:57 | openstackgerrit | Michael Still proposed openstack/nova master: Move the last_bytes util method to the libvirt driver. https://review.openstack.org/472228 | |
| 12:13:44 | openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | |
| 12:16:30 | mriedem | it's p-2 day | |
| 12:16:44 | mriedem | any burning regressions or bugs we need to get fixed before we tag it? | |
| 12:23:31 | bauzas | mriedem: honestly, I'm still looking at all the bugs | |
| 12:23:38 | bauzas | for the moment, none | |
| 12:23:49 | bauzas | but that's a today triage day for me | |
| 12:25:33 | mriedem | lyarwood: i just noticed the git author thing again while rebasing https://review.openstack.org/#/c/436094/ | |
| 12:25:41 | mriedem | i pulled that down with git review -d | |
| 12:25:47 | mriedem | rebased it to master: git rebase -i master | |
| 12:26:12 | mriedem | resolved merge conflicts, added the resolved files and did git commit, then git rebase --continue | |
| 12:26:16 | mriedem | and then i was the author | |
| 12:27:04 | mriedem | i do'nt know what's going on there | |
| 12:29:04 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup update_instance cell mapping handling https://review.openstack.org/467380 | |
| 12:29:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove cells v2 transition code from update_instance https://review.openstack.org/467382 | |
| 12:29:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells https://review.openstack.org/467383 | |
| 12:29:12 | kashyap | cfriesen: Have you booted an UEFI image lately? | |
| 12:29:17 | kashyap | (Or if ever) | |
| 12:29:40 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | |
| 12:30:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Pull out code that builds VIF in _build_network_info_model https://review.openstack.org/466449 | |
| 12:30:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle network-changed event for a specific port https://review.openstack.org/465787 | |
| 12:36:17 | openstack | Launchpad bug 1687581 in OpenStack Compute (nova) "Newton > Ocata upgrade. secret_uuid empty" [High,New] | |
| 12:36:17 | sdague | mriedem: https://bugs.launchpad.net/nova/+bug/1687581 seems like something we should probably fix sooner rather than later | |
| 12:37:25 | sdague | people upgrading newton -> ocata ceph apparently can't start newton guests | |
| 12:37:44 | mriedem | i know what caused that | |
| 12:38:15 | mriedem | https://review.openstack.org/#/c/389399/ | |
| 12:38:37 | mriedem | i think | |
| 12:40:51 | mriedem | so it looks like in newton they had rbd auth enabled on the cinder side, but didn't have the secret uuid coming from cinder, it was coming from nova.conf | |
| 12:42:07 | sdague | right | |
| 12:42:17 | mriedem | i can work up a patch | |
| 12:42:24 | sdague | mriedem: there is a patch in the bug | |
| 12:42:30 | sdague | they just didn't gerrit it | |
| 12:43:04 | sdague | http://paste.ubuntu.com/24497744/ | |
| 12:45:10 | mriedem | i know, but that needs tests and comments, etc | |
| 12:45:15 | mriedem | i can co-author them | |
| 12:48:49 | openstack | Launchpad bug 1696001 in OpenStack Compute (nova) ocata "duplicate for #1696685 nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 12:48:49 | bauzas | claudiub: see https://bugs.launchpad.net/nova/+bug/1696685 | |
| 12:49:01 | bauzas | claudiub: you can review the stable/ocata change | |
| 12:50:02 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | |
| 12:50:14 | openstack | Launchpad bug 1696417 in OpenStack Compute (nova) "nova-manage db online_data_migrations can fail when upgrading to newton under certain conditions" [Undecided,Incomplete] | |
| 12:50:14 | bauzas | mriedem: we have a super weird issue for stable/newton with db migrations when you have different TZs https://bugs.launchpad.net/nova/+bug/1696417 | |
| 12:50:26 | bauzas | mriedem: for sure we could just say "meh" | |
| 12:50:48 | lyarwood | mriedem: huh, are you using an alias for git commit or something? | |
| 12:52:54 | sdague | mriedem: git author or git committer? | |
| 12:54:51 | lyarwood | sdague: both, https://review.openstack.org/#/c/441204/ is an example. | |
| 12:55:00 | mriedem | lyarwood: nope | |
| 12:55:03 | mriedem | just git commit | |
| 12:55:07 | mriedem | sdague: author | |
| 12:55:21 | sdague | mriedem: author shouldn't change unless you --reset-author | |
| 12:55:27 | mriedem | which i didn't do | |
| 12:55:31 | mriedem | just git review | |
| 12:55:33 | mriedem | git rebase | |
| 12:55:35 | lyarwood | yeah that's weird | |
| 12:55:35 | mriedem | git commit | |
| 12:55:45 | mriedem | sdague: somehow stole author from lyarwood on https://review.openstack.org/#/c/441204/ | |
| 12:55:54 | mriedem | and noticed the same this morning when rebasing a change of dan's in devstack | |
| 12:55:57 | mriedem | but caught it this time | |
| 12:56:03 | sdague | mriedem: and nothing in the ~/.gitconfig? | |
| 12:56:20 | mriedem | user@ubuntu:~/git/nova$ cat ~/.gitconfig | |
| 12:56:21 | mriedem | [core] | |
| 12:56:21 | mriedem | username = mriedem | |
| 12:56:21 | mriedem | [gitreview] | |
| 12:56:21 | mriedem | email = mriedem.os@gmail.com | |
| 12:56:21 | mriedem | name = Matt Riedemann | |
| 12:56:21 | mriedem | [user] | |
| 12:56:22 | mriedem | excludesfile = /home/user/.gitignore | |
| 12:56:40 | sdague | and this is all at the cli and not using some other tool? | |
| 12:56:47 | openstack | Launchpad bug 1696605 in OpenStack Compute (nova) "TypeError: unorderable types: int() < NoneType() possible during quota limit check" [High,In progress] - Assigned to melanie witt (melwitt) | |
| 12:56:47 | bauzas | mriedem: possibly merging https://bugs.launchpad.net/nova/+bug/1696605 before pike-2 too | |
| 13:03:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Log a warning if there is only one cell when listing instances https://review.openstack.org/440652 | |
| 13:07:16 | bauzas | does the VM really supports the order of NICs that you provided when booting ? | |
| 13:07:24 | bauzas | looks to me hard to respect | |
| 13:07:28 | openstack | Launchpad bug 1696664 in OpenStack Compute (nova) "Order of the interfaces is not honored when using virt and SR-IOV interfaces" [Undecided,New] | |
| 13:07:28 | bauzas | https://bugs.launchpad.net/nova/+bug/1696664 | |
| 13:07:49 | bauzas | sfinucan: any idea of the above ? ^ | |
| 13:07:59 | bauzas | I mean, I suspect that as a non-issue | |
| 13:08:15 | bauzas | but it's not really related to SR-IOV | |
| 13:08:58 | sfinucan | bauzas: I concur - we don't say anywhere that the order is important | |
| 13:09:17 | sfinucan | /respected | |
| 13:10:18 | bauzas | well, we don't say anything in https://developer.openstack.org/api-ref/compute/?expanded=create-server-detail#create-server | |
| 13:10:26 | bauzas | and when looking at code, we're just getting the dict | |
| 13:10:41 | claudiub | bauzas: ah, cool, ty. It's in merge conflict though. :) https://review.openstack.org/#/c/471790/ | |
| 13:10:52 | bauzas | sfinucan: so I'm pretty sure we're not exactly verifying the order | |
| 13:10:59 | sfinucan | bauzas: Agreed. You can close it as invalid, IMO | |
| 13:11:09 | bauzas | sfinucan: will do | |
| 13:11:21 | bauzas | if the user cares about the order, that would be a feature to me | |
| 13:11:32 | bauzas | and good luck with that. | |
| 13:11:43 | bauzas | claudiub: oh good point, will rebase it | |
| 13:15:40 | mriedem | bauzas: sfinucan: there is some network sorting in the neutronv2.api code | |
| 13:15:59 | mriedem | https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L2522 | |
| 13:16:40 | mriedem | that's for networks, but when given ports we get the networks for those ports too so maybe that's what they are thinking | |
| 13:19:32 | bauzas | humpg | |
| 13:20:30 | bauzas | mriedem: so there are two possible outcomes | |