| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-19 | |||
| 21:34:53 | mnaser | mriedem: what object should it be? | |
| 21:35:05 | mriedem | mnaser: nvm, i guess it doesn't recreate it | |
| 21:35:40 | mriedem | for one thing the test wasn't going through get_stashed_volume_connector when deleting the server, | |
| 21:35:48 | mriedem | and _local_cleanup_bdm_volumes swallows the error anyway | |
| 21:36:04 | mriedem | but if the bdm.connection_info were 'null' the test would fail before that anyway | |
| 21:36:08 | mriedem | so i guess i didn't recreate it | |
| 21:39:15 | efried | I think that newton midcycle was my first in-person encounter with you people. | |
| 21:39:39 | mriedem | "you people" | |
| 21:40:32 | sean-k-mooney | efried: does it feel longer or shorter then that :) | |
| 21:41:06 | efried | With my dying breath I shall continue to deny that "you people" is an acceptable PC violation, under any circumstances. | |
| 21:41:07 | Nick_A | config drive - is there a way to only use it at instance creation and not let instance mount it again after? | |
| 21:42:02 | efried | sean-k-mooney: I thought I started somewhere around liberty/mitaka, but that may have been way in the background, no community involvement at that point. | |
| 21:42:14 | efried | sean-k-mooney: I do remember understanding ZERO of what went on in that room though. | |
| 21:42:21 | efried | (Some things never change) | |
| 21:43:59 | sean-k-mooney | i think that is common. both initally not following the supper detailed conversation and not getting involved in the comunity right away | |
| 21:46:47 | sean-k-mooney | i stared playing with openstack at teh end fo havana playing with quantum but it was after icehouse had shipped in early juno that i submitted my first patch i think | |
| 21:48:12 | sean-k-mooney | huh my first patch was a neutron spec apparently | |
| 21:48:13 | sean-k-mooney | https://review.opendev.org/#/c/95121/ | |
| 21:53:45 | mriedem | seems pypi mirrors have exploded, | |
| 21:53:54 | mriedem | so for anyone waiting all day for a CI result, they likely have to recheck | |
| 22:10:34 | sean-k-mooney | that proably explains why my recheck early never started | |
| 23:56:25 | alex_xu | sean-k-mooney: thanks, I'm ok with default to not copy the data | |
| 23:57:31 | sean-k-mooney | i think in the context of cross cell resize it nice that the default beahvior would be the same as intra cell resize | |
| 23:57:52 | sean-k-mooney | and its also consistent with snapshotting and shelve | |
| #openstack-nova - 2019-06-20 | |||
| 00:21:38 | openstackgerrit | sean mooney proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 00:24:32 | openstackgerrit | sean mooney proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 00:38:05 | openstackgerrit | sean mooney proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/644881 | |
| 00:39:49 | sean-k-mooney | not with spell checking ^ :) | |
| 00:40:00 | sean-k-mooney | or rather grammerly checking | |
| 01:07:17 | openstackgerrit | sean mooney proposed openstack/os-traits master: add libvirt image metadata traits https://review.opendev.org/652996 | |
| 01:11:08 | openstackgerrit | sean mooney proposed openstack/os-traits master: add GOP, NONE and VIRTIO grapics models https://review.opendev.org/665830 | |
| 02:02:07 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove deprecated arguments in db sync command https://review.opendev.org/666470 | |
| 06:11:43 | openstackgerrit | mashuting proposed openstack/nova master: VMware: Support volumes backed by VStorageObject https://review.opendev.org/521690 | |
| 08:31:35 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Grab fresh power state info from the driver https://review.opendev.org/665975 | |
| 08:40:17 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Grab fresh power state info from the driver https://review.opendev.org/665975 | |
| 08:41:24 | stephenfin | efried: When do you _sleep_? | |
| 08:41:47 | stephenfin | :) | |
| 08:59:53 | moshele | sean-k-mooney | |
| 09:00:02 | moshele | sean-k-mooney: hi | |
| 09:02:13 | ohwhyosa | Morning guys and gals! | |
| 09:15:20 | kashyap | jroll: Just saw the ping in scrollback; I don't know the answer to where the keys are stored. I'll inquire and let you know. | |
| 09:22:39 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove global state from the FakeDriver https://review.opendev.org/656709 | |
| 09:23:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552 | |
| 09:23:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add functional test coverage for bug 1724172 https://review.opendev.org/512553 | |
| 09:23:28 | openstack | bug 1724172 in OpenStack Compute (nova) rocky "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,Confirmed] https://launchpad.net/bugs/1724172 | |
| 09:38:14 | openstackgerrit | guangchao.pan proposed openstack/nova master: Add a name for the automatically created volume https://review.opendev.org/660096 | |
| 10:16:26 | mgoddard | Hello novatons, I have a question about upgrades using [upgrade_levels]compute=auto. Is there a programatic way to know when all computes are running the new RPC version, so that I can restart services to remove the version cap? | |
| 10:16:33 | mgoddard | johnthetubaguy: ^ | |
| 10:17:14 | mgoddard | I'm looking for something like 'nova-manage wait for computes [--all-cells]' | |
| 10:17:18 | johnthetubaguy | there is the services API, does that not return you the version? | |
| 10:17:22 | mgoddard | nope | |
| 10:17:46 | mgoddard | the only potentially useful thing there is updated_at, but that's a bit kludgey | |
| 10:17:48 | johnthetubaguy | checking if its in a new microversion | |
| 10:17:56 | mgoddard | ah | |
| 10:18:15 | mgoddard | I'd quite like to not need to care what the latest version is | |
| 10:19:09 | johnthetubaguy | so if you do controllers upgrade, then wait for them all to be the same | |
| 10:19:12 | johnthetubaguy | its quite close | |
| 10:19:17 | johnthetubaguy | but I get your point | |
| 10:19:39 | mgoddard | service_version = service_obj.get_minimum_version_all_cells( | |
| 10:19:42 | mgoddard | context.get_admin_context(), ['nova-compute']) | |
| 10:19:58 | mgoddard | I'd quite like a command that does that, compares with the latest version and says yes/no | |
| 10:20:13 | johnthetubaguy | yeah, that would be a good nova-manage cmd | |
| 10:20:15 | mgoddard | or even waits with a timeout | |
| 10:20:43 | mgoddard | I'm leaning towards short sleep + nova RFE at this point :) | |
| 10:21:29 | johnthetubaguy | RFE? what is that | |
| 10:22:30 | mgoddard | blueprint | |
| 10:22:37 | mgoddard | :p | |
| 10:22:49 | johnthetubaguy | yeah, I am wondering about nova status as a place for this kind of thing | |
| 10:22:56 | johnthetubaguy | "is every upgraded" check, makes sense | |
| 10:23:06 | mgoddard | yeah | |
| 10:24:57 | johnthetubaguy | i am not seeing anything useful just yet | |
| 10:27:25 | johnthetubaguy | I am sure we used to have a nova-manage cmd for this, I guess we killed it | |
| 10:27:49 | mgoddard | that's a shame | |
| 10:30:25 | johnthetubaguy | https://github.com/openstack/nova/blob/fc8cd8f3b346c8f53e2dfc8e3de9fdcaedb0d35d/nova/cmd/manage.py#L613 | |
| 10:30:38 | johnthetubaguy | although doesn't seem like that had the version either | |
| 10:32:52 | mgoddard | that was just pushed into the API wasn't it? | |
| 10:34:00 | mgoddard | if the version was available in the service API that would be fine, but ideally I'd want a way to discover what the version should be | |
| 10:34:35 | mgoddard | Looks like it takes 10+ seconds to update the service version in an AIO setup | |
| 10:36:11 | johnthetubaguy | there is a reporting delay config | |
| 10:36:33 | johnthetubaguy | yeah, so I can't find anything useful in the API or nova-manage, need to add a new thing | |
| 10:36:59 | johnthetubaguy | I mean SIGHUP should be enough, but not sure how we do that via docker? | |
| 10:37:39 | mgoddard | no SIGHUP, it's busted. Need to restart | |
| 10:37:59 | mgoddard | we do that already in kolla, the problem is we're not waiting long enough to actually uncap | |
| 10:38:19 | johnthetubaguy | ah, right | |
| 10:39:11 | johnthetubaguy | https://github.com/openstack/nova/blob/74aebe0d4e5a978a40011e890aee9e70e98246c4/nova/servicegroup/api.py#L35 | |
| 10:39:13 | mgoddard | so I think my band aid is 'sleep ${nova_compute_startup_delay:-30}' | |
| 10:39:25 | johnthetubaguy | so we do wait 5 seconds before reporting | |
| 10:39:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Ignore hw_vif_type for direct, direct-physical vNIC types https://review.opendev.org/609460 | |
| 10:39:51 | johnthetubaguy | that is once its called, and in a timer that might trigger at somepoint after that | |
| 10:40:26 | mgoddard | although I think the thing I'm waiting for happens here: https://github.com/openstack/nova/blob/74aebe0d4e5a978a40011e890aee9e70e98246c4/nova/service.py#L77 | |
| 10:41:20 | johnthetubaguy | init_host is quite expensive | |
| 10:41:33 | johnthetubaguy | it might be just the cost of waiting for that to finish | |
| 10:41:50 | johnthetubaguy | but yeah, its not affected by what I was pointing at | |
| 10:43:33 | mgoddard | would you call this a bug or blueprint? | |
| 10:44:27 | johnthetubaguy | its either really, the release notes are independent these days, so it matters little | |
| 10:44:32 | johnthetubaguy | (to me) | |
| 10:45:19 | johnthetubaguy | the API change would be a spec, but that is a separate issue, a quick fix to nova-manage to list all the services not currently at the latest version would be a simple enough patch | |
| 10:46:20 | johnthetubaguy | you have to target a cell with hosts in it, so maybe its a cell command? | |
| 10:46:47 | johnthetubaguy | get_non_upgrade_cell_computes() or ideally some less rubbish name | |
| 10:48:26 | mgoddard | so there are two cases. API, scheduler & super conductor need to wait for all computes. Cell conductor & computes need to wait just for the cell | |