| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-12 | |||
| 17:24:28 | dansmith | right | |
| 17:24:38 | dansmith | and I would think that we could just do away with the migration entirely, | |
| 17:24:50 | dansmith | especially since that's kindof a placement-side thing | |
| 17:24:51 | dansmith | and being in nova is kinda icky from a separation point of view | |
| 17:26:06 | melwitt | yeah ... just thinking, if a compute is down, that means its allocations don't get included in the person's usage. which seems right on the surface, but if it's a temporary thing then when the compute comes back up, they could go over quota | |
| 17:26:29 | dansmith | melwitt: just like losing a cell, but it's only a window where it matters, | |
| 17:26:37 | dansmith | and if they're over quota they're over quota | |
| 17:26:50 | dansmith | would be an easy reno I think | |
| 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 | rajinir | https://bugs.launchpad.net/nova/+bug/1682222 is causing the ironic CIs to fail | |
| 19:53:41 | openstack | Launchpad bug 1682222 in OpenStack Compute (nova) "Instance deployment failure due to neutron syntax error" [Undecided,New] | |
| 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 | clarkb | since they start on old branch | |
| 23:42:12 | mriedem | oh crap that's grenade | |
| 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" | |