Monday, 2016-12-19

« Sunday, 2016-12-18 Index Tuesday, 2016-12-20 »
*** ducttape_ has joined #puppet-openstack00:08
openstackgerritMerged openstack/puppet-ceph: Move enable_sig and release to ceph::params  https://review.openstack.org/41045400:13
*** ducttape_ has quit IRC00:52
*** ducttape_ has joined #puppet-openstack01:04
*** ducttape_ has quit IRC01:15
*** saneax-_-|AFK is now known as saneax01:18
openstackgerritzhongshengping proposed openstack/puppet-cinder: Use ``enabled_backends`` option to set up backends  https://review.openstack.org/41205001:21
openstackgerritzhongshengping proposed openstack/puppet-cinder: Use ``enabled_backends`` option to set up backends  https://review.openstack.org/41205001:26
*** ducttape_ has joined #puppet-openstack01:29
*** ducttape_ has quit IRC01:50
*** ducttape_ has joined #puppet-openstack02:12
*** ducttape_ has quit IRC02:37
*** ducttape_ has joined #puppet-openstack02:37
*** imcsk8 has quit IRC02:40
*** imcsk8 has joined #puppet-openstack02:40
*** saneax is now known as saneax-_-|AFK02:45
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:48
*** ducttape_ has quit IRC02:52
*** saneax-_-|AFK is now known as saneax02:58
openstackgerritMerged openstack/puppet-panko: Fix the case statement  https://review.openstack.org/40957003:00
openstackgerritMerged openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/41205403:01
*** michapma_alt has quit IRC03:05
*** michchap_alt_ has joined #puppet-openstack03:05
*** saneax is now known as saneax-_-|AFK03:09
*** ducttape_ has joined #puppet-openstack03:15
*** tdb has quit IRC03:16
*** tdb has joined #puppet-openstack03:16
openstackgerritzhongshengping proposed openstack/puppet-openstack-integration: Remove redundant code  https://review.openstack.org/41226503:17
*** ducttape_ has quit IRC03:26
*** mwhahaha has quit IRC03:26
*** mwhahaha has joined #puppet-openstack03:29
*** rlaurika has quit IRC03:31
*** rlaurika has joined #puppet-openstack03:31
openstackgerritMerged openstack/puppet-glance: Add oslo_messaging_notifications transport_url  https://review.openstack.org/41086203:33
openstackgerritzhongshengping proposed openstack/puppet-barbican: Unknown variable: '::barbican::database_pool_size'  https://review.openstack.org/41227203:46
openstackgerritzhongshengping proposed openstack/puppet-barbican: Unknown variable: '::barbican::database_pool_size'  https://review.openstack.org/41227203:54
*** ducttape_ has joined #puppet-openstack04:26
*** ducttape_ has quit IRC04:31
*** snarwade has joined #puppet-openstack04:56
*** ducttape_ has joined #puppet-openstack05:00
*** jaganathan has joined #puppet-openstack05:17
*** saneax-_-|AFK is now known as saneax05:35
*** ducttape_ has quit IRC05:39
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: Add dependency to openvswitch service for adding DPDK_OPTIONS  https://review.openstack.org/40977905:51
*** jaosorior has joined #puppet-openstack06:06
openstackgerritzhongshengping proposed openstack/puppet-barbican: Unknown variable: '::barbican::database_pool_size'  https://review.openstack.org/41227206:10
*** snarwade is now known as snarwade-afk06:20
*** ducttape_ has joined #puppet-openstack06:57
*** snarwade-afk is now known as snarwade06:58
*** ducttape_ has quit IRC07:02
*** tesseract has joined #puppet-openstack07:04
*** fxpester has joined #puppet-openstack07:04
*** tesseract is now known as Guest3325407:05
*** jaosorior has quit IRC07:58
*** shardy has joined #puppet-openstack08:02
*** yprokule has joined #puppet-openstack08:14
*** rasca has joined #puppet-openstack08:16
*** jaosorior has joined #puppet-openstack08:24
*** ducttape_ has joined #puppet-openstack08:28
*** ntpttr__ has joined #puppet-openstack08:30
*** jpena|off is now known as jpena08:32
*** ducttape_ has quit IRC08:33
*** ntpttr__ has quit IRC08:35
*** jaosorior has quit IRC08:45
*** jaosorior has joined #puppet-openstack08:46
*** jpich has joined #puppet-openstack08:50
*** jtomasek has joined #puppet-openstack08:57
*** rasca has quit IRC09:07
*** gfidente has joined #puppet-openstack09:18
*** gfidente has quit IRC09:18
*** gfidente has joined #puppet-openstack09:18
*** jtomasek has quit IRC09:19
*** dgurtner has joined #puppet-openstack09:25
*** lucas-afk is now known as lucasagomes09:26
*** ducttape_ has joined #puppet-openstack09:58
*** chem has joined #puppet-openstack10:00
*** chem has quit IRC10:03
*** chem has joined #puppet-openstack10:03
*** ducttape_ has quit IRC10:03
*** paramite_ has joined #puppet-openstack10:19
*** paramite has joined #puppet-openstack10:20
*** derekh has joined #puppet-openstack10:43
*** panda|off is now known as panda10:49
*** ducttape_ has joined #puppet-openstack11:03
*** amoralej is now known as amoralej|brb11:05
*** pester has joined #puppet-openstack11:08
*** fxpester has quit IRC11:10
*** ducttape_ has quit IRC11:21
*** hecliunyx has joined #puppet-openstack11:31
*** ducttape_ has joined #puppet-openstack11:33
EmilienMo/11:33
*** hjensas has quit IRC11:47
*** ducttape_ has quit IRC11:49
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Do not merge -- debug  https://review.openstack.org/41241711:51
*** lucasagomes is now known as lucas-hungry11:55
*** fultonj has joined #puppet-openstack12:05
*** shardy is now known as shardy_lunch12:15
*** amoralej|brb is now known as amoralej12:15
*** dprince has joined #puppet-openstack12:22
*** iurygregory has joined #puppet-openstack12:29
iurygregorygood morning12:32
*** iberezovskiy|off is now known as iberezovskiy12:35
iberezovskiyhi12:35
*** panda is now known as panda|afk12:48
*** ducttape_ has joined #puppet-openstack12:49
*** ducttape_ has quit IRC12:54
*** dgurtner has quit IRC12:56
*** jtomasek has joined #puppet-openstack12:56
*** hjensas has joined #puppet-openstack12:57
*** hjensas has quit IRC12:57
*** hjensas has joined #puppet-openstack12:57
*** skramaja_ has joined #puppet-openstack13:00
*** skramaja has quit IRC13:00
*** lucas-hungry is now known as lucasagomes13:01
*** johnmilton has joined #puppet-openstack13:11
*** trown|outtypewww is now known as trown13:14
*** pester has quit IRC13:15
*** morazi has quit IRC13:25
*** pradk has joined #puppet-openstack13:31
*** skramaja has joined #puppet-openstack13:36
*** skramaja_ has quit IRC13:36
*** fultonj has quit IRC13:43
*** fultonj has joined #puppet-openstack13:45
*** jpena is now known as jpena|lunch13:46
dmsimard¯\_(ツ)_/¯13:46
* dmsimard still happy about aliases13:46
dmsimardgood morning :p13:46
EmilienMlol hey13:47
*** ducttape_ has joined #puppet-openstack13:50
*** ducttape_ has quit IRC13:55
*** panda|afk is now known as panda13:58
*** amoralej is now known as amoralej|lunch14:10
*** shardy_lunch is now known as shardy14:16
*** yprokule has quit IRC14:21
*** pradk has quit IRC14:25
*** morazi has joined #puppet-openstack14:25
EmilienMcould someone review https://review.openstack.org/#/c/406300/ please?14:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Re-enable Designate on CentOS7  https://review.openstack.org/39373814:26
iurygregoryEmilienM, i'll after my lunch o/14:26
iurygregorydo you have any docs about the nova placement?14:26
EmilienMiurygregory: in the commit message.14:27
EmilienMiurygregory: oh no14:27
EmilienMI removed it it sounds like14:27
iurygregorynope haha14:27
iurygregoryi'll look in past patchs :D14:27
EmilienMwell, somewher e:D14:27
iurygregorydocs.openstack.org/developer/nova/placement.html14:27
iurygregoryafter read this i'll review :D14:28
*** social has quit IRC14:29
*** morazi has quit IRC14:29
*** ducttape_ has joined #puppet-openstack14:32
*** andreww has quit IRC14:40
*** ducttape_ has quit IRC14:42
*** morazi has joined #puppet-openstack14:43
*** mkarpin has quit IRC14:50
*** jtomasek has quit IRC14:50
*** amoralej|lunch is now known as amoralej14:56
*** jtomasek has joined #puppet-openstack14:59
*** vinsh has joined #puppet-openstack15:01
*** jpena|lunch is now known as jpena15:03
*** pradk has joined #puppet-openstack15:04
mwhahahawoo monday15:10
mwhahahawhat's broken today :D15:10
*** jtomasek has quit IRC15:13
EmilienMmwhahaha: puppet CI looks pretty stable15:14
EmilienMmwhahaha: I sent summary on openstack-dev15:14
mwhahahacool15:14
vinshdon't curse it... ;p15:15
EmilienMmwhahaha: when you have time https://review.openstack.org/#/c/406300/15:15
EmilienMmwhahaha: also i'm trying to see why https://review.openstack.org/#/c/410467/ fails15:15
*** ducttape_ has joined #puppet-openstack15:17
jaganathanEmilienM, Hi15:18
mwhahahaEmilienM: I bet you not everyone supports those new options correctly, the output looks like it's got stderr or something mixed into it. pethaps the use_syslog needs to still be defined15:19
EmilienMmwhahaha: sounds like tempest15:20
EmilienMi'll look later, low prio15:20
mwhahahaprobably15:21
*** jaganathan has quit IRC15:22
*** aarefiev is now known as aarefiev_afk15:41
openstackgerritVladimir Kozhukalov proposed openstack/puppet-ceilometer: Add memcached_servers for keystone authtoken  https://review.openstack.org/41250215:43
*** saneax is now known as saneax-_-|AFK15:43
openstackgerritTobias Urdin proposed openstack/puppet-nova: Add resize_confirm_window option to nova::compute  https://review.openstack.org/41250315:44
iurygregoryhttps://review.openstack.org/412502 WTF15:44
iurygregoryO.o15:44
mwhahahamitaka15:45
iurygregoryohhh15:45
iurygregoryi didn't saw the branch lol15:45
iurygregoryXD15:45
iurygregorymaybe i should eat now haha15:45
mwhahaha:D15:45
EmilienMhttp://logs.openstack.org/38/393738/6/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/b35736e/console.html#_2016-12-19_15_24_33_01197315:46
EmilienMsounds like scenario003 on xenial fails15:46
EmilienMscenario002 I mean15:46
mwhahahadid xenial get new ocata-2 packages?15:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/41250515:48
EmilienMmwhahaha: I would be surprised :D15:48
EmilienMlet's see15:48
*** dfisher has joined #puppet-openstack15:48
EmilienMthey have a nice HTML page lol15:48
EmilienMreqorts.qa.ubuntu.com/reports/ubuntu-server/cloud-archive/ocata_versions.html15:48
EmilienMzaqar 3.0.0-1~cloud0 -- still the same15:49
EmilienM3.0.0 is newton, wtf15:49
EmilienMAttributeError: 'Manager' object has no attribute 'default_params_with_timeout_values'15:50
EmilienMthat happens when they removing something in tempest, still in-use by the plugin15:51
* EmilienM looks in logstash15:53
mwhahahamaybe a tempest change15:53
EmilienMit sounds like we're not alone15:54
EmilienMhttp://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22default_params_with_timeout_values%5C%2215:54
* EmilienM switches on #openstask-qa15:54
mwhahahai have never gotten opesntack logstash to work for me15:54
mwhahahalike that comes back with nothing everytime15:54
*** andreww has joined #puppet-openstack15:59
EmilienMmwhahaha: you need to change the timeframe16:00
EmilienMto be 2 days for ex16:00
EmilienMdefault is Last 5m16:00
*** _ducttape_ has joined #puppet-openstack16:00
*** xarses_ has joined #puppet-openstack16:00
mwhahahait was the last 15m16:01
mwhahahai guess it's not very current16:01
*** Guest33254 has quit IRC16:02
*** ducttape_ has quit IRC16:04
*** andreww has quit IRC16:04
*** agrebennikov has joined #puppet-openstack16:07
* mjblack needs more coffee 16:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test https://review.openstack.org/#/c/412511/  https://review.openstack.org/41251316:11
EmilienMmwhahaha: ^ attempt to test a fix in tempest16:11
mwhahahak16:11
*** jtomasek has joined #puppet-openstack16:12
openstackgerritAlex Schultz proposed openstack/puppet-nova: Add cell_v2 simple_cell_setup  https://review.openstack.org/40994816:13
mwhahahathis stupid cell v2 setup this is such a terrible thing16:14
mjblackEmilienM: it looks like for puppet-octavia that there is a prior releasenote, do you still want me to remove mine?16:14
mwhahahaespecially because it needs the transport url for some reason16:14
*** jtomasek has quit IRC16:16
openstackgerritMatthew J Black proposed openstack/puppet-octavia: Use correct section for host and port settings  https://review.openstack.org/41216416:16
EmilienMmjblack: it's a mistake16:17
EmilienMyeah thx16:17
mjblackok just submitted it16:17
openstackgerritMatthew J Black proposed openstack/puppet-octavia: Added parameters to configure ip and port  https://review.openstack.org/41216616:19
*** jtomasek has joined #puppet-openstack16:22
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova: Add novajoin class  https://review.openstack.org/39678916:25
*** iberezovskiy is now known as iberezovskiy|off16:26
*** _ducttape_ has quit IRC16:27
*** ducttape_ has joined #puppet-openstack16:28
*** kozhukalov has joined #puppet-openstack16:29
kozhukalovmwhahaha: hi, could you please take a look at this https://review.openstack.org/#/c/412502/ ?16:29
mwhahahakozhukalov: sure16:32
kozhukalovthanks very much16:33
*** jtomasek has quit IRC16:35
openstackgerritMatthew J Black proposed openstack/puppet-octavia: Added parameters to configure ip and port  https://review.openstack.org/41216616:40
iurygregorymjblack, why not have the default port value? '-'16:44
iurygregorywhy $::os_service_default?16:44
mjblackdefault port value is 555516:45
mjblackthe default bind ip is 127.0.0.116:45
mjblackso to Zhong's review, the only one that needs to be set is the bind ip since we're not using the default value16:46
iurygregoryyeah i saw you used 5555 in test, but why not set the parameter to this value?16:46
*** jaosorior has quit IRC16:48
*** jaosorior has joined #puppet-openstack16:49
openstackgerritMatthew J Black proposed openstack/puppet-octavia: Added parameters to configure ip and port  https://review.openstack.org/41216616:55
openstackgerritAlex Schultz proposed openstack/puppet-nova: Enable cell_v2 setup by default  https://review.openstack.org/41032416:56
iurygregoryEmilienM, do we have a ubunu bug for https://review.openstack.org/#/c/406300/13/spec/acceptance/nova_wsgi_apache_spec.rb ? i think it would be usefull we have in the comments =)17:02
EmilienMiurygregory: I can post a patch on top of it with a comment17:04
*** jpich has quit IRC17:04
EmilienMit's in the commit message otherwise, but I'll post a patch on top, so we can merge this one17:04
iurygregoryEmilienM, i'll +A ;)17:05
EmilienMiurygregory: /me posting a patch right now17:05
EmilienMiurygregory: on top17:05
*** _ducttape_ has joined #puppet-openstack17:08
*** ducttape_ has quit IRC17:11
*** rwsu has joined #puppet-openstack17:22
*** rwsu has quit IRC17:23
*** rwsu has joined #puppet-openstack17:36
*** panda is now known as panda|bbl17:38
*** _ducttape_ has quit IRC17:40
*** chem has quit IRC17:40
*** chem has joined #puppet-openstack17:41
*** ducttape_ has joined #puppet-openstack17:41
*** derekh has quit IRC17:44
*** trown is now known as trown|lunch17:48
*** lucasagomes is now known as lucas-afk17:50
EmilienMmwhahaha: their patch in tempest doesn't work17:51
EmilienMI propose we pin tempest17:51
EmilienMour CI is really broken now17:51
mwhahahaK17:51
dmsimard /topic CI status: rainbow - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/17:52
mwhahahamore like /topic CI status: questionable at best - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/17:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Pin tempest to unblock CI scenarios  https://review.openstack.org/41255517:54
EmilienMlol17:54
EmilienMout for lunch17:54
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Add until_complete to archive_deleted_rows.pp  https://review.openstack.org/41255617:56
openstackgerritMerged openstack/puppet-ceilometer: Add memcached_servers for keystone authtoken  https://review.openstack.org/41250217:58
* mjblack looks around nervously 18:01
mjblackI didnt break CI18:01
mwhahahaa likely story18:01
mjblackyou'll never prove anything copper!18:05
*** shardy has quit IRC18:08
*** jpena is now known as jpena|away18:09
*** gfidente is now known as gfidente|afk18:23
openstackgerritCarlos Camacho proposed openstack/puppet-nova: Add until_complete to archive_deleted_rows.pp  https://review.openstack.org/41255618:29
*** ansmith has joined #puppet-openstack18:38
EmilienMmwhahaha: why do we need to backport https://review.openstack.org/#/c/410532/ ?18:40
mwhahahaso we can backport https://review.openstack.org/#/c/412006/18:41
mwhahahasince the cell v2 was added before Ocata, we need to support it in newton18:41
mwhahahawe don't have to enable it but, but it should exist18:41
EmilienMmwhahaha: for upgrades. fair enough18:42
mwhahahayea18:42
EmilienMdmsimard or iurygregory: please review https://review.openstack.org/#/c/412555/ asap18:44
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib: Improve os_transport_url ssl support  https://review.openstack.org/41257618:45
dmsimardEmilienM: what's that hash refer to ?18:45
dmsimardwhatever was before that breaking commit ?18:45
EmilienMdmsimard: the previous commit before the breakage18:45
EmilienMI'm tired of tempest18:46
EmilienMI told them 100 times to look our jobs18:46
dmsimardId518a6d87d0949737cd1c50cb6a83149b85e5f85 removes this class variable18:46
dmsimardbecause we thought no one was using it anymore. We were wrong and broke18:46
dmsimardsome Tempest plugins. But this variable shouldn"t have been used in the18:46
dmsimardfirst place because it's a private/internal variable only. So we18:46
dmsimardexpect plugins will eventually remove their dependency on that var.18:46
dmsimardNo kidding, that definitely hasn't happened before (a plugin using a private/internal var)18:46
*** fultonj has quit IRC18:46
mwhahahai've been trying to keep an eye on the tempest reviews but i guess i missed that one18:47
mwhahahadidn't we know that one was going to break something at some point?18:47
EmilienMdmsimard: thx.18:47
mwhahahayea i told them on Nov 14th that was going to break stuff18:48
EmilienM'recheck' party in ~40 min18:48
iurygregoryso we should aprove?18:48
EmilienMiurygregory: done18:48
dmsimard(╯°□°）╯︵ ʞɔǝɥɔǝɹ18:48
EmilienMlol18:48
mwhahahaEmilienM, dmsimard: https://review.openstack.org/#/c/355586/18:48
mwhahahait passed18:48
mwhahahaour CI18:48
EmilienMI wonder why18:48
EmilienMand how18:48
dmsimardEmilienM: plugins != tempest18:48
mwhahahadmsimard: but i've seen that error on a centos scenario002 today i thought18:49
mwhahahaso shouldn't it puked?18:49
EmilienMif it would have been a plugin it would have failed in weirdo18:49
EmilienMwe haven't promoted in 4 days, more issues to come!18:51
dmsimardmwhahaha: trying to understand will always result in despair18:51
EmilienMjobs are running now on https://ci.centos.org/view/rdo/view/weirdo-pipelines/view/weirdo-promote-puppet/18:51
mwhahahadmsimard: too late, already in the land of despair and disrepair18:51
EmilienMdmsimard: I'm sure weirdo jobs will fail18:53
EmilienMit needs the pin stuff we're merging now18:53
EmilienMdmsimard: any chance to manually trigger jobs again when the poi patch is merged?18:53
dmsimardEmilienM: yeah.18:54
EmilienMmerci18:54
*** fultonj has joined #puppet-openstack18:58
EmilienMdmsimard: scenario002 in weirdo fails as expeted18:59
EmilienMexpected*18:59
dmsimardgood18:59
EmilienMdmsimard: I think this workflow pretty good after a while of running it19:00
dmsimardI don't like it :(19:00
EmilienMdmsimard: it would be a bit better if we could send a patch in poi when it fails19:00
EmilienMdmsimard: what would you do?19:00
dmsimardIt doesn't have the attention it deserves and there are some differences with upstream, even if little19:01
EmilienMwe should run weirdo upstream!19:01
EmilienMis it inception?19:01
dmsimardah, yes, a project to run gate jobs outside of the gate in the gate19:02
EmilienMgated?19:02
dmsimardmy parrot is gesturing no with his head19:03
*** openstackstatus has quit IRC19:13
*** openstack has joined #puppet-openstack19:15
openstackgerritVladimir Kozhukalov proposed openstack/puppet-cinder: Add memcached_servers for keystone authtoken  https://review.openstack.org/41259319:20
*** trown|lunch is now known as trown19:25
*** ducttape_ has quit IRC19:37
*** panda|bbl is now known as panda19:38
*** gfidente|afk has quit IRC19:51
*** jaosorior has quit IRC19:52
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Revert "Move Mistral to authtoken"  https://review.openstack.org/41260219:55
*** rcernin has joined #puppet-openstack19:56
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Revert "Move Mistral to authtoken"  https://review.openstack.org/41260219:56
openstackgerritMerged openstack/puppet-openstack-integration: Pin tempest to unblock CI scenarios  https://review.openstack.org/41255519:59
*** fultonj has quit IRC19:59
EmilienMdmsimard: ^19:59
EmilienMcan you kick off jobs please? :)20:00
*** fultonj has joined #puppet-openstack20:01
dmsimarddone20:04
EmilienMdmsimard: merci Sir20:05
*** ducttape_ has joined #puppet-openstack20:05
EmilienMdmsimard: where can I see them? i can't find it20:08
dmsimardEmilienM: they're queued20:08
dmsimardslaves are full20:08
EmilienMahh ok20:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Don't pin Tempest if jobs are run in Tempest gate  https://review.openstack.org/41260520:12
EmilienMmwhahaha, dmsimard ^ I found useful to do that ^20:12
dmsimard-120:14
dmsimard(╯°□°）╯︵ ┻━┻20:14
dfisherthe best emoticon.20:14
EmilienMdmsimard: ah right, nice catch20:16
*** amoralej is now known as amoralej|off20:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Don't pin Tempest if jobs are run in Tempest gate  https://review.openstack.org/41260520:18
EmilienMdmsimard: I can do that ^ or test if the variable is not empty before20:19
dmsimardwait, why aren't we using the tempest_version var anyway ?20:19
dmsimardhttps://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L2720:20
dmsimardbecause then that eventually leads to actually getting tempest at the right version https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L83-L10020:20
dmsimardand what do you know, it's inside an if nodepool block20:20
EmilienMoh nice I'll change that20:21
EmilienMlet me rework the code20:21
dmsimardyeah fix the pin to use tempest_version instead and you can add your pin prevention in that block20:21
dmsimardwe won't be able to do a clean revert but that's okay20:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Don't pin Tempest if jobs are run in Tempest gate  https://review.openstack.org/41260520:23
EmilienMdmsimard: might be better ^20:23
dmsimardEmilienM: so that does the reverse of what you want, it'll reset to TEMPEST_VERSION if the zuul_project is tempest20:24
dmsimardEmilienM: I'd move the git reset sha1 inside the if/else block20:24
dmsimardso you don't need to check for zuul twice20:25
dmsimardnot sure if you understand what I mean20:25
EmilienMdmsimard: can you push over?20:27
dmsimardgoing out in 2 mins to pick up kid at school but I can check later20:27
EmilienMdmsimard: nothing urgent20:29
*** ccamacho has quit IRC20:30
*** iurygregory has quit IRC20:42
*** ccamacho has joined #puppet-openstack20:46
*** chem has quit IRC20:50
*** chem has joined #puppet-openstack20:50
*** social has joined #puppet-openstack20:56
*** jpena|away is now known as jpena20:57
*** johnmilton has quit IRC21:00
*** fultonj has quit IRC21:07
*** morazi has quit IRC21:14
*** ccamacho has quit IRC21:15
*** _ducttape_ has joined #puppet-openstack21:29
*** xarses has joined #puppet-openstack21:30
*** ducttape_ has quit IRC21:33
*** fultonj has joined #puppet-openstack21:33
*** xarses_ has quit IRC21:33
*** _ducttape_ has quit IRC21:34
*** saneax-_-|AFK is now known as saneax21:51
*** xarses has quit IRC21:52
*** ansmith has quit IRC21:56
*** xarses has joined #puppet-openstack21:58
*** xarses_ has joined #puppet-openstack21:59
EmilienMmwhahaha: can you +2 https://review.openstack.org/#/c/412602/ ? in case it pass CI, i'll approve it tonight, so we can get tripleo CI green again21:59
*** trown is now known as trown|outtypewww21:59
EmilienMplease22:00
*** xarses_ has quit IRC22:00
mwhahahak22:00
*** xarses_ has joined #puppet-openstack22:00
*** dprince has quit IRC22:02
*** xarses has quit IRC22:03
*** xarses_ has quit IRC22:03
*** panda is now known as panda|Zz22:03
*** chem has quit IRC22:12
*** chem has joined #puppet-openstack22:12
*** jpena is now known as jpena|off22:26
*** fultonj has quit IRC22:27
*** paramite has quit IRC22:36
*** paramite_ has quit IRC22:36
mjblackEmilienM: I didnt set it to os_service_default because of https://github.com/openstack/puppet-octavia/blob/master/manifests/api.pp#L3322:55
mjblackI can make the change if you still want but thats why it was set22:56
*** pradk has quit IRC23:01
openstackgerritMerged openstack/puppet-octavia: Use correct section for host and port settings  https://review.openstack.org/41216423:02
EmilienMmwhahaha: it sounds like https://review.openstack.org/#/c/412602/ would break tripleo scenario00323:10
mwhahahaof course it would23:10
EmilienMmwhahaha: why it didn't before?23:12
EmilienMoh23:12
EmilienMmaybe the instack patch is not packaged yet23:12
EmilienMthe revert is not packaged I mean23:12
EmilienMmjblack: no strong opinion, but it would be nice to be consistent, yes23:13
mjblackok I will leave it as-is then :)23:13
EmilienMmwhahaha: yeah, nevermind. It was old commit.23:14
EmilienMmwhahaha: https://dashboards.rdoproject.org/rdo-dev23:19
EmilienMwe got promotion23:19
mwhahahago us :D23:20
EmilienMyeah23:21
*** dfisher has quit IRC23:21
*** ducttape_ has joined #puppet-openstack23:31
EmilienMdmsimard: I want to merge my patch before tempest think they fixed their stuff23:34
EmilienMso either you push a new PS or we merge my patch23:34
EmilienMhttps://review.openstack.org/#/c/412605/23:34
dmsimardEmilienM: whatever I can fix it later23:34
*** ducttape_ has quit IRC23:34
EmilienMok, so please approve it23:34
dmsimardsorry, I've been in meetings after meeting and stuff23:34
EmilienMI see tempest folks working on a fix23:34
EmilienMI don't want them happy because it's green while it's actually red23:34
*** ducttape_ has joined #puppet-openstack23:35
dmsimardlol23:35
dmsimardapproved23:35
dmsimardttyl23:35
EmilienMthx23:36
*** agrebennikov has quit IRC23:38
*** johnmilton has joined #puppet-openstack23:39
*** ducttape_ has quit IRC23:39
openstackgerritMerged openstack/puppet-cinder: Add memcached_servers for keystone authtoken  https://review.openstack.org/41259323:51
*** ducttape_ has joined #puppet-openstack23:55
*** johnmilton has quit IRC23:55
*** ducttape_ has quit IRC23:59
« Sunday, 2016-12-18 Index Tuesday, 2016-12-20 »

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