| *** vnogin has joined #puppet-openstack | 01:19 | |
| *** vnogin has quit IRC | 01:23 | |
| *** dpawlik has joined #puppet-openstack | 01:44 | |
| *** ilbot3 has quit IRC | 01:47 | |
| *** dpawlik has quit IRC | 01:48 | |
| *** ilbot3 has joined #puppet-openstack | 01:58 | |
| *** dpawlik has joined #puppet-openstack | 03:10 | |
| *** dpawlik has quit IRC | 03:14 | |
| *** jaganathan has joined #puppet-openstack | 04:16 | |
| *** jaganathan has quit IRC | 04:16 | |
| *** jaganathan has joined #puppet-openstack | 04:17 | |
| *** vnogin has joined #puppet-openstack | 04:19 | |
| *** vnogin has quit IRC | 04:23 | |
| *** ykarel has joined #puppet-openstack | 04:27 | |
| *** jaganathan has quit IRC | 04:29 | |
| *** jpena|off has quit IRC | 04:29 | |
| *** lhinds has quit IRC | 04:29 | |
| *** jaganathan has joined #puppet-openstack | 04:30 | |
| *** vkhanna has quit IRC | 04:30 | |
| *** vkhanna has joined #puppet-openstack | 04:30 | |
| *** rasca has quit IRC | 04:30 | |
| *** rnoriega_ has quit IRC | 04:31 | |
| *** lhinds has joined #puppet-openstack | 04:31 | |
| *** radez has quit IRC | 04:31 | |
| *** jpena|off has joined #puppet-openstack | 04:31 | |
| *** rook has quit IRC | 04:31 | |
| *** rnoriega has joined #puppet-openstack | 04:33 | |
| *** radez has joined #puppet-openstack | 04:33 | |
| *** rook has joined #puppet-openstack | 04:34 | |
| *** rook is now known as Guest94983 | 04:34 | |
| *** rasca has joined #puppet-openstack | 04:36 | |
| *** skramaja has joined #puppet-openstack | 04:55 | |
| *** markvoelker has quit IRC | 05:08 | |
| *** dpawlik has joined #puppet-openstack | 05:11 | |
| *** dpawlik has quit IRC | 05:15 | |
| *** armaan has joined #puppet-openstack | 05:28 | |
| *** agurenko has joined #puppet-openstack | 05:32 | |
| *** jtomasek has quit IRC | 05:59 | |
| *** markvoelker has joined #puppet-openstack | 06:09 | |
| *** jtomasek has joined #puppet-openstack | 06:11 | |
| *** jtomasek has quit IRC | 06:18 | |
| *** jtomasek has joined #puppet-openstack | 06:19 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix placement URL https://review.openstack.org/565509 | 06:26 |
|---|---|---|
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon master: Debian is using python3-memcache https://review.openstack.org/559421 | 06:27 |
| *** markvoelker has quit IRC | 06:44 | |
| *** rcernin has quit IRC | 06:58 | |
| *** dpawlik has joined #puppet-openstack | 07:12 | |
| *** dpawlik has quit IRC | 07:16 | |
| *** tesseract has joined #puppet-openstack | 07:18 | |
| openstackgerrit | Pranali Deore proposed openstack/puppet-glance master: Add node_staging_uri parameter https://review.openstack.org/565761 | 07:22 |
| *** tosky has joined #puppet-openstack | 07:29 | |
| *** jpena|off is now known as jpena | 07:35 | |
| *** ykarel is now known as ykarel|lunch | 07:38 | |
| *** markvoelker has joined #puppet-openstack | 07:41 | |
| *** amoralej|off is now known as amoralej | 07:50 | |
| *** shardy has joined #puppet-openstack | 08:08 | |
| *** gfidente has joined #puppet-openstack | 08:13 | |
| *** gfidente has quit IRC | 08:13 | |
| *** gfidente has joined #puppet-openstack | 08:13 | |
| *** vnogin has joined #puppet-openstack | 08:13 | |
| *** markvoelker has quit IRC | 08:13 | |
| *** salmankhan has joined #puppet-openstack | 08:25 | |
| *** ykarel|lunch is now known as ykarel | 08:25 | |
| *** derekh has joined #puppet-openstack | 08:36 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Remove ironic_host_manager usage https://review.openstack.org/565832 | 08:45 |
| *** vnogin has quit IRC | 08:51 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Remove ironic_host_manager usage https://review.openstack.org/565832 | 08:51 |
| zigo | mnaser: https://review.openstack.org/#/c/564328/ <--- Passes the CI thanks to tobasco ! | 08:55 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon master: Debian is using python3-memcache https://review.openstack.org/559421 | 08:59 |
| *** salmankhan has quit IRC | 09:01 | |
| *** salmankhan has joined #puppet-openstack | 09:04 | |
| *** markvoelker has joined #puppet-openstack | 09:10 | |
| *** dpawlik has joined #puppet-openstack | 09:13 | |
| *** dpawlik has quit IRC | 09:17 | |
| *** vnogin has joined #puppet-openstack | 09:22 | |
| *** khappone has joined #puppet-openstack | 09:32 | |
| *** khappone_ has quit IRC | 09:36 | |
| *** markvoelker has quit IRC | 09:44 | |
| *** morazi has quit IRC | 10:02 | |
| *** salmankhan has quit IRC | 10:14 | |
| *** salmankhan has joined #puppet-openstack | 10:15 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Remove ironic_host_manager usage https://review.openstack.org/565832 | 10:23 |
| *** markvoelker has joined #puppet-openstack | 10:41 | |
| *** dtantsur|afk is now known as dtantsur | 10:50 | |
| *** jpena is now known as jpena|lunch | 10:58 | |
| openstackgerrit | Merged openstack/puppet-nova master: Remove references to Exact* filters https://review.openstack.org/565512 | 11:01 |
| *** vnogin has quit IRC | 11:08 | |
| *** markvoelker has quit IRC | 11:14 | |
| *** vnogin has joined #puppet-openstack | 11:24 | |
| *** morazi has joined #puppet-openstack | 11:28 | |
| *** atoth has joined #puppet-openstack | 11:30 | |
| *** amoralej is now known as amoralej|off | 11:53 | |
| *** amoralej|off is now known as amoralej|lunch | 11:54 | |
| *** armaan has quit IRC | 12:00 | |
| *** armaan has joined #puppet-openstack | 12:01 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix placement URL https://review.openstack.org/565509 | 12:03 |
| *** jpena|lunch is now known as jpena | 12:11 | |
| *** markvoelker has joined #puppet-openstack | 12:11 | |
| *** markvoelker has quit IRC | 12:20 | |
| *** markvoelker has joined #puppet-openstack | 12:20 | |
| *** vinsh_ has joined #puppet-openstack | 12:31 | |
| *** vinsh has quit IRC | 12:31 | |
| *** vinsh_ has quit IRC | 12:31 | |
| *** vinsh has joined #puppet-openstack | 12:31 | |
| *** armaan has quit IRC | 12:54 | |
| *** salmankhan has quit IRC | 12:56 | |
| *** beagles is now known as beagles_mtg | 12:57 | |
| *** salmankhan has joined #puppet-openstack | 12:57 | |
| openstackgerrit | Alan Bishop proposed openstack/puppet-cinder master: Revert "Restore iscsi loopback LVM volume group on startup" https://review.openstack.org/566056 | 13:09 |
| *** dpawlik has joined #puppet-openstack | 13:12 | |
| *** dpawlik has quit IRC | 13:17 | |
| *** amoralej|lunch is now known as amoralej | 13:26 | |
| *** ykarel is now known as ykarel|away | 13:27 | |
| *** ykarel|away has quit IRC | 13:32 | |
| *** alee_afk is now known as alee | 13:32 | |
| *** salmankhan has quit IRC | 13:49 | |
| *** salmankhan has joined #puppet-openstack | 13:51 | |
| *** skramaja has quit IRC | 13:55 | |
| *** rajinir has joined #puppet-openstack | 14:00 | |
| *** beagles_mtg is now known as beagles | 14:13 | |
| *** vnogin has quit IRC | 14:34 | |
| *** vnogin has joined #puppet-openstack | 14:35 | |
| *** agurenko has quit IRC | 14:37 | |
| *** vnogin has quit IRC | 14:40 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix placement URL https://review.openstack.org/565509 | 14:55 |
| *** vnogin has joined #puppet-openstack | 14:59 | |
| *** vnogin has quit IRC | 15:00 | |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch to using the cached version of CirrOS https://review.openstack.org/566095 | 15:13 |
| *** dpawlik has joined #puppet-openstack | 15:13 | |
| *** dpawlik has quit IRC | 15:17 | |
| *** vnogin has joined #puppet-openstack | 15:26 | |
| *** vnogin has quit IRC | 16:04 | |
| *** salmankhan has quit IRC | 16:06 | |
| *** agurenko has joined #puppet-openstack | 16:11 | |
| mwhahaha | mnaser: can you lift your -2 on this one https://review.openstack.org/#/c/558454/ we landed all the stuff elsewhere | 16:27 |
| *** tesseract has quit IRC | 16:28 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Also install python3-tempest-horizon https://review.openstack.org/565859 | 16:40 |
| *** tesseract has joined #puppet-openstack | 16:57 | |
| *** derekh has quit IRC | 16:59 | |
| *** jpena is now known as jpena|off | 17:01 | |
| *** dtantsur is now known as dtantsur|afk | 17:02 | |
| *** shardy has quit IRC | 17:03 | |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-openstack-integration master: Revert "Update the cirros version to "0.4.0"" https://review.openstack.org/566122 | 17:11 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-tempest master: Revert "Update the cirros version to "0.4.0"" https://review.openstack.org/566123 | 17:12 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-glance master: Revert "Update the cirros version to "0.4.0"" https://review.openstack.org/566124 | 17:12 |
| mnaser | mwhahaha: sorry, landed | 17:12 |
| mwhahaha | gracias | 17:13 |
| *** dpawlik has joined #puppet-openstack | 17:14 | |
| *** trown is now known as trown|lunch | 17:17 | |
| *** dpawlik has quit IRC | 17:18 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon master: Debian is using python3-memcache https://review.openstack.org/559421 | 17:19 |
| *** tesseract has quit IRC | 17:25 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Allow to test Debian package on top of Ubuntu https://review.openstack.org/560282 | 17:33 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-openstack-integration master: Debian: add puppet system user https://review.openstack.org/566126 | 17:40 |
| *** salmankhan has joined #puppet-openstack | 17:41 | |
| *** salmankhan has quit IRC | 17:46 | |
| *** Guest94983 is now known as rook | 17:54 | |
| openstackgerrit | Merged openstack/puppet-ceilometer master: Deprecate auth_uri option https://review.openstack.org/558454 | 18:24 |
| *** trown|lunch is now known as trown | 18:29 | |
| *** amoralej is now known as amoralej|off | 18:42 | |
| *** morazi has quit IRC | 19:03 | |
| *** openstackgerrit has quit IRC | 19:05 | |
| *** agurenko has quit IRC | 19:12 | |
| *** dpawlik has joined #puppet-openstack | 19:15 | |
| *** atoth has quit IRC | 19:17 | |
| *** atoth has joined #puppet-openstack | 19:17 | |
| *** dpawlik has quit IRC | 19:19 | |
| *** openstackgerrit has joined #puppet-openstack | 19:37 | |
| openstackgerrit | Merged openstack/puppet-glance master: Add node_staging_uri parameter https://review.openstack.org/565761 | 19:37 |
| *** gfidente has quit IRC | 19:52 | |
| *** trown is now known as trown|outtypewww | 21:06 | |
| *** dpawlik has joined #puppet-openstack | 21:16 | |
| openstackgerrit | Thiago da Silva proposed openstack/puppet-swift master: Update s3api configuration https://review.openstack.org/498567 | 21:18 |
| openstackgerrit | Thiago da Silva proposed openstack/puppet-swift master: Update s3api configuration https://review.openstack.org/498567 | 21:20 |
| *** dpawlik has quit IRC | 21:21 | |
| zigo | tobasco: mnaser: mwhahaha: EmilienM: Hey, got a bunch of waiting-to-be-reviewed patches: https://review.openstack.org/#/c/559421/ https://review.openstack.org/#/c/564328/ https://review.openstack.org/#/c/565509/ https://review.openstack.org/#/c/566126/ | 21:28 |
| zigo | :) | 21:29 |
| mnaser | Let me see what I can help push from a phone lol | 21:29 |
| zigo | :) | 21:30 |
| mwhahaha | so demanding | 21:30 |
| *** jtomasek has quit IRC | 21:30 | |
| zigo | mwhahaha: With these patches, ./all-in-one.sh passes in my Virtualbox ! :) | 21:31 |
| mwhahaha | so what you're saying is that it works in devstack | 21:31 |
| * mwhahaha trolls while reviewing | 21:32 | |
| zigo | Where's my post from 2012 where I wrote as Subject: "devstack should die" ? | 21:32 |
| mnaser | zigo: one tiny bit | 21:34 |
| mnaser | If you wanna address it now to get a +2 while I’m here | 21:34 |
| mnaser | On the last patch | 21:34 |
| zigo | Sure, doing it. | 21:35 |
| zigo | Sorry for the extra lines, I'm not even sure how I added them... :P | 21:35 |
| zigo | mnaser: Done. | 21:36 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-openstack-integration master: Debian: add puppet system user https://review.openstack.org/566126 | 21:36 |
| mnaser | zigo: done, one patch left but it’s a bit too hard to review on mobile | 21:37 |
| zigo | mwhahaha: What does the thing you linked to proves? | 21:37 |
| mwhahaha | the nova code | 21:38 |
| zigo | mwhahaha: Sure, VNC should be disabled, and SPICE enabled so that you can run VMs with SPICE. But it says nothing about the spicehtml5proxy needs it, no? | 21:38 |
| mwhahaha | zigo: but how are you configuring that for nova-compute then? | 21:38 |
| zigo | mwhahaha: It's done in compute.pp | 21:38 |
| mwhahaha | zigo: that class configure nova.conf not just spice proxy | 21:38 |
| zigo | mwhahaha: Compute nodes will use compute.pp, right? | 21:39 |
| mwhahaha | it's not in nova::compute, it's in nova::compute::spice | 21:39 |
| mwhahaha | so ok i guess we don't need it there | 21:39 |
| mwhahaha | cause it's duplicated | 21:40 |
| zigo | My understanding is that proxy nodes will not for sure use compute.pp, but that compute nodes will always do. | 21:40 |
| mwhahaha | so i think the issue is that next time let's comment where it's actually be in configured if it's duped | 21:40 |
| zigo | I can do that ! | 21:40 |
| zigo | :) | 21:40 |
| * zigo ammends to patch to add the comment | 21:41 | |
| mwhahaha | these bits are a bit bad because we are duplicating some of the configs but yea we should metion where it's really configured :) | 21:41 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian https://review.openstack.org/564328 | 21:45 |
| zigo | mwhahaha: There you go, comment added, and I didn't touch anything else in the patch (so the CI run will continue to be green and you can review still...) | 21:45 |
| mwhahaha | zigo: so shouldn't we be using nova::compute::spice? | 21:46 |
| zigo | mwhahaha: I've set it in ::nova::compute, I didn't know it was in nova::compute::spice. :/ | 21:47 |
| mwhahaha | i think the previous interaction was to just set vnc to false for nova::compute and include ::nova::compute::spiace | 21:47 |
| zigo | mwhahaha: Why would we have ::nova::compute::spice and not ::nova::compute::vnc? It doesn't really make sence to me, these things are supposed to be quite the same. | 21:48 |
| mwhahaha | so if you're going to use spice, that's how you'd do it because there appears to be additional bits to configure | 21:48 |
| mwhahaha | these are some really old classes | 21:48 |
| mwhahaha | that we never touch :D | 21:48 |
| zigo | :) | 21:48 |
| mwhahaha | Dec 8, 2015 | 21:48 |
| mwhahaha | so yea | 21:48 |
| mwhahaha | i think you're the first to try and improve them | 21:49 |
| zigo | mwhahaha: Not even, it's from 2013 !!! | 21:49 |
| zigo | (as per git blame...) | 21:49 |
| zigo | Mehdi Abaakouk ... | 21:49 |
| zigo | :P | 21:49 |
| mwhahaha | yea so | 21:49 |
| mwhahaha | that's why :D | 21:50 |
| zigo | At that time, I believe I was working with eNovance ... | 21:50 |
| mwhahaha | spice has more configs than vnc | 21:50 |
| zigo | Last time, we tried using SPICE instead of VNC, because the Swiss keyboard was badly handle with VNC. | 21:51 |
| zigo | Funny stuff was, it was ALSO a nightmare with SPICE. | 21:51 |
| mwhahaha | ha | 21:51 |
| zigo | The only way to use these consoles is with a US keyboard. | 21:51 |
| zigo | It's been years like that, and nobody seems to care. | 21:52 |
| zigo | It's also not trivial. The Javascrpit gets the keyboards from the browser, which is not really a OS keyboard mapping, then you got to find out what was the key that was in use, then translate that to whatever the keybaord pair is... | 21:53 |
| zigo | Add special keys into the mix, and about a hundred keyboards... | 21:54 |
| zigo | mwhahaha: So what do you suggest I do for that patch? | 21:54 |
| mwhahaha | don't specify spice/enabled in nova::compute | 21:55 |
| mwhahaha | and use nova::compute::spice to configure it | 21:55 |
| zigo | Ok. | 21:55 |
| zigo | mwhahaha: But no ! :) | 21:56 |
| zigo | mwhahaha: The issue is that, in the Debian package, [spice]/enable is set to True by default. | 21:57 |
| zigo | That's the issue I'm attempting to fix to begin with. | 21:57 |
| mwhahaha | and this is why we don't like things set in config files in packages | 21:57 |
| * mwhahaha shakes fist | 21:57 | |
| zigo | puppet-openstack is not forcing [spice]/enable to false if you're just using VNC... | 21:57 |
| zigo | LOL ! | 21:58 |
| mwhahaha | so the problem is adding it to compute.pp will cause a duplicate declaration | 21:58 |
| mwhahaha | it's complicated | 21:58 |
| zigo | mwhahaha: I could get nova::compute::spice read the new value from nova::compute, no? | 21:59 |
| * zigo is thinking | 21:59 | |
| mwhahaha | if you inherited it, yea | 21:59 |
| mwhahaha | but you still run into catalog problems | 21:59 |
| zigo | mwhahaha: So what's the solution? :/ | 22:02 |
| mwhahaha | fix the packaging to stop including true :D | 22:02 |
| zigo | mwhahaha: Why would my package stop to set the best default? | 22:02 |
| zigo | SPICE is better than VNC. | 22:02 |
| zigo | There's no reason to activate VNC by default... | 22:03 |
| mwhahaha | i didn't say it isn't | 22:03 |
| mwhahaha | but enabling any of the services assumes too much | 22:03 |
| * mwhahaha prefers a less opinionated approach | 22:03 | |
| zigo | Well, ok, let's say you're right. | 22:03 |
| zigo | But then, we would still have a real *bug* in puppet-openstack. | 22:03 |
| mwhahaha | so think what you need to do is drop it from nova::compute::spice | 22:03 |
| mwhahaha | set it in nova::compute | 22:03 |
| zigo | Just consider someone that wants to try SPICE, and then decides to go back to VNC. | 22:04 |
| mwhahaha | and add a release note | 22:04 |
| zigo | In this case, puppet-openstack will miss fixing [spice]/enable. | 22:04 |
| mwhahaha | we have ways around that | 22:04 |
| mwhahaha | ie purge => true on nova_config | 22:04 |
| mwhahaha | but anyway | 22:04 |
| mwhahaha | drop it from nova::compute::spice, leave your setting in nova::compute | 22:04 |
| zigo | Really? Then this should be fixed in puppet-openstack-integration, no? | 22:04 |
| zigo | Ok. | 22:05 |
| mwhahaha | and add a release note that says if you used to rely on ::nova::compute::spice to enable spice in nova.conf, you must now specify $nevar | 22:05 |
| mwhahaha | s/nevar/newvar | 22:05 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian https://review.openstack.org/564328 | 22:07 |
| zigo | Yeah, done. | 22:07 |
| mwhahaha | i'll check it out later | 22:07 |
| * mwhahaha wanders off | 22:07 | |
| *** rcernin has joined #puppet-openstack | 22:07 | |
| zigo | mwhahaha: Thanks man, for taking the time. | 22:08 |
| openstackgerrit | Merged openstack/puppet-horizon master: Debian is using python3-memcache https://review.openstack.org/559421 | 22:44 |
| openstackgerrit | Merged openstack/puppet-nova master: Fix placement URL https://review.openstack.org/565509 | 22:58 |
| *** rajinir has quit IRC | 23:09 | |
| *** dpawlik has joined #puppet-openstack | 23:14 | |
| *** dpawlik has quit IRC | 23:18 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!