Tuesday, 2016-01-26

*** rfolco has joined #openstack-sprint00:21
*** rfolco has quit IRC00:26
*** mrmartin has joined #openstack-sprint06:02
*** mrmartin has quit IRC06:42
*** _degorenko|afk is now known as degorenko09:56
*** rfolco has joined #openstack-sprint10:56
EmilienMo/12:16
*** sbadia has joined #openstack-sprint13:16
EmilienMdhellmann: hey! I have 2 patches that you could review for using reno: https://review.openstack.org/#/c/272261/ and https://review.openstack.org/#/c/272203/14:26
*** yarkot has quit IRC14:35
EmilienMchem: how is your progress on CI?14:37
chemok, I upload the new version, I tryed harder to understand what I was doing.  Sorry for the delay.14:39
EmilienMdon't be sorry, I'm asking if you need help14:40
EmilienMand if you managed to work on it with mwhahaha14:40
chemEmilienM: that should be ok, I'll ask him when this new version is up if there are optimization to be done14:41
EmilienMchem: fyi, this is what I did yesterday for our gem CI: https://review.openstack.org/#/c/272546/14:50
EmilienM(see the 2 jobs)14:50
EmilienMthe code is here: https://github.com/openstack/puppet-openstack_spec_helper/blob/master/run_beaker_tests.sh14:51
EmilienMand https://github.com/openstack/puppet-openstack_spec_helper/blob/master/run_unit_tests.sh14:51
chemthanks14:53
EmilienMso I thought we could 1/ re-use the bits from bootstrap.sh to generate a module in cookiecutter14:59
EmilienMand 2/ run beaker/unit tests after that14:59
chemEmilienM: I think the validation from the created module directory ought to be enough15:00
EmilienMchem: did you forget to push on https://review.openstack.org/#/c/272146/ ? or just not ready yet?15:05
EmilienMjust wondering since you replied you took comments in consideration15:05
chemhad hard time rebasing, and then slipped away and recheck my code.  Uploading in the next 15min15:06
EmilienMno pressure, just checking15:06
chemmwhahaha: when zuul trigger the jenkins the workspace is already populated with the source code or do I need the zuul clone macro ?15:31
chemmwhahaha: when zuul trigger the jenkins' job, is the workspace already populated with the source code or do I need the zuul clone macro ?15:31
mwhahahait's already populated with that project's code15:31
mwhahahabut you'd need to pull down your dependencies15:31
mwhahahawhich is where zuul cloner comes in15:31
chemmwhahaha: thanks15:35
_odyGot my PR in for our production cloud before heading the gym yesterday.  All sprint all day today.15:55
_odyCan I insert myself into something already being worked on or just grab something off the stack?15:56
EmilienMdegorenko, mwhahaha: can we land https://review.openstack.org/#/c/272553/ please ?16:13
EmilienMand https://review.openstack.org/#/c/272552/16:13
degorenkoEmilienM, done :)16:15
EmilienM_ody: we're tracking the work on https://etherpad.openstack.org/p/puppet-happy-new-year-201616:16
EmilienM_ody: degorenko wanted to make progress on Mitaka deprecations + removals16:17
EmilienMmaybe you guys can work together on it16:17
degorenkosure, _ody if here is anything that i can help you - you're welcome16:18
degorenkoi'm sorry, that i can't lead this activity as it was planned, but i have unexpected internal stuff :(16:18
_odyOk.  Sounds good to me.  I doubt we have a way to identify deprecations added post liberty besides "git blame"16:18
_odydegorenko: np.  As did I yesterday.  I'll start digging into it.16:19
degorenko_ody, ok, please ping me or add to review in case of you need help :)16:19
chemmwhahaha: if you have time to review https://review.openstack.org/#/c/272156/, that would be great.  I let a comment inside the code, I don't know if it doable...17:11
chemmwhahaha: I have to go right now, but I will check back later on gerrit17:11
mwhahahasure17:12
_odydegorenko: You called out specifically keytone endpoint types in you mitaka deprecation topic but all master branches seemt to already have them set in their corresponding *::keystone::auth classes.17:19
_odyDid you have a specific example?17:19
degorenko_ody, it wasn't me :( it was mfisch17:20
mfischwhat did I do?17:20
mfischoh17:20
mfischyeah that was fixed in between the time I wrote that and now17:20
mfischwe can delete it17:20
_odyOk.  Check!17:21
* _ody goes back to one module at a time17:21
*** EmilienM has quit IRC17:30
*** EmilienM has joined #openstack-sprint17:34
mfisch_ody: whats the docs scrub?17:35
*** degorenko is now known as _degorenko|afk17:56
_odymfisch: I just recalled last sprint when hunner and I were doing the PL module approved thing, docs weren't complete enough to actually use them as a source of information as a beginner operator/automator.18:01
_odyI do not have a specific example.18:01
_odymfisch, EmilienM https://github.com/openstack/puppet-cinder/blob/master/manifests/api.pp#L158-L15918:26
_odyYou were the last two to touch those lines.18:26
_odyIs the parameter on line 159 actually deprecated?18:26
_odyThe rest of the code doesn't indicate as such.18:26
_odyNeither does the documentation.18:26
EmilienMI think we can drop this code18:26
EmilienMI'm sure nobody use the validation18:27
_odyoh.  You mean all together.18:27
EmilienMand we can send a warning first18:27
_odySo take it out of openstacklib.18:27
EmilienM_ody: yeah we can18:42
EmilienMI tried to do something after paris summit18:42
EmilienMbut it was useless18:42
EmilienMwe could drop the code18:42
EmilienMwhat do you think about https://review.openstack.org/#/c/272203/ ? Specially the fact we have python code in the puppet module19:34
EmilienMcrinkle: ^19:34
crinkleEmilienM: i think using reno is good19:35
EmilienMnice19:35
crinkleEmilienM: you're the only one who would have to make releases, so it's only you who needs to interact with the python stuff right?19:36
EmilienMcrinkle: dhellmann helped me to make it work in puppet-keystone, the patch is ready for review19:36
EmilienMcrinkle: yes19:36
crinklewe don't need to require our developers to have both python and ruby environments set up, so it seems fine to me19:36
EmilienMcrinkle: and our gate, that build doc19:36
crinkleright19:36
EmilienMnibalizer: when you around, could you look https://review.openstack.org/#/c/272261/ please?19:50
nibalizersure19:54
mfischEmilienM: reading your email19:56
mfischwhy switch to IPv6?19:56
mfischI'd like to keep both rather than switch19:56
EmilienMit was an idea19:56
mfischI would show you the "Why Not Both?" meme but the commercial probably doesnt show in Canada ;)19:58
*** rfolco has quit IRC19:59
mwhahahafortunately there's this thing called youtube, https://www.youtube.com/watch?v=vqgSO8_cRio20:00
mfischlol20:01
EmilienMnice one20:01
mfischthats in the original spanish even20:01
EmilienMmfisch: how do you see it?20:02
EmilienMdual stack?20:02
mfischwell I'm thinking about that20:02
mfischI dont have a good solution20:02
mfischare lots of people using ipv6 these days?20:02
EmilienMmfisch: redhat deploys openstack on ipv6 networks for customers20:03
EmilienMwe added ipv6 support in tripleo20:03
EmilienMmight be worth to investigate20:03
mfischits probably ok for loopback ayway20:05
mfischanyway20:05
_odydeprecations are time consumeing.21:15
mwhahahayes21:23
_odyI am going to be lucky to finish even two modules.21:29
EmilienM_ody: can I help?21:41
_odyEmilienM: Nothing right now.  Just having to go through each file one at a time takes time.21:51
_odyRemoving any tests related to deprecated things that I am removing then I'll have a part 1 for cinder.21:52
mfischdo any of our modules make openrc files?22:18
EmilienMnope22:18
_odyOnly openstack_extras.22:18
mfischIm using the ancient puppet-openstack module but am about to just do it myself22:18
mfischah extras22:18
_odyHmm...afternoon tea or coffee.22:19
* _ody has already had morning tea and coffee22:20
mfischopenstack extras doesnt handle some of the more advanced settings for openstack client22:20
mfischlike identity api version22:20
mfisch_ody: tea22:20
_odymfisch: Nope.  It actually pretty much breaks stuff.22:20
_odyI wouldn't be offended if we deprecated that from extras.22:21

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!