Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-07-23

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

All times shown according to UTC.

Time Nick Message
00:07 xarses joined #puppet-openstack
00:49 jasondotstar joined #puppet-openstack
01:00 dprince joined #puppet-openstack
01:18 xiaohua joined #puppet-openstack
01:34 xarses joined #puppet-openstack
01:41 xarses_ joined #puppet-openstack
01:53 EmilienM crinkle, richm: hey what's up?
02:09 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update default notification_driver for ceilometer  https://review.openstack.org/204808
02:10 EmilienM mwhahaha: thx ^
02:10 xingchao joined #puppet-openstack
02:11 EmilienM mwhahaha: though you're missing the 'Cherry-picked from'
02:11 richm EmilienM: welcome back
02:11 EmilienM richm: thanks!
02:12 EmilienM richm: I was about to step out for the night but can you tell me the issue /status with p-keystone?
02:13 richm EmilienM: https://review.openstack.org/198561
02:13 richm EmilienM: We can't get this to work right
02:14 richm EmilienM: It seems that the problem is that keystone_user/keystone_tenant resources are being created before the default domain is set
02:15 richm even if we just specify the default_domain parameter of class keystone, and not try to create another domain with is_default => true
02:15 richm I even tried to rebase that patch on top of the patch that adds anchor support - still no luck
02:18 EmilienM richm: that does a nice spot
02:49 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: Enhancements to caching support  https://review.openstack.org/204815
02:53 openstackgerrit Merged openstack/puppet-murano: Fix rspec 3.x syntax  https://review.openstack.org/204671
03:46 ducttape_ joined #puppet-openstack
03:51 andreww joined #puppet-openstack
04:03 mesamoo joined #puppet-openstack
04:11 francois joined #puppet-openstack
04:47 xiaohua joined #puppet-openstack
04:57 gfidente joined #puppet-openstack
05:08 xiaohua joined #puppet-openstack
05:11 xingchao joined #puppet-openstack
05:45 xiaohua joined #puppet-openstack
05:48 stamak joined #puppet-openstack
06:04 xingchao joined #puppet-openstack
06:55 jasondotstar joined #puppet-openstack
07:18 dgurtner joined #puppet-openstack
07:18 dgurtner joined #puppet-openstack
07:28 vinsh_ joined #puppet-openstack
07:35 vinsh joined #puppet-openstack
07:52 bogdando crinkle, dgurtner I agreed with Colleen, thank you for bringing this out
07:53 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Creation of ceilometer::db::sync  https://review.openstack.org/196607
07:58 vinsh_ joined #puppet-openstack
07:59 vinsh joined #puppet-openstack
08:00 openstackgerrit Yanis Guenane proposed openstack/puppet-glance: Creation of glance::db::sync  https://review.openstack.org/196629
08:01 dgurtner bogdando: great, thank you :)
08:02 jistr joined #puppet-openstack
08:02 paramite|afk joined #puppet-openstack
08:05 dgurtner EmilienM: mgagne can I get some more core reviews on https://review.openstack.org/#/c/171877/4
08:05 gchamoul sbadia: ^^^
08:05 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/203220
08:12 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync  https://review.openstack.org/196646
08:16 openstackgerrit Yanis Guenane proposed openstack/puppet-ironic: Creation of ironic::db::sync  https://review.openstack.org/196657
08:19 openstackgerrit Yanis Guenane proposed openstack/puppet-nova: Creation of nova::db::sync  https://review.openstack.org/196679
08:21 openstackgerrit Yanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/196683
08:22 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Sync cinder::db::sync with new standard  https://review.openstack.org/196693
08:24 openstackgerrit Yanis Guenane proposed openstack/puppet-neutron: Make package change run the db_sync  https://review.openstack.org/202920
08:25 derekh joined #puppet-openstack
08:40 openstackgerrit Yanis Guenane proposed openstack/puppet-designate: Creation of designate::db::sync  https://review.openstack.org/196622
09:07 linkedinyou joined #puppet-openstack
09:38 jasondotstar joined #puppet-openstack
09:38 stamak joined #puppet-openstack
09:44 paramite joined #puppet-openstack
10:01 gfidente joined #puppet-openstack
10:03 paramite joined #puppet-openstack
10:13 sbadia morning here \o/
10:15 sbadia mfisch: hum, it's only in order to by « semver » compliant© :-) (pin before the superior major, after if it's not useful can stop doing it)
10:19 saneax joined #puppet-openstack
10:27 paramite joined #puppet-openstack
10:31 jasondotstar joined #puppet-openstack
10:31 gchamoul spredzy: you only tag openstack-something packages?
10:32 gchamoul spredzy: for example https://review.openstack.org​/#/c/204756/1/manifests/init.pp,cm
10:33 gchamoul spredzy: you didn't tag python-nova package resource
10:39 spredzy gchamoul, python-nova is a miss from my side
10:39 spredzy thanks for pointing it out
10:41 openstackgerrit Yanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources  https://review.openstack.org/204756
10:44 spredzy hmmm actually python-nova doesn't seem to pull nova-common so it shouldnt be tagged. gchamoul could you confirm ti doesn't pull it ?
11:02 gchamoul spredzy: I confirm!
11:27 stamak joined #puppet-openstack
11:32 openstackgerrit Yanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources  https://review.openstack.org/204756
11:42 xarses_ joined #puppet-openstack
11:44 tlbr joined #puppet-openstack
11:58 linkedinyou joined #puppet-openstack
12:02 linkedinyou|2 joined #puppet-openstack
12:17 ducttape_ joined #puppet-openstack
12:23 EmilienM good morning!
12:23 dprince joined #puppet-openstack
12:41 prad joined #puppet-openstack
12:44 iurygregory good morning EmilienM o/
12:50 ferest joined #puppet-openstack
12:54 guimaluf hey guys, when running test in my composition module I'm getting this error: cannot load such file -- puppet/provider/openstack on node
12:54 guimaluf how can I fix this?
12:58 openstackgerrit Yanis Guenane proposed openstack/puppet-modulesync-configs: Enable a default modulesync.yml for infra purpose  https://review.openstack.org/205048
12:59 samuelBartel joined #puppet-openstack
13:04 bklei joined #puppet-openstack
13:09 mattymo_ joined #puppet-openstack
13:15 openstackgerrit Merged openstack/puppet-monasca: Fix metadata.json lint issues  https://review.openstack.org/204025
13:15 openstackgerrit Merged openstack/puppet-monasca: Puppet-Monasca Project Namespace Change  https://review.openstack.org/204032
13:16 mwhahaha EmilienM: hey for that change,it looks like the verification failed because a timeout downloading some gems. https://review.openstack.org/#/c/203807/
13:16 mwhahaha is there an easy way to poke it with a stick?
13:19 bklei joined #puppet-openstack
13:19 spredzy mwhahaha, reverify would do it here
13:20 mwhahaha so just comment with reverify?
13:20 spredzy mwhahaha, correct
13:20 mwhahaha we shall see :)
13:20 rhefner joined #puppet-openstack
13:21 gchamoul EmilienM, spredzy, sbadia: do we have rules about parameters validation?
13:23 gchamoul Some modules are using the validate_* functions sometimes and others not ...
13:24 spredzy gchamoul, is that for you to add one or is that for a review?
13:24 spredzy if its for you just do it :p
13:24 spredzy if its for a review do not -1 but recommend it
13:24 spredzy in a comment I would say
13:26 ducttape_ joined #puppet-openstack
13:27 gchamoul spredzy: the question was targeted a review
13:28 gchamoul spredzy: ok but it's not a requirements?
13:28 gchamoul s/requirements/requirement/
13:28 spredzy I mean if other parameter in the module do not have it it sounds unfair to enforce it on a specific review
13:29 spredzy in order to do that one should do a review where it enforces all of the param then it can become a requirement
13:29 spredzy imo
13:30 jasondotstar joined #puppet-openstack
13:30 sergmelikyan joined #puppet-openstack
13:33 morazi joined #puppet-openstack
13:34 rcallawa joined #puppet-openstack
13:36 xingchao joined #puppet-openstack
13:37 ducttape_ joined #puppet-openstack
13:38 guimaluf hey guys I'm getting this error when testing my module using puppet-keystone. Could not autoload puppet/type/keystone_endpoint: Could not autoload puppet/provider/keystone_endpoint/openstack: cannot load such file -- puppet/provider/openstack on node
13:38 guimaluf hey guys I'm getting this error when testing my module using puppet-keystone. Could not autoload puppet/type/keystone_endpoint: Could not autoload puppet/provider/keystone_endpoint/openstack: cannot load such file -- puppet/provider/openstack on node. ANyone knows how to fixit or what can be happening?
13:41 xingchao joined #puppet-openstack
13:44 richm joined #puppet-openstack
13:54 bogdando Folks, how should we track such changes in docs? https://review.openstack.org/132351 Do we use a DocImpact tag?
13:54 bogdando or if patch changes defaults behavior
13:55 bogdando It adds some functionality to providers, but there is no place to describe such things in the structure of a puppet module
13:56 bogdando maybe README.md?
14:07 EmilienM spredzy, mwhahaha: no reverify but recheck
14:07 mwhahaha reverify started it
14:08 EmilienM cool, I just know they kept it for backward compatibility
14:08 mwhahaha hopefully it works this time and then i'll re-cherry pick
14:09 gchamoul bogdando: to me, it should be documented too in the @doc instance variable of the custom type.
14:10 gchamoul bogdando: https://docs.puppetlabs.com/guides/c​ustom_types.html#type-documentation
14:11 openstackgerrit Merged openstack/puppet-cinder: Update default notification_driver for ceilometer  https://review.openstack.org/203807
14:11 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update default notification_driver for ceilometer  https://review.openstack.org/204808
14:18 openstackgerrit Merged openstack/puppet-nova: Make libvirt migration security configurable  https://review.openstack.org/171877
14:20 rbrooker joined #puppet-openstack
14:20 mfisch sbadia: can you close out a couple trivial reviews?  https://review.openstack.org/#/c/199806/
14:20 mfisch and https://review.openstack.org/#/c/201751/
14:21 ducttape_ joined #puppet-openstack
14:21 sbadia mfisch: oups
14:21 sbadia yes sure!
14:22 sbadia hum, I've a 404 on puppetlabs/manila
14:22 sbadia https://forge.puppetlabs.com/puppetlabs/manila
14:22 mfisch let me look I may need to fix the link too
14:23 mfisch yeah it moved
14:23 mfisch I think recently
14:23 sbadia yep
14:23 sbadia np
14:24 mfisch we need to change the icons on these to not be Emilien's head
14:24 gchamoul lol
14:24 EmilienM mfisch: it's on my todo for today
14:24 mfisch welcome back EmilienM
14:25 sbadia haha :D
14:25 sbadia EmilienM: \o/
14:25 gchamoul mfisch: EmilienM has a ton of selfies ... so we could change it every day ;-0
14:25 EmilienM mfisch: thanks!
14:25 EmilienM gchamoul: lol
14:26 gchamoul EmilienM: I would like to see this one with the bear! ;-)
14:26 sbadia gchamoul: haha :D
14:26 openstackgerrit Matt Fischer proposed openstack/puppet-manila: Note that manila is on Puppet Forge  https://review.openstack.org/201751
14:26 mfisch sbadia: updated
14:27 sbadia thx!
14:28 mfisch I also redid the keystone caching code yesterday
14:29 mfisch for some dumb reason there is a 2nd cache server list setting
14:29 mfisch with a different name
14:29 mfisch and it seems like you need both
14:29 EmilienM gchamoul: and you haven't see everything yet
14:30 gchamoul EmilienM: hmmm, I don't wanna know what the bear did to you! :-p
14:31 EmilienM mfisch: the avatar is linked to gravatar which is my personal email so my personal picture
14:31 EmilienM mff
14:34 openstackgerrit Merged openstack/puppet-ceilometer: fix module install reference  https://review.openstack.org/199806
14:37 EmilienM mfisch: done: https://forge.puppetlabs.com/openstack
14:37 mfisch ;)
14:38 spredzy EmilienM, n00b question. Isn't recheck used if test fail before validation and reverify if test fails after validation?
14:38 EmilienM I use recheck all the time
14:38 spredzy or simply we should use recheck all the time ?
14:38 spredzy ack
14:39 EmilienM spredzy: http://docs.openstack.org/infra/manu​al/developers.html#automated-testing
14:39 EmilienM "To re-run check or gate jobs, leave a comment on the review with the form “recheck”."
14:40 iurygregory richm, are you here? i have looked https://github.com/puppetlabs/puppetl​abs-apache/tree/master/manifests/mod and I did not find  the openidc and mellon mods, this is the repo that crinkle was saying right?
14:40 spredzy yep thanks that is an authoritative answer heh
14:56 crinkle iurygregory: I think richm was saying that you could use the generalized apache::mod defined type for those https://github.com/puppetlabs/puppe​tlabs-apache#defined-type-apachemod
14:58 iurygregory make sense crinkle thanks, i'll add a note saying that we will use this according to the module chosen ok?
14:59 crinkle iurygregory: great, thanks!
15:01 Fin1te joined #puppet-openstack
15:02 mdorman joined #puppet-openstack
15:02 richm iurygregory: crinkle: yes, that's right
15:05 rcallawa_ joined #puppet-openstack
15:06 morazi joined #puppet-openstack
15:20 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-federation  https://review.openstack.org/190361
15:37 saneax joined #puppet-openstack
15:38 rbrooker joined #puppet-openstack
15:42 rcallawa joined #puppet-openstack
15:45 jasondotstar joined #puppet-openstack
15:51 xarses_ joined #puppet-openstack
16:02 angdraug joined #puppet-openstack
16:06 openstackgerrit Merged openstack/puppet-monasca: Add tag to package and service resources  https://review.openstack.org/204746
16:09 openstackgerrit Merged openstack/puppet-ceilometer: Add tag to package and service resources  https://review.openstack.org/204592
16:16 finnx joined #puppet-openstack
16:23 openstackgerrit Merged openstack/puppet-swift: Add tag to package and service resources  https://review.openstack.org/204668
16:35 finnx left #puppet-openstack
16:37 xingchao joined #puppet-openstack
16:38 simonmcc joined #puppet-openstack
16:55 openstackgerrit Erickson Filipe Guedes dos Santos proposed openstack/puppet-heat: Fix module install reference  https://review.openstack.org/205157
16:58 aimon_ joined #puppet-openstack
16:58 guimaluf hey guys, I've realize that swift_{account,container,object}_config resource must be applied after Concat['/etc/swift/${type}-server.conf']. Which is the best way to accomplish this? I did a case which declares Concat -> Swift_type_config <||>, but I can't generate a rspec to test this. Anyone could help me?
17:08 sergmelikyan joined #puppet-openstack
17:18 finnx joined #puppet-openstack
17:18 finnx joined #puppet-openstack
17:25 openstackgerrit Matt Fischer proposed openstack/puppet-keystone: Enhancements to caching support  https://review.openstack.org/204815
17:30 sergmelikyan joined #puppet-openstack
17:39 rcallawa_ joined #puppet-openstack
17:55 sergmelikyan joined #puppet-openstack
17:56 angdraug joined #puppet-openstack
18:08 EmilienM crinkle, pabelanger: have you any update about puppet-openstack-integration work done lately? I'm interested to chat a bit
18:09 crinkle i haven't done any work on it, i think i've seen a couple of patches from pabelanger though
18:11 EmilienM crinkle: so I dropped a comment on https://review.openstack.org/#/c/201298/
18:11 EmilienM crinkle: I'm concerned at how do we test the repo itself, again
18:12 EmilienM with beaker jobs, that would test the first scenario
18:12 EmilienM otherwise, I don't see how we can certify the integration manifest are actually right
18:12 rcallawa joined #puppet-openstack
18:13 Fin1te joined #puppet-openstack
18:15 pabelanger Ya, have something local. Also working to get new nodes online in infra
18:16 pabelanger talking to crinkle, I don't actually thing we need beaker to lauch the testing.  We can leverage existing infra tools for it.  I have a gate test written local, on how it would look
18:16 pabelanger EmilienM: Also have approval for MTL too.
18:17 EmilienM pabelanger: cool for mtl
18:17 EmilienM pabelanger: can you develop/show 'leverage existing infra tools' ?
18:17 EmilienM pabelanger: do you mean to run the puppet-apply jobs?
18:19 pabelanger Ya, basically we'll launch a new node using nodepool (configured via jenkins), install puppet modules required via zuul-cloner. Then run puppet apply on the manifest.  Check it is sucessful, then launch tempest to validate the cloud
18:20 EmilienM pabelanger: how do you provision the node with puppet modules?
18:21 EmilienM pabelanger: why not using beaker? it's exactly what we already need and use
18:22 pabelanger EmilienM: personally, looking into beaker, my understanding there is no different in functionality in how the current zuul / nodepool / jenkins workflow works.
18:22 EmilienM pabelanger: so let's use what we already use in Puppet OpenStack: beaker
18:22 pabelanger beaker launches nodes into sandbox enviornment, then invokes puppet I believe.
18:22 pabelanger We can do that with nodepool and jenkins
18:22 pabelanger if i understand right, beaker just allows you to do that locally
18:22 EmilienM pabelanger: we are already doing functional testing with puppet modules right now, with beaker
18:23 pabelanger currently nodepool / zuul does not
18:23 ducttape_ joined #puppet-openstack
18:23 clayton does beaker support an ssh provisioner?
18:23 pabelanger EmilienM: Right, I am not suggestion changing those modules. Just simple create a gate job for the integration module that runs puppet apply directly on a bare node
18:24 EmilienM pabelanger: and how do you runs tests ? (ie serverspec)
18:24 EmilienM clayton: yes
18:24 EmilienM clayton: it does ssh
18:24 openstackgerrit Merged openstack/puppet-tuskar: Add tag to package and service resources  https://review.openstack.org/204660
18:25 pabelanger EmilienM: my understanding is we have manifests for each deployment.  In this case, scenario0001.pp. I see it working, once the bare jenkins node is up and running.  You run puppet apply --module-path=<modules> --verbose scenario001.pp
18:25 EmilienM pabelanger: I'm afk ~1h, bbl
18:26 pabelanger I am unsure what serverspec are too, outside the scenario001.pp
18:27 pabelanger Beaker is an acceptance testing harness for Puppet PE and other Puppet Projects. It can also be used as a virtual machine provisioner - setting up machines, running any configuration on those machines and then exiting.
18:27 pabelanger This is basically what nodepool does, right?
18:28 sergmelikyan joined #puppet-openstack
18:30 crinkle so what beaker-rspec gives us is a ruby dsl to provision a vm, run puppet apply, and assert state
18:30 crinkle we don't really need it to provision a vm, nodepool does that for us
18:31 crinkle we don't really need it to assert state, tempest can do that for us
18:31 crinkle so it sort of makes sense to cut out beaker and just treat this more like how devstack is tested, which is just to run devstack and then run tempest
18:32 crinkle beaker makes sense for functional testing individual modules but i think it offers less to us for integration testing
18:33 pabelanger Right, I see this like how devstack is tested too.  Get bare node, run puppet (directly) to provision, then run tempest against it (either local or external)
18:34 pabelanger A simple bash script called run_puppet.sh (like we have in system-config) is the most basic launcher that is needed, IMO
18:37 openstackgerrit Merged openstack/puppet-tripleo: Implement firewalling in tripleo::firewall  https://review.openstack.org/202027
18:37 openstackgerrit Merged openstack/puppet-designate: Add hooks for external install & svc management  https://review.openstack.org/197172
18:37 openstackgerrit Merged openstack/puppet-glance: Add tag to package and service resources  https://review.openstack.org/204489
18:37 openstackgerrit Merged openstack/puppet-cinder: Add tag to package and service resources  https://review.openstack.org/204532
18:39 openstackgerrit Merged openstack/puppet-keystone: Add tag to package and service resources  https://review.openstack.org/204598
18:40 openstackgerrit Merged openstack/puppet-heat: Add tag to package and service resources  https://review.openstack.org/204497
18:41 openstackgerrit Merged openstack/puppet-trove: Add region and resource url related options  https://review.openstack.org/200346
18:45 openstackgerrit Merged openstack/puppet-manila: Add support to configure GlusterFS drivers with Manila shares.  https://review.openstack.org/200811
18:55 bitblt joined #puppet-openstack
18:56 ducttape_ joined #puppet-openstack
18:58 clayton EmilienM: thanks for the merge on the hooks patch.   Matt and I were talking about it recently and I think we'll probably be working on keystone next
19:07 openstackgerrit Guilherme Balzana proposed openstack/puppet-swift: Swift_${type}_config should be applied after /etc/swift/${type}-server.conf  https://review.openstack.org/205243
19:19 bklei joined #puppet-openstack
19:21 sergmelikyan joined #puppet-openstack
19:22 ducttape_ joined #puppet-openstack
19:34 sergmelikyan joined #puppet-openstack
19:42 Fin1te joined #puppet-openstack
19:47 sergmelikyan joined #puppet-openstack
20:03 sergmelikyan joined #puppet-openstack
20:04 ducttape_ joined #puppet-openstack
20:10 openstackgerrit Matt Fischer proposed openstack/puppet-ceilometer: Enable support for memcached_servers  https://review.openstack.org/199835
20:14 ducttape_ joined #puppet-openstack
20:19 ducttape_ joined #puppet-openstack
20:22 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-nova: Update Nova README  https://review.openstack.org/204262
20:27 sergmelikyan joined #puppet-openstack
20:34 sergmelikyan joined #puppet-openstack
20:36 sergmelikyan joined #puppet-openstack
20:37 openstackgerrit Guilherme Balzana proposed openstack/puppet-swift: Add a blank space at begining of each filter  https://review.openstack.org/205284
20:39 richm joined #puppet-openstack
20:39 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Update Swift README  https://review.openstack.org/205285
20:39 ducttape_ joined #puppet-openstack
20:41 dalgaaf joined #puppet-openstack
20:42 dalgaaf left #puppet-openstack
20:44 sergmelikyan joined #puppet-openstack
20:45 openstackgerrit Guilherme Balzana proposed openstack/puppet-swift: Add a blank space at begining of each filter  https://review.openstack.org/205284
21:03 colby__ joined #puppet-openstack
21:04 sergmelikyan joined #puppet-openstack
21:11 EmilienM clayton: sounds good
21:12 EmilienM crinkle: "assert state" == testing the services ?
21:20 mgarza_ joined #puppet-openstack
21:26 crinkle EmilienM: yes
21:26 rcallawa_ joined #puppet-openstack
21:27 EmilienM crinkle: so you want to use the puppet apply jobs to gate puppet-openstack-integration?
21:28 sergmelikyan joined #puppet-openstack
21:28 crinkle EmilienM: no those are different
21:28 crinkle EmilienM: we'd need a new job
21:29 sergmelikyan joined #puppet-openstack
21:29 EmilienM crinkle: with new scripts, etc?
21:30 openstackgerrit Merged openstack/puppet-openstacklib: Add a proxy inifile provider  https://review.openstack.org/202488
21:31 crinkle EmilienM: yes
21:32 EmilienM crinkle: mhhh ok
21:32 sergmelikyan joined #puppet-openstack
21:36 jasondotstar joined #puppet-openstack
21:38 openstackgerrit Merged openstack/puppet-nova: Support allow_start and allow_end parameters  https://review.openstack.org/135123
21:40 sergmelikyan joined #puppet-openstack
21:43 sergmelikyan joined #puppet-openstack
21:44 openstackgerrit Alex Schultz proposed openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift  https://review.openstack.org/198669
21:45 sergmelikyan joined #puppet-openstack
21:46 sergmelikyan joined #puppet-openstack
21:46 EmilienM crinkle: sorry if I don't understand well, but I just don't see the difference between testing puppet-keystone or puppet-openstack-integration, they are still manifests
21:47 EmilienM and I have the feeling these scripts will implement what we sort of do with beaker right now, at least for provisioning part
21:47 sergmelikyan joined #puppet-openstack
21:47 sergmelikyan joined #puppet-openstack
21:51 crinkle the functional tests for say puppet-keystone only test that module, not all the modules together, and we have tests like 'describe port(500) { it { is_expected.to be_listening } }' which is part of the serverspec dsl
21:52 crinkle the puppet-openstack-integration tests would have manifests that test all the modules in different scenarios, and instead of writing tests in rspec we just run tempest against it
21:52 ducttape_ joined #puppet-openstack
21:52 crinkle so the job would just be puppet apply ; run tempest
21:52 crinkle there's not a lot of need for beaker anymore because we're not using most of its features
21:53 crinkle this is what pabelanger and i were thinking, possibly nibalizer has a different vision of it he could express?
21:54 EmilienM crinkle: 1/ we still need the provision part of beaker, which is already in place now and 2/ I still see some plus-value to use serverspec, before tempest though (or even run tempest from beaker)
21:55 EmilienM I just feel now we would have to write bash scripts to execute some things already done by beaker, and the code is already everywhere in our module
21:55 crinkle EmilienM: why do we need the provision part? we're just hacking around that anyway
21:55 EmilienM crinkle: because we use zuul-cloner you mean?
21:55 crinkle EmilienM: no with the hack to ssh into localhost
21:55 linkedinyou joined #puppet-openstack
21:56 crinkle EmilienM: how does devstack get tested? it just runs a shell script and then runs tempest
21:56 crinkle it seems like our integration test should be similar
21:58 EmilienM crinkle: ok so let me summarize our eventual plan: 1/ write some scripts (eventually in puppet-openstack-integration) to prepare a VM with our modules (using zuul cloner), 2/ write a scenario manifest in puppet-openstack-integration/manifests, 3/ run puppet apply on the node 4/ run tempest at the end
21:59 crinkle EmilienM: we already have #1 done
21:59 EmilienM crinkle: where? sorry I've missed that part
21:59 EmilienM oh
21:59 sergmelikyan joined #puppet-openstack
21:59 EmilienM crinkle: your work on zuul cloner, right
21:59 crinkle right
22:00 crinkle the rest of that plan sounds good to me
22:00 EmilienM crinkle: that's sounds a good plan and the risk is low I guess
22:00 crinkle cool
22:00 EmilienM crinkle: I would have done beaker if I would have been alone
22:00 EmilienM but indeed, we can do that way too, wfm
22:02 sergmelikyan joined #puppet-openstack
22:04 EmilienM crinkle: I'm about to patch project-config for a new voting job, running only on puppet-openstack-integration that will run a script *.sh and right after it's merged, I'll push the *.sh script that is returning 0 so we can start working from here
22:04 EmilienM crinkle: or we prefer to start by non voting and see later?
22:05 crinkle EmilienM: I think pabelanger is working on something, let's maybe wait for his input?
22:06 EmilienM oh sure
22:06 crinkle in any case non voting would be better
22:07 EmilienM crinkle: can you reply to us on https://review.openstack.org/#/c/199676/ please ,
22:13 crinkle EmilienM: done
22:13 EmilienM crinkle: I'm more concerned about the bug I'm fixing: https://bugs.launchpad.net/p​uppet-keystone/+bug/1474049
22:13 openstack Launchpad bug 1474049 in puppet-keystone "if admin_project_domain == admin_user_domain, admin_domain is never created" [High,In progress] - Assigned to Emilien Macchi (emilienm)
22:16 linkedinyou|2 joined #puppet-openstack
22:23 crinkle EmilienM: I don't understand the bug report, admin_domain isn't mentioned in keystone::roles::admin
22:26 EmilienM crinkle: admin domain is not a param actually
22:26 EmilienM it's just to say, 'we won't  have any domain at all'
22:28 timfreund joined #puppet-openstack
22:30 nibalizer hi
22:30 nibalizer the above looks like stuff I should read
22:30 nibalizer which I will do later
22:30 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Fix catalog compilation when not configuring endpoint  https://review.openstack.org/190308
22:36 ducttape_ joined #puppet-openstack
22:38 openstackgerrit Emilien Macchi proposed stackforge/puppet-ceph: Use beaker-puppet_install_helper to install puppet  https://review.openstack.org/197332
22:51 openstackgerrit Merged openstack/puppet-ceilometer: Enable support for memcached_servers  https://review.openstack.org/199835
22:53 linkedinyou joined #puppet-openstack
22:57 imcsk8 richm: are you around?
22:58 imcsk8 richm: what's the value that should on packstack's CONFIG_KEYSTONE_LDAP_SUFFIX ??
22:58 imcsk8 richm: for example this is my user dn: uid=openstack, ou=openstack, dc=my-domain, dc=com
23:17 sergmelikyan joined #puppet-openstack
23:18 serg_melikyan joined #puppet-openstack
23:23 ducttape_ joined #puppet-openstack
23:44 sergmelikyan joined #puppet-openstack
23:56 jasondotstar joined #puppet-openstack

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