| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-26 | |||
| 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 | |
| 09:53:50 | bauzas | I'm not that seasoned with docs | |
| 09:53:59 | bauzas | although I patched docs in 2013 :p | |
| 09:54:28 | bauzas | but IIRC, it was far less complicated, either you deployed with Ubuntu or Fedora | |
| 09:55:35 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in floating_ip files https://review.openstack.org/459791 | |
| 09:55:50 | bauzas | johnthetubaguy: asettle: so I guess if it's that simple, we should just have variable doc depending on which distro they use nope? | |
| 09:56:05 | johnthetubaguy | alex_xu: cool, glad we got that deprecation sorted, makes those docs much simpler ^ | |
| 09:56:10 | asettle | bauzas: the variables are easy. We can manage that. We already do. So. Yep? | |
| 09:56:17 | johnthetubaguy | bauzas: thats what we do | |
| 09:56:41 | bauzas | asettle: could we then just try to have that reviewed by most of the distros ? | |
| 09:56:56 | bauzas | and having endpoints different between them ? | |
| 09:57:11 | bauzas | idk, just seems to me the easiest solution | |
| 09:58:14 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrate_server.py https://review.openstack.org/459711 | |
| 09:58:36 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in migrate_server.py https://review.openstack.org/459711 | |
| 10:04:02 | cdent | asettle, bauzas, johnthetubaguy did you resolve the above discussion satisfactorily? I tend to agree with johnthetubaguy: make the docs match the packages and leave it at that (for now) | |
| 10:04:16 | asettle | cdent: uh not quite. I think bauzas lost me with his last comments. | |
| 10:11:44 | Dinesh_Bhor | johnthetubaguy, sfinucan: Hi, if you get time then please take a look at it: https://review.openstack.org/#/c/457890/ | |
| 10:12:00 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | |