| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: DNM - CI test https://review.openstack.org/216907 | 00:06 |
|---|---|---|
| *** sergmelikyan has quit IRC | 00:06 | |
| *** ducttape_ has joined #puppet-openstack | 00:08 | |
| *** asd112z has joined #puppet-openstack | 00:14 | |
| *** xingchao_ has quit IRC | 00:15 | |
| *** xingchao_ has joined #puppet-openstack | 00:15 | |
| *** xingchao_ has quit IRC | 00:16 | |
| *** TC01 has quit IRC | 00:18 | |
| *** gildub_ has joined #puppet-openstack | 00:22 | |
| *** TC01 has joined #puppet-openstack | 00:22 | |
| *** gildub has quit IRC | 00:25 | |
| *** openstack has joined #puppet-openstack | 00:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 00:38 |
| *** sergmelikyan has joined #puppet-openstack | 00:38 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 00:39 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 00:39 |
| *** Zhang has joined #puppet-openstack | 00:42 | |
| *** xarses has quit IRC | 00:43 | |
| *** sergmelikyan has quit IRC | 00:43 | |
| *** aimon has joined #puppet-openstack | 00:54 | |
| *** sergmelikyan has joined #puppet-openstack | 01:00 | |
| *** sergmelikyan has quit IRC | 01:03 | |
| *** sergmelikyan has joined #puppet-openstack | 01:08 | |
| *** sergmelikyan has quit IRC | 01:10 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 01:10 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 01:11 |
| *** xarses has joined #puppet-openstack | 01:17 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 01:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 01:24 |
| *** gildub_ has quit IRC | 01:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 01:37 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 01:37 |
| *** gildub_ has joined #puppet-openstack | 01:43 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 01:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 01:47 |
| michchap | xarses: myatsenko thanks, looking. My main question is whether the host setting is relevant for other backends beyond RBD. I understand it's needed for RBD, but if it's also needed for other backends putting it in rbd.pp is not right. | 01:47 |
| EmilienM | michchap++ | 01:48 |
| EmilienM | michchap: I was waiting you wake up before reviewing | 01:48 |
| michchap | EmilienM: I didn't get time to review yesterday | 01:48 |
| *** xarses has quit IRC | 01:48 | |
| *** gildub_ has quit IRC | 01:49 | |
| *** xarses has joined #puppet-openstack | 01:50 | |
| *** xarses has quit IRC | 01:51 | |
| *** xarses has joined #puppet-openstack | 01:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 02:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 02:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 02:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 02:03 |
| *** xarses has quit IRC | 02:03 | |
| *** gildub_ has joined #puppet-openstack | 02:06 | |
| *** gildub_ has quit IRC | 02:10 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: tempest changes https://review.openstack.org/216844 | 02:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 02:17 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ https://review.openstack.org/216828 | 02:17 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 02:18 |
| *** xarses has joined #puppet-openstack | 02:20 | |
| *** aimon_ has joined #puppet-openstack | 02:23 | |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-heat: Add hooks for external install & svc management https://review.openstack.org/216926 | 02:25 |
| *** aimon has quit IRC | 02:26 | |
| EmilienM | clayton: do you have manifests to run ^ with docker? | 02:29 |
| *** richm has quit IRC | 02:29 | |
| clayton | yes, working on finalizing that now | 02:29 |
| clayton | probably have it done tomorrow | 02:29 |
| EmilienM | clayton: cool - we could create testing jobs for that too | 02:29 |
| EmilienM | nice | 02:29 |
| clayton | EmilienM: we're going to be putting all of our puppet-openstack docker integration work here - https://github.com/twc-openstack/puppet-os_docker | 02:33 |
| clayton | right now that's just a rework of the existing designate work we've already done (which went to prod today) | 02:33 |
| clayton | the heat support will probably get merged tomorrow or day after | 02:33 |
| EmilienM | clayton: can you install multiple designate-api containers on the same host, with different configurations? | 02:34 |
| clayton | not with our os_docker module, no. We don't have a use case for that. | 02:35 |
| EmilienM | ok | 02:35 |
| EmilienM | that's the only limitation I see now in your code | 02:35 |
| clayton | well, I think it's a limitation in the existing config providers also | 02:35 |
| EmilienM | dprince is also working on tripleo/puppet/docker | 02:35 |
| EmilienM | and he did not have to patch anything in puppet | 02:35 |
| EmilienM | his approach is different | 02:35 |
| EmilienM | a sec | 02:35 |
| EmilienM | clayton: https://review.openstack.org/208774 | 02:36 |
| EmilienM | clayton: commit message explains his approach | 02:37 |
| clayton | so he's using puppet to generate the config files to go into the containers? | 02:37 |
| EmilienM | yes | 02:37 |
| EmilienM | and iiuc it does support multiple config files on the same host | 02:37 |
| clayton | nod. probably makes sense for tripleo, but I don't think we'd want to do that | 02:37 |
| EmilienM | but I'm not sure about the last statement | 02:37 |
| EmilienM | yeah, just wanted to highlight another use case | 02:38 |
| clayton | nod, he's basically only using puppet for the config file generation in that case it seems | 02:38 |
| EmilienM | yes | 02:38 |
| clayton | if we were going to go down that path, I'd probably use something like consul template and not use the puppet openstack modules at all | 02:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 02:39 |
| EmilienM | clayton: I tend to prefer your approach | 02:39 |
| clayton | but our use case is different. We don't really need super flexible config files, we need the same config file in all environments + some environment specific tweaks | 02:39 |
| EmilienM | clayton: so you're hardcoding config file? | 02:41 |
| EmilienM | clayton: why to use puppet then? just for orchestration? | 02:41 |
| clayton | in os_docker, or in the hypothetical approach above? | 02:41 |
| EmilienM | clayton: in os_docker | 02:41 |
| clayton | the config files can be overridden, they're there because you need to have something to start with, normall packages provide them | 02:42 |
| clayton | in fact, we do override the default ones for designate | 02:42 |
| clayton | os_docker::designate takes a config_files hash that allows you to provide new sources for them, or new behavior | 02:43 |
| EmilienM | oh I see | 02:43 |
| EmilienM | clayton: I think we might need to update our inifile providers to change the file path | 02:44 |
| EmilienM | so we can create multiple file and mount them into containers | 02:44 |
| *** gildub has joined #puppet-openstack | 02:51 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: docfix: update default image_service param in doc https://review.openstack.org/216935 | 03:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 03:16 |
| *** v1k0d3n has quit IRC | 03:19 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 03:20 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 03:20 |
| *** ducttap__ has joined #puppet-openstack | 03:22 | |
| *** ducttape_ has quit IRC | 03:23 | |
| *** gildub has quit IRC | 03:38 | |
| *** ducttap__ has quit IRC | 03:47 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_endpoint https://review.openstack.org/216488 | 03:48 |
| *** gildub has joined #puppet-openstack | 03:55 | |
| *** markir has joined #puppet-openstack | 03:56 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_endpoint https://review.openstack.org/216488 | 03:59 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_endpoint https://review.openstack.org/216488 | 04:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Make sure Nova_admin_tenant_id_setter is executed after Keystone_endpoint https://review.openstack.org/216950 | 04:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 04:03 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 04:05 |
| *** chandankumar has joined #puppet-openstack | 04:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: Make sure Glance_image is executed after Keystone_endpoint https://review.openstack.org/216488 | 04:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Make sure Nova_admin_tenant_id_setter is executed after Keystone_endpoint https://review.openstack.org/216950 | 04:22 |
| *** xingchao_ has joined #puppet-openstack | 04:30 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 04:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 04:38 |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints can be APIs version-less https://review.openstack.org/213957 | 04:48 |
| *** gildub has quit IRC | 04:49 | |
| *** gildub has joined #puppet-openstack | 04:49 | |
| *** chandankumar has quit IRC | 04:54 | |
| *** xingchao_ has quit IRC | 05:01 | |
| *** sergmelikyan has joined #puppet-openstack | 05:07 | |
| *** sergmelikyan has quit IRC | 05:09 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API https://review.openstack.org/200996 | 05:13 |
| *** v1k0d3n has joined #puppet-openstack | 05:20 | |
| *** v1k0d3n has quit IRC | 05:25 | |
| *** markir has left #puppet-openstack | 05:28 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: Endpoints can be APIs version-less https://review.openstack.org/213957 | 05:31 |
| *** saneax has joined #puppet-openstack | 05:31 | |
| *** asd112z has quit IRC | 05:35 | |
| xarses | michchap: probably, but it depends on each backend if it supports having multiple cinder-volume agents acting at the same time, on each volume might be worked on at the same time. | 05:55 |
| xarses | michchap: I asked one of the guys about the EMC driver and they didn't active/active but active/backup (but still wanted to use host | 05:56 |
| michchap | xarses: so that would be a 'no' for LVM, but a 'maybe' for most of the others, right? | 05:56 |
| xarses | michchap: yes | 05:56 |
| xarses | michchap: we would need driver SME's to pipe up about it | 05:57 |
| *** jtomasek has joined #puppet-openstack | 06:08 | |
| michchap | xarses: so we could just put it in rbd anyway, and the result is that each driver will need to add that parameter if they need it, or we can put it in init.pp and expect composition layers / users to know that they need to configure that option (which imo is very obscure) | 06:15 |
| xarses | michchap: thats about my take, in the case with rbd, it turns on whenever the pool is duplicate | 06:17 |
| michchap | xarses: (sorry I got pulled away briefly) | 06:17 |
| xarses | so it might be best to push it in the driver and let it determine | 06:17 |
| michchap | is there any other situation where that param is needed? | 06:18 |
| xarses | we will want to pull some of the common drivers maintainers to see if they can comment about theirs | 06:18 |
| xarses | not that I'm aware of | 06:18 |
| xarses | its used exclusivly to identify the providers unique name, and it defaults to `hostname` | 06:19 |
| michchap | final question: does the naming scheme need to be configurable? | 06:19 |
| xarses | we haven't seen a need for it in fuel, as it's here we set it to the pool name | 06:20 |
| michchap | I guess that would be trivially patchable if someone saw the need for it | 06:20 |
| xarses | I might see someone wanting control over it but it seems weird, and as you note too, its easy to patch | 06:21 |
| michchap | Thanks for the explanation and sorry to take so much time. | 06:21 |
| xarses | its fine | 06:21 |
| xarses | I'm happy to help | 06:21 |
| xarses | enjoy your day | 06:21 |
| * xarses is off to bed | 06:21 | |
| michchap | night! | 06:22 |
| *** chandankumar has joined #puppet-openstack | 06:30 | |
| *** simon-AS559 has joined #puppet-openstack | 06:31 | |
| myatsenko | michchap: hello, this should be used only for rbd backend to solve the problems described in https://bugs.launchpad.net/mos/+bug/1371328, https://bugs.launchpad.net/cinder/+bug/1280367 | 06:33 |
| openstack | Launchpad bug 1371328 in Mirantis OpenStack 5.0.x "On controller failure, some RBD backed Cinder volumes may become unmanageable" [Medium,In progress] - Assigned to Dmitry Borodaenko (dborodaenko) | 06:33 |
| openstack | Launchpad bug 1280367 in Cinder "cinder backend by ceph HA setup" [Undecided,Fix released] | 06:33 |
| michchap | myatsenko: thanks. I left a +2 :) | 06:40 |
| *** Zhang has quit IRC | 06:40 | |
| myatsenko | michchap: thanks for +2 :) | 06:45 |
| *** ducttape_ has joined #puppet-openstack | 06:52 | |
| *** gildub has quit IRC | 06:52 | |
| *** ducttape_ has quit IRC | 07:01 | |
| *** Zhang has joined #puppet-openstack | 07:09 | |
| *** jpena has joined #puppet-openstack | 07:15 | |
| *** Zhang has quit IRC | 07:16 | |
| *** v1k0d3n has joined #puppet-openstack | 07:21 | |
| *** stamak has joined #puppet-openstack | 07:21 | |
| *** sac has quit IRC | 07:23 | |
| *** sac has joined #puppet-openstack | 07:23 | |
| *** Zhang has joined #puppet-openstack | 07:24 | |
| *** v1k0d3n has quit IRC | 07:26 | |
| *** chem has quit IRC | 07:53 | |
| *** chem has joined #puppet-openstack | 07:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/puppet-keystone: add --domain option when calling user list https://review.openstack.org/213906 | 07:54 |
| openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 07:55 |
| *** stamak has quit IRC | 07:56 | |
| openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 07:57 |
| *** rochaporto has joined #puppet-openstack | 07:59 | |
| *** dgurtner has joined #puppet-openstack | 07:59 | |
| *** dgurtner has quit IRC | 07:59 | |
| *** dgurtner has joined #puppet-openstack | 07:59 | |
| *** rochaporto has quit IRC | 08:03 | |
| *** rochaporto has joined #puppet-openstack | 08:04 | |
| *** rochaporto has quit IRC | 08:06 | |
| *** rochaporto has joined #puppet-openstack | 08:07 | |
| *** derekh has joined #puppet-openstack | 08:09 | |
| *** chem has quit IRC | 08:33 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Update dependencies and spec_helpers https://review.openstack.org/217033 | 08:33 |
| *** chem has joined #puppet-openstack | 08:33 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation https://review.openstack.org/211043 | 08:38 |
| *** chem has quit IRC | 08:38 | |
| *** jistr has joined #puppet-openstack | 08:39 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Murano puppet module implementation https://review.openstack.org/211043 | 08:41 |
| *** Zhang has quit IRC | 08:43 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add api and engine class implementation https://review.openstack.org/216682 | 08:43 |
| *** stamak has joined #puppet-openstack | 08:44 | |
| *** LimorStotland has quit IRC | 09:03 | |
| openstackgerrit | Javier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service https://review.openstack.org/211549 | 09:09 |
| *** Zhang has joined #puppet-openstack | 09:12 | |
| *** Zhang has quit IRC | 09:14 | |
| *** Zhang has joined #puppet-openstack | 09:19 | |
| *** Zhang has quit IRC | 09:22 | |
| *** v1k0d3n has joined #puppet-openstack | 09:22 | |
| *** v1k0d3n has quit IRC | 09:26 | |
| *** iurygregory has quit IRC | 09:28 | |
| *** linkedinyou has quit IRC | 09:28 | |
| *** LimorStotland has joined #puppet-openstack | 09:28 | |
| *** iurygregory has joined #puppet-openstack | 09:28 | |
| *** chem has joined #puppet-openstack | 09:32 | |
| *** _matthias__ is now known as _matthias_ | 09:36 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Module skeleton update https://review.openstack.org/217033 | 09:37 |
| *** LimorStotland has quit IRC | 09:47 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 09:55 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 10:02 |
| *** paramite has joined #puppet-openstack | 10:14 | |
| *** Zhang has joined #puppet-openstack | 10:16 | |
| *** LimorStotland has joined #puppet-openstack | 10:18 | |
| *** vmbrasseur has quit IRC | 10:36 | |
| *** vmbrasseur has joined #puppet-openstack | 10:37 | |
| *** cgfbees has joined #puppet-openstack | 10:41 | |
| *** cgfbees has quit IRC | 10:51 | |
| *** LimorStotland has quit IRC | 10:56 | |
| *** cgfbee has quit IRC | 11:09 | |
| *** paramite has quit IRC | 11:12 | |
| *** Vinsh has quit IRC | 11:12 | |
| *** cgfbee has joined #puppet-openstack | 11:15 | |
| *** v1k0d3n has joined #puppet-openstack | 11:23 | |
| *** dprince has joined #puppet-openstack | 11:24 | |
| *** paramite has joined #puppet-openstack | 11:24 | |
| *** saneax has quit IRC | 11:25 | |
| *** v1k0d3n has quit IRC | 11:27 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 11:36 |
| *** Zhang has quit IRC | 11:42 | |
| *** cfloare has joined #puppet-openstack | 11:47 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 11:49 |
| *** LimorStotland has joined #puppet-openstack | 11:52 | |
| *** paramite has quit IRC | 11:56 | |
| *** paramite has joined #puppet-openstack | 12:00 | |
| *** saneax has joined #puppet-openstack | 12:04 | |
| *** Zhang has joined #puppet-openstack | 12:08 | |
| EmilienM | good morning folks | 12:13 |
| *** _skolekonov is now known as skolekonov | 12:14 | |
| skolekonov | morning | 12:14 |
| *** ferest has joined #puppet-openstack | 12:14 | |
| *** ducttape_ has joined #puppet-openstack | 12:15 | |
| *** ferest has quit IRC | 12:19 | |
| *** ducttape_ has quit IRC | 12:26 | |
| *** sergmelikyan has joined #puppet-openstack | 12:28 | |
| *** LimorStotland has quit IRC | 12:30 | |
| *** LimorStotland has joined #puppet-openstack | 12:30 | |
| *** jayg|g0n3 is now known as jayg | 12:30 | |
| *** jpena is now known as jpena|lunch | 12:35 | |
| *** pradk has joined #puppet-openstack | 12:37 | |
| *** Vinsh has joined #puppet-openstack | 12:40 | |
| *** Vinsh has quit IRC | 12:41 | |
| *** Vinsh has joined #puppet-openstack | 12:41 | |
| *** paramite is now known as paramite|afk | 12:41 | |
| *** v1k0d3n has joined #puppet-openstack | 12:47 | |
| degorenko | morning | 12:51 |
| *** Vinsh has quit IRC | 12:52 | |
| iurygregory | morning =) | 12:53 |
| *** Vinsh has joined #puppet-openstack | 12:53 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Initial module implementation https://review.openstack.org/211043 | 13:00 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Implementaion of api and engine classes https://review.openstack.org/216682 | 13:02 |
| *** rochaporto has quit IRC | 13:03 | |
| *** paramite|afk is now known as paramite | 13:04 | |
| *** richm has joined #puppet-openstack | 13:04 | |
| *** Zhang has quit IRC | 13:04 | |
| *** ferest has joined #puppet-openstack | 13:12 | |
| *** richm has quit IRC | 13:13 | |
| *** ferest has quit IRC | 13:16 | |
| *** paramite is now known as paramite|afk | 13:24 | |
| *** Zhang has joined #puppet-openstack | 13:24 | |
| EmilienM | chem: I'm reviewing https://review.openstack.org/#/c/202689 | 13:26 |
| chem | EmilienM: oki | 13:26 |
| EmilienM | chem: you say you needed to use API because some versions of osclient does not have remote-id | 13:26 |
| chem | yep | 13:26 |
| EmilienM | chem: in liberty, do ubuntu & centos7 have the right version? | 13:26 |
| chem | yep | 13:27 |
| chem | (not tested, but should be) | 13:27 |
| openstackgerrit | Clayton O'Neill proposed openstack/puppet-heat: Add hooks for external install & svc management https://review.openstack.org/216926 | 13:27 |
| EmilienM | chem: because we try to be consistent and stop using API for using CLI (osclient) | 13:27 |
| chem | EmilienM: yes, I know, we went through this before my vacation | 13:28 |
| *** paramite|afk is now known as paramite | 13:28 | |
| chem | EmilienM: in previous patchset there are version which doesn't use the API | 13:28 |
| chem | EmilienM: only the cli | 13:29 |
| EmilienM | I had this discussion with crinkle yesterday - using CLI was our choice because we thought it's the way to go for longterm, so we don't manage API changes | 13:29 |
| EmilienM | your patch is really good, but I worry about future and the API changes in identity provider api | 13:30 |
| EmilienM | I would wait for richm & crinkle to have also their feedback | 13:31 |
| EmilienM | but I think we should go for CLI only | 13:31 |
| EmilienM | even if we don't support Kilo | 13:31 |
| degorenko | EmilienM, hi! I want ask you about https://review.openstack.org/192721 . I want to add new run mode for Sahara, which uses not only combined API&Engine (as it right now), but also divide this daemon on two separate (api and engine) | 13:34 |
| degorenko | and what way is more appropriate: 1) deprecate old configuring in init.pp and raise warning to use proper way(for example, use ::sahara::service::what_ever_you_need) | 13:35 |
| degorenko | 2) raise deprecate warning, but if manage_service = true use old way? and if false, do nothing | 13:36 |
| EmilienM | degorenko: you need to deprecate whas is in stable/kilo | 13:36 |
| *** jpena|lunch is now known as jpena | 13:36 | |
| degorenko | yes, i know it | 13:36 |
| degorenko | but it can be done by two ways | 13:37 |
| EmilienM | so what is in master is not supposed to be stable | 13:37 |
| degorenko | both ways of run mode are in master | 13:37 |
| EmilienM | ok | 13:37 |
| chem | EmilienM: richm +1 the patch and the switch to the API was done under his supervision. | 13:37 |
| degorenko | so, we keep possibility to run both ways | 13:37 |
| degorenko | but i want move out it from init.pp | 13:37 |
| EmilienM | chem: we will ask him again to reconsider that option - again, this is for a good reason we want CLI - OpenStack APIs are changing every day, it's really painful to maintain providers | 13:38 |
| EmilienM | degorenko: 1 seems cool | 13:38 |
| degorenko | EmilienM, ok, thanks | 13:39 |
| EmilienM | degorenko: wait, I need to see code before :) | 13:40 |
| EmilienM | but I think it's reasonable | 13:40 |
| degorenko | EmilienM, my patch is outdated now, i'm just want to know how it should be update | 13:40 |
| degorenko | so, if we choose 1) i'll remove service from init.pp, but keep old parameters and mark them as deprecated, also i'll create new manifests service::{api|all|engine}.pp and they should be called separately from init.pp | 13:42 |
| degorenko | is it ok? | 13:42 |
| EmilienM | chem: please look http://www.slideshare.net/Colleen_Murphy/vancouver-presentation-48498077 | 13:42 |
| chem | EmilienM: you already sent me the link weeks ago and I red it and we discussed about it already | 13:43 |
| openstackgerrit | Merged openstack/puppet-cinder: Add 'host' parameter to cinder.conf https://review.openstack.org/209412 | 13:43 |
| *** dfisher has joined #puppet-openstack | 13:44 | |
| *** ducttape_ has joined #puppet-openstack | 13:57 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 14:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 14:02 |
| *** sergmelikyan has quit IRC | 14:12 | |
| *** sergmelikyan has joined #puppet-openstack | 14:16 | |
| *** linkedinyou has joined #puppet-openstack | 14:24 | |
| *** zacdev has joined #puppet-openstack | 14:25 | |
| *** Zhang has quit IRC | 14:37 | |
| *** richm1 has joined #puppet-openstack | 14:37 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: Use zuul-cloner for tempest https://review.openstack.org/217242 | 14:41 |
| *** Zhang has joined #puppet-openstack | 14:46 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-openstack-integration: Use zuul-cloner for tempest https://review.openstack.org/217242 | 14:46 |
| *** Zhang has quit IRC | 14:47 | |
| *** Zhang has joined #puppet-openstack | 14:48 | |
| *** Zhang has quit IRC | 14:49 | |
| *** Zhang has joined #puppet-openstack | 14:49 | |
| *** Zhang has quit IRC | 14:57 | |
| *** topshare has joined #puppet-openstack | 14:58 | |
| *** topshare has quit IRC | 14:58 | |
| *** topshare has joined #puppet-openstack | 14:58 | |
| *** paramite is now known as paramite|afk | 15:00 | |
| *** topshare has quit IRC | 15:01 | |
| *** LimorStotland has quit IRC | 15:01 | |
| *** topshare has joined #puppet-openstack | 15:01 | |
| *** paramite|afk is now known as paramite | 15:01 | |
| *** asd112z has joined #puppet-openstack | 15:02 | |
| *** topshare has quit IRC | 15:02 | |
| *** topshare has joined #puppet-openstack | 15:03 | |
| *** topshare has quit IRC | 15:03 | |
| *** topshare has joined #puppet-openstack | 15:04 | |
| EmilienM | richm1: I looked https://review.openstack.org/#/c/202689 - don't you think using CLI would be more consistent and safe for future maintenance of this code? | 15:04 |
| *** topshare has quit IRC | 15:06 | |
| *** Zhang has joined #puppet-openstack | 15:07 | |
| crinkle | I would prefer to stick with the CLI, making http calls directly makes it inconsistent and introduces complexity especially surrounding https and ipv6 | 15:09 |
| crinkle | the openstackclient people are pretty responsive to feature requests | 15:09 |
| richm1 | the osc people are responsive, yes - but unfortunately we are gated by rdo/uca packaging :P | 15:10 |
| *** richm1 is now known as richm | 15:10 | |
| *** asd112z has quit IRC | 15:10 | |
| crinkle | we're on the liberty staging packages so in theory we can get that a little faster | 15:11 |
| crinkle | as long as we don't need to backport | 15:11 |
| *** asd112z has joined #puppet-openstack | 15:11 | |
| richm | I really dislike having to struggle with rdo/uca every time we run into a missing feature of osc | 15:11 |
| *** paramite is now known as paramite|afk | 15:11 | |
| EmilienM | +1 with crinkle | 15:12 |
| richm | . . . but I suppose that is the price of choosing osc over using the REST API directly . . . | 15:12 |
| EmilienM | we have the right version in liberty to use CLI | 15:12 |
| crinkle | I'm just worried using net::http in random places will introduce weird bugs for people with different environments, where openstackclient has already solved those types of problems | 15:12 |
| EmilienM | not in Kilo ok, but we already released Kilo and iirc, nobody asked for service provider in kilo | 15:12 |
| *** jistr is now known as jistr|mtg | 15:12 | |
| richm | EmilienM: in both the current uca and rdo that we are using in the gate? | 15:12 |
| EmilienM | richm: afik, yes, we have 1.5 | 15:13 |
| *** paramite|afk is now known as paramite | 15:13 | |
| *** Zhang has quit IRC | 15:14 | |
| openstackgerrit | Ramkumar Gowrishankar proposed openstack/puppet-neutron: Support for Nuage Neutron plugin and Nuage VRS in puppet-neutron https://review.openstack.org/214798 | 15:15 |
| *** paramite has quit IRC | 15:15 | |
| *** zhangjn has joined #puppet-openstack | 15:15 | |
| *** zhangjn has quit IRC | 15:16 | |
| *** zhangjn has joined #puppet-openstack | 15:16 | |
| *** zhangjn has quit IRC | 15:17 | |
| pabelanger | so, looking at our gate jobs now. Each puppet module runs 14 check jobs. IMO, I think we can maybe reduce some of it. | 15:18 |
| pabelanger | For example, why don't we combined our puppet-unit gate into a single node, and iterate over each version of puppet we want to support | 15:18 |
| *** zhangjinnan has joined #puppet-openstack | 15:19 | |
| pabelanger | we currently launch 7 nodes today, but I think we can safely sandbox the required gems as needed | 15:19 |
| *** zhangjinnan has quit IRC | 15:19 | |
| pabelanger | into a single node job | 15:19 |
| clayton | makes sense and the other tests take so long to run that it shouldn't be any slower than the others | 15:19 |
| *** zhangjn has joined #puppet-openstack | 15:20 | |
| pabelanger | Ya, I'm more thinking of the nodepool burden right now. 7 nodes, per modules, to download a gem into a local file system, then run ruby code, seems a little expensive | 15:21 |
| *** zhangjn has quit IRC | 15:22 | |
| *** zhangjn has joined #puppet-openstack | 15:22 | |
| *** zhangjn has quit IRC | 15:22 | |
| *** jistr|mtg is now known as jistr | 15:22 | |
| EmilienM | it sounds also good to me, if we provide easy ways to debug failure, like separate (by puppet version) logs | 15:22 |
| EmilienM | also, I think we should keep non-voting the latest version of puppet | 15:23 |
| *** zhangjn has joined #puppet-openstack | 15:23 | |
| pabelanger | that is fine, 7 down to 2 is still worth it | 15:23 |
| EmilienM | pabelanger: I like the idea | 15:24 |
| EmilienM | _ody, Hunner ^ | 15:24 |
| *** zhangjn has quit IRC | 15:25 | |
| *** zhangjn has joined #puppet-openstack | 15:26 | |
| *** zhangjn has quit IRC | 15:26 | |
| *** zhangjn has joined #puppet-openstack | 15:26 | |
| *** zhangjn has quit IRC | 15:26 | |
| *** zhangjn has joined #puppet-openstack | 15:27 | |
| *** zhangjn has quit IRC | 15:27 | |
| *** ferest has joined #puppet-openstack | 15:28 | |
| *** zhangjn has joined #puppet-openstack | 15:28 | |
| *** zhangjn has quit IRC | 15:28 | |
| *** ferest has quit IRC | 15:29 | |
| *** zhangjn has joined #puppet-openstack | 15:29 | |
| *** ferest has joined #puppet-openstack | 15:29 | |
| EmilienM | richm, chem: can we clarify the situation for service provider? | 15:32 |
| chem | EmilienM: I have to revert my code to patchset 21 | 15:33 |
| EmilienM | richm, chem: don't you think openstackclient is the best option for us in longterm? | 15:33 |
| chem | EmilienM: by cherry picking | 15:33 |
| EmilienM | chem: cool, but I just want to make sure we are all on the same page now | 15:33 |
| richm | EmilienM: yes, provided we can get the packaging repos to help us | 15:33 |
| *** ferest has quit IRC | 15:33 | |
| *** zhangjn has quit IRC | 15:33 | |
| EmilienM | chem: ^ ok so here is the thing: let's revert to CLI for now (and I'm sorry again if I missed that) - for future keep that rule in mind: use CLI when we can. If it's to implement a new feature that is for a previous release and can't work with previous packages but only with newer (like now) : use CLI. If CLI is not supporting what we want 1/ report immediately the thing to OSclient team 2/ evaluate when we have the feature in OSclient and acc | 15:37 |
| EmilienM | ordingly wait or use HTTP API instead of CLI | 15:37 |
| chem | EmilienM: we went to the API road because of the gate, but now the gate has changed to liberty so everything is fine :) | 15:39 |
| *** chandankumar has quit IRC | 15:40 | |
| richm | chem: yeah, sorry about jerking you around like that :P | 15:40 |
| chem | richm: no problem :) | 15:41 |
| EmilienM | richm: it's also my fault, apologize | 15:42 |
| EmilienM | chem: so far your patch is good and I like your tests too | 15:42 |
| chem | richm: I was working on keystone_config and see you comment about the API domain configuration. I assume that you mean that we have to check that openstack cli >1.5 has the necessary verbs to manage the domain configuration. | 15:42 |
| chem | richm: as far as the direct API is concerned, it's working in kilo, but not available in the cli. | 15:43 |
| chem | richm: I'm spawning liberty to check the api | 15:43 |
| chem | the *cli* sorry | 15:43 |
| *** stamak has quit IRC | 15:47 | |
| *** simon-AS559 has quit IRC | 16:01 | |
| *** jistr has quit IRC | 16:12 | |
| *** simon-AS559 has joined #puppet-openstack | 16:25 | |
| *** jpena is now known as jpena|away | 16:31 | |
| *** simon-AS559 has quit IRC | 16:42 | |
| *** derekh has quit IRC | 16:46 | |
| pabelanger | EmilienM: you can rebase a top of 217242. should work as expected | 16:56 |
| EmilienM | pabelanger: ack thx | 16:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy & test glance https://review.openstack.org/216418 | 17:22 |
| EmilienM | pabelanger: done^ | 17:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy RabbitMQ https://review.openstack.org/216828 | 17:22 |
| *** xarses has quit IRC | 17:23 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 17:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Nova https://review.openstack.org/216938 | 17:24 |
| *** aimon_ has quit IRC | 17:35 | |
| EmilienM | crinkle: puppet-neutron gate is still broken - rdo folks are fixing that atm | 17:38 |
| EmilienM | but it's basically a dependency issue | 17:39 |
| EmilienM | where they updated alembic, and then it caused a lot of dependency updates requirement | 17:39 |
| EmilienM | anyway... should be fixed soon | 17:39 |
| crinkle | woot | 17:41 |
| EmilienM | it's really painful | 17:41 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy Neutron https://review.openstack.org/216831 | 17:57 |
| *** zacdev has quit IRC | 18:00 | |
| *** linkedinyou has quit IRC | 18:00 | |
| *** ducttape_ has quit IRC | 18:03 | |
| *** ducttape_ has joined #puppet-openstack | 18:13 | |
| *** aimon has joined #puppet-openstack | 18:14 | |
| *** openstackgerrit has quit IRC | 18:17 | |
| *** openstackgerrit has joined #puppet-openstack | 18:17 | |
| EmilienM | crinkle: neutron is back | 18:20 |
| EmilienM | but for a while, they are still updating deps so it might break again, RECHECK RECHECK ! | 18:21 |
| pabelanger | EmilienM: looks to be successful | 18:22 |
| EmilienM | yeah :) | 18:22 |
| EmilienM | https://jenkins07.openstack.org/job/gate-puppet-openstack-integration-dsvm-centos7/37/consoleFull | 18:22 |
| EmilienM | pabelanger: I've got nova, neutron, glance, keystone | 18:22 |
| EmilienM | now, let's run tempest | 18:22 |
| nibalizer | EmilienM: dprince so you want a revert on puppetlabs mongo then? | 18:28 |
| EmilienM | might be an option | 18:28 |
| nibalizer | can you propose the PR? | 18:28 |
| EmilienM | not the best one probably | 18:28 |
| EmilienM | dprince ^ | 18:29 |
| nibalizer | Hunner: thoughts? | 18:30 |
| nibalizer | dprince: EmilienM can you run your tests with a reverted version and see if it resolves your issue? | 18:30 |
| Hunner | If we can't move forward, then backward is the next best option | 18:31 |
| Hunner | I haven't looked at the actual issue yet though | 18:31 |
| EmilienM | nibalizer: I'm not the person who is really affected - tripleo team is | 18:31 |
| EmilienM | but I think we should carefully test patches without tests | 18:31 |
| dprince | @EmilienM: yes | 18:31 |
| dprince | nibalizer: https://github.com/puppetlabs/puppetlabs-mongodb/pull/228 | 18:33 |
| Hunner | totes valid; merged | 18:38 |
| EmilienM | thx Hunner | 18:39 |
| nibalizer | dprince: how could we get a feedback loop set up that tripleo tests can feed information back to mongodb prs? | 18:39 |
| Hunner | ugh, so many mongodb pcci failures | 18:40 |
| dprince | nibalizer: sounds like a new CI system | 18:40 |
| nibalizer | Hunner: yea | 18:41 |
| dprince | nibalizer: we are fairly wired into infra right now so the github workflow would require some work | 18:41 |
| nibalizer | we can make pcci give the red dot on all mongo prs just to terrify them about refactors maybe? | 18:41 |
| Hunner | nibalizer: Do you still need 217? | 18:41 |
| nibalizer | Hunner: I'm not sure what that is | 18:42 |
| nibalizer | oh the PR? | 18:42 |
| Hunner | nibalizer: sorry, I mean https://github.com/puppetlabs/puppetlabs-mongodb/commit/934516e46b2b0c2029076294304f1e7faef7e283 | 18:43 |
| Hunner | yeah | 18:43 |
| nibalizer | nah I'm okay with the revert and hopefully iberezovskiy can come up with a way to add that such that mongo doesn't explode | 18:43 |
| *** asd112z_ has joined #puppet-openstack | 18:52 | |
| *** asd112z has quit IRC | 18:56 | |
| *** dgurtner has quit IRC | 18:58 | |
| rgowrishankar | Emilien: When you get the chance can you look at https://review.openstack.org/#/c/214798/ I fixed the whitespace issues also | 18:59 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 19:00 |
| *** angdraug has joined #puppet-openstack | 19:00 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 19:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 19:04 |
| *** RichardRaseley has joined #puppet-openstack | 19:11 | |
| *** RichardRaseley has quit IRC | 19:14 | |
| *** saneax has quit IRC | 19:29 | |
| mfisch | dmsimard: you around? | 19:32 |
| *** linkedinyou has joined #puppet-openstack | 19:40 | |
| *** dgurtner has joined #puppet-openstack | 19:55 | |
| *** dgurtner has quit IRC | 19:55 | |
| *** dgurtner has joined #puppet-openstack | 19:55 | |
| *** xarses has joined #puppet-openstack | 19:59 | |
| *** jayg is now known as jayg|g0n3 | 20:04 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 20:06 |
| Hunner | Hmm. There doesn't appear to be a way to update a mongodb users password on 2.4 without using plaintext | 20:24 |
| *** dfisher has quit IRC | 20:29 | |
| *** zeus- has joined #puppet-openstack | 20:29 | |
| *** zeus- is now known as zeus | 20:31 | |
| rcarrillocruz | SpamapS, crinkle: heya, mind reviewing https://review.openstack.org/#/c/215575/ when you get a sec? EmilienM was awaiting for your input prior to +2ing... | 20:31 |
| *** zeus is now known as Guest92463 | 20:31 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider https://review.openstack.org/208054 | 20:36 |
| _ody | 2.2(5b) | 20:44 |
| *** simonmcc has quit IRC | 20:45 | |
| *** Guest92463 has quit IRC | 20:45 | |
| _ody | oops | 20:46 |
| _ody | rw | 20:46 |
| *** simonmcc has joined #puppet-openstack | 20:47 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: Use inifile for tempest_*_id_setter provider https://review.openstack.org/217389 | 20:50 |
| *** asd112z_ has quit IRC | 20:51 | |
| *** asd112z has joined #puppet-openstack | 20:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 20:52 |
| *** asd112z has quit IRC | 20:52 | |
| *** jpena|away has quit IRC | 20:52 | |
| *** asd112z has joined #puppet-openstack | 20:53 | |
| *** mihgen has quit IRC | 20:55 | |
| *** mihgen has joined #puppet-openstack | 20:55 | |
| *** openstackgerrit has quit IRC | 21:01 | |
| *** openstackgerrit has joined #puppet-openstack | 21:02 | |
| *** dgurtner has quit IRC | 21:23 | |
| *** zeus has joined #puppet-openstack | 21:29 | |
| *** zeus is now known as Guest53419 | 21:29 | |
| SpamapS | rcarrillocruz: done | 21:30 |
| rcarrillocruz | Thx sir | 21:31 |
| crinkle | rcarrillocruz: also done | 21:34 |
| rcarrillocruz | ++ | 21:34 |
| *** dgurtner has joined #puppet-openstack | 21:37 | |
| *** dgurtner has quit IRC | 21:37 | |
| *** dgurtner has joined #puppet-openstack | 21:37 | |
| *** dprince has quit IRC | 21:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: run Tempest with compute tests https://review.openstack.org/217352 | 22:36 |
| *** ducttape_ has quit IRC | 22:45 | |
| *** markvoelker has quit IRC | 22:46 | |
| *** dgurtner has quit IRC | 22:47 | |
| *** saneax has joined #puppet-openstack | 22:51 | |
| *** markvoelker has joined #puppet-openstack | 22:54 | |
| *** dgurtner has joined #puppet-openstack | 22:59 | |
| *** sergmelikyan has quit IRC | 23:09 | |
| *** gildub has joined #puppet-openstack | 23:09 | |
| *** aimon has quit IRC | 23:38 | |
| *** dgurtner has quit IRC | 23:49 | |
| *** zhangjn has joined #puppet-openstack | 23:55 | |
| *** asd112z has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!