| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-12 | |||
| 17:28:00 | melwitt | dansmith: what about what I mentioned earlier, do you think there's a way we could do an if/else to decide whether to call to placement for usages based on whether the service version is new enough or something? | |
| 17:28:36 | dansmith | melwitt: well, kinda have to have some gate for that anyway, even if we had the online migration, since that runs late | |
| 17:28:57 | dansmith | it's unfortunate to have counting code we only use in that window, but.. otherwise we have to wait a cycle, so probably worth it | |
| 17:30:00 | melwitt | yeah | |
| 17:32:18 | mriedem | i'm wholly dissatisfied with the new bed set that just showed up | |
| 17:32:21 | mriedem | le sigh | |
| 17:33:06 | mriedem | melwitt: yeah i meant a conditional, like we have in the filter scheduler in ocata | |
| 17:36:13 | melwitt | mriedem: k, I'll update the spec. looks like I would have benefited from an "Upgrade impact" template section but maybe I still wouldn't have though of these :P | |
| 17:36:41 | mriedem | melwitt: we could/should add one to the template | |
| 17:36:44 | mriedem | to think about stuff like this | |
| 17:37:03 | mriedem | you can totally take the lead on adding that :) | |
| 17:37:12 | melwitt | yeah. can't hurt, could possibly help | |
| 17:37:30 | melwitt | yes, I can do that :) | |
| 17:44:53 | melwitt | mriedem: do you have an opinion on whether or not to have the online migration? dansmith pointed out that new computes will heal existing allocation records. the only allocations that wouldn't get updated are ones for say, a down compute | |
| 18:03:52 | mriedem | reading back | |
| 18:04:38 | mriedem | yeah i think we could drop the online migration if it's self healing | |
| 18:04:53 | mriedem | old computes won't self heal for this, but we won't use it if we have old computes | |
| 18:04:56 | mriedem | so that problem solves itself | |
| 18:04:57 | mriedem | i think | |
| 18:05:08 | mriedem | melwitt: dansmith: ^ | |
| 18:05:32 | melwitt | thanks mriedem | |
| 18:57:41 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Libvirt: Native LUKS file and host device decryption by QEMU https://review.openstack.org/437070 | |
| 19:04:29 | mriedem | dansmith: ^ lgtm now | |
| 19:04:34 | mriedem | pretty straightforward | |
| 19:05:07 | mriedem | i'm a gonna run to home depot to get some plywood, bbiab | |
| 19:05:10 | dansmith | okay | |
| 19:27:56 | openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | |
| 19:37:10 | openstackgerrit | Mathieu Gagné proposed openstack/nova-specs master: Nova support for attached volume extend https://review.openstack.org/453272 | |
| 19:53:41 | openstack | Launchpad bug 1682222 in OpenStack Compute (nova) "Instance deployment failure due to neutron syntax error" [Undecided,New] | |
| 19:53:41 | rajinir | https://bugs.launchpad.net/nova/+bug/1682222 is causing the ironic CIs to fail | |
| 19:55:15 | openstackgerrit | Mario Villaplana proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | |
| 20:19:19 | sdague | anyone else want to +A to fix the dhcp_v6 lie? https://review.openstack.org/#/c/450297/ :) | |
| 20:19:51 | bauzas | in case people wanna bite a spec proposal :) https://review.openstack.org/#/c/446446/ | |
| 20:20:45 | bauzas | oh oops PS5 is already commented :) | |
| 20:44:06 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 20:56:40 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/357151 | |
| 20:59:42 | openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove backend dependency for key types https://review.openstack.org/456204 | |
| 21:00:16 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 21:15:00 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 21:19:02 | openstackgerrit | Steven Webster proposed openstack/nova master: Race condition between audit and migrate/resize revert https://review.openstack.org/400995 | |
| 21:23:27 | rajathagasthya | Hello! I just ran a bandit check on nova master and I see a couple of high severity jinja2 autoescape warning: http://paste.openstack.org/show/606352/ | |
| 21:23:30 | rajathagasthya | Would there be any objection to submitting a patch setting autoescapt=True? | |
| 21:23:50 | openstackgerrit | Steven Webster proposed openstack/nova master: Fix mitaka online migration for PCI devices https://review.openstack.org/456397 | |
| 21:33:07 | jangutter | Hi! I'm the person who submitted the netronome-smartnic-enablement spec. Would anyone perhaps have spare cycles to review? | |
| 21:51:39 | mriedem | where is the admin_api policy rule defined, because i don't see it in nova's policy.json and i don't see it in keystone | |
| 21:52:37 | mriedem | dansmith: ew see this? https://review.openstack.org/#/c/456397/ | |
| 21:56:15 | dansmith | mriedem: well, I guess we now know how many people are(n't) using those kinds of devices | |
| 21:56:48 | efried | jangutter Reviewed, for what it's worth :) | |
| 21:56:53 | mriedem | 1s of 2s of users | |
| 21:57:04 | jangutter | Thanks! | |
| 21:58:38 | openstackgerrit | melanie witt proposed openstack/nova-specs master: Amend spec for placement project/user association https://review.openstack.org/454352 | |
| 22:14:18 | openstackgerrit | Jan Gutter proposed openstack/nova-specs master: Spec to enable Netronome SmartNIC support https://review.openstack.org/456386 | |
| 22:14:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | |
| 22:39:04 | suresh | hi, I am not able to install nova-api and nova-placement-api together in ubuntu, due to conflict of packages........is it a bug or can someone help me with it.....please | |
| 22:40:11 | mriedem | dansmith: claims in conductor was approved http://forumtopics.openstack.org/cfp/details/63 | |
| 22:40:21 | mriedem | even though it should have probably been renamed now :) | |
| 22:40:25 | dansmith | cool | |
| 22:40:36 | dansmith | meh, we'll surprise people | |
| 22:40:43 | mriedem | suresh: check https://docs.openstack.org/ocata/install-guide-ubuntu/ | |
| 22:40:54 | mriedem | the docs team hit some package issues when writing the install guide | |
| 22:41:01 | mriedem | so those are either fixed or documented in there now | |
| 22:41:03 | mriedem | hopefully anyway | |
| 22:43:05 | suresh | thanks mriedem, but I am following the same document, still am getting this error | |
| 22:44:25 | mriedem | suresh: i'd report a bug against ubuntu then | |
| 22:44:44 | mriedem | https://launchpad.net/ubuntu/+source/nova ? | |
| 22:46:12 | suresh | nova-placement-api : Conflicts: nova-api but 2:14.0.4-0ubuntu1.2 is to be installed | |
| 22:48:17 | suresh | mriedem, is there any woraround for this | |
| 22:48:42 | suresh | mriedem, is there any work around for this | |
| 22:48:46 | openstackgerrit | Clark Boylan proposed openstack/nova master: Fix libvirt group selection in live migration test https://review.openstack.org/456408 | |
| 22:48:59 | mriedem | suresh: i haven't used the ubuntu packages so i don't know | |
| 22:49:04 | clarkb | dansmith: that nova change plus my devstack change will fix the live migration job I think | |
| 22:49:50 | mriedem | clarkb: hmm, but we don't run with ceph in the live migration job | |
| 22:50:01 | dansmith | clarkb: rock on | |
| 22:50:08 | clarkb | mriedem: thats what you think :P | |
| 22:50:17 | clarkb | mriedem: the live migration job uses the ceph scripts (I don't know why) | |
| 22:50:19 | suresh | mriedem, thanks, I will try to find any work around :) | |
| 22:50:20 | mriedem | thinking of https://review.openstack.org/#/c/389582/ | |
| 22:50:42 | mriedem | clarkb: oh weird, ok i'll have to look | |
| 22:50:47 | mriedem | tdurakov: ^ | |
| 23:36:53 | clarkb | dansmith: mriedem https://review.openstack.org/456408 just passed so getting that stack in soonish would likely be good | |
| 23:40:11 | mriedem | hmm | |
| 23:41:17 | mriedem | let me check the old failopotamus on gate-grenade-dsvm-neutron-multinode-live-migration-nv | |
| 23:41:33 | mriedem | clarkb: that doesn't show a big spike in failures | |
| 23:41:35 | openstackgerrit | Clark Boylan proposed openstack/nova master: Remove backend dependency for key types https://review.openstack.org/456204 | |
| 23:42:02 | clarkb | grenade is unaffected because they will run the old libvirt | |
| 23:42:12 | mriedem | oh crap that's grenade | |
| 23:42:12 | clarkb | since they start on old branch | |
| 23:42:34 | mriedem | oh huh i didn't think gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial was voting | |
| 23:42:52 | mriedem | oh shit there i see the spike | |
| 23:43:01 | mriedem | is this because of the UCA change? | |
| 23:43:31 | clarkb | yes, debian appears to have used libvirt instead of libvirtd, UCA pciked up that change and newer ubuntu as well | |
| 23:43:40 | clarkb | so you can't just blindly use "libvirtd" | |
| 23:43:54 | clarkb | the switch to UCA tried to address that but didn't realize noav was doing its own hard coding of the value too | |
| 23:45:02 | mriedem | ok fast approved | |
| 23:45:18 | mriedem | clarkb: thanks for noticing and actually fixing it | |
| 23:45:24 | mriedem | i didn't even know we were busted all day | |
| 23:45:27 | mriedem | that's what i call service! | |
| 23:46:03 | clarkb | mriedem: can you reassign https://bugs.launchpad.net/nova/+bug/1681946 to kfarr? I helpfully set the depends on there but then got the bug assigned to me :) | |
| 23:46:03 | openstack | Launchpad bug 1681946 in OpenStack Compute (nova) "py27 unit test failure: ImageMetaProps has a system dependent hash signature." [High,In progress] - Assigned to Clark Boylan (cboylan) | |
| 23:46:05 | mriedem | mtreinish: oomichi: can you poke https://review.openstack.org/#/c/456388/ ? | |