Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-06-16

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:04 markvoelker_ joined #puppet-openstack
00:04 cdelatte joined #puppet-openstack
00:08 markvoelker joined #puppet-openstack
00:13 xarses_ joined #puppet-openstack
00:13 markvoelker_ joined #puppet-openstack
00:18 markvoelker joined #puppet-openstack
00:21 markvoelker_ joined #puppet-openstack
00:22 cdelatte joined #puppet-openstack
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now CI status: red - all beaker jobs broken - we need https://review.openstack.org/#/c/330088/ - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
03:06 ducttape_ joined #puppet-openstack
03:39 v1k0d3n joined #puppet-openstack
03:50 jfluhmann joined #puppet-openstack
05:03 saneax folks, wanted to know what is the best approach in puppet to get h/w details
05:03 saneax ie sriov pci address and vendor id
05:08 saneax I understand that the approach is for the tooling above puppet should be passing these values?
05:20 jaosorior joined #puppet-openstack
05:37 psanchez joined #puppet-openstack
05:41 rain joined #puppet-openstack
05:46 xarses_ joined #puppet-openstack
05:57 psanchez joined #puppet-openstack
05:57 psanchez joined #puppet-openstack
05:58 psanchez joined #puppet-openstack
05:58 rain joined #puppet-openstack
05:59 openstack joined #puppet-openstack
06:02 jmikolajczak joined #puppet-openstack
06:14 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
06:18 yolanda joined #puppet-openstack
06:23 yolanda joined #puppet-openstack
06:24 rcernin joined #puppet-openstack
06:34 zhongshengping joined #puppet-openstack
06:51 dgurtner joined #puppet-openstack
07:01 markvoelker joined #puppet-openstack
07:09 tesseract joined #puppet-openstack
07:15 cgfbee joined #puppet-openstack
07:19 ifarkas joined #puppet-openstack
07:23 hjensas__ joined #puppet-openstack
07:30 shardy joined #puppet-openstack
07:32 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic: Allow to configure cleaning_network_uuid  https://review.openstack.org/328916
07:32 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic: Allow to set provisioning_network_uuid  https://review.openstack.org/328921
07:34 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/330351
07:37 jaosorior anybody around yet?
07:41 acabot joined #puppet-openstack
07:47 danpawlik joined #puppet-openstack
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/327029
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/327069
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/327473
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/327482
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/328859
07:47 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add fact to get the fqdn for a host in the different networks  https://review.openstack.org/329299
07:53 samuelBartel joined #puppet-openstack
08:05 queretaro joined #puppet-openstack
08:08 jaosorior joined #puppet-openstack
08:09 permalac joined #puppet-openstack
08:11 hjensas_ joined #puppet-openstack
08:12 chem```` joined #puppet-openstack
08:20 karimb joined #puppet-openstack
08:23 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/330351
08:24 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Small fixes to SSL related options  https://review.openstack.org/330351
08:27 paramite joined #puppet-openstack
08:33 openstackgerrit Bartek Żurawski proposed openstack/puppet-nova: Add support for MultiStrOpt for nova_config  https://review.openstack.org/329991
08:41 openstackgerrit Vasyl Saienko proposed openstack/puppet-ironic: Add ironic::drivers::interfaces class  https://review.openstack.org/329775
08:47 derekh joined #puppet-openstack
09:02 dmk0202 joined #puppet-openstack
09:33 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Switch defaults for nova upgrade parameters to $::os_service_default  https://review.openstack.org/330408
09:57 ifarkas joined #puppet-openstack
10:30 jtomasek_ joined #puppet-openstack
10:42 hjensas_ joined #puppet-openstack
10:50 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: test  https://review.openstack.org/330452
10:50 xb_ joined #puppet-openstack
10:57 cdelatte joined #puppet-openstack
11:00 openstackgerrit Brad P. Crochet proposed openstack/puppet-tripleo: Add Mistral profiles  https://review.openstack.org/323431
11:26 johnmilton joined #puppet-openstack
11:29 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support UCA Newton  https://review.openstack.org/330088
11:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/327069
11:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/327473
11:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/327482
11:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/328859
11:39 jaosorior degorenko, iberezovksiy: ping
11:43 johnmilton joined #puppet-openstack
11:46 jaosorior iberezovskiy: Hey, I was looking at the puppet-cinder SSL options
11:46 iberezovskiy hey
11:47 jaosorior it is a bit awkward that they are in init.pp though. Most of the other services (keystone, glance, heat) have those options in api.pp
11:47 jaosorior would it be appropriate to move the SSL options to cinder's api.pp?
11:47 jaosorior it would be the same case for puppet-neutron
11:47 iberezovskiy yes, but you should make this changes backward-compatible
11:48 jaosorior iberezovskiy: I was thinking of passing these parameters in init.pp to a deprecated section
11:48 iberezovskiy it still hould be possible to use init clsss to set these params
11:48 jaosorior ok
11:48 iberezovskiy and use pick in api class
11:48 iberezovskiy smth like this
11:49 jaosorior ??
11:51 iberezovskiy pick($::heat::ssl_key, $::heat::api::ssl_key) in api class
11:51 iberezovskiy just an example
11:52 iberezovskiy jaosorior, https://github.com/openstack/puppet-cinder/blob/master/manifests/db.pp#L54
11:53 jaosorior iberezovskiy: excellent
11:53 jaosorior will do
11:53 jaosorior thanks man
11:53 iberezovskiy thanks
11:54 dprince joined #puppet-openstack
11:54 ducttape_ joined #puppet-openstack
11:58 iurygregory morning guys =D
11:59 jaosorior iberezovskiy: in init.pp do I move those parameters to deprecated?
12:13 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/330498
12:15 danpawlik joined #puppet-openstack
12:16 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/330498
12:17 samuelBartel joined #puppet-openstack
12:17 dgurtner joined #puppet-openstack
12:17 dgurtner joined #puppet-openstack
12:35 EmilienM good morning
12:35 degorenko morning :)
12:36 ducttape_ joined #puppet-openstack
12:37 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-neutron: Move SSL related options to api.pp  https://review.openstack.org/330511
12:38 rcernin joined #puppet-openstack
12:46 iberezovskiy jaosorior, yep
12:46 iberezovskiy EmilienM, morning
12:46 igajsin joined #puppet-openstack
12:47 igajsin left #puppet-openstack
12:47 EmilienM iberezovskiy: can I have a tl;dr on ci status please?
12:48 iurygregory morning EmilienM o/
12:48 iberezovskiy EmilienM, 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 iberezovskiy I've recheck and now it should work
12:50 iberezovskiy patch is https://review.openstack.org/#/c/330088/
12:51 EmilienM iberezovskiy: what is the issue with floating ip?
12:51 iberezovskiy not ip :)
12:51 EmilienM iberezovskiy: gate-puppet-openstack-integration-3-scenario003-tempest-centos-7 is failing
12:51 EmilienM 7 tempest tests
12:51 iberezovskiy so when you run include ::openstack_integration::repos sometimes it doesn't create file ubuntu-cloud-archive.list
12:52 iberezovskiy and as result newton repo isn't enabled
12:52 iberezovskiy I am on it already
12:52 EmilienM what?
12:52 EmilienM sorry
12:52 EmilienM WHAAATTT
12:52 iurygregory lol
12:52 iurygregory 'sometimes'... =X
12:53 EmilienM iberezovskiy: is it because the repo is not reachable?
12:53 iberezovskiy http://logs.openstack.org/88/330088/3/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/d9e1d2c/console.html#_2016-06-15_18_02_43_105930
12:53 iberezovskiy http://logs.openstack.org/88/330088/4/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/3630e7d/console.html#_2016-06-16_11_36_51_032003
12:53 iberezovskiy see difference in puppet logs
12:53 iberezovskiy resource apt::source wasn't called
12:54 EmilienM mh weird
12:54 EmilienM and you have a solution?
12:54 iberezovskiy nope
12:55 iberezovskiy I am going investigate after we fix beaker tests
12:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001/ubuntu: re-enable telemetry  https://review.openstack.org/330188
12:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001/ubuntu: re-enable telemetry  https://review.openstack.org/330188
12:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration:  scenario002/ubuntu: re-enable ironic  https://review.openstack.org/330189
12:56 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003/ubuntu: re-enable trove  https://review.openstack.org/330191
12:56 EmilienM iberezovskiy: ack
12:57 EmilienM iberezovskiy: I'm reading https://review.openstack.org/#/c/330088/4/manifests/init.pp
12:58 EmilienM iberezovskiy: so on centos7 you still allow to run keystone eventlet and not on ubuntu, where you hardcode apache?
12:58 iberezovskiy yep, because rdo provide service script for keystone
12:58 EmilienM hum I don't think so
12:58 iberezovskiy o_O
12:59 EmilienM I think now keystone scripts does not exist
12:59 zhongshengping joined #puppet-openstack
12:59 EmilienM let me propose something:
13:00 EmilienM if $service_name = 'httpd'; then httpd bits (remove keystone::service bits)
13:00 EmilienM else: warning('puppet-keystone only manage deployments with httpd, otherwise manage the service yourself') or something like this
13:01 EmilienM we reduce code & complexity
13:01 iberezovskiy heh, it was my first idea :D
13:01 bkero sounds like git reverting vinsh's keystone service managing patch
13:02 vinsh Not I
13:02 iberezovskiy I even started to work on here https://review.openstack.org/#/c/329567 :D
13:02 iberezovskiy so if you are ok, I fine as well :)
13:05 EmilienM iberezovskiy: I commented https://review.openstack.org/#/c/329567
13:05 EmilienM but yeah I prefer this approach
13:06 iberezovskiy EmilienM, you are right about rdo packages, I missed this. so I'm working on this soltuion
13:06 EmilienM excellent
13:06 EmilienM iurygregory: do you have time to discuss about auth token?
13:06 rcernin joined #puppet-openstack
13:17 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/327473
13:17 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/327482
13:17 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for cinder-api  https://review.openstack.org/328859
13:20 iurygregory EmilienM, yep
13:25 openstackgerrit Sanjay Upadhyay proposed openstack/puppet-neutron: Cleanup for sriov-agent sriov-agent-required  https://review.openstack.org/330544
13:25 EmilienM iurygregory: ok cool
13:25 v1k0d3n joined #puppet-openstack
13:25 EmilienM iurygregory: so the plan is to first cleanup puppet-keystone
13:26 iurygregory ok
13:26 EmilienM https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/authtoken.pp
13:26 Guest48217 joined #puppet-openstack
13:26 EmilienM we could use os_service_default
13:26 EmilienM and avoid all the conditionals
13:26 iurygregory oh yeah big +1
13:26 iurygregory :D
13:26 EmilienM good so that's the first step.
13:27 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova: Use correct conf group for SSL options  https://review.openstack.org/330550
13:27 EmilienM in the same patch we can also update parameters to be really compliant with latest keystone middleware
13:27 EmilienM docs.openstack.org/developer/keystonemiddleware/middlewarearchitecture.html
13:27 EmilienM www.jamielennox.net/blog/2015/02/23/v3-authentication-with-auth-token-middleware/
13:27 iurygregory thanks for the links =D
13:27 EmilienM when we have the patch ready, we can ask richm and/or ayoung to review it
13:27 ayoung EmilienM, you can always ask
13:28 iurygregory tomorrow morning i'll have the patch o/
13:28 EmilienM iurygregory: and when we land it, we'll take over richm's patch in puppet-glance
13:28 EmilienM but split it in 2
13:28 EmilienM 1) update config first, using puppet-keyste define
13:28 EmilienM 2) then update provider
13:28 iurygregory ok =)
13:29 EmilienM when 1) is done, reviewed & merged, we can do other modules (and distribute the work) like we did for oslo things
13:29 EmilienM and do 2) in parallel
13:29 iurygregory https://review.openstack.org/#/c/176150/ ?
13:29 iurygregory this is the patch right?
13:29 EmilienM yes
13:29 EmilienM https://review.openstack.org/#/c/176150/ will be 2). We'll rebase it on top of 1)
13:30 iurygregory EmilienM, ack
13:31 EmilienM iurygregory: 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 Define
13:31 iurygregory EmilienM, ok o/
13:33 iurygregory EmilienM, thanks for all directions =D
13:33 EmilienM iurygregory: cool
13:35 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/329567
13:41 rmart04 joined #puppet-openstack
13:43 ducttape_ joined #puppet-openstack
13:50 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Add user & user_role parameters in auth manifest and related tests  https://review.openstack.org/324296
13:52 dfisher joined #puppet-openstack
13:52 bkero radez: 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:53 bkero Were you planning on adding the content back at some point? :)
13:58 richm joined #puppet-openstack
14:02 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/329567
14:11 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-cinder: Move SSL parameters to api.pp  https://review.openstack.org/330498
14:13 EmilienM iberezovskiy: https://review.openstack.org/#/c/329567/6/manifests/init.pp
14:13 EmilienM so imho you should not deprecate it
14:13 EmilienM oh wait
14:13 EmilienM wait wait wait
14:14 EmilienM yeah so
14:14 EmilienM the way you're doing it, you enforce people to deploy puppet-keystone with apache
14:14 EmilienM it's not good
14:14 EmilienM so you need to keep service_name parameter
14:14 EmilienM do not deprecate it, keep it
14:14 EmilienM iberezovskiy: does it make sense?
14:15 iberezovskiy EmilienM, what this parameter should do in that case?
14:17 EmilienM iberezovskiy: ok I'll comment in gerrit, easier
14:19 bzurawski joined #puppet-openstack
14:22 EmilienM iberezovskiy: look
14:22 EmilienM let me know if you get it
14:24 iberezovskiy EmilienM, I can't put httpd by default, because 'service httpd restart' will fail on ubuntu
14:24 iberezovskiy let's use keystone::params::httpd_service_name
14:24 EmilienM no
14:24 EmilienM see my comment on the left
14:24 EmilienM L995
14:25 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: Implement api.pp  https://review.openstack.org/329830
14:26 iberezovskiy ah, I got what I've missed before, so I'll fix
14:26 iberezovskiy I'll wait for CI, then update
14:26 EmilienM iberezovskiy: well, you can go ahead
14:27 danpawlik EmilienM: I really try to push small commits ;)
14:28 EmilienM danpawlik++
14:47 xarses_ joined #puppet-openstack
14:51 openstackgerrit Vasyl Saienko proposed openstack/puppet-openstacklib: Introduce openstack_resource_id_getter function  https://review.openstack.org/330623
14:53 iberezovskiy EmilienM, good to know that beaker&integration jobs are green on my patch :) will upload patch with you comments resolved soon
14:54 EmilienM excellent
15:00 chem joined #puppet-openstack
15:05 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Keystone could be deployed only under apache now  https://review.openstack.org/329567
15:06 v1k0d3n joined #puppet-openstack
15:17 dgurtner joined #puppet-openstack
15:17 jmiu_ joined #puppet-openstack
15:24 ifarkas joined #puppet-openstack
15:27 evidex joined #puppet-openstack
15:29 EmilienM iberezovskiy: reviewing
15:31 zhongshengping joined #puppet-openstack
15:32 EmilienM iberezovskiy: commented, almost good
15:33 vinsh joined #puppet-openstack
15:33 andreww joined #puppet-openstack
15:33 andreww joined #puppet-openstack
15:34 iberezovskiy EmilienM, I accidently move some lines. will fix
15:34 EmilienM thx
15:34 iberezovskiy s/move/moved
15:34 openstackgerrit joined #puppet-openstack
15:34 EmilienM iberezovskiy: I had more comments also, refresh :P
15:34 EmilienM we're almost there
15:35 iberezovskiy EmilienM, yep, but I've broke fuel :D
15:35 EmilienM ah
15:35 EmilienM your patch should be backward compatible
15:35 EmilienM try again!
15:36 iberezovskiy a bit now, it forces to use httpd because I'm changing default
15:36 iberezovskiy s/a bit now/a bit not
15:36 EmilienM yeah, but I guess fuel is deploying keystone with httpd, no?
15:36 iberezovskiy not on the master node
15:36 EmilienM o_O
15:36 iberezovskiy yeah...
15:36 EmilienM you use eventlet?
15:36 EmilienM just don't tell ayoung
15:39 iberezovskiy yep, eventlet
15:39 iberezovskiy it's not hard to solve I think
15:40 vinsh_ joined #puppet-openstack
15:40 iberezovskiy I'll make the change not-breakable for Fuel
15:40 EmilienM iberezovskiy: 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_ joined #puppet-openstack
15:41 EmilienM and then rebase your patch on top of the fix
15:43 EmilienM iberezovskiy: you ok?^
15:44 iberezovskiy I can adapt fuel fast
15:46 EmilienM iberezovskiy: as long as we have a fix merged by today, I'm fine
15:46 iberezovskiy I think it's possible because mwhahaha can merge fuel patch right after keystone one :)
15:49 vinsh joined #puppet-openstack
15:50 iberezovskiy mwhahaha, please take a look on https://review.openstack.org/#/c/330654/
15:50 mwhahaha k
15:51 iberezovskiy I don't need to add depens-on probably, because it can be merged before
15:53 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Limit CentOS SIG Storage support to Ceph hammer  https://review.openstack.org/330041
15:53 mwhahaha which was the upstream change that broken this one?
15:54 iberezovskiy mwhahaha, probably this one will be https://review.openstack.org/#/c/329567/
15:54 iberezovskiy noop tests are also broken, damn
15:55 mwhahaha soooo i'm going to say -1 on that due to backwards compatibility breakage
15:55 EmilienM iberezovskiy: but you realize the keystone eventlet service won't be managed anymore? you manage it in fuel itself?
15:55 evidex joined #puppet-openstack
15:56 iberezovskiy EmilienM, we still have mitaka keystone on the master node
15:56 iberezovskiy that's the problem :(
15:56 EmilienM this problem is more complex than expected
15:56 mwhahaha but this isn't just a fuel thing, i don't think we should make that hard definition as the default
15:56 EmilienM can we just push a little patch to fix beaker jobs?
15:57 iberezovskiy EmilienM, the smaller patch is https://review.openstack.org/#/c/330088
15:58 iberezovskiy need to replace keystone::params:httpd:service_name to ::apache...
15:59 EmilienM iberezovskiy: ok let's push https://review.openstack.org/#/c/330088 first and you'll rebase on top of this one
15:59 iberezovskiy ok
15:59 EmilienM so you have time tonight or tomorrow to fix at your convenience
16:00 EmilienM iberezovskiy: do you need another patchset?
16:00 iberezovskiy yep, will prepare in a few minutes
16:00 EmilienM ok
16:00 EmilienM cool!
16:02 dprince joined #puppet-openstack
16:10 dmsimard EmilienM, dgurtner, xarses: kilo is going EOL in RDO land soon, puppet-ceph integration tests will start failing
16:10 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profiles/nova/pacemaker/consoleauth: add missing require  https://review.openstack.org/330669
16:10 dmsimard feel free to revive my patch https://review.openstack.org/#/c/251531/ or something
16:11 EmilienM dmsimard: imho, you should split your patches
16:11 EmilienM dmsimard: to 1) bump to liberty or mitaka 2) re-enable ignored tests
16:11 dmsimard imho I have no time for this :(
16:11 EmilienM who does?
16:12 EmilienM if it start failing, we'll drop beaker jobs
16:12 EmilienM we still have good coverage on puppet-openstack-integ
16:12 dmsimard up to you
16:13 dmsimard at least giving you a fair and friendly warning :)
16:13 EmilienM cool thx
16:16 EmilienM colonwq: ^ fyi
16:17 xarses dmsimard: should we just drop the openstack integration testing in puppet-ceph and use the puppet integration scenarios?
16:18 dmsimard xarses: I think the openstack tests have value in puppet-ceph, just that we shouldn't bother "setting up" openstack ourselves like we're doing
16:18 EmilienM xarses: no
16:19 dmsimard we should just re-use puppet-openstack-integration manifests
16:19 EmilienM we should not drop puppet-openstack-integration jobs for puppet-ceph
16:19 EmilienM they have high value, as we test ceph deployment with openstack
16:19 xarses EmilienM: not drop the puppet-openstack-intergration jobs
16:19 EmilienM we should rather move the bits from beaker manifests to p-o-i
16:19 colonwq EmilienM, thank. I am taking a look at the patch.
16:19 xarses drop the internal tests that we have in the repo
16:19 xarses that set up kilo openstack
16:21 EmilienM we really need someone looking at puppet-ceph
16:21 xarses https://github.com/openstack/puppet-ceph/blob/master/spec/acceptance/ceph_rgw_spec.rb
16:21 dprince joined #puppet-openstack
16:22 EmilienM we need to move that ^ into a scenario or something
16:22 EmilienM colonwq: ^
16:23 xarses Ya, that would probably be better
16:35 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support Newton deployment  https://review.openstack.org/330088
16:35 iberezovskiy this keystone makes me crazy
16:37 EmilienM iberezovskiy: mhh
16:37 EmilienM looking at your patch..
16:40 iberezovskiy it really looks like a workaround, but we don't have another choice...
16:40 EmilienM iberezovskiy: I commented
16:41 EmilienM byt why do you modify the big conditional?
16:41 EmilienM do we need it?
16:41 EmilienM ok let me push something
16:45 iberezovskiy EmilienM, I'll rework if condition
16:46 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/330694
16:46 EmilienM iberezovskiy: I was just thinking about this ^
16:46 EmilienM iberezovskiy: will it fix beaker jobs?
16:46 iberezovskiy nope
16:47 EmilienM ah right the keystone::service thing in the conditional
16:47 EmilienM ok
16:47 iberezovskiy https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L1005
16:47 EmilienM let's just drop it
16:47 iberezovskiy that's the problem
16:47 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/330694
16:47 EmilienM iberezovskiy: better? ^
16:48 EmilienM that way, it won't break fuel I thikn
16:48 mwhahaha what exactly is getting broken? cause shouldn't the policy rc.d not break if you apply it on services that aren't configured
16:49 mwhahaha the way we manage our stupid hack around this for ubuntu doesn't break if we define it but the service isn't configured
16:52 EmilienM mwhahaha: 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_762
16:52 iberezovskiy EmilienM, better, but I have one suggestion, 1 minute
16:53 EmilienM we try to disable aservice that doesn't exist anymore
16:53 EmilienM iberezovskiy: I think my approach can work and is much simpler.
16:53 mwhahaha ah i missed the service thing
16:54 mwhahaha yea https://review.openstack.org/#/c/330694 should be the fix
16:55 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/330694
16:55 EmilienM with unit tests updated
16:55 mwhahaha for 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 tart
16:55 mwhahaha but the policyrc.d should have handled that
16:55 iberezovskiy EmilienM, do we need to care about supporting mitaka deployment in that case?
16:55 EmilienM no
16:55 EmilienM master == Newton
16:56 EmilienM stable/mitaka == Mitaka
16:56 iberezovskiy ah, than your patch is ok
16:56 mwhahaha well newton should still be backwards compatible
16:56 EmilienM http://specs.openstack.org/openstack/puppet-openstack-specs/specs/kilo/master-policy.html
16:56 mwhahaha and if not, we need a release note on it
16:56 EmilienM mwhahaha: ok /me adding it
16:56 iberezovskiy I just thought if wanna to support mitaka deployment
16:57 iberezovskiy we probably don't need to remove keystone::service class usage
16:57 iberezovskiy just need to improve 'if' condition
16:57 mwhahaha the ifcondition should have been removed with the addition of policyrcd
16:59 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/330694
16:59 EmilienM mwhahaha: can you look please? ^
17:00 colonwq EmilienM, I think I can break 251531 into two patches.
17:00 EmilienM colonwq: excellent
17:00 EmilienM colonwq: or just focus on p-o-i work
17:01 EmilienM to add a new scenario
17:01 EmilienM and we can run this scenario only on puppet-ceph
17:01 EmilienM I think it's better, so we re-use all p-o-i framework
17:01 EmilienM instead of beaker
17:01 EmilienM we can keep beaker for basic testing
17:02 EmilienM but we reached a very complex set of beaker tests for tpuppe-ceph
17:02 EmilienM puppet-ceph*
17:02 EmilienM that we're not able to maintain, nobody has time to do it
17:02 colonwq EmilienM, it is frustratingly complex.The various files have drifted.
17:02 EmilienM so 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 scenario
17:02 EmilienM colonwq: you ok with ^?
17:03 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Update keystone module to support Newton deployment  https://review.openstack.org/330088
17:03 colonwq EmilienM, I have a terminology question: p-o-i ?
17:03 EmilienM puppet openstack integration
17:03 EmilienM https://github.com/openstack/puppet-openstack-integration
17:04 colonwq Ah. Thanks.
17:04 EmilienM iberezovskiy: I think we'll take my patch to fix beaker jobs
17:05 EmilienM iberezovskiy: and we'll rebase your patch on top and drop keystone service at all
17:05 EmilienM iberezovskiy: wdyt?
17:05 EmilienM colonwq: you ok to work on it? /me trying to drive it a bit so we're making progress on this area
17:05 iberezovskiy I'm just wondering about backward compatibility, it can broke smth
17:05 iberezovskiy let's see
17:05 EmilienM iberezovskiy: in my patch?
17:05 EmilienM iberezovskiy: yes, see in release not what is breaks
17:05 iberezovskiy I saw
17:06 EmilienM "if you deploys UCA on Xenial and you want Mitaka, you're broken"
17:06 xarses joined #puppet-openstack
17:06 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: migration: split libvirt & nova configs  https://review.openstack.org/330172
17:06 EmilienM iberezovskiy: let's see if my patch helps for beaker jobs ^
17:06 EmilienM I added depends-on on my patch
17:07 colonwq EmilienM, I can put some time on it. Let me see what I can do with it.
17:08 EmilienM colonwq: really, 1) should be easy as we'll remove code
17:08 EmilienM colonwq: I can certainly help with 2)
17:08 iberezovskiy EmilienM, yeah, let's see CI, that's all I want now :)
17:08 EmilienM colonwq: 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 beaker
17:08 colonwq EmilienM, do you foresee a scenario or two for just ceph/rgw tests?
17:09 EmilienM colonwq: keep scenario001 for nova/neutron/ceilometer/glance/cinder using ceph backend (so only mon & osd are needed)
17:09 EmilienM colonwq: and add scenario004 with only keystone/rgw/ceph
17:10 EmilienM and maybe run some tests or something
17:10 EmilienM yeah, maybe also add swift
17:10 EmilienM and run tempest
17:10 * EmilienM looking for a use case
17:11 EmilienM colonwq: can we use ceph with rgw as a backend for glance?
17:11 colonwq EmilienM, I'll make an 004 and let's see how we can expand it.
17:11 colonwq EmilienM, Yes.
17:11 EmilienM colonwq: what is the use case with openstack + rgw again?
17:12 colonwq EmilienM, It will be to use rgw for object storage instead of swift.
17:12 EmilienM colonwq: right, but can we tell Glance to store images in there?
17:13 colonwq EmilienM, I duno about telling glance to use rgw for storage. It uses rbd directly.
17:14 EmilienM yeah, I was looking for a use case with openstack
17:14 EmilienM colonwq: if we don't have it, nevermind. We can do another thing :
17:15 colonwq EmilienM, I am hard pressed for a openstack direct usage of rgw. I think it is an 'also provided' service.
17:15 EmilienM colonwq: keep rgw beaker tests but modify the code to use bits from p-o-i (ie: include ::openstack_integration::keystone
17:15 EmilienM colonwq: look https://github.com/openstack/puppet-nova/blob/master/spec/acceptance/nova_wsgi_apache_spec.rb#L9-L13
17:15 EmilienM I think we could do the same for puppet-ceph
17:16 EmilienM instead of deploying Kilo, we could deploy OpenStack like we do in all other jobs
17:17 EmilienM colonwq: see what I mean?
17:17 colonwq EmilienM, I think so. I'll make a branch and start on it.
17:17 EmilienM great
17:18 colonwq EmilienM, in short: no 004, keep beaker, use p-o-i for needed bits. Am I correct?
17:19 EmilienM colonwq: yes for now
17:19 EmilienM we'll iterate later if needed
17:20 colonwq Cool. I'll go eat lunch then see what I can knock out.
17:20 EmilienM colonwq: the goal is to bring back our beaker jobs alive
17:20 EmilienM and make them work on jewel
17:21 EmilienM colonwq: have a good lunch! and thx for your help
17:21 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: WIP: Don't start nova service right after package installation  https://review.openstack.org/330715
17:22 iberezovskiy EmilienM, trying to fix puppet-4 jobs ^
17:23 EmilienM iberezovskiy: oh yeah? how is it related?
17:23 iberezovskiy nova-api starts before apache
17:23 iberezovskiy and then apache can't bind on 8774 port
17:23 EmilienM oh damnn
17:23 EmilienM iberezovskiy: nice catch
17:23 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: WIP: Don't start nova service right after package installation  https://review.openstack.org/330715
17:23 iberezovskiy skolekonov also helped me :)
17:24 EmilienM skolekonov, iberezovskiy ++ guys
17:24 * EmilienM afk ~1h
17:29 leanderthal|afk joined #puppet-openstack
17:32 danpawlik joined #puppet-openstack
17:32 iberezovskiy will look at CI for patches, but won't be at irc probably
18:07 rcernin joined #puppet-openstack
18:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
18:39 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Try hack to prevent nova services from autostart  https://review.openstack.org/330715
18:41 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Try hack to prevent nova services from autostart  https://review.openstack.org/330715
18:41 EmilienM mwhahaha: can you  look https://review.openstack.org/#/c/330694/ please? and see why fuel jobs fail?
18:43 iberezovskiy|off EmilienM, not deployment issue, rechecked
18:43 EmilienM iberezovskiy|off: hey
18:43 EmilienM the beaker patch fails https://review.openstack.org/#/c/330172/
18:43 iberezovskiy|off I'm partially here, but I can't see hostory, damn
18:43 EmilienM but it's maybe related to your work
18:43 iberezovskiy|off looking
18:43 EmilienM iberezovskiy|off: no worries
18:43 EmilienM I think we need https://review.openstack.org/330715
18:44 dmk0202 joined #puppet-openstack
18:45 iberezovskiy|off yeah, but see what's weird solution I have there
18:45 iberezovskiy|off we can't call policyrcd class many times
18:45 EmilienM iberezovskiy|off: in the meantime, we can fix other beaker jobs with https://review.openstack.org/#/c/330694/
18:46 EmilienM can you review it please? and maybe mwhahaha -- so when p-o-ijobs are green we can land it
18:47 iberezovskiy|off patch is fine, but we need to figure our what to do with policyrcd
18:47 iberezovskiy|off it should be possible to call it as much time as we need
18:49 EmilienM iberezovskiy|off: using <+ can be good
18:49 iberezovskiy|off let's see
18:51 xarses joined #puppet-openstack
18:52 iberezovskiy|off Error: Only subclasses can override parameters :(
19:02 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/329682
19:02 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/329714
19:06 xarses_ joined #puppet-openstack
19:10 EmilienM iberezovskiy|off: maybe we can adjust https://github.com/openstack/puppet-openstacklib/blob/master/manifests/policyrcd.pp
19:11 EmilienM iberezovskiy|off: because we're going to have the same issue for all our jobs, that run in WSGI
19:12 EmilienM ironic nova ceilometer aodh gnocchi etc
19:13 iberezovskiy|off the problem is only: how to get all service names
19:13 EmilienM iberezovskiy|off: I have another idea
19:14 EmilienM it's a bit crazy
19:14 EmilienM see https://github.com/devopsconsulting/puppet-disablestart/blob/master/files/usr/sbin/policy-rc.d
19:14 EmilienM we disable everything lol
19:14 iberezovskiy|off I thought about this
19:15 iurygregory woa
19:15 iberezovskiy|off but to be honest with users, we need to remove this file after puppet run
19:15 iberezovskiy|off which will make our manifests non-idempotent
19:16 EmilienM ok let me talk to canonical again
19:16 EmilienM -> #ubuntu-servr
19:16 EmilienM -> #ubuntu-server
19:16 iberezovskiy|off I'm there
19:19 EmilienM iberezovskiy: or something else
19:19 EmilienM iberezovskiy: we can add this in run_tests.sh:
19:20 EmilienM echo exit 101 | sudo tee /usr/sbin/policy-rc.d && sudo chmod +x /usr/sbin/policy-rc.d
19:20 EmilienM oh wait, I have another idea
19:21 iberezovskiy I'm all ears :D
19:22 EmilienM a sec
19:22 EmilienM I'm writting it
19:22 iberezovskiy sure
19:24 EmilienM it's ugly
19:24 EmilienM lol
19:26 iberezovskiy also solution could be ugly because we trying to fix packaging issue
19:29 johnmilton joined #puppet-openstack
19:31 chem|off` joined #puppet-openstack
19:32 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: api/httpd: stop nova-api service after package install  https://review.openstack.org/330757
19:32 EmilienM iberezovskiy: I'm finishing the patch but this is what I propose ^
19:33 EmilienM I think it can work
19:34 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: api/httpd: stop nova-api service after package install  https://review.openstack.org/330757
19:35 EmilienM let's see if it works
19:40 EmilienM _ody: w00t your puppet patch is merged \o/
19:40 dmk0202 joined #puppet-openstack
19:40 _ody Wow.  Kinda surprised.
19:40 _ody Not a single person had even touched it as of this morning.
19:42 _ody Set out this morning to see how many nova services I could get running inside docker using nothing but the upstream puppet modules.
19:42 _ody This going A LOT smoother than I expected
19:42 _ody Part of a very last minute constructed demo for OpenStack PDX tonight.
19:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
20:03 iberezovskiy EmilienM, I am off for today, I'll monitor your patches tomorrow
20:03 EmilienM thx!
20:03 EmilienM iberezovskiy: good night
20:04 iberezovskiy thanks )
20:04 mwhahaha we 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 installed
20:04 mwhahaha which we use a custom function for (is not pretty)
20:20 openstackstatus joined #puppet-openstack
20:22 xarses_ joined #puppet-openstack
20:23 dmk0202 joined #puppet-openstack
20:26 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Update to limit CentOS SIG to hammer  https://review.openstack.org/330168
21:04 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/330785
21:08 EmilienM mwhahaha: https://review.openstack.org/#/c/330694/ fails
21:08 EmilienM because of job failure
21:09 EmilienM I don't see why
21:09 EmilienM I'm reporting it on infra
21:09 EmilienM http://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_161119
21:11 mwhahaha thats weird
21:14 EmilienM yeah
21:15 iurygregory EmilienM, I'll start the work at home o/ tomorrow I'll give an update
21:15 EmilienM iurygregory: excellent, please let me know if you need any help
21:16 iurygregory EmilienM, sure :D have a good night
21:16 EmilienM cheers
21:26 karimb joined #puppet-openstack
21:38 openstackgerrit Keith Schincke proposed openstack/puppet-openstack-integration: Add ext4 parameters to openstack_integration::ceph  https://review.openstack.org/330797
21:45 EmilienM ok infra is really in bad shape
21:45 EmilienM timeout again for another reason
21:50 chem joined #puppet-openstack
22:02 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/330785
22:04 dmk0202 joined #puppet-openstack
22:07 xarses_ joined #puppet-openstack
22:07 ayoung joined #puppet-openstack
22:09 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Libvirt profile  https://review.openstack.org/329682
22:10 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Create libvirt micro-service  https://review.openstack.org/329714
22:16 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: nova/api: include ::nova::network::neutron  https://review.openstack.org/329529
22:19 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328591
22:22 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Update ceph_usecases to use puppet-openstack-integration  https://review.openstack.org/330806
22:23 colonwq EmilienM, are 330806 and 330797 about what you are looking for?
22:24 EmilienM colonwq: it's a good start
22:26 colonwq EmilienM, Ok. I'll keep working in that direction.
22:27 EmilienM colonwq: I commented on https://review.openstack.org/#/c/330168/
22:28 EmilienM I'm off, cheers
22:58 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: manage_policyrcd: stop eventlet support from Newton  https://review.openstack.org/330694
23:00 xarses_ joined #puppet-openstack
23:34 iurygregory_ joined #puppet-openstack
23:44 ayoung joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary