Posted	Nick	Remark
#openstack-nova - 2017-04-26
07:24:20	openstackgerrit	Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569
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

Earlier   Later