Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:05 fedexo joined #puppet-openstack
00:16 bklei joined #puppet-openstack
00:19 xingchao joined #puppet-openstack
00:25 sergmelikyan joined #puppet-openstack
00:25 aimon joined #puppet-openstack
00:28 markvoelker joined #puppet-openstack
00:30 tiswanso joined #puppet-openstack
00:52 jasondotstar joined #puppet-openstack
01:01 tiswanso joined #puppet-openstack
01:03 xarses joined #puppet-openstack
01:04 tiswanso joined #puppet-openstack
01:25 xingchao joined #puppet-openstack
01:27 jasondotstar joined #puppet-openstack
01:33 xingchao joined #puppet-openstack
01:45 xiaohua joined #puppet-openstack
01:45 openstackgerrit Xiaohua Yuan proposed openstack/puppet-designate: Add related parameters to designate::central  https://review.openstack.org/209727
01:49 gildub joined #puppet-openstack
01:56 openstackgerrit Xiaohua Yuan proposed openstack/puppet-designate: add designate_rootwrap_config in designate::config  https://review.openstack.org/208178
01:59 openstackgerrit Merged openstack/puppet-openstack-integration: Initial run_tests.sh script  https://review.openstack.org/206093
02:25 xingchao joined #puppet-openstack
02:55 dh64 joined #puppet-openstack
02:58 bklei joined #puppet-openstack
03:30 markvoelker joined #puppet-openstack
03:35 xingchao joined #puppet-openstack
03:40 xiaohua joined #puppet-openstack
04:01 xiaohua joined #puppet-openstack
04:20 xiaohua joined #puppet-openstack
04:22 xingchao joined #puppet-openstack
04:41 xiaohua joined #puppet-openstack
04:46 openstackgerrit Xingchao Yu proposed openstack/puppet-trove: Make taskmanager_queue option configurable  https://review.openstack.org/210196
05:01 xiaohua joined #puppet-openstack
05:16 jasondotstar joined #puppet-openstack
05:18 serg_melikyan joined #puppet-openstack
05:27 xiaohua joined #puppet-openstack
05:31 markvoelker joined #puppet-openstack
06:07 xingchao joined #puppet-openstack
06:18 sergmelikyan joined #puppet-openstack
06:32 xiaohua joined #puppet-openstack
06:51 dgurtner joined #puppet-openstack
06:51 dgurtner joined #puppet-openstack
06:52 xiaohua joined #puppet-openstack
06:59 paramite joined #puppet-openstack
07:05 jasondotstar joined #puppet-openstack
07:19 sergmelikyan joined #puppet-openstack
07:29 saneax joined #puppet-openstack
07:32 markvoelker joined #puppet-openstack
07:50 gfidente joined #puppet-openstack
07:57 xiaohua joined #puppet-openstack
08:06 jistr joined #puppet-openstack
08:12 xingchao joined #puppet-openstack
08:17 xiaohua joined #puppet-openstack
08:38 xiaohua joined #puppet-openstack
08:49 dkaigarodsev_ joined #puppet-openstack
08:55 paramite joined #puppet-openstack
08:58 xiaohua joined #puppet-openstack
08:59 openstackgerrit Lukas Bezdicka proposed openstack/puppet-nova: Revert "Support allow_start and allow_end parameters"  https://review.openstack.org/210256
09:00 jasondotstar joined #puppet-openstack
09:15 dgurtner joined #puppet-openstack
09:15 dgurtner joined #puppet-openstack
09:18 xiaohua joined #puppet-openstack
09:18 openstackgerrit Sergii Golovatiuk proposed openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/208462
09:20 sergmelikyan joined #puppet-openstack
09:33 markvoelker joined #puppet-openstack
09:35 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add use_stderr parameter  https://review.openstack.org/210274
09:39 xiaohua joined #puppet-openstack
09:40 chem joined #puppet-openstack
09:42 jasondotstar joined #puppet-openstack
09:43 openstackgerrit Sergey Kolekonov proposed openstack/puppet-nova: Add use_stderr parameter  https://review.openstack.org/210276
09:59 xiaohua joined #puppet-openstack
10:28 xiaohua joined #puppet-openstack
10:28 jistr joined #puppet-openstack
10:28 jeblair joined #puppet-openstack
10:43 ferest joined #puppet-openstack
10:49 nigelb joined #puppet-openstack
10:49 duritong_ joined #puppet-openstack
10:49 jeblair joined #puppet-openstack
10:49 jistr joined #puppet-openstack
10:49 chem joined #puppet-openstack
10:49 dgurtner joined #puppet-openstack
10:49 paramite joined #puppet-openstack
10:49 dkaigarodsev_ joined #puppet-openstack
10:49 gfidente joined #puppet-openstack
10:49 saneax joined #puppet-openstack
10:49 dh64 joined #puppet-openstack
10:49 xarses joined #puppet-openstack
10:49 aimon joined #puppet-openstack
10:49 logan2 joined #puppet-openstack
10:49 kbyrne joined #puppet-openstack
10:49 hogepodge joined #puppet-openstack
10:49 iurygregory joined #puppet-openstack
10:49 degorenko joined #puppet-openstack
10:49 guimaluf joined #puppet-openstack
10:49 vinsh joined #puppet-openstack
10:49 higgins joined #puppet-openstack
10:49 prad joined #puppet-openstack
10:49 beddari joined #puppet-openstack
10:49 barthalion joined #puppet-openstack
10:49 fvollero joined #puppet-openstack
10:49 rlaurika joined #puppet-openstack
10:49 zigo joined #puppet-openstack
10:49 trash joined #puppet-openstack
10:49 rcarrillocruz joined #puppet-openstack
10:49 michchap joined #puppet-openstack
10:49 dmsimard joined #puppet-openstack
10:49 IBerezovskiy joined #puppet-openstack
10:49 social joined #puppet-openstack
10:49 vsaienko joined #puppet-openstack
10:49 sac joined #puppet-openstack
10:49 joshsamuelson joined #puppet-openstack
10:49 ric` joined #puppet-openstack
10:49 crinkle joined #puppet-openstack
10:49 mattymo joined #puppet-openstack
10:49 zino_ joined #puppet-openstack
10:49 tdb joined #puppet-openstack
10:49 colby_ joined #puppet-openstack
10:49 ianw joined #puppet-openstack
10:49 dachary joined #puppet-openstack
10:49 Madkiss joined #puppet-openstack
10:49 bogdando joined #puppet-openstack
10:49 linkedinyou joined #puppet-openstack
10:49 sanjayu_ joined #puppet-openstack
10:49 blentz joined #puppet-openstack
10:49 greghaynes joined #puppet-openstack
10:49 spredzy joined #puppet-openstack
10:49 gchamoul joined #puppet-openstack
10:49 larsks joined #puppet-openstack
10:49 Guest73505 joined #puppet-openstack
10:49 SpamapS joined #puppet-openstack
10:49 clayton joined #puppet-openstack
10:49 openstack joined #puppet-openstack
10:49 _nick joined #puppet-openstack
10:49 aarefiev joined #puppet-openstack
10:49 tchaypo joined #puppet-openstack
10:49 saneax_-_ joined #puppet-openstack
10:49 Hunner joined #puppet-openstack
10:49 _morgan joined #puppet-openstack
10:49 blkperl joined #puppet-openstack
10:49 pleia2 joined #puppet-openstack
10:49 mwhahaha joined #puppet-openstack
10:49 rhefner joined #puppet-openstack
10:49 simonmcc joined #puppet-openstack
10:49 EmilienM joined #puppet-openstack
10:49 fc__ joined #puppet-openstack
10:49 jayg|g0n3 joined #puppet-openstack
10:49 Mosibi joined #puppet-openstack
10:49 ddmitriev joined #puppet-openstack
10:49 leanderthal joined #puppet-openstack
10:49 ken_barber joined #puppet-openstack
10:49 amaksimov joined #puppet-openstack
10:49 aderyugin joined #puppet-openstack
10:49 nibalizer joined #puppet-openstack
10:49 tristanC joined #puppet-openstack
10:49 mattymo_ joined #puppet-openstack
10:49 timfreund joined #puppet-openstack
10:49 tlbr joined #puppet-openstack
10:49 francois joined #puppet-openstack
10:49 _stowa joined #puppet-openstack
10:49 mihgen joined #puppet-openstack
10:49 pasquier-s joined #puppet-openstack
10:49 krotscheck joined #puppet-openstack
10:49 khappone joined #puppet-openstack
10:49 _ody joined #puppet-openstack
10:49 finchd joined #puppet-openstack
10:49 mjb joined #puppet-openstack
10:49 mgagne joined #puppet-openstack
10:49 pabelanger joined #puppet-openstack
10:49 ikke-t joined #puppet-openstack
10:49 mwoodson joined #puppet-openstack
10:49 nigelk joined #puppet-openstack
10:49 partner joined #puppet-openstack
10:49 sbadia joined #puppet-openstack
10:49 sgolovatiuk joined #puppet-openstack
10:49 mfisch joined #puppet-openstack
10:49 asilenkov joined #puppet-openstack
10:49 bodepd joined #puppet-openstack
10:49 jgrassler joined #puppet-openstack
10:51 sergmelikyan joined #puppet-openstack
10:52 _morgan joined #puppet-openstack
11:03 vmbrasseur joined #puppet-openstack
11:04 openstackgerrit Jiri Stransky proposed openstack/puppet-cinder: Revert "Add lock_path to cinder config"  https://review.openstack.org/210328
11:04 xiaohua joined #puppet-openstack
11:31 sergmelikyan joined #puppet-openstack
11:34 markvoelker joined #puppet-openstack
11:46 openstackgerrit Jiri Stransky proposed openstack/puppet-cinder: Add lock_path to cinder config  https://review.openstack.org/210350
11:47 openstackgerrit Jiri Stransky proposed openstack/puppet-cinder: Reintroduce "Add lock_path to cinder config"  https://review.openstack.org/210350
11:49 dprince joined #puppet-openstack
11:52 clayton jistr ?
11:52 jistr clayton: hi :)
11:53 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:55 clayton I'd prefer to just fix it
11:56 jistr clayton: yeah that's what i did in https://review.openstack.org/210350
11:56 clayton looking
11:57 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:58 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
12:00 clayton is it just me, or has infra gerrit been really slow the last two days?
12:01 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:02 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: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:05 morazi joined #puppet-openstack
12:09 xiaohua joined #puppet-openstack
12:15 openstackgerrit Jiri Stransky proposed openstack/puppet-cinder: Reintroduce "Add lock_path to cinder config"  https://review.openstack.org/210350
12:23 EmilienM good morning
12:24 EmilienM jistr: what's up?
12:25 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:26 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: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:28 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:29 markvoelker joined #puppet-openstack
12:29 xiaohua joined #puppet-openstack
12:30 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:31 sergmelikyan joined #puppet-openstack
12:33 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:34 EmilienM jistr: this is ugly
12:35 jistr EmilienM: just checked, it's the same on RHEL
12:41 EmilienM jistr: patch in the gate
12:41 EmilienM jistr: CI unblocked in ~ min
12:41 jistr cool
12:44 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:47 openstackgerrit Sergey Kolekonov proposed openstack/puppet-heat: Add an ability to manage use_stderr parameter  https://review.openstack.org/210377
12:48 openstackgerrit Sergey Kolekonov proposed openstack/puppet-ceilometer: Add an ability to manage use_stderr parameter  https://review.openstack.org/210376
12:49 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 joined #puppet-openstack
12:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Put install_modules.sh in / to unlock CI jobs"  https://review.openstack.org/210380
12:51 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:56 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:57 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
13:04 openstackgerrit Sergey Kolekonov proposed openstack/puppet-keystone: Add an ability to manage use_stderr parameter  https://review.openstack.org/210403
13:05 openstackgerrit Sergey Kolekonov proposed openstack/puppet-sahara: Add an ability to manage use_stderr parameter  https://review.openstack.org/210404
13:09 tiswanso joined #puppet-openstack
13:10 xiaohua joined #puppet-openstack
13:12 logan2 joined #puppet-openstack
13:13 EmilienM pabelanger, jistr: I think Puppetfile is not found too
13:15 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:16 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 joined #puppet-openstack
13:17 EmilienM actually, CI is fixed now I guess. Though my changes are not ready yet
13:18 clayton spredzy: sounds like we should talk about notify behavior for db::sync classes when you're around
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:22 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:23 EmilienM pabelanger: I totally get what you mean
13:24 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:25 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Revert "Initial run_tests.sh script""  https://review.openstack.org/210412
13:30 xiaohua joined #puppet-openstack
13:35 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Remove httpchk option from haproxy listeners  https://review.openstack.org/197904
13:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Move intall_modules.sh to tools directory  https://review.openstack.org/210380
13:38 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 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:40 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:41 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:43 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:44 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:46 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 joined #puppet-openstack
13:47 dfisher joined #puppet-openstack
13:47 EmilienM pabelanger: we still need a repo to host our script that installs puppet modules with zuul
13:48 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:49 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:51 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 joined #puppet-openstack
13:51 pabelanger https://etherpad.openstack.org/p/puppet-modules-global-requirements
13:52 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:53 dfisher do they just get installed and are then unused?
13:53 xingchao joined #puppet-openstack
14:00 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Remove httpchk option from haproxy listeners  https://review.openstack.org/197904
14:02 sergmelikyan joined #puppet-openstack
14:03 openstackgerrit Merged openstack/puppet-openstack-integration: Revert "Initial run_tests.sh script"  https://review.openstack.org/210411
14:06 EmilienM should be fix now
14:10 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Run CI test - do not merge  https://review.openstack.org/210454
14:10 xiaohua joined #puppet-openstack
14:12 richm joined #puppet-openstack
14:21 iurygregory hello people =)
14:23 tiswanso joined #puppet-openstack
14:24 pabelanger sbadia: crinkle https://github.com/puppet-community/modulesync/pull/53
14:25 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:29 ferest joined #puppet-openstack
14:34 EmilienM pabelanger: https://review.openstack.org/210472
14:36 EmilienM pabelanger, jistr: CI is fixed for real now...
14:37 EmilienM I did all needed rechecks
14:37 jistr EmilienM: thanks!
14:41 mdorman joined #puppet-openstack
14:44 tiswanso joined #puppet-openstack
14:44 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release  https://review.openstack.org/209286
14:45 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:46 ferest joined #puppet-openstack
14:47 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: acceptance: bump to Liberty release  https://review.openstack.org/209287
14:48 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:49 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: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:52 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:53 crinkle I like the idea of adding a couple of modules as canary gate jobs to the integration repo
14:54 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:55 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:56 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:57 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:58 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
15:00 pabelanger crinkle: caught?
15:00 crinkle pabelanger: we would have caught the error in the patch
15:01 pabelanger crinkle: right, but we didn't.  Only after it merged.
15:01 EmilienM let's create a canary job, it sounds good
15:02 EmilienM crinkle: this job would be run for integ repo or also all p-* ?
15:03 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 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:04 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:05 crinkle pabelanger: okay, let's push it down the road then :)
15:06 pabelanger meh
15:06 crinkle i'd rather take things as they come than try to prepare for the unknown
15:08 EmilienM crinkle: http://goo.gl/Xwl36O
15:09 EmilienM I guess this approach is a good start
15:10 tiswanso 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:11 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:12 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:13 openstackgerrit Merged openstack/puppet-neutron: Jumbo frames between instances  https://review.openstack.org/199999
15:14 pabelanger EmilienM: it doesn't have to be msync, the Puppetfile needs to live upstream to both puppet-openstack-integration and puppet-* modules
15:15 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:16 pabelanger to me, if openstack python is copying files from there requirements project, don't see why puppet cannot
15:16 xiaohua 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:17 xingchao joined #puppet-openstack
15:19 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:20 aimon joined #puppet-openstack
15:22 pabelanger crinkle: ack
15:24 darkxploit joined #puppet-openstack
15:26 ducttape_ joined #puppet-openstack
15:27 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: acceptance: bump to Liberty release  https://review.openstack.org/209286
15:29 dtalmage_ joined #puppet-openstack
15:29 dtalmage_ left #puppet-openstack
15:31 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:33 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: acceptance: bump to Liberty release  https://review.openstack.org/209291
15:34 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:36 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:39 crinkle EmilienM: yes
15:40 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:41 EmilienM dprince: ping - https://review.openstack.org/#/c/207861/
15:41 logan2 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:44 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:45 EmilienM crinkle: I was happy my patch almost worked out of the box actually
15:46 crinkle EmilienM: :D
15:49 aimon joined #puppet-openstack
15:53 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: drop neutron_plugin_ovs type/provider  https://review.openstack.org/210506
15:55 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:59 linkedinyou 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
16:05 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:08 iurygregory Hey EmilienM, if you have make the changes you required in the spec, when you have please take a look =)
16:09 EmilienM iurygregory: will look thx
16:20 xiaohua joined #puppet-openstack
16:40 EmilienM w00t - 5 blockers and liberty is supposed to work everywhere https://etherpad.openstack.org/p/puppet-libery-blocker
16:40 xiaohua joined #puppet-openstack
16:40 dprince EmilienM: ack, thanks for fixing that
16:41 EmilienM dprince: my question is, how could it pas beaker tests before? they are running kilo
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-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:44 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:45 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:46 openstackgerrit Sergey Kolekonov proposed openstack/puppet-ceilometer: Add an ability to manage use_stderr parameter  https://review.openstack.org/210376
16:48 pabelanger EmilienM: sbadia: this is required: https://github.com/puppet-community/modulesync/pull/53
16:48 EmilienM crinkle might want to see also ^
16:49 EmilienM pabelanger: looks cool
16:51 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:53 pabelanger then we add a propose-puppet-modulesync-configs-updates post job and we'll be laughing
16:57 openstackgerrit Paul Belanger proposed openstack/puppet-modulesync-configs: Create sync-projects.sh script  https://review.openstack.org/210528
17:02 xiaohua joined #puppet-openstack
17:03 imcsk8 joined #puppet-openstack
17:09 xarses joined #puppet-openstack
17:14 openstackgerrit Merged openstack/puppet-sahara: Creation of sahara::db::sync  https://review.openstack.org/196683
17:15 openstackgerrit Merged openstack/puppet-nova: Creation of nova::db::sync  https://review.openstack.org/196679
17:17 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:18 EmilienM crinkle: though I wonder if it's related to the unscheduled refresh of glance-api
17:19 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:20 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:21 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 joined #puppet-openstack
17:23 EmilienM clayton: can you revisit again https://review.openstack.org/#/c/172580/ please ?
17:24 clayton looking
17:25 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:26 EmilienM crinkle: rebasing?
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:27 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:28 crinkle okay it should be good then
17:41 xingchao joined #puppet-openstack
17:43 xiaohua joined #puppet-openstack
17:47 openstackgerrit Merged openstack/puppet-glance: Use OpenstackClient for glance_image auth  https://review.openstack.org/172580
18:00 EmilienM crinkle: wOOt ^
18:00 EmilienM crinkle: https://review.openstack.org/#/c/209320/ pass CI, maybe you can look
18:02 xiaohua joined #puppet-openstack
18:09 iurygregory left #puppet-openstack
18:09 iurygregory joined #puppet-openstack
18:12 clayton is Denis Egorenko usually on here?  not sure what his irc nick would be
18:22 EmilienM clayton: degorenko
18:22 EmilienM clayton: but offline now
18:23 xiaohua joined #puppet-openstack
18:36 EmilienM clayton: also https://review.openstack.org/#/c/196657
18:38 openstackgerrit Richard Megginson proposed openstack/puppet-glance: support for keystone v3 api - glance  https://review.openstack.org/176150
18:43 xiaohua joined #puppet-openstack
18:58 openstackgerrit Sergey Kolekonov proposed openstack/puppet-keystone: Add an ability to manage use_stderr parameter  https://review.openstack.org/210403
19:04 xiaohua joined #puppet-openstack
19:10 openstackgerrit Merged openstack/puppet-ironic: Creation of ironic::db::sync  https://review.openstack.org/196657
19:11 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release  https://review.openstack.org/209292
19:21 EmilienM clayton: have you read dprince's email about docker?
19:22 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:24 xiaohua 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:25 EmilienM clayton: see my reply
19:25 EmilienM clayton: I talk about your patch
19:26 EmilienM clayton: in designate
19:26 clayton ah, I see.  Thanks :)
19:27 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:30 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:44 xiaohua joined #puppet-openstack
20:04 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance  https://review.openstack.org/209320
20:05 xiaohua joined #puppet-openstack
20:10 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:12 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:24 xiaohua joined #puppet-openstack
20:27 dtalmage_ joined #puppet-openstack
20:33 EmilienM same issue with sahara
20:36 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:44 xiaohua joined #puppet-openstack
21:05 xiaohua joined #puppet-openstack
21:06 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: acceptance: bump to Liberty release  https://review.openstack.org/209296
21:26 xiaohua joined #puppet-openstack
21:31 EmilienM crinkle: can you open https://review.openstack.org/#/c/209296/ ? it does not work for me
21:39 logan2 joined #puppet-openstack
21:46 xiaohua joined #puppet-openstack
21:53 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 EmilienM when I ssh the host, I run puppet, and it installs sahara
22:10 xiaohua joined #puppet-openstack
22:27 xiaohua joined #puppet-openstack
22:28 markvoelker joined #puppet-openstack
22:31 xingchao joined #puppet-openstack
22:35 pabelanger joined #puppet-openstack
22:40 ducttape_ joined #puppet-openstack
22:46 xiaohua joined #puppet-openstack
22:52 mdorman joined #puppet-openstack
23:07 xiaohua joined #puppet-openstack
23:27 xiaohua joined #puppet-openstack
23:47 xiaohua joined #puppet-openstack
23:59 markvoelker joined #puppet-openstack

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