Thursday, 2016-10-20

« Wednesday, 2016-10-19 Index Friday, 2016-10-21 »
*** ducttape_ has quit IRC00:03
*** ducttape_ has joined #puppet-openstack00:16
*** agrebennikov has quit IRC00:31
*** agrebennikov_ has joined #puppet-openstack00:31
dmsimardmwhahaha: I THINK THIS IS DEPRECATED http://logs.openstack.org/89/388589/1/gate/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/fc2567b/console.html#_2016-10-19_22_24_58_03285400:33
*** saneax is now known as saneax-_-|AFK00:36
*** ducttape_ has quit IRC00:37
mwhahahadmsimard: OK? :D it's not related to this failure but yea we need to fix that00:39
*** snarwade has quit IRC00:40
*** ducttape_ has joined #puppet-openstack00:44
*** fultonj has quit IRC00:45
*** xarses has joined #puppet-openstack00:47
*** cdelatte has quit IRC00:56
*** agrebennikov_ has quit IRC01:15
*** rpittau has quit IRC01:25
*** michapma_alt has joined #puppet-openstack02:06
openstackgerritSteve Baker proposed openstack/puppet-tripleo: Use HAProxy for docker-registry endpoint  https://review.openstack.org/38896302:11
*** hjensas has quit IRC02:27
*** cgfbee has quit IRC02:32
*** cgfbee has joined #puppet-openstack02:33
*** ducttape_ has quit IRC02:49
*** ducttape_ has joined #puppet-openstack02:54
openstackgerritXingchao Yu proposed openstack/puppet-cloudkitty: This is the initial commit for puppet-cloudkitty.  https://review.openstack.org/38714902:54
*** ducttape_ has quit IRC03:14
*** rpittau has joined #puppet-openstack03:21
*** rpittau has quit IRC03:38
*** saneax-_-|AFK is now known as saneax03:54
*** richm1 has joined #puppet-openstack04:04
*** richm has quit IRC04:05
*** ducttape_ has joined #puppet-openstack04:15
*** ducttape_ has quit IRC04:20
*** saneax is now known as saneax-_-|AFK04:37
*** markvoelker_ has quit IRC04:49
*** jaosorior has joined #puppet-openstack05:09
*** ducttape_ has joined #puppet-openstack05:15
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-openstacklib: Add tls options to mysql user creation  https://review.openstack.org/38788005:15
*** skramaja has joined #puppet-openstack05:18
*** ducttape_ has quit IRC05:21
*** markvoelker_ has joined #puppet-openstack05:30
*** rpittau has joined #puppet-openstack05:33
*** jtomasek has quit IRC05:39
*** richm1 has quit IRC05:39
*** markvoelker_ has quit IRC05:46
*** yolanda has quit IRC06:20
*** rpittau has quit IRC06:36
*** imcsk8_PTO has quit IRC07:02
*** imcsk8_PTO has joined #puppet-openstack07:02
*** dgurtner has joined #puppet-openstack07:05
*** dgurtner has quit IRC07:05
*** dgurtner has joined #puppet-openstack07:05
*** fxpester has joined #puppet-openstack07:10
*** tesseract has joined #puppet-openstack07:11
*** tesseract is now known as Guest1406907:12
*** ducttape_ has joined #puppet-openstack07:17
*** amoralej|off is now known as amoralej07:21
*** ducttape_ has quit IRC07:22
*** jpena|off is now known as jpena07:37
*** danpawlik has joined #puppet-openstack07:42
*** exed_ has joined #puppet-openstack07:48
*** rmart04 has joined #puppet-openstack07:48
openstackgerritAlfredo Moralejo proposed openstack/puppet-neutron: Fix domain_id in server manifest  https://review.openstack.org/38904707:49
*** exed_ has left #puppet-openstack07:50
*** fxpester has quit IRC07:51
*** jpich has joined #puppet-openstack08:04
*** paramite has joined #puppet-openstack08:08
*** hjensas has joined #puppet-openstack08:16
*** hjensas has quit IRC08:16
*** hjensas has joined #puppet-openstack08:16
*** ducttape_ has joined #puppet-openstack08:18
*** markvoelker has joined #puppet-openstack08:19
*** ccamacho|afk is now known as ccamacho08:19
*** ducttape_ has quit IRC08:23
*** colonwq has quit IRC08:24
*** rpittau has joined #puppet-openstack08:32
*** hjensas has quit IRC08:38
*** hjensas has joined #puppet-openstack08:40
*** hjensas has joined #puppet-openstack08:40
*** derekh has joined #puppet-openstack08:46
*** saneax-_-|AFK is now known as saneax08:56
*** dtantsur|sick is now known as dtantsur08:56
*** ducttape_ has joined #puppet-openstack09:19
*** social has joined #puppet-openstack09:20
*** jtomasek_ has joined #puppet-openstack09:21
*** ducttape_ has quit IRC09:23
*** hjensas has quit IRC09:27
*** hjensas has joined #puppet-openstack09:33
*** hjensas has joined #puppet-openstack09:33
*** jtomasek_ is now known as jtomasek09:35
*** markvoelker has quit IRC09:39
*** exed_ has joined #puppet-openstack09:41
*** derekh has quit IRC09:43
*** jaosorior has quit IRC09:44
*** colonwq has joined #puppet-openstack09:44
*** jaosorior has joined #puppet-openstack09:44
*** derekh has joined #puppet-openstack09:47
*** ddmitriev has joined #puppet-openstack09:57
*** b00tcat has joined #puppet-openstack10:16
b00tcathi, quick question: what's the format of this parameter in puppet-tripleo? https://github.com/openstack/puppet-tripleo/blob/master/manifests/haproxy.pp#L42710:17
b00tcatit says "Filename of an HAProxy-compatible certificate and key file", does this mean the file contains the two things?10:17
*** saneax is now known as saneax-_-|AFK10:19
*** ducttape_ has joined #puppet-openstack10:20
*** degorenko|afk is now known as degorenko10:20
jaosoriorb00tcat: yep10:24
*** ducttape_ has quit IRC10:25
jaosoriorb00tcat: that's the way haproxy reads it :/10:25
jaosoriorpretty weird, but it's the way it is10:25
Kvislefirst proposal-question; my proposal now has 5 points on code, 1 point on verified, but it needs workflow ... is there anything I need to do, or will it eventually be merged at this point? https://review.openstack.org/#/c/387263/10:26
b00tcatthanks jaosorior10:27
*** derekh has quit IRC10:31
*** derekh has joined #puppet-openstack10:34
*** markvoelker has joined #puppet-openstack10:37
*** saneax-_-|AFK is now known as saneax10:37
*** iberezovskiy|off is now known as iberezovskiy10:49
*** dprince has joined #puppet-openstack10:58
*** markvoelker_ has joined #puppet-openstack11:07
*** lucas-sick is now known as lucasagomes11:10
*** markvoelker has quit IRC11:11
EmilienMI'm failing to see where scenarios are failing11:18
EmilienMwell, it sounds like they are passing now11:18
EmilienMdegorenko: I saw your recheck :)11:19
degorenkoEmilienM, so, if i see failed jobs - i can recheck right?11:20
*** ducttape_ has joined #puppet-openstack11:20
EmilienMdegorenko: I don't know, I just read backlog and saw conversation between dmsimard and mwhahaha11:21
degorenkoyeah, i see11:21
degorenkohm11:22
EmilienMwell, I saw some jobs running now, so let's see :)11:22
*** gfidente has joined #puppet-openstack11:25
degorenkoyeah11:25
*** ducttape_ has quit IRC11:25
*** davideagle_ has joined #puppet-openstack11:36
*** bogdando has quit IRC11:38
*** ccamacho is now known as ccamacho|lunch11:39
*** jpena is now known as jpena|lunch11:43
*** johnmilton has quit IRC11:45
openstackgerritMerged openstack/puppet-gnocchi: Release 9.4.1 (newton)  https://review.openstack.org/38852011:51
*** jayg|g0n3 is now known as jayg11:56
*** bogdando has joined #puppet-openstack11:56
*** johnmilton has joined #puppet-openstack12:06
iurygregorygood morning12:07
*** amoralej is now known as amoralej|lunch12:08
*** ducttape_ has joined #puppet-openstack12:10
iurygregoryEmilienM, did you recived the mascot logo?12:11
*** markvoelker has joined #puppet-openstack12:12
*** markvoelker_ has quit IRC12:16
*** trown|outtypewww is now known as trown12:25
*** fultonj has joined #puppet-openstack12:27
*** ducttape_ has quit IRC12:35
openstackgerritMerged openstack/puppet-ironic: Release 9.4.1 (newton)  https://review.openstack.org/38852212:37
*** davideagle_ has quit IRC12:39
*** richm has joined #puppet-openstack12:41
*** ccamacho|lunch is now known as ccamacho12:42
*** ducttape_ has joined #puppet-openstack12:44
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Release 9.4.1 (newton)  https://review.openstack.org/38916812:46
EmilienMbackport ^12:46
iurygregory+212:46
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [WIP] Remove "dhcp" command from the iPXE script  https://review.openstack.org/38916912:47
*** jpena|lunch is now known as jpena12:48
*** derekh has quit IRC12:50
*** rcernin has joined #puppet-openstack12:53
openstackgerritDan Prince proposed openstack/puppet-tripleo: Set memcached_servers for nova API  https://review.openstack.org/38917613:00
*** jaosorior is now known as jaosorior_brb13:01
*** ducttape_ has quit IRC13:03
*** morazi has joined #puppet-openstack13:05
*** rmart04 has quit IRC13:06
*** jistr is now known as jistr|biab13:08
*** mkoderer has joined #puppet-openstack13:08
*** derekh has joined #puppet-openstack13:09
*** davideagle_ has joined #puppet-openstack13:09
*** richm1 has joined #puppet-openstack13:10
*** richm has quit IRC13:10
*** jistr|biab is now known as jistr13:11
*** morazi has quit IRC13:12
*** morazi has joined #puppet-openstack13:25
*** ducttape_ has joined #puppet-openstack13:26
*** jtriley has joined #puppet-openstack13:27
*** dgurtner has quit IRC13:36
*** dgurtner has joined #puppet-openstack13:38
*** dgurtner has quit IRC13:38
*** dgurtner has joined #puppet-openstack13:38
*** amoralej|lunch is now known as amoralej13:43
*** ducttape_ has quit IRC13:44
*** dgurtner has quit IRC13:53
*** jaosorior_brb is now known as jaosorior14:05
*** ducttape_ has joined #puppet-openstack14:06
*** xarses has quit IRC14:10
*** ducttape_ has quit IRC14:10
mwhahahahey it looks like infra fixed the test problems, yay14:16
*** agrebennikov_ has joined #puppet-openstack14:17
dmsimardmwhahaha: so it was not our fault ?14:17
iurygregory \o/14:17
dmsimardmwhahaha: also re last night: I know the deprecations are not related to the issue14:17
dmsimardmwhahaha: they just started happening recently so I thought I'd highlight them :p14:17
mwhahahayea it wasn't our fault, ansible was erroring14:17
iurygregoryansible...14:18
iurygregory=P14:18
mwhahahadmsimard: yea i saw those yesterday, i'll take a look today if i get time14:18
openstackgerritMerged openstack/puppet-ironic: Release 9.4.1 (newton)  https://review.openstack.org/38916814:20
*** cgfbee has quit IRC14:22
*** skramaja has quit IRC14:27
*** lucasagomes is now known as lucas-hungry14:28
*** xarses has joined #puppet-openstack14:30
*** cgfbee has joined #puppet-openstack14:30
*** bogdando has quit IRC14:33
*** bogdando has joined #puppet-openstack14:36
*** ducttape_ has joined #puppet-openstack14:38
*** davideagle_ has quit IRC14:41
*** ducttape_ has quit IRC14:52
*** davideagle has joined #puppet-openstack14:54
*** social has quit IRC15:16
*** social has joined #puppet-openstack15:19
*** ducttape_ has joined #puppet-openstack15:21
*** _ducttape_ has joined #puppet-openstack15:28
*** saneax is now known as saneax-_-|AFK15:30
*** ducttape_ has quit IRC15:31
*** lucas-hungry is now known as lucasagomes15:32
*** danpawlik has quit IRC15:33
openstackgerritMerged openstack/puppet-tripleo: Fixes missing ODL ML2 Authentication info  https://review.openstack.org/38682015:34
*** sbadia has joined #puppet-openstack15:35
*** priteau has joined #puppet-openstack15:39
*** dgurtner has joined #puppet-openstack15:40
*** _ducttape_ has quit IRC15:42
*** ducttape_ has joined #puppet-openstack15:43
mwhahahaHunner: just thought i'd comment that the stdlib deprecation function spits totally unreadable stack traces, http://logs.openstack.org/89/388589/1/gate/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/fc2567b/console.html#_2016-10-19_22_24_59_66797515:43
mwhahahaHunner: so i have absolutely no idea what is deprecated15:44
*** jaosorior has quit IRC15:44
dmsimardHunner: this man speaks the truth ^15:48
mwhahahai think the is_array function is deprecated15:48
mwhahahawhich is lunacy15:48
mwhahahajust make it puppet4 compatible so we don't have to see all these notices15:48
fultonjdo any more cores want to review https://review.openstack.org/#/c/387540/ ?15:51
fultonjthanks mwhahaha for looking at that15:52
iurygregoryfultonj, looking now15:52
fultonjiurygregory: thanks15:52
*** rcernin has quit IRC15:52
mwhahahahttps://tickets.puppetlabs.com/browse/MODULES-3961 ಠ_ಠ15:55
mwhahahalooks like all the validate_* calls are deprecated15:56
iurygregoryand what we should use?15:56
iurygregory'-'15:56
*** dgurtner has quit IRC15:57
*** Guest14069 has quit IRC15:59
*** dgurtner has joined #puppet-openstack15:59
mwhahahai have no idea16:00
mwhahahait claims validate_legacy with some sort of type16:00
iurygregoryody, do you have any idea ^16:01
iurygregory?16:01
mwhahahahttps://github.com/puppetlabs/puppetlabs-stdlib#validate_legacy16:01
mwhahahaody: Hunner -^ who thought this was a good idea16:01
*** michapma_alt has quit IRC16:02
*** dgurtner has quit IRC16:04
*** jpich has quit IRC16:04
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Remove "dhcp" command from the iPXE script  https://review.openstack.org/38916916:06
*** dfisher has joined #puppet-openstack16:07
mwhahahaguess it's time to tack on yet another annoying mass update to address this16:08
*** ducttape_ has quit IRC16:09
iurygregoryyeah lets add16:09
iurygregorynow we should follow: validate_legacy("Optional[String]", "validate_re", "Value to be validated", ["."]) ? O.o omg16:09
mwhahahai guess, trying to figure out that out now16:11
mwhahahalike i said, who thought that was a good idea16:11
*** ducttape_ has joined #puppet-openstack16:11
mwhahahai guess it's validate_legacy(<puppet4 type>, "validate_<old function type>", $var)16:14
mwhahahahttps://dzone.com/articles/ntp-a-puppet-4-language-update16:14
mwhahahanot sure what Optional[string] is i guess that's a puppet4 language thing16:15
iurygregorya better example in the readme would be good16:16
mwhahahapretty much16:16
mwhahahahttps://docs.puppet.com/puppet/latest/reference/lang_data_type.html16:18
*** openstackgerrit has quit IRC16:18
mwhahahamy god that's overly complicated16:19
*** openstackgerrit has joined #puppet-openstack16:19
iurygregorymaybe we should ask in #puppet ?16:19
mwhahahaeh we're just hitting the puppet3 eol, puppet4 push16:19
mwhahahajust annoying16:20
mwhahahalooks like the heredoc syntax has also chaged, https://docs.puppet.com/puppet/latest/reference/lang_data_string.html#heredocs16:21
mwhahahafortunately i don't think we use that anywhere16:22
*** hjensas has quit IRC16:22
*** derekh has quit IRC16:25
*** hjensas has joined #puppet-openstack16:27
*** hjensas has joined #puppet-openstack16:27
mwhahahagah all the is_* functions are also deprecated, you're supposed to use if $var =~ Stdlib::Compat::<type> instead16:27
mwhahahai wonder if that works in puppet316:28
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Switch to stdlib validate_legacy  https://review.openstack.org/38927116:30
* mwhahaha cries16:30
mwhahahaguess i figured out what i'm going to do today16:30
Hunnermwhahaha: :(16:30
Hunnermwhahaha: I'll pass it on (dev_el_ops is the one behind it)16:30
mwhahahaHunner: this is a nightmare, jfyi16:31
Hunnermwhahaha: oh, the p4 typing?16:32
mwhahahano having to switch all these functions16:32
mwhahahatyping is annoying16:32
mwhahahabut this, this is crap16:32
Hunnermwhahaha: Or you could use the real types instead of the Stdlib::Compat::* types16:32
Hunnermwhahaha: are you at pconf? dev_el_ops is there and probably happy to chat about it16:33
mwhahahano16:33
mwhahahaHunner: what's the p4 friendly way to validate_re16:34
mwhahahathe example in stdlib doesn't really make sense16:34
*** dgurtner has joined #puppet-openstack16:36
openstackgerritAlex Schultz proposed openstack/puppet-barbican: Switch to stdlib validate_legacy  https://review.openstack.org/38927316:38
Hunnermwhahaha: If you want p4 way and not just validate_legacy way, Variant[Boolean, Enum['true', 'false', 'running', 'stopped']]16:38
Hunnerfor example is a way that people often do validation16:38
*** jtomasek has quit IRC16:38
Hunnermwhahaha: as well as https://docs.puppet.com/puppet/4.7/reference/lang_data_abstract.html#pattern16:38
*** dtantsur is now known as dtantsur|afk16:38
Hunnerthe abstract data types specifically have a lot of flexibility16:39
iurygregoryi'll miss validate_* functions =(16:39
mwhahahaHunner: they might, but we don't need any of that heh16:39
*** mdorman_ is now known as mdorman16:39
dmsimardiurygregory: welcome to the future16:39
iurygregorydmsimard, not very welcome right? haha XD16:39
mwhahahathe validate_* functions where always a basic check and we weren't relying on the annoying special cases so for us just making them continue to function would have been better than having to switch all of them16:40
mwhahahai'm glad we've really not been to strict about these checks or this could have been worse16:40
iurygregoryyep16:40
*** mkoderer has quit IRC16:42
Hunnermwhahaha: want to start a puppet-dev@ mail? David is UK tz usually and at pconf doing a talk today16:43
Hunnergetting user feedback is ++16:43
mwhahahai probably need to signup for that :D16:43
mwhahahawhere is that16:43
Hunnerpuppet-dev@groups.google.com16:43
*** exed_ has quit IRC16:43
HunnerIf you don't want to sign up, CC yourself and mention that you're not subscribed to the list. Replies should CC you as swell16:44
mwhahahai'll just subscribe16:45
amoralejmay i get your reviews on https://review.openstack.org/#/c/389047/ ?16:47
amoralejeasy one, but is blocking promotion of mitaka in rdo-ci16:47
mwhahahaamoralej: commented16:48
dmsimardamoralej: https://review.openstack.org/#/c/389047/ straight to mitaka ?16:49
amoralejlemme explain it in the change...16:50
iurygregoryalso there is a OOO failure =(16:50
dmsimardamoralej: oh, mwhahaha beat me to it16:50
*** dgurtner has quit IRC16:56
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Move rbd_default_features to ensure_resources  https://review.openstack.org/38754016:57
*** _ducttape_ has joined #puppet-openstack17:01
mwhahahaugh validate_legacy isn't added until 4.13 meaning that becomes the minium version of stdlib we need17:04
*** ducttape_ has quit IRC17:04
mwhahahaeff all this17:04
mwhahahaoh and it doesn't work in puppet317:05
*** jpena is now known as jpena|off17:10
*** trown is now known as trown|lunch17:12
mwhahahaso i guess we need to figure out an EOL of puppet3 for us as well17:12
iurygregoryyeah17:13
iurygregorymwhahaha, you will participate in puppet sessions via hangouts or something? maybe is a topic we should discuss17:13
mwhahahai could try17:15
mwhahahathe first session is at 4 am my time17:16
iurygregoryurgh17:16
iurygregoryomg17:16
iurygregoryvery hard for you =(17:16
mwhahahathe second is 10:3017:16
mwhahahawhich is more reasonable :D17:16
iurygregoryoh yes XD17:16
*** iberezovskiy is now known as iberezovskiy|off17:17
mwhahahai added it to the etherpad, i figure we might want to mention in O but actually do it in P17:20
iurygregorynice :D17:21
mjblackmwhahaha: have a link to the etherpad?17:22
amoralejmwhahaha, i think cherry-picking https://review.openstack.org/#/c/336958/ is the right way17:22
dmsimardmwhahaha: oh, not going to BCN ?17:23
mwhahahamjblack: https://etherpad.openstack.org/p/ocata-puppet17:23
mwhahahadmsimard: nope17:23
dmsimard:(17:23
mwhahahaamoralej: yea17:23
*** degorenko is now known as _degorenko|afk17:23
mjblackmwhahaha: so one suggestion for octa release17:24
mjblackare we open to using hiera data binding?17:24
*** vinsh has joined #puppet-openstack17:24
dmsimardmjblack: where ?17:25
mwhahahamjblack: tripleo uses it extensively, but what do you have in mind?17:25
mwhahahaswitching just the tests or something?17:25
mjblackmwhahaha: like a community supported hiera that people can use to do least amount of composition to make workable17:25
mjblackCI could benefit from it too17:25
dmsimardmjblack: we chose not to use it in puppet-openstack-integration and acceptance tests because it adds another layer of complexity17:25
mjblackit does17:26
mjblackbut it also makes it much more flexible17:26
dmsimardlike, when troubleshooting things.. okay, where's the issue, is it in packaging, is it in the project, is it in the gate vm, is it an external puppet module, is it in hiera, etc. etc.17:26
mjblackor maybe its in the fixtures17:26
dmsimardright17:26
mjblackor the p-o-i manifests17:27
mjblackthe argument doesnt negate that it is already complex17:27
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: WIP Glance NFS  https://review.openstack.org/38928417:27
mwhahahai would assume that doing hiera for p-o-i  would actually simplify it17:27
mwhahahabut maybe i'm wrong17:27
mjblackwont know till we take a look at it17:27
*** _ducttape_ has quit IRC17:28
dmsimardmwhahaha: simplify what for who17:28
mjblackdmsimard: well for starters, you could put in the hiera.yaml a distro and distro-release to add flexiblity, like for example...turning something on for xenial that was off for trusty17:28
dmsimardmwhahaha: if it's because there's too many parameters, we can just create a params class17:28
*** ducttape_ has joined #puppet-openstack17:28
mjblackits more than just parameters17:29
dmsimardmjblack: I know what hiera is and what it provides, but CI is already hard enough to troubleshoot17:29
dmsimardthat's my opinion, anyway17:30
mwhahahai guess the question is what exactly to move to hiera and what not to17:30
mwhahahafor me  the hardcoded crud like https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/mistral.pp#L29-L4117:30
mwhahahawould make sense in hiera17:30
mjblackwell, we're not talking about just the CI though in my opinion17:30
mwhahahaall those passwords and usernames17:30
dmsimardit'd make things cleaner, simpler /for [me, you, select individuals]/ but increases the required ramp up for people to help17:30
dmsimardmjblack: I'm not against the idea but I need to be convinced :)17:32
mwhahahadmsimard: we already had to do it for apache btw17:32
mwhahahahttps://github.com/openstack/puppet-openstack-integration/blob/master/hiera/common.yaml17:33
dmsimardmwhahaha: yup, and I even hook into that to extend p-o-i17:33
dmsimardI believe there was a debate around hiera a while back.. I'm trying to find where it is17:34
dmsimardprobably find arguments for and against there17:34
dmsimardall I remember is that Colleen shared my opinion :(17:34
mwhahahai think things have changed a bit since it was last discussed :D17:36
dmsimardwas it that long ago ?17:36
mjblackit was probably before puppet 417:37
openstackgerritAlfredo Moralejo proposed openstack/puppet-neutron: Add user_domain_name and project_domain_name  https://review.openstack.org/38929017:38
*** ducttape_ has quit IRC17:44
*** ducttape_ has joined #puppet-openstack17:44
*** ducttape_ has quit IRC17:49
*** exed_ has joined #puppet-openstack17:49
amoralejmwhahaha thanks for your help!17:54
*** markvoelker has quit IRC17:56
*** amoralej is now known as amoralej|off18:00
*** ducttape_ has joined #puppet-openstack18:02
openstackgerritcraig delatte proposed openstack/puppet-manila: WIP - Manila hooks support  https://review.openstack.org/38930218:04
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted now in an attempt to resolve some mismatched merge states on a few changes, but should return momentarily.18:09
*** davideagle has quit IRC18:10
*** trown|lunch is now known as trown18:13
openstackgerritAlex Schultz proposed openstack/puppet-ceph: Make pid_max settable on OSD nodes.  https://review.openstack.org/37579818:46
*** markvoelker has joined #puppet-openstack18:46
mwhahahaguess we had a bunch of stuff stuck18:47
*** markvoelker has quit IRC18:47
*** markvoelker has joined #puppet-openstack18:47
openstackgerritMerged openstack/puppet-ceph: Ensure wget is installed for RGW Keystone integration  https://review.openstack.org/32673218:48
openstackgerritMerged openstack/puppet-ceph: Change conf from create to ensure  https://review.openstack.org/38768618:48
openstackgerritMerged openstack/puppet-ceph: Notify radosgw service on configuration change  https://review.openstack.org/38808018:48
mwhahahalol wat18:48
*** _ducttape_ has joined #puppet-openstack18:49
xarsessticky stuck18:51
*** ducttape_ has quit IRC18:52
openstackgerritMerged openstack/puppet-cloudkitty: This is the initial commit for puppet-cloudkitty.  https://review.openstack.org/38714918:53
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Fix package version on Fedora  https://review.openstack.org/28861918:57
*** lucasagomes is now known as lucas-afk18:58
*** _ducttape_ has quit IRC18:59
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/38858918:59
*** ducttape_ has joined #puppet-openstack18:59
mwhahahachem: you going to continue with this or should it just be aboandoned? https://review.openstack.org/#/c/29637018:59
chemmwhahaha: hum .... good question.  give me till end of next week to see if I can finish this up.  Thanks for poping this up.19:02
mwhahahasure19:02
*** paramite has quit IRC19:02
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Make fernet the default token provider  https://review.openstack.org/38932219:10
*** johnmilton has quit IRC19:21
openstackgerritTim Rozet proposed openstack/puppet-tripleo: Fixes missing ODL ML2 Authentication info  https://review.openstack.org/38932819:26
openstackgerritAlex Schultz proposed openstack/puppet-swift: Allow a replication network to be specified  https://review.openstack.org/30248319:32
*** paramite has joined #puppet-openstack19:38
*** johnmilton has joined #puppet-openstack19:39
openstackgerritAlex Schultz proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/24648019:40
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Fix package version on Fedora  https://review.openstack.org/28861919:48
*** paramite has quit IRC20:01
openstackgerritAlex Schultz proposed openstack/puppet-vswitch: rework service and fix service for debian jessie  https://review.openstack.org/32249520:02
mwhahahavinsh: you still working on https://review.openstack.org/#/c/264966/ or should it be abandoned?20:03
vinshI am working on it.  I will land the storage policy change in the next week, then this one follows.20:05
vinshIt's a priority for me20:05
vinshI have a massive update to the storage policy change to push up still20:07
vinsham bringing up some new swift clusters today though, so gotta get that done.20:07
mwhahahak20:09
vinshand then comes ini provider/remove templates for all the object/account/container classes20:10
vinshTHEN the change for dedicated replication networks can be re done20:10
vinshThat ring one that you updated today is good/cool20:10
*** jayg is now known as jayg|g0n320:15
dmsimardEmilienM, mwhahaha, iurygregory: fyi I just fixed an issue that prevent puppet repository promotion in RDO for the past ~2 days, promotion just occured after passing all three scenarios20:17
iurygregorydmsimard, thank you man :D20:17
dmsimardiurygregory: well take that for what it is, I'm the one who also introduced the issue20:17
dmsimardso yeah :p20:17
iurygregorydmsimard, you don't need to tell me that part =P hahaha20:18
EmilienMdmsimard: you rocks20:18
dmsimardhey gotta take credit20:18
dmsimardI break things and I fix things20:18
dmsimardI don't discriminate against who broke things20:18
iurygregoryyeah that's true20:18
dmsimardeven if that's myself20:18
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Change worker defaults to ::os_workers  https://review.openstack.org/38394820:20
*** ccamacho has quit IRC20:22
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Update NetApp configuration parameters  https://review.openstack.org/34177720:23
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Re-enable Mistral testing  https://review.openstack.org/33981320:26
EmilienMmwhahaha: they were flappy last time I tried ^20:27
mwhahahayea i know just rebasing so we can keep testing20:27
mwhahahagood ol' merge conflicts20:27
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Re-enable SSL testing on Ubuntu  https://review.openstack.org/37946320:29
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Re-enable linuxbridge testing on Ubuntu  https://review.openstack.org/37947220:31
openstackgerritAlex Schultz proposed openstack/puppet-openstack-cookiecutter: Added ensure parameter to pass to service identity  https://review.openstack.org/34276820:41
*** dprince has quit IRC20:48
Kvisledo I understand it correct that "Needs Review" mean I need a second core dev to ack on my proposal? I'm a bit confused, because there's a green checkmark on the Code Review-column in gerrit20:49
Kvisle(first proposal)20:49
iurygregoryKvisle, another core need to +A your change =)20:51
Kvisleok! is this a part of the process I should actively engage in, or is it something that will happen eventually when someone gets to it?20:52
iurygregorywell when you have change that need review you can ask here for reviews and when we have time we will give feedback =)20:53
xarsesKvisle: we do two +2 code review +1 verified, and +1 (A) workflow to get merged20:53
iurygregorycan you put the link for review?20:53
Kvislehttps://review.openstack.org/#/c/387263/20:53
KvisleI don't think I understand what the difference between code review and workflow20:54
Kvisle... is20:54
*** priteau has quit IRC20:54
xarsescode review is people reviewing, cores can +/-2, others can +/-120:55
xarsesworkflow is telling gerrit to start the gate validation for merging20:55
xarsesfor most projects, workflow involves rebasing the current CR onto master and re-running the same check CI and requiring that it all pass before it can be merged20:57
KvisleI see.20:58
xarsesit's an additional step that helps ensure that we don't land a patch because master changed in a conflicting way from when verification CI ran maybe some days ago.20:58
*** ilbot3 has quit IRC21:00
*** ducttape_ has quit IRC21:00
Kvislethanks! :)21:03
iurygregoryKvisle, soo iscsi_ip_address will only need to be set if i'm using cinder.volume.drivers.emc.emc_cli_iscsi.EMCCLIISCSIDriver?21:03
*** ducttape_ has joined #puppet-openstack21:04
openstackgerritAlex Schultz proposed openstack/puppet-vswitch: rework service and fix service for debian jessie  https://review.openstack.org/32249521:04
openstackgerritAlex Schultz proposed openstack/puppet-openstack-cookiecutter: Added ensure parameter to pass to service identity  https://review.openstack.org/34276821:05
iurygregorythere is another EMC drivers and all other does not requires iscsi_ip_address?21:06
mwhahahadmsimard: is designate fixed yet for centos?21:06
dmsimardmwhahaha: no21:06
mwhahaha:(21:06
dmsimardmwhahaha: I don't really want to package monasca if I can help it21:06
Kvisleiurygregory: the emc vnx backend has two volume drivers - one for fibre channel and one for iscsi.21:06
dmsimardmwhahaha: there's a bug open on their end to make it an optional dependency21:06
mwhahahadmsimard: k maybe i'll dust off my python to fix it :D21:07
dmsimardmwhahaha: I have the bug here somewhere, hang on21:07
iurygregoryKvisle, right and the only one that requires iscsi_ip_address is cinder.volume.drivers.emc.emc_cli_iscsi.EMCCLIISCSIDriver ?21:07
dmsimardmwhahaha: https://bugs.launchpad.net/designate/+bug/163066621:07
openstackLaunchpad bug 1630666 in Designate "monasca-statsd is a non-optional import even when disabled" [Low,Triaged]21:07
dmsimardmwhahaha: it's so stupid IMO21:08
dmsimardmwhahaha: bad bad implementation21:08
dmsimardlet's make an optional backend a non-optional requirement21:08
mwhahahasounds about right21:08
dmsimardit's almost like if we forced ceilometermiddleware to be installed with swift21:09
* dmsimard coughs coughs21:09
mwhahaha:o21:09
*** ilbot3 has joined #puppet-openstack21:10
dmsimardit's not like we're doing that or anything21:10
* dmsimard still very unhappy with the outcome of that issue21:10
*** trown is now known as trown|outtypewww21:11
Kvisleiurygregory: the "cinder.volume.drivers.emc.emc_cli_fc.EMCCLIFCDriver" does not require it at least --- the documentation does not say that it is required for ISCSI, and I don't have the hardware to test behaviour without it -- so I don't want to make it completely optional21:12
Kvisle(maybe I should?)21:12
*** ayoung has quit IRC21:12
iurygregoryKvisle, maybe ask cinder devs about it?21:13
iurygregoryif the other drive does not require the parameter the change is fine =) can you confirm and put in the review?21:13
*** jtriley has quit IRC21:14
openstackgerritGiulio Fidente proposed openstack/puppet-ceph: Always allow setting keystone token/user/pass/project/domain  https://review.openstack.org/38935521:16
Kvisleiurygregory: put in the review?21:17
openstackgerritMerged openstack/puppet-keystone: set 0600 permissions on fernet keys & folder  https://review.openstack.org/38213621:17
openstackgerritMerged openstack/puppet-ceph: Make pid_max settable on OSD nodes.  https://review.openstack.org/37579821:17
openstackgerritMerged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/38858921:18
iurygregoryKvisle, ask someone in cinder if the parameter is optional for the other driver, when you get the answer you can reply the review with this information or just ping me here in the channel21:18
Kvislein that case, I could create another change making iscsi_ip_address completely optional?21:20
iurygregoryWhen you acess https://review.openstack.org/#/c/387263/ you can just click in Reply.. and add a message with the answer you got from cinder and just click Post =D21:20
Kvisleah, ok!21:20
iurygregoryif is optional for both you need just to update your patch :D21:20
Kvisleunless it has been introduced as a required parameter, it's optional ... in mitaka it's a valid parameter, but a noop21:26
*** davideagle has joined #puppet-openstack21:26
Kvisleseems like the backend has been completely reworked since then though21:26
Kvisleiurygregory: looks like I'll rework the patch and make it completely optional -- the parameter does nothing in mitaka, and in master it isn't even there21:30
Kvislethrow warning that it has been removed, and will be removed on next release perhaps21:31
* Kvisle leaves with these thoughts in mind21:31
openstackgerritMerged openstack/puppet-ironic: Remove "dhcp" command from the iPXE script  https://review.openstack.org/38916921:32
openstackgerritMerged openstack/puppet-neutron: Add user_domain_name and project_domain_name  https://review.openstack.org/38929021:32
*** davideagle has quit IRC21:36
openstackgerritMerged openstack/puppet-swift: Release 9.4.3 (newton)  https://review.openstack.org/38887921:45
openstackgerritMerged openstack/puppet-tripleo: pacemaker/mysql: wait step 2 to remove default accounts  https://review.openstack.org/38604221:51
*** davideagle has joined #puppet-openstack21:53
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Uses keystone user/pass/project instead of token for RadosGW  https://review.openstack.org/38937222:06
openstackgerritMerged openstack/puppet-swift: Change worker defaults to ::os_workers  https://review.openstack.org/38282922:11
*** ducttape_ has quit IRC22:17
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Update NetApp configuration parameters  https://review.openstack.org/34177722:20
*** morazi has quit IRC22:22
*** ayoung has joined #puppet-openstack22:23
*** xarses has quit IRC22:39
*** blkperl_ is now known as blkperl22:51
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Uses keystone user/pass/project instead of token for RadosGW  https://review.openstack.org/38937222:59
*** gfidente has quit IRC23:00
*** ayoung has quit IRC23:06
*** davideagle has quit IRC23:09
*** davideagle_ has joined #puppet-openstack23:11
*** saneax-_-|AFK is now known as saneax23:16
*** davideagle_ has quit IRC23:21
*** davideagle_ has joined #puppet-openstack23:23
*** dfisher has quit IRC23:24
*** davideagle_ has quit IRC23:25
*** markvoelker has quit IRC23:37
*** rpittau has quit IRC23:44
*** agrebennikov_ has quit IRC23:46
*** ayoung has joined #puppet-openstack23:59
« Wednesday, 2016-10-19 Index Friday, 2016-10-21 »

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