Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-06-30

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

All times shown according to UTC.

Time Nick Message
00:05 beddari joined #puppet-openstack
00:10 gchamoul joined #puppet-openstack
00:15 beddari joined #puppet-openstack
00:16 openstackgerrit Matteus Sthefano Leite da Silva proposed openstack/puppet-barbican: Add Xenial to operatingsystemrelease  https://review.openstack.org/335723
00:24 openstackgerrit Caio Oliveira proposed openstack/puppet-ironic: Remove 12.04 and add 16.04 to metadatajson  https://review.openstack.org/335727
00:26 beddari_ joined #puppet-openstack
00:29 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-cinder: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335732
00:35 beddari joined #puppet-openstack
00:42 ducttape_ joined #puppet-openstack
00:44 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-congress: Add Xenial to metadata  https://review.openstack.org/335735
00:46 beddari joined #puppet-openstack
00:56 beddari joined #puppet-openstack
01:04 beddari joined #puppet-openstack
01:09 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-glance: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335741
01:11 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: Add Xenial to metadata  https://review.openstack.org/335742
01:14 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-heat: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335743
01:14 openstackgerrit Matteus Sthefano Leite da Silva proposed openstack/puppet-designate: Add Xenial to operatingsystemrelease  https://review.openstack.org/335744
01:14 beddari joined #puppet-openstack
01:15 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-horizon: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335745
01:16 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335746
01:18 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-manila: Add Xenial to metadata  https://review.openstack.org/335747
01:23 ducttape_ joined #puppet-openstack
01:25 beddari joined #puppet-openstack
01:26 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-mistral: Add Xenial to metadata  https://review.openstack.org/335752
01:28 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-neutron: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335754
01:35 xingchao joined #puppet-openstack
01:35 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-octavia: Update metadata  https://review.openstack.org/335756
01:35 beddari joined #puppet-openstack
01:37 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Add Xenial to metadata  https://review.openstack.org/335757
01:38 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Add Xenial to metadata  https://review.openstack.org/335758
01:39 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack_extras: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335759
01:41 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstacklib: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335760
01:46 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-oslo: Add Xenial to metadata  https://review.openstack.org/335761
01:46 beddari joined #puppet-openstack
01:47 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-rally: Update metatada  https://review.openstack.org/335763
01:55 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Remove 12.04 and add 16.04  https://review.openstack.org/335766
01:55 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-tempest: Remove 12.04 and add 16.04  https://review.openstack.org/335767
01:56 beddari joined #puppet-openstack
01:58 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-tacker: Update metadata  https://review.openstack.org/335768
02:00 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-trove: Remove 12.04 and add 16.04 from metadata  https://review.openstack.org/335769
02:05 beddari joined #puppet-openstack
02:07 ducttape_ joined #puppet-openstack
02:09 _ducttape_ joined #puppet-openstack
02:10 EmilienM iurygregory: do you have progress on authtoken thing?
02:10 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Support Xenial  https://review.openstack.org/335770
02:12 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-watcher: Update metadata  https://review.openstack.org/335772
02:13 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-zaqar: Add xenial to metadata  https://review.openstack.org/335773
02:13 iurygregory_ EmilienM, waiting for reviews from keystone people
02:14 EmilienM ayoung: ^
02:14 ayoung U hoh
02:14 iurygregory_ i talked to my friend in HP to review =)
02:15 ayoung Um...which one?  iurygregory_ has filled this page
02:15 iurygregory_ ayoung, 1min
02:15 iurygregory_ ayoung, https://review.openstack.org/#/c/328620/
02:15 beddari joined #puppet-openstack
02:16 ayoung -2 A...godd to go?
02:16 EmilienM thx Adam :)
02:16 iurygregory_ O.o what?
02:16 iurygregory_ -2 omg kkkkkkkkk
02:16 ayoung Heh
02:16 ayoung I don't have +2/-2 on puppet-keystone
02:16 ayoung sorry
02:17 iurygregory_ maybe one day you will have ayoung o/
02:17 iurygregory_ EmilienM, sorry for the spam =P
02:17 ayoung iurygregory_, don't you threaten me!
02:17 EmilienM iurygregory_: denis let comments
02:17 EmilienM ok I'm afk now
02:18 EmilienM iurygregory_: please let me know blockers, etc, maybe we can do this work during newton cycle
02:18 EmilienM thx
02:18 EmilienM good night folks
02:18 ayoung iurygregory_, explain this to me...hows it work?
02:18 iurygregory_ EmilienM, good night
02:18 iurygregory_ ayoung, ok let's go o/
02:19 iurygregory_ this class hould configure the section keystone_authtoken  for other services (glance nova etc)
02:19 iurygregory_ we need to make sure we have all necessary options
02:20 iurygregory_ we add all parameters according to http://docs.openstack.org/developer/keystonemidd​leware/middlewarearchitecture.html#configuration
02:21 iurygregory_ but in this docs we have some examples from things that maybe are deprecated but still have effect i think =)
02:24 iurygregory_ also looking at nova options in http://docs.openstack.org/mitaka/config-reference​/compute/config-options.html#nova-authentication i see "admin_<option>" but in the keystone doc i don't see =(
02:25 beddari joined #puppet-openstack
02:34 xarses joined #puppet-openstack
02:34 iurygregory_ ayoung, did you understood? .-.
02:35 beddari joined #puppet-openstack
02:36 ayoung iurygregory_, one sec
02:36 iurygregory_ ayoung, sure =)
02:36 vinsh joined #puppet-openstack
02:37 ducttape_ joined #puppet-openstack
02:45 beddari joined #puppet-openstack
02:51 ayoung iurygregory_, so this is going to be used for all of the services, or just Keystone?
02:51 iurygregory_ all services =)
02:52 iurygregory_ glance nova ironic ...
02:52 iurygregory_ :D
02:53 ayoung iurygregory_, any drawbacks?
02:53 ayoung iurygregory_, I wish this could be automated.  I hate having to visually check to see that it is right.
02:54 iurygregory_ ayoung, about drawbacks i don't think we  have, it's hard to read all parameters .-.
02:55 ayoung could we autogenerate these or check somehow?  How does testing work for these?
02:56 beddari joined #puppet-openstack
02:57 ayoung iurygregory_, I'm tempted to say "run with it" and then talk with the oslo-config folks about support to try and come up with a way to confirm "running puppet apply should set exactly the values that oslo-config expects in this case"
02:58 iurygregory_ auto generate i don't have any idea, to test we have added each param in tests
02:59 * iurygregory_ thinking..
03:01 iurygregory_ i wish i had documentation with  what are cases and what configurations are expected XD
03:03 ayoung iurygregory_, so, apachectl -t will tell you fid syntax is good.  Maybe something like that?
03:04 ayoung use puppet to generate a config file, then run something to confirm the config
03:04 iurygregory_ oh i see =)
03:05 beddari joined #puppet-openstack
03:06 markvoelker joined #puppet-openstack
03:06 iurygregory_ not 100% automated but i can apply some manifests using the define and see if keystone is working fine?
03:07 ayoung so, I can't see anything needing domain_name, as that would be for domain_scoped tokens, and everything out there use project scoped tokens.  Not sure why that is even in middleware
03:08 ayoung default_domain_name might be useful for v2 to v3 compat, but Keystone server knows that, and I would not expect that to be used eiother
03:08 ayoung catfile and certfile are pki tokens...deprecated would not add in now but meh
03:08 iurygregory_ ayoung, ack =)
03:08 ayoung looks right, though
03:09 iurygregory_ tomorrow i'll update a patch and i'll try to see if i can run some tests with configurations =)
03:09 ayoung god I hate that we have to have sections like this: 'keystone_authtoken/memcache_use_advanced_pool'     => {'value' => $memcache_use_advanced_pool},
03:09 ayoung endless copy of lists of parameters
03:09 iurygregory_ yeah kkkkk
03:09 iurygregory_ it's the life =(
03:10 ayoung its puppet, and I don't like it
03:10 ayoung $::os_service_default.  what is that anyway?
03:10 ayoung I'd seriously rather copy config files around than the stuff we do with hiera and metadata
03:10 iurygregory_ well some parameters have default values in configuration right? you don't need to explicit set
03:11 ayoung so puppet does not know what they are, so if the user does not specifuy..what happens?  It leaves the value out of the end document?
03:11 iurygregory_ using $::os_service_default we dont change the configuration
03:11 ayoung iurygregory_, "don
03:12 iurygregory_ because the user wants the default =)
03:12 ayoung don't change means that if someone were to edit the file by hand, what happens?
03:12 iurygregory_ he should not do this because the file is managed by puppet LOL
03:13 iurygregory_ what will happens is that the user can break but is his fault o.o
03:13 iurygregory_ if he wants to change he should apply a manifest and put the value for the parameter
03:14 iurygregory_ i don't know about hiera =(
03:16 beddari joined #puppet-openstack
03:23 beddari joined #puppet-openstack
03:23 ayoung iurygregory_, what do you mean "I don't know about hiera?"  That statement scares me more than anything
03:24 iurygregory_ ayoung, i never used =P
03:24 ayoung iurygregory_, I understand that "he should not" i was asking what puppet would do
03:24 ayoung iurygregory_, how do you use puppet, then?
03:25 iurygregory_ puppet apply .-.
03:25 ayoung iurygregory_, that seems like a complete waste of time
03:25 ayoung just edit the damn files
03:25 iurygregory_ lol
03:25 ayoung config files are intentionally suppused to be lowest common denominator
03:26 ayoung adding in all these layers of indirection make my life harder
03:26 iurygregory_ maybe i should take a look at https://docs.puppet.com/hiera/3.1/complet​e_example.html#what-can-we-do-with-hiera XD
03:26 xingchao_ joined #puppet-openstack
03:26 ayoung I make somethine work in keystone, and then find that someone changed the default in Puppet ... happend a few years back, and I've never forgiven puppet
03:26 ayoung iurygregory_, so, I have beeen workgin with TripHellO!
03:27 iurygregory_ ayoung, please forgive puppet =P
03:27 iurygregory_ give another chance =D
03:27 ayoung (need to say Hello that like Mike Meyers)
03:27 ayoung iurygregory_, I have no choice
03:27 iurygregory_ sahusahsuah
03:27 ayoung anyway, I am trying to figure out how Tripleo works, and I think that like, 4 people actually understand it
03:27 ducttape_ joined #puppet-openstack
03:28 iurygregory_ only 4?
03:28 iurygregory_ O.o
03:28 ayoung but, near as I can tell, heat generates a huge chunk of hiera data, sticks it in a metadata server, and then os-collect-config pulls it off the metadata server and sticks it in the local file system, then some other process does the hiera-to-puppet-apply trick
03:29 ayoung iurygregory_, steve hardy is the only one I know of for certain, but I suspect Dan Prince does as well.  beyond that the couple people i thought knew it claimed not to
03:30 iurygregory_ ayoung, omg
03:30 ayoung iurygregory_, I started trying to map it out here:
03:30 ayoung http://adam.younglogic.com​/2016/06/overcloud-deploy/
03:31 ayoung "os-collect-config never runs puppet, it runs os-refresh-config only, which then invokes a script that runs puppet."
03:31 iurygregory_ ayoung, thanks for the link :D
03:32 iurygregory_ it's magic \o/
03:32 ayoung iurygregory_, So you can see the complain I have about copying endless lists of parameters around is multiplied
03:33 iurygregory_ yeah .-.
03:34 iurygregory_ going to sleep o/ have good night ayoung tks for your time =)
03:34 ayoung iurygregory_, it does make me wonder if that method of posting to metadata ->os-collect-config->os-refres​h-config->script->puppet-apply is sane
03:34 ayoung iurygregory_, anyway, address the other -1 issue, but I'll give this review a +1
03:35 iurygregory_ tks ayoung o/
03:35 iurygregory_ good luck in TripleO o/
03:36 ayoung Trip HELLO!
03:36 beddari joined #puppet-openstack
03:44 beddari joined #puppet-openstack
03:55 beddari joined #puppet-openstack
04:04 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-octavia: Update metadata  https://review.openstack.org/335756
04:05 beddari_ joined #puppet-openstack
04:07 markvoelker joined #puppet-openstack
04:08 vinsh joined #puppet-openstack
04:11 xingchao joined #puppet-openstack
04:16 beddari joined #puppet-openstack
04:25 beddari joined #puppet-openstack
04:35 beddari_ joined #puppet-openstack
04:39 vinsh joined #puppet-openstack
04:45 beddari joined #puppet-openstack
04:55 beddari_ joined #puppet-openstack
05:05 beddari_ joined #puppet-openstack
05:12 sanjayu joined #puppet-openstack
05:15 beddari joined #puppet-openstack
05:26 beddari_ joined #puppet-openstack
05:26 rcernin joined #puppet-openstack
05:36 beddari joined #puppet-openstack
05:46 beddari joined #puppet-openstack
05:46 ducttape_ joined #puppet-openstack
05:56 beddari joined #puppet-openstack
06:04 jmikolajczak joined #puppet-openstack
06:09 markvoelker joined #puppet-openstack
06:09 rcernin joined #puppet-openstack
06:09 vinsh joined #puppet-openstack
06:16 beddari_ joined #puppet-openstack
06:17 openstackgerrit Andreas Jaeger proposed openstack/puppet-openstack-release-tools: List system dependencies for running common tests  https://review.openstack.org/335806
06:20 xingchao joined #puppet-openstack
06:26 beddari joined #puppet-openstack
06:36 beddari joined #puppet-openstack
06:43 openstackgerrit zhongshengping proposed openstack/puppet-ec2api: Add purge_config option  https://review.openstack.org/335812
06:45 beddari joined #puppet-openstack
06:48 ducttape_ joined #puppet-openstack
06:51 shardy joined #puppet-openstack
06:56 beddari joined #puppet-openstack
07:02 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists?  https://review.openstack.org/335600
07:06 beddari_ joined #puppet-openstack
07:10 markvoelker joined #puppet-openstack
07:15 ifarkas joined #puppet-openstack
07:15 tesseract- joined #puppet-openstack
07:16 beddari joined #puppet-openstack
07:26 beddari joined #puppet-openstack
07:36 beddari joined #puppet-openstack
07:38 xingchao joined #puppet-openstack
07:42 danpawlik joined #puppet-openstack
07:46 beddari joined #puppet-openstack
07:50 ducttape_ joined #puppet-openstack
07:56 beddari_ joined #puppet-openstack
08:04 gfidente joined #puppet-openstack
08:04 derekh joined #puppet-openstack
08:06 beddari joined #puppet-openstack
08:10 markvoelker joined #puppet-openstack
08:12 vinsh joined #puppet-openstack
08:16 beddari joined #puppet-openstack
08:19 xingchao joined #puppet-openstack
08:25 beddari joined #puppet-openstack
08:33 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
08:34 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Nova API and Neutron server should run database creation in step 3  https://review.openstack.org/335419
08:35 beddari joined #puppet-openstack
08:36 xingchao joined #puppet-openstack
08:38 dgurtner joined #puppet-openstack
08:38 dgurtner joined #puppet-openstack
08:45 beddari_ joined #puppet-openstack
08:50 ducttape_ joined #puppet-openstack
08:56 beddari joined #puppet-openstack
08:57 imcsk8_ joined #puppet-openstack
09:00 openstackgerrit Daniel Pawlik proposed openstack/puppet-keystone: Add misssing parameters to keystone_authtoken  https://review.openstack.org/328620
09:00 bapalm joined #puppet-openstack
09:04 abramley_ joined #puppet-openstack
09:06 beddari joined #puppet-openstack
09:09 Kvisle joined #puppet-openstack
09:14 ericksonsantos joined #puppet-openstack
09:14 bkero joined #puppet-openstack
09:15 iurygregory joined #puppet-openstack
09:16 beddari joined #puppet-openstack
09:21 partner joined #puppet-openstack
09:25 xingchao joined #puppet-openstack
09:26 beddari joined #puppet-openstack
09:34 psanchez joined #puppet-openstack
09:34 amoralej bkero, any objection to cherry-pick https://review.openstack.org/#/c/333406/ to stable/mitaka?
09:36 beddari_ joined #puppet-openstack
09:46 xingchao joined #puppet-openstack
09:46 beddari_ joined #puppet-openstack
09:48 chem hi, I really don't understand what happen here https://review.openstack.org/#/c/335600/
09:49 chem it looks like python-pysaml2 is installed over and over again
09:49 chem can't get it. It's installed there  http://logs.openstack.org/00/335600/4/gate/gate​-puppet-keystone-puppet-beaker-rspec-centos-7/a​4df69e/console.html#_2016-06-30_02_46_40_985265
09:50 chem and then reinstall on the idempotency test again : http://logs.openstack.org/00/335600/4/gate/gate​-puppet-keystone-puppet-beaker-rspec-centos-7/a​4df69e/console.html#_2016-06-30_02_48_18_178835
09:50 chem does anybody has already seen anything like that ?
09:52 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: DEBUG: test ci  https://review.openstack.org/335910
09:55 beddari joined #puppet-openstack
10:06 beddari joined #puppet-openstack
10:10 vinsh joined #puppet-openstack
10:13 degorenko iurygregory, EmilienM do we need to drop 14.04 for master here https://review.openstack.or​g/#/q/topic:update_metadata ?
10:14 xb_ joined #puppet-openstack
10:16 beddari joined #puppet-openstack
10:17 openstackgerrit Alfredo Moralejo proposed openstack/puppet-tempest: Allow virtuals during python-setuptools install  https://review.openstack.org/335918
10:26 beddari joined #puppet-openstack
10:31 degorenko chem, hey
10:31 degorenko strange error
10:32 degorenko and i don't see any notifies for this packages, which can trigger restart of installation
10:32 degorenko chem, haev you tried to reproduce this issue localy? May be there are some other changes in puppet or errors on first run
10:36 beddari joined #puppet-openstack
10:42 openstackgerrit Merged openstack/puppet-ec2api: Add purge_config option  https://review.openstack.org/335812
10:42 chem joined #puppet-openstack
10:46 aernhart joined #puppet-openstack
10:46 beddari joined #puppet-openstack
10:47 aernhart joined #puppet-openstack
10:54 partner joined #puppet-openstack
10:56 beddari joined #puppet-openstack
10:59 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack_extras: Assume Debian if no osfamily  https://review.openstack.org/335665
10:59 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack_extras: Assume Debian if no osfamily  https://review.openstack.org/335665
11:06 beddari joined #puppet-openstack
11:12 aderyugin joined #puppet-openstack
11:14 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack_extras: Assume Debian if no osfamily  https://review.openstack.org/335665
11:16 beddari joined #puppet-openstack
11:21 degorenko chem, same error on another patch https://review.openstack.org/#/c/328620/ as your
11:24 openstackgerrit Merged openstack/puppet-neutron: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335754
11:26 beddari joined #puppet-openstack
11:29 ducttape_ joined #puppet-openstack
11:30 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues  https://review.openstack.org/335960
11:35 beddari joined #puppet-openstack
11:40 vinsh joined #puppet-openstack
11:46 paramite joined #puppet-openstack
11:46 beddari joined #puppet-openstack
11:51 EmilienM hello
11:52 degorenko hey
11:52 degorenko Hunner, ping
11:54 beddari joined #puppet-openstack
11:57 johnmilton joined #puppet-openstack
12:06 beddari joined #puppet-openstack
12:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
12:06 EmilienM mwhahaha: we should be able to promote today ^
12:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Test https://review.openstack.org/#/c/334209/"  https://review.openstack.org/335016
12:08 openstackgerrit Merged openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues  https://review.openstack.org/335960
12:13 markvoelker joined #puppet-openstack
12:15 markvoelker joined #puppet-openstack
12:16 beddari_ joined #puppet-openstack
12:17 openstackgerrit Caio Oliveira proposed openstack/puppet-ironic: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335727
12:20 johnmilton joined #puppet-openstack
12:20 vinsh joined #puppet-openstack
12:20 iurygregory degorenko, i asked mwhahaha maybe we should drop before release Newton
12:20 iurygregory morning guys
12:23 ericksonsantos joined #puppet-openstack
12:26 beddari joined #puppet-openstack
12:35 beddari joined #puppet-openstack
12:37 ericksonsantos joined #puppet-openstack
12:43 openstackgerrit Merged openstack/puppet-tripleo: Remove base/neutron/ml2.pp  https://review.openstack.org/332870
12:46 beddari joined #puppet-openstack
12:47 jmikolajczak left #puppet-openstack
12:48 chem So, CI is broken for puppet-keystone: https://bugs.launchpad.net/p​uppet-keystone/+bug/1597753
12:48 openstack Launchpad bug 1597753 in puppet-keystone "Broken CI: python-pysaml2 is always re-installed" [Undecided,New]
12:49 mdavidson joined #puppet-openstack
12:51 openstackgerrit Merged openstack/puppet-tripleo: neutron plugins: include neutron base  https://review.openstack.org/332897
12:51 iurygregory chem, yeah i saw that =(
12:52 iurygregory did you check locally?
12:53 openstackgerrit Merged openstack/puppet-tripleo: Nova API and Neutron server should run database creation in step 3  https://review.openstack.org/335419
12:53 EmilienM chem: nice shot
12:54 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/315527
12:56 beddari joined #puppet-openstack
12:57 iurygregory EmilienM,  fyi barbican CI is also broken i think https://review.openstack.org/#/c/335723/
12:57 openstackgerrit Giulio Fidente proposed openstack/puppet-tripleo: Default ceph_osd_selinux_permissive to false  https://review.openstack.org/333326
12:58 ducttape_ joined #puppet-openstack
12:58 EmilienM iurygregory: ack
12:59 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: Use Debian repositories instead of UCA  https://review.openstack.org/331195
13:05 chem for the broken CI #rdo folks are investigating a possible issue with the repo definition
13:05 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: keystone: deploy composable firewall rules  https://review.openstack.org/330785
13:09 openstackgerrit Dan Radez proposed openstack/puppet-congress: Populating init.pp and parameters  https://review.openstack.org/330195
13:16 beddari joined #puppet-openstack
13:24 johnmilton joined #puppet-openstack
13:26 beddari joined #puppet-openstack
13:36 beddari joined #puppet-openstack
13:37 degorenko mwhahaha, according to https://review.openstack.org/#/c/335665/4/​spec/acceptance/openstack_extras_repo_spec.rb
13:37 degorenko have you tried to use require 'facter' and then Facter.value?
13:37 degorenko or we can call bash facter :D
13:37 mwhahaha not sure how that works with beaker
13:38 mwhahaha the issue is, http://logs.openstack.org/65/335665/4/c​heck/gate-puppet-openstack_extras-puppe​t-beaker-rspec-ubuntu-xenial/73d3af8/co​nsole.html#_2016-06-30_11_24_11_762280
13:38 mwhahaha was trying to work around it to let the fact just fail but then i managed to break centos :D
13:38 mwhahaha i'll take some time to look at it later today
13:39 degorenko mwhahaha, http://paste.openstack.org/show/524268/
13:39 degorenko this can work probably
13:39 mwhahaha but that's when you're running rspec
13:40 mwhahaha i thought beaker worked differently
13:40 mwhahaha could try it though
13:40 degorenko mwhahaha, also http://paste.openstack.org/show/524269/ works
13:40 EmilienM mwhahaha: working on promoting our repos today, only scenario003 is blocking now
13:40 mwhahaha the system call won't work due to the underlying pathing issues
13:41 mwhahaha since the puppetlabs repo installs in /opt/puppetlabs and doesn't update the PATH
13:41 degorenko then i will try test it with beaker
13:41 mwhahaha the unfortunate thing is it's a nodepool issue
13:41 mwhahaha it works fine locally :D
13:41 degorenko as i remember, there was some way to run beaker tests right on local machine
13:43 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/328347
13:44 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: nova/compute: add NFS bits  https://review.openstack.org/332954
13:46 beddari joined #puppet-openstack
13:48 ayoung joined #puppet-openstack
13:50 emccormick joined #puppet-openstack
13:51 mwhahaha chem: is pysaml2 a virtual package now? i thought i saw something similar where if it's a virtual package it messes with idempotency
13:51 degorenko mwhahaha, afaik, it was for python-setuptools
13:51 degorenko or something like that
13:51 mwhahaha yea just wondering if python-pysaml2 got converted or something :D
13:51 degorenko and it was set to virtual :-)
13:52 mwhahaha cause centos is failing on his keystone patch cause the package keeps getting reinstalled
13:52 mwhahaha http://logs.openstack.org/00/335600/5/check/gate​-puppet-keystone-puppet-beaker-rspec-centos-7/e9​6497e/console.html#_2016-06-30_10_59_56_022071
13:52 chem mwhahaha: hum dunno, just than the name is python2-pysaml2 and it has a "Provides: python-pysaml2"
13:52 degorenko yeah
13:52 chem mwhahaha: does it qualify for virtual package ?
13:52 jmiu_ joined #puppet-openstack
13:52 mwhahaha i have no idea :D i wouldn't think so but just wondering
13:53 chem mwhahaha: hehe, ack I'll ask on #rdo
13:56 beddari joined #puppet-openstack
13:57 emccormick does anyone know of a good way to get puppet-nova to allow rbd cinder volumes on computes without fully enabling ephemeral storage in ceph?
13:57 emccormick it seems that the module removes the pool and config path if you don't set ephemeral to true
13:58 EmilienM what?
13:59 EmilienM emccormick: I think we support this use case, we do it in tripleo
13:59 ducttape_ joined #puppet-openstack
13:59 emccormick i  if $ephemeral_storage {
13:59 emccormick nova_config {
13:59 emccormick 'libvirt/images_type':          value => 'rbd';
13:59 emccormick 'libvirt/images_rbd_pool':      value => $libvirt_images_rbd_pool;
13:59 emccormick 'libvirt/images_rbd_ceph_conf': value => $libvirt_images_rbd_ceph_conf;
13:59 emccormick }
13:59 emccormick } else {
13:59 emccormick nova_config {
13:59 emccormick 'libvirt/images_rbd_pool':      ensure => absent;
14:00 emccormick 'libvirt/images_rbd_ceph_conf': ensure => absent;
14:00 emccormick }
14:00 emccormick sorry probably not the best way to do that but that nukes the config options
14:00 emccormick I get around it by just getting rid of that bit but obviously I'd prefer not to stay forked
14:02 EmilienM you don't need libvirt/images_rbd_* when using cinder rbd volumes
14:02 EmilienM you only need rbd user and key
14:02 EmilienM or rbd_secret_uuid
14:02 emccormick I do if my ceph config isn't /etc/ceph/ceph.conf
14:02 emccormick and my image pool isn't "images"
14:03 emccormick err
14:03 emccormick "volumes" sorry
14:03 EmilienM we don't hardcode 'volumes'
14:03 EmilienM and it's not harcoded in nova, isn't?
14:03 richm joined #puppet-openstack
14:03 emccormick I believe it's a nova default, yes
14:03 EmilienM I agree for /etc/ceph/ceph.conf though
14:04 EmilienM emccormick: feel free to submit a patch
14:04 EmilienM don't stay on your fork, please send a patch
14:04 emccormick I'll see if I can make it appealing. The way I did it it just works for the way I'm doing it
14:05 emccormick emergency hack ;)
14:06 beddari joined #puppet-openstack
14:07 EmilienM ok promotion will be hard
14:07 EmilienM trove tests does not exist anymore in tempest
14:08 EmilienM mtreinish: http://logs.openstack.org/52/334852/4/ch​eck/gate-puppet-openstack-integration-4-​scenario003-tempest-centos-7-nv/4212f38/​console.html#_2016-06-30_12_31_28_142339
14:08 EmilienM is it related with your trove work?
14:09 degorenko mwhahaha, looks like they are using bash right now for fact method:  https://github.com/puppetlabs/beaker/blob/master/​lib/beaker/dsl/helpers/facter_helpers.rb#L39-L52 https://github.com/puppetlabs/beaker/blob/​master/lib/beaker/dsl/wrappers.rb#L16-L21
14:09 mwhahaha yea cause usually it gets run on a parent host to connect to the host that it runs on
14:09 mtreinish EmilienM: probably, I think the plugin patch landed. I told them it was untested and they needed to fix it (I just pushed it as a starting point)
14:09 mwhahaha in nodepool i don't think that's necessary but i could be wrong
14:10 EmilienM we're going to align stars today
14:10 EmilienM mtreinish: it's a regression for us, as we used to test trove... but nevermind.
14:11 mtreinish EmilienM: heh, you tested trove. But trove didn't test trove :)
14:11 degorenko mwhahaha, then we can try to use native Facter module
14:11 mwhahaha but that would fail if you run beaker locally i think
14:11 * mwhahaha shrugs
14:11 EmilienM mtreinish: lol
14:11 EmilienM sad story
14:12 degorenko mwhahaha, do you know how to run beaker locally? :D
14:12 mtreinish EmilienM: that file is missing an import (assuming that's the only issue) https://github.com/openstack/trove/blob/mast​er/trove/tests/tempest/tests/api/base.py#L18
14:12 degorenko without vagrant
14:12 mwhahaha if i do a bundle exec rake acceptance, where does the .rb run?
14:12 mwhahaha i thought it ran on my OSX box
14:12 mtreinish should have from trove.tests.tempest.services.database.json import flavors_client
14:13 mtreinish err for all the clients actually. It's not importing any of them :)
14:13 degorenko mwhahaha, after bundle exec rspec spec/acceptance
14:13 degorenko i see No such file or directory - vagrant (Errno::ENOENT)
14:14 mwhahaha oh i have vagrant installed :D
14:14 degorenko boom :)
14:15 degorenko mwhahaha, can you share settings for acceptance then? :D vagrant, or something else
14:15 mwhahaha just install vagrant
14:15 mwhahaha :D
14:16 degorenko mwhahaha, yeah, but Error was No usable default provider could be found for your system.
14:16 degorenko :D
14:16 mwhahaha sooo do you not have virtualbox installed?
14:16 degorenko i have no experience with vagrant, and i have no ideas how to configure that :D
14:16 degorenko no
14:16 beddari joined #puppet-openstack
14:17 mwhahaha https://www.vagrantup.com/docs/getting-started/
14:17 mwhahaha it uses virtualbox by default or you can try and use libvirt or whatever
14:17 mwhahaha https://www.vagrantup.com/docs/​getting-started/providers.html
14:19 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True  https://review.openstack.org/336045
14:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
14:21 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Allow virtual package for python-pysaml2  https://review.openstack.org/336048
14:21 EmilienM chem: I think https://review.openstack.org/336045 can fix puppet-keystone CI
14:21 EmilienM no don't do that ^
14:22 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Allow virtual package for python-pysaml2  https://review.openstack.org/336048
14:22 EmilienM in your patch you fix the problem in ONE place, while RDO has a bunch of virtual packages
14:22 mwhahaha on second thought, tis a silly place
14:22 chem EmilienM: I did a patch in puppet-keystone to in order to have a working module even outside CI.
14:22 EmilienM chem: https://review.openstack.org/#/c/336045/ is executed outside CI
14:22 chem EmilienM: well puppet-openstack-integration is not used in production :)
14:23 emccormick <EmilienM> just fyi, this is in nova's imagebackend.py for images_rbd_pool: default='rbd',
14:23 EmilienM chem: we won't land this patch
14:23 emccormick EmilienM: so will need to take that into account also
14:23 EmilienM because it's too opinionated
14:23 EmilienM emccormick: ok cool
14:23 chem EmilienM: but it uses the openstack_integration module, which is not necessarily the case
14:24 EmilienM right but your patch is too opinionated
14:24 EmilienM puppet 4 set it to True by default
14:24 EmilienM so what you're doing is forcing everyone using puppet-keystone to allow virtual packaging
14:24 EmilienM while I prefer our user managing it themselves
14:24 chem EmilienM: nope
14:25 EmilienM the thing is that we have this packaging issue for a lot of packages, not one
14:25 chem EmilienM: when installing pysaml2 for federation I force it because without federation won't be idempotent
14:26 chem EmilienM: well, one way or another we have to fix it as we install the package
14:26 chem EmilienM: inside the module itself, I mean
14:26 EmilienM I'll remove my -2 for now, and let others give feedbac
14:26 beddari joined #puppet-openstack
14:26 EmilienM but I don't like it because we're fixing the issue only for one package
14:26 EmilienM while we have other cases
14:27 EmilienM ie: saharaclient, etc
14:27 EmilienM chem: in the meantime, I suggest you to update your patchs that needs to land with Depends-On https://review.openstack.org/#/c/336045
14:27 chem EmilienM: well so we should force virtual package to true everywhere, ie in each and every module
14:27 EmilienM chem: https://review.openstack.org/#/c/336045 will fix our CI
14:28 chem EmilienM: as my  patch :)
14:29 EmilienM mhh
14:29 rcernin joined #puppet-openstack
14:31 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists?  https://review.openstack.org/335600
14:32 EmilienM chem: I think your patch is safe to merge too
14:32 EmilienM I just want to find a way to fix it for all RDO packages
14:32 EmilienM maybe in openstacklib or something
14:32 EmilienM or in openstack extras
14:32 EmilienM when using RDO repos
14:33 chem EmilienM: I think openstacklib would be the place.  Is there one of the manifest that is loaded by every other module ?
14:35 EmilienM mhh no
14:35 mwhahaha Package<| tag == 'openstack'|> { allow_virtual => true } ?
14:35 mwhahaha still seems dangerous
14:35 EmilienM yeah
14:35 EmilienM and where putting it
14:36 beddari joined #puppet-openstack
14:37 chem mwhahaha: EmilienM maybe we should add a include ::openstacklib::default in every module to have a place to do this kind of thing ?
14:37 xarses joined #puppet-openstack
14:38 mwhahaha that's a lot of updates
14:38 chem and the package allow_virtual could be pinned to centos, and puppet version
14:38 ducttape_ joined #puppet-openstack
14:38 mwhahaha who thought using virtual packages was a good thing :D
14:42 EmilienM chem's solution would work indeed
14:42 chem mwhahaha: well ... yes, but I don't see any another way, beside adding Package {allow_virtul=>true} in each and every module (if we want a generic solution)
14:42 EmilienM I'm trying to find 1) a flexible solution 2) a work out of the box solution
14:42 EmilienM and chem's solution is 1) and 2) I think
14:43 mwhahaha yea i'm not saying it's bad per say, but it seems like we're having to solve for an upstream package decision
14:43 cdelatte joined #puppet-openstack
14:43 mwhahaha forcing yet more workarounds in puppet because something RDO has chosen to do
14:43 mwhahaha see policyrc.d
14:44 EmilienM yeah...
14:45 mwhahaha so just wondering if the usage of virtual packages was properly vetted upstream ;)
14:45 mwhahaha not sure if it's causing more issues than its solving but since i don't live in that world i can only ask
14:45 mwhahaha the ::openstacklib::defaults seems like a good idea, it's just a ton of work
14:45 mwhahaha should make the package owners do it
14:46 beddari joined #puppet-openstack
14:46 EmilienM maybe we can ask for help from RDO folks
14:47 EmilienM amoralej: are you interested to help on it? ^
14:49 * amoralej reading up...
14:51 mwhahaha tl;dr create a single class and include it everywhere so we can put in a single change to address virtual packages
14:51 EmilienM amoralej: yeah, don't fix it in packstack, it's not the best way
14:51 EmilienM imho -> fix it in puppet modules
14:54 chem mwhahaha: cd ~/Src && find . -maxdepth 3 -wholename '*/manifests/init.pp'  | xargs -n 1 sed -i.back -e "s/^}$/  include ::openstacklib::default\n}\n/"
14:54 chem mwhahaha: and you're done :)
14:54 mwhahaha no you should include it in more places than just init
14:54 mwhahaha techincally if we are doing this, it should be in every .pp
14:55 chem mwhahaha: then that's getting insane
14:55 mwhahaha now you see  my questioning this :D
14:55 mwhahaha you can't assume that someone will always include the init class
14:56 chem mwhahaha: EmilienM so the problem now is that we won't see those failure anymore as the CI will pass them ok, but the module may be broken when used outside of it because of virtual package...
14:56 beddari joined #puppet-openstack
14:56 EmilienM right
14:57 amoralej that's a good point chem
14:57 EmilienM that's why your proposal is not too bad
14:57 EmilienM mwhahaha: that's really bad practice
14:58 rcernin joined #puppet-openstack
14:59 mwhahaha not necessarily
14:59 mwhahaha techincally for this issue we need to include it wherever the packages are included
14:59 chem I tend to agree with mwhahaha, we cannot assume that init.pp will always be included
15:00 mwhahaha for this issue the minimum is that i must be included whereever there is a package definition in the modules
15:00 mwhahaha you could probably get away with including it in params
15:01 mwhahaha as i think almost all of the package definitions use some form of params
15:05 ducttape_ joined #puppet-openstack
15:05 chem mwhahaha: I've got this one now : find . -maxdepth 3 -name '*.pp'  | xargs grep -li 'package.*{' | xargs -n 1 sed -e "s/^}$/  include ::openstacklib::default\n}\n/"
15:05 chem mwhahaha: and we're done :)
15:06 mwhahaha params might be cleaner, but that's an option
15:06 beddari joined #puppet-openstack
15:06 chem mwhahaha: params is cleaner ... and in some way it's a param
15:08 chem mwhahaha: EmilienM with a script it's not a big deal to update all modules if we want to go that way : update all params.pp to include openstacklib::default
15:08 mwhahaha call it openstacklib::defaults and i'm ok
15:09 EmilienM params.pp ?
15:09 EmilienM argh
15:09 EmilienM welcome to durty world
15:09 EmilienM params.pp is not used for this purpose afik but well...
15:10 mwhahaha correct but it's the one place we include everywhere already
15:10 EmilienM yeah
15:10 EmilienM let's go for it...
15:10 amoralej let me know how can i help
15:10 mwhahaha so rather than add yet another class, lets limit scope
15:11 EmilienM amoralej: I think chem has it covered but you can sync each others if you want to spread the load
15:11 amoralej ok
15:11 amoralej so solution will be params.pp to include openstacklib::defaults
15:11 chem EmilienM: amoralej the only thing I'm missing is a clean list of all puppet modules
15:11 ducttape_ joined #puppet-openstack
15:11 amoralej wich wil turn on allow_virtual, right?
15:12 chem mwhahaha: what do you mean by limit scope ?
15:12 amoralej well, i can give you the list of packages we package in rdo
15:12 mwhahaha chem: limit what we have to update to accomplish this
15:12 EmilienM chem: curl https://raw.githubusercontent.com/​openstack/puppet-openstack-integra​tion/master/openstack_modules.txt
15:13 chem mwhahaha: ack, by using only the params.pp.. got it
15:14 chem EmilienM: that's it ?  I always though there were more ... anyway thanks.
15:15 EmilienM chem: sorry, we *only* manage 33 puppet modules
15:15 dfisher joined #puppet-openstack
15:15 chem EmilienM: hehe :)
15:16 beddari joined #puppet-openstack
15:25 beddari joined #puppet-openstack
15:35 chem EmilienM: mwhahaha in the meantime could we merge https://review.openstack.org/#/c/336048/ as it unlock the CI for puppet keystone ?
15:35 beddari joined #puppet-openstack
15:36 mwhahaha k
15:36 chem EmilienM: mwhahaha I will revert when the params.pp stuff is in the place.
15:36 EmilienM chem: ok
15:38 chem EmilienM: I think the package {virtula... } default setting doesn't work : look there https://review.openstack.org/#/c/335600/ I still have the issue while there https://review.openstack.org/#/c/336048/ it works.  So it mustn't be taken into account .
15:43 chem EmilienM: oki, so why it doesn't work :)
15:46 beddari joined #puppet-openstack
15:47 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken  https://review.openstack.org/336087
15:52 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken  https://review.openstack.org/336087
15:56 beddari joined #puppet-openstack
15:58 chem EmilienM: mwhahaha the include openstacklib::defaults won't work
15:58 mwhahaha why?
15:59 chem mwhahaha: http://paste.openstack.org/show/524298/
15:59 chem mwhahaha: EmilienM the "setting default idiom" is local
15:59 mwhahaha right you need to use...
16:00 mwhahaha Package<| tag == 'openstack'|> { allow_virtual => true }
16:01 chem mwhahaha: right this is working
16:01 mwhahaha so we'd need to put that in openstacklib::defaults
16:03 chem mwhahaha: all package are tagged with openstack ?
16:03 mwhahaha should be
16:03 mwhahaha if not that's a bug
16:03 mwhahaha also would limit the scope of who gets this allow virtual target since we only want to affect the openstack related packages
16:04 chem mwhahaha: for one those are tagged differently https://review.openstack.org/#/c/336048/​2/manifests/federation/identity_provider.pp
16:05 mwhahaha hmm so maybe we should use openstack and openstack-dependency or something
16:06 chem mwhahaha: well, now we have to go over all package to make sure they are tagged all right ...
16:06 beddari joined #puppet-openstack
16:07 mwhahaha remember when i said this is a lot of work
16:07 chem mwhahaha: hehe, right :)
16:07 chem mwhahaha: what about : we fix them as they show up like I did in the keystone package and when 3.6 support is dropped we're happy ?
16:08 chem mwhahaha: puppet>3.6 virtual package is true by default right ? or is it 4 ?
16:08 mwhahaha thought it was a 3/4 split
16:08 mwhahaha not sure
16:10 chem mwhahaha: you're right https://docs.puppet.com/puppet/3.6/re​ference/release_notes.html#changes-to​-rpm-behavior-with-virtual-packages
16:10 chem mwhahaha: from 4 on ...
16:14 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Add memcached_servers for keystone authtoken  https://review.openstack.org/336101
16:15 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken  https://review.openstack.org/336087
16:15 EmilienM chem: so https://review.openstack.org​/#/c/336045/1/manifests/init.ppis bad?
16:15 beddari joined #puppet-openstack
16:16 EmilienM we need to use the collector?
16:16 chem EmilienM: http://paste.openstack.org/show/524298/ doesn't work yes.  we need a collector.
16:16 EmilienM ack
16:16 mwhahaha should also wrap a puppet version check
16:16 EmilienM yeah?
16:16 mwhahaha so we only do it for puppet 3
16:17 EmilienM grmllll
16:17 chem EmilienM: mwhahaha but all packages are not tagged openstack
16:17 EmilienM grmmllllllll
16:17 EmilienM right
16:17 mwhahaha remember when i said i don't know if this was taken into consideration when they decided to use virtual packages...
16:18 mwhahaha ‎(ノಥ益ಥ)ノ ┻━┻
16:18 chem mwhahaha: why is it better to use virtual packages ?
16:18 chem mwhahaha: (you have amazing UTF8-fu!)
16:18 mwhahaha i have no idea, thats a question as to why it was decided to switch that out
16:19 jpena EmilienM: if I understood https://docs.puppet.com/puppet/3.6/re​ference/release_notes.html#changes-to​-rpm-behavior-with-virtual-packages correctly, shouldn't the allow_virtual thing be put in the scenarioxxx.pp files?
16:19 jpena it says "in the main manifest"
16:19 mwhahaha because then end users have to know to do it
16:20 mwhahaha that doesn't fix it for people who just pull the modules down
16:20 mwhahaha people don't deploy with the scenario files
16:20 mwhahaha that fixes CI but does not fix it for the end user
16:20 mwhahaha so the modules will no longer work out of the box :(
16:20 jpena yes, I was thinking short-term solution vs long-term one
16:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True  https://review.openstack.org/336045
16:20 EmilienM chem: better ^ ?
16:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
16:21 mwhahaha the short term is fix the package declaration that were virtualized
16:21 mwhahaha long term is figure out the openstacklib::defaults thing
16:21 mwhahaha and fix all the packages
16:21 mwhahaha er package declarations
16:21 mwhahaha in puppet
16:22 amoralej according to https://docs.puppet.com/puppet/3.6/re​ference/release_notes.html#changes-to​-rpm-behavior-with-virtual-packages, it can be declared as before, but in the main manifest
16:22 openstackgerrit Merged openstack/puppet-keystone: Allow virtual package for python-pysaml2  https://review.openstack.org/336048
16:22 amoralej the one called with puppet apply
16:22 mwhahaha it would have to be declared in a site.pp
16:22 chem EmilienM: yep, but for instance it wouldn't have solved my pysaml2 problem as it's not tagged openstack https://review.openstack.org/#/c/336048/​2/manifests/federation/identity_provider.pp
16:23 EmilienM ok
16:23 amoralej whouldn't fixtures/scenario00*.pp work
16:23 amoralej ?
16:23 mwhahaha the point is we can't just put a single instance somewhere and expect that it fixes CI and end users for all modules at the moment
16:24 amoralej yes, i was only thinking in CI
16:24 mwhahaha once again that might fix CI but not end users
16:24 EmilienM right, oslib will fix it for end users
16:26 chem EmilienM: oki, so we have to include the openstacklib::defaults *and* make sure that all package are tagged correctly
16:26 EmilienM mwhahaha: is https://review.openstack.org/336045 ok to fix CI until we sort things out in oslib?
16:26 EmilienM chem: I think so yes
16:26 beddari joined #puppet-openstack
16:27 EmilienM mwhahaha: https://review.openstack.org/#/c/336047/ is going to be merged
16:27 EmilienM which means no more blocker
16:27 EmilienM I'll update Promotion patch once it's in delorean
16:27 mwhahaha yea we can fix it with that for now
16:27 EmilienM ok
16:27 mwhahaha it might not catch everything tho
16:27 EmilienM right :(
16:27 mwhahaha since we found not all the things are tagged correctly
16:27 EmilienM we'll see bit by bit
16:27 EmilienM we'll fix tag as long as we find issues
16:27 mwhahaha you could just drop the tag as a short term fix for now
16:28 EmilienM Package<||> { allow_virtual => true }
16:28 EmilienM this ? ^
16:28 mwhahaha yea
16:28 chem EmilienM: yep
16:29 EmilienM ok
16:29 chem mwhahaha: EmilienM but then ... it's going to be hard to spot any problem :)
16:29 mwhahaha yea i wouldn't drop it just yet :D
16:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True  https://review.openstack.org/336045
16:29 mwhahaha oh well
16:29 mwhahaha heh
16:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
16:30 EmilienM ah
16:30 chem hehe
16:32 EmilienM so what?
16:32 EmilienM tag or not tag
16:33 EmilienM ok let's put tag now
16:33 chem tag
16:33 EmilienM we'll figure later
16:33 iurygregory +1 to tag
16:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True  https://review.openstack.org/336045
16:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
16:33 EmilienM cool
16:33 EmilienM afk lunch
16:33 chem I'm going to make the massive update and do my best to add the missing opentack tag to all package
16:33 iurygregory chem, need help?
16:34 chem iurygregory: for the moment no, I see where I can go with a script.  If I fail, then yes :)
16:34 iurygregory chem, ack
16:35 EmilienM cool :)
16:36 beddari joined #puppet-openstack
16:40 openstackgerrit Matthew J Black proposed openstack/puppet-neutron: Added geneve type_driver config options  https://review.openstack.org/336115
16:41 cdelatte joined #puppet-openstack
16:47 beddari joined #puppet-openstack
16:52 rwsu joined #puppet-openstack
16:57 beddari joined #puppet-openstack
16:58 _ducttape_ joined #puppet-openstack
17:07 beddari joined #puppet-openstack
17:09 openstackgerrit Alex Schultz proposed openstack/puppet-ceilometer: Add memcached_servers for keystone authtoken  https://review.openstack.org/336125
17:09 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Add memcached_servers for keystone authtoken  https://review.openstack.org/336101
17:12 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken  https://review.openstack.org/336087
17:12 ducttape_ joined #puppet-openstack
17:15 ducttape_ joined #puppet-openstack
17:17 beddari joined #puppet-openstack
17:24 _ducttape_ joined #puppet-openstack
17:26 beddari joined #puppet-openstack
17:27 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists?  https://review.openstack.org/335600
17:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: disable trove  https://review.openstack.org/336131
17:30 EmilienM chem|off, mwhahaha: ok collector worked
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True  https://review.openstack.org/336045
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/334852
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Test https://review.openstack.org/#/c/334209/"  https://review.openstack.org/335016
17:31 EmilienM ok this chain of patches should promote our CI ^
17:32 EmilienM I'll try to reneable trove asap
17:37 beddari joined #puppet-openstack
17:37 aernhart joined #puppet-openstack
17:42 openstackgerrit Alex Schultz proposed openstack/puppet-heat: Add memcached_servers for keystone authtoken  https://review.openstack.org/336135
17:47 beddari joined #puppet-openstack
17:56 beddari joined #puppet-openstack
18:01 xarses mwhahaha: can you look over https://review.openstack.org/#/c/333542 I added reno file to it so I think that excludes me from +2
18:02 mwhahaha K I'll look in a bit
18:03 xarses whats the fuel-ci recheck passphrase?
18:04 iurygregory mwhahaha, can you look at 335727  335744? ^^
18:07 beddari joined #puppet-openstack
18:15 hjensas joined #puppet-openstack
18:15 hjensas joined #puppet-openstack
18:17 beddari joined #puppet-openstack
18:17 mwhahaha fuel-ci: recheck
18:25 beddari joined #puppet-openstack
18:36 openstackgerrit Merged openstack/puppet-designate: Add Xenial to operatingsystemrelease  https://review.openstack.org/335744
18:36 beddari joined #puppet-openstack
18:44 openstackgerrit Merged openstack/puppet-ceph: Add Xenial to operatingsystemrelease  https://review.openstack.org/335716
18:46 beddari joined #puppet-openstack
18:57 beddari joined #puppet-openstack
18:59 aernhart joined #puppet-openstack
19:06 beddari joined #puppet-openstack
19:12 openstackgerrit Alex Schultz proposed openstack/puppet-murano: Add memcached_servers for keystone authtoken  https://review.openstack.org/336261
19:15 iurygregory EmilienM, you around?
19:17 beddari joined #puppet-openstack
19:21 mwhahaha he said he'd be afk for some of the afternoon
19:21 mwhahaha what's up?
19:21 iurygregory oh i didn't saw =(
19:22 iurygregory mwhahaha, in murano and sahara why we dont have ubuntu in metadata? =(
19:22 mwhahaha probably cause they didn't have packages for a while :D
19:22 mwhahaha should add it
19:23 iurygregory tks o/ i have the patch in my notebook at home i'll push =)
19:27 beddari joined #puppet-openstack
19:28 mwhahaha sure
19:29 vinsh joined #puppet-openstack
19:36 beddari joined #puppet-openstack
19:46 beddari_ joined #puppet-openstack
19:49 hjensas joined #puppet-openstack
19:56 beddari joined #puppet-openstack
20:02 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Add memcached_servers for keystone authtoken  https://review.openstack.org/336270
20:07 beddari joined #puppet-openstack
20:09 ducttape_ joined #puppet-openstack
20:15 beddari joined #puppet-openstack
20:27 beddari joined #puppet-openstack
20:28 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-murano: Add Ubuntu to metadata  https://review.openstack.org/336279
20:30 cdelatte joined #puppet-openstack
20:31 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-sahara: Add Ubuntu to metadata  https://review.openstack.org/336280
20:37 beddari joined #puppet-openstack
20:39 cdelatte joined #puppet-openstack
20:41 rcernin joined #puppet-openstack
20:46 beddari joined #puppet-openstack
20:56 beddari joined #puppet-openstack
21:07 beddari joined #puppet-openstack
21:09 openstackgerrit Alex Schultz proposed openstack/puppet-sahara: Add memcached_servers for keystone authtoken  https://review.openstack.org/336290
21:14 openstackgerrit Alex Schultz proposed openstack/puppet-heat: Add memcached_servers for keystone authtoken  https://review.openstack.org/336135
21:15 openstackgerrit Alex Schultz proposed openstack/puppet-ironic: Add memcached_servers for keystone authtoken  https://review.openstack.org/336191
21:16 beddari joined #puppet-openstack
21:23 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335746
21:24 aernhart joined #puppet-openstack
21:26 beddari_ joined #puppet-openstack
21:26 openstackgerrit Alex Schultz proposed openstack/puppet-murano: Add memcached_servers for keystone authtoken  https://review.openstack.org/336261
21:35 openstackgerrit Merged openstack/puppet-congress: Add Xenial to metadata  https://review.openstack.org/335735
21:36 beddari joined #puppet-openstack
21:40 openstackgerrit Merged openstack/puppet-ec2api: Add Xenial to metadata  https://review.openstack.org/335757
21:40 openstackgerrit Merged openstack/puppet-tacker: Update metadata  https://review.openstack.org/335768
21:42 openstackgerrit Merged openstack/puppet-rally: Update metatada  https://review.openstack.org/335763
21:45 openstackgerrit Merged openstack/puppet-vitrage: Add vitrage notifier class  https://review.openstack.org/332054
21:45 openstackgerrit Merged openstack/puppet-watcher: Update metadata  https://review.openstack.org/335772
21:45 openstackgerrit Merged openstack/puppet-octavia: Update metadata  https://review.openstack.org/335756
21:47 beddari joined #puppet-openstack
21:51 cdelatte joined #puppet-openstack
21:57 beddari joined #puppet-openstack
21:59 EmilienM m
21:59 EmilienM mwhahaha: what's up?
21:59 EmilienM Iury pinged me oops
22:01 iurygregory joined #puppet-openstack
22:03 EmilienM iurygregory: hey sorry I was afk
22:03 iurygregory EmilienM, no problem o/
22:05 openstackgerrit Merged openstack/puppet-ceilometer: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335715
22:05 openstackgerrit Merged openstack/puppet-cinder: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335732
22:06 beddari joined #puppet-openstack
22:13 openstackgerrit Merged openstack/puppet-glance: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335741
22:13 openstackgerrit Merged openstack/puppet-gnocchi: Add Xenial to metadata  https://review.openstack.org/335742
22:13 openstackgerrit Merged openstack/puppet-heat: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335743
22:13 openstackgerrit Merged openstack/puppet-horizon: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335745
22:14 openstackgerrit Merged openstack/puppet-ironic: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335727
22:14 iurygregory danpawlik, i'm updating the keystone middleware patch o/
22:14 openstackgerrit Merged openstack/puppet-keystone: Add retry to keystone_user.exists?  https://review.openstack.org/335600
22:15 openstackgerrit Merged openstack/puppet-openstacklib: Remove 12.04 and add 16.04 to metadata  https://review.openstack.org/335760
22:15 EmilienM colonwq: buildlogs.centos.org/centos/​7/storage/x86_64/ceph-jewel/
22:15 EmilienM we have jewel packages in SIG !!
22:16 bkero \o/
22:16 beddari_ joined #puppet-openstack
22:17 iurygregory \o/
22:17 iurygregory nice
22:18 openstackgerrit Merged openstack/puppet-tempest: Remove 12.04 and add 16.04  https://review.openstack.org/335767
22:19 openstackgerrit Merged openstack/puppet-trove: Remove 12.04 and add 16.04 from metadata  https://review.openstack.org/335769
22:19 bkero EmilienM: are they psasing tests?
22:21 openstackgerrit Merged openstack/puppet-zaqar: Add xenial to metadata  https://review.openstack.org/335773
22:22 derekh joined #puppet-openstack
22:23 EmilienM bkero: dunno
22:23 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-mistral: Add Xenial to metadata  https://review.openstack.org/335752
22:26 beddari joined #puppet-openstack
22:27 cdelatte joined #puppet-openstack
22:31 ntpttr joined #puppet-openstack
22:32 openstackgerrit Merged openstack/puppet-openstack-integration: scenario003: disable trove  https://review.openstack.org/336131
22:36 beddari joined #puppet-openstack
22:46 beddari joined #puppet-openstack
22:52 iurygregory richm, you around
22:52 iurygregory ?
22:56 beddari joined #puppet-openstack
23:03 iurygregory EmilienM, richm, I'm wondering why it's possible to pass domain in username and project_name using "::".. do we still need this for some special reason?
23:04 iurygregory sorry to bother you guys at this time XD
23:07 beddari joined #puppet-openstack
23:09 ayoung joined #puppet-openstack
23:17 beddari joined #puppet-openstack
23:27 richm iurygregory: I don't know if it is still needed - I think the work that chem did makes this not necessary anymore
23:27 beddari joined #puppet-openstack
23:28 iurygregory richm good to know, but what is the work that chem|off have done? (I would like to know)
23:28 iurygregory richm, i'll remove so we can have only the parameters from keystonemiddleware =)
23:28 iurygregory tks
23:29 richm The "::domain" was required to make resource names/titles unique
23:29 richm but chem figured out how to use multiple parameters to make a unique resource
23:30 richm I think the "::domain" naming was supposed to be deprecated?
23:30 richm In N?  and then removed in O?  I can't remember
23:33 iurygregory humm i'll update the patch and ask about this to chem =) tks richm
23:36 beddari joined #puppet-openstack
23:44 cdelatte joined #puppet-openstack
23:46 richm iurygregory: ok
23:47 beddari joined #puppet-openstack
23:57 beddari joined #puppet-openstack
23:59 hogepodge joined #puppet-openstack

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