| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-27 | |||
| 15:00:15 | zioproto | dansmith, http://paste.openstack.org/show/600630/ | |
| 15:00:23 | zioproto | it is not many records, which one should I delete ? | |
| 15:00:32 | satyar | johnthetubaguy: i attached the screenshot how to create boot from volume from horizon | |
| 15:00:58 | zioproto | dansmith, 11 and 12 look weird, no ? | |
| 15:01:10 | dansmith | zioproto: ah, so you have a deleted conductor record too, which will do it as well.. so really I think you want "DELETE FROM services WHERE deleted != 0" | |
| 15:02:58 | zioproto | dansmith, http://paste.openstack.org/show/600632/ | |
| 15:03:16 | zioproto | but if I run nova-manage db online_data_migrations this still fails | |
| 15:03:35 | mriedem | cached? | |
| 15:03:35 | zioproto | 2017-02-27 16:02:14.167 16580 ERROR nova.db.sqlalchemy.api [req-cbc733c3-8c99-4f93-8b51-07d61d77102b - - - - -] Data migrations for PciDevice are not safe, likely because not all services that access the DB directly are updated to the latest version | |
| 15:03:43 | mriedem | the service version is cached right? | |
| 15:03:55 | zioproto | cached where ? | |
| 15:03:56 | dansmith | mriedem: not in nova-manage | |
| 15:04:24 | dansmith | zioproto: 11 and 12 do look very weird yeah | |
| 15:04:33 | zioproto | should I delete those too ? | |
| 15:04:47 | zioproto | I will | |
| 15:05:06 | dansmith | actually, the api one might be okay | |
| 15:05:12 | dansmith | it's definitely weird because it doesn't get updated periodically | |
| 15:05:21 | dansmith | neither of these really should be in effect though, which is strange | |
| 15:05:36 | zioproto | same error | |
| 15:05:47 | dansmith | ah, and the metadata one is probably weird for the same reason | |
| 15:05:57 | dansmith | zioproto: are you deploying from git or distro packages or what? | |
| 15:06:02 | zioproto | ubuntu packages | |
| 15:06:05 | zioproto | mitaka | |
| 15:06:33 | dansmith | zioproto: can you give me an exact version string so I can try to hunt down a git sha? | |
| 15:06:47 | zioproto | 2:13.1.2-0ubuntu2 | |
| 15:07:15 | zioproto | the repo is http://ch.archive.ubuntu.com/ubuntu xenial-updates/main amd64 Packages | |
| 15:09:08 | zioproto | should be the nova tag 13.1.2 | |
| 15:09:47 | dansmith | should be, but... | |
| 15:11:22 | zioproto | I am checking the ubuntu patches in the deb packages on top of 13.1.2, nothing related to this thing AFAIK | |
| 15:11:57 | zioproto | dansmith, git://git.launchpad.net/~ubuntu-server-dev/ubuntu/+source/nova and see branch stable/mitaka | |
| 15:13:51 | belmoreira | testing "db sync" in a copy of our production DBs from liberty to mitaka and getting an error in the migration 317->318: error: Index 'allocations_resource_provider_class_used_idx' already exists! | |
| 15:14:09 | belmoreira | Is this a known issue? google is not being my friend | |
| 15:14:40 | dansmith | zioproto: just to be clear, you have this applied to the code you're running nova-manage from? https://review.openstack.org/#/c/385195/1/nova/objects/pci_device.py | |
| 15:15:10 | mriedem | oh wow that's not released yet.. | |
| 15:15:52 | dansmith | mriedem: which? | |
| 15:15:55 | mriedem | ^ | |
| 15:16:00 | mriedem | that's in stable/mitaka but not released | |
| 15:16:05 | mriedem | in a 13.x point release | |
| 15:16:08 | mriedem | so might not be in distros | |
| 15:16:10 | dansmith | mriedem: the index thing or my link? | |
| 15:16:14 | zioproto | dansmith, let me check | |
| 15:16:14 | mriedem | your link | |
| 15:16:17 | dansmith | okay | |
| 15:16:42 | dansmith | zioproto: that could definitely be your problem if you don't.. now that my brain re-caffeinates, it would cause us to always get zero as the minimum version regardless of what is in the db | |
| 15:17:35 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects https://review.openstack.org/394480 | |
| 15:17:36 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added nova objects for intance diagnostics https://review.openstack.org/434745 | |
| 15:17:37 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | |
| 15:17:38 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | |
| 15:17:39 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: Add RPC version aliases for Ocata https://review.openstack.org/425958 | |
| 15:17:46 | zioproto | dansmith, yes, I applied that patch, and also regenrated the pyc file. And still the problem is there | |
| 15:18:10 | dansmith | zioproto: since this is pre-prod can you share with me a dump of the db by chance? | |
| 15:18:50 | zioproto | sure, as long as I share it just with you and not will the all world in public it is okay :)( | |
| 15:18:53 | zioproto | :) | |
| 15:18:58 | dansmith | zioproto: sure, will pm | |
| 15:20:35 | dansmith | mriedem: that definitely should get out to people, but it also shouldn't be blocking zioproto from getting to the migration he needs, so it really should only affect pci-using people | |
| 15:20:40 | dansmith | but still important of course | |
| 15:26:37 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 15:26:37 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 15:26:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 15:26:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 15:26:39 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 15:26:40 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 15:26:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 16:17:12 | openstackgerrit | Dan Smith proposed openstack/nova master: Ignore deleted services in minimum version calculation https://review.openstack.org/438578 | |
| 16:17:24 | dansmith | mriedem: I think we want that ^ back to mitaka | |
| 16:18:13 | mriedem | ok i'll hold https://review.openstack.org/#/c/438570/ up for that | |
| 16:18:56 | dansmith | cool | |
| 16:19:50 | openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | |
| 16:19:50 | akijak | hi, I've found a bug in nova concerning rebuilding and 'hw_disk_bus' image property (https://bugs.launchpad.net/nova/+bug/1667667), I'm looking for someone who could give me some directions how to fix that :) can you help? | |
| 16:21:21 | mriedem | dansmith: +2, just need another taker | |
| 16:24:26 | sdague | mriedem: where are these patches? | |
| 16:24:44 | sfinucan | macsz: Any thoughts on https://review.openstack.org/#/c/429042/? | |
| 16:28:08 | mriedem | sdague: https://review.openstack.org/438578 | |
| 16:28:16 | sdague | mriedem: yeh, I found it | |
| 16:33:10 | ildikov | johnthetubaguy: sorry, I passed out, but yes it is, the link was added to one the earlier comments of the patch | |
| 16:33:19 | ildikov | johnthetubaguy: and thanks for approving :) | |
| 16:34:38 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 16:36:11 | efried | mriedem mikal: Per request, split out a spawn/delete change set that's a mere 362LOC. Managed to make it pass jenkins *and* our CI. https://review.openstack.org/#/c/438119/ | |
| 16:39:42 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | |
| 16:41:16 | macsz | sfinucan: hmm merging them together sounds like a good idea, let me check the hyper-v code, let's see if it behaves the same as the libvirt's counterpart | |
| 16:51:54 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 16:55:14 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 16:57:30 | mriedem | sdague: you know how you renamed gate-tempest-dsvm-nnet-ubuntu-xenial ? | |
| 16:57:41 | sdague | mriedem: yep | |
| 16:57:46 | sdague | what now? | |
| 16:57:50 | mriedem | something was broken in the branch filtering | |
| 16:58:02 | mriedem | we haven't been running nova-net since ocata unless it's in the cells v1 job | |
| 16:58:09 | mriedem | so i'm guessing gate-tempest-dsvm-nnet-ubuntu-xenial is running neutron | |
| 16:58:11 | mriedem | else it would be failing | |
| 16:58:20 | mriedem | it should be running on pike or ocata though | |
| 16:58:23 | mriedem | *shouldn't | |
| 16:59:11 | sdague | mriedem: yep, seems to be running neutron | |
| 16:59:19 | sdague | where was the branch filtering for that? | |
| 16:59:43 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 16:59:51 | mriedem | i assume layout.yaml | |
| 16:59:54 | mriedem | looking | |
| 17:01:44 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 17:01:44 | mriedem | L2051 | |
| 17:02:07 | sdague | mriedem: ok, pushing a change | |
| 17:02:15 | sdague | https://review.openstack.org/438621 ? | |