| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-19 | |||
| 20:08:16 | dansmith | mriedem: right, but isn't that the expectation of the api user? | |
| 20:08:17 | melwitt | but how does that hurt anything? if we never delete instance mappings? | |
| 20:08:24 | dansmith | or is this only in the multi-create case? | |
| 20:08:52 | melwitt | *if we never delete instance mappings anyway? | |
| 20:09:07 | dansmith | for single create, once you get back a uuid you expect to be able to list that thing (with deleted=yes) until archive | |
| 20:09:24 | dansmith | but I guess maybe if you've deleted it it could be up for immediate archive and thus deleting the mapping is equivalent | |
| 20:09:26 | dansmith | so yeah, either I guess | |
| 20:09:42 | mriedem | melwitt: it could be a problem in a tight window if you got that mapping via build request as a marker when listing/paging | |
| 20:09:54 | melwitt | oh, I see | |
| 20:10:20 | mriedem | dansmith: if you delete the build request before it's an instance, i don't think we honor reading it as deleted=yes anyway, | |
| 20:10:24 | mriedem | because BRs are hard deleted | |
| 20:10:50 | mriedem | melwitt: at least that was part of the reasoning behind https://review.opendev.org/#/c/575556/ | |
| 20:11:13 | dansmith | mriedem: ack yeah I guess I remember that bit of cheating | |
| 20:11:51 | mriedem | it would have been like newton-era talk about that cheating being ok, likely in a small hotel conf room in hillsboro | |
| 20:11:53 | mriedem | if i remember correctly | |
| 20:12:00 | mriedem | with jpenick squashed on the floor in a corner | |
| 20:12:07 | melwitt | lol | |
| 20:12:27 | dansmith | yar | |
| 20:12:44 | mriedem | the good old days... | |
| 20:12:54 | melwitt | it was wharm in there | |
| 20:12:56 | mriedem | anyway, i'm about to abandon https://review.opendev.org/#/c/575556/ but was just looking at my last comment there | |
| 20:13:11 | mriedem | melwitt: unless you were directly underneath where the air shot out and down on you, which was my case | |
| 20:13:18 | mriedem | then it was cold and contact-drying | |
| 20:13:25 | melwitt | lol! | |
| 20:13:30 | melwitt | well I'll be | |
| 20:32:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete InstanceMapping in conductor if BuildRequest is already deleted https://review.opendev.org/666438 | |
| 20:35:29 | openstackgerrit | melanie witt proposed openstack/nova master: rbd: use MAX_AVAIL stat for reporting bytes available https://review.opendev.org/556692 | |
| 20:40:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate https://review.opendev.org/571265 | |
| 21:00:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: don't log error if guest gone during interface detach https://review.opendev.org/610727 | |
| 21:04:31 | mriedem | relatively simple older patch here that has a +2 on it - it's mostly docstring https://review.opendev.org/#/c/633212/ | |
| 21:28:27 | sean-k-mooney | mriedem: just back after dinner ill take a look at https://review.opendev.org/#/c/644881/ now | |
| 21:31:13 | mriedem | i'll be gone in 30 minutes | |
| 21:31:56 | sean-k-mooney | mriedem: the use of hybrid plug is based on the local config on the compute host which is read by the agent and hybrid plug amoung other things is stored in the neutron db | |
| 21:32:15 | sean-k-mooney | so when the ml2 driver bind the port to the hsot is pass an agent context like our host_state object | |
| 21:32:19 | mriedem | mnaser: figured out why my recreate test in https://review.opendev.org/#/c/647603/ wasn't failing | |
| 21:32:26 | sean-k-mooney | which it check to determin if hybid plug should be used | |
| 21:32:31 | mriedem | mnaser: only took a couple of months | |
| 21:32:46 | mriedem | sean-k-mooney: but that's per neutron agent right? | |
| 21:32:53 | sean-k-mooney | yes | |
| 21:33:03 | mriedem | the commit message makes it sound like if you're ovs hybrid plug, it's deployment-wide | |
| 21:33:23 | sean-k-mooney | it does. ill comment on the comment and correct it | |
| 21:33:46 | sean-k-mooney | for tools like triplo it typeiclay configred globally | |
| 21:33:54 | sean-k-mooney | but it actully set per host | |
| 21:34:49 | sean-k-mooney | at bind time the ml2 driver is passed the agent context for the agent on the host that nova selected and it determins what to do based on the configurtion of that host | |
| 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 | |