| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-20 | |||
| 21:14:21 | efried | melwitt: https://review.openstack.org/#/c/490057/ | |
| 21:14:23 | melwitt | it's maybe not "as noticeable" since it's likely only this sync power states periodic that results in notifications maybe | |
| 21:14:54 | melwitt | efried: okay, cool. | |
| 21:15:21 | melwitt | so only need to backport to queens | |
| 21:15:27 | efried | melwitt: And here's the change that removed api_servers from devstack: https://review.openstack.org/#/c/490031/ | |
| 21:16:12 | efried | melwitt: Well, I'll be a little surprised if that's the only place we're using a tokenless auth context to look up a glance endpoint. | |
| 21:16:20 | efried | given the scope of 490057 | |
| 21:19:43 | melwitt | efried: yeah ... probably. but since it's so recent, I think you're right this is being caught a lot sooner than most people upgrade to queens. I was worried it was going to go back to ocata or something crazy. I know you linked the patch earlier but I didn't notice it was pretty recent | |
| 21:22:14 | melwitt | mriedem: what I was thinking with the nullable thing, would be in the worst case if someone had automation parsing image_ref_url and we make it not a url, but empty string isn't one either. not sure if they're equally bad. just thinking out loud | |
| 21:22:40 | mriedem | if someone blindly takes the image_ref_url and makes a GET curl request with it, it's going to blow up for all volume-backed instances | |
| 21:23:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle EndpointNotFound when building image_ref_url in notifications https://review.openstack.org/554703 | |
| 21:23:14 | mriedem | i think in the long ago, you could also create servers by passing an image URL to the imageRef parameter | |
| 21:23:15 | melwitt | yeah. I was thinking before that as far as the format of it. probably overthinking it | |
| 21:23:16 | mriedem | and nova would parse it | |
| 21:29:15 | mriedem | multiattach is fixed with the queens UCA | |
| 21:29:19 | mriedem | https://review.openstack.org/#/c/554667/ | |
| 21:29:30 | mriedem | that's the only thing blocking us from using the queens UCA in devstack now | |
| 21:30:08 | imacdonn | ooh | |
| 21:31:01 | imacdonn | is there any actually need to use the UCA? ZFSSA CI seems to be doing OK with out .. but that's cinder | |
| 21:31:32 | mriedem | well, a few things, | |
| 21:31:37 | mriedem | we get newer libvirt and qemu, | |
| 21:31:51 | mriedem | testing rocky against pike UCA seems weird, we should at least use queens if it's there | |
| 21:32:13 | mriedem | and it allows us to remove these weird workarounds in devstack for the multiattach job to *not* use the UCA b/c the pike UCA didn't have the right package versions, but queens UCA does https://review.openstack.org/#/c/554317/ | |
| 21:32:19 | mriedem | https://review.openstack.org/#/c/554314/ | |
| 21:32:39 | imacdonn | yeah, I have one of those workarounds (per your recommendation) | |
| 21:32:51 | imacdonn | I suppose it does make sense to test nova stuff with the latest virt stuff | |
| 21:33:09 | mriedem | s/nova/openstack/ | |
| 21:33:36 | imacdonn | seems it'd matter more for nova... but yeah | |
| 21:33:54 | mriedem | sure, for libvirt and qemu yes, but you also get newer things like rados and tgtd | |
| 21:33:57 | mriedem | ovs | |
| 21:33:58 | mriedem | etc | |
| 21:35:34 | imacdonn | OK. Has devstack master already been updated to use the Queens UCA? | |
| 21:35:45 | mriedem | https://review.openstack.org/#/c/554314/ | |
| 21:35:52 | mriedem | depends on fixing this bug in nova first | |
| 21:36:06 | imacdonn | k... I'll subscribe to the bug | |
| 21:36:20 | mriedem | get stvnoyes on the oracle phone | |
| 21:37:07 | imacdonn | not sure if he's still in EU, but even if not, he's probably gone for the day (east coast) | |
| 21:37:17 | melwitt | mriedem: +2 | |
| 21:37:51 | mriedem | thanks | |
| 21:39:08 | openstackgerrit | Merged openstack/nova stable/queens: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553817 | |
| 21:39:21 | imacdonn | Something completely unrelated to bounce off you, mriedem (or anyone else) | |
| 21:40:11 | imacdonn | per Queens release notes, I tried to remove neutron.url from my nova.conf, but it seems to be unable to get the neutron endpoint from the service catalog | |
| 21:40:28 | mriedem | efried: replied to all comments in https://review.openstack.org/#/c/554703/ and i don't think any of them are worth changing | |
| 21:40:31 | imacdonn | I had someone in #openstack buddy-check my config, and it seems sane | |
| 21:40:45 | mriedem | imacdonn: there is a bug fix that you need, | |
| 21:40:46 | mriedem | sec | |
| 21:40:55 | imacdonn | k ;) | |
| 21:40:59 | mriedem | imacdonn: https://github.com/openstack/nova/commit/3a3b0f09db318faf1a1ea711a73bb365cab8b233 | |
| 21:41:56 | imacdonn | mriedem: Interesting, Looks pertinent. Will try it. Thanks! | |
| 21:44:32 | efried | mriedem: Brain fart, sorry 'bout that. +1. | |
| 21:45:13 | mriedem | efried: np, thanks for the quick review | |
| 21:47:36 | mriedem | hongbin: yikun: Kevin_Zheng: easy bug https://bugs.launchpad.net/nova/+bug/1757273 | |
| 21:47:38 | openstack | Launchpad bug 1757273 in OpenStack Compute (nova) "nova-compute fails to start even if [placement]/region_name is set" [Medium,Triaged] | |
| 21:53:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove RequestContext.instance_lock_checked https://review.openstack.org/554378 | |
| 22:29:37 | mriedem | i wonder if anyone that uses DVR also uses shared ephemeral storage (rbd imagebackend) and live migration (HP cloud anyone?) | |
| 22:29:53 | mriedem | because i'm pretty sure we don't cleanup on failed live migration properly in that case | |
| 22:30:26 | mriedem | https://github.com/openstack/nova/blob/3fd863d8bf2fa1fc09acd08d976689462cffd2e3/nova/compute/manager.py#L6506 will also cleanup some stuff we put in the port's binding profile for DVR during live migration, | |
| 22:30:42 | mriedem | but if do_cleanup is False, which it is if you're using shared ephemeral storage, then we don't clean that up | |
| 22:40:59 | efried | mriedem: Are you talking specifically libvirt? | |
| 22:41:35 | efried | Cause that sounds like something we either support already or will support shortly in PowerVM (out of tree). | |
| 22:41:58 | mriedem | for shared local disk? | |
| 22:42:14 | mriedem | https://github.com/openstack/nova/blob/3fd863d8bf2fa1fc09acd08d976689462cffd2e3/nova/compute/manager.py#L6185 | |
| 22:42:39 | mriedem | looks like that is also checked for xen and hyperv | |
| 22:43:33 | mriedem | anyway, https://github.com/openstack/nova/blob/3fd863d8bf2fa1fc09acd08d976689462cffd2e3/nova/compute/manager.py#L6524 is totally doing more than just cleaning up local disk created on the dest host for non-shared storage | |
| 22:43:39 | mriedem | it's also cleaning up network stuff on the dest host | |
| 22:45:09 | mriedem | added back in mitaka https://review.openstack.org/#/c/227897/ | |
| 22:45:18 | mriedem | oh nvm, that was the error handling | |
| 22:47:24 | mriedem | wow added long ago https://review.openstack.org/#/c/4646/ | |
| 22:47:35 | mriedem | essex | |
| 22:48:20 | efried | Gotta run | |
| 23:27:11 | openstackgerrit | Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | |
| 23:27:12 | openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241 | |
| 23:27:14 | openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242 | |
| 23:27:14 | openstackgerrit | Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078 | |
| 23:27:15 | openstackgerrit | Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | |
| 23:27:15 | openstackgerrit | Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605 | |
| 23:27:23 | openstackgerrit | Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438 | |
| 23:27:25 | openstackgerrit | Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | |
| 23:59:36 | openstackgerrit | melanie witt proposed openstack/nova master: Remove useless run_periodic_tasks call in ClientRouter https://review.openstack.org/554381 | |
| #openstack-nova - 2018-03-21 | |||
| 00:23:38 | mriedem | alex_xu_: gmann_: there is a question in here about API validation outside of the json schema, would be nice to get your input https://review.openstack.org/#/c/546925/ | |
| 00:49:31 | gmann_ | mriedem: sure, ll check | |
| 01:08:31 | Kevin_Zheng | mriedem got it | |
| 01:38:17 | mriedem | johnthetubaguy: thanks for the review on https://review.openstack.org/#/c/520248/ - i didn't see the -1 until now; as for the script thing where someone starts using an IP before the server is ready, we don't show the server address while it's building | |
| 01:38:54 | mriedem | because of this fun guy https://github.com/openstack/nova/blob/3fd863d8bf2fa1fc09acd08d976689462cffd2e3/nova/conf/api.py#L264 | |
| 01:39:40 | mriedem | gmann_: btw, i think you can remove that option now, it's been over 3 months | |
| 01:39:42 | mriedem | ^ | |
| 01:47:57 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add [placement]/region_name to compute manager placement config check https://review.openstack.org/554759 | |
| 01:56:42 | yikun | mriedem, sorry, I misunderstand your means to have to use the schema to validate per-policy rule. : ) | |
| 01:56:43 | yikun | so, you mean it's just a code validation after schema validation, right? | |
| 01:57:11 | mriedem | yikun: yes, since we can't define a mapping in the json schema document | |
| 01:57:26 | mriedem | not that i know of anyway, without some custom schema code | |
| 01:59:21 | yikun | ok, got it. :) | |
| 02:06:31 | mriedem | but alex_xu_ might have other ideas | |
| 02:06:40 | mriedem | anyway, time for me to stop working, ttyl | |
| 02:39:14 | openstackgerrit | Julia Kreger proposed openstack/nova master: WIP: Add microversion to ironic client wrapper call https://review.openstack.org/554762 | |
| 02:48:56 | vivsoni_ | if we have two compute node, compute1 & compute2 | |
| 02:49:13 | vivsoni_ | and if we create nova instance1 from compute1 | |
| 02:49:31 | vivsoni_ | then is that instance1 visible from compute2 as well | |
| 02:49:32 | vivsoni_ | ? | |
| 02:50:16 | vivsoni_ | i mean when i execute 'nova list' command from compute2, will that instance1 will be listed ? | |