Thursday, 2016-06-16

« Wednesday, 2016-06-15 Index Friday, 2016-06-17 »
*** openstack has joined #puppet-openstack05:59
*** jmikolajczak has joined #puppet-openstack06:02
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859106:14
*** yolanda has joined #puppet-openstack06:18
*** yolanda has quit IRC06:21
*** yolanda has joined #puppet-openstack06:24
*** rcernin has joined #puppet-openstack06:24
*** saneax_AFK is now known as saneax06:25
*** zhongshengping has joined #puppet-openstack06:34
*** zhongshengping has quit IRC06:34
*** dgurtner has joined #puppet-openstack06:51
*** cgfbee has quit IRC06:55
*** markvoelker_ has quit IRC07:01
*** markvoelker has joined #puppet-openstack07:01
*** amoralej|off is now known as amoralej07:05
*** tesseract has joined #puppet-openstack07:09
*** foxhunt_away is now known as foxhunt7207:13
*** cgfbee has joined #puppet-openstack07:15
*** permalac has quit IRC07:19
*** ifarkas has joined #puppet-openstack07:19
*** jpena|off is now known as jpena07:23
*** hjensas__ has joined #puppet-openstack07:23
*** Guest61676 is now known as leanderthal07:25
*** shardy has joined #puppet-openstack07:30
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to configure cleaning_network_uuid  https://review.openstack.org/32891607:32
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Allow to set provisioning_network_uuid  https://review.openstack.org/32892107:32
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/33035107:34
jaosorioranybody around yet?07:37
*** acabot has joined #puppet-openstack07:42
*** danpawlik has joined #puppet-openstack07:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/32702907:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/32706907:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747307:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748207:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885907:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/32929907:47
*** dtantsur|afk is now known as dtantsur07:50
*** samuelBartel has joined #puppet-openstack07:53
*** saneax is now known as saneax_AFK07:56
*** queretaro has joined #puppet-openstack08:05
*** osp has quit IRC08:06
*** jaosorior has quit IRC08:07
*** jaosorior has joined #puppet-openstack08:08
*** permalac has joined #puppet-openstack08:09
*** hjensas_ has joined #puppet-openstack08:11
*** chem```` has joined #puppet-openstack08:12
*** chem``` has quit IRC08:14
*** hjensas__ has quit IRC08:14
*** chem```` is now known as chem08:20
*** karimb has joined #puppet-openstack08:21
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/33035108:23
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/33035108:24
*** paramite has joined #puppet-openstack08:27
openstackgerritBartek Żurawski proposed openstack/puppet-nova: Add support for MultiStrOpt for nova_config  https://review.openstack.org/32999108:33
openstackgerritVasyl Saienko proposed openstack/puppet-ironic: Add ironic::drivers::interfaces class  https://review.openstack.org/32977508:41
*** saneax_AFK is now known as saneax08:44
*** derekh has joined #puppet-openstack08:47
*** foxhunt72 is now known as foxhunt_away08:54
*** foxhunt_away is now known as foxhunt7208:55
*** dtantsur is now known as dtantsur|brb09:00
*** dmk0202 has joined #puppet-openstack09:02
*** ifarkas has quit IRC09:06
*** foxhunt72 is now known as foxhunt_away09:15
*** foxhunt_away is now known as foxhunt7209:18
*** iberezovskiy|off is now known as iberezovskiy09:31
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Switch defaults for nova upgrade parameters to $::os_service_default  https://review.openstack.org/33040809:33
*** hjensas_ has quit IRC09:38
*** samuelBartel has quit IRC09:56
*** ifarkas has joined #puppet-openstack09:57
*** _degorenko|afk is now known as degorenko10:11
*** dtantsur|brb is now known as dtantsur10:13
*** jtomasek_ has joined #puppet-openstack10:30
*** hjensas_ has joined #puppet-openstack10:42
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: test  https://review.openstack.org/33045210:50
*** xb_ has joined #puppet-openstack10:50
*** cdelatte has joined #puppet-openstack10:57
*** jtomasek_ has quit IRC10:59
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/32343111:00
*** dgurtner has quit IRC11:26
*** johnmilton has joined #puppet-openstack11:27
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support UCA Newton  https://review.openstack.org/33008811:29
*** johnmilton has quit IRC11:31
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/32706911:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747311:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748211:37
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885911:37
jaosoriordegorenko, iberezovksiy: ping11:39
*** johnmilton has joined #puppet-openstack11:43
*** danpawlik has quit IRC11:44
jaosorioriberezovskiy: Hey, I was looking at the puppet-cinder SSL options11:46
iberezovskiyhey11:46
jaosoriorit is a bit awkward that they are in init.pp though. Most of the other services (keystone, glance, heat) have those options in api.pp11:47
jaosoriorwould it be appropriate to move the SSL options to cinder's api.pp?11:47
jaosoriorit would be the same case for puppet-neutron11:47
iberezovskiyyes, but you should make this changes backward-compatible11:47
jaosorioriberezovskiy: I was thinking of passing these parameters in init.pp to a deprecated section11:48
iberezovskiyit still hould be possible to use init clsss to set these params11:48
jaosoriorok11:48
iberezovskiyand use pick in api class11:48
iberezovskiysmth like this11:48
jaosorior??11:49
*** jpena is now known as jpena|lunch11:50
iberezovskiypick($::heat::ssl_key, $::heat::api::ssl_key) in api class11:51
iberezovskiyjust an example11:51
iberezovskiyjaosorior, https://github.com/openstack/puppet-cinder/blob/master/manifests/db.pp#L5411:52
jaosorioriberezovskiy: excellent11:53
jaosoriorwill do11:53
jaosoriorthanks man11:53
iberezovskiythanks11:53
*** ducttape_ has quit IRC11:54
*** dprince has joined #puppet-openstack11:54
*** ducttape_ has joined #puppet-openstack11:54
iurygregorymorning guys =D11:58
*** amoralej is now known as amoralej|lunch11:58
jaosorioriberezovskiy: in init.pp do I move those parameters to deprecated?11:59
*** shardy has quit IRC12:04
*** ducttape_ has quit IRC12:07
*** jayg|g0n3 is now known as jayg12:08
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/33049812:13
*** danpawlik has joined #puppet-openstack12:15
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/33049812:16
*** samuelBartel has joined #puppet-openstack12:17
*** dgurtner has joined #puppet-openstack12:17
*** dgurtner has quit IRC12:17
*** dgurtner has joined #puppet-openstack12:17
*** rcernin has quit IRC12:24
*** trown|outtypewww is now known as trown12:30
EmilienMgood morning12:35
degorenkomorning :)12:35
*** ducttape_ has joined #puppet-openstack12:36
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-neutron: Move SSL related options to api.pp  https://review.openstack.org/33051112:37
*** ducttape_ has quit IRC12:38
*** rcernin has joined #puppet-openstack12:38
iberezovskiyjaosorior, yep12:46
iberezovskiyEmilienM, morning12:46
*** igajsin has joined #puppet-openstack12:47
*** igajsin has left #puppet-openstack12:47
EmilienMiberezovskiy: can I have a tl;dr on ci status please?12:47
iurygregorymorning EmilienM o/12:48
iberezovskiyEmilienM, so the yesterday failure on my patch was related to floating problem, that newton repo wasn't apllied (you can go through logs and see this)12:48
iberezovskiyI've recheck and now it should work12:48
iberezovskiypatch is https://review.openstack.org/#/c/330088/12:50
EmilienMiberezovskiy: what is the issue with floating ip?12:51
iberezovskiynot ip :)12:51
EmilienMiberezovskiy: gate-puppet-openstack-integration-3-scenario003-tempest-centos-7 is failing12:51
EmilienM7 tempest tests12:51
iberezovskiyso when you run include ::openstack_integration::repos sometimes it doesn't create file ubuntu-cloud-archive.list12:51
iberezovskiyand as result newton repo isn't enabled12:52
iberezovskiyI am on it already12:52
EmilienMwhat?12:52
EmilienMsorry12:52
EmilienMWHAAATTT12:52
iurygregorylol12:52
iurygregory'sometimes'... =X12:52
EmilienMiberezovskiy: is it because the repo is not reachable?12:53
iberezovskiyhttp://logs.openstack.org/88/330088/3/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/d9e1d2c/console.html#_2016-06-15_18_02_43_10593012:53
iberezovskiyhttp://logs.openstack.org/88/330088/4/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/3630e7d/console.html#_2016-06-16_11_36_51_03200312:53
iberezovskiysee difference in puppet logs12:53
iberezovskiyresource apt::source wasn't called12:53
*** rcernin has quit IRC12:54
EmilienMmh weird12:54
EmilienMand you have a solution?12:54
iberezovskiynope12:54
iberezovskiyI am going investigate after we fix beaker tests12:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001/ubuntu: re-enable telemetry  https://review.openstack.org/33018812:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001/ubuntu: re-enable telemetry  https://review.openstack.org/33018812:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration:  scenario002/ubuntu: re-enable ironic  https://review.openstack.org/33018912:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario003/ubuntu: re-enable trove  https://review.openstack.org/33019112:56
EmilienMiberezovskiy: ack12:56
EmilienMiberezovskiy: I'm reading https://review.openstack.org/#/c/330088/4/manifests/init.pp12:57
EmilienMiberezovskiy: so on centos7 you still allow to run keystone eventlet and not on ubuntu, where you hardcode apache?12:58
iberezovskiyyep, because rdo provide service script for keystone12:58
EmilienMhum I don't think so12:58
iberezovskiyo_O12:59
EmilienMI think now keystone scripts does not exist12:59
*** zhongshengping has joined #puppet-openstack12:59
EmilienMlet me propose something:12:59
EmilienMif $service_name = 'httpd'; then httpd bits (remove keystone::service bits)13:00
EmilienMelse: warning('puppet-keystone only manage deployments with httpd, otherwise manage the service yourself') or something like this13:00
EmilienMwe reduce code & complexity13:01
iberezovskiyheh, it was my first idea :D13:01
bkerosounds like git reverting vinsh's keystone service managing patch13:01
vinshNot I13:02
*** zhongshengping has quit IRC13:02
iberezovskiyI even started to work on here https://review.openstack.org/#/c/329567 :D13:02
iberezovskiyso if you are ok, I fine as well :)13:02
*** jpena|lunch is now known as jpena13:03
EmilienMiberezovskiy: I commented https://review.openstack.org/#/c/32956713:05
EmilienMbut yeah I prefer this approach13:05
*** amoralej|lunch is now known as amoralej13:05
iberezovskiyEmilienM, you are right about rdo packages, I missed this. so I'm working on this soltuion13:06
EmilienMexcellent13:06
EmilienMiurygregory: do you have time to discuss about auth token?13:06
*** rcernin has joined #puppet-openstack13:06
*** dgurtner has quit IRC13:10
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/32747313:17
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/32748213:17
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/32885913:17
iurygregoryEmilienM, yep13:20
openstackgerritSanjay Upadhyay proposed openstack/puppet-neutron: Cleanup for sriov-agent sriov-agent-required  https://review.openstack.org/33054413:25
EmilienMiurygregory: ok cool13:25
*** v1k0d3n has joined #puppet-openstack13:25
EmilienMiurygregory: so the plan is to first cleanup puppet-keystone13:25
*** v1k0d3n has quit IRC13:25
iurygregoryok13:26
EmilienMhttps://github.com/openstack/puppet-keystone/blob/master/manifests/resource/authtoken.pp13:26
*** Guest48217 has joined #puppet-openstack13:26
EmilienMwe could use os_service_default13:26
EmilienMand avoid all the conditionals13:26
iurygregoryoh yeah big +113:26
iurygregory:D13:26
EmilienMgood so that's the first step.13:26
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova: Use correct conf group for SSL options  https://review.openstack.org/33055013:27
EmilienMin the same patch we can also update parameters to be really compliant with latest keystone middleware13:27
EmilienMdocs.openstack.org/developer/keystonemiddleware/middlewarearchitecture.html13:27
EmilienMwww.jamielennox.net/blog/2015/02/23/v3-authentication-with-auth-token-middleware/13:27
iurygregorythanks for the links =D13:27
EmilienMwhen we have the patch ready, we can ask richm and/or ayoung to review it13:27
ayoungEmilienM, you can always ask13:28
iurygregorytomorrow morning i'll have the patch o/13:28
EmilienMiurygregory: and when we land it, we'll take over richm's patch in puppet-glance13:28
EmilienMbut split it in 213:28
EmilienM1) update config first, using puppet-keyste define13:28
EmilienM2) then update provider13:28
iurygregoryok =)13:28
EmilienMwhen 1) is done, reviewed & merged, we can do other modules (and distribute the work) like we did for oslo things13:29
EmilienMand do 2) in parallel13:29
iurygregoryhttps://review.openstack.org/#/c/176150/ ?13:29
iurygregorythis is the patch right?13:29
EmilienMyes13:29
EmilienMhttps://review.openstack.org/#/c/176150/ will be 2). We'll rebase it on top of 1)13:29
iurygregoryEmilienM, ack13:30
EmilienMiurygregory: I'll let you do the first step and let me know when we can start reviewing. I'll help in updating our modules with the new Define13:31
iurygregoryEmilienM, ok o/13:32
iurygregoryEmilienM, thanks for all directions =D13:33
EmilienMiurygregory: cool13:33
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/32956713:35
*** furlongm_ is now known as furlongm13:41
*** rmart04 has joined #puppet-openstack13:41
*** ducttape_ has joined #puppet-openstack13:43
*** jmikolajczak has quit IRC13:48
openstackgerritVenkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Add user & user_role parameters in auth manifest and related tests  https://review.openstack.org/32429613:50
*** ducttape_ has quit IRC13:52
*** dfisher has joined #puppet-openstack13:52
bkeroradez: was looking at making integration tests for congress, but it looks like your latest commit nerfed the whole thing.13:52
bkero(puppet-congress)13:52
bkeroWere you planning on adding the content back at some point? :)13:53
*** richm has joined #puppet-openstack13:58
*** xarses_ has quit IRC13:59
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/32956714:02
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/33049814:11
EmilienMiberezovskiy: https://review.openstack.org/#/c/329567/6/manifests/init.pp14:13
EmilienMso imho you should not deprecate it14:13
EmilienMoh wait14:13
EmilienMwait wait wait14:13
EmilienMyeah so14:14
EmilienMthe way you're doing it, you enforce people to deploy puppet-keystone with apache14:14
EmilienMit's not good14:14
EmilienMso you need to keep service_name parameter14:14
EmilienMdo not deprecate it, keep it14:14
EmilienMiberezovskiy: does it make sense?14:14
iberezovskiyEmilienM, what this parameter should do in that case?14:16
EmilienMiberezovskiy: ok I'll comment in gerrit, easier14:17
*** bzurawski has joined #puppet-openstack14:19
*** Guest48217 has quit IRC14:20
EmilienMiberezovskiy: look14:22
EmilienMlet me know if you get it14:22
iberezovskiyEmilienM, I can't put httpd by default, because 'service httpd restart' will fail on ubuntu14:24
iberezovskiylet's use keystone::params::httpd_service_name14:24
EmilienMno14:24
EmilienMsee my comment on the left14:24
EmilienML99514:24
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Implement api.pp  https://review.openstack.org/32983014:25
iberezovskiyah, I got what I've missed before, so I'll fix14:26
iberezovskiyI'll wait for CI, then update14:26
EmilienMiberezovskiy: well, you can go ahead14:26
danpawlikEmilienM: I really try to push small commits ;)14:27
*** jistr is now known as jistr|mtg14:28
*** ifarkas has quit IRC14:28
EmilienMdanpawlik++14:28
*** trown is now known as trown|brb14:44
*** xarses_ has joined #puppet-openstack14:47
*** jaosorior has quit IRC14:49
openstackgerritVasyl Saienko proposed openstack/puppet-openstacklib: Introduce openstack_resource_id_getter function  https://review.openstack.org/33062314:51
*** trown|brb is now known as trown14:52
iberezovskiyEmilienM, good to know that beaker&integration jobs are green on my patch :) will upload patch with you comments resolved soon14:53
EmilienMexcellent14:54
*** dtantsur is now known as dtantsur|bbl14:58
*** chem has quit IRC14:59
*** chem has joined #puppet-openstack15:00
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/32956715:05
*** v1k0d3n has joined #puppet-openstack15:06
*** mattymo_ is now known as mattymo15:10
*** danpawlik has quit IRC15:14
*** dgurtner has joined #puppet-openstack15:17
*** jmiu_ has joined #puppet-openstack15:17
*** jistr|mtg is now known as jistr15:19
*** ifarkas has joined #puppet-openstack15:24
*** evidex has joined #puppet-openstack15:27
EmilienMiberezovskiy: reviewing15:29
*** zhongshengping has joined #puppet-openstack15:31
*** vinsh has quit IRC15:32
EmilienMiberezovskiy: commented, almost good15:32
*** zhongshengping has quit IRC15:32
*** vinsh has joined #puppet-openstack15:33
*** andreww has joined #puppet-openstack15:33
*** andreww has quit IRC15:33
*** xarses_ has quit IRC15:33
*** andreww has joined #puppet-openstack15:34
*** openstackgerrit has quit IRC15:34
iberezovskiyEmilienM, I accidently move some lines. will fix15:34
EmilienMthx15:34
iberezovskiys/move/moved15:34
*** openstackgerrit has joined #puppet-openstack15:34
EmilienMiberezovskiy: I had more comments also, refresh :P15:34
EmilienMwe're almost there15:34
iberezovskiyEmilienM, yep, but I've broke fuel :D15:35
EmilienMah15:35
EmilienMyour patch should be backward compatible15:35
EmilienMtry again!15:35
iberezovskiya bit now, it forces to use httpd because I'm changing default15:36
iberezovskiys/a bit now/a bit not15:36
EmilienMyeah, but I guess fuel is deploying keystone with httpd, no?15:36
iberezovskiynot on the master node15:36
EmilienMo_O15:36
iberezovskiyyeah...15:36
EmilienMyou use eventlet?15:36
EmilienMjust don't tell ayoung15:36
*** leanderthal is now known as leanderthal|afk15:38
*** rmart04 has quit IRC15:38
*** foxhunt72 is now known as foxhunt_away15:39
iberezovskiyyep, eventlet15:39
iberezovskiyit's not hard to solve I think15:39
*** vinsh_ has joined #puppet-openstack15:40
*** vinsh_ has quit IRC15:40
iberezovskiyI'll make the change not-breakable for Fuel15:40
EmilienMiberezovskiy: for now our beaker jobs are broken and we might need them fixed by today. Can we do baby steps and fix at least beaker jobs?15:40
*** vinsh_ has joined #puppet-openstack15:41
EmilienMand then rebase your patch on top of the fix15:41
EmilienMiberezovskiy: you ok?^15:43
*** vinsh has quit IRC15:44
iberezovskiyI can adapt fuel fast15:44
*** saneax is now known as saneax_AFK15:44
EmilienMiberezovskiy: as long as we have a fix merged by today, I'm fine15:46
iberezovskiyI think it's possible because mwhahaha can merge fuel patch right after keystone one :)15:46
*** evidex has quit IRC15:48
*** vinsh has joined #puppet-openstack15:49
iberezovskiymwhahaha, please take a look on https://review.openstack.org/#/c/330654/15:50
mwhahahak15:50
iberezovskiyI don't need to add depens-on probably, because it can be merged before15:51
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Limit CentOS SIG Storage support to Ceph hammer  https://review.openstack.org/33004115:53
mwhahahawhich was the upstream change that broken this one?15:53
*** vinsh_ has quit IRC15:53
iberezovskiymwhahaha, probably this one will be https://review.openstack.org/#/c/329567/15:54
iberezovskiynoop tests are also broken, damn15:55
mwhahahasoooo i'm going to say -1 on that due to backwards compatibility breakage15:55
EmilienMiberezovskiy: but you realize the keystone eventlet service won't be managed anymore? you manage it in fuel itself?15:55
*** evidex has joined #puppet-openstack15:55
iberezovskiyEmilienM, we still have mitaka keystone on the master node15:56
iberezovskiythat's the problem :(15:56
EmilienMthis problem is more complex than expected15:56
mwhahahabut this isn't just a fuel thing, i don't think we should make that hard definition as the default15:56
EmilienMcan we just push a little patch to fix beaker jobs?15:56
*** andreww is now known as xarses15:57
iberezovskiyEmilienM, the smaller patch is https://review.openstack.org/#/c/33008815:57
iberezovskiyneed to replace keystone::params:httpd:service_name to ::apache...15:58
EmilienMiberezovskiy: ok let's push https://review.openstack.org/#/c/330088 first and you'll rebase on top of this one15:59
iberezovskiyok15:59
EmilienMso you have time tonight or tomorrow to fix at your convenience15:59
EmilienMiberezovskiy: do you need another patchset?16:00
*** evidex has quit IRC16:00
iberezovskiyyep, will prepare in a few minutes16:00
EmilienMok16:00
EmilienMcool!16:00
*** tesseract has quit IRC16:00
*** dprince has quit IRC16:01
*** dprince has joined #puppet-openstack16:02
*** samuelBartel has quit IRC16:02
dmsimardEmilienM, dgurtner, xarses: kilo is going EOL in RDO land soon, puppet-ceph integration tests will start failing16:10
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profiles/nova/pacemaker/consoleauth: add missing require  https://review.openstack.org/33066916:10
dmsimardfeel free to revive my patch https://review.openstack.org/#/c/251531/ or something16:11
EmilienMdmsimard: imho, you should split your patches16:11
EmilienMdmsimard: to 1) bump to liberty or mitaka 2) re-enable ignored tests16:11
dmsimardimho I have no time for this :(16:11
EmilienMwho does?16:11
EmilienMif it start failing, we'll drop beaker jobs16:12
EmilienMwe still have good coverage on puppet-openstack-integ16:12
dmsimardup to you16:12
dmsimardat least giving you a fair and friendly warning :)16:13
EmilienMcool thx16:13
*** permalac has quit IRC16:15
EmilienMcolonwq: ^ fyi16:16
xarsesdmsimard: should we just drop the openstack integration testing in puppet-ceph and use the puppet integration scenarios?16:17
dmsimardxarses: I think the openstack tests have value in puppet-ceph, just that we shouldn't bother "setting up" openstack ourselves like we're doing16:18
EmilienMxarses: no16:18
dmsimardwe should just re-use puppet-openstack-integration manifests16:19
EmilienMwe should not drop puppet-openstack-integration jobs for puppet-ceph16:19
EmilienMthey have high value, as we test ceph deployment with openstack16:19
xarsesEmilienM: not drop the puppet-openstack-intergration jobs16:19
EmilienMwe should rather move the bits from beaker manifests to p-o-i16:19
colonwqEmilienM, thank. I am taking a look at the patch.16:19
xarsesdrop the internal tests that we have in the repo16:19
xarsesthat set up kilo openstack16:19
*** dprince has quit IRC16:20
EmilienMwe really need someone looking at puppet-ceph16:21
xarseshttps://github.com/openstack/puppet-ceph/blob/master/spec/acceptance/ceph_rgw_spec.rb16:21
*** dprince has joined #puppet-openstack16:21
EmilienMwe need to move that ^ into a scenario or something16:22
EmilienMcolonwq: ^16:22
xarsesYa, that would probably be better16:23
*** dtantsur|bbl is now known as dtantsur16:32
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support Newton deployment  https://review.openstack.org/33008816:35
iberezovskiythis keystone makes me crazy16:35
EmilienMiberezovskiy: mhh16:37
EmilienMlooking at your patch..16:37
iberezovskiyit really looks like a workaround, but we don't have another choice...16:40
EmilienMiberezovskiy: I commented16:40
EmilienMbyt why do you modify the big conditional?16:41
EmilienMdo we need it?16:41
EmilienMok let me push something16:41
*** dmk0202 has quit IRC16:42
iberezovskiyEmilienM, I'll rework if condition16:45
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/33069416:46
EmilienMiberezovskiy: I was just thinking about this ^16:46
EmilienMiberezovskiy: will it fix beaker jobs?16:46
iberezovskiynope16:46
EmilienMah right the keystone::service thing in the conditional16:47
EmilienMok16:47
iberezovskiyhttps://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L100516:47
EmilienMlet's just drop it16:47
iberezovskiythat's the problem16:47
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/33069416:47
EmilienMiberezovskiy: better? ^16:47
EmilienMthat way, it won't break fuel I thikn16:48
mwhahahawhat exactly is getting broken? cause shouldn't the policy rc.d not break if you apply it on services that aren't configured16:48
mwhahahathe way we manage our stupid hack around this for ubuntu doesn't break if we define it but the service isn't configured16:49
EmilienMmwhahaha: http://logs.openstack.org/72/330172/1/check/gate-puppet-nova-puppet-beaker-rspec-ubuntu-xenial/188417e/console.html#_2016-06-15_20_00_29_76216:52
iberezovskiyEmilienM, better, but I have one suggestion, 1 minute16:52
EmilienMwe try to disable aservice that doesn't exist anymore16:53
EmilienMiberezovskiy: I think my approach can work and is much simpler.16:53
mwhahahaah i missed the service thing16:53
mwhahahayea https://review.openstack.org/#/c/330694 should be the fix16:54
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/33069416:55
EmilienMwith unit tests updated16:55
mwhahahafor newton on, i guess we need a note about using mitaka with that change the end user would need to ensure that keystone eventlet doesnt's tart16:55
mwhahahabut the policyrc.d should have handled that16:55
iberezovskiyEmilienM, do we need to care about supporting mitaka deployment in that case?16:55
EmilienMno16:55
EmilienMmaster == Newton16:55
EmilienMstable/mitaka == Mitaka16:56
iberezovskiyah, than your patch is ok16:56
mwhahahawell newton should still be backwards compatible16:56
EmilienMhttp://specs.openstack.org/openstack/puppet-openstack-specs/specs/kilo/master-policy.html16:56
mwhahahaand if not, we need a release note on it16:56
EmilienMmwhahaha: ok /me adding it16:56
iberezovskiyI just thought if wanna to support mitaka deployment16:56
iberezovskiywe probably don't need to remove keystone::service class usage16:57
iberezovskiyjust need to improve 'if' condition16:57
mwhahahathe ifcondition should have been removed with the addition of policyrcd16:57
*** derekh has quit IRC16:59
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/33069416:59
EmilienMmwhahaha: can you look please? ^16:59
colonwqEmilienM, I think I can break 251531 into two patches.17:00
EmilienMcolonwq: excellent17:00
EmilienMcolonwq: or just focus on p-o-i work17:00
EmilienMto add a new scenario17:01
EmilienMand we can run this scenario only on puppet-ceph17:01
EmilienMI think it's better, so we re-use all p-o-i framework17:01
EmilienMinstead of beaker17:01
EmilienMwe can keep beaker for basic testing17:01
EmilienMbut we reached a very complex set of beaker tests for tpuppe-ceph17:02
EmilienMpuppet-ceph*17:02
EmilienMthat we're not able to maintain, nobody has time to do it17:02
colonwqEmilienM, it is frustratingly complex.The various files have drifted.17:02
EmilienMso 1) let's simplify the beaker tests to test osd/mon/rgw with basic stuffs and 2) work on p-o-i for a more complex & realistic scenario17:02
EmilienMcolonwq: you ok with ^?17:02
openstackgerritIvan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support Newton deployment  https://review.openstack.org/33008817:03
colonwqEmilienM, I have a terminology question: p-o-i ?17:03
EmilienMpuppet openstack integration17:03
EmilienMhttps://github.com/openstack/puppet-openstack-integration17:03
colonwqAh. Thanks.17:04
EmilienMiberezovskiy: I think we'll take my patch to fix beaker jobs17:04
EmilienMiberezovskiy: and we'll rebase your patch on top and drop keystone service at all17:05
EmilienMiberezovskiy: wdyt?17:05
EmilienMcolonwq: you ok to work on it? /me trying to drive it a bit so we're making progress on this area17:05
iberezovskiyI'm just wondering about backward compatibility, it can broke smth17:05
iberezovskiylet's see17:05
EmilienMiberezovskiy: in my patch?17:05
EmilienMiberezovskiy: yes, see in release not what is breaks17:05
iberezovskiyI saw17:05
*** xarses has quit IRC17:05
EmilienM"if you deploys UCA on Xenial and you want Mitaka, you're broken"17:06
*** xarses has joined #puppet-openstack17:06
openstackgerritEmilien Macchi proposed openstack/puppet-nova: migration: split libvirt & nova configs  https://review.openstack.org/33017217:06
EmilienMiberezovskiy: let's see if my patch helps for beaker jobs ^17:06
EmilienMI added depends-on on my patch17:06
colonwqEmilienM, I can put some time on it. Let me see what I can do with it.17:07
EmilienMcolonwq: really, 1) should be easy as we'll remove code17:08
EmilienMcolonwq: I can certainly help with 2)17:08
*** trown is now known as trown|lunch17:08
iberezovskiyEmilienM, yeah, let's see CI, that's all I want now :)17:08
*** jpena is now known as jpena|off17:08
*** ifarkas has quit IRC17:08
EmilienMcolonwq: I'll do all necessary in openstack-infra for the CI jobs thing. I'll need some help to compose the scenario manifest, basically copy/paste from beaker17:08
colonwqEmilienM, do you foresee a scenario or two for just ceph/rgw tests?17:08
EmilienMcolonwq: keep scenario001 for nova/neutron/ceilometer/glance/cinder using ceph backend (so only mon & osd are needed)17:09
EmilienMcolonwq: and add scenario004 with only keystone/rgw/ceph17:09
EmilienMand maybe run some tests or something17:10
EmilienMyeah, maybe also add swift17:10
EmilienMand run tempest17:10
* EmilienM looking for a use case17:10
EmilienMcolonwq: can we use ceph with rgw as a backend for glance?17:11
colonwqEmilienM, I'll make an 004 and let's see how we can expand it.17:11
colonwqEmilienM, Yes.17:11
EmilienMcolonwq: what is the use case with openstack + rgw again?17:11
colonwqEmilienM, It will be to use rgw for object storage instead of swift.17:12
EmilienMcolonwq: right, but can we tell Glance to store images in there?17:12
*** queretaro has quit IRC17:13
colonwqEmilienM, I duno about telling glance to use rgw for storage. It uses rbd directly.17:13
EmilienMyeah, I was looking for a use case with openstack17:14
EmilienMcolonwq: if we don't have it, nevermind. We can do another thing :17:14
colonwqEmilienM, I am hard pressed for a openstack direct usage of rgw. I think it is an 'also provided' service.17:15
EmilienMcolonwq: keep rgw beaker tests but modify the code to use bits from p-o-i (ie: include ::openstack_integration::keystone17:15
EmilienMcolonwq: look https://github.com/openstack/puppet-nova/blob/master/spec/acceptance/nova_wsgi_apache_spec.rb#L9-L1317:15
EmilienMI think we could do the same for puppet-ceph17:15
EmilienMinstead of deploying Kilo, we could deploy OpenStack like we do in all other jobs17:16
*** leanderthal|afk has quit IRC17:16
EmilienMcolonwq: see what I mean?17:17
colonwqEmilienM, I think so. I'll make a branch and start on it.17:17
EmilienMgreat17:17
colonwqEmilienM, in short: no 004, keep beaker, use p-o-i for needed bits. Am I correct?17:18
EmilienMcolonwq: yes for now17:19
EmilienMwe'll iterate later if needed17:19
colonwqCool. I'll go eat lunch then see what I can knock out.17:20
EmilienMcolonwq: the goal is to bring back our beaker jobs alive17:20
EmilienMand make them work on jewel17:20
*** dtantsur is now known as dtantsur|afk17:20
EmilienMcolonwq: have a good lunch! and thx for your help17:21
*** rcernin has quit IRC17:21
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: WIP: Don't start nova service right after package installation  https://review.openstack.org/33071517:21
iberezovskiyEmilienM, trying to fix puppet-4 jobs ^17:22
EmilienMiberezovskiy: oh yeah? how is it related?17:23
iberezovskiynova-api starts before apache17:23
iberezovskiyand then apache can't bind on 8774 port17:23
EmilienMoh damnn17:23
EmilienMiberezovskiy: nice catch17:23
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: WIP: Don't start nova service right after package installation  https://review.openstack.org/33071517:23
iberezovskiyskolekonov also helped me :)17:23
EmilienMskolekonov, iberezovskiy ++ guys17:24
* EmilienM afk ~1h17:24
*** leanderthal|afk has joined #puppet-openstack17:29
*** danpawlik has joined #puppet-openstack17:32
iberezovskiywill look at CI for patches, but won't be at irc probably17:32
*** iberezovskiy is now known as iberezovskiy|off17:32
*** dgurtner has quit IRC17:50
*** paramite has quit IRC18:03
*** amoralej is now known as amoralej|off18:04
*** rcernin has joined #puppet-openstack18:07
*** trown|lunch is now known as trown18:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859118:36
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Try hack to prevent nova services from autostart  https://review.openstack.org/33071518:40
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Try hack to prevent nova services from autostart  https://review.openstack.org/33071518:41
EmilienMmwhahaha: can you  look https://review.openstack.org/#/c/330694/ please? and see why fuel jobs fail?18:42
iberezovskiy|offEmilienM, not deployment issue, rechecked18:43
EmilienMiberezovskiy|off: hey18:43
EmilienMthe beaker patch fails https://review.openstack.org/#/c/330172/18:43
iberezovskiy|offI'm partially here, but I can't see hostory, damn18:43
EmilienMbut it's maybe related to your work18:43
iberezovskiy|offlooking18:43
EmilienMiberezovskiy|off: no worries18:43
EmilienMI think we need https://review.openstack.org/33071518:44
*** dmk0202 has joined #puppet-openstack18:44
iberezovskiy|offyeah, but see what's weird solution I have there18:45
iberezovskiy|offwe can't call policyrcd class many times18:45
EmilienMiberezovskiy|off: in the meantime, we can fix other beaker jobs with https://review.openstack.org/#/c/330694/18:45
*** dmk0202 has quit IRC18:45
*** chem is now known as chem|off18:46
*** saneax_AFK is now known as saneax18:46
EmilienMcan you review it please? and maybe mwhahaha -- so when p-o-ijobs are green we can land it18:46
iberezovskiy|offpatch is fine, but we need to figure our what to do with policyrcd18:47
iberezovskiy|offit should be possible to call it as much time as we need18:47
EmilienMiberezovskiy|off: using <+ can be good18:49
iberezovskiy|offlet's see18:49
*** xarses has quit IRC18:50
*** xarses has joined #puppet-openstack18:51
iberezovskiy|offError: Only subclasses can override parameters :(18:52
*** yolanda has quit IRC19:00
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/32968219:02
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/32971419:02
*** danpawlik has quit IRC19:03
*** xarses_ has joined #puppet-openstack19:06
*** saneax is now known as saneax_AFK19:08
*** xarses has quit IRC19:09
EmilienMiberezovskiy|off: maybe we can adjust https://github.com/openstack/puppet-openstacklib/blob/master/manifests/policyrcd.pp19:10
EmilienMiberezovskiy|off: because we're going to have the same issue for all our jobs, that run in WSGI19:11
EmilienMironic nova ceilometer aodh gnocchi etc19:12
*** xarses_ has quit IRC19:12
*** johnmilton has quit IRC19:13
iberezovskiy|offthe problem is only: how to get all service names19:13
EmilienMiberezovskiy|off: I have another idea19:13
EmilienMit's a bit crazy19:14
EmilienMsee https://github.com/devopsconsulting/puppet-disablestart/blob/master/files/usr/sbin/policy-rc.d19:14
EmilienMwe disable everything lol19:14
iberezovskiy|offI thought about this19:14
iurygregorywoa19:15
iberezovskiy|offbut to be honest with users, we need to remove this file after puppet run19:15
iberezovskiy|offwhich will make our manifests non-idempotent19:15
EmilienMok let me talk to canonical again19:16
EmilienM-> #ubuntu-servr19:16
EmilienM-> #ubuntu-server19:16
iberezovskiy|offI'm there19:16
*** iberezovskiy|off is now known as iberezovskiy19:17
*** xb_ has quit IRC19:17
EmilienMiberezovskiy: or something else19:19
EmilienMiberezovskiy: we can add this in run_tests.sh:19:20
EmilienMecho exit 101 | sudo tee /usr/sbin/policy-rc.d && sudo chmod +x /usr/sbin/policy-rc.d19:20
EmilienMoh wait, I have another idea19:20
iberezovskiyI'm all ears :D19:21
EmilienMa sec19:22
EmilienMI'm writting it19:22
iberezovskiysure19:22
EmilienMit's ugly19:24
EmilienMlol19:24
iberezovskiyalso solution could be ugly because we trying to fix packaging issue19:26
*** dprince has quit IRC19:27
*** johnmilton has joined #puppet-openstack19:30
*** chem|off` has joined #puppet-openstack19:31
openstackgerritEmilien Macchi proposed openstack/puppet-nova: api/httpd: stop nova-api service after package install  https://review.openstack.org/33075719:32
EmilienMiberezovskiy: I'm finishing the patch but this is what I propose ^19:32
EmilienMI think it can work19:33
*** chem|off has quit IRC19:33
openstackgerritEmilien Macchi proposed openstack/puppet-nova: api/httpd: stop nova-api service after package install  https://review.openstack.org/33075719:35
EmilienMlet's see if it works19:35
EmilienM_ody: w00t your puppet patch is merged \o/19:40
*** dmk0202 has joined #puppet-openstack19:40
_odyWow.  Kinda surprised.19:40
_odyNot a single person had even touched it as of this morning.19:40
_odySet out this morning to see how many nova services I could get running inside docker using nothing but the upstream puppet modules.19:42
_odyThis going A LOT smoother than I expected19:42
_odyPart of a very last minute constructed demo for OpenStack PDX tonight.19:42
*** dmk0202 has quit IRC19:43
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859119:59
*** jayg is now known as jayg|g0n320:03
iberezovskiyEmilienM, I am off for today, I'll monitor your patches tomorrow20:03
EmilienMthx!20:03
EmilienMiberezovskiy: good night20:03
iberezovskiythanks )20:04
*** iberezovskiy is now known as iberezovskiy|off20:04
mwhahahawe do the policyrc.d thing for all patches during a deployment in fuel but we also cheat for idempotency and only do it when the package is not installed20:04
mwhahahawhich we use a custom function for (is not pretty)20:04
*** ayoung has quit IRC20:18
*** openstackstatus has joined #puppet-openstack20:20
*** ChanServ sets mode: +v openstackstatus20:20
*** xarses_ has joined #puppet-openstack20:22
*** dmk0202 has joined #puppet-openstack20:23
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Update to limit CentOS SIG to hammer  https://review.openstack.org/33016820:26
*** trown is now known as trown|outtypewww21:00
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/33078521:04
*** chem|off` has quit IRC21:07
EmilienMmwhahaha: https://review.openstack.org/#/c/330694/ fails21:08
EmilienMbecause of job failure21:09
EmilienMI don't see why21:09
EmilienMI'm reporting it on infra21:09
EmilienMhttp://logs.openstack.org/94/330694/4/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/652634b/console.html#_2016-06-16_20_40_28_16111921:09
mwhahahathats weird21:11
EmilienMyeah21:14
iurygregoryEmilienM, I'll start the work at home o/ tomorrow I'll give an update21:15
EmilienMiurygregory: excellent, please let me know if you need any help21:15
iurygregoryEmilienM, sure :D have a good night21:16
EmilienMcheers21:16
*** karimb has quit IRC21:25
*** karimb has joined #puppet-openstack21:26
*** rcernin has quit IRC21:36
*** dmk0202 has quit IRC21:38
openstackgerritKeith Schincke proposed openstack/puppet-openstack-integration: Add ext4 parameters to openstack_integration::ceph  https://review.openstack.org/33079721:38
EmilienMok infra is really in bad shape21:45
EmilienMtimeout again for another reason21:45
*** chem has joined #puppet-openstack21:50
*** chem is now known as chem|off21:52
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/33078522:02
*** dmk0202 has joined #puppet-openstack22:04
*** karimb has quit IRC22:06
*** xarses_ has quit IRC22:06
*** xarses_ has joined #puppet-openstack22:07
*** ayoung has joined #puppet-openstack22:07
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/32968222:09
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/32971422:10
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: nova/api: include ::nova::network::neutron  https://review.openstack.org/32952922:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/32859122:19
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Update ceph_usecases to use puppet-openstack-integration  https://review.openstack.org/33080622:22
colonwqEmilienM, are 330806 and 330797 about what you are looking for?22:23
EmilienMcolonwq: it's a good start22:24
colonwqEmilienM, Ok. I'll keep working in that direction.22:26
EmilienMcolonwq: I commented on https://review.openstack.org/#/c/330168/22:27
EmilienMI'm off, cheers22:28
*** dmk0202 has quit IRC22:39
*** dfisher has quit IRC22:47
*** saneax_AFK is now known as saneax22:49
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/33069422:58
*** xarses_ has quit IRC22:59
*** xarses_ has joined #puppet-openstack23:00
*** ayoung has quit IRC23:07
*** iurygregory_ has joined #puppet-openstack23:34
*** ayoung has joined #puppet-openstack23:44
*** xarses_ has quit IRC23:48
« Wednesday, 2016-06-15 Index Friday, 2016-06-17 »

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