| *** hoangcx has joined #openstack-nova | 00:41 | |
| *** fried_rolls1 has joined #openstack-nova | 00:48 | |
| *** fried_rolls has quit IRC | 00:50 | |
| *** fried_rolls1 is now known as fried_rolls | 00:50 | |
| *** fried_rolls is now known as efried | 01:07 | |
| *** mhen has quit IRC | 01:15 | |
| *** mhen has joined #openstack-nova | 01:16 | |
| *** imacdonn has quit IRC | 01:17 | |
| *** imacdonn has joined #openstack-nova | 01:17 | |
| *** mrsoul has joined #openstack-nova | 01:22 | |
| *** brinzhang has joined #openstack-nova | 01:34 | |
| *** rcernin has quit IRC | 01:40 | |
| *** rcernin has joined #openstack-nova | 01:40 | |
| *** hongbin has joined #openstack-nova | 01:49 | |
| *** Bhujay has joined #openstack-nova | 02:04 | |
| *** Bhujay has quit IRC | 02:12 | |
| openstackgerrit | Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844 | 02:15 |
|---|---|---|
| *** imacdonn has quit IRC | 02:35 | |
| *** psachin has joined #openstack-nova | 02:40 | |
| *** imacdonn has joined #openstack-nova | 02:50 | |
| *** sapd1 has joined #openstack-nova | 03:05 | |
| *** vivsoni has joined #openstack-nova | 03:25 | |
| *** hongbin has quit IRC | 03:39 | |
| *** psachin has quit IRC | 03:48 | |
| *** udesale has joined #openstack-nova | 03:50 | |
| *** jdillaman1 has quit IRC | 03:55 | |
| *** icey has quit IRC | 03:56 | |
| *** icey has joined #openstack-nova | 03:57 | |
| *** kevinbenton has quit IRC | 04:00 | |
| *** kevinbenton has joined #openstack-nova | 04:00 | |
| *** psachin has joined #openstack-nova | 04:00 | |
| *** vivsoni has quit IRC | 04:10 | |
| *** pooja_jadhav has joined #openstack-nova | 04:14 | |
| *** vivsoni has joined #openstack-nova | 04:30 | |
| *** vivsoni has quit IRC | 04:37 | |
| *** vivsoni has joined #openstack-nova | 04:37 | |
| *** vivsoni has quit IRC | 04:43 | |
| *** vivsoni has joined #openstack-nova | 04:46 | |
| *** whoami-rajat has joined #openstack-nova | 04:46 | |
| *** Bhujay has joined #openstack-nova | 04:58 | |
| *** spsurya has joined #openstack-nova | 05:02 | |
| *** dave-mccowan has joined #openstack-nova | 05:36 | |
| *** jiapei has joined #openstack-nova | 05:46 | |
| *** Bhujay has quit IRC | 05:53 | |
| *** Bhujay has joined #openstack-nova | 05:59 | |
| *** dpawlik has joined #openstack-nova | 06:03 | |
| *** ajo has joined #openstack-nova | 06:03 | |
| *** pcaruana has joined #openstack-nova | 06:05 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687 | 06:23 |
| *** Luzi has joined #openstack-nova | 06:28 | |
| *** belmoreira has joined #openstack-nova | 06:36 | |
| *** belmoreira has quit IRC | 06:45 | |
| *** belmoreira has joined #openstack-nova | 06:47 | |
| *** ralonsoh has joined #openstack-nova | 06:48 | |
| *** giblet is now known as gibi | 06:53 | |
| *** sahid has joined #openstack-nova | 06:55 | |
| *** sahid has quit IRC | 06:55 | |
| *** sahid has joined #openstack-nova | 06:56 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687 | 06:57 |
| gibi | mriedem: thanks for the review, I'm planning to focus on that patch series this week | 06:58 |
| gibi | cdent: bedisdes the use-nested-allocation-candidates I think bauzas working on the vgpu support that will utilize nested scheduling too | 06:59 |
| *** maciejjozefczyk has joined #openstack-nova | 07:00 | |
| gibi | s/bedisdes/besides/ :) | 07:05 |
| *** rcernin has quit IRC | 07:06 | |
| *** ircuser-1 has joined #openstack-nova | 07:20 | |
| *** holser_ has joined #openstack-nova | 07:23 | |
| *** helenafm has joined #openstack-nova | 07:26 | |
| *** dtantsur|afk is now known as dtantsur | 07:46 | |
| *** jpena|off is now known as jpena | 07:49 | |
| *** alexchadin has joined #openstack-nova | 07:51 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: placement: Always reset conf.CONF when starting the wsgi app https://review.openstack.org/604693 | 08:03 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: placement: Always reset conf.CONF when starting the wsgi app https://review.openstack.org/604694 | 08:03 |
| *** lyarwood has joined #openstack-nova | 08:10 | |
| *** alexchadin has quit IRC | 08:14 | |
| *** openstackgerrit has quit IRC | 08:22 | |
| bauzas | good morning nova | 08:31 |
| bauzas | I wave a bit late tho | 08:31 |
| *** janki has joined #openstack-nova | 08:33 | |
| *** janki has quit IRC | 08:34 | |
| *** janki has joined #openstack-nova | 08:34 | |
| *** holser_ has quit IRC | 08:35 | |
| *** holser_ has joined #openstack-nova | 08:36 | |
| *** holser_ has left #openstack-nova | 08:36 | |
| *** Bhujay has quit IRC | 08:38 | |
| *** finucannot is now known as stephenfin | 08:38 | |
| *** alexchadin has joined #openstack-nova | 08:42 | |
| *** derekh has joined #openstack-nova | 08:44 | |
| gibi | bauzas: good morning | 08:46 |
| *** openstackgerrit has joined #openstack-nova | 08:52 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 08:52 |
| *** ratailor has joined #openstack-nova | 08:52 | |
| *** openstackgerrit has quit IRC | 09:05 | |
| *** zioproto has joined #openstack-nova | 09:05 | |
| zioproto | hello | 09:05 |
| zioproto | I am upgrading my cluster from Newton to Ocata, and I see weird failures in Nova like this http://paste.openstack.org/show/730608/ | 09:06 |
| zioproto | does anyone had such problems with database foreign keys ? | 09:06 |
| zioproto | this seems to be related to heat | 09:06 |
| *** s10 has joined #openstack-nova | 09:08 | |
| *** kukacz has quit IRC | 09:10 | |
| *** jaosorior has quit IRC | 09:11 | |
| *** kukacz has joined #openstack-nova | 09:12 | |
| *** rpittau has joined #openstack-nova | 09:20 | |
| *** Bhujay has joined #openstack-nova | 09:32 | |
| *** openstackgerrit has joined #openstack-nova | 09:34 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687 | 09:34 |
| *** Bhujay has quit IRC | 09:37 | |
| *** sapd1__ has quit IRC | 09:54 | |
| *** ondrejme has joined #openstack-nova | 09:56 | |
| ondrejme | Hello, I am trying to build Rocky from source and deploy with Kolla-ansible. I have tried to create image for nova (rhel-based), but there's a version requirement for qemu - a "qemu-img-ev". I don't want to use -ev so i built it with older version called "qemu-img". What problems can I encounter? Thanks ahead. | 09:57 |
| *** sapd1_ has joined #openstack-nova | 09:59 | |
| openstackgerrit | Merged openstack/osc-placement master: import zuul job settings from project-config https://review.openstack.org/601374 | 09:59 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Specifies the storage backend to boot instance https://review.openstack.org/579360 | 10:04 |
| egonzalez | ondrejme better use #openstack-kolla channel, sort question is to remove package and append your with template overrides {% set nova_compute_packages_remove = ['qemu-img-ev'] %} {% set nova_compute_packages_append = ['qemu-img'] %} | 10:06 |
| openstackgerrit | Merged openstack/osc-placement master: switch documentation job to new PTI https://review.openstack.org/601375 | 10:06 |
| openstackgerrit | Merged openstack/osc-placement master: add python 3.6 unit test job https://review.openstack.org/601376 | 10:06 |
| *** brinzhang has quit IRC | 10:08 | |
| johnthetubaguy | gibi: I had thoughts on this one, but really close to approve, what do you think, if you have time to take a peak at that? https://review.openstack.org/#/c/476459/29 | 10:09 |
| ondrejme | egonzales: thats what I did, and I managed to deploy as well with successful instance launch, that instance is up and running just fine, I was just wondering if you know of any functions that will not work that way | 10:11 |
| *** janki is now known as janki|lunch | 10:14 | |
| *** sean-k-mooney has quit IRC | 10:14 | |
| *** sean-k-mooney has joined #openstack-nova | 10:15 | |
| *** jaosorior has joined #openstack-nova | 10:22 | |
| gibi | johnthetubaguy: looking | 10:23 |
| *** jiapei has quit IRC | 10:23 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 10:34 |
| *** Bhujay has joined #openstack-nova | 10:40 | |
| gibi | johnthetubaguy: responeded inline. I think you raised the same issue melwitt raised in PS21 then had some back and forth and ended up passing delete_type instead on NotificationAction | 10:41 |
| *** erlon has joined #openstack-nova | 10:50 | |
| *** sapd1 has quit IRC | 10:55 | |
| johnthetubaguy | gibi: cool, I was struggling to work out all the context in the previous revisions | 10:57 |
| johnthetubaguy | gibi: I am OK with add the host param, but wasn't sure what you thought, can be a follow up patch | 10:58 |
| gibi | johnthetubaguy: yeah I can do a followup right away | 10:58 |
| johnthetubaguy | gibi: sounds good, I wasn't 100% if my reading of that was correct | 10:59 |
| gibi | johnthetubaguy: I double checked and I think you are correct | 10:59 |
| johnthetubaguy | gibi: given its been cooking since jun17, lets just merge where we are now! | 10:59 |
| gibi | johnthetubaguy: I totally agree | 11:00 |
| johnthetubaguy | cool | 11:00 |
| *** Bhujay has quit IRC | 11:02 | |
| *** Bhujay has joined #openstack-nova | 11:05 | |
| *** Bhujay has quit IRC | 11:06 | |
| *** Bhujay has joined #openstack-nova | 11:07 | |
| *** Bhujay has quit IRC | 11:08 | |
| *** Bhujay has joined #openstack-nova | 11:08 | |
| *** Bhujay has quit IRC | 11:09 | |
| *** janki|lunch is now known as janki | 11:10 | |
| *** Bhujay has joined #openstack-nova | 11:10 | |
| *** jpena is now known as jpena|lunch | 11:10 | |
| *** Bhujay has quit IRC | 11:11 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Follow up for Ib6f95c22ffd3ea235b60db4da32094d49c2efa2a https://review.openstack.org/604743 | 11:11 |
| gibi | johnthetubaguy: thanks for the reviews, here is the followup ^ | 11:11 |
| *** Bhujay has joined #openstack-nova | 11:11 | |
| *** mvkr has quit IRC | 11:17 | |
| *** alexchadin has quit IRC | 11:20 | |
| *** udesale has quit IRC | 11:25 | |
| *** mvkr has joined #openstack-nova | 11:29 | |
| kashyap | gibi: Thanks for the review here: https://review.openstack.org/#/c/562313/3 | 11:36 |
| * kashyap is a bit embarrassed | 11:37 | |
| gibi | kashyap: don't worry, it is easy to mix those variables | 11:37 |
| kashyap | So two things there: (a) first bump the version for Stein, as advertized in commit: 28d337b (Pick next minimum libvirt / QEMU versions for "Stein") | 11:38 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova stable/pike: Fix resize_instance rpcapi call https://review.openstack.org/603439 | 11:38 |
| *** Roamer` has quit IRC | 11:39 | |
| kashyap | gibi: ... (b) Need to pick version for the 'T' release. | 11:39 |
| kashyap | gibi: Can't bump without first picking the future MIN versions for the 'T' release. | 11:39 |
| kashyap | johnthetubaguy: ^ You have a comment here? | 11:39 |
| gibi | kashyap: so you will propose a patch that set MIN based on NEXT_MIN but for that we have to figure out what will be the new value of NEXT_MIN | 11:43 |
| kashyap | gibi: Yes. First we need to pick NEXT_MIN_* versions | 11:44 |
| kashyap | gibi: Like I sent out an email to 'operators' list in the past: http://lists.openstack.org/pipermail/openstack-operators/2018-April/015089.html | 11:44 |
| gibi | kashyap: OK this make sense | 11:44 |
| kashyap | (Operators + Dev list) | 11:44 |
| kashyap | Do we have a name for the 'T' release yet? | 11:44 |
| gibi | kashyap: I think we are in the name suggestion phase | 11:45 |
| kashyap | gibi: I just need to spend some tedious time to go through all distros that matter / upstream cares about, and arrive at the new enough version that is supported by all distros.. | 11:45 |
| gibi | kashyap: sounds like fun | 11:46 |
| kashyap | gibi: It is mind-warpingly tedious, but necessary :D | 11:46 |
| gibi | kashyap: OK, that is a better description than "fun" :D | 11:46 |
| kashyap | Heh | 11:47 |
| * kashyap in the middle of different context (to deprecate a command in QEMU), will get to this, once he 'flushes' that cache to 'disk' | 11:48 | |
| *** zul has joined #openstack-nova | 11:53 | |
| kashyap | gibi: 'T' will be at the end of 2019, isn't it? | 11:56 |
| *** alexchadin has joined #openstack-nova | 11:59 | |
| gibi | kashyap: autumn 2019 | 12:00 |
| kashyap | Yeah. So I'm writing an email to the list, to see if we even want to pick MIN_ versions so far out (one year ahead) | 12:00 |
| kashyap | There will be 13+ libvirt releases (including maintenance streams), IIRC. And some 4 QEMU releases | 12:01 |
| kashyap | I'll just write the email to the list; MattR will definitely have an opinion. As he also did this work in the past. | 12:02 |
| *** alexchadin has quit IRC | 12:05 | |
| gibi | kashyap: I agree that it is a good way forward | 12:06 |
| kashyap | gibi: What do you mean? I think you mean "let's write to the list and talk there"? | 12:06 |
| *** alexchadin has joined #openstack-nova | 12:06 | |
| gibi | kashyap: yeah, lets describe the issue as you descibed above (e.g. we need a new NEXT_MIN but there will be still a lot of libvirt release until 2019 autumn) on the ML | 12:07 |
| kashyap | Yeah, almost half-way through the e-mail | 12:08 |
| gibi | cool | 12:08 |
| *** psachin has quit IRC | 12:13 | |
| *** jpena|lunch is now known as jpena | 12:14 | |
| *** zul has quit IRC | 12:15 | |
| *** Bhujay has quit IRC | 12:15 | |
| *** Bhujay has joined #openstack-nova | 12:16 | |
| *** ratailor has quit IRC | 12:16 | |
| *** leakypipes is now known as jaypipes | 12:24 | |
| johnthetubaguy | kashyap: from the wiki page, looks like the question is basically when do we drop Ubuntu 16.04 support? | 12:27 |
| kashyap | johnthetubaguy: Is it just that? | 12:28 |
| kashyap | johnthetubaguy: Or we can just stick to the next LTS release, that is 18.04, Bionic? | 12:28 |
| kashyap | johnthetubaguy: But I like your phrasing, which gives a much simplified view of the problem :-) | 12:30 |
| johnthetubaguy | kashyap: yeah, I just reading through the wiki page | 12:30 |
| kashyap | johnthetubaguy: Based on that, I think we can simply just go with 'Bionic'. | 12:30 |
| kashyap | I'll double-check all the other distros, too | 12:31 |
| *** mdbooth has joined #openstack-nova | 12:32 | |
| johnthetubaguy | so, I think we said 3.0.0 and 2.8.0 which is Debian Stretch being the lower bound next, so its jump from Ubuntu Xenial to Debian Stretch as the lowest (including dropping SUSE leap 42.2) | 12:33 |
| johnthetubaguy | https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | 12:33 |
| mdbooth | Just debugging a functional test where I'm trying to force an instance to a particular host. I'm getting NoValidHost. It seems we always ask placement for candidates even if we're forcing a specific host, is that correct? | 12:33 |
| johnthetubaguy | kashyap: I just updated that for Rocky | 12:33 |
| *** zul has joined #openstack-nova | 12:34 | |
| kashyap | johnthetubaguy: Ah, thanks! | 12:34 |
| johnthetubaguy | kashyap: hmm, KVM for IBM Z... that seems to be a problem | 12:34 |
| kashyap | johnthetubaguy: Hmm, so we can't settle with Bionic | 12:35 |
| johnthetubaguy | well, unless they updated or we don't care about them | 12:35 |
| *** sambetts_ is now known as sambetts | 12:35 | |
| kashyap | johnthetubaguy: Yeah. I'll ask on the list about what their plan is | 12:36 |
| *** Bhujay has quit IRC | 12:39 | |
| *** Bhujay has joined #openstack-nova | 12:40 | |
| *** Bhujay has quit IRC | 12:41 | |
| *** Bhujay has joined #openstack-nova | 12:41 | |
| kashyap | johnthetubaguy: For Stein we are yet to bump to the advertized versions, as you may have noticed. | 12:42 |
| *** jaosorior has quit IRC | 12:42 | |
| *** alexchadin has quit IRC | 12:43 | |
| *** jaosorior has joined #openstack-nova | 12:44 | |
| johnthetubaguy | kashyap: so we often do that every other cycle, historically | 12:45 |
| kashyap | johnthetubaguy: Yeah. I was just trying to stick to what we said in this commit: | 12:45 |
| kashyap | http://git.openstack.org/cgit/openstack/nova/commit/?h=master&id=28d337b ("Pick next minimum libvirt / QEMU versions for "Stein"") | 12:45 |
| *** udesale has joined #openstack-nova | 12:46 | |
| *** jiapei has joined #openstack-nova | 12:47 | |
| *** tbachman has joined #openstack-nova | 12:50 | |
| *** jdillaman has joined #openstack-nova | 12:53 | |
| *** alexchadin has joined #openstack-nova | 12:53 | |
| johnthetubaguy | kashyap: I found this: https://www-01.ibm.com/common/ssi/cgi-bin/ssialias?infotype=OC&subtype=NA&htmlfid=897/ENUS5648-KVM&appname=totalstorage#lifecycl | 12:57 |
| *** icey3 has joined #openstack-nova | 12:58 | |
| * kashyap clicks | 12:58 | |
| *** afazekas has quit IRC | 12:58 | |
| kashyap | johnthetubaguy: It doesn't tell us version info of QEMU and libvirt, does it? | 12:59 |
| johnthetubaguy | kashyap: seems to suggest it is discontinued, and no longer supported though | 12:59 |
| kashyap | johnthetubaguy: Oh, yeah. | 13:01 |
| *** icey3 has quit IRC | 13:01 | |
| kashyap | For completeness' sake I'll ask the open question for KVM for IBM folks. | 13:01 |
| *** zul has quit IRC | 13:01 | |
| kashyap | If there's no response, we can pick with a version that aligns with the rest of all the distros. | 13:01 |
| *** awaugama has joined #openstack-nova | 13:01 | |
| johnthetubaguy | kashyap: http://kvmonz.blogspot.com/2017/03/kvm-for-ibm-z-withdrawal.html | 13:02 |
| * kashyap clicks | 13:02 | |
| kashyap | Ah, it's the blurb | 13:02 |
| johnthetubaguy | kashyap: I think we can ignore it now, basically its all upstream, use a regular distro, appears to be the message | 13:02 |
| kashyap | Okay, then. We're good. | 13:03 |
| kashyap | Thanks for digging! | 13:03 |
| johnthetubaguy | I was curious what it was | 13:03 |
| johnthetubaguy | kashyap: next question is specify the next next, if you get me, seems like Oracle and SLES need a dig | 13:06 |
| kashyap | johnthetubaguy: Right, I'll ask the SLES and Oracle folks to comment | 13:07 |
| kashyap | johnthetubaguy: I'm pretty damn sure they'll also align, if I see their historical releases | 13:07 |
| kashyap | The community edition, openSUSE, already ships the desired "Bionic" versions. | 13:07 |
| * johnthetubaguy nods | 13:08 | |
| *** mriedem has joined #openstack-nova | 13:09 | |
| kashyap | Thanks for helping me dig! | 13:11 |
| *** lbragstad has joined #openstack-nova | 13:18 | |
| bauzas | mriedem: quick question, when calling update_provider_tree_for_vgpus() we pass a mutable dict of allocations, and then we modify the allocations directly | 13:22 |
| bauzas | mriedem: that means that we will call the placement API with the allocations dict once we call it ? | 13:22 |
| bauzas | I mean, it means that the method will have to modify directly the allocations dict | 13:23 |
| bauzas | ? | 13:23 |
| kashyap | johnthetubaguy: gibi: As promised: http://lists.openstack.org/pipermail/openstack-operators/2018-September/015929.html | 13:23 |
| bauzas | mriedem: so we expect to have the allocations dict to be modified ? | 13:23 |
| mriedem | bauzas: yes https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L943 | 13:24 |
| bauzas | mriedem: cool | 13:24 |
| openstackgerrit | Merged openstack/nova master: nova-status - don't count deleted compute_nodes https://review.openstack.org/604495 | 13:29 |
| openstackgerrit | Merged openstack/nova master: Imported Translations from Zanata https://review.openstack.org/604577 | 13:29 |
| *** belmorei_ has joined #openstack-nova | 13:30 | |
| mriedem | imacdonn: you want to backport https://review.openstack.org/#/c/604495/ as well? | 13:30 |
| *** zul has joined #openstack-nova | 13:30 | |
| *** cfriesen has joined #openstack-nova | 13:31 | |
| *** belmoreira has quit IRC | 13:32 | |
| *** icey has quit IRC | 13:34 | |
| *** SteelyDan is now known as dansmith | 13:34 | |
| *** munimeha1 has joined #openstack-nova | 13:42 | |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/rocky: nova-status - don't count deleted compute_nodes https://review.openstack.org/604785 | 13:48 |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: nova-status - don't count deleted compute_nodes https://review.openstack.org/604786 | 13:48 |
| *** alex_xu has joined #openstack-nova | 13:48 | |
| *** edmondsw has joined #openstack-nova | 13:48 | |
| mriedem | s10: thanks | 13:50 |
| dansmith | mriedem: no test on this? https://review.openstack.org/#/c/554380/ | 13:53 |
| openstackgerrit | Vlad Gusev proposed openstack/nova stable/pike: nova-status - don't count deleted compute_nodes https://review.openstack.org/604788 | 13:54 |
| bauzas | mriedem: just another point, you reshape first and then you only update the child inventories after that | 13:55 |
| mriedem | i guess not? i wrote it in march. | 13:55 |
| dansmith | ...and march is the month of no tests? :D | 13:55 |
| *** jaosorior has quit IRC | 13:55 | |
| bauzas | mriedem: is this because you don't want to have a child inventory in case the reshape provides an exception ? | 13:55 |
| mriedem | that's what we agreed in dublin | 13:55 |
| dansmith | I didn't realize it was a backport so was about to -1 it out of existence | 13:55 |
| efried | n-sch/placement meeting in 5 minutes in #openstack-meeting-alt | 13:55 |
| openstackgerrit | Merged openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 13:56 |
| *** beekneemech is now known as bnemec | 13:57 | |
| mriedem | dansmith: before you are gone for the rest of the week, we should probably figure out what we expect users to pass for the bdm volume_type value in the compute API, | 13:59 |
| mriedem | because cinder's volume create API allows passing the volume type name or ID | 13:59 |
| mriedem | i was thinking the compute API would just take volume type name | 14:00 |
| dansmith | it would suck to not take either | 14:00 |
| dansmith | just name? | 14:00 |
| mriedem | i didn't realize the volume create API took either | 14:00 |
| mriedem | until 30 seconds ago | 14:00 |
| dansmith | why is it hard for us to take either? | 14:00 |
| mriedem | it's not | 14:00 |
| mriedem | it came up while reviewing the db model changes for nova b/c he had originally restricted the volume_type column to 36 characters for volume type id | 14:01 |
| mriedem | i got him to change it to 255 to allow name | 14:01 |
| dansmith | ah | 14:01 |
| mriedem | anyway, i asked the question on https://review.openstack.org/#/c/604687/ | 14:01 |
| dansmith | and that's their name restriction? | 14:01 |
| mriedem | yes | 14:01 |
| dansmith | that _is_ the downside of proxy apis I suppose, but .. :) | 14:01 |
| dansmith | okay | 14:01 |
| dansmith | surely we're not merging that until the rest of the patches are stacked on top right? | 14:02 |
| mriedem | yes he split this out b/c i asked him to | 14:02 |
| mriedem | and i asked him to rebase the rest back on top | 14:02 |
| dansmith | okay cool | 14:02 |
| *** Luzi has quit IRC | 14:03 | |
| mriedem | the other thing is if nova-api is going to validate that the requested volume type exists, we'll need to know if it's an id or a name, which kind of sucks | 14:07 |
| mriedem | we can do is_uuid_like for that, but ... | 14:07 |
| s10 | mriedem: can I ask to add volume_name in https://review.openstack.org/#/c/604687/ or it would be too much? | 14:09 |
| *** icey has joined #openstack-nova | 14:09 | |
| mriedem | s10: and eventually description and az and hints and metadata... | 14:10 |
| *** ondrejme has quit IRC | 14:14 | |
| mriedem | s10: if people are going to want to also pass volume name for the next several years, i'd rather us just add that now in the same microversion, | 14:15 |
| mriedem | dansmith: ^ what do you think? | 14:15 |
| mriedem | this is the definition of the slippery slope with these proxy apis | 14:15 |
| dansmith | I think I said name+volume already right? | 14:16 |
| dansmith | did I miss other discussion? | 14:16 |
| dansmith | oh | 14:16 |
| dansmith | volume name? | 14:16 |
| mriedem | name + volume_type? | 14:16 |
| mriedem | yes, he's asking that we also proxy a volume name, | 14:16 |
| dansmith | ffs | 14:17 |
| mriedem | today nova-compute doesn't give name/description to any volumes it creates | 14:17 |
| mriedem | ftersin had a patch to at least name the volumes that nova created | 14:17 |
| s10 | mriedem: that's what we are doing :( we have our patch for volume name since 2014 and volume type since 2015. I will be happy to drop it and make our OpenStack more close to the upstream... | 14:17 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: docs: Add redirects https://review.openstack.org/604796 | 14:18 |
| dansmith | and we have to handle the multi-create case where they can't be the same name yeah? | 14:18 |
| mriedem | depends on what cinder allows, checking the cinder db model | 14:18 |
| dansmith | presumably that means we have to have all the handling for races even against volumes we didn't reate | 14:18 |
| dansmith | I guess duplicate names may be allowed as long as we refer to them by uuid | 14:19 |
| *** belmorei_ has quit IRC | 14:19 | |
| dansmith | but still... | 14:19 |
| mriedem | yeah i don't see any unique constraint on volume names in the cinder db | 14:19 |
| dansmith | we're five minutes into this and already inches are being given | 14:19 |
| mriedem | nova has that weird config to restrict server names by project or global | 14:19 |
| mriedem | i suppose that's more for fqdns | 14:19 |
| *** mrjk_ has joined #openstack-nova | 14:20 | |
| *** mrjk has quit IRC | 14:20 | |
| *** ratailor has joined #openstack-nova | 14:22 | |
| dansmith | well, I dunno | 14:23 |
| dansmith | tbh, taking name or setting it is not something I've heard asked before | 14:23 |
| dansmith | I would tend to think that if we have no real restrictions we could name it after the instance without another param | 14:24 |
| dansmith | but | 14:24 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | 14:25 |
| mriedem | https://review.openstack.org/#/c/213433/ | 14:26 |
| mriedem | ^ ftersin's old patch to name the volumes that nova creates | 14:26 |
| dansmith | yeah, you said that, but... what about the volume/ | 14:26 |
| dansmith | meaning, lots of people ask for volume_type, but was that the only one ask for name | 14:26 |
| dansmith | ? | 14:26 |
| mriedem | idk | 14:27 |
| mriedem | i wouldn't be surprised if others would come out of the woodwork asking for proxying the name later | 14:27 |
| dansmith | where does it end? | 14:28 |
| mriedem | that's what i said above | 14:29 |
| dansmith | I know | 14:29 |
| *** belmoreira has joined #openstack-nova | 14:30 | |
| *** mlavalle has joined #openstack-nova | 14:34 | |
| *** dpawlik has quit IRC | 14:37 | |
| *** dulek has quit IRC | 14:38 | |
| *** dulek has joined #openstack-nova | 14:39 | |
| *** dpawlik has joined #openstack-nova | 14:40 | |
| *** mchlumsky has joined #openstack-nova | 14:40 | |
| *** dpawlik has quit IRC | 14:40 | |
| *** mchlumsky has quit IRC | 14:45 | |
| *** mchlumsky has joined #openstack-nova | 14:47 | |
| *** dave-mccowan has quit IRC | 14:50 | |
| efried | jaypipes: "efried: I specifically left out the "identification of the provider before you need it" because the clients of such a descriptor file would undoubtedly have different ideas of how to map local identifiers to RP identifiers." | 15:02 |
| efried | jaypipes: If we're talking about RPs representing devices, yeah, which is what those last two specs are trying to define. | 15:02 |
| efried | jaypipes: And those specs are attempting to account for the differences in hypervisors etc. | 15:03 |
| efried | jaypipes: But what about e.g. NUMA node RPs? | 15:03 |
| jaypipes | efried: yes. that is why I didn't put multiple providers in a single provider descriptor file and left it up to the caller to determine whether they use something like a directory with descriptor files named for the provider UUID or the provider's "local name" (NUMA0, compute_node, some PCI address, whatever...) | 15:03 |
| jaypipes | efried: each hypervisor (or thing like Cyborg) is going to have its own way of identifying local devices. | 15:04 |
| efried | I guess the same thing applies: as long as we've specified/documented how those RPs are going to be named, presumably the consumer can figure out what the names are going to be beforehand. | 15:04 |
| jaypipes | efried: and therefore each hypervisor needs to "own" the mapping of its local device name to the resource provider UUID | 15:04 |
| jaypipes | efried: yes, exactly my point. | 15:05 |
| efried | well, yeah, but not everything is a device. | 15:05 |
| bauzas | mmmm | 15:05 |
| jaypipes | efried: if libvirt wants to call its root compute node provider "compute_node_{hostname}" cool. but Xen might call it, e.g. "dom0_{hostname}" | 15:05 |
| jaypipes | efried: my point being we don't want to hard-code the names of things. | 15:06 |
| bauzas | so the problem is to know which is which, right? | 15:06 |
| efried | cdent's concern was that we don't want to require the consumer to go get a report from placement in order to figure out what's named what and then populate the file. His point was that that would be a PITA for deployment tools like ansible. | 15:06 |
| efried | jaypipes: Oh, certainly don't want to hardcode the name of anything - couldn't if we wanted to. | 15:06 |
| *** _hemna has quit IRC | 15:07 | |
| efried | but I guess we *do* need to make sure that the names are generated in a deterministic fashion that an operator can reproduce. | 15:07 |
| jaypipes | efried: well, that's essentially what cdent's gripe involves: hard-coding the name of the compute node resource provider so tools like ansible can have a stable way of calling things like `openstack provider-inventory $HOSTNAME` | 15:07 |
| *** Bhujay has quit IRC | 15:07 | |
| efried | We've got to have a starting point. | 15:08 |
| efried | and we've already established that part, I thought. | 15:08 |
| jaypipes | efried: what part? | 15:08 |
| efried | The root compute RP's UUID (the host UUID) and name (the node UUID) - right? | 15:08 |
| efried | in non-ironic those are the same | 15:09 |
| jaypipes | efried: yes, that ship is sailed. | 15:09 |
| efried | okay, and I don't think that's a bad thing. | 15:09 |
| jaypipes | efried: even for ironic. | 15:09 |
| jaypipes | it's just the compute_nodes.uuid value that is used. | 15:09 |
| efried | um, I thought for ironic the node UUID was different because multiple nodes managed by one host. | 15:09 |
| jaypipes | efried: hold up, that's not true. | 15:09 |
| efried | yeah | 15:10 |
| efried | anyway, beside the point | 15:10 |
| jaypipes | efried: we use compute_nodes.hypervisor_hostname | 15:10 |
| efried | point is that we've already established a rule | 15:10 |
| jaypipes | as the RP name. | 15:10 |
| jaypipes | https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L1417 | 15:10 |
| jaypipes | efried: ^ | 15:11 |
| *** ratailor has quit IRC | 15:12 | |
| efried | yeah, but didn't we decide hypervisor_hostname was actually the node UUID in the ironic case? | 15:12 |
| jaypipes | efried: so we already use hypervisor_hostname as RP name. | 15:12 |
| mriedem | efried: yes it is | 15:12 |
| jaypipes | efried: oh, I see what you mean. | 15:12 |
| jaypipes | efried: yes, that is true. | 15:12 |
| *** dave-mccowan has joined #openstack-nova | 15:12 | |
| jaypipes | it's the Ironic node UUID. | 15:12 |
| efried | right, okay, so I was wrong before that the RP UUID and name were the same for non-ironic; the name is actually the hypervisor hostname which is like text. | 15:13 |
| efried | anyway, point is, we've got an established rule for both the name and the UUID of the root RP. | 15:13 |
| efried | And in Denver we talked about using the parent RP's UUID as part of the child RP's name, to ensure name uniqueness | 15:13 |
| efried | So that's one part of the naming rules | 15:14 |
| efried | For each type of child RP as long as we make the remainder of the naming convention deterministic *and document it*, that should allow consumers to figure out how to identify them for purposes of their inventory file. | 15:14 |
| efried | It may be kind of brittle in practice, but I can't think of a better solution. | 15:15 |
| jaypipes | efried: I guess I'd prefer to use the root provider's *name*, not UUID, in the child provider's name. | 15:16 |
| jaypipes | efried: and yes, I realize for Ironic, any child providers would have a UUID-based name since the root provider name is a UUID. | 15:16 |
| efried | if we want the convention to be extensible, we should use something about the parent provider, not the root provider | 15:17 |
| *** dave-mccowan has quit IRC | 15:17 | |
| efried | which only gets different when we have >2 levels of tree | 15:17 |
| jaypipes | efried: yes, the parent. | 15:17 |
| efried | but if we use the name, then we'll have N-1 UUIDs in the name for a provider at the Nth level down, which is :( | 15:17 |
| jaypipes | efried: so for a SRIOV NIC under the first NUMA node on a compute node might be cn1_numa0_sriovpf0 or something like that? | 15:18 |
| jaypipes | efried: I don't understand your last comment. could you elaborate? | 15:18 |
| efried | Sorry, I wasn't being as clever as you. | 15:19 |
| efried | I was thinking dumbly $PARENT_NAME - $MY_NAME | 15:19 |
| efried | Which means your example would have ended up being cn1_cn1_numa0_sriovpf0 | 15:19 |
| efried | um | 15:20 |
| jaypipes | ah. no, just use the parent name, not the concat of the ancestry tree | 15:20 |
| efried | that's not even right. | 15:20 |
| efried | yeah | 15:20 |
| jaypipes | so, the "formula" would be just: {$PARENT_NAME}_{$CHILD_NAME} | 15:20 |
| efried | well, since names have to be unique, that should work just fine. | 15:21 |
| jaypipes | or whatever separator you wanted to use instead of _ | 15:21 |
| efried | So we have to establish the naming convention for $CHILD_NAME, which will be different for any given type of provider x hypervisor/virt | 15:21 |
| efried | and document it | 15:21 |
| efried | and then operators ought to be able to figure it out from there | 15:21 |
| efried | if they need to | 15:21 |
| jaypipes | efried: do we really though? | 15:21 |
| efried | well, if we don't, then they have to do the two-step | 15:22 |
| efried | ask placement, figure it out, use that to populate the file, which then gets used to update placement. | 15:22 |
| openstackgerrit | Aditya Vaja proposed openstack/nova stable/queens: fix typo in IVS related privsep method https://review.openstack.org/604817 | 15:22 |
| jaypipes | efried: I mean if the "use case" is just for deployment tooling to be able to list the inventory for a compute node and its children, we already have the one heuristic that would be needed. | 15:22 |
| jaypipes | efried: i.e. ansbile would always call `openstack provider inventory list $HYPERVISOR_HOSTNAME` | 15:22 |
| efried | isn't there a chicken/egg though? | 15:23 |
| jaypipes | efried: and to find the child providers it would be: | 15:23 |
| *** dave-mccowan has joined #openstack-nova | 15:23 | |
| jaypipes | `openstack provider list --in-tree $HYPERVISOR_HOSTNAME` | 15:23 |
| jaypipes | efried: well, of course, there's not going to be any inventory or provider records until the nova-compute runs, but I don't think that's any different from today's landscape for ansible/deploy tools | 15:24 |
| efried | it is | 15:24 |
| efried | because in today's tooling, you can set cpu_allocation_ratio beforehand | 15:24 |
| jaypipes | and? | 15:25 |
| efried | Well, I don't know if we really care enough to feel it's worth the hoops of fire it would take to make this new world similarly configurable-before-deployment. | 15:25 |
| efried | But if I'm understanding cdent's concern correctly, that was it. | 15:25 |
| jaypipes | what regarding the provider descriptor file format would prevent ansible from writing out an allocation ratio override for the compute node? | 15:25 |
| efried | oh, that's exactly the point. The allocation ratio for CPU may not *live* in the compute node provider. | 15:26 |
| efried | It may live in the (multiple) NUMA node providers | 15:26 |
| efried | which ansible would need a way to figure out how to identify before deployment, if we're wanting to keep this paradigm of pre-deployment configurability. | 15:26 |
| jaypipes | efried: right, but we don't currently set any allocation ratio for NUMA node providers anywhere (certainly not in any config file) | 15:26 |
| efried | NUMA is just an example. | 15:27 |
| efried | and | 15:27 |
| efried | we don't today, but might tomorrow | 15:27 |
| jaypipes | efried: this is why I think setting data for inventory records using configuration files is silly. | 15:27 |
| efried | you think it should be done by invoking placement after all the auto setup is done? | 15:27 |
| jaypipes | as opposed to setting them en-masse after the records have been created. | 15:28 |
| jaypipes | yes. | 15:28 |
| jaypipes | but we have deployers who are insisting on the configuration files... soo.... | 15:28 |
| efried | I'd be cool with that. Requires a fundamental shift in the philosophy | 15:28 |
| efried | basically each virt's update_provider_tree would have to agree to only muck with total and reserved. | 15:34 |
| efried | which is not what we're doing today - today we overwrite everything. | 15:34 |
| efried | but there's still the concern about how to set initial values. | 15:34 |
| efried | The difficult part of that being how to tell that it's initial. | 15:34 |
| efried | cdent suggested something about making use of the updated_at field. | 15:35 |
| efried | bbar (that's "be back after reboot" - something's slowly eating up all my swap space) | 15:36 |
| *** efried has quit IRC | 15:37 | |
| *** dpawlik has joined #openstack-nova | 15:38 | |
| *** efried has joined #openstack-nova | 15:39 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 15:40 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 15:42 |
| *** dpawlik has quit IRC | 15:42 | |
| efried | belmoreira: did you get past your max_unit snafu from last week? | 15:43 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 15:44 |
| belmoreira | efried: we decided to not change it for now | 15:46 |
| efried | belmoreira: Meaning you still can't deploy those flavors to those nodes? | 15:47 |
| *** alexchadin has quit IRC | 15:47 | |
| belmoreira | efried: correct. People with these flavors can't use them. We're informing the users for now and suggesting smaller flavors | 15:48 |
| efried | okey | 15:48 |
| efried | belmoreira: Have you crystallized any sense of how you'd ideally like to see this handled big-picture/long-term? | 15:49 |
| belmoreira | efried: our main issue is that our compute nodes may be very overcommited (CPU) and for now is saffer to stop scheduling these large flavors | 15:49 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 15:50 |
| belmoreira | efried: last week I was debugging and we just realised the issue in the channel... I believe that letting the operator decide is a good practice. In my specific use case this is not a priority. If we allow these flavors again this is small downstream patch | 15:52 |
| *** sapd1 has joined #openstack-nova | 15:53 | |
| efried | roger that. Thanks for the update. | 15:53 |
| mriedem | dansmith: so circling back on https://review.openstack.org/#/c/604687/3/nova/db/sqlalchemy/migrate_repo/versions/391_add_volume_type_to_bdm.py - you're cool with the compute API taking volume type name or ID yes? | 15:53 |
| dansmith | mriedem: yes, but I'm not sure what that has to do with that migration.. it already has char(255) right? | 15:54 |
| mriedem | yes | 15:54 |
| mriedem | that's just what initiated the question | 15:54 |
| belmoreira | efried cdent also was testing if there's any limitation creating VMs with more vcpus than cpus available in the node and can't find any issue | 15:54 |
| efried | good to know. | 15:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consumer gen support for delete instance allocations https://review.openstack.org/591597 | 16:02 |
| *** helenafm has quit IRC | 16:03 | |
| *** lbragstad has quit IRC | 16:10 | |
| bauzas | gibi: I'll start review your series tomorrow morning, can I ? | 16:13 |
| gibi | bauzas: sure. There are comment already that I need to fix through the series but the content of the patches are ready for review | 16:13 |
| bauzas | all cool then ++ | 16:14 |
| bauzas | aaaand then, calling it a day ! | 16:14 |
| gibi | bauzas: have a nice evening | 16:16 |
| *** lbragstad has joined #openstack-nova | 16:18 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consumer gen support for delete instance allocations https://review.openstack.org/591597 | 16:18 |
| *** janki has quit IRC | 16:20 | |
| *** spotz is now known as spotz_ | 16:25 | |
| *** spotz_ is now known as spotz | 16:25 | |
| *** icey has quit IRC | 16:29 | |
| *** adrianc has joined #openstack-nova | 16:30 | |
| *** icey has joined #openstack-nova | 16:33 | |
| *** dave-mccowan has quit IRC | 16:38 | |
| *** ttsiouts has joined #openstack-nova | 16:38 | |
| *** s10 has quit IRC | 16:40 | |
| *** sahid has quit IRC | 16:41 | |
| *** jiapei has quit IRC | 16:43 | |
| melwitt | o/ | 16:45 |
| *** ttsiouts has quit IRC | 16:45 | |
| sean-k-mooney | o/ | 16:45 |
| *** s10 has joined #openstack-nova | 16:47 | |
| s10 | https://bugs.launchpad.net/nova/+bug/1209101 - please, reopen this bug | 16:48 |
| openstack | Launchpad bug 1209101 in OpenStack Compute (nova) "Non-public flavor cannot be used in created tenant" [High,Fix released] - Assigned to Sumanth Nagadavalli (sumanth-nagadavalli) | 16:48 |
| *** s10 has quit IRC | 16:53 | |
| *** hamzy_ has joined #openstack-nova | 16:58 | |
| *** adrianc has quit IRC | 16:59 | |
| *** derekh has quit IRC | 16:59 | |
| *** hamzy has quit IRC | 16:59 | |
| *** adrianc has joined #openstack-nova | 16:59 | |
| *** dpawlik has joined #openstack-nova | 17:00 | |
| cfriesen | mriedem: stephenfin: any chance you could take a look at a robustness fix around port binding in rebuild/reboot? https://review.openstack.org/603844 | 17:02 |
| *** belmoreira has quit IRC | 17:04 | |
| *** alex_xu has quit IRC | 17:05 | |
| openstackgerrit | Merged openstack/nova stable/pike: Fix message for unexpected external event https://review.openstack.org/589503 | 17:06 |
| openstackgerrit | Merged openstack/nova master: Rename "polling_changes-since_parameter.rst" https://review.openstack.org/604606 | 17:06 |
| *** adrianc has quit IRC | 17:06 | |
| *** udesale has quit IRC | 17:10 | |
| *** jpena is now known as jpena|off | 17:21 | |
| *** mvkr has quit IRC | 17:25 | |
| *** ralonsoh has quit IRC | 17:34 | |
| *** dave-mccowan has joined #openstack-nova | 17:35 | |
| *** sambetts is now known as sambetts|afk | 17:39 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Always check return of wait_for_versioned_notifications https://review.openstack.org/604859 | 17:42 |
| mdbooth | ^^^ took me all day :/ | 17:43 |
| mdbooth | The failure in my test, that is. I haven't actually checked locally if the above patch works. | 17:43 |
| mdbooth | I wonder if it should just raise an exception instead, tbh. Would make it harder to misuse. | 17:45 |
| *** munimeha1 has quit IRC | 17:47 | |
| *** dpawlik has quit IRC | 17:51 | |
| *** dtantsur is now known as dtantsur|afk | 17:56 | |
| efried | mdbooth: "it" the fixture or "it" the original method? | 18:10 |
| efried | oh. it's only in the fixture | 18:12 |
| *** coreycb has joined #openstack-nova | 18:13 | |
| *** mvkr has joined #openstack-nova | 18:15 | |
| *** dpawlik has joined #openstack-nova | 18:17 | |
| *** dpawlik_ has joined #openstack-nova | 18:17 | |
| efried | makes sense for it to raise on None, I reckon. But if you're going that far, you may as well make _Sub.wait_n raise on timeout too. | 18:18 |
| *** dpawlik has quit IRC | 18:18 | |
| *** AJaeger has joined #openstack-nova | 18:23 | |
| AJaeger | mriedem, bauzas, melwitt , lyarwood, could you help reviewing the stable python3-first changes, please? https://review.openstack.org/#/q/topic:python3-first+status:open+(openstack/nova+OR+project:openstack/nova-specs+OR+openstack/os-traits+OR+openstack/os-vif+OR+openstack/osc-placement+OR+openstack/python-novaclient) gives list of open changes | 18:25 |
| sean-k-mooney | AJaeger: looking at the list everything that is left is for stable branches | 18:27 |
| sean-k-mooney | AJaeger: the trove change is likely the wrong channel | 18:27 |
| AJaeger | sean-k-mooney: yeah, don't know why the query includes that one ;( Adn yes, it's all stable changes... | 18:27 |
| lyarwood | AJaeger: ack will do | 18:27 |
| AJaeger | thanks, lyarwood. If you have questions, feel free to ask ... | 18:28 |
| AJaeger | lyarwood: and ignore the trove one, please | 18:28 |
| lyarwood | AJaeger: ack, I can't +2 that anyway :) | 18:28 |
| openstackgerrit | Alessandro Pilotti proposed openstack/python-novaclient master: Fixes Python3 issue in decoding password https://review.openstack.org/604870 | 18:29 |
| melwitt | AJaeger: thanks for the heads up | 18:31 |
| AJaeger | melwitt: once those 15 changes are in, the python3-first goal is done for nova ;) | 18:32 |
| AJaeger | Only 13, I miscounted | 18:33 |
| melwitt | coolness, I'll make sure we get those in | 18:33 |
| AJaeger | great | 18:33 |
| AJaeger | you have at least changes that pass everywhere - compared to other projects ;/ | 18:33 |
| melwitt | that's fortunate :) | 18:34 |
| AJaeger | indeed | 18:35 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression for bug 1787606 https://review.openstack.org/593073 | 18:38 |
| openstack | bug 1787606 in OpenStack Compute (nova) "Multi instance creation rescheduling fails due to a lack of alternates" [Medium,In progress] https://launchpad.net/bugs/1787606 - Assigned to Lee Yarwood (lyarwood) | 18:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: scheduler: Increase alternate count in smaller environments https://review.openstack.org/593074 | 18:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | 18:45 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 18:45 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 18:45 |
| AJaeger | all approved - thanks, mriedem and lyarwood ! | 18:54 |
| *** panda has quit IRC | 18:56 | |
| openstackgerrit | Merged openstack/os-traits stable/rocky: import zuul job settings from project-config https://review.openstack.org/601403 | 19:11 |
| openstackgerrit | Merged openstack/os-traits stable/queens: import zuul job settings from project-config https://review.openstack.org/601398 | 19:11 |
| openstackgerrit | Merged openstack/os-traits stable/pike: import zuul job settings from project-config https://review.openstack.org/601393 | 19:11 |
| karimull | mreidem : thanks for the info..will look into hooks. | 19:13 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add support changes-before for microversion 2.66 https://review.openstack.org/603549 | 19:14 |
| *** sapd1 has quit IRC | 19:15 | |
| *** pcaruana has quit IRC | 19:17 | |
| *** s10 has joined #openstack-nova | 19:17 | |
| karimull | mreidem : efried: is there a way in nova I can branch out of normal processing of instance launch and try to work on the image before libvirt is called or with in libvirt is also fine | 19:19 |
| karimull | mreidem:efried : basically I'm looking to decrypt an image before it is launched... | 19:20 |
| efried | karimull: I have way more questions than answers. | 19:22 |
| efried | Are you the only one who has ever wanted to work with encrypted images? | 19:22 |
| efried | Is the image encrypted in glance, and then you want to decrypt it while/after you copy it to the instance's boot disk? | 19:22 |
| karimull | may be :) | 19:23 |
| karimull | efried : exactly | 19:23 |
| efried | I guess what I'm getting at is, either what you're doing is wild and crazy and you shouldn't be doing it - upstream or down - or it's something that more people want to do and is either already supported or should be proposed formally upstream. | 19:23 |
| efried | Me, I don't know anything about it, I'm afraid. | 19:24 |
| efried | seems weird that you're maintaining the image encrypted in glance, but want it decrypted *before* you boot the instance. | 19:25 |
| efried | It's as if you trust glance less than you trust instances | 19:25 |
| karimull | efried :I could see volume encryption blue but nothing on image | 19:25 |
| karimull | efried : I'm making sure if this is feasible before proposing a formal blue print | 19:28 |
| efried | karimull: Okay, so you do intend to propose it upstream? | 19:29 |
| karimull | efried : yes | 19:29 |
| efried | I see. Have you talked to the glance folks about it? | 19:29 |
| karimull | efried : not yet | 19:30 |
| melwitt | we added support for trusted image certificate validation in rocky https://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/nova-validate-certificates.html | 19:31 |
| dansmith | presumably they want encryption | 19:31 |
| dansmith | but that came before, AFAIK | 19:31 |
| melwitt | but I don't know of any support for encrypted images in glance | 19:32 |
| melwitt | yeah, was just mentioning it in case it might be useful | 19:32 |
| melwitt | that's the extent of the handling of "untrusted glance" that I know about | 19:33 |
| dansmith | oh I thought the encryption support was already there | 19:33 |
| dansmith | maybe I'm thinking of encrypted block | 19:33 |
| melwitt | I'm not sure, it might be there. trying to find out. an earlier iteration of the trusted certs stuff mentioned image encryption | 19:34 |
| dansmith | yeah | 19:35 |
| dansmith | looks like just signatures though in the tree | 19:35 |
| karimull | I have not seen any support for encrypted image in glance.. | 19:35 |
| efried | assuming the decrypt would happen chunk-wise, it's not in the nova glance code. | 19:35 |
| karimull | wanted to support user defined encryption of image at nova compute for more flexibility | 19:36 |
| efried | karimull: Point is, assuming it's not already there, you would likely be looking to make your changes in a lot of the same places as the bp melwitt mentioned ( https://review.openstack.org/#/q/topic:bp/nova-validate-certificates+(status:open+OR+status:merged) ) | 19:36 |
| *** panda has joined #openstack-nova | 19:37 | |
| karimull | by using Castellan which support key manager interface and by having a plugin in nova to perform user defined decryption process it will be more transparent..just a thought still framing on all possibilities | 19:39 |
| karimull | efried: will look into that blueprint.. | 19:39 |
| melwitt | karimull: are you thinking this would be transparent to glance? like you would encrypt the image before uploading to glance using your nova keypair, for example, and then you'd like nova to decrypt it? we would need the private key for that though and we don't store them | 19:39 |
| karimull | yes | 19:40 |
| dansmith | that's where castellan or barbican comes in | 19:40 |
| dansmith | nova gets a key the user provides there to decrypt | 19:41 |
| melwitt | right.. ok | 19:41 |
| dansmith | AFAIK, glance needs to look at the image when you upload it so it's not like you can do this without glance at all I think | 19:41 |
| dansmith | unless there is some way to tell glance not to look at the image, but I'm not sure | 19:41 |
| karimull | user will get the key from either barbican or from their own KMS and encrypt and upload the image with information in meta data , using that information and castellan libraries key will be retrieved for decryption of image | 19:42 |
| dansmith | unless you care about hiding the boot content from everything other than nova, this is pretty easy to do internal to the image without a lot of fanfare | 19:42 |
| dansmith | also, you'd probably want to make sure we don't cache the decrypted image, especially if the cache is on shared storage | 19:43 |
| dansmith | gets out of hand pretty quick :) | 19:43 |
| karimull | ok | 19:43 |
| karimull | dansmith: wanted to decrypt the image at compute host before it is launched..is this possible?..if we can have hooks at libvirt or nova-compute level wanted to make it a plugin | 19:46 |
| dansmith | karimull: we don't have plugins | 19:46 |
| dansmith | we have some aging hooks that are slowly being removed from the code | 19:47 |
| dansmith | but obviously doing the decryption on the compute host is where it would need to happen | 19:47 |
| karimull | having a plugin kind of functionality will give user flexibility to use their own decryption process..hence look in that way..do we have any similar way to do it in Nova | 19:49 |
| karimull | dansmith : looking* | 19:50 |
| dansmith | we don't have plugins | 19:50 |
| *** dpawlik_ has quit IRC | 19:51 | |
| *** dpawlik has joined #openstack-nova | 19:55 | |
| dansmith | mriedem: jaypipes: what's the fix for this? https://bugs.launchpad.net/nova/+bug/1793747 | 19:55 |
| openstack | Launchpad bug 1793747 in OpenStack Compute (nova) "Fails to boot instance using Blazar flavor if compute host names are in uppercase" [High,Triaged] - Assigned to Neha Alhat (nehaalhat) | 19:55 |
| *** mlavalle has quit IRC | 19:56 | |
| *** dpawlik has quit IRC | 19:56 | |
| dansmith | I don't even think I get what the problem is | 19:58 |
| *** mlavalle has joined #openstack-nova | 19:58 | |
| openstackgerrit | Merged openstack/nova stable/ocata: Cleanup RP and HM records while deleting a compute service. https://review.openstack.org/603749 | 19:59 |
| dansmith | oh, I see, I was looking at the wrong thing.. we're lower()ing all the hostnames | 19:59 |
| dansmith | s10: okay I got all those backports you tagged me on | 20:02 |
| s10 | dansmith: thank you, finally we will get this fixes in queens after two month of waiting :) | 20:04 |
| dansmith | s10: we just got a queens release this morning though right? | 20:04 |
| dansmith | might already be time to queue up another one :) | 20:04 |
| *** erlon has quit IRC | 20:05 | |
| mriedem | and we just released that blazar regression https://review.openstack.org/#/c/585334/ | 20:06 |
| mriedem | dansmith: i don't know what the fix is for that bug | 20:07 |
| jaypipes | dansmith: the fix for this is not having such fragile friggin code? :( | 20:07 |
| jaypipes | fix one thing, breaks another. :( | 20:07 |
| dansmith | jaypipes: yeah we should totes just depend on our backend database ignoring case for us :) | 20:07 |
| dansmith | mriedem: we could try to lower() the hostname everywhere else, but I kinda think the original "fix" was broken | 20:07 |
| jaypipes | dansmith: the user expects a case-insensitive search. | 20:08 |
| dansmith | if they pass a hostname that is different from what the machine reports, they should expect it to not work | 20:08 |
| dansmith | jaypipes: I don't | 20:08 |
| dansmith | the aggregate code must not be validating hostnames when you go to add one right? | 20:08 |
| dansmith | in which case maybe the fix is just to make host-add fail if you specify something wrong? | 20:09 |
| jaypipes | dansmith: this isn't about that. this is about the collection of host aggregate states in the scheduler (in Python, not in the DB) | 20:09 |
| dansmith | jaypipes: the original | 20:09 |
| jaypipes | and Python is case-sensitive, as we know. | 20:09 |
| dansmith | fix and the new regression are all about us allowing you to add a host with a non-matching case, | 20:09 |
| dansmith | and then us not also ignoring case when we go to join it up right? | 20:09 |
| dansmith | if we just refuse to let them add non-matching hostnames in the first place, everything else can be consistent right? | 20:10 |
| jaypipes | I need to look (again) at the code. it's a giant ball of turds. | 20:10 |
| jaypipes | dansmith: I don't think this is about them adding non-case-matching hostnames. | 20:11 |
| dansmith | I don't expect to have case ignored. what I do expect is for nova to tell me "that's, like, not a host maan" when I go to add one to an aggregate | 20:11 |
| mriedem | non-matching by looking up the host from the compute_nodes table? | 20:11 |
| dansmith | jaypipes: it is.. the original fix says "accidentally typed COMPUTE0 instead of compute0" | 20:11 |
| jaypipes | dansmith: no, I'm talking about the bug 1793747 | 20:11 |
| openstack | bug 1793747 in OpenStack Compute (nova) "Fails to boot instance using Blazar flavor if compute host names are in uppercase" [High,Triaged] https://launchpad.net/bugs/1793747 - Assigned to Neha Alhat (nehaalhat) | 20:11 |
| dansmith | jaypipes: and the regression is that blazar is taking the mixed-case hostname from the hypervisors api, and using that to add the host to an aggregate | 20:11 |
| jaypipes | dansmith: there's no indication that that bug reporter has used non-matching hostname... | 20:12 |
| dansmith | jaypipes: they're the same thing | 20:12 |
| dansmith | jaypipes: blazar is looking at hypervisors and using that value.. | 20:12 |
| mriedem | fwiw, bug 1709260 wouldn't be possible by default if they were using postgresql :P | 20:12 |
| dansmith | blazar host-create Openstack-VirtualBox | 20:12 |
| openstack | bug 1709260 in OpenStack Compute (nova) queens "Addition of host to host-aggregate should be case -sensitive" [Low,Fix committed] https://launchpad.net/bugs/1709260 - Assigned to Rajesh Tailor (ratailor) | 20:12 |
| jaypipes | dansmith: that's the correct hostname. | 20:12 |
| dansmith | jaypipes: right exactrly | 20:13 |
| dansmith | jaypipes: but we're mangling it internally by lower()ing it | 20:13 |
| dansmith | and they can't see that | 20:13 |
| *** dpawlik has joined #openstack-nova | 20:13 | |
| jaypipes | dansmith: where are we mangling it internally other than the host manager's host aggregate state internal map? | 20:13 |
| dansmith | exactly there | 20:13 |
| dansmith | that's the problem right? | 20:14 |
| jaypipes | dansmith: and how exactly would PG vs. MySQL "solve" this problem? | 20:14 |
| dansmith | jaypipes: mriedem said that not me | 20:14 |
| dansmith | I don't think it would | 20:14 |
| dansmith | unless PG honors case, but rejects duplicates that differ only by case | 20:14 |
| jaypipes | "<dansmith> jaypipes: yeah we should totes just depend on our backend database ignoring case for us :)" | 20:14 |
| mriedem | PG is case sensitive by default | 20:14 |
| openstackgerrit | Merged openstack/python-novaclient stable/queens: Switch to stestr https://review.openstack.org/601933 | 20:15 |
| openstackgerrit | Merged openstack/python-novaclient stable/queens: import zuul job settings from project-config https://review.openstack.org/601400 | 20:15 |
| mriedem | so fat fingering COMPUTE0 should result in HostNotFound | 20:15 |
| dansmith | mriedem: I don't think it would if we're not checking | 20:15 |
| mriedem | 1709260 | 20:15 |
| mriedem | oops | 20:15 |
| dansmith | or maybe you mean we're "checking" by just looking it up? | 20:15 |
| mriedem | mapping = objects.HostMapping.get_by_host(context, host_name) | 20:15 |
| mriedem | yes | 20:15 |
| dansmith | gotcha | 20:15 |
| dansmith | then yeah, mysql _is_ hurting us here | 20:15 |
| dansmith | (IMHO) | 20:15 |
| mriedem | there are lots of things like this in the api | 20:16 |
| dansmith | yup | 20:16 |
| jaypipes | I still don't see how MySQL is hurting us | 20:16 |
| mriedem | because adding the host to the aggregate with the wrong name doesn't puke in the api | 20:16 |
| jaypipes | the user expects a case-insenstive API. | 20:17 |
| dansmith | I don't and I'm a user | 20:17 |
| mriedem | you keep saying that but i'm not sure why | 20:17 |
| dansmith | yeah | 20:17 |
| jaypipes | the user that reported all these bugs. | 20:17 |
| mriedem | heh | 20:17 |
| dansmith | that user would be fine if nova told it it was wrong | 20:17 |
| mriedem | if i create a server with name Foo i don't expect to find it using FOO | 20:17 |
| openstackgerrit | Merged openstack/os-vif stable/queens: import zuul job settings from project-config https://review.openstack.org/601399 | 20:17 |
| openstackgerrit | Merged openstack/os-vif stable/pike: import zuul job settings from project-config https://review.openstack.org/601394 | 20:17 |
| openstackgerrit | Merged openstack/os-vif stable/ocata: import zuul job settings from project-config https://review.openstack.org/601388 | 20:17 |
| dansmith | the original bug even said "accidentally typed" | 20:17 |
| mriedem | yeah i think rajesh's patch / bug report was saying he expected HostNotFound | 20:18 |
| *** dpawlik has quit IRC | 20:18 | |
| dansmith | like because they haven't properly mapped caps lock to control like everyone should | 20:18 |
| mriedem | "While adding compute0 to host-aggregate, if I provide hostname as "COMPUTE0.example.com", instead of | 20:18 |
| mriedem | throwing HostNotFound error, it is added to host-aggregate." | 20:18 |
| dansmith | face it jaypipes, PG rules and mysql drools | 20:18 |
| jaypipes | perhaps that is because URIs and hostnames are case-insensitive and always have been? ... | 20:19 |
| * dansmith is TOTALLY joking | 20:19 | |
| jaypipes | GOOGLE.com == goOGle.com | 20:19 |
| dansmith | DNS is case insensitive | 20:19 |
| mriedem | so one fix for https://bugs.launchpad.net/nova/+bug/1793747 is to revert https://review.openstack.org/#/q/Iee4b9bbf412adfdc6fdc62ea3429fb960d6ac2a2 right? | 20:19 |
| openstack | Launchpad bug 1793747 in OpenStack Compute (nova) "Fails to boot instance using Blazar flavor if compute host names are in uppercase" [High,Triaged] - Assigned to Neha Alhat (nehaalhat) | 20:19 |
| dansmith | URIs are not, AFAIK, in apache | 20:19 |
| jaypipes | but whatevers, I need to go puke on chef again. | 20:19 |
| dansmith | mriedem: yes I think we should revert rajesh's patch and fix the NotFound on original add | 20:20 |
| mriedem | or, we need to lowercase aggregate members everywhere...? | 20:20 |
| mriedem | how do we fix the not found on original add if the db is case insensitive? | 20:20 |
| dansmith | we just check the result | 20:21 |
| dansmith | if host_name == compute.host | 20:21 |
| dansmith | or whatever | 20:21 |
| jaypipes | dansmith: and you'd need to fix PG manually since it will undoubtedly have allowed 2 records, one for compute0 and one for COMPUTE0 | 20:21 |
| mriedem | jaypipes: mysql manually | 20:21 |
| mriedem | PG won't allow this by default | 20:21 |
| jaypipes | in the host_aggregate_hosts table (or whatever the heck it's named) | 20:21 |
| dansmith | yeah PG is already fine | 20:21 |
| dansmith | no | 20:21 |
| dansmith | you won't have ever found COMPUTE0 | 20:21 |
| jaypipes | what are you guys talking about? | 20:21 |
| dansmith | so we would fail | 20:21 |
| dansmith | jaypipes: we look up the host before we add the aggegate_host | 20:22 |
| jaypipes | PG will allow multiple records to be created, one for agg -> COMPUTE0 and one for agg -> compute0 | 20:22 |
| dansmith | in mysql we find it wrongly, case insensitive | 20:22 |
| dansmith | in PG, we never find it, | 20:22 |
| jaypipes | ah. if that fetch is done in the API layer... | 20:22 |
| jaypipes | is it? | 20:22 |
| dansmith | so we won't add the aggregate_host | 20:22 |
| dansmith | we don't need the constraint to help us in PG, | 20:22 |
| dansmith | because we never get that far | 20:22 |
| dansmith | right | 20:22 |
| dansmith | mriedem says it is | 20:22 |
| dansmith | [13:15:46] <mriedem>mapping = objects.HostMapping.get_by_host(context, host_name) | 20:23 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L5289 | 20:23 |
| mriedem | dansmith: so you want to add: if mapping.host == host_name? in the api? | 20:24 |
| mriedem | for the case check? | 20:24 |
| dansmith | yeah | 20:24 |
| mriedem | yeah that seems easy enough | 20:24 |
| dansmith | wait, are you asking if I want to write that patch? | 20:24 |
| dansmith | or just if I like that approach? | 20:24 |
| mriedem | i was confirming that was your idea | 20:24 |
| dansmith | indeed | 20:24 |
| mriedem | i expect you're already half dressed for vacation | 20:24 |
| dansmith | indeed :) | 20:25 |
| mriedem | (un)dressed | 20:25 |
| mriedem | dansmith: are you going to propose the revert at least? | 20:25 |
| melwitt | will this change in anyway cause existing tooling to stop working? that is, do we need a microversion? | 20:25 |
| dansmith | I can if you want | 20:25 |
| mriedem | we already regressed existing tooling | 20:25 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1793747 | 20:25 |
| openstack | Launchpad bug 1793747 in OpenStack Compute (nova) "Fails to boot instance using Blazar flavor if compute host names are in uppercase" [High,Triaged] - Assigned to Neha Alhat (nehaalhat) | 20:25 |
| dansmith | melwitt: this is making the api consistent with the database, if your database is PG | 20:25 |
| dansmith | right | 20:26 |
| openstackgerrit | Merged openstack/python-novaclient stable/pike: import zuul job settings from project-config https://review.openstack.org/601395 | 20:26 |
| openstackgerrit | Merged openstack/python-novaclient stable/ocata: Use generic user for both zuul v2 and v3 https://review.openstack.org/601932 | 20:26 |
| openstackgerrit | Merged openstack/python-novaclient stable/ocata: import zuul job settings from project-config https://review.openstack.org/601391 | 20:26 |
| melwitt | I mean the addition of the case sensitive check, not the revert | 20:26 |
| dansmith | the check is ^ | 20:26 |
| mriedem | shouldn't need to opt out of broken behavior | 20:26 |
| mriedem | so this isn't a microversion | 20:26 |
| dansmith | right | 20:26 |
| melwitt | ok, just wondering if this might somehow break anyone's existing automation | 20:26 |
| dansmith | we already broke it | 20:27 |
| dansmith | this makes it work again | 20:27 |
| dansmith | anyone's automation is going to be using the actual hostname, | 20:27 |
| dansmith | either from the system or from our own API | 20:27 |
| dansmith | those things would have broken silently just now | 20:27 |
| dansmith | this will unbreak them | 20:27 |
| jaypipes | this is partly why I hate even reviewing anything like that patch any more... always seems that they get reverted. | 20:27 |
| dansmith | the combo | 20:28 |
| mriedem | i should have sniffed this out sooner on the backports given i knew about the earlier work from HPE to try and solve the case insensitivity issues in the api | 20:28 |
| mriedem | mdbooth was also working on something related to this recently | 20:29 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2018-August/thread.html#133202 | 20:29 |
| melwitt | yeah, that was the metadata keys case insensitive thing | 20:29 |
| dansmith | mriedem: | 20:29 |
| mriedem | beware then https://review.openstack.org/#/c/504885/ | 20:30 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/604898 | 20:30 |
| melwitt | it was confusing thinking about it, but the proposal was to make the key column case sensitive by changing the collation, but that would break anyone who was relying on the case insensitive behavior. and then I got fatigued thinking about it | 20:30 |
| dansmith | yep, | 20:31 |
| melwitt | so I was trying to determine whether there would be any similar thing here | 20:31 |
| dansmith | the difference here is that a hostname with a different name is not legit | 20:31 |
| dansmith | in an aggregate mapping | 20:31 |
| dansmith | since metadata is free-form, it is legit (although silly) to use two keys that differ only by case | 20:31 |
| dansmith | er, "hostname with a different case" | 20:32 |
| melwitt | ok, I see | 20:32 |
| dansmith | mriedem: I can stack the check on top of this if you'll write the tests when I run out of time | 20:33 |
| dansmith | mriedem: we could also just use the name from the mapping instead of the one they asked for, | 20:35 |
| dansmith | which means they won't get a NotFound, and we'll add the mapping they intended | 20:35 |
| dansmith | but they may fail similarly later if they keep using the wrong name | 20:35 |
| dansmith | so I tend to err on the side of fail fast, | 20:35 |
| dansmith | especially since PG will already cause us to behave that way | 20:35 |
| dansmith | thoughts? | 20:35 |
| mriedem | agree on the api being explicitly | 20:35 |
| mriedem | *explicit | 20:35 |
| melwitt | yeah it sounds like fail fast would be better | 20:38 |
| *** dpawlik has joined #openstack-nova | 20:40 | |
| openstackgerrit | Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds/reboots https://review.openstack.org/603844 | 20:40 |
| jaypipes | yeah, I definitely won't be reviewing 504885. Tired of approving these patches and then needing them to be reverted. | 20:47 |
| jaypipes | I'll just stick to reviewing anything that isn't in the API layer I guess. | 20:47 |
| jaypipes | since our API layer is so friggin eggshells. | 20:47 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906 | 20:49 |
| dansmith | mriedem: all yours if that fails other tests | 20:49 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906 | 20:50 |
| mriedem | ack | 20:55 |
| *** slaweq has quit IRC | 21:01 | |
| melwitt | reminder to everyone: forum topic submission deadline is Wed Sep 26 | 21:05 |
| melwitt | https://etherpad.openstack.org/p/nova-forum-stein | 21:06 |
| *** dpawlik has quit IRC | 21:12 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/604906 | 21:15 |
| mriedem | i don't know if we're going to do a placement extraction session at the forum | 21:15 |
| mriedem | kind of a big thing *not* to have a session about it for at least a status check and fyi | 21:16 |
| mriedem | for people that weren't at the ptg | 21:16 |
| melwitt | that's fair enough | 21:18 |
| *** awaugama has quit IRC | 21:18 | |
| melwitt | a status check would be good, at the least | 21:18 |
| mriedem | so i guess i'm submitting that one | 21:20 |
| * dansmith readies his voting finger | 21:20 | |
| melwitt | if you could, that would be greeeeaaat | 21:21 |
| melwitt | honestly, it's kinda hard to be thinking about what we need to talk about at the forum when we literally were just at the ptg | 21:22 |
| melwitt | other than picking out, which of these topics should be a repeat at summit time | 21:23 |
| melwitt | or a generic "nova ops/users feedback session" | 21:23 |
| mriedem | https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/22780 | 21:31 |
| melwitt | thanks | 21:33 |
| openstackgerrit | Merged openstack/osc-placement stable/queens: import zuul job settings from project-config https://review.openstack.org/601397 | 21:44 |
| *** takashin has joined #openstack-nova | 21:48 | |
| *** mriedem is now known as mriedem_away | 21:57 | |
| *** elod has quit IRC | 22:04 | |
| *** AJaeger_ has joined #openstack-nova | 22:10 | |
| *** elod has joined #openstack-nova | 22:11 | |
| *** AJaeger has quit IRC | 22:11 | |
| *** dpawlik has joined #openstack-nova | 22:16 | |
| *** dpawlik has quit IRC | 22:20 | |
| *** mgariepy has quit IRC | 22:20 | |
| *** mgariepy has joined #openstack-nova | 22:33 | |
| *** panda is now known as panda|off | 22:38 | |
| *** rcernin has joined #openstack-nova | 22:45 | |
| *** mgariepy has quit IRC | 22:49 | |
| *** pooja-jadhav has joined #openstack-nova | 22:51 | |
| *** owalsh_ has joined #openstack-nova | 22:53 | |
| *** kukacz_ has joined #openstack-nova | 22:55 | |
| *** dims_ has joined #openstack-nova | 22:59 | |
| *** jamiec_ has joined #openstack-nova | 22:59 | |
| *** mmedvede_ has joined #openstack-nova | 23:00 | |
| *** _d34dh0r53_ has joined #openstack-nova | 23:00 | |
| *** stephenfin_ has joined #openstack-nova | 23:00 | |
| *** kukacz has quit IRC | 23:00 | |
| *** pooja_jadhav has quit IRC | 23:00 | |
| *** mhen has quit IRC | 23:00 | |
| *** owalsh has quit IRC | 23:00 | |
| *** dims has quit IRC | 23:00 | |
| *** d34dh0r53 has quit IRC | 23:00 | |
| *** jlvillal has quit IRC | 23:00 | |
| *** jamiec has quit IRC | 23:00 | |
| *** sorrison has quit IRC | 23:00 | |
| *** cburgess has quit IRC | 23:00 | |
| *** naichuans_ has quit IRC | 23:00 | |
| *** fanzhang has quit IRC | 23:00 | |
| *** bandini has quit IRC | 23:00 | |
| *** wznoinsk has quit IRC | 23:00 | |
| *** mmedvede has quit IRC | 23:00 | |
| *** stephenfin has quit IRC | 23:00 | |
| *** ujjain has quit IRC | 23:00 | |
| *** mmedvede_ is now known as mmedvede | 23:00 | |
| *** andreykurilin has quit IRC | 23:03 | |
| *** purplerbot has quit IRC | 23:04 | |
| *** mgariepy has joined #openstack-nova | 23:04 | |
| *** andreykurilin has joined #openstack-nova | 23:05 | |
| *** tetsuro has joined #openstack-nova | 23:24 | |
| *** mgariepy has quit IRC | 23:26 | |
| *** hshiina has joined #openstack-nova | 23:37 | |
| *** s10 has quit IRC | 23:38 | |
| *** s10 has joined #openstack-nova | 23:39 | |
| *** mgariepy has joined #openstack-nova | 23:39 | |
| *** s10 has quit IRC | 23:39 | |
| *** s10 has joined #openstack-nova | 23:39 | |
| *** s10 has quit IRC | 23:40 | |
| *** s10 has joined #openstack-nova | 23:40 | |
| *** s10 has quit IRC | 23:41 | |
| *** s10 has joined #openstack-nova | 23:41 | |
| *** s10 has quit IRC | 23:41 | |
| *** s10 has joined #openstack-nova | 23:42 | |
| *** s10 has quit IRC | 23:42 | |
| *** erlon has joined #openstack-nova | 23:46 | |
| *** mgariepy has quit IRC | 23:47 | |
| *** mgariepy has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!