| *** sergmelikyan has quit IRC | 00:03 | |
| *** aimon has quit IRC | 00:03 | |
| *** fedexo has joined #puppet-openstack | 00:05 | |
| *** xarses has quit IRC | 00:09 | |
| *** gfidente has quit IRC | 00:14 | |
| *** bklei has quit IRC | 00:14 | |
| *** bklei has joined #puppet-openstack | 00:16 | |
| *** xingchao has joined #puppet-openstack | 00:19 | |
| *** xingchao has quit IRC | 00:24 | |
| *** sergmelikyan has joined #puppet-openstack | 00:25 | |
| *** aimon has joined #puppet-openstack | 00:25 | |
| *** markvoelker has quit IRC | 00:27 | |
| *** mdorman has quit IRC | 00:27 | |
| *** markvoelker has joined #puppet-openstack | 00:28 | |
| *** tiswanso has joined #puppet-openstack | 00:30 | |
| *** tiswanso has quit IRC | 00:41 | |
| *** jasondotstar has quit IRC | 00:50 | |
| *** jasondotstar has joined #puppet-openstack | 00:52 | |
| *** jasondotstar has quit IRC | 00:56 | |
| *** tiswanso has joined #puppet-openstack | 01:01 | |
| *** tiswanso has quit IRC | 01:03 | |
| *** xarses has joined #puppet-openstack | 01:03 | |
| *** tiswanso has joined #puppet-openstack | 01:04 | |
| *** angdraug has quit IRC | 01:23 | |
| *** xingchao has joined #puppet-openstack | 01:25 | |
| *** jasondotstar has joined #puppet-openstack | 01:27 | |
| *** xingchao has quit IRC | 01:33 | |
| *** xingchao has joined #puppet-openstack | 01:33 | |
| *** xiaohua has joined #puppet-openstack | 01:43 | |
| openstackgerrit | Xiaohua Yuan proposed openstack/puppet-designate: Add related parameters to designate::central https://review.openstack.org/209727 | 01:44 |
|---|---|---|
| *** gildub has joined #puppet-openstack | 01:49 | |
| *** xingchao has quit IRC | 01:51 | |
| openstackgerrit | Xiaohua Yuan proposed openstack/puppet-designate: add designate_rootwrap_config in designate::config https://review.openstack.org/208178 | 01:56 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Initial run_tests.sh script https://review.openstack.org/206093 | 01:59 |
| *** xingchao has joined #puppet-openstack | 02:25 | |
| *** markvoelker has quit IRC | 02:30 | |
| *** bklei has quit IRC | 02:30 | |
| *** xingchao has quit IRC | 02:41 | |
| *** dh64 has joined #puppet-openstack | 02:55 | |
| *** bklei has joined #puppet-openstack | 02:58 | |
| *** bklei has quit IRC | 03:22 | |
| *** markvoelker has joined #puppet-openstack | 03:30 | |
| *** xingchao has joined #puppet-openstack | 03:35 | |
| *** markvoelker has quit IRC | 03:35 | |
| *** xiaohua has quit IRC | 03:39 | |
| *** xiaohua has joined #puppet-openstack | 03:40 | |
| *** tiswanso has quit IRC | 03:40 | |
| *** xiaohua has quit IRC | 03:48 | |
| *** xiaohua has joined #puppet-openstack | 04:01 | |
| *** xiaohua has quit IRC | 04:08 | |
| *** jasondotstar has quit IRC | 04:09 | |
| *** xingchao has quit IRC | 04:13 | |
| *** xiaohua has joined #puppet-openstack | 04:20 | |
| *** xingchao has joined #puppet-openstack | 04:22 | |
| *** xiaohua has quit IRC | 04:28 | |
| *** xiaohua has joined #puppet-openstack | 04:41 | |
| openstackgerrit | Xingchao Yu proposed openstack/puppet-trove: Make taskmanager_queue option configurable https://review.openstack.org/210196 | 04:46 |
| *** xiaohua has quit IRC | 04:48 | |
| *** xingchao has quit IRC | 04:58 | |
| *** xiaohua has joined #puppet-openstack | 05:01 | |
| *** xiaohua has quit IRC | 05:14 | |
| *** sergmelikyan has quit IRC | 05:15 | |
| *** jasondotstar has joined #puppet-openstack | 05:16 | |
| *** serg_melikyan has joined #puppet-openstack | 05:18 | |
| *** fedexo has quit IRC | 05:19 | |
| *** jasondotstar has quit IRC | 05:21 | |
| *** serg_melikyan has quit IRC | 05:22 | |
| *** xiaohua has joined #puppet-openstack | 05:27 | |
| *** markvoelker has joined #puppet-openstack | 05:31 | |
| *** xiaohua has quit IRC | 05:36 | |
| *** markvoelker has quit IRC | 05:36 | |
| *** xingchao has joined #puppet-openstack | 06:07 | |
| *** sergmelikyan has joined #puppet-openstack | 06:18 | |
| *** sergmelikyan has quit IRC | 06:23 | |
| *** xiaohua has joined #puppet-openstack | 06:32 | |
| *** xiaohua has quit IRC | 06:40 | |
| *** dgurtner has joined #puppet-openstack | 06:51 | |
| *** xiaohua has joined #puppet-openstack | 06:52 | |
| *** paramite has joined #puppet-openstack | 06:59 | |
| *** xiaohua has quit IRC | 07:00 | |
| *** jasondotstar has joined #puppet-openstack | 07:05 | |
| *** jasondotstar has quit IRC | 07:09 | |
| *** sergmelikyan has joined #puppet-openstack | 07:19 | |
| *** sergmelikyan has quit IRC | 07:24 | |
| *** saneax has joined #puppet-openstack | 07:29 | |
| *** markvoelker has joined #puppet-openstack | 07:32 | |
| *** markvoelker has quit IRC | 07:37 | |
| *** gfidente has joined #puppet-openstack | 07:50 | |
| *** xiaohua has joined #puppet-openstack | 07:57 | |
| *** xiaohua has quit IRC | 08:05 | |
| *** jistr has joined #puppet-openstack | 08:06 | |
| *** gildub has quit IRC | 08:08 | |
| *** xingchao has quit IRC | 08:08 | |
| *** xingchao has joined #puppet-openstack | 08:12 | |
| *** xiaohua has joined #puppet-openstack | 08:17 | |
| *** dgurtner has quit IRC | 08:18 | |
| *** xiaohua has quit IRC | 08:26 | |
| *** xingchao has quit IRC | 08:35 | |
| *** xiaohua has joined #puppet-openstack | 08:38 | |
| *** paramite has quit IRC | 08:42 | |
| *** xiaohua has quit IRC | 08:46 | |
| *** dkaigarodsev_ has joined #puppet-openstack | 08:49 | |
| *** paramite has joined #puppet-openstack | 08:55 | |
| *** xiaohua has joined #puppet-openstack | 08:58 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-nova: Revert "Support allow_start and allow_end parameters" https://review.openstack.org/210256 | 08:59 |
| *** jasondotstar has joined #puppet-openstack | 09:00 | |
| *** imcsk8 has quit IRC | 09:03 | |
| *** xiaohua has quit IRC | 09:06 | |
| *** dgurtner has joined #puppet-openstack | 09:15 | |
| *** dgurtner has quit IRC | 09:15 | |
| *** dgurtner has joined #puppet-openstack | 09:15 | |
| *** xiaohua has joined #puppet-openstack | 09:18 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/puppet-horizon: Make redirect type configurable https://review.openstack.org/208462 | 09:18 |
| *** sergmelikyan has joined #puppet-openstack | 09:20 | |
| *** sergmelikyan has quit IRC | 09:24 | |
| *** xiaohua has quit IRC | 09:26 | |
| *** jasondotstar has quit IRC | 09:29 | |
| *** markvoelker has joined #puppet-openstack | 09:33 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add use_stderr parameter https://review.openstack.org/210274 | 09:35 |
| *** markvoelker has quit IRC | 09:37 | |
| *** xiaohua has joined #puppet-openstack | 09:39 | |
| *** chem has joined #puppet-openstack | 09:40 | |
| *** jasondotstar has joined #puppet-openstack | 09:42 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-nova: Add use_stderr parameter https://review.openstack.org/210276 | 09:43 |
| *** xiaohua has quit IRC | 09:46 | |
| *** xiaohua has joined #puppet-openstack | 09:59 | |
| *** xiaohua has quit IRC | 10:07 | |
| *** jasondotstar has quit IRC | 10:13 | |
| *** xiaohua has joined #puppet-openstack | 10:18 | |
| *** jistr has quit IRC | 10:20 | |
| *** jistr has joined #puppet-openstack | 10:20 | |
| *** jeblair has quit IRC | 10:21 | |
| *** jeblair has joined #puppet-openstack | 10:22 | |
| *** xiaohua has quit IRC | 10:26 | |
| *** duritong_ has joined #puppet-openstack | 10:28 | |
| *** duritong has quit IRC | 10:28 | |
| *** nigelb has quit IRC | 10:28 | |
| *** nigelb has joined #puppet-openstack | 10:28 | |
| jistr | clayton: ping re cinder lock_path | 10:32 |
| *** xiaohua has joined #puppet-openstack | 10:39 | |
| *** mtreinish has quit IRC | 10:45 | |
| *** ilbot3 has quit IRC | 10:45 | |
| *** openstackgerrit has quit IRC | 10:45 | |
| *** ddmitriev1 has quit IRC | 10:45 | |
| *** _matthias_ has quit IRC | 10:45 | |
| *** kfjohnson has quit IRC | 10:45 | |
| *** dweaver has quit IRC | 10:45 | |
| *** danieru has quit IRC | 10:45 | |
| *** wabu has quit IRC | 10:45 | |
| *** abramley has quit IRC | 10:45 | |
| *** vmbrasseur has quit IRC | 10:45 | |
| *** sc` has quit IRC | 10:45 | |
| *** xiaohua has quit IRC | 10:48 | |
| *** vmbrasseur has joined #puppet-openstack | 10:49 | |
| *** mtreinish has joined #puppet-openstack | 10:49 | |
| *** ilbot3 has joined #puppet-openstack | 10:49 | |
| *** openstackgerrit has joined #puppet-openstack | 10:49 | |
| *** ddmitriev1 has joined #puppet-openstack | 10:49 | |
| *** _matthias_ has joined #puppet-openstack | 10:49 | |
| *** kfjohnson has joined #puppet-openstack | 10:49 | |
| *** dweaver has joined #puppet-openstack | 10:49 | |
| *** danieru has joined #puppet-openstack | 10:49 | |
| *** wabu has joined #puppet-openstack | 10:49 | |
| *** abramley has joined #puppet-openstack | 10:49 | |
| *** sc` has joined #puppet-openstack | 10:49 | |
| *** sergmelikyan has joined #puppet-openstack | 10:51 | |
| *** _morgan has quit IRC | 10:52 | |
| *** _morgan has joined #puppet-openstack | 10:52 | |
| *** sergmelikyan has quit IRC | 10:55 | |
| *** vmbrasseur has quit IRC | 10:58 | |
| *** vmbrasseur has joined #puppet-openstack | 11:03 | |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-cinder: Revert "Add lock_path to cinder config" https://review.openstack.org/210328 | 11:04 |
| *** xiaohua has joined #puppet-openstack | 11:04 | |
| *** xiaohua has quit IRC | 11:12 | |
| *** sergmelikyan has joined #puppet-openstack | 11:31 | |
| *** markvoelker has joined #puppet-openstack | 11:34 | |
| *** sergmelikyan has quit IRC | 11:35 | |
| *** markvoelker has quit IRC | 11:39 | |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-cinder: Add lock_path to cinder config https://review.openstack.org/210350 | 11:46 |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-cinder: Reintroduce "Add lock_path to cinder config" https://review.openstack.org/210350 | 11:47 |
| *** dprince has joined #puppet-openstack | 11:49 | |
| clayton | jistr ? | 11:52 |
| jistr | clayton: hi :) | 11:52 |
| jistr | clayton: the lock_path patch breaks TripleO CI, i submitted a revert and un-revert | 11:53 |
| clayton | why does it break tripleo? | 11:53 |
| jistr | clayton: because the default path is wrong for Fedora based distros | 11:54 |
| jistr | clayton: i saw the revert not pass CI though... | 11:54 |
| clayton | I'd prefer to just fix it | 11:55 |
| jistr | clayton: yeah that's what i did in https://review.openstack.org/210350 | 11:56 |
| clayton | looking | 11:56 |
| jistr | clayton: i'm used to reverting things first in case there's some trouble with the fix too... I'd prefer to have this solved soon because ATM all TripleO CI is red because of this. | 11:57 |
| jistr | so my reasoning was "we can revert fast and reintroduce it when reviewers are happy with the way it's done" | 11:57 |
| clayton | what is the valid value for fedora? | 11:58 |
| clayton | and is it the same for fedora as all other RH OSes? | 11:59 |
| jistr | clayton: /var/lib/cinder/tmp is the package default | 11:59 |
| jistr | [root@overcloud-controller-0 ~]# grep lock_path /usr/share/cinder/cinder-dist.conf | 11:59 |
| jistr | lock_path = /var/lib/cinder/tmp | 11:59 |
| jistr | clayton: i don't know if it's the same for other RH OSes but i'd assume so | 11:59 |
| clayton | is it just me, or has infra gerrit been really slow the last two days? | 12:00 |
| jistr | it runs fairly ok for me. Not blazingly fast but not too slow either. | 12:01 |
| clayton | I believe the install_modules.sh is something that is known, I saw patches about that yesterday | 12:01 |
| jistr | ack | 12:01 |
| jistr | but i don't suppose we can push the revert in spite of those jobs failing | 12:01 |
| clayton | that would be emilien's call | 12:02 |
| jistr | i might need to cherry pick the fixed version in TripleO CI for now, until the puppet CI is ok | 12:02 |
| clayton | I suspect anyone that is core can push direct, but I doubt he'll want to do that | 12:02 |
| *** chem has quit IRC | 12:03 | |
| clayton | I'll be back in about an hour, have to ferry the kids to camp | 12:03 |
| jistr | clayton: ack, thanks so far :) | 12:03 |
| *** morazi has joined #puppet-openstack | 12:05 | |
| *** xiaohua has joined #puppet-openstack | 12:09 | |
| openstackgerrit | Jiri Stransky proposed openstack/puppet-cinder: Reintroduce "Add lock_path to cinder config" https://review.openstack.org/210350 | 12:15 |
| *** xiaohua has quit IRC | 12:17 | |
| EmilienM | good morning | 12:23 |
| EmilienM | jistr: what's up? | 12:24 |
| jistr | the patch which adds lock_path to puppet-cinder breaks tripleo, because the default path is not correct on Fedora-based systems. I submitted a revert https://review.openstack.org/210328 and also reintroduced the patch in a distro agnostic way https://review.openstack.org/210350 | 12:25 |
| jistr | (wanted to revert first in case there's some problem with the fix too) | 12:26 |
| jistr | EmilienM: but seems like puppet CI is down too | 12:26 |
| EmilienM | puppet ci down? | 12:26 |
| EmilienM | mhh | 12:26 |
| EmilienM | let me loo | 12:26 |
| EmilienM | look* | 12:26 |
| *** paramite is now known as paramite|afk | 12:27 | |
| jistr | EmilienM: i also submitted a patch to pin down puppet-cinder in tripleo CI for now https://review.openstack.org/210362 | 12:27 |
| jistr | so we don't necessarily have to push the puppet-cinder revert through red CI | 12:27 |
| EmilienM | jistr: ok I know what is failing | 12:28 |
| EmilienM | jistr: let me 10 min and it's fixed. | 12:28 |
| jistr | EmilienM: great, thx! then we can recheck the two puppet-cinder patches and roll on | 12:28 |
| *** markvoelker has joined #puppet-openstack | 12:29 | |
| *** xiaohua has joined #puppet-openstack | 12:29 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Put install_modules.sh in / to unlock CI jobs https://review.openstack.org/210368 | 12:30 |
| EmilienM | jistr: done ^ I'm checking if it's fine when it's merged - in the meantime I'm looking at your 2 patches | 12:30 |
| *** sergmelikyan has joined #puppet-openstack | 12:31 | |
| EmilienM | jistr: '/var/lib/cinder/tmp' ? seriously? | 12:33 |
| jistr | EmilienM: yup... at least on Fedora. I'd assume it's the same in others | 12:33 |
| EmilienM | jistr: this is ugly | 12:34 |
| jistr | EmilienM: just checked, it's the same on RHEL | 12:35 |
| *** sergmelikyan has quit IRC | 12:36 | |
| *** xiaohua has quit IRC | 12:38 | |
| EmilienM | jistr: patch in the gate | 12:41 |
| EmilienM | jistr: CI unblocked in ~ min | 12:41 |
| jistr | cool | 12:41 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-glance: Add an ability to manage use_stderr parameter https://review.openstack.org/210374 | 12:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-cinder: Add an ability to manage use_stderr parameter https://review.openstack.org/210375 | 12:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-ceilometer: Add an ability to manage use_stderr parameter https://review.openstack.org/210376 | 12:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-heat: Add an ability to manage use_stderr parameter https://review.openstack.org/210377 | 12:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-heat: Add an ability to manage use_stderr parameter https://review.openstack.org/210377 | 12:47 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-ceilometer: Add an ability to manage use_stderr parameter https://review.openstack.org/210376 | 12:48 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Put install_modules.sh in / to unlock CI jobs https://review.openstack.org/210368 | 12:49 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-cinder: Add an ability to manage use_stderr parameter https://review.openstack.org/210375 | 12:49 |
| EmilienM | jistr: should be good now | 12:49 |
| *** xiaohua has joined #puppet-openstack | 12:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Put install_modules.sh in / to unlock CI jobs" https://review.openstack.org/210380 | 12:50 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add an ability to manage use_stderr parameter https://review.openstack.org/210274 | 12:51 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-nova: Add an ability to manage use_stderr parameter https://review.openstack.org/210276 | 12:51 |
| jistr | EmilienM: thanks! | 12:51 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: update install_modules.sh path https://review.openstack.org/210384 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: acceptance: update install_modules.sh path https://review.openstack.org/210385 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: acceptance: update install_modules.sh path https://review.openstack.org/210386 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: update install_modules.sh path https://review.openstack.org/210387 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: acceptance: update install_modules.sh path https://review.openstack.org/210388 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: acceptance: update install_modules.sh path https://review.openstack.org/210389 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: update install_modules.sh path https://review.openstack.org/210390 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: acceptance: update install_modules.sh path https://review.openstack.org/210391 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: update install_modules.sh path https://review.openstack.org/210392 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: update install_modules.sh path https://review.openstack.org/210393 | 12:56 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: acceptance: update install_modules.sh path https://review.openstack.org/210394 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: acceptance: update install_modules.sh path https://review.openstack.org/210395 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: acceptance: update install_modules.sh path https://review.openstack.org/210396 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: acceptance: update install_modules.sh path https://review.openstack.org/210397 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: acceptance: update install_modules.sh path https://review.openstack.org/210398 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: acceptance: update install_modules.sh path https://review.openstack.org/210399 | 12:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: acceptance: update install_modules.sh path https://review.openstack.org/210400 | 12:57 |
| *** xiaohua has quit IRC | 12:58 | |
| *** saneax_-_ has quit IRC | 13:02 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-keystone: Add an ability to manage use_stderr parameter https://review.openstack.org/210403 | 13:04 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-sahara: Add an ability to manage use_stderr parameter https://review.openstack.org/210404 | 13:05 |
| *** tiswanso has joined #puppet-openstack | 13:10 | |
| *** xiaohua has joined #puppet-openstack | 13:10 | |
| *** paramite|afk is now known as paramite | 13:10 | |
| *** logan2 has quit IRC | 13:11 | |
| *** logan2 has joined #puppet-openstack | 13:12 | |
| EmilienM | pabelanger, jistr: I think Puppetfile is not found too | 13:13 |
| EmilienM | pabelanger: the Puppetfile is in SCRIPT_DIR | 13:15 |
| pabelanger | BRB: coffee, but yes. I have some concerns about this dependency | 13:15 |
| EmilienM | well, we will solve this dependency issue after fixing the path | 13:15 |
| EmilienM | pabelanger: I'm pushing a patch to have the script in tools but the Puppetfile in root | 13:16 |
| EmilienM | pabelanger: is that fine? | 13:16 |
| *** skolekonov has joined #puppet-openstack | 13:16 | |
| EmilienM | actually, CI is fixed now I guess. Though my changes are not ready yet | 13:17 |
| clayton | spredzy: sounds like we should talk about notify behavior for db::sync classes when you're around | 13:18 |
| *** xiaohua has quit IRC | 13:18 | |
| EmilienM | err, my bad.. CI is still broken due to Puppetfile | 13:18 |
| clayton | it's fairly inconsistent right now, it'd probably make sense to have all the modules do it the same way | 13:18 |
| pabelanger | EmilienM: I would suggest reverting the commit from last night, until I better understand the dependency betweek beaker and puppet-openstack-integration. Right now we are just jumping around to fix broken gates will out properly testing patches in the gate | 13:22 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Initial run_tests.sh script" https://review.openstack.org/210411 | 13:22 |
| EmilienM | pabelanger: done^ | 13:22 |
| EmilienM | pabelanger: I totally get what you mean | 13:23 |
| pabelanger | EmilienM: let that merge, reset jobs. By then the coffee will kick in and start looking to see why things broken | 13:24 |
| pabelanger | broke* | 13:24 |
| EmilienM | yes | 13:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Revert "Initial run_tests.sh script"" https://review.openstack.org/210412 | 13:25 |
| *** xiaohua has joined #puppet-openstack | 13:30 | |
| *** paramite is now known as paramite|afk | 13:30 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Remove httpchk option from haproxy listeners https://review.openstack.org/197904 | 13:35 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Move intall_modules.sh to tools directory https://review.openstack.org/210380 | 13:36 |
| *** paramite|afk is now known as paramite | 13:37 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: update install_modules.sh path https://review.openstack.org/210419 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: acceptance: update install_modules.sh path https://review.openstack.org/210420 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: acceptance: update install_modules.sh path https://review.openstack.org/210421 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-glance: acceptance: update install_modules.sh path https://review.openstack.org/210422 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: acceptance: update install_modules.sh path https://review.openstack.org/210423 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: acceptance: update install_modules.sh path https://review.openstack.org/210424 | 13:38 |
| *** xiaohua has quit IRC | 13:38 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: update install_modules.sh path https://review.openstack.org/210425 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: acceptance: update install_modules.sh path https://review.openstack.org/210426 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: update install_modules.sh path https://review.openstack.org/210427 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: update install_modules.sh path https://review.openstack.org/210428 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: acceptance: update install_modules.sh path https://review.openstack.org/210429 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstacklib: acceptance: update install_modules.sh path https://review.openstack.org/210430 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: acceptance: update install_modules.sh path https://review.openstack.org/210431 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: acceptance: update install_modules.sh path https://review.openstack.org/210432 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: acceptance: update install_modules.sh path https://review.openstack.org/210433 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: acceptance: update install_modules.sh path https://review.openstack.org/210434 | 13:38 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-vswitch: acceptance: update install_modules.sh path https://review.openstack.org/210435 | 13:38 |
| pabelanger | EmilienM: so, if I understand, we are using puppet-openstack-integration to resolve the global dependencys for all puppet modules | 13:40 |
| EmilienM | our gate is currently broken - will be fixed once https://review.openstack.org/#/c/210411/ is merged | 13:40 |
| EmilienM | pabelanger: yes | 13:40 |
| pabelanger | EmilienM: is our msync process working yet? | 13:40 |
| sbadia | hi | 13:40 |
| EmilienM | pabelanger: what do you mean by working? | 13:41 |
| sbadia | pabelanger: not yet, the initial msync is in progress | 13:41 |
| EmilienM | sbadia: o/ | 13:41 |
| sbadia | \o | 13:41 |
| pabelanger | EmilienM: sbadia: okay, so to avoid the circular dependency, I think we need to more Puppetfile and install script into msync and install into each puppet-* module. Other wise, every puppet module beaker job would need to be added as a check / gate job for puppet-openstack-integration | 13:43 |
| EmilienM | pabelanger: so I did https://review.openstack.org/#/c/210380/ | 13:43 |
| EmilienM | pabelanger: and all the "acceptance: update install_modules.sh path" patches with the Depends-On https://review.openstack.org/#/c/210380/ | 13:43 |
| EmilienM | with the goal to test the new path | 13:43 |
| EmilienM | pabelanger: so I guess the only one piece we're missing, is to gate at least one pupet module in openstack-integ | 13:44 |
| EmilienM | what do you think? | 13:44 |
| pabelanger | I'd rather break the dependency beaker-jobs has on puppet-openstack-integration. Personally, I see them as 2 independent processes. Only way I can think of that is using msync to keep Puppetfile and install scirpt sync across all modules | 13:46 |
| *** cdelatte has joined #puppet-openstack | 13:46 | |
| *** dfisher has joined #puppet-openstack | 13:47 | |
| EmilienM | pabelanger: we still need a repo to host our script that installs puppet modules with zuul | 13:47 |
| EmilienM | so you want to duplicate the script? | 13:48 |
| EmilienM | and use msync to keep it synced | 13:48 |
| EmilienM | it sounds risky | 13:48 |
| EmilienM | pabelanger: I suggest for now we wait crinkle wakes up so we can talk with her. She did that script, I guess she deserves to be in the discussion | 13:49 |
| pabelanger | I think we need to revisit what we are trying to do today. From the sounds of it, the goal is to have a single global requirements file for all puppet-* modules to install dependencies from | 13:49 |
| EmilienM | yeah, sure, we need to revisit that | 13:50 |
| EmilienM | we have the proof our system is broken :) | 13:50 |
| EmilienM | well, I would rather say "not solid enough" | 13:50 |
| pabelanger | well, we exposed case where we don't have test coverage | 13:50 |
| EmilienM | yep | 13:50 |
| pabelanger | which is fine, we just need to plug the hole now :) | 13:50 |
| EmilienM | pabelanger: we continue when crinkle is online | 13:50 |
| pabelanger | Ya, going to start composing my thoughts | 13:51 |
| EmilienM | pabelanger: until then, we need to make sure to restart our CI | 13:51 |
| EmilienM | pabelanger: cool | 13:51 |
| *** xiaohua has joined #puppet-openstack | 13:51 | |
| pabelanger | https://etherpad.openstack.org/p/puppet-modules-global-requirements | 13:51 |
| dfisher | are those requirements for puppet modules themselves? | 13:52 |
| EmilienM | yes | 13:52 |
| dfisher | how do you handle requiring a module that is a functional noop on some OSes? | 13:52 |
| dfisher | puppetlabs-apt, for example, on Solaris/RHEL/Fedora ? | 13:52 |
| dfisher | do they just get installed and are then unused? | 13:53 |
| *** xingchao has joined #puppet-openstack | 13:53 | |
| *** xiaohua has quit IRC | 13:58 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Remove httpchk option from haproxy listeners https://review.openstack.org/197904 | 14:01 |
| *** sergmelikyan has joined #puppet-openstack | 14:02 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Revert "Initial run_tests.sh script" https://review.openstack.org/210411 | 14:03 |
| EmilienM | should be fix now | 14:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Run CI test - do not merge https://review.openstack.org/210454 | 14:10 |
| *** xiaohua has joined #puppet-openstack | 14:10 | |
| *** paramite is now known as paramite|afk | 14:12 | |
| *** paramite|afk is now known as paramite | 14:12 | |
| *** richm has joined #puppet-openstack | 14:12 | |
| *** xiaohua has quit IRC | 14:18 | |
| iurygregory | hello people =) | 14:21 |
| *** tiswanso has quit IRC | 14:22 | |
| *** tiswanso has joined #puppet-openstack | 14:23 | |
| pabelanger | sbadia: crinkle https://github.com/puppet-community/modulesync/pull/53 | 14:24 |
| pabelanger | not sure that is the correct way, but gets us a way to stop msync from cloning git repos | 14:25 |
| pabelanger | well _should_ untested right now | 14:25 |
| *** ferest has joined #puppet-openstack | 14:29 | |
| EmilienM | pabelanger: https://review.openstack.org/210472 | 14:34 |
| EmilienM | pabelanger, jistr: CI is fixed for real now... | 14:36 |
| EmilienM | I did all needed rechecks | 14:37 |
| jistr | EmilienM: thanks! | 14:37 |
| *** ferest has quit IRC | 14:38 | |
| *** mdorman has joined #puppet-openstack | 14:41 | |
| *** tiswanso has quit IRC | 14:43 | |
| *** tiswanso has joined #puppet-openstack | 14:44 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release https://review.openstack.org/209286 | 14:44 |
| EmilienM | pabelanger: also, this kind of patch is dangerous: https://review.openstack.org/#/c/209209/ | 14:45 |
| EmilienM | pabelanger: and having the Puppetfile in a repo we can actually gate is useful | 14:45 |
| EmilienM | not sure how to handle it with msync | 14:45 |
| EmilienM | since we don't test anything | 14:45 |
| *** ferest has joined #puppet-openstack | 14:46 | |
| *** paramite is now known as paramite|afk | 14:46 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-cinder: acceptance: bump to Liberty release https://review.openstack.org/209287 | 14:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-designate: acceptance: bump to Liberty release https://review.openstack.org/209288 | 14:48 |
| pabelanger | EmilienM: I see it working the same way openstack/requirements does. We'd add a gate job into puppet-modulesync-configs that gate both on puppet-openstack-integration and, maybe a puppet-* modules (all?). | 14:48 |
| pabelanger | See: https://review.openstack.org/#/c/209797/ | 14:48 |
| crinkle | EmilienM: pabelanger can i get a tldr? | 14:49 |
| EmilienM | good morning crinkle | 14:49 |
| pabelanger | what ever uses that files, jobs need to be added into gate | 14:49 |
| EmilienM | crinkle: https://etherpad.openstack.org/p/puppet-modules-global-requirements | 14:49 |
| pabelanger | crinkle: TL;DR. found issue with Puppetfile breaking because of depend issue | 14:50 |
| *** ferest has quit IRC | 14:51 | |
| crinkle | I would give a strong -1 to using modulesync to solve this, it is much much preferable to centralize as much as we can rather than copying the same files all over the place | 14:51 |
| crinkle | it will end up making things harder since we have to approve 25 changes every time we change something | 14:52 |
| crinkle | and the puppetfile is likely to change frequently | 14:52 |
| pabelanger | I would disagree, in fact copying files is how openstack python projects handle dependencies | 14:52 |
| pabelanger | however, I agree there is overhead with that approach | 14:52 |
| *** paramite|afk is now known as paramite | 14:53 | |
| crinkle | I like the idea of adding a couple of modules as canary gate jobs to the integration repo | 14:53 |
| pabelanger | Having a global repo, which other puppet modules clone, can work. We just need to add gate jobs for ALL puppet modules that use those files, to ensure 100% coverage for testing | 14:54 |
| EmilienM | link for anyone interested: https://etherpad.openstack.org/p/puppet-libery-blocker | 14:54 |
| crinkle | the error in the patch in question would have been caught with a single depends-on patch in puppet-keystone or puppet-nova | 14:55 |
| pabelanger | crinkle: I would be interested to see the feedback on puppet modules on approval of requirements.txt | 14:56 |
| pabelanger | additionally, I'd have to check with infra, but there could be a way to auto approve the changes | 14:56 |
| pabelanger | like we do with translation jobs today | 14:56 |
| EmilienM | crinkle: yes, my bad, I've merged it last night | 14:56 |
| crinkle | auto approving sounds very dangerous | 14:56 |
| pabelanger | they still need to pass the gate | 14:57 |
| pabelanger | if they do, they are merge | 14:57 |
| pabelanger | merged* | 14:57 |
| pabelanger | if failed, code review still exists | 14:57 |
| EmilienM | crinkle: we did not auto approve that one: https://review.openstack.org/#/c/206093/ | 14:57 |
| EmilienM | which is the patch that broke CI | 14:58 |
| EmilienM | I auto approved the revert this morning, to unblock our CI quickly | 14:58 |
| crinkle | I think we're making a big fuss over something that would have been caught very easily with either manually making a couple of depends-on patches or adding some canary gate jobs | 14:58 |
| pabelanger | crinkle: caught? | 15:00 |
| crinkle | pabelanger: we would have caught the error in the patch | 15:00 |
| pabelanger | crinkle: right, but we didn't. Only after it merged. | 15:01 |
| EmilienM | let's create a canary job, it sounds good | 15:01 |
| EmilienM | crinkle: this job would be run for integ repo or also all p-* ? | 15:02 |
| pabelanger | Agreed, we can start with one. To see what coverage we get. | 15:03 |
| crinkle | EmilienM: well the point is it's the same job that's already being run on the puppet modules | 15:03 |
| *** ferest has joined #puppet-openstack | 15:03 | |
| crinkle | so adding like the beaker jobs from puppet-keystone and puppet-nova to puppet-openstack-integration would help catch things | 15:03 |
| EmilienM | crinkle: it is what I proposed in the etherpad | 15:04 |
| EmilienM | crinkle: L13 | 15:04 |
| pabelanger | my concern is, we're just pushing the issue down the road until we expose the next issue. puppet-openstack-integration is a stand alone module, we don't have have any project depending on it. | 15:04 |
| crinkle | pabelanger: okay, let's push it down the road then :) | 15:05 |
| pabelanger | meh | 15:06 |
| crinkle | i'd rather take things as they come than try to prepare for the unknown | 15:06 |
| *** tiswanso has quit IRC | 15:08 | |
| EmilienM | crinkle: http://goo.gl/Xwl36O | 15:08 |
| EmilienM | I guess this approach is a good start | 15:09 |
| *** ferest has quit IRC | 15:09 | |
| *** tiswanso has joined #puppet-openstack | 15:10 | |
| EmilienM | if we feel it's not enough we can think to the canary | 15:10 |
| EmilienM | the msync is definitely not the right option | 15:10 |
| crinkle | heh | 15:10 |
| pabelanger | Okay, still don't think adding beaker-jobs to puppet-openstack-integration is going to work. We have a circular dependency still. How are we going to land changes that require us to update beaker? | 15:11 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Add ability to specify ttl and timeout parameters https://review.openstack.org/207898 | 15:12 |
| pabelanger | lets add a non-voting beaker-job, so we can test | 15:12 |
| openstackgerrit | Merged openstack/puppet-neutron: Jumbo frames between instances https://review.openstack.org/199999 | 15:13 |
| pabelanger | EmilienM: it doesn't have to be msync, the Puppetfile needs to live upstream to both puppet-openstack-integration and puppet-* modules | 15:14 |
| pabelanger | however, I am not a fan of having external dependencies outside of project repos. Makes things hard once you start branching and such | 15:15 |
| pabelanger | to me, if openstack python is copying files from there requirements project, don't see why puppet cannot | 15:16 |
| *** xiaohua has joined #puppet-openstack | 15:16 | |
| pabelanger | vs creating our own processes | 15:16 |
| openstackgerrit | Merged openstack/puppet-keystone: Allow to change archive destination https://review.openstack.org/209129 | 15:16 |
| EmilienM | social: is delorean repo down? https://jenkins03.openstack.org/job/gate-puppet-ceilometer-puppet-beaker-rspec-dsvm-centos7/20/console | 15:16 |
| *** xingchao has quit IRC | 15:17 | |
| *** xingchao has joined #puppet-openstack | 15:17 | |
| crinkle | pabelanger: it might be worth taking this to #openstack-infra as the design for this affects them and was worked out with jeblair and nibalizer | 15:19 |
| *** aimon has quit IRC | 15:19 | |
| *** aimon has joined #puppet-openstack | 15:20 | |
| pabelanger | crinkle: ack | 15:22 |
| *** xiaohua has quit IRC | 15:23 | |
| *** linkedinyou has quit IRC | 15:24 | |
| *** darkxploit has joined #puppet-openstack | 15:24 | |
| *** aimon has quit IRC | 15:25 | |
| *** ducttape_ has joined #puppet-openstack | 15:26 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release https://review.openstack.org/209286 | 15:27 |
| *** sergmelikyan has quit IRC | 15:28 | |
| *** paramite has quit IRC | 15:29 | |
| *** dtalmage_ has joined #puppet-openstack | 15:29 | |
| *** dtalmage_ has left #puppet-openstack | 15:29 | |
| EmilienM | crinkle: what do you think about https://review.openstack.org/209320 and particularly the fact i'm testing only WSGi setup to be more real since we have one single node in our CI | 15:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-horizon: acceptance: bump to Liberty release https://review.openstack.org/209291 | 15:33 |
| crinkle | EmilienM: could we do the same thing we do with keystone and have two tests? then the upgrade from eventlet to wsgi is also tested | 15:34 |
| EmilienM | crinkle: in fact beaker jobs run the test alphabetically, we need to make sure to run first eventlet, then wsgi so we actually test the upgrade | 15:36 |
| *** dh64 has quit IRC | 15:36 | |
| *** logan2 has quit IRC | 15:38 | |
| crinkle | EmilienM: yes | 15:39 |
| EmilienM | crinkle: I plan to patch all *wsgi supported* projects | 15:40 |
| EmilienM | crinkle: I've seen a lot of API could be leveraged by WSGI | 15:40 |
| EmilienM | in OpenStack I mean | 15:40 |
| EmilienM | and I guess our operators would love having that feature in our modules | 15:40 |
| EmilienM | clayton: if you around ^ | 15:40 |
| EmilienM | dprince: ping - https://review.openstack.org/#/c/207861/ | 15:41 |
| *** logan2 has joined #puppet-openstack | 15:41 | |
| clayton | yes, although we're currently planning to just put that into the docker containers as we move in that direction, so I don't think we'll use direct wsgi support in the modules | 15:41 |
| crinkle | EmilienM: ++ to adding the feature | 15:44 |
| degorenko | spredzy, hey o/ ping | 15:44 |
| EmilienM | degorenko: spredzy is off today | 15:44 |
| degorenko | spredzy, oh, thanks | 15:44 |
| degorenko | EmilienM, ^ | 15:44 |
| EmilienM | crinkle: will do | 15:44 |
| EmilienM | crinkle: I was happy my patch almost worked out of the box actually | 15:45 |
| crinkle | EmilienM: :D | 15:46 |
| *** aimon has joined #puppet-openstack | 15:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: drop neutron_plugin_ovs type/provider https://review.openstack.org/210506 | 15:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Support for configuring openvswitch_agent.ini https://review.openstack.org/207861 | 15:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: acceptance: bump to Liberty release https://review.openstack.org/209294 | 15:55 |
| *** linkedinyou has joined #puppet-openstack | 15:59 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: acceptance: bump to Liberty release https://review.openstack.org/209290 | 15:59 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release https://review.openstack.org/209292 | 15:59 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 16:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 16:06 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 16:06 |
| iurygregory | Hey EmilienM, if you have make the changes you required in the spec, when you have please take a look =) | 16:08 |
| EmilienM | iurygregory: will look thx | 16:09 |
| *** xiaohua has joined #puppet-openstack | 16:20 | |
| *** xiaohua has quit IRC | 16:28 | |
| *** jistr has quit IRC | 16:28 | |
| EmilienM | w00t - 5 blockers and liberty is supposed to work everywhere https://etherpad.openstack.org/p/puppet-libery-blocker | 16:40 |
| *** xiaohua has joined #puppet-openstack | 16:40 | |
| dprince | EmilienM: ack, thanks for fixing that | 16:41 |
| *** xingchao has quit IRC | 16:41 | |
| EmilienM | dprince: my question is, how could it pas beaker tests before? they are running kilo | 16:41 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-modulesync-configs: Create sync-projects.sh script https://review.openstack.org/210528 | 16:43 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add an ability to manage use_stderr parameter https://review.openstack.org/210274 | 16:43 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-modulesync-configs: Create sync-projects.sh script https://review.openstack.org/210528 | 16:43 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-nova: Add an ability to manage use_stderr parameter https://review.openstack.org/210276 | 16:43 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-glance: Add an ability to manage use_stderr parameter https://review.openstack.org/210374 | 16:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-cinder: Add an ability to manage use_stderr parameter https://review.openstack.org/210375 | 16:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-heat: Add an ability to manage use_stderr parameter https://review.openstack.org/210377 | 16:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-keystone: Add an ability to manage use_stderr parameter https://review.openstack.org/210403 | 16:44 |
| pabelanger | EmilienM: sbadia: a much simpler script to run msync across all the puppet-modules | 16:44 |
| pabelanger | ^ | 16:44 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-sahara: Add an ability to manage use_stderr parameter https://review.openstack.org/210404 | 16:45 |
| EmilienM | pabelanger: looking | 16:45 |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-ceilometer: Add an ability to manage use_stderr parameter https://review.openstack.org/210376 | 16:46 |
| *** xarses has quit IRC | 16:47 | |
| pabelanger | EmilienM: sbadia: this is required: https://github.com/puppet-community/modulesync/pull/53 | 16:48 |
| EmilienM | crinkle might want to see also ^ | 16:48 |
| *** xiaohua has quit IRC | 16:49 | |
| EmilienM | pabelanger: looks cool | 16:49 |
| pabelanger | EmilienM: sbadia: crinkle: here is it working: http://pastebin.com/XdG9uCdn | 16:51 |
| pabelanger | using --noop mode | 16:51 |
| pabelanger | however, once gate is merged, we'll use --offline support so zuul deals with cloning | 16:51 |
| pabelanger | then we add a propose-puppet-modulesync-configs-updates post job and we'll be laughing | 16:53 |
| *** skolekonov has quit IRC | 16:56 | |
| openstackgerrit | Paul Belanger proposed openstack/puppet-modulesync-configs: Create sync-projects.sh script https://review.openstack.org/210528 | 16:57 |
| *** dprince has quit IRC | 17:01 | |
| *** xiaohua has joined #puppet-openstack | 17:02 | |
| *** degorenko has quit IRC | 17:02 | |
| *** imcsk8 has joined #puppet-openstack | 17:03 | |
| *** xiaohua has quit IRC | 17:09 | |
| *** xarses has joined #puppet-openstack | 17:09 | |
| openstackgerrit | Merged openstack/puppet-sahara: Creation of sahara::db::sync https://review.openstack.org/196683 | 17:14 |
| openstackgerrit | Merged openstack/puppet-nova: Creation of nova::db::sync https://review.openstack.org/196679 | 17:16 |
| EmilienM | crinkle: how come we have SUCCESS while there is a Puppet error ? http://logs.openstack.org/89/209289/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-centos7/ca777c0/console.html#_2015-08-05_01_01_08_901 | 17:17 |
| EmilienM | crinkle: though I wonder if it's related to the unscheduled refresh of glance-api | 17:18 |
| EmilienM | maybe a race condition where the provider tries to list images while glance-api is still starting | 17:19 |
| crinkle | EmilienM: it's an error in prefetch, puppet just eats that and keeps going | 17:19 |
| EmilienM | crinkle: ah ok | 17:19 |
| crinkle | it would have caused an error if there was already a glance image there | 17:19 |
| EmilienM | mhh ok | 17:19 |
| EmilienM | crinkle: still, I think the provider prefetch glance api while it's still starting | 17:20 |
| crinkle | EmilienM: ya probably | 17:20 |
| EmilienM | I file a bug | 17:20 |
| crinkle | EmilienM: i think my openstackclient patch for glance will fix it | 17:20 |
| crinkle | since it has the retries built in | 17:20 |
| EmilienM | I don't file a bug | 17:20 |
| * crinkle will go fix that up | 17:20 | |
| EmilienM | crinkle: when can we have it merged ? | 17:21 |
| crinkle | EmilienM: idk it has your +2 already :) | 17:21 |
| EmilienM | crinkle: ok, trying to rebase on top of yours, because it make jobs failing with liberty (don't ask me why please) | 17:21 |
| *** xiaohua has joined #puppet-openstack | 17:21 | |
| EmilienM | clayton: can you revisit again https://review.openstack.org/#/c/172580/ please ? | 17:23 |
| clayton | looking | 17:24 |
| crinkle | it's been sitting for so long i should make sure there aren't changes in lib/puppet/provider/glance_image/glance.rb that need to go in | 17:25 |
| EmilienM | crinkle: rebasing? | 17:26 |
| *** dgurtner has quit IRC | 17:26 | |
| EmilienM | crinkle: our gate will fail if we try to merge it | 17:26 |
| crinkle | EmilienM: problem is it just deletes the old file, so rebasing won't move changes into the new file | 17:26 |
| EmilienM | ah | 17:26 |
| EmilienM | yes | 17:26 |
| EmilienM | git log <lib> :) | 17:26 |
| EmilienM | crinkle: we did not change anything in lib for long time | 17:27 |
| crinkle | oh good | 17:27 |
| EmilienM | crinkle: you're the last commiter | 17:27 |
| EmilienM | June | 17:27 |
| EmilienM | we were so young! | 17:27 |
| crinkle | :) | 17:27 |
| crinkle | okay it should be good then | 17:28 |
| *** xiaohua has quit IRC | 17:31 | |
| *** xingchao has joined #puppet-openstack | 17:41 | |
| *** IBerezovskiy has quit IRC | 17:42 | |
| *** xiaohua has joined #puppet-openstack | 17:43 | |
| *** xingchao has quit IRC | 17:46 | |
| openstackgerrit | Merged openstack/puppet-glance: Use OpenstackClient for glance_image auth https://review.openstack.org/172580 | 17:47 |
| *** xiaohua has quit IRC | 17:50 | |
| *** prad has quit IRC | 17:52 | |
| EmilienM | crinkle: wOOt ^ | 18:00 |
| EmilienM | crinkle: https://review.openstack.org/#/c/209320/ pass CI, maybe you can look | 18:00 |
| *** xiaohua has joined #puppet-openstack | 18:02 | |
| *** iurygregory has left #puppet-openstack | 18:09 | |
| *** iurygregory has joined #puppet-openstack | 18:09 | |
| *** xiaohua has quit IRC | 18:10 | |
| clayton | is Denis Egorenko usually on here? not sure what his irc nick would be | 18:12 |
| EmilienM | clayton: degorenko | 18:22 |
| EmilienM | clayton: but offline now | 18:22 |
| *** xiaohua has joined #puppet-openstack | 18:23 | |
| *** xiaohua has quit IRC | 18:31 | |
| EmilienM | clayton: also https://review.openstack.org/#/c/196657 | 18:36 |
| openstackgerrit | Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance https://review.openstack.org/176150 | 18:38 |
| *** xiaohua has joined #puppet-openstack | 18:43 | |
| *** cdelatte has quit IRC | 18:50 | |
| *** xiaohua has quit IRC | 18:51 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-keystone: Add an ability to manage use_stderr parameter https://review.openstack.org/210403 | 18:58 |
| *** xiaohua has joined #puppet-openstack | 19:04 | |
| openstackgerrit | Merged openstack/puppet-ironic: Creation of ironic::db::sync https://review.openstack.org/196657 | 19:10 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release https://review.openstack.org/209292 | 19:11 |
| *** xiaohua has quit IRC | 19:12 | |
| EmilienM | clayton: have you read dprince's email about docker? | 19:21 |
| clayton | no, os-dev? | 19:22 |
| EmilienM | clayton: yeah | 19:22 |
| EmilienM | clayton: see puppet/docker/tripleo | 19:22 |
| EmilienM | clayton: Re: [openstack-dev] [TripleO] [Puppet] [kolla] Deploying OpenStack with Puppet modules on Docker with Heat | 19:22 |
| *** xiaohua has joined #puppet-openstack | 19:24 | |
| clayton | found it | 19:24 |
| clayton | that's interesting, I had mostly planned on using templates for config files once we had consul or something similar for service discovery in place | 19:24 |
| EmilienM | clayton: see my reply | 19:25 |
| EmilienM | clayton: I talk about your patch | 19:25 |
| EmilienM | clayton: in designate | 19:26 |
| clayton | ah, I see. Thanks :) | 19:26 |
| clayton | I think once mfisch gets back from vacation we'll probably be working on keystone. I've started some of the docker side of things internally already | 19:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 19:30 |
| EmilienM | crinkle: ^ | 19:30 |
| *** xiaohua has quit IRC | 19:32 | |
| *** xiaohua has joined #puppet-openstack | 19:44 | |
| *** dfisher has quit IRC | 19:46 | |
| *** xiaohua has quit IRC | 19:52 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance https://review.openstack.org/209320 | 20:04 |
| *** xiaohua has joined #puppet-openstack | 20:05 | |
| EmilienM | I wonder if there is a puppet-manila expert here | 20:10 |
| EmilienM | with liberty packages, puppet needs to be run 2 times | 20:10 |
| EmilienM | http://logs.openstack.org/93/209293/2/check/gate-puppet-manila-puppet-beaker-rspec-dsvm-centos7/3ea1b43/console.html | 20:10 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: acceptance: bump to Liberty release https://review.openstack.org/209293 | 20:12 |
| openstackgerrit | Paul Belanger proposed openstack/puppet-modulesync-configs: Create sync-projects.sh script https://review.openstack.org/210528 | 20:12 |
| *** xiaohua has quit IRC | 20:13 | |
| *** xiaohua has joined #puppet-openstack | 20:24 | |
| *** dtalmage_ has joined #puppet-openstack | 20:27 | |
| *** xiaohua has quit IRC | 20:32 | |
| EmilienM | same issue with sahara | 20:33 |
| mgagne | bodepd: ping. I wish to create a custom type based on ini which accepts both string and array as value. When munging the value, it looks like the value is converted to String regardless if the original value was an array or not. I found about :array_matching but now it converts everything to an array. an idea or example I can look at? | 20:36 |
| *** xiaohua has joined #puppet-openstack | 20:44 | |
| *** xiaohua has quit IRC | 20:53 | |
| *** tiswanso has quit IRC | 20:54 | |
| *** logan2 has quit IRC | 20:56 | |
| *** xiaohua has joined #puppet-openstack | 21:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-sahara: acceptance: bump to Liberty release https://review.openstack.org/209296 | 21:06 |
| *** xiaohua has quit IRC | 21:13 | |
| *** xiaohua has joined #puppet-openstack | 21:26 | |
| *** markvoelker has quit IRC | 21:28 | |
| EmilienM | crinkle: can you open https://review.openstack.org/#/c/209296/ ? it does not work for me | 21:31 |
| *** xiaohua has quit IRC | 21:33 | |
| *** morazi has quit IRC | 21:37 | |
| *** logan2 has joined #puppet-openstack | 21:39 | |
| *** xiaohua has joined #puppet-openstack | 21:46 | |
| *** ducttape_ has quit IRC | 21:52 | |
| crinkle | EmilienM: http://logs.openstack.org/96/209296/2/check/gate-puppet-sahara-puppet-beaker-rspec-dsvm-centos7/b9c2ca5/console.html#_2015-08-07_15_23_11_684 | 21:53 |
| crinkle | EmilienM: it looks like something is misconfigured and the service quits between puppet runs | 21:53 |
| EmilienM | crinkle: https://etherpad.openstack.org/p/puppet-libery-blocker | 21:53 |
| EmilienM | ah | 21:53 |
| *** xiaohua has quit IRC | 21:53 | |
| EmilienM | when I ssh the host, I run puppet, and it installs sahara | 21:53 |
| *** xiaohua has joined #puppet-openstack | 22:10 | |
| *** xiaohua has quit IRC | 22:14 | |
| *** xiaohua has joined #puppet-openstack | 22:27 | |
| *** guimaluf has quit IRC | 22:28 | |
| *** gfidente has quit IRC | 22:28 | |
| *** markvoelker has joined #puppet-openstack | 22:28 | |
| *** xingchao has joined #puppet-openstack | 22:31 | |
| *** markvoelker has quit IRC | 22:33 | |
| *** pabelanger has quit IRC | 22:33 | |
| *** xiaohua has quit IRC | 22:34 | |
| *** pabelanger has joined #puppet-openstack | 22:35 | |
| *** xingchao has quit IRC | 22:36 | |
| *** ducttape_ has joined #puppet-openstack | 22:40 | |
| *** xiaohua has joined #puppet-openstack | 22:46 | |
| *** mdorman has quit IRC | 22:49 | |
| *** mdorman has joined #puppet-openstack | 22:52 | |
| *** xiaohua has quit IRC | 22:55 | |
| *** ducttape_ has quit IRC | 22:58 | |
| *** xiaohua has joined #puppet-openstack | 23:07 | |
| *** logan2 has quit IRC | 23:15 | |
| *** xiaohua has quit IRC | 23:15 | |
| *** xiaohua has joined #puppet-openstack | 23:27 | |
| *** xiaohua has quit IRC | 23:35 | |
| *** xiaohua has joined #puppet-openstack | 23:47 | |
| *** aimon has quit IRC | 23:48 | |
| *** mdorman has quit IRC | 23:49 | |
| *** xiaohua has quit IRC | 23:56 | |
| *** markvoelker has joined #puppet-openstack | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!