| *** sai_p has quit IRC | 00:03 | |
| *** atoth has quit IRC | 00:53 | |
| openstackgerrit | Merged openstack/puppet-ganesha master: Retire the project https://review.openstack.org/556771 | 00:58 |
|---|---|---|
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558419 | 01:06 |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Deprecate auth_uri option https://review.openstack.org/558344 | 01:07 |
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558361 | 01:08 |
| *** armaan has quit IRC | 01:09 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558419 | 01:12 |
| *** gyan__ has joined #puppet-openstack | 01:13 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Deprecate auth_uri option https://review.openstack.org/558344 | 01:15 |
| *** armaan has joined #puppet-openstack | 01:35 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558419 | 01:39 |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Deprecate auth_uri option https://review.openstack.org/558344 | 01:40 |
| *** ilbot3 has quit IRC | 01:44 | |
| *** ilbot3 has joined #puppet-openstack | 01:57 | |
| *** ykarel|afk has joined #puppet-openstack | 02:16 | |
| *** ykarel|afk has quit IRC | 02:29 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Deprecate auth_uri option https://review.openstack.org/558344 | 02:31 |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Add puppet-openstack-integration-jobs-all job to the repo https://review.openstack.org/558674 | 02:36 |
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558361 | 02:42 |
| *** armaan has quit IRC | 02:43 | |
| *** armaan has joined #puppet-openstack | 02:43 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-openstack-integration master: Move puppet5 jobs into check pipeline as non-voting https://review.openstack.org/556341 | 02:46 |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Use Debian's repo when using Debian https://review.openstack.org/558485 | 02:49 |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Fixed puppet repo & key in Debian https://review.openstack.org/558457 | 02:49 |
| *** ykarel|afk has joined #puppet-openstack | 04:09 | |
| *** ykarel|afk is now known as ykarel | 04:10 | |
| *** asilenkov has quit IRC | 04:12 | |
| *** asilenkov has joined #puppet-openstack | 04:13 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558361 | 04:33 |
| openstackgerrit | zhongshengping proposed openstack/puppet-keystone master: Add puppet-openstack-integration-jobs-all job to the repo https://review.openstack.org/558674 | 04:34 |
| *** tonyb has quit IRC | 04:45 | |
| *** jtomasek has joined #puppet-openstack | 05:39 | |
| *** PagliaccisCloud has quit IRC | 05:40 | |
| *** PagliaccisCloud has joined #puppet-openstack | 05:45 | |
| *** agurenko has joined #puppet-openstack | 05:50 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ec2api master: Configure metadata_proxy_shared_secret as secret https://review.openstack.org/558697 | 05:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/558702 | 06:09 |
| *** jaosorior has quit IRC | 06:25 | |
| *** shardy has quit IRC | 06:36 | |
| *** shardy has joined #puppet-openstack | 06:36 | |
| openstackgerrit | Merged openstack/puppet-keystone master: Deprecate auth_uri option https://review.openstack.org/558344 | 06:42 |
| *** jaosorior has joined #puppet-openstack | 06:58 | |
| openstackgerrit | Sulaiman Radwan proposed openstack/puppet-cinder master: Deprecate iscsi praramters in cider backend https://review.openstack.org/558714 | 06:59 |
| *** armaan has quit IRC | 07:12 | |
| *** armaan has joined #puppet-openstack | 07:12 | |
| *** skramaja has joined #puppet-openstack | 07:22 | |
| *** rcernin has quit IRC | 07:23 | |
| *** tesseract has joined #puppet-openstack | 07:26 | |
| *** amoralej|off is now known as amoralej | 07:34 | |
| *** shardy has quit IRC | 07:45 | |
| *** shardy has joined #puppet-openstack | 07:45 | |
| *** jpena|off is now known as jpena | 07:46 | |
| openstackgerrit | Merged openstack/puppet-neutron master: Deprecate auth_uri parameter https://review.openstack.org/558361 | 07:57 |
| openstackgerrit | zhongshengping proposed openstack/puppet-aodh master: Deprecate auth_uri option https://review.openstack.org/558340 | 08:08 |
| *** agurenko has quit IRC | 08:09 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: ACK Debian's switch to Python 3 for Neutron https://review.openstack.org/558733 | 08:10 |
| *** agurenko has joined #puppet-openstack | 08:12 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ceilometer master: Deprecate auth_uri option https://review.openstack.org/558454 | 08:15 |
| *** gfidente has joined #puppet-openstack | 08:42 | |
| *** gfidente has quit IRC | 08:42 | |
| *** gfidente has joined #puppet-openstack | 08:42 | |
| *** vnogin has joined #puppet-openstack | 09:07 | |
| zigo | Any idea what's going on in the gate? Everything is failing now, with this error: | 09:11 |
| zigo | "Puppet (err): Failed to apply catalog: File: /etc/neutron/neutron.conf does not contain all required sections. Neutron types will not work if neutron is not correctly configured." | 09:11 |
| zigo | It doesn't feel right, as I inspected neutron.conf from the logs, and it seemed correct. So there's something else going on. | 09:11 |
| zigo | https://review.openstack.org/#/c/558483 | 09:11 |
| *** vnogin has quit IRC | 09:11 | |
| *** ykarel_ has joined #puppet-openstack | 09:14 | |
| *** ykarel has quit IRC | 09:17 | |
| openstackgerrit | Merged openstack/puppet-keystone master: Add puppet-openstack-integration-jobs-all job to the repo https://review.openstack.org/558674 | 09:17 |
| *** vnogin has joined #puppet-openstack | 09:28 | |
| *** salmankhan has joined #puppet-openstack | 09:28 | |
| *** ykarel__ has joined #puppet-openstack | 09:39 | |
| *** ykarel_ has quit IRC | 09:43 | |
| *** dtantsur|afk is now known as dtantsur | 09:54 | |
| openstackgerrit | Sulaiman Radwan proposed openstack/puppet-cinder master: Deprecate iscsi praramters in cinder backend https://review.openstack.org/558714 | 09:54 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-glance master: Fix README to use api and registry classes https://review.openstack.org/558774 | 09:58 |
| *** jaganathan has quit IRC | 10:05 | |
| *** HW_Peter has joined #puppet-openstack | 10:06 | |
| *** jtomasek_ has joined #puppet-openstack | 10:06 | |
| *** vnogin_ has joined #puppet-openstack | 10:07 | |
| *** armaan_ has joined #puppet-openstack | 10:07 | |
| *** beekneemech has joined #puppet-openstack | 10:10 | |
| *** tdb_ has joined #puppet-openstack | 10:11 | |
| *** jgrassle1 has joined #puppet-openstack | 10:11 | |
| *** m3ntat has joined #puppet-openstack | 10:13 | |
| *** gfidente^2nd has joined #puppet-openstack | 10:14 | |
| *** gfidente has quit IRC | 10:15 | |
| *** jtomasek has quit IRC | 10:15 | |
| *** owalsh has quit IRC | 10:15 | |
| *** mentat has quit IRC | 10:15 | |
| *** bnemec has quit IRC | 10:15 | |
| *** vnogin has quit IRC | 10:15 | |
| *** armaan has quit IRC | 10:15 | |
| *** cgfbee has quit IRC | 10:15 | |
| *** HW-Peter has quit IRC | 10:15 | |
| *** mkarpin has quit IRC | 10:15 | |
| *** tdb has quit IRC | 10:15 | |
| *** dmsimard has quit IRC | 10:15 | |
| *** jgrassler has quit IRC | 10:15 | |
| *** owalsh has joined #puppet-openstack | 10:17 | |
| *** mkarpin has joined #puppet-openstack | 10:18 | |
| *** cgfbee has joined #puppet-openstack | 10:18 | |
| *** dmsimard has joined #puppet-openstack | 10:18 | |
| *** salmankhan has quit IRC | 10:28 | |
| *** salmankhan has joined #puppet-openstack | 10:32 | |
| *** rcernin has joined #puppet-openstack | 10:44 | |
| openstackgerrit | Merged openstack/puppet-ironic master: Deprecate auth_uri option https://review.openstack.org/558450 | 10:47 |
| *** gfidente^2nd has quit IRC | 11:11 | |
| *** vnogin_ has quit IRC | 11:20 | |
| *** gfidente has joined #puppet-openstack | 11:22 | |
| *** gfidente has quit IRC | 11:22 | |
| *** gfidente has joined #puppet-openstack | 11:22 | |
| *** gyan__ has quit IRC | 11:38 | |
| *** ykarel__ has quit IRC | 11:43 | |
| *** vnogin has joined #puppet-openstack | 11:45 | |
| *** ykarel has joined #puppet-openstack | 11:50 | |
| *** jpena is now known as jpena|lunch | 11:53 | |
| *** dprince has joined #puppet-openstack | 11:55 | |
| *** vnogin has quit IRC | 11:57 | |
| *** alee has joined #puppet-openstack | 12:10 | |
| *** atoth has joined #puppet-openstack | 12:13 | |
| *** amoralej is now known as amoralej|lunch | 12:29 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-neutron master: Revert "Deprecate auth_uri parameter" https://review.openstack.org/558803 | 12:38 |
| mwhahaha | I think -^ broke the gate | 12:40 |
| mwhahaha | we'll see | 12:40 |
| *** vnogin has joined #puppet-openstack | 12:41 | |
| *** trown|outtypewww is now known as trown | 12:43 | |
| *** colonwq has quit IRC | 12:44 | |
| *** jpena|lunch is now known as jpena | 12:53 | |
| *** shardy is now known as shardy_mtg | 12:58 | |
| jpena | mwhahaha: the initial issue is https://review.openstack.org/558344. That changed auth_uri to www_authenticate_uri, and several providers seem to depend on that | 13:08 |
| mwhahaha | Yea that's what I figured | 13:10 |
| jpena | we've seen that in Packstack jobs, too | 13:11 |
| *** morazi has joined #puppet-openstack | 13:12 | |
| *** salmankhan has quit IRC | 13:15 | |
| *** ykarel has quit IRC | 13:21 | |
| *** amoralej|lunch is now known as amoralej | 13:21 | |
| *** armaan_ has quit IRC | 13:22 | |
| *** colonwq has joined #puppet-openstack | 13:23 | |
| *** armaan has joined #puppet-openstack | 13:23 | |
| *** salmankhan has joined #puppet-openstack | 13:37 | |
| *** vnogin has quit IRC | 13:45 | |
| *** vnogin has joined #puppet-openstack | 13:45 | |
| mwhahaha | hmm my revert fails ci now | 13:48 |
| mwhahaha | i wonder if we're not testing something | 13:48 |
| *** armaan has quit IRC | 13:49 | |
| *** armaan has joined #puppet-openstack | 13:49 | |
| *** ykarel has joined #puppet-openstack | 13:49 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-glance master: Add proxy support to glance_image resource https://review.openstack.org/558825 | 13:51 |
| mwhahaha | ok so the keystone change needs to be modified to set both | 13:53 |
| mwhahaha | until we clean all of them up | 13:53 |
| * mwhahaha will propose a fix | 13:53 | |
| EmilienM | mwhahaha: why did he auto approve? https://review.openstack.org/#/c/558361/ | 13:54 |
| EmilienM | same for https://review.openstack.org/#/c/558344/ | 13:54 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-keystone master: Continue to define auth_uri https://review.openstack.org/558827 | 13:55 |
| * mwhahaha shrugs | 13:55 | |
| EmilienM | mwhahaha: also why did he remove p-o-i jobs in puppet-keystone?? | 13:56 |
| mwhahaha | do i look like a mind reader? | 13:56 |
| EmilienM | and he's not on IRC AFIK | 13:56 |
| EmilienM | I'll send an email | 13:56 |
| jaosorior | mwhahaha: you kinda do. It's the beard. | 13:56 |
| jaosorior | :P | 13:57 |
| mwhahaha | ಠ_ಠ | 13:57 |
| mwhahaha | at the moment, http://i0.kym-cdn.com/photos/images/original/000/282/392/b8f.jpg would be very accurate if i drank coffee | 13:58 |
| jaosorior | hahaha daaamn | 13:58 |
| jaosorior | gonna use that gif myself | 13:58 |
| *** hamdyk has joined #puppet-openstack | 14:16 | |
| zigo | mwhahaha: Hi there! Any idea what's going on with testing at https://review.openstack.org/#/c/558483/ ? | 14:16 |
| zigo | Some tests passed this morning, but now none of them do even after a few rechecks... :( | 14:17 |
| mwhahaha | yes we're aware | 14:17 |
| mwhahaha | stuffs broke | 14:17 |
| mwhahaha | working on it | 14:17 |
| zigo | Ah ok ! :) | 14:17 |
| zigo | Thanks. | 14:17 |
| zigo | If you remember, ping me when fixed... | 14:17 |
| zigo | Oh, I see in the backlog ... :P | 14:18 |
| hamdyk | Hi all, how can I use a not yet merged patch from puppet-cinder in puppet-tripleo ? | 14:19 |
| hamdyk | I need to use resources from my patch in puppet-cinder | 14:20 |
| hamdyk | in a new patch in puppet-tripleo | 14:20 |
| mwhahaha | hamdyk: you should be able to depends-on | 14:20 |
| hamdyk | unit testing fails because there is no such resource in master puppet-cinder | 14:20 |
| mwhahaha | hamdyk: locally? | 14:21 |
| mwhahaha | or in ci | 14:21 |
| hamdyk | locally | 14:21 |
| mwhahaha | you'd have to manually pull that in | 14:21 |
| mwhahaha | you can do bundle exec rake spec_prep | 14:21 |
| mwhahaha | then manually update the fixtures/modules/cinder to have your fix | 14:22 |
| hamdyk | ahaa | 14:22 |
| hamdyk | I see, thank you | 14:22 |
| hamdyk | I'll try that now | 14:22 |
| mwhahaha | then test with bundle exec rake spec_standalone | 14:22 |
| mwhahaha | if you do that it won't cleanup the modules | 14:22 |
| mwhahaha | so you can just iterate on it | 14:22 |
| mwhahaha | bundle exec rake spec includes the prep/testing/cleanup | 14:23 |
| mwhahaha | but you can manually do those phases | 14:23 |
| mwhahaha | hidden unit testing tricks :D | 14:23 |
| hamdyk | actually I'm using PDK to run tests but I got your idea | 14:23 |
| *** armaan has quit IRC | 14:25 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-keystone master: Continue to define auth_uri https://review.openstack.org/558827 | 14:31 |
| hamdyk | 12345 | 14:31 |
| *** rcernin has quit IRC | 14:32 | |
| *** vnogin has quit IRC | 14:47 | |
| *** vnogin has joined #puppet-openstack | 14:48 | |
| *** vnogin has quit IRC | 14:53 | |
| *** morazi has quit IRC | 14:55 | |
| *** ChanServ sets mode: +o mwhahaha | 15:01 | |
| *** mwhahaha changes topic to "CI Status: RED | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/" | 15:01 | |
| *** hamdyk has quit IRC | 15:08 | |
| *** vnogin has joined #puppet-openstack | 15:09 | |
| *** morazi_onsite has joined #puppet-openstack | 15:10 | |
| *** morazi_onsite is now known as morazi | 15:11 | |
| EmilienM | mnaser: if you around: https://review.openstack.org/#/c/558827/ | 15:23 |
| *** salmankhan has quit IRC | 15:25 | |
| *** salmankhan has joined #puppet-openstack | 15:25 | |
| mnaser | EmilienM: checking | 15:34 |
| *** jaganathan has joined #puppet-openstack | 15:35 | |
| EmilienM | mnaser: I'm really unhappy with his self approving thing on that one. | 15:35 |
| mnaser | EmilienM: i agree, but i think that we kinda have to set an example, we've self approved things in the past too, so we have to all set an example :\ | 15:36 |
| mnaser | i know sometimes that we're pressed on time but yeah, this is pretty critical | 15:36 |
| mnaser | mistakes happen and he's put a lot of good work in the past but it has to be clear that this should *not* happen | 15:36 |
| mnaser | EmilienM, mwhahaha: so just to follow up, the issue is that providers are broken right now because they pulled auth_uri to run the api requests, correct? | 15:38 |
| *** tesseract has quit IRC | 15:38 | |
| mwhahaha | they look for it in the config | 15:38 |
| mwhahaha | yea | 15:38 |
| *** dtantsur is now known as dtantsur|brb | 15:38 | |
| mwhahaha | if it's not provided that's how it's getting pulled out for auth in the providers | 15:39 |
| mnaser | gotcha | 15:39 |
| mnaser | and this wasn't caught because integration tests were removed | 15:39 |
| mwhahaha | we need to continue to declare it until it's been cleaned up everywhere | 15:39 |
| mwhahaha | yea it seems that way | 15:39 |
| mwhahaha | either way, when those renames happen there's also package promotion problems as well | 15:40 |
| mwhahaha | we're still using queens ubuntu things | 15:40 |
| mnaser | okay, well this fix helps unblock things | 15:40 |
| mwhahaha | so in this case it shouldn't have been approved/merged so quickly | 15:40 |
| mwhahaha | we needed to keep some sort of backwards compatibility in place | 15:40 |
| mnaser | yeah it affects downstream a bunch | 15:40 |
| *** tesseract has joined #puppet-openstack | 15:41 | |
| mnaser | mwhahaha: i left a comment on the bug, +W'd (it looks like the change is almost done so itll get right into gate) | 15:42 |
| EmilienM | mnaser: I agree with you, we have done the same thing in the past, but we also kind of maintain our CI (and not only puppet but also tripleo etc), so... | 15:42 |
| EmilienM | mnaser: anyway, I hope my email was nice :) | 15:42 |
| mnaser | i will ask him if he can push the appropriate changes to all the providers to use either auth_url or www_authenticate_uri with fallback | 15:42 |
| mwhahaha | there are changes for the www_authenticate_uri | 15:43 |
| mwhahaha | we just needed this auth_uri bit a bit longer | 15:43 |
| * mnaser is in the middle of a meeting so i'll try to digest this when im done | 15:43 | |
| mwhahaha | yea we'll get it sorted today :D | 15:43 |
| mnaser | EmilienM: it was clear and nice :) | 15:43 |
| *** agurenko has quit IRC | 15:44 | |
| *** armaan has joined #puppet-openstack | 15:46 | |
| *** skramaja has quit IRC | 15:50 | |
| *** salmankhan has quit IRC | 15:56 | |
| *** vnogin has quit IRC | 15:57 | |
| *** atoth has quit IRC | 16:00 | |
| *** salmankhan has joined #puppet-openstack | 16:03 | |
| *** jaganathan has quit IRC | 16:10 | |
| *** trown is now known as trown|lunch | 16:14 | |
| *** atoth has joined #puppet-openstack | 16:15 | |
| *** beekneemech is now known as bnemec | 16:42 | |
| *** dprince has quit IRC | 16:59 | |
| EmilienM | I think https://review.openstack.org/#/c/558827/ failed on horizon :( | 16:59 |
| mwhahaha | stupid horizon | 17:10 |
| *** salmankhan has quit IRC | 17:13 | |
| *** jpena is now known as jpena|off | 17:19 | |
| *** trown|lunch is now known as trown | 17:26 | |
| *** dprince has joined #puppet-openstack | 17:26 | |
| *** tdb_ has quit IRC | 17:31 | |
| *** tdb has joined #puppet-openstack | 17:31 | |
| *** agurenko has joined #puppet-openstack | 17:40 | |
| *** gfidente is now known as gfidente|afk | 17:42 | |
| *** vnogin has joined #puppet-openstack | 17:44 | |
| *** sai_p has joined #puppet-openstack | 17:44 | |
| *** atoth has quit IRC | 17:56 | |
| *** shardy_mtg has quit IRC | 18:06 | |
| *** vnogin has quit IRC | 18:08 | |
| *** atoth has joined #puppet-openstack | 18:10 | |
| *** tesseract has quit IRC | 18:14 | |
| *** armaan has quit IRC | 18:16 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Use openstacklib::wsgi::apache for keystone wsgi https://review.openstack.org/558887 | 18:16 |
| *** vnogin has joined #puppet-openstack | 18:24 | |
| *** vnogin has quit IRC | 18:28 | |
| *** dtantsur|brb is now known as dtantsur | 18:29 | |
| *** armaan has joined #puppet-openstack | 18:30 | |
| *** armaan has quit IRC | 18:35 | |
| *** armaan has joined #puppet-openstack | 18:36 | |
| *** amoralej is now known as amoralej|off | 18:51 | |
| *** dtantsur is now known as dtantsur|afk | 18:52 | |
| *** gfidente|afk has quit IRC | 18:56 | |
| *** agurenko has quit IRC | 19:11 | |
| *** atoth has quit IRC | 19:16 | |
| *** armaan has quit IRC | 19:31 | |
| openstackgerrit | Merged openstack/puppet-keystone master: Continue to define auth_uri https://review.openstack.org/558827 | 19:34 |
| *** vnogin has joined #puppet-openstack | 19:34 | |
| *** ykarel is now known as ykarel|away | 19:35 | |
| *** armaan has joined #puppet-openstack | 19:38 | |
| mwhahaha | yay it's finally merged | 19:43 |
| *** alee has quit IRC | 19:51 | |
| *** armaan has quit IRC | 19:54 | |
| *** colonwq has quit IRC | 20:05 | |
| *** alee has joined #puppet-openstack | 20:08 | |
| *** colonwq has joined #puppet-openstack | 20:20 | |
| *** vnogin has quit IRC | 20:21 | |
| *** ykarel|away has quit IRC | 20:25 | |
| *** vnogin has joined #puppet-openstack | 20:25 | |
| *** colonwq has quit IRC | 20:35 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Deprecate auth_uri option https://review.openstack.org/558451 | 20:35 |
| *** vnogin has quit IRC | 20:35 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Use openstacklib::wsgi::apache for keystone wsgi https://review.openstack.org/558887 | 20:42 |
| *** jtomasek_ has quit IRC | 20:44 | |
| zigo | mwhahaha: Thanks for the work! :) | 20:48 |
| * zigo does a recheck. | 20:48 | |
| zigo | That's one full day of work wasted because of this ... :/ | 20:48 |
| zigo | Oh, it passes ! :) | 20:48 |
| zigo | What's zhongshengping's IRC name? | 20:49 |
| zigo | Thanks for the recheck man. | 20:49 |
| zigo | I'd appreciate a +2 ... :P | 20:49 |
| zigo | EmilienM: https://review.openstack.org/#/c/558483/ would you mind? | 20:49 |
| zigo | Then I can try again the openstack-integration's ./all-in-one.sh script in my VM. | 20:50 |
| zigo | (once this one is merged) | 20:50 |
| EmilienM | there is a comment from mwhahaha https://review.openstack.org/#/c/558483/7/manifests/repo/debian/params.pp | 20:50 |
| mwhahaha | EmilienM: well i made the change :D just a thought | 20:50 |
| mwhahaha | we don't have to remove it | 20:50 |
| mwhahaha | but we could | 20:50 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-glance master: Add proxy support to glance_image resource https://review.openstack.org/558825 | 20:51 |
| zigo | EmilienM: I intend to fix this in a better way after it's fixed, because I do want to keep $deb_location in params.pp | 20:51 |
| zigo | ie: still continue to make it possible to override the default mirror. | 20:51 |
| zigo | (which currently defaults to stretch-queens.debian.net) | 20:52 |
| EmilienM | +2 | 20:52 |
| EmilienM | zigo: are you using Puppet? | 20:52 |
| mwhahaha | zigo: actually the params.pp is unnecessary because https://review.openstack.org/#/c/558483/7/manifests/repo/debian/debian.pp@50 | 20:52 |
| mwhahaha | we don't actually use that var which was my comment | 20:53 |
| zigo | EmilienM: Yeah, I'm currently working on building Infomaniak's public network with Stretch + Queens + puppet-openstack. | 20:54 |
| zigo | EmilienM: So far, the ./all-in-one.sh can install quite a lot of things, but fails because some stuff needs to use python3- instead of python-. | 20:55 |
| zigo | It shouldn't be hard to patch, and it's easy to test for me by tweaking the Puppetfile with the proposed patches. | 20:55 |
| zigo | (ie: not use master but ref/xxx/yyy) | 20:55 |
| zigo | And so far, it's been a lot of fun for me to learn puppet ! :) | 20:56 |
| zigo | After all these years, I finally have the opportunity to do it. | 20:56 |
| EmilienM | cool | 20:56 |
| EmilienM | yeah the modules are awesome | 20:56 |
| EmilienM | and I don't say it because... you know | 20:56 |
| zigo | Quite, indeed. | 20:56 |
| EmilienM | but really they are stable | 20:56 |
| EmilienM | people don't complain much | 20:56 |
| EmilienM | they do their job | 20:56 |
| EmilienM | and cost us low maintenance AFICT | 20:56 |
| zigo | I don't think they need a lot of work to make it all working. | 20:57 |
| zigo | How about upgrades? Is it smooth? | 20:57 |
| zigo | Is it gated? | 20:57 |
| mwhahaha | we don't really cover upgrades | 20:57 |
| mwhahaha | since a bunch of that is specific to the environment | 20:57 |
| mwhahaha | but upgrading the modules is pretty simple | 20:58 |
| mwhahaha | since we try to maintain backwards compatibility | 20:58 |
| mwhahaha | for at least one release | 20:58 |
| zigo | I have packaged all the modules in Debian, I don't know if you're aware of that. | 20:58 |
| zigo | To setup a puppet-master, I just "apt-get install openstack-puppet-modules", and it's done. | 20:58 |
| zigo | :) | 20:59 |
| EmilienM | why upgrading? it's cloud | 20:59 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Refactor package installations to bindep.txt https://review.openstack.org/511687 | 20:59 |
| EmilienM | just re-install | 20:59 |
| * EmilienM leaves | 20:59 | |
| zigo | What I intend to do is to also provide hiera roles. | 20:59 |
| zigo | Just bundle everything, then people will just have to define hostnames and roles, start a script to generate passwords, then the cluster will automatically get installed. | 21:00 |
| zigo | That's my current plan, at least. | 21:00 |
| zigo | Hopefully, I'll get there. | 21:00 |
| *** rcernin has joined #puppet-openstack | 21:00 | |
| mwhahaha | magic :D | 21:01 |
| zigo | :P | 21:01 |
| zigo | EmilienM: Why upgrading: because it's a public cloud with already existing workload, Pushing customers to move from one cloud to another is too painful. | 21:02 |
| zigo | But probably we'll do that, yeah. | 21:02 |
| EmilienM | I was joking. | 21:02 |
| zigo | EmilienM: Yeah, though I still don't have an answer to my question: is upgrading from one release to the next is gated in infra? | 21:04 |
| zigo | We'll end up doing upgrade tests on our dev cloud before doing the production upgrade, of course, but still... | 21:05 |
| zigo | So, what's zhongshengping's IRC nick? :) | 21:06 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet https://review.openstack.org/510651 | 21:11 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet https://review.openstack.org/510651 | 21:17 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS https://review.openstack.org/510652 | 21:17 |
| zigo | mwhahaha: How do I fix this? http://logs.openstack.org/33/558733/1/check/puppet-openstack-unit-4.8-centos-7/20d6f3f/job-output.txt.gz#_2018-04-04_08_23_19_259771 | 21:26 |
| zigo | ie: how do I make facts available from the unit test? | 21:26 |
| mwhahaha | depends on the test | 21:27 |
| zigo | Is it the topmost: require 'spec_helper' ? | 21:27 |
| zigo | mwhahaha: spec/classes/horizon_init_spec.rb in puppet-neutron | 21:27 |
| zigo | Sorry, no. | 21:28 |
| zigo | spec/classes/neutron_db_spec.rb in puppet-neutron | 21:28 |
| mwhahaha | so debian is not part of our default tests | 21:28 |
| zigo | ie: https://review.openstack.org/#/c/558733/1/spec/classes/neutron_db_spec.rb | 21:28 |
| mwhahaha | line 96-99 | 21:29 |
| mwhahaha | those are where the facts come in | 21:29 |
| mwhahaha | but what you're attempting to do isn't going to work in the structure of those tests | 21:29 |
| mwhahaha | many of those tests need to get updated to our newer syntax | 21:29 |
| *** rcernin has quit IRC | 21:29 | |
| zigo | Hum... | 21:29 |
| mwhahaha | https://review.openstack.org/#/c/558733/1/spec/classes/neutron_plugins_ml2_bagpipe_spec.rb@51 | 21:29 |
| mwhahaha | we use rspec puppet facts to automatically populate the tests | 21:30 |
| zigo | mwhahaha: The important bit is this: https://review.openstack.org/#/c/558733/1/manifests/params.pp | 21:30 |
| mwhahaha | and run through the rhel/ubuntu tests | 21:30 |
| zigo | I was scared it would break if I just push that. | 21:30 |
| mwhahaha | you'd need to include debian in the supported os | 21:30 |
| mwhahaha | well if the tests work, you wouldn't be breaking anything | 21:30 |
| mwhahaha | :D | 21:30 |
| zigo | Right. | 21:31 |
| mwhahaha | however i think you are improperly changing things | 21:31 |
| zigo | So, I'll revert these changes | 21:31 |
| mwhahaha | we won't have os_package_type == debian under redhat | 21:31 |
| zigo | You'll have os_package_type == rpm | 21:32 |
| zigo | Which is fine. | 21:32 |
| mwhahaha | i'll comment on the review | 21:32 |
| zigo | This was discussed and done in the Hongkong summit. | 21:32 |
| mwhahaha | right | 21:32 |
| mwhahaha | but there's already an os family block | 21:32 |
| zigo | This makes it possible to run Debian packages on top of Ubuntu. | 21:32 |
| * mwhahaha is commenting | 21:32 | |
| mwhahaha | you just have a single bad change in that | 21:32 |
| zigo | Ah... | 21:33 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Use openstacklib::wsgi::apache for keystone wsgi https://review.openstack.org/558887 | 21:33 |
| * zigo re-read the change | 21:33 | |
| mwhahaha | commented | 21:34 |
| mwhahaha | at least on params | 21:34 |
| mwhahaha | now for the tests i think you'd need to actually rewrite a bunch of the tests to properly test the debian/ubuntu changes | 21:35 |
| mwhahaha | we might have some logic for the ml2 stuff previously | 21:35 |
| mwhahaha | that you could try and reuse | 21:35 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Use openstacklib::wsgi::apache for keystone wsgi https://review.openstack.org/558887 | 21:36 |
| zigo | mwhahaha: Actually, I have just uploaded networking-baremetal to Sid ! :) | 21:36 |
| zigo | Because I tried Ironic, and it is now required. | 21:36 |
| zigo | It's sitting in the ftpmaster NEW queue for review. | 21:36 |
| zigo | mwhahaha: And also, there's currently no way Ironic can work in Ubuntu (like... at all !!!). | 21:36 |
| mwhahaha | :( | 21:36 |
| zigo | Bionic will simply not have support for it. | 21:37 |
| zigo | They do MaaS you know... | 21:37 |
| mwhahaha | not sure we test it under ubuntu | 21:37 |
| zigo | They don't care about Ironic. | 21:37 |
| zigo | Impossible that you do. | 21:37 |
| zigo | There's just no package that would support it. | 21:37 |
| zigo | I checked for that last week. | 21:37 |
| zigo | And fixed so many things for Ironic to work in Debian. | 21:38 |
| zigo | Unfortunately, you can't ask Ubuntu to just "sync from Sid", because I've switched all to Python 3. | 21:38 |
| zigo | Also, Biionic will be out in a few weeks... | 21:38 |
| zigo | Sad story... | 21:39 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Use openstacklib::wsgi::apache for keystone wsgi https://review.openstack.org/558887 | 21:41 |
| *** colonwq has joined #puppet-openstack | 21:44 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: ACK Debian's switch to Python 3 for Neutron https://review.openstack.org/558733 | 21:46 |
| *** alee has quit IRC | 21:50 | |
| zigo | Oh, sorry, just understood your comment about networking-baremetal | 21:51 |
| zigo | :) | 21:51 |
| mwhahaha | i think you still missed it, let me comment again | 21:52 |
| mwhahaha | 106-110 shouldn't be there | 21:52 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: ACK Debian's switch to Python 3 for Neutron https://review.openstack.org/558733 | 21:54 |
| mwhahaha | i posted an alternative on ps2 | 21:57 |
| *** morazi has quit IRC | 22:02 | |
| zigo | mwhahaha: I very much like your idea of the $ver_mod thing, that will make everything so much easier to read. | 22:08 |
| mwhahaha | yea | 22:08 |
| mwhahaha | it's also something we might need to do for python3 under fedora as well at some point | 22:08 |
| * zigo scraps his parrams.pp and restarts from scratch. | 22:10 | |
| zigo | mwhahaha: Wouldn't it be nicer to just have a $pyvers variable set globally as a fact? ie: ${::pyvers}. | 22:15 |
| zigo | Or something like that ... | 22:16 |
| mwhahaha | ehhhh not necessarily | 22:16 |
| mwhahaha | but that could work as well | 22:16 |
| zigo | In openstacklib or something. | 22:16 |
| mwhahaha | the problem is that it's not 100% | 22:16 |
| zigo | mwhahaha: Well, neutron is the first one that I'm fixing, but there will be others. | 22:16 |
| mwhahaha | yea | 22:16 |
| mwhahaha | i'm sure | 22:16 |
| mwhahaha | we went through and actually removed the version from the package a while back | 22:16 |
| mwhahaha | anyway | 22:17 |
| * mwhahaha wanders off | 22:17 | |
| zigo | mwhahaha: I can first try like you suggested, see how it goes with neutron (which was failing the openstack-integration script), and then attempt a more global thing... | 22:17 |
| zigo | This wont go away in Debian. | 22:17 |
| zigo | Not before Py2 is killed. | 22:17 |
| zigo | And that will be ... in 3 decades? :) | 22:17 |
| zigo | (hopefully, not...) | 22:17 |
| *** alee has joined #puppet-openstack | 22:24 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: ACK Debian's switch to Python 3 for Neutron https://review.openstack.org/558733 | 22:30 |
| * zigo goes to sleep | 22:31 | |
| *** vnogin has joined #puppet-openstack | 22:36 | |
| *** vnogin has quit IRC | 22:41 | |
| *** rcernin has joined #puppet-openstack | 22:45 | |
| *** colonwq has quit IRC | 23:47 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
