| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-23 | |||
| 19:21:40 | openstackgerrit | melanie witt proposed openstack/nova master: Correct the release notes related to nova-consoleauth https://review.openstack.org/595455 | |
| 19:24:02 | openstackgerrit | melanie witt proposed openstack/nova master: Correct the release notes related to nova-consoleauth https://review.openstack.org/595455 | |
| 19:28:36 | melwitt | mriedem: ^ updated the release note patch | |
| 19:29:25 | mriedem | so the config option help doesn't need to be updated? | |
| 19:30:40 | melwitt | oh, sorry. I missed that | |
| 19:32:45 | mriedem | looks ok otherwise | |
| 19:32:50 | mriedem | just update the option help and i think we're good | |
| 19:37:43 | openstackgerrit | melanie witt proposed openstack/nova master: Correct the release notes related to nova-consoleauth https://review.openstack.org/595455 | |
| 19:41:00 | melwitt | mriedem ^ | |
| 20:17:06 | openstackgerrit | Merged openstack/nova stable/rocky: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590801 | |
| 20:17:12 | openstackgerrit | Merged openstack/nova stable/queens: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590803 | |
| 20:38:44 | mriedem | melwitt: +2 | |
| 20:38:49 | mriedem | now to find another core | |
| 20:38:54 | melwitt | thanks | |
| 20:39:12 | melwitt | jaypipes: are you still around? can you hit this for rc3 please? https://review.openstack.org/595455 | |
| 20:46:09 | melwitt | going to have to wait for APAC folk, methinks | |
| 20:50:06 | mriedem | is dansmith around? | |
| 20:50:57 | melwitt | no, he started his day at 5am today so is off by now | |
| 20:51:02 | mriedem | oh | |
| 20:51:08 | melwitt | had a 5am meeting | |
| 20:51:37 | mriedem | and efried_afk is afk | |
| 20:51:40 | mriedem | how about i just +W? | |
| 20:52:13 | melwitt | could. I was thinking I have to be around to get stable cores for the stable/rocky backports anyway, so wasn't sure | |
| 20:52:46 | mriedem | tonyb is here for stable | |
| 20:52:49 | melwitt | https://review.openstack.org/594508 hasn't merged yet, still need review on the backport | |
| 20:52:56 | melwitt | once that merges | |
| 20:54:03 | mriedem | https://review.openstack.org/#/c/595455/ approved, can you propose the backport? | |
| 20:54:28 | melwitt | yarp. I should have proposed the other one too so you could review it. wasn't thinking | |
| 20:55:13 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Correct the release notes related to nova-consoleauth https://review.openstack.org/595890 | |
| 20:55:29 | melwitt | dangit, cherry-pick button doesn't do -x | |
| 20:55:36 | mriedem | not until it's merged | |
| 20:55:42 | melwitt | oh ok | |
| 20:55:52 | melwitt | that's what I get for trying buttons | |
| 20:57:48 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Correct the release notes related to nova-consoleauth https://review.openstack.org/595890 | |
| 21:01:24 | dansmith | mriedem: melwitt I put another +2 on 595455 | |
| 21:01:27 | dansmith | is that all? | |
| 21:02:57 | mriedem | and https://review.openstack.org/#/c/595864/ | |
| 21:03:03 | melwitt | dansmith: need stable review for this https://review.openstack.org/595864 master change not merged yet, so we'll have to hold till later for the second +2 | |
| 21:03:20 | melwitt | meaning, I'll ask tonyb after the master change lands | |
| 21:03:32 | melwitt | for the +W | |
| 21:03:34 | openstackgerrit | Surya Seetharaman proposed openstack/nova-specs master: Handling a down cell https://review.openstack.org/595892 | |
| 21:04:57 | dansmith | melwitt: okay I slapped a +2 on there | |
| 21:05:14 | melwitt | thx | |
| 21:06:01 | melwitt | ok, cool. just have to babysit the patches through the gate, get tonyb for the backports, then should be good to go | |
| 21:16:06 | openstackgerrit | Merged openstack/nova master: Normalize dashless 'resource provider create' uuid https://review.openstack.org/567191 | |
| 21:30:44 | openstackgerrit | Merged openstack/nova master: libvirt: Don't react to VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED events https://review.openstack.org/594508 | |
| 21:30:50 | openstackgerrit | Merged openstack/nova master: Update contributor guide for Stein https://review.openstack.org/591258 | |
| 21:30:56 | openstackgerrit | Merged openstack/nova master: [placement] Add functional test to verify presence of policy https://review.openstack.org/595559 | |
| 21:34:07 | mriedem | tonyb: can you hit this for RC3? https://review.openstack.org/#/c/595864/ | |
| 21:37:55 | tonyb | mriedem: done | |
| 21:38:34 | tonyb | anything else you need for RC3 you knwo where I am | |
| 21:40:31 | mriedem | the bottom of the world? | |
| 22:50:29 | openstackgerrit | Merged openstack/nova master: Add functional test for affinity with multiple cells https://review.openstack.org/585073 | |
| #openstack-nova - 2018-08-24 | |||
| 13:21:07 | mnaser | we have plenty of capacity of both, but i have a proposed solution where it just defaults to requiring hardware virt but allowing user to control that value if need be | |
| 13:21:38 | sean-k-mooney | mnaser: right but that is not something you can do via an image today | |
| 13:21:49 | mnaser | sean-k-mooney: it is, using the `img_hv_type` property | |
| 13:21:57 | sean-k-mooney | mnaser: that would be better handeled in a weigher | |
| 13:22:12 | mnaser | if you set it to `kvm`, it'll filter only those that can do hardware acceleration | |
| 13:22:20 | sean-k-mooney | mnaser: no that just allowas you to select the hypervior not the arch | |
| 13:22:27 | mnaser | sean-k-mooney: combined with hw_architecture :) | |
| 13:22:33 | mnaser | and https://review.openstack.org/#/c/566425/ | |
| 13:22:52 | sean-k-mooney | hw_arcitecture is menat to defien the arch of the vm. | |
| 13:23:37 | sean-k-mooney | mnaser: your relying on the fact that today kvm acclerate does not work when you are emulating a different architecture | |
| 13:23:41 | openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291 | |
| 13:24:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Add subtree filter for GET /resource_providers https://review.openstack.org/595236 | |
| 13:24:31 | mnaser | sean-k-mooney: i don't think that's likely going to change anytime soon™ | |
| 13:24:55 | sean-k-mooney | mnaser: well my first respocne to that patch is we should revert it with out a spec | |
| 13:25:12 | mnaser | and that would mean we can't deliver multiarchitecture in our cloud | |
| 13:25:17 | mnaser | so i'm not sure how that benefits anyone | |
| 13:25:33 | mnaser | the patch doesn't hurt anyone except help users deliver a better openstack experience | |
| 13:25:34 | sean-k-mooney | mnaser: you can you just need a weigher. | |
| 13:26:00 | mnaser | doesnt the scheduler filter out N number of items to weight | |
| 13:26:08 | sean-k-mooney | mnaser: my issue is you did this in a filter so its a hard requirement instead of a perference | |
| 13:26:15 | mnaser | it is not | |
| 13:26:32 | mnaser | if the hw_architecture field is empty | |
| 13:26:41 | mnaser | or there is no default configured value | |
| 13:26:43 | mnaser | there is no enforcement | |
| 13:27:03 | sean-k-mooney | yes but i cant set hw_architecture and hv_type anymore | |
| 13:27:09 | mnaser | yes you can | |
| 13:27:19 | mnaser | this patch only uses the configured default IF none is specified in the image. | |
| 13:27:32 | mnaser | you are more than welcome to set hw_arch to aarc64 and hv_type to qemu | |
| 13:27:40 | mnaser | and you'll get software virtualized arm64 vm | |
| 13:27:59 | mnaser | this is just the default values if *none* are specified, which the current openstack nova behavior = put it anywhere. | |
| 13:28:22 | sean-k-mooney | so i can set hw_architecture=x86 and hv_type=kvm and land on a powerkvm node? | |
| 13:28:49 | mnaser | sean-k-mooney: no, if no values are set, you can end up anywhere yes | |
| 13:28:57 | mnaser | but the thing is, 99% of our customers *dont* send hw_arch and hv_type | |
| 13:29:11 | mnaser | because they assume that the cloud is x86_64/kvm which is a reasonable assumption. | |
| 13:29:36 | sean-k-mooney | mnaser: its not how rackspace used to run there public cloud | |
| 13:29:38 | mnaser | multiarchitecture clouds are rare, so asking every single customer/user to tag their images with an architecture is not really smooth user experience, especially in a transition | |
| 13:30:21 | mnaser | sure. so you're saying that for someone to upload an image into openstack, we need to start asking for 3 more parameters | |
| 13:30:23 | sean-k-mooney | they had a large amount of powerpc nodes at one point with xen as the hypervior running with x86 guests | |
| 13:31:15 | mnaser | well, i'd say that's more of an outlier. i don't see how this helps solve things and make them easier for users overall in a multiarch environment | |
| 13:31:34 | mnaser | because the rax env you're describing sounds still like a single arch one for the user facing stuff | |
| 13:32:45 | sean-k-mooney | mnaser: well my point is if you implemented this as a weigher you would land on the same acrh in 99% of cases and fall back to hardware acllerated emulaton if not native nodes were available. | |
| 13:33:06 | mnaser | sean-k-mooney: but don't weighers run on a subset of compute nodes? | |
| 13:33:19 | sean-k-mooney | the chage is merged in anycase but i really dont think this should have been a bug. | |
| 13:33:30 | sean-k-mooney | mnaser: am the run after all the filters | |
| 13:33:49 | sean-k-mooney | so you go through placment then the filters then the weighers | |
| 13:33:57 | mnaser | i was pretty sure that weighers ran on a subset of hosts | |
| 13:33:58 | sean-k-mooney | they just reorder what the filters pass | |