| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-13 | |||
| 10:54:55 | sfinucan | pkoniszewski: https://github.com/openstack/nova/commit/df334b4f414b8c341709df37ca10065d3b50fcef for reference | |
| 10:56:36 | pkoniszewski | sfinucan: so my point was to deprecate and finally remove live_migration_uri which is used for hacking the interface used for live migrations, now there is live_migration_inbound_addr to choose an interface | |
| 10:56:45 | pkoniszewski | also rationale behind df334b4 was to keep an option to change scheme for live migration, some of operators use SSH instead of TCP | |
| 10:56:57 | pkoniszewski | i didn't want to force anyone to switch to TCP from SSH | |
| 10:57:48 | sfinucan | pkoniszewski: Sounds fair. Unfortunately though some operators also use extraparamters, e.g. '?command=xyz' https://bugs.launchpad.net/nova/+bug/1671288 | |
| 10:57:48 | openstack | Launchpad bug 1671288 in OpenStack Compute (nova) "live_migration_uri to live_migration_scheme SSH settings" [Undecided,Confirmed] - Assigned to Stephen Finucane (stephenfinucane) | |
| 10:58:20 | pkoniszewski | yeah, i noticed that like two weeks ago | |
| 10:58:27 | pkoniszewski | so it's a trade off right now | |
| 10:58:57 | sfinucan | pkoniszewski: trade-off between...? :) | |
| 11:00:42 | sfinucan | pkoniszewski: I was going to add a 'live_migration_extraparamters' DictOpt to allow folks to specify those options if they want them | |
| 11:00:48 | pkoniszewski | keep live_migration_uri and document how it can be used/let people google how to use it, or implement a few configuration options to cover operators use cases (like SSH username and key) and remove live_migration_uri entirely | |
| 11:01:02 | pkoniszewski | or a DictOpt, it also sounds good | |
| 11:01:08 | sfinucan | ...but then I though, wouldn't it be easier to keep live_migration_uri and mark it as 'advanced' | |
| 11:01:16 | sfinucan | https://docs.openstack.org/developer/oslo.config/opts.html#oslo_config.cfg.Opt | |
| 11:01:40 | sfinucan | pkoniszewski: seems we're weighing up similar trade-offs so :) | |
| 11:03:20 | johnthetubaguy | sfinucan: pkoniszewski: I was thinking about that before as well | |
| 11:03:35 | johnthetubaguy | honestly, folks seem used to using the URL, maybe we just keep it | |
| 11:03:50 | johnthetubaguy | and add a description about how you change SSH params in there | |
| 11:04:16 | sfinucan | right, I'll revert the deprecation so, mark it as advanced, and expand the help with a link to the libvirt docs | |
| 11:04:29 | sfinucan | that'll keep both standard and "advanced" operators happy | |
| 11:04:47 | sfinucan | with minimal code changes (read: easy backporting) | |
| 11:04:48 | johnthetubaguy | I am tempted to include the SSH case in the description | |
| 11:05:00 | sfinucan | johnthetubaguy: e.g.? | |
| 11:05:03 | johnthetubaguy | makes sense to have the same key for migration and livemigration | |
| 11:05:13 | johnthetubaguy | sfinucan: it was in one of the bugs somewhere | |
| 11:05:49 | johnthetubaguy | sfinucan: here we go live_migration_uri = qemu+ssh://nova@%s/system?keyfile=/var/lib/nova/.ssh/id_rsa&no_tty=1&no_verify=1 | |
| 11:10:37 | johnthetubaguy | sfinucan: yeah, maybe leave those stranger bits off the example | |
| 11:26:43 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 11:27:53 | voelzmo | Hey there | |
| 11:28:18 | voelzmo | A while back, there was some blueprint about updateable metadata for servers. Is that still a thing that's being pursued? | |
| 11:31:29 | voelzmo | s/metadata/userdata/ | |
| 11:31:36 | voelzmo | I guess this might be the one https://blueprints.launchpad.net/nova/+spec/userdata-modification | |
| 11:32:11 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 11:35:43 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Don't deprecate the 'live_migration_uri' https://review.openstack.org/456571 | |
| 11:35:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | |
| 11:35:44 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | |
| 11:35:57 | sfinucan | johnthetubaguy, pkoniszewski, owalsh: ^ | |
| 11:43:56 | openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | |
| 11:45:35 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 11:50:45 | EmilienM | hey folks, this patch is really critical for TripleO CI to run correctly again: https://review.openstack.org/#/c/456227/ | |
| 11:51:04 | EmilienM | https://bugs.launchpad.net/tripleo/+bug/1682001 | |
| 11:51:04 | openstack | Launchpad bug 1682001 in tripleo "[promotion] nova hypervisors/detail fails when host is not mapped to any cell" [Critical,In progress] - Assigned to Oliver Walsh (owalsh) | |
| 11:57:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-extraspecs endpoint instead of stevedore https://review.openstack.org/456585 | |
| 12:07:51 | openstackgerrit | NidhiMittalHada proposed openstack/python-novaclient master: Nova volume-attach doesn't care for device name https://review.openstack.org/454048 | |
| 12:09:05 | peter-hamilton | johnthetubaguy: if you have any further questions on the cert validation spec just ping me | |
| 12:09:43 | johnthetubaguy | peter-hamilton: cool, I should hit that one now | |
| 12:14:22 | johnthetubaguy | peter-hamilton: do we want to return a 400 error when a user gives us a list of certs, but our cert checking is turned off? | |
| 12:16:23 | peter-hamilton | johnthetubaguy: good question; does it make sense to throw a warning and store the certs with the instance anyway? | |
| 12:17:00 | johnthetubaguy | peter-hamilton: I am bit worried the user would assume the cloud has done the verification, but it hasn't actually done it | |
| 12:17:04 | peter-hamilton | johnthetubaguy: that way cert validation could be enabled at a later date and boot would still work? | |
| 12:17:42 | johnthetubaguy | although we do allow folks to specify the cert list when the image doesn't require verification anyways | |
| 12:17:52 | johnthetubaguy | its probably more consistent to just log a warning | |
| 12:18:05 | johnthetubaguy | I mean the user doesn't see that I guess | |
| 12:18:08 | peter-hamilton | johnthetubaguy: i could go either way i think | |
| 12:18:16 | johnthetubaguy | peter-hamilton: honestly, me too | |
| 12:18:43 | johnthetubaguy | wonders which side of the fence to sit | |
| 12:18:54 | peter-hamilton | johnthetubaguy: should we hash this out during implementation? | |
| 12:19:35 | johnthetubaguy | peter-hamilton: thats probably the right call | |
| 12:20:17 | peter-hamilton | johnthetubaguy: i'll make a note of it for when i get to the API patches | |
| 12:26:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for flavors-access endpoint instead of stevedore https://review.openstack.org/456589 | |
| 12:31:43 | openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456593 | |
| 12:43:59 | dane-fichter | johnthetubaguy: Can you take a look at peter's spec when you get a chance? https://review.openstack.org/#/c/357151/ | |
| 12:44:12 | peter-hamilton | dane-fichter: he's already on it :) | |
| 12:44:18 | johnthetubaguy | dane-fichter: yeah, have thinking through that now :) | |
| 12:44:25 | dane-fichter | oh lol sorry | |
| 12:44:39 | johnthetubaguy | no problem, its the right day to chase me down :) | |
| 12:44:53 | openstackgerrit | Bryan Song proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 12:47:14 | peter-hamilton | johnthetubaguy: awesome, thanks for the +2! | |
| 12:48:06 | johnthetubaguy | peter-hamilton: no problem, glad we are getting there, its a good feature folks need! | |
| 12:48:18 | peter-hamilton | johnthetubaguy: i agree :) | |
| 12:48:27 | peter-hamilton | johnthetubaguy: now, where is jaypipes... | |
| 12:49:12 | johnthetubaguy | hopefully he is having a coffee, putting on his flip flops, and about to open his laptop | |
| 12:57:40 | openstackgerrit | Yasufumi Ogawa proposed openstack/nova-specs master: Merge "Libvirt options for DPDK" https://review.openstack.org/456601 | |
| 13:01:04 | johnthetubaguy | sean-k-mooney: sfinucan: any chance we could get use cases added in here quickly? https://review.openstack.org/#/c/361140 | |
| 13:01:37 | sfinucan | johnthetubaguy: Sure. Gimme 10 | |
| 13:01:37 | johnthetubaguy | sean-k-mooney: sfinucan: something to describe the scenario(s) where each type of behaviour is what you want | |
| 13:02:12 | johnthetubaguy | sfinucan: sweet, I have got all confused after re-reading them, particular around the good points cdent raised in there | |
| 13:06:33 | sfinucan | johnthetubaguy: Dumb question, but is there another name of the service that boots a VM other than 'nova-compute'? | |
| 13:07:13 | johnthetubaguy | sfinucan: hmm, duno really, n-cpu? | |
| 13:07:19 | sfinucan | i.e. we'd say "the scheduler" rather than 'nova-scheduler' | |
| 13:07:30 | johnthetubaguy | "the compute node" gets used a bit | |
| 13:07:37 | johnthetubaguy | or host | |
| 13:07:51 | johnthetubaguy | or just "in compute" | |
| 13:08:03 | johnthetubaguy | not sure thats a good thing though | |
| 13:09:01 | sfinucan | maybe I should just use nova-compute and nova-scheduler consistently instead | |
| 13:14:17 | johnthetubaguy | sfinucan: I guess thats always clear | |
| 13:17:12 | bauzas | sfinucan: if you talk about a running service having a RPC MQ, it's nova-compute | |
| 13:17:25 | bauzas | sfinucan: if you talk about an hypervisor, it's a node | |
| 13:37:04 | peter-hamilton | jaypipes: if you have any questions on cert validation just ping me | |
| 13:37:28 | jaypipes | peter-hamilton: will do. was on PTO yesterday. catching up today. | |
| 13:38:55 | mriedem | johnthetubaguy: as you probably saw i marked up your remove scope check patch, let me know if you want to talk about it | |
| 13:39:32 | johnthetubaguy | mriedem: so I didn't see that, looking now | |
| 13:39:41 | mriedem | you might want to sit down | |
| 13:40:02 | johnthetubaguy | funny you say that, I just adjusted my desk to sit down mode | |
| 13:45:07 | artom | For tagged attach, I need to invalidate the metadata API's cache - however, as far as I can tell, its cache is just a dict local to that process. | |
| 13:45:42 | artom | I'm trying to thinking of a good way to solve that, without suddenly requiring all deployment to run memcache.' | |
| 13:46:07 | bauzas | jaypipes: in case you have some brain time free, I'd love to get your insights on https://review.openstack.org/#/c/450122/15 | |
| 13:46:53 | bauzas | jaypipes: my biggest concern is how a virt driver can return a single dict for a common RC if it has two distinct devices behind it | |
| 13:47:04 | bauzas | unless it's not really a problem and then meh. | |
| 13:48:04 | mriedem | artom: i wonder if we could set a flag in the db to signal when the cache should be dumped | |