Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-10-09

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

All times shown according to UTC.

Time Nick Message
01:36 archit joined #puppet-openstack
01:55 ilbot3 joined #puppet-openstack
01:55 Topic for #puppet-openstack is now CI Status: YELLOWish - scenarios 1/4 failing (Most jobs are fixed for Zuulv3, but we could still have edge cases) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
03:40 udesale joined #puppet-openstack
03:46 openstackgerrit Merged openstack/puppet-neutron master: Allow setting log files for apache vhost  https://review.openstack.org/510071
03:47 openstackgerrit Merged openstack/puppet-nova master: Remove nova-start  https://review.openstack.org/508754
03:55 markvoelker joined #puppet-openstack
03:58 openstackgerrit Merged openstack/puppet-nova master: Allow setting log files for nova-placement apache vhost  https://review.openstack.org/509151
03:58 openstackgerrit Merged openstack/puppet-nova master: Remove some useless parameters  https://review.openstack.org/510143
04:07 jaosorior joined #puppet-openstack
04:28 etingof joined #puppet-openstack
04:32 ratailor joined #puppet-openstack
04:33 ratailor_ joined #puppet-openstack
04:34 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-glance master: Accept empty strings for log_file  https://review.openstack.org/510404
04:37 ratailor_ joined #puppet-openstack
04:39 ratailor__ joined #puppet-openstack
04:42 ykarel_ joined #puppet-openstack
04:48 ntpttr_laptop joined #puppet-openstack
04:48 ratailor_ joined #puppet-openstack
05:05 ratailor__ joined #puppet-openstack
05:23 openstackgerrit Trygve Vea proposed openstack/puppet-nova master: Deprecate dedicated database for placement API  https://review.openstack.org/510372
05:26 markvoelker joined #puppet-openstack
05:40 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnc-backend  https://review.openstack.org/510406
05:41 ratailor_ joined #puppet-openstack
05:54 spectr joined #puppet-openstack
05:56 jtomasek joined #puppet-openstack
06:00 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/510409
06:02 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnx-backend  https://review.openstack.org/510406
06:03 udesale__ joined #puppet-openstack
06:07 openstackgerrit Merged openstack/puppet-nova master: Remove parameter nova::compute::pci_passthrough  https://review.openstack.org/510151
06:12 etingof joined #puppet-openstack
06:21 skramaja joined #puppet-openstack
06:27 yprokule joined #puppet-openstack
06:28 ykarel_ joined #puppet-openstack
06:32 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Fix README, was referring to deprecated class  https://review.openstack.org/510417
06:35 ratailor__ joined #puppet-openstack
06:40 rcernin joined #puppet-openstack
06:52 ccamacho joined #puppet-openstack
06:55 markvoelker joined #puppet-openstack
06:59 udesale joined #puppet-openstack
06:59 udesale joined #puppet-openstack
07:01 jaganathan joined #puppet-openstack
07:01 udesale joined #puppet-openstack
07:13 etingof joined #puppet-openstack
07:18 ratailor_ joined #puppet-openstack
07:25 ratailor__ joined #puppet-openstack
07:29 ratailor joined #puppet-openstack
07:55 shardy joined #puppet-openstack
07:59 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Fix README, was referring to deprecated class  https://review.openstack.org/510417
08:03 etingof joined #puppet-openstack
08:14 ykarel_ joined #puppet-openstack
08:26 markvoelker joined #puppet-openstack
08:32 derekh joined #puppet-openstack
08:34 ratailor_ joined #puppet-openstack
08:35 gfidente joined #puppet-openstack
08:36 chem joined #puppet-openstack
08:50 ratailor joined #puppet-openstack
08:51 salmankhan joined #puppet-openstack
08:55 openstackgerrit Saverio Proto proposed openstack/puppet-cinder stable/pike: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/510446
08:55 openstackgerrit Saverio Proto proposed openstack/puppet-cinder stable/ocata: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/510447
08:55 openstackgerrit Saverio Proto proposed openstack/puppet-cinder stable/newton: Read os_region_name from cinder.conf when doing requests with the Openstack client  https://review.openstack.org/510448
08:57 salmankhan joined #puppet-openstack
09:15 salmankhan joined #puppet-openstack
09:43 ccamacho joined #puppet-openstack
09:53 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/510409
09:56 markvoelker joined #puppet-openstack
10:15 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated classes for setting up backends  https://review.openstack.org/510409
10:18 * Kvisle . o O ( How did that happen? I'm in the tree for 510406. )
10:18 Kvisle Anyone with gerrit-fu who knows how to make 510406 depend on 510409? The tests in 510406 fail, because 510409 needs to be merged first.
10:21 ykarel Kvisle, you can add Depends-On: If09687d9b58c6ac8ea08a4cca94956f0e1ccbc9c in 510406
10:21 Kvisle thanks!
10:21 ykarel commit message
10:21 Kvisle I tried with git review -d 510409 -- but that didn't seem to work
10:22 ykarel Kvisle, this would checkout 510409 locally
10:23 openstackgerrit Trygve Vea proposed openstack/puppet-cinder master: Remove deprecated option from emc_vnx-backend  https://review.openstack.org/510406
11:05 jaosorior joined #puppet-openstack
11:06 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
11:14 openstackgerrit Trygve Vea proposed openstack/puppet-nova master: Deprecate dedicated database for placement API  https://review.openstack.org/510372
11:27 markvoelker joined #puppet-openstack
11:46 ansiwen[q] joined #puppet-openstack
11:51 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova master: Expose log_outputs parameter for libvirt  https://review.openstack.org/510544
12:13 udesale joined #puppet-openstack
12:21 etingof joined #puppet-openstack
12:21 markvoelker joined #puppet-openstack
12:28 ratailor joined #puppet-openstack
12:32 iurygregory joined #puppet-openstack
12:32 morazi joined #puppet-openstack
12:34 ratailor_ joined #puppet-openstack
13:01 ansmith joined #puppet-openstack
13:10 dprince joined #puppet-openstack
13:10 ratailor_ joined #puppet-openstack
13:35 ratailor__ joined #puppet-openstack
13:40 ratailor_ joined #puppet-openstack
13:40 bnemec joined #puppet-openstack
13:57 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
14:07 etingof joined #puppet-openstack
14:07 morazi joined #puppet-openstack
14:09 ratailor__ joined #puppet-openstack
14:31 amoralej mnaser, i'm removing the depends-on https://review.openstack.org/#/c/509978
14:32 amoralej as murano is disabled
14:32 openstackgerrit Alfredo Moralejo proposed openstack/puppet-openstack-integration master: Use --os-identity-api-version 3 explicitly in osc  https://review.openstack.org/509978
14:39 ntpttr_laptop joined #puppet-openstack
14:40 archit joined #puppet-openstack
14:50 mnaser amoralej yep!  thank you
14:50 mnaser amoralej i wonder if i should reapprove or it went into gate
14:51 morazi joined #puppet-openstack
14:52 amoralej mnaser, it's in gate queue
14:52 mnaser oh cool
15:44 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
15:53 ntpttr_laptop joined #puppet-openstack
15:54 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-zaqar master: Allow setting log files for apache vhost  https://review.openstack.org/510073
15:57 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
16:09 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
16:14 openstackgerrit Merged openstack/puppet-glance master: Accept empty strings for log_file  https://review.openstack.org/510404
16:14 openstackgerrit Merged openstack/puppet-horizon master: Add parameter to configure instance defaults  https://review.openstack.org/510098
16:15 jaosorior mwhahaha: could you check this out https://review.openstack.org/#/c/510066/ ?
16:34 jaosorior joined #puppet-openstack
16:40 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
16:40 jaosorior joined #puppet-openstack
16:49 EmilienM mnaser, mwhahaha: do we plan to release puppet-openstack newton/ocata/pike more often?
16:49 EmilienM I do tripleo every 2 weeks
16:49 mwhahaha no
16:50 EmilienM bou
16:50 mwhahaha would just need to evaulate the changes in puppet-tripleo to see which modules to release
16:50 mwhahaha we should not do all of them because we do not backport enough to warrant that
17:10 xarses joined #puppet-openstack
17:31 jtomasek joined #puppet-openstack
18:08 mnaser i think evaluating puppet-tripleo changes and releasing as-need-be works for everyone, it avoids the extra churn
18:10 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
18:12 Kvisle is there a term one could use for a parameter that is not deprecated, does not do anything, but may do something in the future? (this is about https://review.openstack.org/#/c/510372/, I'm the proposer)
18:35 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
18:35 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/510651
18:35 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/510652
18:35 mnaser Kvisle i would just warn that it is currently un-used and remove any mention of deprecation.  it is a very weird scenario
18:36 Kvisle mnaser: as in have a warning() whenever it's in use?
18:36 Kvisle or warning through a line in the docs?
18:36 mnaser Kvisle yes, similar to how you've done but avoid saying that it's deprecated
18:36 Kvisle ack.
18:39 Kvisle "has no effect as of pike, and may be removed in a future release"
18:49 dprince joined #puppet-openstack
18:49 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
18:49 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/510651
18:50 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/510652
18:50 mnaser Kvisle but also maybe readded according to Nova's team, that sounds okay to me though
18:51 mnaser in other good news im testing out the patch above and it might eliminate out r10k github timeouts 100% ^^ :D
18:56 openstackgerrit Trygve Vea proposed openstack/puppet-nova master: Warn when configuring dedicated database connection for placement API  https://review.openstack.org/510372
18:57 mwhahaha there's a deprecation function we should be using instead of a warning
18:57 mwhahaha jfyi
18:57 mwhahaha https://github.com/puppetlabs/puppetlabs-stdlib#deprecation
18:58 Kvisle oh, TIL
18:59 Kvisle ... this is however, not a deprecation warning, though.
18:59 Kvisle but nice to know.
18:59 mwhahaha yea just incase it was
19:00 * Kvisle have had two patches approved today, maybe not wait almost a year until next time.
19:07 mnaser Kvisle fyi no need to recheck zuul :) the only failing job was a tripleo job and i think tripleo folks are working to get those fixed up
19:07 mnaser zuul isn't voting at the moment anyawys
19:07 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Switch R10K external module installation to Ansible  https://review.openstack.org/510611
19:07 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Drop unnecessary clean-ups for Puppet  https://review.openstack.org/510651
19:07 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Fix MANAGE_REPOS for Debian-based OS  https://review.openstack.org/510652
19:07 Kvisle mnaser: it doesn't count? I saw a big '-1' on the V, so I assumed it counted.
19:08 mnaser Kvisle nope, because infra rolled back the deployment of zuulv3, it is currently not meaning much other than "hey your zuul jobs are ok"
19:08 mnaser it's a bit confusing for reviewing code i agree, cause you dont know if the -1 is really zuul failing or jenkins
19:09 Kvisle cool!
19:25 mnaser EmilienM https://review.openstack.org/#/c/508170/ can i borrow a +w?
19:25 openstackgerrit Mohammed Naser proposed openstack/puppet-cinder stable/pike: Always set keymgr_api_class  https://review.openstack.org/510665
19:25 EmilienM mnaser: sure thing
19:26 mnaser the -1s left by zuulv3 make finding things +1d by jenkins so much harder
19:26 EmilienM mnaser: done
19:26 mnaser thank you
19:26 openstackgerrit Mohammed Naser proposed openstack/puppet-cinder stable/ocata: Always set keymgr_api_class  https://review.openstack.org/510666
19:27 openstackgerrit Mohammed Naser proposed openstack/puppet-cinder stable/newton: Always set keymgr_api_class  https://review.openstack.org/510667
19:42 mnaser oh this is so awesome
19:44 mnaser with that r10k + ansible patch, we make 5 attempts with a 5 minute timeout, which means that we have a chance to recover github timeouts
19:44 mnaser and if it fails completetly then its only 25 minutes to failure, not JOB_TIMEOUT
19:48 mwhahaha that's handy
19:55 openstackgerrit Trygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/510675
19:58 openstackgerrit Trygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/510675
20:19 xarses joined #puppet-openstack
20:22 andreww joined #puppet-openstack
20:22 morazi joined #puppet-openstack
21:49 ansmith joined #puppet-openstack
21:55 openstackgerrit Merged openstack/puppet-cinder master: Configure *_domain_name to Default by default  https://review.openstack.org/510119
22:06 etingof joined #puppet-openstack
22:32 openstackgerrit Trygve Vea proposed openstack/puppet-neutron master: Deprecate parameters deprecated in Ocata  https://review.openstack.org/510675
23:01 etingof joined #puppet-openstack
23:58 openstackgerrit Merged openstack/puppet-cinder master: Allow setting log files for apache vhost  https://review.openstack.org/510066
23:59 xarses joined #puppet-openstack

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