| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-26 | |||
| 07:34:02 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 07:34:18 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Specify keymap on server boot https://review.openstack.org/448830 | |
| 07:39:24 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050 | |
| 07:39:29 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | |
| 07:42:28 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | |
| 07:44:39 | openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-interface endpoint instead of stevedore https://review.openstack.org/459979 | |
| 07:50:33 | openstackgerrit | fengzhr proposed openstack/nova master: Modify the description of flat_injected in nova.conf https://review.openstack.org/458789 | |
| 08:49:14 | lyarwood | sfinucan: morning, would you mind taking a look at https://review.openstack.org/#/c/458807/ this morning? It's the actual bugfix in the series you started reviewing yesterday. | |
| 09:08:49 | asettle | Ah, my second home, the nova channel.... | |
| 09:09:24 | asettle | bauzas: we've got a fix for those placement vs. port numbers https://review.openstack.org/#/c/459903/ if you have time to look | |
| 09:09:27 | asettle | I can't see cdent in yet | |
| 09:11:09 | openstackgerrit | John Garbutt proposed openstack/nova master: Remove all discoverable policy rules https://review.openstack.org/459608 | |
| 09:11:36 | johnthetubaguy | asettle: he was up late with TC stuff | |
| 09:11:57 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791 | |
| 09:12:49 | asettle | johnthetubaguy: ahuh! makes sense :) | |
| 09:13:35 | johnthetubaguy | I followed through on my election promise of not attending the meeting... I was in a pub talking about openstack, so didn't make the best use of my time as such | |
| 09:14:51 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Check the instance task status to release the memory quota for vram https://review.openstack.org/456050 | |
| 09:17:22 | asettle | johnthetubaguy: at least you tried :P | |
| 09:17:23 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 09:19:30 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 09:19:42 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801 | |
| 09:20:38 | openstackgerrit | Ritesh proposed openstack/nova master: rbd: flatten images when unshelving https://review.openstack.org/457886 | |
| 09:20:55 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | |
| 09:22:57 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791 | |
| 09:24:31 | bauzas | asettle: morning, sorry was in $meeting | |
| 09:25:03 | asettle | How dare you work! | |
| 09:25:05 | asettle | I have patches! | |
| 09:25:07 | asettle | :p | |
| 09:25:27 | bauzas | asettle: you know that port 8778 clashes with some other openstack project? | |
| 09:25:37 | bauzas | in devstack I mean | |
| 09:25:48 | asettle | bauzas: I did not. Which I guess is why I'm asking you guys to review :/ | |
| 09:25:51 | asettle | Oh no. What does it clash with? | |
| 09:26:01 | bauzas | asettle: sec, giving you the ML thread | |
| 09:26:29 | bauzas | asettle: long thread, enjoy the read http://lists.openstack.org/pipermail/openstack-dev/2016-December/109219.html | |
| 09:26:41 | bauzas | asettle: tl;dr: senlin was gating on this port | |
| 09:26:47 | asettle | Gosh, my morning just got so much more exciting | |
| 09:26:56 | bauzas | asettle: so we changed the placement devstack module to not use a port | |
| 09:27:22 | bauzas | asettle: but given some operators already looked at the previous version, they began to use this port | |
| 09:27:24 | asettle | Right. Right. | |
| 09:27:29 | asettle | That's. Not great. | |
| 09:27:34 | asettle | *head desk* | |
| 09:27:41 | asettle | So what port *do* you guys use now? | |
| 09:27:43 | bauzas | asettle: most of them are saying it's fine because they don't use Senlin | |
| 09:27:50 | asettle | Yes, right. | |
| 09:27:54 | asettle | But we can't have htat in the official docs. | |
| 09:27:56 | bauzas | asettle: in devstack, we don't use a port | |
| 09:28:14 | bauzas | we use the httpd virtualhost config | |
| 09:28:43 | bauzas | asettle: https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template | |
| 09:29:09 | bauzas | asettle: unfortunately, some operators didn't changed that | |
| 09:29:14 | bauzas | (yet) | |
| 09:30:49 | asettle | Honest to god the more i learn each morning about this the more I want to pretend I don't know :P | |
| 09:30:50 | asettle | Okay. | |
| 09:30:55 | asettle | Thanks for hte contenxt bauzas :) | |
| 09:30:57 | asettle | context* | |
| 09:32:35 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791 | |
| 09:34:29 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrations.py https://review.openstack.org/459727 | |
| 09:36:54 | bauzas | asettle: so, about documentation, I don't know what to say | |
| 09:36:59 | johnthetubaguy | asettle: the wider context, is we get everyone off ports as we get all the APIs onto apache, I think | |
| 09:37:00 | bauzas | asettle: because some use that port | |
| 09:37:05 | bauzas | asettle: some are not | |
| 09:37:14 | sfinucan | lyarwood: on it | |
| 09:37:15 | johnthetubaguy | bauzas: its tempting to say, find URL that your placement API is hosted on, and add it in here | |
| 09:37:34 | bauzas | johnthetubaguy: yup, it's definitely a config | |
| 09:37:43 | asettle | I think what we might have to do is write a very big IMPORTANT note that reminds people you can use that port, but do not use it if you intend to use Senlin. Whihc is really messy but I"m not seeing too many ways around it, to be honest. Using the placement is *not* working, we can't argue that right now. | |
| 09:37:47 | asettle | We need to have some working documentation. | |
| 09:37:52 | lyarwood | sfinucan: cheers | |
| 09:38:04 | bauzas | johnthetubaguy: but unfortunately, that depends on distros | |
| 09:38:41 | bauzas | asettle: deployers don't have choices, it's untrue to tell them that | |
| 09:38:44 | johnthetubaguy | bauzas: right, thats what I mean by find the URL | |
| 09:38:59 | johnthetubaguy | we can give its usually this or this, etc | |
| 09:39:07 | bauzas | asettle: it's rather, deployers need to know the related option depending on their distro | |
| 09:39:35 | asettle | Okay. So how do we communicate that in a short, concise, documented, step? | |
| 09:39:40 | bauzas | (at least until all distros move to a non-port URL) | |
| 09:40:00 | bauzas | asettle: the main question is, how deployers can discover that ? | |
| 09:40:15 | bauzas | I'd be tempted to say all distros use httpd | |
| 09:40:30 | bauzas | so we could say to deployers to look at the /etc/httpd/ path | |
| 09:41:03 | bauzas | either sites.de or sites-enabled depending on the OS | |
| 09:41:46 | bauzas | or grep something in /etc/{httpd|apache} | |
| 09:42:14 | bauzas | either way, the idea is to find the placement virtualhost configuration | |
| 09:42:23 | bauzas | *if they use httpd* | |
| 09:42:40 | bauzas | because if they use nginx+uswsgi, then it's totally different | |
| 09:43:54 | johnthetubaguy | so these docs are for specific packages right? | |
| 09:44:01 | johnthetubaguy | can't we just find out that answer | |
| 09:49:48 | bauzas | johnthetubaguy: the other problem is that it's unrelated to packages :p | |
| 09:50:07 | johnthetubaguy | bauzas: I thought the packages installed the apache conf? | |
| 09:50:32 | bauzas | johnthetubaguy: unfortunately no, because TripleO wipes that packaged conf and provides its own | |
| 09:50:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | |
| 09:50:47 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 09:50:53 | johnthetubaguy | it can be overriden, for sure, but I thought they general added one | |
| 09:51:02 | johnthetubaguy | if not, our guide should tell them how to set that up | |
| 09:51:14 | johnthetubaguy | so we just need the URL to match the guide, and its easier? | |
| 09:51:20 | bauzas | johnthetubaguy: in other words, openstack-nova in RDO uses port 8778 AFAIK but if you puppet-nova on top of it, it will use /placement | |
| 09:51:30 | johnthetubaguy | sure, thats fine | |
| 09:51:44 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.soft_delete notifications https://review.openstack.org/460029 | |
| 09:51:49 | johnthetubaguy | this is about using RDO packages manually, so seems like 8778 is the correct answer | |
| 09:52:06 | johnthetubaguy | now a warning say, there be dragons here if you change the package defaults is a good idea | |
| 09:52:24 | bauzas | johnthetubaguy: this is fine, you mean ? http://imgur.com/c4jt321 :D | |
| 09:52:57 | johnthetubaguy | so the guide is about using the packages manaully | |
| 09:53:06 | johnthetubaguy | if you don't do that, the guide will be very wrong in many ways | |
| 09:53:40 | bauzas | good point | |