| *** xarses has quit IRC | 00:06 | |
| *** xarses has joined #puppet-openstack | 00:07 | |
| *** sergmelikyan has quit IRC | 00:08 | |
| *** xingchao has quit IRC | 00:08 | |
| *** jasondotstar has quit IRC | 00:47 | |
| *** jasondotstar has joined #puppet-openstack | 00:49 | |
| *** jasondotstar has quit IRC | 00:54 | |
| *** xarses has quit IRC | 00:59 | |
| *** dprince has joined #puppet-openstack | 01:00 | |
| *** angdraug has quit IRC | 01:05 | |
| *** xiaohua has joined #puppet-openstack | 01:18 | |
| *** dprince has quit IRC | 01:21 | |
| *** mesamoo has quit IRC | 01:27 | |
| *** xarses has joined #puppet-openstack | 01:34 | |
| *** xarses_ has joined #puppet-openstack | 01:41 | |
| *** xarses has quit IRC | 01:44 | |
| *** EmilienM|off is now known as EmilienM | 01:52 | |
| *** EmilienM sets mode: +o crinkle | 01:52 | |
| *** EmilienM sets mode: +o clayton | 01:52 | |
| *** EmilienM sets mode: +o mgagne | 01:52 | |
| EmilienM | crinkle, richm: hey what's up? | 01:53 |
|---|---|---|
| openstackgerrit | Alex Schultz proposed openstack/puppet-cinder: Update default notification_driver for ceilometer https://review.openstack.org/204808 | 02:09 |
| EmilienM | mwhahaha: thx ^ | 02:10 |
| *** xingchao has joined #puppet-openstack | 02:10 | |
| EmilienM | mwhahaha: though you're missing the 'Cherry-picked from' | 02:11 |
| richm | EmilienM: welcome back | 02:11 |
| EmilienM | richm: thanks! | 02:11 |
| EmilienM | richm: I was about to step out for the night but can you tell me the issue /status with p-keystone? | 02:12 |
| richm | EmilienM: https://review.openstack.org/198561 | 02:13 |
| richm | EmilienM: We can't get this to work right | 02:13 |
| richm | EmilienM: It seems that the problem is that keystone_user/keystone_tenant resources are being created before the default domain is set | 02:14 |
| 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:15 |
| EmilienM | richm: that does a nice spot | 02:18 |
| *** richm has quit IRC | 02:47 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: Enhancements to caching support https://review.openstack.org/204815 | 02:49 |
| openstackgerrit | Merged openstack/puppet-murano: Fix rspec 3.x syntax https://review.openstack.org/204671 | 02:53 |
| *** ducttape_ has joined #puppet-openstack | 03:46 | |
| *** andreww has joined #puppet-openstack | 03:51 | |
| *** xarses_ has quit IRC | 03:54 | |
| *** mesamoo has joined #puppet-openstack | 04:03 | |
| *** ducttape_ has quit IRC | 04:06 | |
| *** francois has quit IRC | 04:10 | |
| *** francois has joined #puppet-openstack | 04:11 | |
| *** xiaohua has quit IRC | 04:47 | |
| *** xiaohua has joined #puppet-openstack | 04:47 | |
| *** xingchao has quit IRC | 04:54 | |
| *** xiaohua has quit IRC | 04:56 | |
| *** gfidente has joined #puppet-openstack | 04:57 | |
| *** xiaohua has joined #puppet-openstack | 05:08 | |
| *** xingchao has joined #puppet-openstack | 05:11 | |
| *** xingchao has quit IRC | 05:16 | |
| *** xiaohua has quit IRC | 05:16 | |
| *** xiaohua has joined #puppet-openstack | 05:45 | |
| *** stamak has joined #puppet-openstack | 05:48 | |
| *** xingchao has joined #puppet-openstack | 06:04 | |
| *** stamak has quit IRC | 06:32 | |
| *** jasondotstar has joined #puppet-openstack | 06:55 | |
| *** dgurtner has joined #puppet-openstack | 07:18 | |
| *** dgurtner has quit IRC | 07:18 | |
| *** dgurtner has joined #puppet-openstack | 07:18 | |
| *** xingchao has quit IRC | 07:26 | |
| *** vinsh_ has joined #puppet-openstack | 07:29 | |
| *** vinsh has quit IRC | 07:31 | |
| *** vinsh has joined #puppet-openstack | 07:35 | |
| *** vinsh_ has quit IRC | 07:36 | |
| bogdando | crinkle, dgurtner I agreed with Colleen, thank you for bringing this out | 07:52 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-ceilometer: Creation of ceilometer::db::sync https://review.openstack.org/196607 | 07:53 |
| *** vinsh_ has joined #puppet-openstack | 07:58 | |
| *** vinsh has quit IRC | 07:59 | |
| *** vinsh has joined #puppet-openstack | 07:59 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-glance: Creation of glance::db::sync https://review.openstack.org/196629 | 08:00 |
| dgurtner | bogdando: great, thank you :) | 08:01 |
| *** jistr has joined #puppet-openstack | 08:02 | |
| *** paramite|afk has joined #puppet-openstack | 08:02 | |
| *** vinsh_ has quit IRC | 08:03 | |
| *** paramite|afk is now known as paramite | 08:04 | |
| 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:05 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-heat: Creation of heat::db::sync https://review.openstack.org/196646 | 08:12 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-ironic: Creation of ironic::db::sync https://review.openstack.org/196657 | 08:16 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-nova: Creation of nova::db::sync https://review.openstack.org/196679 | 08:19 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-sahara: Creation of sahara::db::sync https://review.openstack.org/196683 | 08:21 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-cinder: Sync cinder::db::sync with new standard https://review.openstack.org/196693 | 08:22 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-neutron: Make package change run the db_sync https://review.openstack.org/202920 | 08:24 |
| *** derekh has joined #puppet-openstack | 08:25 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-designate: Creation of designate::db::sync https://review.openstack.org/196622 | 08:40 |
| *** saneax has quit IRC | 09:01 | |
| *** linkedinyou has joined #puppet-openstack | 09:07 | |
| *** jasondotstar has quit IRC | 09:25 | |
| *** jasondotstar has joined #puppet-openstack | 09:38 | |
| *** barthalion has quit IRC | 09:38 | |
| *** stamak has joined #puppet-openstack | 09:38 | |
| *** paramite has quit IRC | 09:42 | |
| *** jasondotstar has quit IRC | 09:43 | |
| *** paramite has joined #puppet-openstack | 09:44 | |
| *** paramite has quit IRC | 09:49 | |
| *** gfidente has quit IRC | 09:59 | |
| *** gfidente has joined #puppet-openstack | 10:01 | |
| *** paramite has joined #puppet-openstack | 10:03 | |
| sbadia | morning here \o/ | 10:13 |
| *** paramite has quit IRC | 10:14 | |
| 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:15 |
| *** saneax has joined #puppet-openstack | 10:19 | |
| *** paramite has joined #puppet-openstack | 10:27 | |
| *** jasondotstar has joined #puppet-openstack | 10:31 | |
| gchamoul | spredzy: you only tag openstack-something packages? | 10:31 |
| gchamoul | spredzy: for example https://review.openstack.org/#/c/204756/1/manifests/init.pp,cm | 10:32 |
| gchamoul | spredzy: you didn't tag python-nova package resource | 10:33 |
| spredzy | gchamoul, python-nova is a miss from my side | 10:39 |
| spredzy | thanks for pointing it out | 10:39 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources https://review.openstack.org/204756 | 10:41 |
| 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 ? | 10:44 |
| gchamoul | spredzy: I confirm! | 11:02 |
| *** paramite is now known as paramite|afk | 11:16 | |
| *** paramite|afk is now known as paramite | 11:18 | |
| *** stamak has quit IRC | 11:23 | |
| *** rcallawa_ has quit IRC | 11:24 | |
| *** stamak has joined #puppet-openstack | 11:27 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-nova: Add tag to package and service resources https://review.openstack.org/204756 | 11:32 |
| *** stamak has quit IRC | 11:40 | |
| *** tlbr has quit IRC | 11:42 | |
| *** xarses_ has joined #puppet-openstack | 11:42 | |
| *** tlbr has joined #puppet-openstack | 11:44 | |
| *** mesamoo has quit IRC | 11:45 | |
| *** andreww has quit IRC | 11:45 | |
| *** linkedinyou has quit IRC | 11:57 | |
| *** linkedinyou has joined #puppet-openstack | 11:58 | |
| *** linkedinyou|2 has joined #puppet-openstack | 12:02 | |
| *** linkedinyou has quit IRC | 12:03 | |
| *** linkedinyou|2 is now known as linkedinyou | 12:04 | |
| *** jasondotstar has quit IRC | 12:11 | |
| *** ducttape_ has joined #puppet-openstack | 12:17 | |
| *** chandankumar is now known as chkumar246 | 12:18 | |
| EmilienM | good morning! | 12:23 |
| *** dprince has joined #puppet-openstack | 12:23 | |
| *** ducttape_ has quit IRC | 12:36 | |
| *** prad has joined #puppet-openstack | 12:41 | |
| iurygregory | good morning EmilienM o/ | 12:44 |
| *** ferest has joined #puppet-openstack | 12:50 | |
| 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:54 |
| *** ferest has quit IRC | 12:57 | |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-modulesync-configs: Enable a default modulesync.yml for infra purpose https://review.openstack.org/205048 | 12:58 |
| *** samuelBartel has joined #puppet-openstack | 12:59 | |
| *** paramite has quit IRC | 12:59 | |
| *** bklei has joined #puppet-openstack | 13:04 | |
| *** xiaohua has quit IRC | 13:04 | |
| *** bklei has quit IRC | 13:08 | |
| *** mattymo_ has quit IRC | 13:09 | |
| *** mattymo_ has joined #puppet-openstack | 13:10 | |
| 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:15 |
| 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:16 |
| *** bklei has joined #puppet-openstack | 13:19 | |
| spredzy | mwhahaha, reverify would do it here | 13:19 |
| mwhahaha | so just comment with reverify? | 13:20 |
| spredzy | mwhahaha, correct | 13:20 |
| mwhahaha | we shall see :) | 13:20 |
| *** linkedinyou has quit IRC | 13:20 | |
| *** rhefner has quit IRC | 13:20 | |
| *** rhefner has joined #puppet-openstack | 13:20 | |
| gchamoul | EmilienM, spredzy, sbadia: do we have rules about parameters validation? | 13:21 |
| gchamoul | Some modules are using the validate_* functions sometimes and others not ... | 13:23 |
| 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:24 |
| *** ducttape_ has joined #puppet-openstack | 13:26 | |
| gchamoul | spredzy: the question was targeted a review | 13:27 |
| gchamoul | spredzy: ok but it's not a requirements? | 13:28 |
| gchamoul | s/requirements/requirement/ | 13:28 |
| *** ducttape_ has quit IRC | 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:28 |
| 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:29 |
| *** jasondotstar has joined #puppet-openstack | 13:30 | |
| *** sergmelikyan has joined #puppet-openstack | 13:30 | |
| *** morazi has joined #puppet-openstack | 13:33 | |
| *** rcallawa has joined #puppet-openstack | 13:34 | |
| *** xingchao has joined #puppet-openstack | 13:36 | |
| *** ducttape_ has joined #puppet-openstack | 13:37 | |
| 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:38 |
| *** xingchao has quit IRC | 13:41 | |
| *** xingchao has joined #puppet-openstack | 13:42 | |
| *** richm has joined #puppet-openstack | 13:44 | |
| *** ducttape_ has quit IRC | 13:52 | |
| 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:54 |
| bogdando | It adds some functionality to providers, but there is no place to describe such things in the structure of a puppet module | 13:55 |
| bogdando | maybe README.md? | 13:56 |
| EmilienM | spredzy, mwhahaha: no reverify but recheck | 14:07 |
| mwhahaha | reverify started it | 14:07 |
| 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:08 |
| gchamoul | bogdando: to me, it should be documented too in the @doc instance variable of the custom type. | 14:09 |
| gchamoul | bogdando: https://docs.puppetlabs.com/guides/custom_types.html#type-documentation | 14:10 |
| 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:11 |
| openstackgerrit | Merged openstack/puppet-nova: Make libvirt migration security configurable https://review.openstack.org/171877 | 14:18 |
| *** rbrooker has 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:20 |
| *** ducttape_ has joined #puppet-openstack | 14:21 | |
| sbadia | mfisch: oups | 14:21 |
| sbadia | yes sure! | 14:21 |
| 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:22 |
| mfisch | yeah it moved | 14:23 |
| mfisch | I think recently | 14:23 |
| sbadia | yep | 14:23 |
| sbadia | np | 14:23 |
| 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:24 |
| 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:25 |
| 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:26 |
| sbadia | thx! | 14:27 |
| mfisch | I also redid the keystone caching code yesterday | 14:28 |
| 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:29 |
| *** samuelBartel has quit IRC | 14:30 | |
| gchamoul | EmilienM: hmmm, I don't wanna know what the bear did to you! :-p | 14:30 |
| EmilienM | mfisch: the avatar is linked to gravatar which is my personal email so my personal picture | 14:31 |
| EmilienM | mff | 14:31 |
| *** vsaienko has quit IRC | 14:31 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: fix module install reference https://review.openstack.org/199806 | 14:34 |
| 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:38 |
| EmilienM | spredzy: http://docs.openstack.org/infra/manual/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:39 |
| iurygregory | richm, are you here? i have looked https://github.com/puppetlabs/puppetlabs-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:40 |
| *** spredzy is now known as spredzy|afk | 14:48 | |
| crinkle | iurygregory: I think richm was saying that you could use the generalized apache::mod defined type for those https://github.com/puppetlabs/puppetlabs-apache#defined-type-apachemod | 14:56 |
| iurygregory | make sense crinkle thanks, i'll add a note saying that we will use this according to the module chosen ok? | 14:58 |
| crinkle | iurygregory: great, thanks! | 14:59 |
| *** saneax has quit IRC | 14:59 | |
| *** Fin1te has joined #puppet-openstack | 15:01 | |
| *** morazi has quit IRC | 15:01 | |
| *** mdorman has joined #puppet-openstack | 15:02 | |
| richm | iurygregory: crinkle: yes, that's right | 15:02 |
| *** rcallawa_ has joined #puppet-openstack | 15:05 | |
| *** morazi has joined #puppet-openstack | 15:06 | |
| *** rcallawa has quit IRC | 15:08 | |
| *** jasondotstar has quit IRC | 15:12 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-federation https://review.openstack.org/190361 | 15:20 |
| *** xingchao has quit IRC | 15:25 | |
| *** xarses_ has quit IRC | 15:28 | |
| *** rbrooker has quit IRC | 15:30 | |
| *** saneax has joined #puppet-openstack | 15:37 | |
| *** rbrooker has joined #puppet-openstack | 15:38 | |
| *** rcallawa_ has quit IRC | 15:40 | |
| *** rcallawa has joined #puppet-openstack | 15:42 | |
| *** jasondotstar has joined #puppet-openstack | 15:45 | |
| *** xarses_ has joined #puppet-openstack | 15:52 | |
| *** xarses_ is now known as xarses | 15:54 | |
| *** dgurtner has quit IRC | 15:58 | |
| *** jistr has quit IRC | 16:01 | |
| *** angdraug has joined #puppet-openstack | 16:02 | |
| *** angdraug has quit IRC | 16:03 | |
| openstackgerrit | Merged openstack/puppet-monasca: Add tag to package and service resources https://review.openstack.org/204746 | 16:06 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Add tag to package and service resources https://review.openstack.org/204592 | 16:09 |
| *** rbrooker has quit IRC | 16:12 | |
| *** finnx has joined #puppet-openstack | 16:16 | |
| openstackgerrit | Merged openstack/puppet-swift: Add tag to package and service resources https://review.openstack.org/204668 | 16:23 |
| *** finnx has left #puppet-openstack | 16:35 | |
| *** xingchao has joined #puppet-openstack | 16:37 | |
| *** simonmcc has quit IRC | 16:38 | |
| *** simonmcc has joined #puppet-openstack | 16:38 | |
| *** xingchao has quit IRC | 16:42 | |
| *** Fin1te has quit IRC | 16:46 | |
| *** derekh has quit IRC | 16:46 | |
| *** sergmelikyan has quit IRC | 16:50 | |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/puppet-heat: Fix module install reference https://review.openstack.org/205157 | 16:55 |
| *** aimon_ has 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? | 16:58 |
| *** aimon has quit IRC | 17:00 | |
| *** jasondotstar has quit IRC | 17:03 | |
| *** sergmelikyan has joined #puppet-openstack | 17:08 | |
| *** finnx has joined #puppet-openstack | 17:18 | |
| *** finnx has quit IRC | 17:18 | |
| *** finnx has joined #puppet-openstack | 17:18 | |
| *** finnx has quit IRC | 17:18 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-keystone: Enhancements to caching support https://review.openstack.org/204815 | 17:25 |
| *** sergmelikyan has quit IRC | 17:30 | |
| *** sergmelikyan has joined #puppet-openstack | 17:31 | |
| *** bklei has quit IRC | 17:37 | |
| *** rcallawa_ has joined #puppet-openstack | 17:39 | |
| *** rcallawa has quit IRC | 17:42 | |
| *** sergmelikyan has quit IRC | 17:46 | |
| *** sergmelikyan has joined #puppet-openstack | 17:55 | |
| *** angdraug has joined #puppet-openstack | 17:56 | |
| *** sergmelikyan has quit IRC | 18:02 | |
| EmilienM | crinkle, pabelanger: have you any update about puppet-openstack-integration work done lately? I'm interested to chat a bit | 18:08 |
| crinkle | i haven't done any work on it, i think i've seen a couple of patches from pabelanger though | 18:09 |
| 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:11 |
| *** rcallawa_ has quit IRC | 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 has joined #puppet-openstack | 18:12 | |
| *** Fin1te has joined #puppet-openstack | 18:13 | |
| pabelanger | Ya, have something local. Also working to get new nodes online in infra | 18:15 |
| 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:16 |
| 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:17 |
| 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:19 |
| *** ducttape_ has quit IRC | 18:19 | |
| *** Fin1te has quit IRC | 18:20 | |
| EmilienM | pabelanger: how do you provision the node with puppet modules? | 18:20 |
| EmilienM | pabelanger: why not using beaker? it's exactly what we already need and use | 18:21 |
| 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_ has 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:23 |
| 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:24 |
| 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:25 |
| pabelanger | I am unsure what serverspec are too, outside the scenario001.pp | 18:26 |
| 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:27 |
| *** sergmelikyan has joined #puppet-openstack | 18:28 | |
| 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:30 |
| 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:31 |
| crinkle | beaker makes sense for functional testing individual modules but i think it offers less to us for integration testing | 18:32 |
| 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:33 |
| 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:34 |
| 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:38 |
| openstackgerrit | Merged openstack/puppet-keystone: Add tag to package and service resources https://review.openstack.org/204598 | 18:39 |
| openstackgerrit | Merged openstack/puppet-heat: Add tag to package and service resources https://review.openstack.org/204497 | 18:40 |
| openstackgerrit | Merged openstack/puppet-trove: Add region and resource url related options https://review.openstack.org/200346 | 18:41 |
| openstackgerrit | Merged openstack/puppet-manila: Add support to configure GlusterFS drivers with Manila shares. https://review.openstack.org/200811 | 18:45 |
| *** leanderthal has quit IRC | 18:48 | |
| *** ducttape_ has quit IRC | 18:51 | |
| *** sergmelikyan has quit IRC | 18:54 | |
| *** bitblt has joined #puppet-openstack | 18:55 | |
| *** ducttape_ has joined #puppet-openstack | 18:56 | |
| *** bitblt has quit IRC | 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 | 18:58 |
| 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:07 |
| *** ducttape_ has quit IRC | 19:08 | |
| *** bklei has joined #puppet-openstack | 19:19 | |
| *** sergmelikyan has joined #puppet-openstack | 19:21 | |
| *** ducttape_ has joined #puppet-openstack | 19:22 | |
| *** gfidente has quit IRC | 19:29 | |
| *** sergmelikyan has quit IRC | 19:31 | |
| *** sergmelikyan has joined #puppet-openstack | 19:34 | |
| *** sergmelikyan has quit IRC | 19:38 | |
| *** Fin1te has joined #puppet-openstack | 19:42 | |
| *** sergmelikyan has joined #puppet-openstack | 19:47 | |
| *** sergmelikyan has quit IRC | 19:50 | |
| *** ducttape_ has quit IRC | 20:00 | |
| *** sergmelikyan has joined #puppet-openstack | 20:03 | |
| *** ducttape_ has joined #puppet-openstack | 20:04 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-ceilometer: Enable support for memcached_servers https://review.openstack.org/199835 | 20:10 |
| *** ducttape_ has quit IRC | 20:12 | |
| *** ducttape_ has joined #puppet-openstack | 20:14 | |
| *** ducttape_ has quit IRC | 20:18 | |
| *** sergmelikyan has quit IRC | 20:19 | |
| *** ducttape_ has joined #puppet-openstack | 20:19 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-nova: Update Nova README https://review.openstack.org/204262 | 20:22 |
| *** sergmelikyan has joined #puppet-openstack | 20:27 | |
| *** dprince has quit IRC | 20:32 | |
| *** richm has quit IRC | 20:33 | |
| *** sergmelikyan has quit IRC | 20:33 | |
| *** sergmelikyan has joined #puppet-openstack | 20:34 | |
| *** sergmelikyan has quit IRC | 20:35 | |
| *** sergmelikyan has joined #puppet-openstack | 20:36 | |
| openstackgerrit | Guilherme Balzana proposed openstack/puppet-swift: Add a blank space at begining of each filter https://review.openstack.org/205284 | 20:37 |
| *** prad has quit IRC | 20:37 | |
| *** ducttape_ has quit IRC | 20:37 | |
| *** richm has joined #puppet-openstack | 20:39 | |
| *** colby__ has quit IRC | 20:39 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Update Swift README https://review.openstack.org/205285 | 20:39 |
| *** ducttape_ has joined #puppet-openstack | 20:39 | |
| *** sergmelikyan has quit IRC | 20:40 | |
| *** dalgaaf has joined #puppet-openstack | 20:41 | |
| *** dalgaaf has left #puppet-openstack | 20:42 | |
| *** markvoelker has quit IRC | 20:42 | |
| *** sergmelikyan has joined #puppet-openstack | 20:44 | |
| openstackgerrit | Guilherme Balzana proposed openstack/puppet-swift: Add a blank space at begining of each filter https://review.openstack.org/205284 | 20:45 |
| *** sergmelikyan has quit IRC | 20:50 | |
| *** colby__ has joined #puppet-openstack | 21:03 | |
| *** sergmelikyan has joined #puppet-openstack | 21:04 | |
| *** sergmelikyan has quit IRC | 21:05 | |
| EmilienM | clayton: sounds good | 21:11 |
| EmilienM | crinkle: "assert state" == testing the services ? | 21:12 |
| *** mgarza_ has joined #puppet-openstack | 21:20 | |
| crinkle | EmilienM: yes | 21:26 |
| *** rcallawa_ has joined #puppet-openstack | 21:26 | |
| EmilienM | crinkle: so you want to use the puppet apply jobs to gate puppet-openstack-integration? | 21:27 |
| *** sergmelikyan has joined #puppet-openstack | 21:28 | |
| crinkle | EmilienM: no those are different | 21:28 |
| crinkle | EmilienM: we'd need a new job | 21:28 |
| *** sergmelikyan has quit IRC | 21:29 | |
| *** sergmelikyan has joined #puppet-openstack | 21:29 | |
| EmilienM | crinkle: with new scripts, etc? | 21:30 |
| *** rcallawa has quit IRC | 21:30 | |
| *** Fin1te has quit IRC | 21:30 | |
| openstackgerrit | Merged openstack/puppet-openstacklib: Add a proxy inifile provider https://review.openstack.org/202488 | 21:30 |
| *** rcallawa_ has quit IRC | 21:31 | |
| *** sergmelikyan has quit IRC | 21:31 | |
| crinkle | EmilienM: yes | 21:31 |
| EmilienM | crinkle: mhhh ok | 21:32 |
| *** sergmelikyan has joined #puppet-openstack | 21:33 | |
| *** jasondotstar has joined #puppet-openstack | 21:36 | |
| *** sergmelikyan has quit IRC | 21:37 | |
| openstackgerrit | Merged openstack/puppet-nova: Support allow_start and allow_end parameters https://review.openstack.org/135123 | 21:38 |
| *** sergmelikyan has joined #puppet-openstack | 21:41 | |
| *** ducttape_ has quit IRC | 21:41 | |
| *** sergmelikyan has quit IRC | 21:42 | |
| *** pleia2 has quit IRC | 21:42 | |
| *** sergmelikyan has joined #puppet-openstack | 21:43 | |
| *** sergmelikyan has quit IRC | 21:44 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-glance: Add swift_store_region parameter to glance::backend::swift https://review.openstack.org/198669 | 21:44 |
| *** sergmelikyan has joined #puppet-openstack | 21:45 | |
| *** sergmelikyan has quit IRC | 21:46 | |
| *** sergmelikyan has 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:46 |
| 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 has joined #puppet-openstack | 21:47 | |
| *** sergmelikyan has quit IRC | 21:47 | |
| *** sergmelikyan has joined #puppet-openstack | 21:47 | |
| *** sergmelikyan has quit IRC | 21:48 | |
| 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:51 |
| 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_ has 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:52 |
| crinkle | this is what pabelanger and i were thinking, possibly nibalizer has a different vision of it he could express? | 21:53 |
| 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:54 |
| 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 has joined #puppet-openstack | 21:55 | |
| 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:56 |
| 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:58 |
| 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 has joined #puppet-openstack | 21:59 | |
| EmilienM | crinkle: your work on zuul cloner, right | 21:59 |
| *** sergmelikyan has quit IRC | 21:59 | |
| crinkle | right | 21:59 |
| 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:00 |
| *** sergmelikyan has joined #puppet-openstack | 22:02 | |
| 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:04 |
| crinkle | EmilienM: I think pabelanger is working on something, let's maybe wait for his input? | 22:05 |
| EmilienM | oh sure | 22:06 |
| crinkle | in any case non voting would be better | 22:06 |
| EmilienM | crinkle: can you reply to us on https://review.openstack.org/#/c/199676/ please , | 22:07 |
| crinkle | EmilienM: done | 22:13 |
| EmilienM | crinkle: I'm more concerned about the bug I'm fixing: https://bugs.launchpad.net/puppet-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:13 |
| *** mgarza_ has quit IRC | 22:15 | |
| *** linkedinyou|2 has joined #puppet-openstack | 22:16 | |
| *** linkedinyou has quit IRC | 22:19 | |
| crinkle | EmilienM: I don't understand the bug report, admin_domain isn't mentioned in keystone::roles::admin | 22:23 |
| 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:26 |
| *** timfreund has joined #puppet-openstack | 22:28 | |
| 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:30 |
| *** ducttape_ has quit IRC | 22:32 | |
| *** ducttape_ has joined #puppet-openstack | 22:36 | |
| openstackgerrit | Emilien Macchi proposed stackforge/puppet-ceph: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/197332 | 22:38 |
| *** jasondotstar has quit IRC | 22:40 | |
| *** morazi has quit IRC | 22:43 | |
| *** linkedinyou|2 has quit IRC | 22:47 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Enable support for memcached_servers https://review.openstack.org/199835 | 22:51 |
| *** linkedinyou has joined #puppet-openstack | 22:53 | |
| imcsk8 | richm: are you around? | 22:57 |
| 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 | 22:58 |
| *** ducttape_ has quit IRC | 23:03 | |
| *** sergmelikyan has quit IRC | 23:15 | |
| *** sergmelikyan has joined #puppet-openstack | 23:17 | |
| *** sergmelikyan has quit IRC | 23:18 | |
| *** serg_melikyan has joined #puppet-openstack | 23:18 | |
| *** serg_melikyan has quit IRC | 23:23 | |
| *** ducttape_ has joined #puppet-openstack | 23:23 | |
| *** rhefner has quit IRC | 23:40 | |
| *** sergmelikyan has joined #puppet-openstack | 23:44 | |
| *** jasondotstar has joined #puppet-openstack | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!