| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-16 | |||
| 19:44:02 | NostawRm | like I'm going to be giving multiple IPs to the interfaces of instances, ant the metadata service just picks the first IP but that may not be the 'main ip' that was set aside for licensing purposes :/ | |
| 19:44:45 | harlowja | cfriesen NostawRm ya, i think smoser (another cloud-init guy, the main guy really) doesn't want config-drive at all, lol | |
| 19:44:51 | cfriesen | mriedem: doing an upgrade from Mitaka to Newton, one of the "type-PF" entries in table pci_devices ended up with a non-null entry under the "parent_addr" column, which is I think unexpected. Not sure yet if it's upstream code or ours, but its odd. | |
| 19:45:33 | mriedem | cfriesen: i believe that's already fixed | |
| 19:45:52 | NostawRm | harlowja, :/ that's odd, I like the config-drive better, partially because I wasn't going to use DHCP | |
| 19:45:59 | mriedem | cfriesen: https://github.com/openstack/nova/commit/ad12fa65f5cc06bdee52be49b7370d703855d618 | |
| 19:46:06 | cfriesen | harlowja: I proposed dropping it and allowing specifying personality files on an evacuate the way we do now for rebuild. (since config drive gets lost on evacuate currently) | |
| 19:46:31 | cfriesen | NostawRm: by the way, config drive gets lost on evacuate. :) | |
| 19:46:34 | harlowja | cfriesen right | |
| 19:46:34 | mriedem | cfriesen: you should be getting the latest stable branch releases before doing your upgrade | |
| 19:46:49 | NostawRm | cfriesen, don't think they do when using rbd, but I'd have to double check | |
| 19:47:13 | harlowja | NostawRm ya, i mean i don't mind config-drive, its just more flakey than a webservice | |
| 19:47:17 | mriedem | eandersson: superdan: mitaka backport https://review.openstack.org/475067 | |
| 19:47:21 | harlowja | especially i think in migrations, live-migrtions.... | |
| 19:47:41 | harlowja | some of those were prob bugs that got fixed, but its always seem to have been more 'finiky?' | |
| 19:48:42 | superdan | mriedem: thanks for doing that | |
| 19:49:36 | mriedem | superdan: here is that expected nova-status upgrade check warning for the CoreFilter thing from bauzas' change http://logs.openstack.org/00/427200/4/check/gate-grenade-dsvm-neutron-ubuntu-xenial/59e0c78/logs/grenade.sh.txt.gz#_2017-06-16_17_32_25_590 | |
| 19:49:40 | mriedem | https://review.openstack.org/#/c/427200/ | |
| 19:49:49 | mriedem | in grenade we ignore everything below failure | |
| 19:50:40 | mriedem | i figure if/when we drop ram/disk filters from enabled_filters defaults, we can bump the allocation_ratios for those 3 to 9999.0 in devstack | |
| 19:51:40 | cfriesen | mriedem: we're based on 14.0.6, nothing in 14.0.7 would affect the pci_devices table migration, I don't think. | |
| 19:51:48 | superdan | mriedem: okay I was kinda thinking maybe a warning if those _were_ enabled to say they really should be removed at this point | |
| 19:51:57 | superdan | mriedem: but if this is what you want that's fine | |
| 19:52:27 | cfriesen | mriedem: in our case the "parent_addr" column was fine before the upgrade, but messed up after | |
| 19:52:32 | mriedem | cfriesen: that fix went into 14.0.6 | |
| 19:52:34 | cfriesen | mriedem: but just for one device | |
| 19:52:51 | mriedem | but i suppose maybe you were already past that migration? | |
| 19:52:55 | cfriesen | mriedem: and that fix came from us. :) | |
| 19:53:43 | mriedem | superdan: we'd have to remove them from the default enabled_filters list first | |
| 19:53:50 | mriedem | otherwise all new installs would start out with warnings | |
| 19:54:04 | mriedem | well, as is the case here with CoreFilter | |
| 19:54:05 | mriedem | ... | |
| 19:54:31 | cfriesen | mriedem: we had run that online data migration in Mitaka to convert all the stuff from the previous release, so the DB looked good in Mitaka. So something got messed up either in the offline or online DB migrations going to Newton. | |
| 19:54:50 | cfriesen | could be internal to us, will have to track it down | |
| 19:57:12 | superdan | mriedem: right | |
| 19:57:26 | mriedem | superdan: so this check should probably be seeing if we're a fresh install... | |
| 19:57:35 | mriedem | and not warn if you don't have those filters enabled | |
| 20:25:26 | openstackgerrit | Eric Fried proposed openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146 | |
| 20:42:47 | cfriesen | are there known issues with 'nova list' pagination in Newton? | |
| 20:45:11 | cfriesen | we seem to have hit a case where "servers_links" is not included even though there are more servers than have been listed so far | |
| 20:48:12 | melwitt | mriedem: replied on the fixed ips quota patch. the NoopQuotaDriver will always 'pass' on the limit check, but the counting will still happen. that's latent, but it's true it gets worse with the series in that more things become CountableResources instead of just a few | |
| 20:55:29 | sdague | mriedem: https://review.openstack.org/#/c/475067 - the mitaka change, is hitting keystone compatiblity issues | |
| 20:56:36 | superdan | um | |
| 20:56:43 | superdan | did gerrit just hang for anyone else? | |
| 20:58:01 | cfriesen | superdan: nope | |
| 20:58:02 | sdague | mriedem / superdan - I'm going to throw this up there as a potential work around - https://review.openstack.org/475073 | |
| 20:58:20 | sdague | if that passes, I'd be fine pushing that through to backport this thing | |
| 20:58:26 | openstackgerrit | Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627 | |
| 21:27:42 | cheran | @here Can anyone help me getting the filter properties from the spec_obj in mitaka for the ScheduleFilter https://docs.openstack.org/releasenotes/nova/mitaka.html | |
| 21:27:58 | cheran | "Filters internal interface changed using now the RequestSpec NovaObject instead of an old filter_properties dictionary. In case you run out-of-tree filters, you need to modify the host_passes() method to accept a new RequestSpec object and modify the filter internals to use that new object. You can see other in-tree filters for getting the logic or ask for help in #openstack-nova IRC channel." | |
| 21:28:19 | cheran | The request_spec object doesn;t have the filter, so my question is that do I need to add a new filter dict into that? | |
| 21:40:13 | melwitt | cheran: I don't think so. the RequestSpec has a get_scheduler_hint() method you can call for getting the hints related to your filter | |
| 21:43:57 | ihrachys | mriedem, howdy. I see this in newton: http://logs.openstack.org/75/474275/2/check/gate-tempest-dsvm-neutron-dvr-ubuntu-xenial/d4c39c7/logs/screen-n-cpu.txt.gz#_2017-06-15_19_53_17_745 shouldn't it have been fixed by newer libvirt? | |
| 21:44:02 | ihrachys | or don't we have it in newton? | |
| 21:47:52 | ihrachys | oh it seems like devstack does enable it in master only :-x | |
| 21:48:27 | cheran | melwitt: Thanks, let me try that.. | |
| 21:49:00 | melwitt | cheran: if you grep for get_scheduler_hint in nova/scheduler/filters/ you can see examples | |
| #openstack-nova - 2017-06-17 | |||
| 00:34:13 | mriedem | melwitt: but that's only if you've configured to use the noop quota driver right? the vlan network manager was hardcoding the usage of the noop quota driver, it wasn't based on config. | |
| 08:14:54 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Fix typo VIFVIFHostDeviceDevType to VIFHostDeviceDevType https://review.openstack.org/475118 | |
| 09:05:51 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Use versionedobjects PCIAddress field https://review.openstack.org/475120 | |
| #openstack-nova - 2017-06-18 | |||
| 09:29:06 | openstackgerrit | Moshe Levi proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 10:18:41 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 12:29:29 | openstackgerrit | Alex Szarka proposed openstack/nova master: [WIP]Transform instance-live_migration_abort notification https://review.openstack.org/420453 | |
| 20:40:04 | openstackgerrit | Merged openstack/os-vif master: Fix typo VIFVIFHostDeviceDevType to VIFHostDeviceDevType https://review.openstack.org/475118 | |
| #openstack-nova - 2017-06-19 | |||
| 01:21:20 | openstackgerrit | Shunli Zhou proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | |
| 02:00:21 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add X-OpenStack-Request-Id description https://review.openstack.org/474847 | |
| 02:13:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 02:13:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 02:13:55 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 02:44:05 | MarginHu | Hi , I met an issuse, maybe it's a bug | |
| 02:47:24 | MarginHu | it reports "Booting with blank volume at /dev/vda" | |
| 04:26:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in API Versions https://review.openstack.org/475226 | |
| 04:27:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix missing parameters in API Versions https://review.openstack.org/475226 | |
| 05:45:23 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/467173 | |
| 06:17:53 | openstackgerrit | Tovin Seven proposed openstack/python-novaclient master: Make --profile load from environment variables https://review.openstack.org/475251 | |
| 06:37:25 | openstackgerrit | Tovin Seven proposed openstack/python-novaclient master: Make --profile load from environment variables https://review.openstack.org/475251 | |
| 07:38:26 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276 | |
| 07:45:37 | artom | I'd like to depend on https://review.openstack.org/#/c/470774/ | |
| 07:45:46 | artom | But I don't want to push a new patchset to that change | |
| 08:09:49 | gibi | artom: this is the way how I would do it. Checkout 470774, then cherry-pick your change, that wil make your change top of 470774 | |
| 08:10:10 | gibi | you can do that via gerrit ui if you wish | |
| 08:10:32 | gibi | in your change select rebase and copy the hash of 470774 to the test box | |
| 08:11:33 | artom | gibi, won't that push a rebase of the dependant change when I git review? | |
| 08:11:46 | artom | And snikitin isn't around, so I can't warn him either | |
| 08:13:24 | gibi | if you push rebase on your change then only your change will have a new patch set | |
| 08:14:02 | artom | gibi, thanks :) Alright, here it goes | |
| 08:18:15 | artom | Woo! | |
| 08:18:22 | artom | Thanks again gibi | |
| 08:19:03 | gibi | :) | |
| 08:20:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644 | |
| 08:23:07 | artom | Now I need to be careful with any future rebases and make sure I keep the same commit id | |
| 08:23:14 | artom | (Of 470774) | |
| 08:23:29 | artom | Unless Sergei uploads a new version, obviously | |
| 08:25:54 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | |
| 08:37:23 | gibi | artom: yes, if your patch needs a rebase then you have to rebase the whole chain which effect Sergei's work | |
| 09:03:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying https://review.openstack.org/345396 | |
| 09:03:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 09:03:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 09:03:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 09:05:15 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | |