*** markvoelker_ has quit IRC | 00:10 | |
*** gildub has joined #puppet-openstack | 00:38 | |
*** zhangjn has joined #puppet-openstack | 00:58 | |
*** Guest56608 is now known as saneax_AFK | 01:07 | |
*** markvoelker has joined #puppet-openstack | 01:10 | |
*** markvoelker has quit IRC | 01:15 | |
*** baker has joined #puppet-openstack | 01:25 | |
*** baker has quit IRC | 01:33 | |
*** saneax_AFK is now known as saneax | 01:33 | |
*** saneax is now known as Guest26785 | 01:34 | |
*** baker has joined #puppet-openstack | 01:39 | |
*** baker has quit IRC | 01:41 | |
*** baker has joined #puppet-openstack | 01:42 | |
*** baker has quit IRC | 01:42 | |
*** baker has joined #puppet-openstack | 01:45 | |
*** baker has quit IRC | 01:46 | |
*** zhangjn has quit IRC | 02:06 | |
*** zhangjn has joined #puppet-openstack | 02:13 | |
*** markvoelker has joined #puppet-openstack | 03:11 | |
*** markvoelker has quit IRC | 03:16 | |
*** zhangjn has quit IRC | 03:56 | |
*** Guest26785 is now known as saneax_AFK | 04:25 | |
*** zhangjn has joined #puppet-openstack | 04:35 | |
*** mgagne has quit IRC | 04:36 | |
*** mentat has quit IRC | 04:37 | |
*** mgagne has joined #puppet-openstack | 04:37 | |
*** mgagne is now known as Guest73233 | 04:37 | |
*** mentat has joined #puppet-openstack | 04:38 | |
*** markvoelker has joined #puppet-openstack | 05:12 | |
*** markvoelker has quit IRC | 05:17 | |
*** zhangjn has quit IRC | 05:19 | |
*** zhangjn has joined #puppet-openstack | 05:21 | |
*** saneax_AFK is now known as saneax | 05:25 | |
*** saneax is now known as Guest88888 | 05:26 | |
*** zhangjn has quit IRC | 05:59 | |
*** zhangjn has joined #puppet-openstack | 06:01 | |
*** gomarivera has quit IRC | 06:04 | |
*** zhangjn has quit IRC | 06:41 | |
*** zhangjn has joined #puppet-openstack | 06:41 | |
*** gildub has quit IRC | 06:49 | |
*** zhangjn has quit IRC | 06:49 | |
*** zhangjn has joined #puppet-openstack | 06:57 | |
*** zhangjn has quit IRC | 06:57 | |
*** zhangjn has joined #puppet-openstack | 06:58 | |
*** mjb has quit IRC | 07:00 | |
*** rcernin has joined #puppet-openstack | 07:01 | |
*** zhangjn has quit IRC | 07:03 | |
*** zhangjn has joined #puppet-openstack | 07:04 | |
*** mjb has joined #puppet-openstack | 07:05 | |
*** openstackstatus has joined #puppet-openstack | 07:06 | |
*** ChanServ sets mode: +v openstackstatus | 07:06 | |
*** openstackgerrit has joined #puppet-openstack | 07:10 | |
*** markvoelker has joined #puppet-openstack | 07:13 | |
*** markvoelker has quit IRC | 07:17 | |
openstackgerrit | Andrew Spiers proposed openstack/puppet-designate: Use actual service names for RH osfamily https://review.openstack.org/257175 | 07:20 |
---|---|---|
*** rmart04 has joined #puppet-openstack | 07:26 | |
openstackgerrit | Andrew Spiers proposed openstack/puppet-designate: Use actual service names for RH osfamily https://review.openstack.org/257175 | 07:47 |
*** foxhunt_away is now known as foxhunt72 | 08:05 | |
*** gchamoul- is now known as gchamoul | 08:18 | |
*** jpena has joined #puppet-openstack | 08:31 | |
*** paramite has joined #puppet-openstack | 08:35 | |
*** paramite has quit IRC | 08:43 | |
openstackgerrit | Mykyta Karpin proposed openstack/puppet-tempest: CI test - do not merge https://review.openstack.org/257221 | 08:49 |
*** gfidente has joined #puppet-openstack | 08:53 | |
*** gfidente has quit IRC | 08:53 | |
*** gfidente has joined #puppet-openstack | 08:53 | |
openstackgerrit | Michael Chapman proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller https://review.openstack.org/249390 | 08:55 |
*** samuelBartel has joined #puppet-openstack | 08:55 | |
*** darkxploit has quit IRC | 08:55 | |
*** zhangjn has quit IRC | 09:00 | |
*** zhangjn has joined #puppet-openstack | 09:03 | |
*** darkxploit has joined #puppet-openstack | 09:07 | |
*** jistr has joined #puppet-openstack | 09:12 | |
*** markvoelker has joined #puppet-openstack | 09:14 | |
*** markvoelker has quit IRC | 09:18 | |
openstackgerrit | Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router https://review.openstack.org/255913 | 09:20 |
*** rochaporto has joined #puppet-openstack | 09:26 | |
*** leanderthal has joined #puppet-openstack | 09:26 | |
*** admin0 has joined #puppet-openstack | 09:30 | |
*** morazi has joined #puppet-openstack | 09:30 | |
*** rahulait has joined #puppet-openstack | 09:41 | |
*** _skolekonov is now known as skolekonov | 09:41 | |
*** leanderthal has quit IRC | 09:42 | |
*** rmart04_ has joined #puppet-openstack | 09:47 | |
*** rmart04 has quit IRC | 09:48 | |
*** rmart04_ is now known as rmart04 | 09:48 | |
*** zhangjn has quit IRC | 09:56 | |
*** zhangjn has joined #puppet-openstack | 10:01 | |
*** zhangjn has quit IRC | 10:01 | |
*** leanderthal has joined #puppet-openstack | 10:01 | |
*** zhangjn has joined #puppet-openstack | 10:03 | |
*** jtomasek has joined #puppet-openstack | 10:06 | |
openstackgerrit | Andrey Shestakov proposed openstack/puppet-neutron: Add 'distributed' parameter for neutron_router https://review.openstack.org/255913 | 10:08 |
*** davideagle__ has joined #puppet-openstack | 10:10 | |
*** paramite has joined #puppet-openstack | 10:16 | |
*** leanderthal has quit IRC | 10:21 | |
*** zeus` has quit IRC | 10:23 | |
*** rcernin has quit IRC | 10:25 | |
*** pleia2 has quit IRC | 10:32 | |
*** _degorenko|afk is now known as degorenko | 10:37 | |
*** samuelBartel has quit IRC | 10:38 | |
*** mgrosz has joined #puppet-openstack | 10:41 | |
mgrosz | Hello everyone, can some one please review and +2 and +workflow change https://review.openstack.org/#/c/255431/? Thanks | 10:42 |
*** rleander has joined #puppet-openstack | 10:43 | |
*** omartsyniuk has joined #puppet-openstack | 10:47 | |
*** rleander has quit IRC | 10:49 | |
*** leanderthal has joined #puppet-openstack | 10:57 | |
*** devvesa has joined #puppet-openstack | 11:01 | |
*** IvanBerezovskiy has joined #puppet-openstack | 11:01 | |
*** rleander has joined #puppet-openstack | 11:04 | |
*** zhangjn has quit IRC | 11:07 | |
*** rmart04 has quit IRC | 11:10 | |
*** zhangjn has joined #puppet-openstack | 11:13 | |
*** markvoelker has joined #puppet-openstack | 11:15 | |
*** zhangjn has quit IRC | 11:18 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: Ability to configure api, collector and notification workers https://review.openstack.org/226011 | 11:19 |
*** markvoelker has quit IRC | 11:20 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-glance: Remove POSIX users, groups, and file modes. https://review.openstack.org/211593 | 11:21 |
*** rahulait has quit IRC | 11:22 | |
devvesa | Hi, anyone knows why https://review.openstack.org/#/c/251319/ | 11:25 |
devvesa | some reviews are failing on puppet tests? | 11:25 |
devvesa | It is something introduced since December 12th | 11:26 |
*** mgrosz has quit IRC | 11:28 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-ceilometer: Remove POSIX users and groups https://review.openstack.org/195135 | 11:28 |
*** mgrosz has joined #puppet-openstack | 11:30 | |
*** admin0 has quit IRC | 11:37 | |
*** leanderthal has quit IRC | 11:43 | |
*** derekh has joined #puppet-openstack | 11:47 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 11:53 |
*** slagle has quit IRC | 11:53 | |
*** slagle has joined #puppet-openstack | 11:54 | |
*** mgrosz has quit IRC | 11:58 | |
*** mgrosz has joined #puppet-openstack | 12:01 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 12:07 |
*** admin0 has joined #puppet-openstack | 12:07 | |
*** dgurtner has joined #puppet-openstack | 12:09 | |
*** dgurtner has quit IRC | 12:09 | |
*** dgurtner has joined #puppet-openstack | 12:09 | |
EmilienM | good morning | 12:22 |
devvesa | Hi EmilienM: Good morning | 12:24 |
devvesa | I noticed that suddenly my gate-puppet jobs failed | 12:24 |
devvesa | Look this one since Dec 13th: https://review.openstack.org/#/c/251319/ | 12:25 |
EmilienM | I'm looking | 12:25 |
devvesa | This is another one in puppet-midonet | 12:25 |
devvesa | sorry | 12:25 |
devvesa | puppet-neutron | 12:25 |
EmilienM | this one is a bad example, it's not a puppet openstack module but tripleo, they way it's tested is different | 12:25 |
devvesa | https://review.openstack.org/#/c/255913/ | 12:25 |
EmilienM | but there is something with psych, indeed | 12:26 |
devvesa | Yes. I solved in puppet-tripleo with this patch | 12:26 |
devvesa | https://review.openstack.org/#/c/257294/ | 12:26 |
devvesa | I don't know the reason why they started to fail | 12:27 |
devvesa | Another example: https://review.openstack.org/#/c/257175/ | 12:27 |
EmilienM | sbadia: I think we also need to add the dependency in our modules | 12:28 |
EmilienM | sbadia: can you double check too? I think it's ok | 12:28 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: CI test - never merge https://review.openstack.org/257312 | 12:29 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Adding psych on Gemfile explicitly https://review.openstack.org/257317 | 12:33 |
EmilienM | devvesa: doing some testing in puppet openstack modules | 12:33 |
devvesa | EmilienM: your test one already has failed | 12:34 |
devvesa | https://jenkins05.openstack.org/job/gate-puppet-tripleo-puppet-unit-3.3-dsvm-centos7/26/console | 12:34 |
devvesa | sorry, bad link | 12:34 |
devvesa | https://jenkins06.openstack.org/job/gate-puppet-neutron-puppet-syntax-3-dsvm-centos7/99/console | 12:35 |
EmilienM | devvesa: taking a coffee and I'm back ;-) | 12:36 |
* devvesa lunch time | 12:37 | |
devvesa | :) | 12:37 |
*** dprince has joined #puppet-openstack | 12:38 | |
*** trown|outttypeww is now known as trown | 12:38 | |
*** ddmitriev has joined #puppet-openstack | 12:40 | |
*** ducttape_ has joined #puppet-openstack | 12:44 | |
*** zhangjn has joined #puppet-openstack | 12:45 | |
*** markvoelker has joined #puppet-openstack | 12:45 | |
mgrosz | @EmilienM: Good Morning! | 12:49 |
*** markvoelker has quit IRC | 12:49 | |
sbadia | hi here | 12:51 |
sbadia | EmilienM: ok, I take a look | 12:51 |
*** IvanBerezovskiy has quit IRC | 12:55 | |
*** ducttape_ has quit IRC | 13:00 | |
*** zhangjn has quit IRC | 13:06 | |
sbadia | EmilienM: hum, only two gems are changed between the two builds (https://review.openstack.org/#/c/257175/ (fail) and https://review.openstack.org/#/c/255393/ (ok)) | 13:09 |
sbadia | EmilienM: specinfra 2.44.7 → specinfra 2.45.0 and bundler 1.10.6 → bundler 1.11.0 | 13:09 |
sbadia | (maybe rubygem itself but I'm not sure, this message isn't present on the « ok » test http://logs.openstack.org/75/257175/2/check/gate-puppet-designate-puppet-syntax-3-dsvm-centos7/f0e38b7/console.html#_2015-12-14_07_47_52_179 ) | 13:10 |
devvesa | sbadia: I would say it is the bundle version | 13:11 |
devvesa | *bundler | 13:11 |
devvesa | https://github.com/bundler/bundler/issues/2068 | 13:11 |
sbadia | indeed :/ | 13:11 |
sbadia | let's pin bundler version \o/ | 13:12 |
sbadia | devvesa: you already made the patch? | 13:12 |
devvesa | That's the other option :) | 13:12 |
devvesa | I made one for puppet-tripleo | 13:12 |
devvesa | https://review.openstack.org/#/c/257294/ | 13:12 |
sbadia | ok! | 13:12 |
*** dgurtner has quit IRC | 13:14 | |
EmilienM | it seems we have 2 options, yes | 13:14 |
EmilienM | I'm in favor of keeping bundle from latest release | 13:14 |
EmilienM | and add psych as a new dep | 13:14 |
EmilienM | devvesa, sbadia, like https://review.openstack.org/#/c/257317/ (look in the code my commend) | 13:15 |
EmilienM | comment* | 13:15 |
sbadia | EmilienM: hum… | 13:15 |
*** jayg|g0n3 is now known as jayg | 13:16 | |
sbadia | this is a bundler bug, let's patch it in project-config (top wide) | 13:16 |
sbadia | in order to not add this in all our projects… | 13:16 |
devvesa | sbadia: In that case, someone could experience the bug locally and once you know it, you would push patches blindly... | 13:19 |
*** IvanBerezovskiy has joined #puppet-openstack | 13:19 | |
sbadia | yep yep sure! | 13:19 |
EmilienM | sbadia: if you patch the fix in project-config (where by the way?), people who run tests locally would fail | 13:21 |
EmilienM | I think we have to fix all Gemfiles | 13:23 |
sbadia | or fix bundler :D | 13:23 |
sbadia | https://github.com/bundler/bundler/issues/4149 | 13:23 |
EmilienM | sbadia: it's fixed but not released iiuc | 13:24 |
devvesa | According to the issue, it will be released on bundle 1.12 | 13:24 |
EmilienM | and we don't know when that happens | 13:25 |
devvesa | I have to wait some reviews on tripleo patches, if you have a spreadsheet or something with the list of puppet modules where to apply | 13:26 |
devvesa | I can help you to push patches in all of them | 13:26 |
mgrosz | @EmilienM: hey. Any chances my puppet-vitrage will be merged today? | 13:29 |
sbadia | devvesa: oh thanks! we can maybe use modulesync for that no? https://github.com/openstack/puppet-modulesync-configs/blob/master/config_defaults.yml | 13:31 |
sbadia | (for the list https://github.com/openstack/puppet-modulesync-configs/blob/master/managed_modules.yml) | 13:32 |
devvesa | sbadia: Oh, I didn't know this project! How does it work? | 13:33 |
sbadia | it's magic :-) | 13:33 |
sbadia | sorry, no it use the tool « modulesync » | 13:33 |
sbadia | https://github.com/puppet-community/modulesync | 13:34 |
devvesa | Being magic is good for good stuff but horrible for bad ones :) | 13:34 |
sbadia | ;-) | 13:34 |
*** igajsin has joined #puppet-openstack | 13:36 | |
*** igajsin has left #puppet-openstack | 13:36 | |
*** zhangjn has joined #puppet-openstack | 13:41 | |
openstackgerrit | Sebastien Badia proposed openstack/puppet-modulesync-configs: Add new managed modules https://review.openstack.org/257357 | 13:43 |
*** zhangjn has quit IRC | 13:44 | |
*** zhangjn has joined #puppet-openstack | 13:45 | |
EmilienM | sbadia: monasca? | 13:45 |
EmilienM | not sure monasca is compliant | 13:46 |
EmilienM | the msync will change a lot of things I think | 13:46 |
*** jaosorior has joined #puppet-openstack | 13:47 | |
*** dslevin_ has joined #puppet-openstack | 13:55 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters https://review.openstack.org/255869 | 13:56 |
devvesa | I have a question regarding on puppet-modulesync-configs: Should I push the change on this module and it will be applied automatically to others with a CI job? | 13:56 |
EmilienM | devvesa: sbadia is working on it | 13:56 |
devvesa | Oh... I finished mine already :) | 13:57 |
*** zeus has joined #puppet-openstack | 13:57 | |
devvesa | (with the --noop) | 13:57 |
devvesa | Ok, all yours sbadia :) | 13:57 |
*** zeus is now known as Guest83257 | 13:57 | |
sbadia | devvesa: ok cool then! | 13:58 |
sbadia | you already patched all our modules? | 13:58 |
devvesa | BTW, EmilienM: Do you have +2 permissions on puppet-tripleo? | 13:58 |
sbadia | devvesa: yep (me too) | 13:58 |
EmilienM | devvesa: yes | 13:58 |
*** Guest83257 is now known as zeus- | 13:58 | |
*** zeus- is now known as zeus` | 13:58 | |
devvesa | Can you merge the puppet-tripleo one, then? | 13:59 |
devvesa | https://review.openstack.org/257294 | 13:59 |
devvesa | sbadia: Yes. I already patches all the modules | 13:59 |
EmilienM | devvesa: I would like to see an inline comment | 13:59 |
devvesa | EmilienM: ok | 13:59 |
sbadia | and a unique topic | 13:59 |
EmilienM | devvesa: see https://review.openstack.org/257317 | 13:59 |
*** jpena is now known as jpena|lunch | 14:00 | |
sbadia | devvesa: then do not hesitate to submit your patchs | 14:00 |
sbadia | it's no use that we will both manage that, I leave you manage in this case. | 14:00 |
michchap | psych issue hit me today | 14:00 |
michchap | I thought it was interesting it only affected the versions < 4.0.0 | 14:01 |
*** zhangjn has quit IRC | 14:02 | |
*** richm has joined #puppet-openstack | 14:02 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 14:02 |
devvesa | sbadia: I have not dared to run the process without the --noop. Will I create a patch for each project? Is this what it is expected to happen? | 14:03 |
sbadia | devvesa: this case is a bit special (because we need to add a comment inline) | 14:04 |
sbadia | we can maybe manage this the old way using a loop on managed module (in order to do that quickly) | 14:05 |
sbadia | devvesa: do you want to split the module, or it's ok? | 14:05 |
devvesa | Uhm, ok. I think I give it to you. I don't want to mess up with all the modules... | 14:05 |
*** zhangjn has joined #puppet-openstack | 14:05 | |
EmilienM | sbadia: hopefully Bundle will provide a release soon | 14:06 |
EmilienM | because it also break stable/* CI | 14:06 |
sbadia | devvesa: ok, ok, let's go then | 14:06 |
jaosorior | richm: Hey man, just saw the CR you sent me: https://review.openstack.org/#/c/256248/ I think I could use that for what I need. | 14:07 |
EmilienM | sbadia: what do we do for stable branches? | 14:07 |
jaosorior | richm: Instead of wiring an exec with sed. Which would work, but somehow I like this better | 14:08 |
sbadia | EmilienM: the latest bundler is always fetched even in stable branch | 14:08 |
EmilienM | sbadia: yeah, that's what I'm saying: stable/liberty, kilo etc are broken too | 14:08 |
sbadia | EmilienM: we can maybe query to bundler team when the release come? | 14:09 |
EmilienM | sbadia: what should we do? | 14:09 |
EmilienM | sbadia ++ | 14:09 |
*** guimaluf has quit IRC | 14:11 | |
*** paramite is now known as paramite|afk | 14:11 | |
*** paramite|afk is now known as paramite | 14:12 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency https://review.openstack.org/251319 | 14:15 |
*** admin0 has quit IRC | 14:16 | |
*** Guest88888 is now known as saneax_AFK | 14:20 | |
*** Vinsh has quit IRC | 14:22 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters https://review.openstack.org/255869 | 14:22 |
*** zhangjn has quit IRC | 14:25 | |
openstackgerrit | Merged openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257294 | 14:26 |
sbadia | spam ready :) | 14:29 |
* EmilienM ready for review | 14:29 | |
openstackgerrit | Sebastien Badia proposed openstack/puppet-aodh: Adding psych on Gemfile explicitly https://review.openstack.org/257379 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-barbican: Adding psych on Gemfile explicitly https://review.openstack.org/257381 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-ceilometer: Adding psych on Gemfile explicitly https://review.openstack.org/257382 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-ceph: Adding psych on Gemfile explicitly https://review.openstack.org/257383 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-cinder: Adding psych on Gemfile explicitly https://review.openstack.org/257384 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-designate: Adding psych on Gemfile explicitly https://review.openstack.org/257385 | 14:29 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-glance: Adding psych on Gemfile explicitly https://review.openstack.org/257386 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-gnocchi: Adding psych on Gemfile explicitly https://review.openstack.org/257387 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-heat: Adding psych on Gemfile explicitly https://review.openstack.org/257388 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-horizon: Adding psych on Gemfile explicitly https://review.openstack.org/257389 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-ironic: Adding psych on Gemfile explicitly https://review.openstack.org/257390 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-keystone: Adding psych on Gemfile explicitly https://review.openstack.org/257391 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-magnum: Adding psych on Gemfile explicitly https://review.openstack.org/257392 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-manila: Adding psych on Gemfile explicitly https://review.openstack.org/257393 | 14:30 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-mistral: Adding psych on Gemfile explicitly https://review.openstack.org/257394 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-monasca: Adding psych on Gemfile explicitly https://review.openstack.org/257395 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-murano: Adding psych on Gemfile explicitly https://review.openstack.org/257396 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-nova: Adding psych on Gemfile explicitly https://review.openstack.org/257397 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-openstacklib: Adding psych on Gemfile explicitly https://review.openstack.org/257398 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-sahara: Adding psych on Gemfile explicitly https://review.openstack.org/257399 | 14:31 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-swift: Adding psych on Gemfile explicitly https://review.openstack.org/257400 | 14:32 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-tempest: Adding psych on Gemfile explicitly https://review.openstack.org/257401 | 14:32 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-trove: Adding psych on Gemfile explicitly https://review.openstack.org/257402 | 14:32 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-tuskar: Adding psych on Gemfile explicitly https://review.openstack.org/257404 | 14:32 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-vswitch: Adding psych on Gemfile explicitly https://review.openstack.org/257405 | 14:33 |
openstackgerrit | Sebastien Badia proposed openstack/puppet-zaqar: Adding psych on Gemfile explicitly https://review.openstack.org/257406 | 14:33 |
EmilienM | sbadia: monasca is ok | 14:34 |
EmilienM | sbadia: oh I actually have another idea | 14:35 |
EmilienM | since it's only temporary | 14:35 |
EmilienM | and it's only for unit tests: maybe we can only patch https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules_unit.sh | 14:35 |
EmilienM | sbadia: or is it also affecting syntax? | 14:36 |
EmilienM | I think we can avoid all these patches | 14:36 |
EmilienM | 1/ patch https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules_unit.sh and install the gem manually | 14:36 |
EmilienM | 2/ patch project-config in syntax jobs and add the gem manually too | 14:36 |
EmilienM | that way, we only have 2 patches, it easily backportable and people can still run tests locally | 14:37 |
EmilienM | devvesa, sbadia: wdyt? | 14:37 |
sbadia | https://review.openstack.org/#/q/topic:bug/1525929,n,z | 14:37 |
sbadia | (sorry scrum meeting right now) | 14:37 |
EmilienM | sbadia: I think we can avoid that in fact | 14:38 |
sbadia | EmilienM: it's also syntax indeed | 14:38 |
EmilienM | yeah but we can fix project-config for that one | 14:38 |
EmilienM | (syntax) | 14:38 |
EmilienM | I'm going to test it | 14:38 |
sbadia | and for the local tests (devlopper laptop) | 14:39 |
*** trown is now known as trown|outttypeww | 14:40 | |
*** samuelBartel has joined #puppet-openstack | 14:40 | |
*** dgurtner has joined #puppet-openstack | 14:41 | |
*** dgurtner has quit IRC | 14:41 | |
*** dgurtner has joined #puppet-openstack | 14:41 | |
*** markvoelker has joined #puppet-openstack | 14:43 | |
iurygregory | hello people o/ | 14:44 |
EmilienM | sbadia: do you think people run syntax tests? | 14:45 |
sbadia | EmilienM: yes :) | 14:45 |
devvesa | Sorry, I don't get it. Why people will be able to run locally? | 14:46 |
devvesa | is puppet-openstack-integration a common dependency? | 14:46 |
EmilienM | yes | 14:46 |
devvesa | Ok, +1 then :) | 14:46 |
EmilienM | see Rakefile | 14:46 |
EmilienM | having 26 patches each time is really heavy | 14:47 |
EmilienM | I would avoid it when possible | 14:47 |
iurygregory | i run syntax tests before send =D | 14:47 |
EmilienM | we need to poke Bundler folks to provide a release | 14:48 |
devvesa | Agree. But as far as I understood, the problem is not the pysch is not installed | 14:48 |
sbadia | … let's abandon all this patchs then… | 14:48 |
EmilienM | sbadia: no! | 14:48 |
EmilienM | sbadia: just 'pause' | 14:48 |
devvesa | But is not installed by bundle | 14:48 |
EmilienM | sbadia: we might need your patches | 14:48 |
EmilienM | I'm just looking for a simple wway | 14:49 |
EmilienM | because we will have this issue again for another dep one day | 14:49 |
EmilienM | and we need a scalable solution | 14:49 |
EmilienM | patching 26 modules (+ backports so ~100 patches) is not scalable | 14:49 |
sbadia | I didn't remember the git clone of integration repo on local laptops | 14:49 |
EmilienM | of course it does | 14:49 |
EmilienM | sbadia: read Rakefile ;-) | 14:49 |
sbadia | but it's the best way to manage that… | 14:49 |
sbadia | so patching all modules is not needed | 14:50 |
EmilienM | you +2'ed the patches | 14:50 |
sbadia | we juste need to patch integration repo for local and project-config for openstack infra | 14:50 |
sbadia | and that it | 14:50 |
EmilienM | sbadia: yeah but people who run syntax on their laptop will be broken | 14:50 |
*** dh64 has quit IRC | 14:50 | |
EmilienM | but that's not related to our modules | 14:50 |
EmilienM | they can install the gem manually I guess? | 14:51 |
sbadia | yes maybe | 14:51 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters https://review.openstack.org/255869 | 14:52 |
*** zhangjn has joined #puppet-openstack | 14:52 | |
*** zhangjn has quit IRC | 14:53 | |
*** ducttape_ has joined #puppet-openstack | 14:55 | |
openstackgerrit | Jaume Devesa proposed openstack/puppet-tripleo: Adding psych on Gemfile explicitly https://review.openstack.org/257420 | 14:56 |
*** xarses has quit IRC | 14:56 | |
EmilienM | sbadia: do not abandon your patches yet | 14:57 |
devvesa | Since puppet-tripleo is outside the puppet-openstack modules, I've backported to stable/liberty the master's patch ^ | 14:57 |
richm | jaosorior: yeah, just trying to figure out some alternatives for you to use | 15:00 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Adding psych on Gemfile explicitly https://review.openstack.org/257421 | 15:01 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Test https://review.openstack.org/257421 https://review.openstack.org/257422 | 15:02 |
EmilienM | sbadia: ^ if the test pass, I think we can go ahead with the change, but let me know what do you think | 15:02 |
*** imcsk8_ is now known as imcsk8 | 15:03 | |
*** zhangjn has joined #puppet-openstack | 15:07 | |
*** jpena|lunch is now known as jpena | 15:08 | |
mwhahaha | ugh looks like it broke the fuel syntax checks as well | 15:09 |
*** bklei has joined #puppet-openstack | 15:09 | |
EmilienM | mwhahaha: mouahahahahah | 15:11 |
mwhahaha | Happy Monday! | 15:11 |
EmilienM | (I had too) | 15:11 |
openstackgerrit | Michael Polenchuk proposed openstack/puppet-keystone: Make request by service token if got unauth error https://review.openstack.org/251490 | 15:17 |
EmilienM | 166 jobs in the gate, we have time to wait :) | 15:20 |
*** dfisher has joined #puppet-openstack | 15:21 | |
mwhahaha | hmm i don't get the psych error locally | 15:22 |
EmilienM | maybe because you already have the gem | 15:23 |
EmilienM | mwhahaha: gem list | grep psych? | 15:23 |
mwhahaha | ah yes because it's a default gem on my system | 15:26 |
mwhahaha | blah | 15:26 |
jaosorior | richm: Thanks dude | 15:28 |
*** baker has joined #puppet-openstack | 15:30 | |
*** migi has quit IRC | 15:32 | |
*** migi has joined #puppet-openstack | 15:33 | |
*** baker has quit IRC | 15:34 | |
*** dslevin_ has quit IRC | 15:36 | |
*** xarses has joined #puppet-openstack | 15:42 | |
*** admin0 has joined #puppet-openstack | 15:43 | |
*** mdorman has joined #puppet-openstack | 15:47 | |
*** baker has joined #puppet-openstack | 15:48 | |
*** imcsk8 is now known as imcsk8|gym | 15:55 | |
*** rleander is now known as leanderthal|afk | 15:55 | |
*** dhollinger has joined #puppet-openstack | 16:02 | |
*** zhangjn has quit IRC | 16:03 | |
*** jistr has quit IRC | 16:03 | |
*** mgrosz has quit IRC | 16:04 | |
*** morazi has quit IRC | 16:06 | |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Fix incorrect default metadata_dir and api_url parameters https://review.openstack.org/255869 | 16:13 |
*** admin0 has quit IRC | 16:19 | |
*** davideagle__ has quit IRC | 16:20 | |
*** davideagle__ has joined #puppet-openstack | 16:21 | |
*** jtomasek has quit IRC | 16:22 | |
*** dgurtner has quit IRC | 16:23 | |
*** vishpat has quit IRC | 16:24 | |
*** rahulait has joined #puppet-openstack | 16:24 | |
*** dh64 has joined #puppet-openstack | 16:25 | |
openstackgerrit | Jay_Clark proposed openstack/puppet-zaqar: continuing work on the initial pass of the zaqar module. https://review.openstack.org/229036 | 16:27 |
openstackgerrit | Jay_Clark proposed openstack/puppet-zaqar: Adding manifest work for Debian installs. https://review.openstack.org/257469 | 16:27 |
*** foxhunt72 is now known as foxhunt_away | 16:33 | |
*** rochaporto has quit IRC | 16:35 | |
*** gfidente has quit IRC | 16:37 | |
*** paramite has quit IRC | 16:39 | |
*** gfidente has joined #puppet-openstack | 16:43 | |
*** gfidente has quit IRC | 16:43 | |
*** gfidente has joined #puppet-openstack | 16:43 | |
*** pleia2 has joined #puppet-openstack | 16:49 | |
*** rahulait has quit IRC | 16:50 | |
*** rahulait1 has joined #puppet-openstack | 16:51 | |
*** cwolferh has joined #puppet-openstack | 16:52 | |
*** rahulait1 is now known as rahulait | 16:53 | |
*** yottatsa has joined #puppet-openstack | 17:03 | |
EmilienM | sbadia: my fix does not work | 17:04 |
EmilienM | http://logs.openstack.org/22/257422/1/check/gate-puppet-neutron-puppet-unit-3.3-dsvm-centos7/d6bfdb4/console.html#_2015-12-14_16_35_34_101 | 17:04 |
EmilienM | see http://logs.openstack.org/22/257422/1/check/gate-puppet-neutron-puppet-unit-3.3-dsvm-centos7/d6bfdb4/console.html#_2015-12-14_16_35_33_436 | 17:04 |
*** rahulait has quit IRC | 17:07 | |
xarses | EmilienM: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20151215 appears to be empty | 17:12 |
EmilienM | xarses: yeah I need to feed it | 17:13 |
EmilienM | doing it right now | 17:13 |
xarses | ok | 17:13 |
*** jtomasek has joined #puppet-openstack | 17:18 | |
*** openstackstatus has quit IRC | 17:20 | |
*** openstackstatus has joined #puppet-openstack | 17:23 | |
*** ChanServ sets mode: +v openstackstatus | 17:23 | |
*** ducttape_ has quit IRC | 17:24 | |
*** samuelBartel has quit IRC | 17:25 | |
*** devvesa has quit IRC | 17:31 | |
*** yottatsa has quit IRC | 17:32 | |
*** jpena has quit IRC | 17:32 | |
*** imcsk8|gym is now known as imcsk8 | 17:35 | |
*** yottatsa has joined #puppet-openstack | 17:38 | |
openstackgerrit | Andrew Woodward proposed openstack/puppet-ceph: Move intergration tests out of acceptance https://review.openstack.org/257511 | 17:42 |
EmilienM | I think we need to pin bundler | 17:44 |
xarses | related to https://launchpad.net/bugs/1525929 ? | 17:45 |
openstack | Launchpad bug 1525929 in puppet-vswitch "Unit-tests fails due to an update of bundler (Psych is double-loaded unless it's in the Gemfile)" [Undecided,In progress] - Assigned to Sebastien Badia (sbadia) | 17:45 |
EmilienM | yes | 17:45 |
EmilienM | except if they do a release | 17:45 |
EmilienM | https://github.com/bundler/bundler/issues/4149#issuecomment-164507268 | 17:46 |
xarses | (off-topic) there was also a tripple-o thread of similar note (about puppetlabs-mysql) they where asking about pinning third-party modules to a release also. I'm not sure if you caught it, it didn't have [puppet] in the topic | 17:47 |
EmilienM | I did, I'll look at it | 17:47 |
EmilienM | thanks | 17:47 |
*** aimon_ has quit IRC | 17:53 | |
*** dgurtner has joined #puppet-openstack | 17:58 | |
*** dgurtner has quit IRC | 17:58 | |
*** dgurtner has joined #puppet-openstack | 17:58 | |
*** davideagle__ has quit IRC | 18:01 | |
*** yottatsa has quit IRC | 18:02 | |
openstackgerrit | Ben Kero proposed openstack/puppet-trove: Typo in README.md https://review.openstack.org/257515 | 18:04 |
*** derekh has quit IRC | 18:04 | |
bkero | >_> drive-by review | 18:11 |
EmilienM | the CI is still broken, unit tests | 18:16 |
EmilienM | I'm on it today, if anyone wants to help | 18:16 |
*** mgrosz has joined #puppet-openstack | 18:16 | |
bkero | Is CI not running, or are jobs just failing? | 18:18 |
*** foxhunt_away is now known as foxhunt72 | 18:19 | |
*** yottatsa has joined #puppet-openstack | 18:19 | |
*** trown|outttypeww is now known as trown | 18:20 | |
xarses | bkero: CI appears to be backlogged | 18:23 |
xarses | http://status.openstack.org/zuul/ | 18:23 |
xarses | you can search for your job there | 18:23 |
*** aimon has joined #puppet-openstack | 18:24 | |
bkero | ok | 18:25 |
openstackgerrit | Dan Prince proposed openstack/puppet-nova: Drop use of deprecated neutron auth options https://review.openstack.org/257522 | 18:30 |
openstackgerrit | Dan Prince proposed openstack/puppet-nova: Drop use of deprecated neutron auth options https://review.openstack.org/257522 | 18:32 |
*** dgurtner has quit IRC | 18:33 | |
*** dfisher has quit IRC | 18:36 | |
*** IvanBerezovskiy has left #puppet-openstack | 18:46 | |
*** dslevin_ has joined #puppet-openstack | 18:48 | |
EmilienM | crinkle, _ody: I would like your insights on 1525929 | 18:51 |
EmilienM | https://launchpad.net/bugs/1525929 | 18:52 |
openstack | Launchpad bug 1525929 in puppet-vswitch "Unit-tests fails due to an update of bundler (Psych is double-loaded unless it's in the Gemfile)" [Undecided,In progress] - Assigned to Sebastien Badia (sbadia) | 18:52 |
EmilienM | we have several options: | 18:52 |
EmilienM | - patch all modules, like sbadia did: https://review.openstack.org/#/q/topic:bug/1525929,n,z | 18:52 |
EmilienM | - try to fix it only in puppet-openstack-integration, like I'm doing (and is failing): https://review.openstack.org/257421 | 18:53 |
EmilienM | - pin bundler in project-config | 18:53 |
EmilienM | nibalizer ^ | 18:54 |
EmilienM | a very quick workaround would be to pin bundler | 18:56 |
crinkle | EmilienM: the unit tests run with `bundle exec` to installing gems with `gem install` won't fix things | 18:56 |
crinkle | i think pinning bundler or doing what sbadia did are both fine | 18:56 |
EmilienM | crinkle: I figured :( | 18:56 |
EmilienM | crinkle: I'm in favor of pin because what sbadia did does not scale for 26 modules (+ stable branches so 100 patches) | 18:57 |
xarses | EmilienM: if pining bundler works, I think fixing it in project-config seems good | 18:57 |
EmilienM | it's a temporary fix, so if we could avoid 100 patches | 18:57 |
EmilienM | cool I just want to make sure our group like it | 18:57 |
xarses | 200, eventually we will back it out right? | 18:57 |
EmilienM | xarses: sigh. | 18:57 |
*** yottatsa has quit IRC | 18:59 | |
*** yottatsa has joined #puppet-openstack | 19:00 | |
*** foxhunt72 is now known as foxhunt_away | 19:00 | |
*** leanderthal|afk has quit IRC | 19:05 | |
*** yottatsa has quit IRC | 19:06 | |
*** yottatsa has joined #puppet-openstack | 19:07 | |
*** yottatsa has quit IRC | 19:09 | |
mdorman | all the keystone_* providers are supposed to work under keystone v2 api as well as v3, right? or are they all coded now such that they only use v3? | 19:10 |
chem | mdorman: right | 19:11 |
chem | mdorman: should work with both | 19:11 |
mdorman | ok. found a bug where keystone_user is calling ‘openstack user list’ w/o a —domain option under v3 api. so i’m trying to figure out how to patch this such that it still works for v2 as well. | 19:14 |
mdorman | https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_user/openstack.rb#L10 does this line imply that keystone_user always uses v3? | 19:14 |
chem | mdorman: not that simple :) | 19:15 |
mdorman | i figured as much | 19:15 |
chem | mdorman: did you do a bug report about what you've found ? | 19:15 |
mdorman | i’m about to. but wanted to have a fix mostly working on my end first so i can make sure i’m describing it correctly | 19:16 |
mdorman | is there some way i can programatically determine what api version it’s using? | 19:16 |
chem | mdorman: most of the time it's going to be /v3, but in some cases '/2.0' endpoint would be used for password checking in kesytone_user | 19:17 |
mdorman | ok. well what i need to do is figure out if it’s 2 or 3 here: https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_user/openstack.rb#L150 | 19:18 |
mdorman | under v2, it works as is. but for v3 we need to do a domain list first and then a user list for each domain individually | 19:18 |
mdorman | i have it working here for v3, but i know it’ll break for v2 (b/c there is no such ‘openstack domain list’ for v2) | 19:18 |
chem | going to lend on '/v3' | 19:21 |
chem | mdorman: but before you go further, I think you got it wrong | 19:21 |
mdorman | ? | 19:22 |
mdorman | i’m writing up the bug now | 19:22 |
chem | mdorman: or maybe you have a use case ... | 19:22 |
mdorman | not sure i follow | 19:23 |
*** yottatsa has joined #puppet-openstack | 19:24 | |
mdorman | ‘openstack user list’ on the command line using the v3 api fails | 19:24 |
mdorman | https://bugs.launchpad.net/puppet-keystone/+bug/1526031 | 19:25 |
openstack | Launchpad bug 1526031 in puppet-keystone "keystone_user provider missing --domain parameter on user list call under v3 API" [Undecided,New] | 19:25 |
chem | mdorman: I'm looking | 19:25 |
EmilienM | our CI should be back very soon | 19:26 |
EmilienM | once https://review.openstack.org/#/c/257465/ is merged | 19:26 |
chem | mdorman: I think this cannot be the case, all the ci would be broken if it was true, as listing user is the basis to not recreate them. | 19:28 |
chem | mdorman: there must be something more | 19:28 |
mdorman | i think the problem is in the test: https://github.com/openstack/puppet-keystone/blob/master/spec/unit/provider/keystone_user/openstack_spec.rb#L82-L87 | 19:28 |
mdorman | this improperly assumes that ‘openstack user list’ will return a proper user listing… when in fact it fails under v3. | 19:28 |
mdorman | (if i am reading that test right) | 19:29 |
*** yottatsa has quit IRC | 19:29 | |
chem | mdorman: you are but it certainly sure this work, for instance, this works https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L123 | 19:30 |
*** dprince has quit IRC | 19:32 | |
mdorman | chem: i wonder if ‘openstack user list’ only fails w/o —domain if keystone actually has more than the default domain configured | 19:35 |
mdorman | let me test that theory | 19:35 |
chem | mdorman: in the ci there are multiple domains configured | 19:36 |
chem | mdorman: those 2: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L25-L34 and the Default one | 19:36 |
mdorman | fair enough | 19:39 |
mdorman | ok, the problem is when domain_specific_drivers_enabled is turned on. when that’s enabled, —domain is needed for user list call. when it’s disabled, user list works fine by defaulting to the default domain. | 19:41 |
mdorman | not sure why it’s not obeying the default domain in that base | 19:42 |
mdorman | *case | 19:42 |
chem | user list will show the user of all the domain if you use the admin user | 19:43 |
chem | mdorman: ^ I think | 19:44 |
mdorman | https://bugs.launchpad.net/keystone/+bug/1479578 | 19:46 |
openstack | Launchpad bug 1479578 in OpenStack Identity (keystone) "Domain-specific config breaks some ops" [Undecided,New] | 19:46 |
mdorman | it doesn’t work if you have domain-specific config turned on in keystone | 19:46 |
chem | mdorman: when you activate domain_specific_drivers_enabled you have to have your backend working for any request to work | 19:46 |
*** yottatsa has joined #puppet-openstack | 19:47 | |
mdorman | they keystone backend? yes, i know. and it is. | 19:48 |
mdorman | it’s a keystone bug. | 19:48 |
chem | mdorman: sorry, it's late :) | 19:48 |
*** imcsk8 has quit IRC | 19:49 | |
*** imcsk8 has joined #puppet-openstack | 19:49 | |
mdorman | np | 19:49 |
mdorman | as usual, this was quite a rabbit hole | 19:50 |
*** yottatsa has quit IRC | 19:50 | |
chem | mdorman: deep down. Could you update the bug report, we have a patch that should manage domain_specific driver, there https://review.openstack.org/#/c/219289/24, it's not fully tested in the CI as we don't setup the proper ldap backend (for instance), so we just check that we create the right files. In the meantime the openstack cli used in the bug report above is old If my memory is ok. So before the patch get commited I would like | 19:52 |
chem | to check if that problem still exists. Thanks. | 19:52 |
chem | mdorman: or when, what and try to have it fixed if necessary | 19:53 |
*** mgrosz has quit IRC | 19:53 | |
chem | mdorman: this is the end fo my day so please put any finding in the bug report and I will get back to you through it | 19:54 |
*** dprince has joined #puppet-openstack | 19:54 | |
mdorman | chem: yup i updated the bug report referencing the keystone bug | 19:55 |
chem | mdorman: thank | 19:55 |
chem | s | 19:55 |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration. https://review.openstack.org/219289 | 19:59 |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend. https://review.openstack.org/238164 | 20:00 |
*** rahulait has joined #puppet-openstack | 20:02 | |
*** rahulait has quit IRC | 20:02 | |
EmilienM | _ody: I think https://review.openstack.org/#/c/256644/ is safe to merge also | 20:06 |
*** yottatsa has joined #puppet-openstack | 20:10 | |
*** vinsh has joined #puppet-openstack | 20:12 | |
openstackgerrit | Sergey Kolekonov proposed openstack/puppet-tempest: [DO NOT MERGE] Test commit https://review.openstack.org/257556 | 20:17 |
*** dslevin_ has quit IRC | 20:21 | |
*** mdorman has quit IRC | 20:22 | |
*** mdorman has joined #puppet-openstack | 20:27 | |
_ody | EmilienM: We talked about one failure with the pick() function last week. What was the actual result of that? | 20:33 |
EmilienM | _ody: I did not investigate yet | 20:33 |
_ody | IMO it didn't look related to the module updates. | 20:33 |
EmilienM | but puppet-openstack_extras has unit tests broken | 20:33 |
EmilienM | nope it did not | 20:33 |
_ody | Ah. | 20:33 |
_ody | So do think it was already broken? | 20:34 |
*** ducttape_ has joined #puppet-openstack | 20:38 | |
_ody | EmilienM: Can you please update https://review.openstack.org/#/c/256644/ and comment that you tested the patch and the results? | 20:42 |
EmilienM | _ody: done | 20:43 |
EmilienM | _ody: yes it was already broken | 20:43 |
EmilienM | _ody: I'll look at it if nobody does before | 20:44 |
*** mflobo has quit IRC | 20:44 | |
*** mflobo has joined #puppet-openstack | 20:44 | |
*** v1k0d3n has quit IRC | 20:46 | |
*** v1k0d3n has joined #puppet-openstack | 20:47 | |
_ody | Lame bundler bug. | 20:47 |
EmilienM | _ody: :) | 20:49 |
EmilienM | _ody: your first +2, heh! | 20:49 |
*** dhollinger has quit IRC | 20:50 | |
_ody | nah. I believe I did one last week too but that failure was on the day's first. | 20:50 |
*** dhollinger has joined #puppet-openstack | 20:52 | |
*** v1k0d3n has quit IRC | 20:53 | |
*** v1k0d3n has joined #puppet-openstack | 20:54 | |
*** v1k0d3n has quit IRC | 20:56 | |
*** v1k0d3n has joined #puppet-openstack | 20:58 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: implement class and defined type for cinder GPFS driver https://review.openstack.org/257565 | 20:58 |
*** karimb has joined #puppet-openstack | 20:58 | |
_ody | EmilienM: weird. I can't comment on https://review.openstack.org/#/c/257565 | 21:01 |
*** jayg is now known as jayg|g0n3 | 21:03 | |
EmilienM | _ody: comment? | 21:04 |
_ody | Nevermind. It was just a UI thing I've never noticed. | 21:05 |
*** chem has quit IRC | 21:08 | |
*** dslevin_ has joined #puppet-openstack | 21:09 | |
*** v1k0d3n has quit IRC | 21:11 | |
*** v1k0d3n has joined #puppet-openstack | 21:12 | |
EmilienM | dmsimard: how do you feel about testing mitaka in puppet CI for centos jobs? Have you heard about some blockers? | 21:13 |
EmilienM | I see https://launchpad.net/~ubuntu-cloud-archive/+archive/ubuntu/mitaka-staging | 21:13 |
EmilienM | maybe worth to test it | 21:13 |
*** yottatsa has quit IRC | 21:23 | |
*** dprince has quit IRC | 21:28 | |
*** yottatsa has joined #puppet-openstack | 21:33 | |
*** ducttape_ has quit IRC | 21:39 | |
*** yottatsa has quit IRC | 21:44 | |
_ody | by randome chance, someone here whould have happened to backport rally to jessie? | 21:49 |
*** yottatsa has joined #puppet-openstack | 21:50 | |
* _ody is down a rabbit hole of dependencies | 21:50 | |
*** yottatsa has quit IRC | 21:51 | |
*** yottatsa has joined #puppet-openstack | 21:53 | |
*** jtomasek has quit IRC | 21:55 | |
*** yottatsa has quit IRC | 21:55 | |
*** yottatsa has joined #puppet-openstack | 21:56 | |
*** baker has quit IRC | 21:56 | |
mwhahaha | zigo could probably answer that | 21:59 |
*** baker has joined #puppet-openstack | 22:02 | |
*** yottatsa has quit IRC | 22:05 | |
*** yottatsa has joined #puppet-openstack | 22:07 | |
*** yottatsa has quit IRC | 22:10 | |
*** trown is now known as trown|outttypeww | 22:14 | |
mdorman | mfisch: ping? | 22:14 |
mfisch | mdorman: yo | 22:15 |
mdorman | re: https://bugs.launchpad.net/puppet-keystone/+bug/1523643 just wondering if you had any patches in flight for that? | 22:16 |
openstack | Launchpad bug 1523643 in puppet-keystone "glance & keystone fighting over openstack client" [High,In progress] - Assigned to Matt Fischer (mfisch) | 22:16 |
mfisch | mdorman: nope | 22:16 |
mfisch | oh wait | 22:16 |
mfisch | sorry yes I do | 22:16 |
mfisch | they all landed | 22:16 |
mfisch | I backported the one that fixes it to liberty | 22:16 |
EmilienM | I had some news from canonical, and they'll have mitaka packaging next week | 22:16 |
mdorman | i am running into a related problem where if you specify a version (vs. ‘present’ or ‘absent’) to keystone::client, it forces openstackclient to be set to the same version. i want to indicate independent version numbers for each. | 22:16 |
EmilienM | in the meantime, RDO packaging seems ready | 22:16 |
EmilienM | I think we could bump our CI during Xmas holidays | 22:17 |
mdorman | mfisch: ok maybe i just need to pull then | 22:17 |
EmilienM | so we'll run mitaka very soon | 22:17 |
mfisch | mdorman: what we did was have Keystone and glance just include ::openstacklib::openstackclient | 22:17 |
mfisch | mdorman: want to BP them? | 22:17 |
EmilienM | maybe I should ask ML | 22:17 |
mfisch | mdorman: https://review.openstack.org/#/c/254824/ https://review.openstack.org/#/c/253311/ | 22:17 |
mfisch | I will +2 a BP | 22:17 |
mdorman | i’m off master i just pulled longer ago than when these langed | 22:19 |
mdorman | *landed | 22:19 |
mfisch | ok | 22:19 |
mdorman | i can create the backport reviews, though, if i can remember/figure out how to do it. | 22:19 |
mfisch | you do it in gerrit but I'd rather not | 22:19 |
openstackgerrit | Mike Dorman proposed openstack/puppet-keystone: Let openstacklib manage openstackclient https://review.openstack.org/257583 | 22:19 |
mfisch | the 3rd review "fixes" it for most people without being intrusive | 22:19 |
mfisch | but if you think people want it then I'm +2 | 22:20 |
openstackgerrit | Mike Dorman proposed openstack/puppet-glance: Let openstacklib manage openstackclient https://review.openstack.org/257584 | 22:20 |
mdorman | the third review? | 22:20 |
EmilienM | mfisch: any patches we can help now? | 22:21 |
mfisch | mdorman: the 3rd review is in liberty | 22:21 |
mdorman | oh. so i should not have backported those to stable/liberty | 22:23 |
mdorman | so no Closes-Bug on those? none of them show up in the bug report | 22:23 |
mfisch | mdorman: well its debatable | 22:24 |
mfisch | give me 1 min | 22:24 |
mfisch | This one solves the actual bug: https://review.openstack.org/#/c/254502/ | 22:24 |
mfisch | the other 2 in general make things cleaner and better but might not need BP | 22:24 |
mfisch | make sense? | 22:24 |
mfisch | like without that last review you can't compile the catalog | 22:24 |
*** gildub has joined #puppet-openstack | 22:26 | |
mdorman | sure you can? if python-openstackclient package is defined in openstacklib::openstackclient and everywhere else it’s just include’d ? | 22:27 |
mfisch | one was including the class, one was including the package | 22:27 |
mfisch | I think glance & keystone respectively | 22:27 |
*** ducttape_ has joined #puppet-openstack | 22:27 | |
mfisch | we covered this in the meeting last week and on the ML to a lesser extent | 22:27 |
mdorman | kk, sorry for dup questions here. i see what you’re saying, i didn’t realize there was a difference in glance. at the moment i’m only concerned with keystone | 22:28 |
*** yottatsa has joined #puppet-openstack | 22:28 | |
mfisch | iirc keystone was doing it wrong | 22:28 |
mdorman | i guess i have no opinion on the backport to stable/liberty for those other two patches. like i said i’m more or less running off master so i don’t really care. | 22:29 |
mfisch | okay lets skip for now then | 22:29 |
mfisch | unless we have a good reason | 22:29 |
mdorman | sounds good. | 22:30 |
*** yottatsa has quit IRC | 22:32 | |
openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp https://review.openstack.org/253216 | 22:43 |
EmilienM | crinkle, _ody, nibalizer: do you do something for puppethack day tomorrow? | 22:43 |
nibalizer | ill be puppeting likely | 22:46 |
nibalizer | not super sure what | 22:46 |
nibalizer | i might just try to slog through puppet community pull requests | 22:46 |
*** baker has quit IRC | 22:46 | |
*** baker has joined #puppet-openstack | 22:47 | |
EmilienM | nice | 22:48 |
nibalizer | https://twitter.com/nibalizer/status/676534071929868288 | 22:48 |
EmilienM | nibalizer: rt'ed | 22:48 |
nibalizer | haha | 22:48 |
openstackgerrit | Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp https://review.openstack.org/253216 | 22:48 |
*** ducttape_ has quit IRC | 22:49 | |
_ody | EmilienM: It kinda just lines up with the day I generally dedicate my whole day to Puppet OpenStack stuff...so I'll coincidentally probably just be hacking on that. | 22:50 |
*** baker has quit IRC | 22:50 | |
EmilienM | \o/ | 22:51 |
*** mgrosz has joined #puppet-openstack | 22:52 | |
*** mgrosz has quit IRC | 22:53 | |
*** dhollinger has quit IRC | 22:57 | |
*** ken_barber has quit IRC | 23:01 | |
*** ric` has quit IRC | 23:01 | |
*** baker has joined #puppet-openstack | 23:01 | |
*** baker has quit IRC | 23:03 | |
*** gfidente has quit IRC | 23:17 | |
*** saneax_AFK is now known as saneax | 23:19 | |
*** saneax is now known as Guest44309 | 23:20 | |
*** mgrosz has joined #puppet-openstack | 23:26 | |
*** ducttape_ has joined #puppet-openstack | 23:26 | |
*** mgrosz has quit IRC | 23:27 | |
*** ducttape_ has quit IRC | 23:30 | |
*** mdorman has quit IRC | 23:31 | |
*** davideagle has joined #puppet-openstack | 23:36 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: Update Puppetfile to latest releases / branches https://review.openstack.org/256644 | 23:54 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: CI test - never merge https://review.openstack.org/257611 | 23:54 |
*** markvoelker has quit IRC | 23:55 | |
*** xarses has quit IRC | 23:56 | |
*** jaosorior has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!