Thursday, 2016-05-05

*** chem has quit IRC00:06
*** fawadkhaliq has quit IRC00:09
*** rahulait has quit IRC00:10
*** ansmith has joined #puppet-openstack00:22
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Acceptence test: rgw/civetweb with keystone  https://review.openstack.org/31118800:33
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Acceptence test: rgw/civetweb with keystone  https://review.openstack.org/31118801:02
*** ansmith has quit IRC01:10
*** xarses has joined #puppet-openstack01:20
EmilienMmwhahaha: integ jobs still fail with testr http://logs.openstack.org/87/312787/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/05277aa/console.html#_2016-05-05_00_07_53_65601:32
bkeroEmilienM: Eep! A ghost!01:34
*** xiaohua has joined #puppet-openstack01:42
*** xiaohua_ has joined #puppet-openstack01:46
mwhahaha:(01:49
*** xiaohua has quit IRC01:49
*** afranc has quit IRC01:49
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add Users/DBs related parameters to guestagent class  https://review.openstack.org/30571701:52
*** xiaohua has joined #puppet-openstack01:53
*** xiaohua_ has quit IRC01:53
*** xiaohua_ has joined #puppet-openstack02:00
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add datastore_registry_ext option to trove-guestagent.conf  https://review.openstack.org/30559102:00
*** xiaohua__ has joined #puppet-openstack02:01
*** xiaohua_ has quit IRC02:01
*** xiaohua has quit IRC02:03
*** ducttape_ has quit IRC02:05
*** ducttape_ has joined #puppet-openstack02:17
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add manifests and class specs  https://review.openstack.org/31002202:25
*** jaosorior has joined #puppet-openstack02:28
*** fawadkhaliq has joined #puppet-openstack02:32
*** jaosorior has quit IRC02:48
*** ducttape_ has quit IRC02:50
*** cdelatte has quit IRC03:03
*** xiaohua__ has quit IRC03:13
*** xiaohua has joined #puppet-openstack03:14
*** xiaohua has quit IRC03:22
*** cwolferh has quit IRC03:26
*** xiaohua has joined #puppet-openstack03:40
*** fawadkhaliq has quit IRC03:41
*** cwolferh has joined #puppet-openstack03:48
*** pleia2 has quit IRC03:51
*** xiaohua has quit IRC04:43
*** xiaohua has joined #puppet-openstack04:43
*** xiaohua has quit IRC04:52
*** fawadkhaliq has joined #puppet-openstack05:18
*** yolanda has joined #puppet-openstack05:26
*** jmikolajczak has joined #puppet-openstack05:40
openstackgerritMichael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/29343605:45
*** xiaohua has joined #puppet-openstack05:50
*** xiaohua has quit IRC05:57
*** cwolferh has quit IRC05:58
openstackgerritNate Potter proposed openstack/puppet-magnum: Update tests with rspec facts  https://review.openstack.org/30959606:01
*** openstackgerrit has quit IRC06:03
*** openstackgerrit has joined #puppet-openstack06:03
*** xiaohua has joined #puppet-openstack06:04
openstackgerritNate Potter proposed openstack/puppet-magnum: Update tests with rspec facts  https://review.openstack.org/30959606:04
*** chandankumar has joined #puppet-openstack06:08
*** saneax_AFK is now known as saneax06:12
*** tesseract has joined #puppet-openstack06:44
*** tesseract is now known as Guest2128806:45
openstackgerritMerged openstack/puppet-tripleo: Create dbs in step 3 for the roles  https://review.openstack.org/31249606:47
*** cwolferh has joined #puppet-openstack06:47
*** amoralej|off is now known as amoralej07:08
*** dtantsur|afk is now known as dtantsur07:15
*** paramite has joined #puppet-openstack07:15
*** paramite has quit IRC07:30
*** yolanda has quit IRC07:36
*** shardy has joined #puppet-openstack07:37
*** shardy has quit IRC07:38
*** shardy has joined #puppet-openstack07:38
*** paramite has joined #puppet-openstack07:44
*** yolanda has joined #puppet-openstack07:44
*** jpena|off is now known as jpena07:48
*** gfidente has joined #puppet-openstack07:49
*** gfidente has quit IRC07:49
*** gfidente has joined #puppet-openstack07:49
*** dmk0202 has joined #puppet-openstack07:57
*** partner has quit IRC07:59
*** dmk0202 has quit IRC08:03
*** fawadkhaliq has quit IRC08:06
*** yolanda has quit IRC08:08
*** dmk0202 has joined #puppet-openstack08:11
*** yolanda has joined #puppet-openstack08:13
*** derekh has joined #puppet-openstack08:23
*** pepijn has joined #puppet-openstack08:32
*** dtantsur is now known as dtantsur|brb08:33
*** jistr has joined #puppet-openstack08:34
*** foxhunt_away is now known as foxhunt7208:45
*** yottatsa has joined #puppet-openstack08:53
*** yottatsa has quit IRC09:00
*** hjensas has quit IRC09:04
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: WIP: Puppet manifest to configure NTP as a composable service  https://review.openstack.org/31072509:12
*** skolekonov_ has quit IRC09:17
*** foxhunt72 is now known as foxhunt_away09:23
*** pepijn has quit IRC09:35
*** leanderthal has quit IRC09:43
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: WIP: Composable roles NTP: PT  https://review.openstack.org/31072509:43
*** dtantsur|brb is now known as dtantsur09:51
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Adds the base and pacemaker profile for the memcached service  https://review.openstack.org/31287709:53
*** evidex has joined #puppet-openstack09:54
*** hjensas has joined #puppet-openstack09:55
*** mgould has joined #puppet-openstack10:06
*** yolanda has quit IRC10:09
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: WIP: Composable roles NTP: PT  https://review.openstack.org/31072510:10
*** xb_ has joined #puppet-openstack10:13
*** yolanda has joined #puppet-openstack10:13
*** yottatsa has joined #puppet-openstack10:28
*** mgould has quit IRC10:43
*** mgould has joined #puppet-openstack10:51
*** johnmilton has joined #puppet-openstack11:19
*** magicboiz has joined #puppet-openstack11:21
*** magicboiz has quit IRC11:24
*** yottatsa has quit IRC11:32
*** yolanda has quit IRC11:32
*** ansmith has joined #puppet-openstack11:33
*** yottatsa has joined #puppet-openstack11:37
*** yolanda has joined #puppet-openstack11:38
*** yottatsa has quit IRC11:44
*** pepijn has joined #puppet-openstack11:52
*** yottatsa has joined #puppet-openstack11:55
*** saneax is now known as saneax_AFK12:02
*** paramite has quit IRC12:06
*** ducttape_ has joined #puppet-openstack12:07
*** saneax_AFK is now known as saneax12:08
*** jayg|g0n3 is now known as jayg12:13
*** dprince has joined #puppet-openstack12:15
*** xiaohua has quit IRC12:21
*** amoralej is now known as amoralej|lunch12:23
*** ansmith has quit IRC12:23
*** alee has quit IRC12:24
*** ducttape_ has quit IRC12:24
*** yolanda has quit IRC12:38
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate unit testing wiki  https://review.openstack.org/31116112:39
*** richm has joined #puppet-openstack12:40
*** jpena is now known as jpena|lunch12:43
*** yolanda has joined #puppet-openstack12:44
*** mattymo` has joined #puppet-openstack12:49
iurygregorybkero, this is my idea i just need to fix problems in the table12:50
EmilienMhello12:50
*** morazi has joined #puppet-openstack12:53
*** bapalm has joined #puppet-openstack12:54
*** trown|outtypewww is now known as trown12:58
*** yolanda has quit IRC13:00
*** hjensas has quit IRC13:04
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux  https://review.openstack.org/30940813:05
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835813:05
*** yolanda has joined #puppet-openstack13:05
*** xiaohua has joined #puppet-openstack13:07
*** ducttape_ has joined #puppet-openstack13:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: run_tests.sh: run testr outside .tox  https://review.openstack.org/31278713:15
*** xiaohua has quit IRC13:26
*** alee has joined #puppet-openstack13:27
*** amoralej|lunch is now known as amoralej13:30
*** xiaohua has joined #puppet-openstack13:31
*** hjensas has joined #puppet-openstack13:35
*** hjensas has quit IRC13:35
*** hjensas has joined #puppet-openstack13:35
*** ducttape_ has quit IRC13:36
*** ducttape_ has joined #puppet-openstack13:44
*** openstackgerrit has quit IRC13:47
*** openstackgerrit has joined #puppet-openstack13:48
*** xiaohua has quit IRC13:49
*** dfisher has joined #puppet-openstack13:52
*** jpena|lunch is now known as jpena13:53
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: WIP: Composable roles NTP: PT  https://review.openstack.org/31072513:54
*** chandankumar has quit IRC13:54
*** moravec has quit IRC13:58
*** moravec1 has joined #puppet-openstack13:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588613:59
EmilienMjpena: testing now with your patch ^13:59
*** dfisher has quit IRC14:04
*** dfisher has joined #puppet-openstack14:05
*** mgould is now known as mgould|afk14:10
*** xiaohua has joined #puppet-openstack14:10
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux  https://review.openstack.org/30940814:14
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835814:14
*** xiaohua has quit IRC14:16
*** xarses has quit IRC14:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588614:25
*** mihgen has quit IRC14:27
*** mjb has quit IRC14:28
*** dfisher1 has joined #puppet-openstack14:29
*** jistr has quit IRC14:29
*** dfisher has quit IRC14:31
*** xarses has joined #puppet-openstack14:34
*** mjb has joined #puppet-openstack14:35
EmilienMdprince: I'm wondering if haproxy/keepalived need to be collocated14:38
EmilienMthinking at container deployments14:38
openstackgerritMerged openstack/puppet-openstack-integration: run_tests.sh: run testr outside .tox  https://review.openstack.org/31278714:39
*** dfisher1 has quit IRC14:45
*** mgould|afk is now known as mgould14:46
*** dfisher has joined #puppet-openstack14:46
mtreinishEmilienM: I honestly didn't think anyone was relying on the tox internals to access the venv14:46
EmilienMmtreinish: no worries14:46
mtreinishI updated devstack, but didn't think about other deployers14:46
EmilienMcool np14:46
EmilienMit's fixed now and merged!14:46
mtreinishEmilienM: fwiw, the new method is a bit easier you can do 'tox -venv-tempest -- pip install foo'14:46
mtreinish(which is why we made the shift to enable devstack to install tempest plugins)14:47
EmilienMmtreinish: maybe if you have time this week or next week, you can review https://github.com/openstack/puppet-openstack-integration/blob/master/run_tests.sh#L205 and send a patch to improve our way to run tempest14:48
openstackgerritNate Potter proposed openstack/puppet-magnum: Update tests with rspec facts  https://review.openstack.org/30959614:48
*** cdelatte has joined #puppet-openstack14:48
*** jistr has joined #puppet-openstack14:51
openstackgerritMatthew Treinish proposed openstack/puppet-openstack-integration: Stop using tox venv in run_tests.sh  https://review.openstack.org/31300914:56
mtreinishEmilienM: ^^^ is the start14:56
mtreinishI'm not sure how well that'll work though14:56
EmilienMoh cool14:56
EmilienMmtreinish: thanks! I'll start from here14:57
mtreinishon stable branches we'll have to move it inside a venv we create by hand14:57
*** ayoung has quit IRC14:59
*** derekh has quit IRC14:59
EmilienMmtreinish: but if we use pip, it will eventually update packages that were installed by RDO packaging, right?15:00
EmilienMwe might want to test the packages, without pip, isn't? it's not a devstack job15:00
EmilienM(even in master)15:00
*** jaosorior has joined #puppet-openstack15:02
jpenaEmilienM: I'm checking the progress of CI tests for https://review.openstack.org/#/c/305886/ . I see scenario002 has failed for centos7, but it looks like the problem is we have upgraded the version of paramiko in the host using pip, and that breaks nova crypto -> http://logs.openstack.org/86/305886/31/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3595c10/logs/nova/nova-api.txt.gz#_2016-05-05_14_51_54_82215:06
*** jaosorior has quit IRC15:06
EmilienMjpena: oh, when we pip install tempest-lib?15:08
EmilienMor something else?15:08
jpenaEmilienM: exact15:09
EmilienMdamn15:09
EmilienMwe need to install it to pin paramiko or install tempest-lib in venv15:09
EmilienMjpena: wdyt?15:10
EmilienMmtreinish: you see ^^^^ that's why I don't want to run pip outside venv15:10
EmilienMbecause we rely on (and want to test) packaging15:10
jpenaI'd run it inside a venv, just to be safe15:10
EmilienM++15:10
mtreinishEmilienM: ok lets venv it then15:11
mtreinishit makes backporting changes easier :)15:11
EmilienMmtreinish: can you update your patch?15:11
mtreinishEmilienM: but we should avoid system site-packages and manually install the plugins I think15:11
EmilienMor should it be done in a separated patch?15:12
EmilienMmtreinish: what do you suggest?15:15
EmilienMrunning everythign in a venv looks safer regading our use case15:15
*** pleia2 has joined #puppet-openstack15:15
EmilienMsince we run packaging15:15
mtreinishEmilienM: yeah I think putting it in a venv makes the most sense, but I think we should have run_tests own the venv for real15:17
*** xarses has quit IRC15:17
mtreinishEmilienM: I'll update the patch in a min, I've got a couple of other threads spinning15:17
EmilienMsure15:17
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles  https://review.openstack.org/31301915:18
*** _ody has quit IRC15:22
*** basilAB has quit IRC15:23
*** _ody has joined #puppet-openstack15:23
*** basilAB has joined #puppet-openstack15:23
*** colonwq has quit IRC15:23
*** colonwq has joined #puppet-openstack15:24
*** agrebennikov__ has joined #puppet-openstack15:25
*** yottatsa has quit IRC15:27
*** xarses has joined #puppet-openstack15:32
*** ayoung has joined #puppet-openstack15:32
*** karimb has joined #puppet-openstack15:33
*** yottatsa has joined #puppet-openstack15:34
openstackgerritzhongshengping proposed openstack/puppet-magnum: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/31302415:35
*** ducttape_ has quit IRC15:37
*** dmk0202 has quit IRC15:38
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Add loadbalancer profile for ha & non-ha  https://review.openstack.org/31302615:42
*** mattymo` has quit IRC15:44
*** ayoung has quit IRC15:46
*** Guest21288 has quit IRC15:46
*** ducttape_ has joined #puppet-openstack15:48
openstackgerritMatthew Treinish proposed openstack/puppet-openstack-integration: Stop using tox venv in run_tests.sh  https://review.openstack.org/31300915:49
mtreinishEmilienM: ^^^ I'm not sure you'll like that. To provide real isolation you have to specify the plugins to install15:49
mtreinishas opposed to today where it just picks up all the system install plugins included in the packages15:49
EmilienMmtreinish: I like it actually15:50
EmilienMmtreinish: as long as it does not break what we installed system-wide15:50
mtreinishit shouldn't you just use the TEMPEST_PLUGINS variable to pass a string of what gets installed inside the tempest venv15:51
EmilienMexcellent15:52
EmilienMlet's see if it pass15:52
mtreinishEmilienM: it's the same basic approach I did for devstack: https://review.openstack.org/31261515:53
openstackgerritNathan Taylor proposed openstack/puppet-nova: Adding a purge_config option for nova_config  https://review.openstack.org/31273415:54
*** mattymo` has joined #puppet-openstack15:56
*** saneax is now known as saneax_AFK15:59
*** mattymo` has quit IRC16:02
*** chandankumar has joined #puppet-openstack16:03
*** dtantsur is now known as dtantsur|afk16:04
*** derekh has joined #puppet-openstack16:06
*** ducttape_ has quit IRC16:06
*** evidex has quit IRC16:08
*** mattymo` has joined #puppet-openstack16:08
*** karimb has quit IRC16:10
*** ducttape_ has joined #puppet-openstack16:11
*** ducttape_ has quit IRC16:12
*** yottatsa has quit IRC16:14
*** xb_ has quit IRC16:20
*** derekh has quit IRC16:22
EmilienMmtreinish: http://logs.openstack.org/09/313009/2/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/7007ebb/console.html#_2016-05-05_16_12_23_72116:22
EmilienMI think the regex is wrong16:22
*** yolanda has quit IRC16:25
*** trown is now known as trown|lunch16:25
EmilienMmtreinish: could we run --no-discover ?16:28
EmilienMso we pass $TESTS16:28
mtreinishEmilienM: no discover takes in a python callable I don't think that would work here16:28
mtreinishEmilienM: we just need quotes around $TESTS16:29
mtreinishargparse is treating the spaces as seperate arguments and we meant for it to be a regex passed in16:29
mtreinishEmilienM: we probably should just use a whitelist file anyway16:30
*** yolanda has joined #puppet-openstack16:30
mtreinishbut I'll do that as a followup16:30
EmilienMokay16:30
*** yottatsa has joined #puppet-openstack16:31
openstackgerritMatthew J Black proposed openstack/puppet-ceilometer: Added support for messaging urls  https://review.openstack.org/30434616:32
EmilienMmtreinish: do you want me to push over your patch for quotes?16:34
EmilienMoh but I realized something16:35
EmilienMpip install -U $TEMPEST_PLUGINS16:35
EmilienMI'm not sure this is correct, because until now we were testing tempest plugins provided by packaging16:35
EmilienMI'm not sure it will be used anymore if we go that way16:36
*** xiaohua has joined #puppet-openstack16:39
*** fedexo has joined #puppet-openstack16:43
*** xiaohua has quit IRC16:43
*** agrebennikov__ has quit IRC16:45
*** yottatsa has quit IRC16:46
*** ducttape_ has joined #puppet-openstack16:46
*** xb_ has joined #puppet-openstack16:47
*** yottatsa has joined #puppet-openstack16:48
*** dprince has quit IRC16:48
*** imcsk8 has quit IRC16:51
*** imcsk8 has joined #puppet-openstack16:51
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Conditionally fall back to redhat service provider  https://review.openstack.org/31228416:55
*** mattymo` has quit IRC16:57
EmilienMskolekonov: can you update p-o-i so we don't have deprecated parameters, related to https://review.openstack.org/#/c/312446/ ? thanks17:01
skolekonovEmilienM, sure. Looks like only verbose opt is deprecated17:02
EmilienMskolekonov: thx17:03
*** fedexo has quit IRC17:04
*** yolanda has quit IRC17:07
*** yolanda has joined #puppet-openstack17:07
*** shardy has quit IRC17:07
*** mattymo` has joined #puppet-openstack17:08
*** gfidente has quit IRC17:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Migrate unit testing wiki  https://review.openstack.org/31116117:13
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: add metadata.json file  https://review.openstack.org/31306817:14
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux  https://review.openstack.org/30940817:17
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835817:17
*** jistr has quit IRC17:17
openstackgerritBen Kero proposed openstack/puppet-openstack-guide: add supported platforms doc  https://review.openstack.org/31227917:19
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add tripleo::selinux  https://review.openstack.org/30940817:19
bkeroEmilienM: should I wait for doc to merge and rebase before I update another doc review? ^17:19
EmilienMor you can do gerrit dependencies17:21
EmilienMbut either way works17:21
*** saneax_AFK is now known as saneax17:24
*** gluytium has quit IRC17:24
*** trown|lunch is now known as trown17:25
openstackgerritMatthew J Black proposed openstack/puppet-neutron: Added availability zone hints  https://review.openstack.org/31044717:26
openstackgerritSergey Kolekonov proposed openstack/puppet-openstack-integration: Remove deprecated parameters for Neutron  https://review.openstack.org/31307717:30
skolekonovGerrit seems to work very slow17:31
*** jpena is now known as jpena|off17:31
EmilienMyeah17:32
EmilienMi'm chatting on infra channel about it17:32
*** ayoung has joined #puppet-openstack17:33
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588617:37
EmilienMmwhahaha: I would like to consider this patch ^17:37
EmilienMthere is a workaround, ugly but it will help us to promote CI downstream and also in our CI, so we can move forward, until they figure how to deal with the tempest patch17:38
EmilienMconsider this workaround very temporary (until jpena|off & mtreinish figure how to deal with the actual bug when SSL is enabled on Glance)17:38
mwhahahak17:39
mwhahahadoing the ci shuffle17:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy keystone with fernet tokens  https://review.openstack.org/30545117:40
EmilienMayoung: ^ fyi, this patch should patch now we have Newton working17:40
mtreinishEmilienM: the patch from jpena|off doesn't make any sense in tempest17:40
EmilienMalee: we're close to have newton tests, please recheck/rebase your patches using https://review.openstack.org/305886 in depends-on17:40
mtreinishit did for glanceclient, where it could be running under eventlet. But tempest isn't multithreaded17:40
EmilienMmtreinish: maybe but it fix Glance tests when SSL is enabled17:40
ayoungEmilienM, awesome.17:40
EmilienMmtreinish: I'll17:42
EmilienMerr17:42
mtreinishEmilienM: it's an accident that it does. The eventlet monkey patched openssl is probably working around the real bug. Also, in the normal case it doesn't even fix tempest17:42
mtreinishit only works on the puppet jobs because they're not venv isolated17:42
EmilienMmtreinish: I'll use this patch just to make our CI happy this week but we'll drop this workaround17:42
mtreinishnormally eventlet isn't even present when running tempest17:42
EmilienMmtreinish: I commented on your patch, using venv for us is complicated because we install packages that deploy tempest plugins17:43
EmilienMmaybe we can try to mix both worlds, by not overriding system packages17:43
EmilienMlike you mentionned before17:43
mtreinishEmilienM: that's what you're doing now, you just need to add the site-packages flag to the virtualenv call17:45
*** ducttape_ has quit IRC17:45
openstackgerritMerged openstack/puppet-neutron: Add logging class  https://review.openstack.org/31244617:45
aleeEmilienM, that should be fun :)17:47
EmilienMalee: if you depends-on this patch, you'll get latest newton from today17:47
aleeEmilienM, can I have multiple depends-on ?17:48
*** dprince has joined #puppet-openstack17:48
EmilienMyes, if it's for different projects17:48
EmilienMalee: http://docs.openstack.org/infra/zuul/gating.html#cross-repository-dependencies17:48
* mjblack kicks fuel ci 17:48
*** amoralej is now known as amoralej|off17:50
* EmilienM brb lunch17:50
*** mattymo` has quit IRC17:52
openstackgerritAde Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346117:53
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346217:53
*** mihgen has joined #puppet-openstack17:55
aleeEmilienM, added depends-on ..17:55
alee^^17:55
*** mattymo` has joined #puppet-openstack18:03
*** yottatsa has quit IRC18:12
*** yottatsa has joined #puppet-openstack18:12
*** yottatsa has quit IRC18:12
openstackgerritMerged openstack/puppet-nova: Conditionally fall back to redhat service provider  https://review.openstack.org/31228418:13
*** yolanda has quit IRC18:15
*** mattymo` has quit IRC18:17
*** mgould has quit IRC18:18
mjblackmwhahaha: looks like fuel ci is broken18:20
mwhahahabetter not be18:20
mwhahahawhich change?18:20
mjblackhttps://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.neutron_vlan_ha/713/console18:20
mwhahahaeh that's just a connectivity problem18:20
mwhahahafuel: recheck18:21
mjblackjust did18:21
mjblackthats the second one18:21
mwhahahais gerrit having problems?18:21
mjblackdont think so, other CI is working18:21
mwhahahai think it's intermittent18:21
mwhahahai'll check on it in a bit18:21
mwhahahareview.o.o was slow for me18:21
_odyOh yeah it is.  Time out slow.18:23
mwhahaharecheck should be going through now18:24
mjblackyay18:25
mwhahahaEmilienM: http://logs.openstack.org/86/305886/32/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/83b4733/console.html#_2016-05-05_18_01_25_628 your ci hack fails on the cherry-pick18:25
*** ducttape_ has joined #puppet-openstack18:27
openstackgerritCody Herriges proposed openstack/puppet-openstack_spec_helper: Explicitly define the fact puppetversion  https://review.openstack.org/31276918:27
*** chandankumar has quit IRC18:31
EmilienMmwhahaha: weird, maybe we need to checkout master first18:34
EmilienMah in fact we should checkout, err18:38
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588618:38
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy keystone with fernet tokens  https://review.openstack.org/30545118:39
*** hjensas has quit IRC18:47
openstackgerritMatthew Treinish proposed openstack/puppet-openstack-integration: Stop using tox venv in run_tests.sh  https://review.openstack.org/31300918:48
openstackgerritMerged openstack/puppet-cinder: Prepare module for publication to the forge  https://review.openstack.org/31220218:50
openstackgerritMerged openstack/puppet-horizon: Prepare module for publication to the forge  https://review.openstack.org/31222318:50
openstackgerritMerged openstack/puppet-sahara: Prepare module for publication to the forge  https://review.openstack.org/31222818:51
openstackgerritMerged openstack/puppet-trove: Prepare module for publication to the forge  https://review.openstack.org/31225218:51
EmilienMmtreinish: what do you suggest for the glance case?18:52
mtreinishEmilienM: I don't fully understand the bug yet, so I don't have a path forward in mind right now18:54
EmilienMmtreinish: your patch looks cool ^18:54
EmilienMmtreinish: so we venv everything except what we already have system-wide?18:54
mtreinishEmilienM: yeah more or less18:55
EmilienMmtreinish: the thing is: we're not devstack-like jobs, we use packaging as we can, because our operators use it.18:56
EmilienMthat's the reason why we want to use packaging and fail if something's wrong. Of course venv'ing everything would be much easier, but that's not how we want to test18:57
EmilienMwe want to make sure that by using our puppet modules, you can actually install, configure and test OpenStack18:57
openstackgerritMerged openstack/puppet-gnocchi: Prepare module for publication to the forge  https://review.openstack.org/31226418:58
openstackgerritMerged openstack/puppet-heat: Prepare module for publication to the forge  https://review.openstack.org/31222018:58
mtreinishfwiw, the last time I deployed a cloud (a couple of years ago) I just used the tarballs. But, maybe I'm weird18:58
EmilienMlol18:58
EmilienMyou're not weird, but most of people use packaging18:58
EmilienMyou can ask operators, they either use packaging provided by UCA or RDO (or they build their own), or they build images for containers (but usually built with packages)18:59
*** xiaohua has joined #puppet-openstack19:02
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: add metadata.json file  https://review.openstack.org/31306819:04
*** haplo37 has joined #puppet-openstack19:04
*** xiaohua has quit IRC19:07
mtreinishEmilienM: on the glance thing can you tell me which version of pyopenssl is installed on the system on the mitaka jobs vs the newton jobs?19:07
EmilienMmtreinish: I already compared both and they were the same, but let me recheck again19:08
EmilienMpyOpenSSL-0.15.1-1.el7.noarch on mitaka19:08
openstackgerritMerged openstack/puppet-glance: Prepare module for publication to the forge  https://review.openstack.org/31220519:09
openstackgerritMerged openstack/puppet-ceilometer: Prepare module for publication to the forge  https://review.openstack.org/31223619:09
openstackgerritMerged openstack/puppet-tempest: Prepare module for publication to the forge  https://review.openstack.org/31224619:09
openstackgerritMerged openstack/puppet-neutron: Prepare module for publication to the forge  https://review.openstack.org/31221319:10
EmilienMand same on newton19:10
mtreinishEmilienM: but are they the same inside the tox venv? Because that's where it really matters19:10
* EmilienM checks again19:10
EmilienMbut the thing is, I checked on a job that ran today19:11
EmilienMso they should have same version19:11
EmilienMsince we only pin Yum repos19:11
mtreinishI'm just wondering if it's pulling in a later version inside the venv for some reason (this is why I dislike venvs with system-site-packages because you never know which version is being used)19:11
*** xb_ has quit IRC19:12
mtreinishEmilienM: http://logs.openstack.org/86/305886/29/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/616bb73/console.html#_2016-05-04_23_25_06_090 it's got 16.0.0 there19:12
mtreinishthat's from the bug so running on newton19:12
openstackgerritRyan Bak proposed openstack/puppet-monasca: Fix for custom agent roles  https://review.openstack.org/31311919:12
EmilienMlet me look in mitaka19:12
mtreinishmy thinking right now is we're pulling in a newer pyopenssl on the newton jobs and that's causing the issue19:13
mtreinishand since we don't test ssl anywhere else it's gone unnoticed until now that tempest with latest pyopenssl doesn't work for glance upload19:13
EmilienMthis is a very recent mitaka job: http://logs.openstack.org/84/312284/4/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/8301524/console.html19:13
EmilienMpyOpenSSL==16.0.019:14
mtreinishok, well there goes my theory :)19:14
EmilienMmtreinish: the question is why does it work on mitaka?19:14
mtreinishright, tempest is the same, so what is different?19:14
EmilienMwhy the same tests work fine19:14
EmilienMjust packaging19:15
EmilienMI can give a list of packages diff19:15
EmilienMgive me a sec19:15
mtreinishanyway I've got to go run an errand, I'll be back in a little while19:15
EmilienMok ttyl19:15
EmilienMpackage diff: https://www.diffchecker.com/4fubwy3o19:18
EmilienMpython2-requestsexceptions is not the same, not sure we care19:19
-openstackstatus- NOTICE: Gerrit is restarting to address performance issues related to a suspected memory leak19:22
*** trown is now known as trown|outtypewww19:26
iurygregorybkero, are you around?19:26
openstackgerritMerged openstack/puppet-nova: Prepare module for publication to the forge  https://review.openstack.org/31220419:27
openstackgerritMerged openstack/puppet-ironic: Prepare module for publication to the forge  https://review.openstack.org/31224219:27
iurygregorybkero, https://review.openstack.org/#/c/311161/ this is my idea, you have problem in abandon https://review.openstack.org/#/c/312285/ and https://review.openstack.org/#/c/312282/ ?19:28
openstackgerritMerged openstack/puppet-keystone: Prepare module for publication to the forge  https://review.openstack.org/31214119:28
*** trown|outtypewww is now known as trown19:29
*** trown is now known as trown|outtypewww19:32
*** hjensas has joined #puppet-openstack19:33
*** hjensas has joined #puppet-openstack19:33
openstackgerritRyan Bak proposed openstack/puppet-monasca: Fix for custom agent roles  https://review.openstack.org/31311919:34
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: Add redis profile  https://review.openstack.org/30475419:39
openstackgerritMerged openstack/puppet-openstacklib: Prepare module for publication to the forge  https://review.openstack.org/31213319:46
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Stop using tox venv in run_tests.sh  https://review.openstack.org/31300919:49
EmilienMmtreinish: trying to make it pass wit ostestr ^19:49
iurygregoryEmilienM, we should migrate https://wiki.openstack.org/wiki/Meetings/PuppetOpenStack ?19:51
EmilienMiurygregory: not yet19:52
EmilienMthis page is very dynamic now19:52
EmilienMnibalizer: are we going to drop wiki entirely?19:52
nibalizerideally19:52
EmilienMmhh, ok19:52
EmilienMiurygregory: yes then19:53
EmilienMgo ahead19:53
iurygregoryok EmilienM o/19:53
nibalizerwe really want to know what use cases can't be filled by other tools in infraa so we caan help get thaat up19:53
bkeroiurygregory: just got back from lunch19:53
nibalizerlike we'd cry if all closing the wiki did was move things into google docs19:53
bkeroiurygregory: looks good to me19:54
*** dprince has quit IRC19:54
iurygregorytks bkero =)19:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588619:57
EmilienMnibalizer: lol19:57
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy keystone with fernet tokens  https://review.openstack.org/30545119:58
_odyIf I could get one more set of eyes on https://review.openstack.org/#/c/312266, it looks like I'll be able to do forge publication.19:59
iurygregorygood luck _ody o/20:04
*** ayoung has quit IRC20:10
_odyiurygregory: Thanks!20:14
*** jayg is now known as jayg|g0n320:26
*** dmk0202 has joined #puppet-openstack20:29
-openstackstatus- NOTICE: Gerrit is restarting to revert incorrect changes to test result displays20:30
*** cdelatte has quit IRC20:34
EmilienMmtreinish: I think I messed up something in my last PS http://logs.openstack.org/09/313009/4/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/1db72d0/console.html#_2016-05-05_20_16_17_09120:39
openstackgerritMerged openstack/puppet-openstack_extras: Prepare module for publication to the forge  https://review.openstack.org/31224320:39
openstackgerritMerged openstack/puppet-aodh: Prepare module for publication to the forge  https://review.openstack.org/31226620:39
*** pepijn has quit IRC20:41
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/31315320:50
*** alee has quit IRC20:55
*** foxhunt_away is now known as foxhunt7220:55
EmilienMmwhahaha: can I get approve on https://review.openstack.org/311786 ?20:56
mwhahahak20:57
*** haplo37 has quit IRC20:57
EmilienMmwhahaha: did you have feedback on https://review.openstack.org/312779 ?20:59
EmilienMwho is good at doc? :-P21:00
mwhahahaha! no one. hence poor documentation all around! :D21:00
mwhahahai ran out of time yesterday to come up with an alternative, not sure if i'll have time today21:00
EmilienMmfisch: who is pushing on puppet-monasca here?21:01
EmilienMmfisch: I'm still wondering when collaboration will start21:01
EmilienMeverytime they tell me now, but we never see it. I still see the module not compliant, not released, etc21:02
EmilienMmwhahaha: no worries it can wait21:03
EmilienMcrinkle: good candidate for backport IMHO https://review.openstack.org/31230021:04
*** johnmilton has quit IRC21:04
*** fawadkhaliq has joined #puppet-openstack21:04
*** saneax is now known as saneax_AFK21:05
*** rcernin has joined #puppet-openstack21:05
EmilienMmfisch: can we have a release note on https://review.openstack.org/#/c/312661/ ?21:07
openstackgerritMerged openstack/puppet-swift: Prepare module for publication to the forge  https://review.openstack.org/31223221:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588621:12
crinkleEmilienM: hrm it looks like the parameter changed from neutron_auth_plugin to neutron_auth_type so it doesn't cherry-pick cleanly to mitaka21:14
mtreinishEmilienM: you're not calling ostestr from inside the venv anymore21:15
mtreinishso it's missing the tempest deps21:15
EmilienMmtreinish: yeah, I did wrong21:15
*** xingchao has joined #puppet-openstack21:19
*** foxhunt72 is now known as foxhunt_away21:19
EmilienMmtreinish: but in your patch, ostestr could not be found21:22
mtreinishright that's the weird thing, I'm thinking it's the system-site-packages thing. ostestr is installed in system-site-packages so pip isn't installing it in the venv (which means no binary)21:24
mtreinishhaving a venv with system site-packages makes things much mroe confusing :)21:24
*** cdelatte has joined #puppet-openstack21:24
EmilienMmwhahaha, crinkle: I'm about to go running now. I would like to start testing Newton packaging, and we need https://review.openstack.org/#/c/305886/ - you'll notice the ugly workaround that aims to be temporary. Please review the patch and give feedback. So maybe we can move forward. Thanks21:25
mwhahahak21:25
EmilienMmtreinish: what do you suggest?21:25
EmilienMif we venv'ing everything, we're stopping to test packages21:25
mtreinishwe can just manually install ostestr, that might work21:26
*** dfisher has left #puppet-openstack21:26
*** dfisher has joined #puppet-openstack21:26
openstackgerritMatthew Treinish proposed openstack/puppet-openstack-integration: Stop using tox venv in run_tests.sh  https://review.openstack.org/31300921:27
mtreinishEmilienM: ^^^ lets see if that does it21:27
EmilienMmtreinish: thanks, I'll look results when i come back /me afk21:27
crinkleEmilienM: will look in a bit21:27
EmilienMcrinkle: if you like it, feel fee to +2 only (if CI did not finished yet), I'll +A myself later tonight, otherwise -121:28
*** xiaohua has joined #puppet-openstack21:30
openstackgerritRyan Bak proposed openstack/puppet-monasca: Fix for custom agent roles  https://review.openstack.org/31311921:31
*** fawadkhaliq has quit IRC21:33
crinkleEmilienM: uh i haven't been following the context but i don't feel good about fetching a tempest patch that mtreinish has -2'd21:34
*** xiaohua has quit IRC21:34
*** xingchao has quit IRC21:36
*** cwolferh has quit IRC21:48
*** alee has joined #puppet-openstack21:52
*** ayoung has joined #puppet-openstack22:03
*** cwolferh has joined #puppet-openstack22:06
*** ducttape_ has quit IRC22:06
*** mattymo` has joined #puppet-openstack22:06
openstackgerritCody Herriges proposed openstack/puppet-manila: Fix typo in metadata.json  https://review.openstack.org/31317422:11
*** jmikolajczak has quit IRC22:18
*** cdelatte has quit IRC22:22
openstackgerritMerged openstack/puppet-nova: Fix default keystone auth_plugin for nova/neutron  https://review.openstack.org/31230022:23
*** rcernin has quit IRC22:29
openstackgerritMerged openstack/puppet-openstack-integration: cinder: deploy cinderv3 on ipv6/ssl  https://review.openstack.org/31265222:31
*** dfisher has quit IRC22:33
openstackgerritMerged openstack/puppet-neutron: Add oslo.messaging notitication related options  https://review.openstack.org/31258322:40
*** mattymo` has quit IRC22:48
*** cdelatte has joined #puppet-openstack22:51
*** ducttape_ has joined #puppet-openstack22:54
EmilienMcrinkle: I don't like it neither22:54
EmilienMcrinkle: the thing is, nobody in upstream CI except us is testing SSL deployments (except tripleo but they don't run tempest)22:55
EmilienMcrinkle: we'll wait tomorrow to see how thing work in the tempest patch and if they manage to find a concensus22:56
EmilienMcrinkle: we have 3 options: 1) disable the Glance tests 2) carry this custom patch until it's sorted in tempest 3) delay Newton testing22:57
EmilienMcrinkle: 1) is a regression because the tests are the only ones that actually test Glance upload22:57
EmilienMI don't like 3) because it's slowing down our development process22:57
EmilienMI hate 2) also but it's ok if it's temporary (less than one week)22:58
EmilienMI'm open to suggestions23:02
*** saneax_AFK is now known as saneax23:03
*** ayoung has quit IRC23:07
mfischEmilienM: yes on release notes23:19
EmilienMmfisch: thanks!23:20
EmilienMsorry for -123:20
EmilienMbut I'm afraid some people don't see notific on +0 reviews23:20
EmilienMso I -1 to make sure it's visible in gerrit23:20
mwhahahapfft, make them really notice with a -2 ;)23:21
EmilienMlol23:21
*** vinsh_ has quit IRC23:31
*** cdelatte has quit IRC23:34
*** dmk0202 has quit IRC23:42
EmilienMyouhouuu puppet4 jobs are green again - thanks _ody for your fix in puppet-nova!23:43
*** dtantsur|afk has quit IRC23:47
*** morazi has quit IRC23:48
*** johnmilton has joined #puppet-openstack23:48
*** mkarpin has quit IRC23:49
*** xarses has quit IRC23:51
*** mkarpin has joined #puppet-openstack23:52
*** johnmilton has quit IRC23:52
*** ayoung has joined #puppet-openstack23:53
*** xiaohua has joined #puppet-openstack23:57
*** karimb has joined #puppet-openstack23:58

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