*** beddari has quit IRC | 00:01 | |
*** gfidente|afk has quit IRC | 00:03 | |
*** beddari has joined #puppet-openstack | 00:05 | |
*** xarses has quit IRC | 00:06 | |
*** gchamoul has joined #puppet-openstack | 00:10 | |
*** beddari has quit IRC | 00:11 | |
*** beddari has joined #puppet-openstack | 00:15 | |
openstackgerrit | Matteus Sthefano Leite da Silva proposed openstack/puppet-barbican: Add Xenial to operatingsystemrelease https://review.openstack.org/335723 | 00:16 |
---|---|---|
*** xingchao has quit IRC | 00:20 | |
openstackgerrit | Caio Oliveira proposed openstack/puppet-ironic: Remove 12.04 and add 16.04 to metadatajson https://review.openstack.org/335727 | 00:24 |
*** beddari has quit IRC | 00:25 | |
*** beddari_ has joined #puppet-openstack | 00:26 | |
*** beddari_ is now known as beddari | 00:26 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-cinder: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335732 | 00:29 |
*** beddari has quit IRC | 00:31 | |
*** beddari has joined #puppet-openstack | 00:35 | |
*** ducttape_ has joined #puppet-openstack | 00:42 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-congress: Add Xenial to metadata https://review.openstack.org/335735 | 00:44 |
*** beddari has quit IRC | 00:45 | |
*** beddari has joined #puppet-openstack | 00:46 | |
*** rwsu has quit IRC | 00:49 | |
*** beddari has quit IRC | 00:55 | |
*** beddari has joined #puppet-openstack | 00:56 | |
*** beddari has quit IRC | 01:01 | |
*** beddari has joined #puppet-openstack | 01:04 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-glance: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335741 | 01:09 |
*** beddari has quit IRC | 01:11 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: Add Xenial to metadata https://review.openstack.org/335742 | 01:11 |
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 has joined #puppet-openstack | 01:14 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-horizon: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335745 | 01:15 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335746 | 01:16 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-manila: Add Xenial to metadata https://review.openstack.org/335747 | 01:18 |
*** ducttape_ has quit IRC | 01:19 | |
*** ducttape_ has joined #puppet-openstack | 01:24 | |
*** beddari has quit IRC | 01:24 | |
*** beddari has joined #puppet-openstack | 01:25 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-mistral: Add Xenial to metadata https://review.openstack.org/335752 | 01:26 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-neutron: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335754 | 01:28 |
*** beddari has quit IRC | 01:31 | |
*** ducttape_ has quit IRC | 01:34 | |
*** xingchao has joined #puppet-openstack | 01:35 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-octavia: Update metadata https://review.openstack.org/335756 | 01:35 |
*** beddari has joined #puppet-openstack | 01:35 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Add Xenial to metadata https://review.openstack.org/335757 | 01:37 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Add Xenial to metadata https://review.openstack.org/335758 | 01:38 |
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:39 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstacklib: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335760 | 01:41 |
*** beddari has quit IRC | 01:41 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-oslo: Add Xenial to metadata https://review.openstack.org/335761 | 01:42 |
*** beddari has joined #puppet-openstack | 01:45 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-rally: Update metatada https://review.openstack.org/335763 | 01:47 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-swift: Remove 12.04 and add 16.04 https://review.openstack.org/335766 | 01:55 |
*** beddari has quit IRC | 01:55 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-tempest: Remove 12.04 and add 16.04 https://review.openstack.org/335767 | 01:55 |
*** beddari has joined #puppet-openstack | 01:56 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-tacker: Update metadata https://review.openstack.org/335768 | 01:58 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-trove: Remove 12.04 and add 16.04 from metadata https://review.openstack.org/335769 | 02:00 |
*** beddari has quit IRC | 02:01 | |
*** beddari has joined #puppet-openstack | 02:05 | |
*** markvoelker has quit IRC | 02:06 | |
*** ducttape_ has joined #puppet-openstack | 02:07 | |
*** _ducttape_ has joined #puppet-openstack | 02:09 | |
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:10 |
*** beddari has quit IRC | 02:11 | |
*** ducttape_ has quit IRC | 02:11 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-watcher: Update metadata https://review.openstack.org/335772 | 02:12 |
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:13 |
EmilienM | ayoung: ^ | 02:14 |
ayoung | U hoh | 02:14 |
iurygregory_ | i talked to my friend in HP to review =) | 02:14 |
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 has joined #puppet-openstack | 02:15 | |
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:16 |
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:17 |
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:18 |
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:19 |
iurygregory_ | we add all parameters according to http://docs.openstack.org/developer/keystonemiddleware/middlewarearchitecture.html#configuration | 02:20 |
iurygregory_ | but in this docs we have some examples from things that maybe are deprecated but still have effect i think =) | 02:21 |
*** beddari has quit IRC | 02:21 | |
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:24 |
*** beddari has joined #puppet-openstack | 02:25 | |
*** _ducttape_ has quit IRC | 02:32 | |
*** vinsh_ has quit IRC | 02:32 | |
*** xarses has joined #puppet-openstack | 02:34 | |
iurygregory_ | ayoung, did you understood? .-. | 02:34 |
*** beddari has quit IRC | 02:35 | |
*** beddari has joined #puppet-openstack | 02:35 | |
ayoung | iurygregory_, one sec | 02:36 |
iurygregory_ | ayoung, sure =) | 02:36 |
*** vinsh has joined #puppet-openstack | 02:36 | |
*** ducttape_ has joined #puppet-openstack | 02:37 | |
*** beddari has quit IRC | 02:41 | |
*** beddari has joined #puppet-openstack | 02:45 | |
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:52 |
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:53 |
iurygregory_ | ayoung, about drawbacks i don't think we have, it's hard to read all parameters .-. | 02:54 |
*** beddari has quit IRC | 02:55 | |
ayoung | could we autogenerate these or check somehow? How does testing work for these? | 02:55 |
*** beddari has joined #puppet-openstack | 02:56 | |
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:57 |
iurygregory_ | auto generate i don't have any idea, to test we have added each param in tests | 02:58 |
*** ducttape_ has quit IRC | 02:59 | |
* iurygregory_ thinking.. | 02:59 | |
iurygregory_ | i wish i had documentation with what are cases and what configurations are expected XD | 03:01 |
*** beddari has quit IRC | 03:01 | |
ayoung | iurygregory_, so, apachectl -t will tell you fid syntax is good. Maybe something like that? | 03:03 |
ayoung | use puppet to generate a config file, then run something to confirm the config | 03:04 |
iurygregory_ | oh i see =) | 03:04 |
*** beddari has joined #puppet-openstack | 03:05 | |
*** markvoelker has 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:06 |
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:07 |
*** vinsh has quit IRC | 03:07 | |
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:08 |
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:09 |
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:10 |
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 |
*** beddari has quit IRC | 03:11 | |
iurygregory_ | using $::os_service_default we dont change the configuration | 03:11 |
ayoung | iurygregory_, "don | 03:11 |
*** markvoelker has quit IRC | 03:11 | |
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:12 |
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:13 |
iurygregory_ | i don't know about hiera =( | 03:14 |
*** beddari has joined #puppet-openstack | 03:16 | |
*** beddari has quit IRC | 03:21 | |
*** xingchao has quit IRC | 03:22 | |
*** beddari has 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:23 |
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:24 |
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:25 |
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/complete_example.html#what-can-we-do-with-hiera XD | 03:26 |
*** xingchao_ has 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:26 |
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_ has 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:28 |
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:29 |
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:30 |
ayoung | "os-collect-config never runs puppet, it runs os-refresh-config only, which then invokes a script that runs puppet." | 03:31 |
*** beddari has quit IRC | 03:31 | |
iurygregory_ | ayoung, thanks for the link :D | 03:31 |
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:32 |
iurygregory_ | yeah .-. | 03:33 |
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-refresh-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:34 |
iurygregory_ | tks ayoung o/ | 03:35 |
iurygregory_ | good luck in TripleO o/ | 03:35 |
ayoung | Trip HELLO! | 03:36 |
*** beddari has joined #puppet-openstack | 03:36 | |
*** iurygregory_ has quit IRC | 03:36 | |
*** beddari has quit IRC | 03:41 | |
*** beddari has joined #puppet-openstack | 03:44 | |
*** beddari has quit IRC | 03:51 | |
*** beddari has joined #puppet-openstack | 03:55 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-octavia: Update metadata https://review.openstack.org/335756 | 04:05 |
*** beddari has quit IRC | 04:05 | |
*** beddari_ has joined #puppet-openstack | 04:05 | |
*** beddari_ is now known as beddari | 04:06 | |
*** richm has quit IRC | 04:07 | |
*** markvoelker has joined #puppet-openstack | 04:07 | |
*** vinsh has joined #puppet-openstack | 04:08 | |
*** xingchao_ has quit IRC | 04:09 | |
*** xingchao has joined #puppet-openstack | 04:11 | |
*** beddari has quit IRC | 04:11 | |
*** markvoelker has quit IRC | 04:12 | |
*** vinsh has quit IRC | 04:13 | |
*** ducttape_ has quit IRC | 04:16 | |
*** beddari has joined #puppet-openstack | 04:16 | |
*** xingchao has quit IRC | 04:16 | |
*** beddari has quit IRC | 04:21 | |
*** beddari has joined #puppet-openstack | 04:25 | |
*** ayoung has quit IRC | 04:29 | |
*** beddari has quit IRC | 04:35 | |
*** beddari_ has joined #puppet-openstack | 04:35 | |
*** beddari_ is now known as beddari | 04:35 | |
*** vinsh has joined #puppet-openstack | 04:39 | |
*** vinsh has quit IRC | 04:43 | |
*** beddari has quit IRC | 04:45 | |
*** beddari has joined #puppet-openstack | 04:45 | |
*** beddari has quit IRC | 04:55 | |
*** beddari_ has joined #puppet-openstack | 04:55 | |
*** beddari_ is now known as beddari | 04:56 | |
*** beddari has quit IRC | 05:05 | |
*** beddari_ has joined #puppet-openstack | 05:06 | |
*** beddari_ is now known as beddari | 05:06 | |
*** beddari has quit IRC | 05:11 | |
*** sanjayu has joined #puppet-openstack | 05:12 | |
*** sanjayu has quit IRC | 05:12 | |
*** beddari has joined #puppet-openstack | 05:16 | |
*** beddari_ has joined #puppet-openstack | 05:26 | |
*** beddari has quit IRC | 05:26 | |
*** beddari_ is now known as beddari | 05:26 | |
*** rcernin has joined #puppet-openstack | 05:26 | |
*** beddari has quit IRC | 05:31 | |
*** rcernin has quit IRC | 05:34 | |
*** beddari has joined #puppet-openstack | 05:36 | |
*** beddari has quit IRC | 05:41 | |
*** beddari has joined #puppet-openstack | 05:46 | |
*** ducttape_ has joined #puppet-openstack | 05:46 | |
*** ducttape_ has quit IRC | 05:52 | |
*** dtantsur|afk is now known as dtantsur | 05:55 | |
*** beddari has quit IRC | 05:55 | |
*** beddari has joined #puppet-openstack | 05:56 | |
*** jmikolajczak has joined #puppet-openstack | 06:04 | |
*** markvoelker has joined #puppet-openstack | 06:09 | |
*** rcernin has joined #puppet-openstack | 06:09 | |
*** vinsh has joined #puppet-openstack | 06:09 | |
*** markvoelker has quit IRC | 06:13 | |
*** vinsh has quit IRC | 06:15 | |
*** beddari_ has joined #puppet-openstack | 06:16 | |
*** beddari has quit IRC | 06:16 | |
*** beddari_ is now known as beddari | 06:16 | |
openstackgerrit | Andreas Jaeger proposed openstack/puppet-openstack-release-tools: List system dependencies for running common tests https://review.openstack.org/335806 | 06:17 |
*** xingchao has joined #puppet-openstack | 06:20 | |
*** beddari has quit IRC | 06:25 | |
*** beddari has joined #puppet-openstack | 06:26 | |
*** beddari has quit IRC | 06:31 | |
*** beddari has joined #puppet-openstack | 06:36 | |
*** beddari has quit IRC | 06:41 | |
openstackgerrit | zhongshengping proposed openstack/puppet-ec2api: Add purge_config option https://review.openstack.org/335812 | 06:43 |
*** dtantsur is now known as dtantsur|brb | 06:43 | |
*** beddari has joined #puppet-openstack | 06:45 | |
*** ducttape_ has joined #puppet-openstack | 06:48 | |
*** shardy has joined #puppet-openstack | 06:51 | |
*** beddari has quit IRC | 06:51 | |
*** ducttape_ has quit IRC | 06:53 | |
*** chem|off is now known as chem | 06:55 | |
*** amoralej|off is now known as amoralej | 06:55 | |
*** beddari has joined #puppet-openstack | 06:56 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists? https://review.openstack.org/335600 | 07:02 |
*** foxhunt_away is now known as foxhunt72 | 07:02 | |
*** beddari has quit IRC | 07:05 | |
*** beddari_ has joined #puppet-openstack | 07:06 | |
*** beddari_ is now known as beddari | 07:06 | |
*** markvoelker has joined #puppet-openstack | 07:10 | |
*** beddari has quit IRC | 07:11 | |
*** markvoelker has quit IRC | 07:14 | |
*** ifarkas has joined #puppet-openstack | 07:15 | |
*** tesseract- has joined #puppet-openstack | 07:15 | |
*** beddari has joined #puppet-openstack | 07:16 | |
*** beddari has quit IRC | 07:21 | |
*** beddari has joined #puppet-openstack | 07:26 | |
*** xingchao has quit IRC | 07:27 | |
*** beddari has quit IRC | 07:31 | |
*** beddari has joined #puppet-openstack | 07:36 | |
*** xingchao has joined #puppet-openstack | 07:38 | |
*** beddari has quit IRC | 07:41 | |
*** danpawlik has joined #puppet-openstack | 07:42 | |
*** jpena|off is now known as jpena | 07:43 | |
*** beddari has joined #puppet-openstack | 07:46 | |
*** ducttape_ has joined #puppet-openstack | 07:50 | |
*** xingchao has quit IRC | 07:53 | |
*** ducttape_ has quit IRC | 07:54 | |
*** beddari has quit IRC | 07:56 | |
*** beddari_ has joined #puppet-openstack | 07:56 | |
*** beddari_ is now known as beddari | 07:56 | |
*** gfidente has joined #puppet-openstack | 08:04 | |
*** derekh has joined #puppet-openstack | 08:04 | |
*** beddari has quit IRC | 08:05 | |
*** beddari has joined #puppet-openstack | 08:06 | |
*** markvoelker has joined #puppet-openstack | 08:11 | |
*** beddari has quit IRC | 08:11 | |
*** vinsh has joined #puppet-openstack | 08:12 | |
*** markvoelker has quit IRC | 08:15 | |
*** vinsh has quit IRC | 08:16 | |
*** beddari has joined #puppet-openstack | 08:16 | |
*** xingchao has joined #puppet-openstack | 08:19 | |
*** beddari has quit IRC | 08:21 | |
*** beddari has joined #puppet-openstack | 08:25 | |
*** beddari has quit IRC | 08:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/334852 | 08:33 |
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:34 |
*** xingchao has quit IRC | 08:35 | |
*** beddari has joined #puppet-openstack | 08:35 | |
*** xingchao has joined #puppet-openstack | 08:36 | |
*** dgurtner has joined #puppet-openstack | 08:38 | |
*** dgurtner has quit IRC | 08:38 | |
*** dgurtner has joined #puppet-openstack | 08:38 | |
*** beddari has quit IRC | 08:45 | |
*** beddari_ has joined #puppet-openstack | 08:45 | |
*** beddari_ is now known as beddari | 08:46 | |
*** ducttape_ has joined #puppet-openstack | 08:50 | |
*** ducttape_ has quit IRC | 08:55 | |
*** beddari has quit IRC | 08:55 | |
*** beddari has joined #puppet-openstack | 08:56 | |
*** imcsk8_ has joined #puppet-openstack | 08:57 | |
*** bapalm has quit IRC | 08:57 | |
openstackgerrit | Daniel Pawlik proposed openstack/puppet-keystone: Add misssing parameters to keystone_authtoken https://review.openstack.org/328620 | 09:00 |
*** bapalm has joined #puppet-openstack | 09:00 | |
*** beddari has quit IRC | 09:01 | |
*** abramley_ has joined #puppet-openstack | 09:04 | |
*** beddari has joined #puppet-openstack | 09:06 | |
*** imcsk8 has quit IRC | 09:08 | |
*** iurygregory has quit IRC | 09:08 | |
*** ericksonsantos has quit IRC | 09:08 | |
*** partner_ has quit IRC | 09:08 | |
*** Kvisle_ has quit IRC | 09:08 | |
*** abramley has quit IRC | 09:08 | |
*** bkero has quit IRC | 09:08 | |
*** abramley_ is now known as abramley | 09:08 | |
*** Kvisle has joined #puppet-openstack | 09:09 | |
*** beddari has quit IRC | 09:11 | |
*** ericksonsantos has joined #puppet-openstack | 09:14 | |
*** bkero has joined #puppet-openstack | 09:14 | |
*** iurygregory has joined #puppet-openstack | 09:15 | |
*** beddari has joined #puppet-openstack | 09:16 | |
*** xingchao has quit IRC | 09:19 | |
*** beddari has quit IRC | 09:21 | |
*** partner has joined #puppet-openstack | 09:21 | |
*** xingchao has joined #puppet-openstack | 09:25 | |
*** beddari has joined #puppet-openstack | 09:26 | |
*** psanchez has joined #puppet-openstack | 09:34 | |
amoralej | bkero, any objection to cherry-pick https://review.openstack.org/#/c/333406/ to stable/mitaka? | 09:34 |
*** beddari has quit IRC | 09:36 | |
*** beddari_ has joined #puppet-openstack | 09:36 | |
*** beddari_ is now known as beddari | 09:36 | |
*** partner has quit IRC | 09:37 | |
*** xingchao has quit IRC | 09:39 | |
*** beddari has quit IRC | 09:46 | |
*** xingchao has joined #puppet-openstack | 09:46 | |
*** beddari_ has joined #puppet-openstack | 09:46 | |
*** beddari_ is now known as beddari | 09:46 | |
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 |
*** dtantsur|brb is now known as dtantsur | 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/a4df69e/console.html#_2016-06-30_02_46_40_985265 | 09:49 |
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/a4df69e/console.html#_2016-06-30_02_48_18_178835 | 09:50 |
chem | does anybody has already seen anything like that ? | 09:50 |
*** beddari has quit IRC | 09:51 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: DEBUG: test ci https://review.openstack.org/335910 | 09:52 |
*** beddari has joined #puppet-openstack | 09:55 | |
*** beddari has quit IRC | 10:01 | |
*** beddari has joined #puppet-openstack | 10:06 | |
*** vinsh has joined #puppet-openstack | 10:10 | |
*** beddari has quit IRC | 10:11 | |
*** _degorenko|afk is now known as degorenko | 10:12 | |
degorenko | iurygregory, EmilienM do we need to drop 14.04 for master here https://review.openstack.org/#/q/topic:update_metadata ? | 10:13 |
*** xb_ has joined #puppet-openstack | 10:14 | |
*** beddari has joined #puppet-openstack | 10:16 | |
openstackgerrit | Alfredo Moralejo proposed openstack/puppet-tempest: Allow virtuals during python-setuptools install https://review.openstack.org/335918 | 10:17 |
*** beddari has quit IRC | 10:21 | |
*** beddari has joined #puppet-openstack | 10:26 | |
*** beddari has quit IRC | 10:31 | |
degorenko | chem, hey | 10:31 |
degorenko | strange error | 10:31 |
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:32 |
*** mdavidson has quit IRC | 10:33 | |
*** beddari has joined #puppet-openstack | 10:36 | |
*** vinsh has quit IRC | 10:39 | |
openstackgerrit | Merged openstack/puppet-ec2api: Add purge_config option https://review.openstack.org/335812 | 10:42 |
*** chem has quit IRC | 10:42 | |
*** chem has joined #puppet-openstack | 10:42 | |
*** aernhart has quit IRC | 10:44 | |
*** aernhart has joined #puppet-openstack | 10:46 | |
*** beddari has quit IRC | 10:46 | |
*** beddari has joined #puppet-openstack | 10:46 | |
*** aernhart has quit IRC | 10:46 | |
*** aernhart has joined #puppet-openstack | 10:47 | |
*** beddari has quit IRC | 10:51 | |
*** partner has joined #puppet-openstack | 10:54 | |
*** beddari has joined #puppet-openstack | 10:56 | |
*** xingchao has quit IRC | 10:57 | |
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 | 10:59 |
*** beddari has quit IRC | 11:05 | |
*** beddari has joined #puppet-openstack | 11:06 | |
*** aderyugin has quit IRC | 11:11 | |
*** beddari has quit IRC | 11:11 | |
*** aderyugin has joined #puppet-openstack | 11:12 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack_extras: Assume Debian if no osfamily https://review.openstack.org/335665 | 11:14 |
*** beddari has joined #puppet-openstack | 11:16 | |
*** jpena is now known as jpena|lunch | 11:16 | |
degorenko | chem, same error on another patch https://review.openstack.org/#/c/328620/ as your | 11:21 |
*** beddari has quit IRC | 11:21 | |
openstackgerrit | Merged openstack/puppet-neutron: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335754 | 11:24 |
*** beddari has joined #puppet-openstack | 11:26 | |
*** ducttape_ has joined #puppet-openstack | 11:29 | |
openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues https://review.openstack.org/335960 | 11:30 |
*** beddari has quit IRC | 11:31 | |
*** beddari has joined #puppet-openstack | 11:35 | |
*** ericksonsantos has quit IRC | 11:38 | |
*** vinsh has joined #puppet-openstack | 11:40 | |
*** beddari has quit IRC | 11:45 | |
*** vinsh has quit IRC | 11:45 | |
*** paramite has quit IRC | 11:46 | |
*** paramite has joined #puppet-openstack | 11:46 | |
*** beddari has joined #puppet-openstack | 11:46 | |
EmilienM | hello | 11:51 |
*** beddari has quit IRC | 11:51 | |
degorenko | hey | 11:52 |
degorenko | Hunner, ping | 11:52 |
*** beddari has joined #puppet-openstack | 11:54 | |
*** ducttape_ has quit IRC | 11:57 | |
*** johnmilton has joined #puppet-openstack | 11:57 | |
*** beddari has quit IRC | 12:01 | |
*** beddari has 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:06 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Test https://review.openstack.org/#/c/334209/" https://review.openstack.org/335016 | 12:07 |
*** johnmilton has quit IRC | 12:07 | |
openstackgerrit | Merged openstack/puppet-tripleo: Resolve duplicate base::ceilometer issues https://review.openstack.org/335960 | 12:08 |
*** markvoelker has joined #puppet-openstack | 12:13 | |
*** markvoelker has quit IRC | 12:14 | |
*** markvoelker has joined #puppet-openstack | 12:15 | |
*** beddari has quit IRC | 12:16 | |
*** beddari_ has joined #puppet-openstack | 12:16 | |
*** beddari_ is now known as beddari | 12:16 | |
*** paramite is now known as paramite|afk | 12:17 | |
openstackgerrit | Caio Oliveira proposed openstack/puppet-ironic: Remove 12.04 and add 16.04 to metadata https://review.openstack.org/335727 | 12:17 |
*** johnmilton has joined #puppet-openstack | 12:20 | |
*** vinsh has joined #puppet-openstack | 12:20 | |
iurygregory | degorenko, i asked mwhahaha maybe we should drop before release Newton | 12:20 |
iurygregory | morning guys | 12:20 |
*** paramite|afk is now known as paramite | 12:21 | |
*** ericksonsantos has joined #puppet-openstack | 12:23 | |
*** ericksonsantos has quit IRC | 12:26 | |
*** beddari has quit IRC | 12:26 | |
*** beddari has joined #puppet-openstack | 12:26 | |
*** johnmilton has quit IRC | 12:30 | |
*** beddari has quit IRC | 12:31 | |
*** beddari has joined #puppet-openstack | 12:35 | |
*** ericksonsantos has joined #puppet-openstack | 12:37 | |
*** jayg|g0n3 is now known as jayg | 12:40 | |
openstackgerrit | Merged openstack/puppet-tripleo: Remove base/neutron/ml2.pp https://review.openstack.org/332870 | 12:43 |
*** beddari has quit IRC | 12:45 | |
*** jpena|lunch is now known as jpena | 12:46 | |
*** beddari has joined #puppet-openstack | 12:46 | |
*** jmikolajczak has left #puppet-openstack | 12:47 | |
chem | So, CI is broken for puppet-keystone: https://bugs.launchpad.net/puppet-keystone/+bug/1597753 | 12:49 |
openstack | Launchpad bug 1597753 in puppet-keystone "Broken CI: python-pysaml2 is always re-installed" [Undecided,New] | 12:49 |
*** mdavidson has joined #puppet-openstack | 12:49 | |
openstackgerrit | Merged openstack/puppet-tripleo: neutron plugins: include neutron base https://review.openstack.org/332897 | 12:51 |
iurygregory | chem, yeah i saw that =( | 12:51 |
*** beddari has quit IRC | 12:51 | |
iurygregory | did you check locally? | 12:52 |
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:53 |
openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Add gnocchi profiles https://review.openstack.org/315527 | 12:54 |
*** amoralej is now known as amoralej|lunch | 12:56 | |
*** beddari has joined #puppet-openstack | 12:56 | |
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_ has joined #puppet-openstack | 12:58 | |
EmilienM | iurygregory: ack | 12:58 |
openstackgerrit | Alexey Deryugin proposed openstack/puppet-openstack-integration: Use Debian repositories instead of UCA https://review.openstack.org/331195 | 12:59 |
*** ducttape_ has quit IRC | 13:02 | |
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:05 |
openstackgerrit | Dan Radez proposed openstack/puppet-congress: Populating init.pp and parameters https://review.openstack.org/330195 | 13:09 |
*** beddari has quit IRC | 13:11 | |
*** trown|outtypewww is now known as trown | 13:13 | |
*** beddari has joined #puppet-openstack | 13:16 | |
*** beddari has quit IRC | 13:21 | |
*** johnmilton has joined #puppet-openstack | 13:24 | |
*** beddari has joined #puppet-openstack | 13:26 | |
*** beddari has quit IRC | 13:31 | |
*** beddari has joined #puppet-openstack | 13:36 | |
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:37 |
mwhahaha | the issue is, http://logs.openstack.org/65/335665/4/check/gate-puppet-openstack_extras-puppet-beaker-rspec-ubuntu-xenial/73d3af8/console.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:38 |
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:39 |
*** amoralej|lunch is now known as amoralej | 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:40 |
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 |
*** beddari has quit IRC | 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:42 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly https://review.openstack.org/328347 | 13:43 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: nova/compute: add NFS bits https://review.openstack.org/332954 | 13:44 |
*** beddari has joined #puppet-openstack | 13:46 | |
*** ayoung has joined #puppet-openstack | 13:48 | |
*** emccormick has joined #puppet-openstack | 13:50 | |
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:51 |
*** beddari has quit IRC | 13:51 | |
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/e96497e/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_ has joined #puppet-openstack | 13:52 | |
mwhahaha | i have no idea :D i wouldn't think so but just wondering | 13:52 |
chem | mwhahaha: hehe, ack I'll ask on #rdo | 13:53 |
*** beddari has joined #puppet-openstack | 13:56 | |
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:57 |
EmilienM | what? | 13:58 |
EmilienM | emccormick: I think we support this use case, we do it in tripleo | 13:59 |
*** ducttape_ has 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 |
*** social has quit IRC | 14:00 | |
emccormick | } | 14:00 |
emccormick | sorry probably not the best way to do that but that nukes the config options | 14:00 |
*** xarses has quit IRC | 14:00 | |
emccormick | I get around it by just getting rid of that bit but obviously I'd prefer not to stay forked | 14:00 |
*** beddari has quit IRC | 14:01 | |
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:02 |
emccormick | err | 14:03 |
emccormick | "volumes" sorry | 14:03 |
EmilienM | we don't hardcode 'volumes' | 14:03 |
*** ducttape_ has quit IRC | 14:03 | |
EmilienM | and it's not harcoded in nova, isn't? | 14:03 |
*** richm has 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:03 |
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:04 |
emccormick | emergency hack ;) | 14:05 |
*** beddari has joined #puppet-openstack | 14:06 | |
EmilienM | ok promotion will be hard | 14:07 |
EmilienM | trove tests does not exist anymore in tempest | 14:07 |
EmilienM | mtreinish: http://logs.openstack.org/52/334852/4/check/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:08 |
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:09 |
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:10 |
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 | |
*** beddari has quit IRC | 14:11 | |
EmilienM | mtreinish: lol | 14:11 |
EmilienM | sad story | 14:11 |
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/master/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:12 |
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:13 |
mwhahaha | oh i have vagrant installed :D | 14:14 |
degorenko | boom :) | 14:14 |
*** rcernin has quit IRC | 14:14 | |
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:15 |
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 has joined #puppet-openstack | 14:16 | |
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:17 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: redhat/packages: set allow_virtual to True https://review.openstack.org/336045 | 14:19 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/334852 | 14:20 |
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:21 |
*** beddari has quit IRC | 14:21 | |
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:22 |
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:23 |
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:24 |
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:25 |
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 has 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:26 |
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:27 |
chem | EmilienM: as my patch :) | 14:28 |
EmilienM | mhh | 14:29 |
*** rcernin has joined #puppet-openstack | 14:29 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists? https://review.openstack.org/335600 | 14:31 |
*** jistr is now known as jistr|mtg | 14:31 | |
*** beddari has quit IRC | 14:31 | |
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:32 |
chem | EmilienM: I think openstacklib would be the place. Is there one of the manifest that is loaded by every other module ? | 14:34 |
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:35 |
*** beddari has joined #puppet-openstack | 14:36 | |
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 has joined #puppet-openstack | 14:37 | |
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_ has joined #puppet-openstack | 14:38 | |
mwhahaha | who thought using virtual packages was a good thing :D | 14:38 |
*** ducttape_ has quit IRC | 14:39 | |
*** beddari has quit IRC | 14:41 | |
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:42 |
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 has 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:43 |
EmilienM | yeah... | 14:44 |
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 |
*** rcernin has quit IRC | 14:46 | |
*** beddari has joined #puppet-openstack | 14:46 | |
EmilienM | maybe we can ask for help from RDO folks | 14:46 |
EmilienM | amoralej: are you interested to help on it? ^ | 14:47 |
* amoralej reading up... | 14:49 | |
*** saneax is now known as saneax_AFK | 14:49 | |
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:51 |
*** beddari has quit IRC | 14:51 | |
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:54 |
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:55 |
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 has joined #puppet-openstack | 14:56 | |
EmilienM | right | 14:56 |
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:57 |
*** rcernin has joined #puppet-openstack | 14:58 | |
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 | 14:59 |
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:00 |
mwhahaha | as i think almost all of the package definitions use some form of params | 15:01 |
*** beddari has quit IRC | 15:01 | |
*** ducttape_ has 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:05 |
mwhahaha | params might be cleaner, but that's an option | 15:06 |
*** beddari has joined #puppet-openstack | 15:06 | |
chem | mwhahaha: params is cleaner ... and in some way it's a param | 15:06 |
*** dtantsur is now known as dtantsur|afk | 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 |
*** ducttape_ has quit IRC | 15:09 | |
EmilienM | welcome to durty world | 15:09 |
EmilienM | params.pp is not used for this purpose afik but well... | 15:09 |
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:10 |
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 |
*** beddari has quit IRC | 15:11 | |
chem | EmilienM: amoralej the only thing I'm missing is a clean list of all puppet modules | 15:11 |
*** ducttape_ has joined #puppet-openstack | 15:11 | |
amoralej | wich wil turn on allow_virtual, right? | 15:11 |
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-integration/master/openstack_modules.txt | 15:12 |
chem | mwhahaha: ack, by using only the params.pp.. got it | 15:13 |
chem | EmilienM: that's it ? I always though there were more ... anyway thanks. | 15:14 |
*** gregfaust has quit IRC | 15:15 | |
EmilienM | chem: sorry, we *only* manage 33 puppet modules | 15:15 |
*** dfisher has joined #puppet-openstack | 15:15 | |
chem | EmilienM: hehe :) | 15:15 |
*** beddari has joined #puppet-openstack | 15:16 | |
*** rcernin has quit IRC | 15:19 | |
*** beddari has quit IRC | 15:22 | |
*** jistr|mtg is now known as jistr | 15:23 | |
*** beddari has joined #puppet-openstack | 15:25 | |
*** tesseract- has quit IRC | 15:27 | |
*** beddari has quit IRC | 15:31 | |
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 has joined #puppet-openstack | 15:35 | |
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:36 |
*** foxhunt72 is now known as foxhunt_away | 15:37 | |
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:38 |
*** beddari has quit IRC | 15:42 | |
chem | EmilienM: oki, so why it doesn't work :) | 15:43 |
*** beddari has joined #puppet-openstack | 15:46 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken https://review.openstack.org/336087 | 15:47 |
*** beddari has quit IRC | 15:51 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken https://review.openstack.org/336087 | 15:52 |
*** beddari has joined #puppet-openstack | 15:56 | |
chem | EmilienM: mwhahaha the include openstacklib::defaults won't work | 15:58 |
mwhahaha | why? | 15:58 |
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... | 15:59 |
mwhahaha | Package<| tag == 'openstack'|> { allow_virtual => true } | 16:00 |
chem | mwhahaha: right this is working | 16:01 |
mwhahaha | so we'd need to put that in openstacklib::defaults | 16:01 |
*** beddari has quit IRC | 16:01 | |
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:03 |
chem | mwhahaha: for one those are tagged differently https://review.openstack.org/#/c/336048/2/manifests/federation/identity_provider.pp | 16:04 |
mwhahaha | hmm so maybe we should use openstack and openstack-dependency or something | 16:05 |
chem | mwhahaha: well, now we have to go over all package to make sure they are tagged all right ... | 16:06 |
*** beddari has joined #puppet-openstack | 16:06 | |
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:07 |
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:08 |
chem | mwhahaha: you're right https://docs.puppet.com/puppet/3.6/reference/release_notes.html#changes-to-rpm-behavior-with-virtual-packages | 16:10 |
chem | mwhahaha: from 4 on ... | 16:10 |
*** beddari has quit IRC | 16:11 | |
*** ifarkas has quit IRC | 16:14 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-aodh: Add memcached_servers for keystone authtoken https://review.openstack.org/336101 | 16:14 |
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 has joined #puppet-openstack | 16:16 | |
*** cdelatte has quit IRC | 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:16 |
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:17 |
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:18 |
jpena | EmilienM: if I understood https://docs.puppet.com/puppet/3.6/reference/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:19 |
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:21 |
*** beddari has quit IRC | 16:22 | |
amoralej | according to https://docs.puppet.com/puppet/3.6/reference/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:22 |
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:23 |
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:24 |
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 has 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:27 |
EmilienM | Package<||> { allow_virtual => true } | 16:28 |
EmilienM | this ? ^ | 16:28 |
mwhahaha | yea | 16:28 |
chem | EmilienM: yep | 16:28 |
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:29 |
EmilienM | ah | 16:30 |
chem | hehe | 16:30 |
*** trown is now known as trown|lunch | 16:30 | |
*** beddari has quit IRC | 16:32 | |
EmilienM | so what? | 16:32 |
EmilienM | tag or not tag | 16:32 |
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:33 |
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:34 |
EmilienM | cool :) | 16:35 |
*** beddari has joined #puppet-openstack | 16:37 | |
openstackgerrit | Matthew J Black proposed openstack/puppet-neutron: Added geneve type_driver config options https://review.openstack.org/336115 | 16:40 |
*** cdelatte has joined #puppet-openstack | 16:41 | |
*** beddari has quit IRC | 16:41 | |
*** chem is now known as chem|off | 16:42 | |
*** beddari has joined #puppet-openstack | 16:47 | |
*** beddari has quit IRC | 16:51 | |
*** rwsu has joined #puppet-openstack | 16:52 | |
*** beddari has joined #puppet-openstack | 16:57 | |
*** _ducttape_ has joined #puppet-openstack | 16:58 | |
*** beddari has quit IRC | 17:01 | |
*** derekh has quit IRC | 17:02 | |
*** ducttape_ has quit IRC | 17:03 | |
*** _ducttape_ has quit IRC | 17:03 | |
*** beddari has joined #puppet-openstack | 17:07 | |
*** jpena is now known as jpena|off | 17:08 | |
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:09 |
*** foxhunt_away is now known as foxhunt72 | 17:10 | |
*** beddari has quit IRC | 17:11 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-neutron: Add memcached_servers for keystone authtoken https://review.openstack.org/336087 | 17:12 |
*** ducttape_ has joined #puppet-openstack | 17:13 | |
*** ducttape_ has quit IRC | 17:13 | |
*** ducttape_ has joined #puppet-openstack | 17:15 | |
*** beddari has joined #puppet-openstack | 17:17 | |
*** degorenko is now known as _degorenko|afk | 17:21 | |
*** beddari has quit IRC | 17:21 | |
*** _ducttape_ has joined #puppet-openstack | 17:24 | |
*** ducttape_ has quit IRC | 17:25 | |
*** foxhunt72 is now known as foxhunt_away | 17:26 | |
*** beddari has joined #puppet-openstack | 17:26 | |
*** foxhunt_away is now known as foxhunt72 | 17:27 | |
*** cdelatte has quit IRC | 17:27 | |
openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Add retry to keystone_user.exists? https://review.openstack.org/335600 | 17:28 |
*** foxhunt72 is now known as foxhunt_away | 17:28 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: disable trove https://review.openstack.org/336131 | 17:28 |
*** foxhunt_away is now known as foxhunt72 | 17:30 | |
*** imcsk8_ is now known as imcsk8|doctor | 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:31 |
*** beddari has quit IRC | 17:31 | |
EmilienM | I'll try to reneable trove asap | 17:32 |
*** aernhart has quit IRC | 17:36 | |
*** beddari has joined #puppet-openstack | 17:37 | |
*** dgurtner has quit IRC | 17:37 | |
*** aernhart has joined #puppet-openstack | 17:37 | |
*** foxhunt72 is now known as foxhunt_away | 17:39 | |
*** trown|lunch is now known as trown | 17:40 | |
*** beddari has quit IRC | 17:42 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-heat: Add memcached_servers for keystone authtoken https://review.openstack.org/336135 | 17:43 |
*** gfidente has quit IRC | 17:44 | |
*** beddari has joined #puppet-openstack | 17:47 | |
*** beddari has quit IRC | 17:52 | |
*** beddari has joined #puppet-openstack | 17:56 | |
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:01 |
*** beddari has quit IRC | 18:02 | |
mwhahaha | K I'll look in a bit | 18:02 |
xarses | whats the fuel-ci recheck passphrase? | 18:03 |
iurygregory | mwhahaha, can you look at 335727 335744? ^^ | 18:04 |
*** beddari has joined #puppet-openstack | 18:07 | |
*** hjensas has joined #puppet-openstack | 18:15 | |
*** hjensas has joined #puppet-openstack | 18:15 | |
*** beddari has quit IRC | 18:16 | |
*** beddari has joined #puppet-openstack | 18:17 | |
mwhahaha | fuel-ci: recheck | 18:17 |
*** beddari has quit IRC | 18:22 | |
*** amoralej is now known as amoralej|off | 18:24 | |
*** beddari has joined #puppet-openstack | 18:25 | |
*** xb_ has quit IRC | 18:26 | |
*** beddari has quit IRC | 18:32 | |
openstackgerrit | Merged openstack/puppet-designate: Add Xenial to operatingsystemrelease https://review.openstack.org/335744 | 18:36 |
*** beddari has joined #puppet-openstack | 18:36 | |
*** shardy has quit IRC | 18:41 | |
*** beddari has quit IRC | 18:42 | |
openstackgerrit | Merged openstack/puppet-ceph: Add Xenial to operatingsystemrelease https://review.openstack.org/335716 | 18:44 |
*** beddari has joined #puppet-openstack | 18:46 | |
*** vinsh has quit IRC | 18:48 | |
*** beddari has quit IRC | 18:56 | |
*** beddari has joined #puppet-openstack | 18:57 | |
*** hjensas has quit IRC | 18:58 | |
*** aernhart has quit IRC | 18:58 | |
*** aernhart has joined #puppet-openstack | 18:59 | |
*** beddari has quit IRC | 19:02 | |
*** beddari has joined #puppet-openstack | 19:06 | |
*** beddari has quit IRC | 19:12 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-murano: Add memcached_servers for keystone authtoken https://review.openstack.org/336261 | 19:12 |
iurygregory | EmilienM, you around? | 19:15 |
*** beddari has joined #puppet-openstack | 19:17 | |
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:21 |
*** beddari has quit IRC | 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:22 |
iurygregory | tks o/ i have the patch in my notebook at home i'll push =) | 19:23 |
*** beddari has joined #puppet-openstack | 19:27 | |
mwhahaha | sure | 19:28 |
*** vinsh has joined #puppet-openstack | 19:29 | |
*** johnmilton has quit IRC | 19:31 | |
*** beddari has quit IRC | 19:32 | |
*** beddari has joined #puppet-openstack | 19:36 | |
*** beddari has quit IRC | 19:46 | |
*** beddari_ has joined #puppet-openstack | 19:46 | |
*** beddari_ is now known as beddari | 19:46 | |
*** hjensas has joined #puppet-openstack | 19:49 | |
*** beddari has quit IRC | 19:56 | |
*** beddari has joined #puppet-openstack | 19:56 | |
*** beddari has quit IRC | 20:02 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-cinder: Add memcached_servers for keystone authtoken https://review.openstack.org/336270 | 20:02 |
*** beddari has joined #puppet-openstack | 20:07 | |
*** _ducttape_ has quit IRC | 20:07 | |
*** ducttape_ has joined #puppet-openstack | 20:09 | |
*** beddari has quit IRC | 20:12 | |
*** beddari has joined #puppet-openstack | 20:15 | |
*** jayg is now known as jayg|g0n3 | 20:21 | |
*** beddari has quit IRC | 20:22 | |
*** beddari has joined #puppet-openstack | 20:27 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-murano: Add Ubuntu to metadata https://review.openstack.org/336279 | 20:28 |
*** cdelatte has joined #puppet-openstack | 20:30 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-sahara: Add Ubuntu to metadata https://review.openstack.org/336280 | 20:31 |
*** beddari has quit IRC | 20:32 | |
*** cdelatte has quit IRC | 20:34 | |
*** beddari has joined #puppet-openstack | 20:37 | |
*** imcsk8|doctor is now known as imcsk8 | 20:39 | |
*** cdelatte has joined #puppet-openstack | 20:39 | |
*** rcernin has joined #puppet-openstack | 20:41 | |
*** beddari has quit IRC | 20:42 | |
*** beddari has joined #puppet-openstack | 20:46 | |
*** beddari has quit IRC | 20:52 | |
*** beddari has joined #puppet-openstack | 20:56 | |
*** beddari has quit IRC | 21:02 | |
*** beddari has joined #puppet-openstack | 21:07 | |
*** cdelatte has quit IRC | 21:09 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-sahara: Add memcached_servers for keystone authtoken https://review.openstack.org/336290 | 21:09 |
*** beddari has quit IRC | 21:12 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-heat: Add memcached_servers for keystone authtoken https://review.openstack.org/336135 | 21:14 |
openstackgerrit | Alex Schultz proposed openstack/puppet-ironic: Add memcached_servers for keystone authtoken https://review.openstack.org/336191 | 21:15 |
*** beddari has joined #puppet-openstack | 21:16 | |
*** trown is now known as trown|outtypewww | 21:23 | |
*** aernhart has quit IRC | 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 has joined #puppet-openstack | 21:24 | |
*** beddari has quit IRC | 21:26 | |
*** beddari_ has joined #puppet-openstack | 21:26 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-murano: Add memcached_servers for keystone authtoken https://review.openstack.org/336261 | 21:27 |
*** beddari_ is now known as beddari | 21:27 | |
*** beddari has quit IRC | 21:32 | |
*** ayoung has quit IRC | 21:34 | |
openstackgerrit | Merged openstack/puppet-congress: Add Xenial to metadata https://review.openstack.org/335735 | 21:35 |
*** beddari has joined #puppet-openstack | 21:36 | |
*** aernhart has quit IRC | 21:36 | |
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:40 |
*** hjensas has quit IRC | 21:42 | |
*** beddari has quit IRC | 21:42 | |
openstackgerrit | Merged openstack/puppet-rally: Update metatada https://review.openstack.org/335763 | 21:42 |
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 |
*** rwsu has quit IRC | 21:45 | |
*** iurygregory has quit IRC | 21:45 | |
*** trown|outtypewww has quit IRC | 21:45 | |
*** jgrassler has quit IRC | 21:45 | |
*** hogepodge has quit IRC | 21:45 | |
*** bogdando has quit IRC | 21:45 | |
openstackgerrit | Merged openstack/puppet-octavia: Update metadata https://review.openstack.org/335756 | 21:45 |
*** beddari has joined #puppet-openstack | 21:47 | |
*** cdelatte has joined #puppet-openstack | 21:51 | |
*** beddari has quit IRC | 21:56 | |
*** beddari has joined #puppet-openstack | 21:57 | |
EmilienM | m | 21:59 |
EmilienM | mwhahaha: what's up? | 21:59 |
EmilienM | Iury pinged me oops | 21:59 |
*** iurygregory has joined #puppet-openstack | 22:01 | |
*** beddari has quit IRC | 22:02 | |
EmilienM | iurygregory: hey sorry I was afk | 22:03 |
iurygregory | EmilienM, no problem o/ | 22:03 |
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:05 |
*** beddari has joined #puppet-openstack | 22:06 | |
*** cdelatte has quit IRC | 22:09 | |
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:14 |
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:15 |
*** beddari has quit IRC | 22:16 | |
bkero | \o/ | 22:16 |
*** beddari_ has joined #puppet-openstack | 22:17 | |
iurygregory | \o/ | 22:17 |
*** beddari_ is now known as beddari | 22:17 | |
iurygregory | nice | 22:17 |
openstackgerrit | Merged openstack/puppet-tempest: Remove 12.04 and add 16.04 https://review.openstack.org/335767 | 22:18 |
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:19 |
openstackgerrit | Merged openstack/puppet-zaqar: Add xenial to metadata https://review.openstack.org/335773 | 22:21 |
*** derekh has joined #puppet-openstack | 22:22 | |
*** beddari has quit IRC | 22:22 | |
EmilienM | bkero: dunno | 22:23 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-mistral: Add Xenial to metadata https://review.openstack.org/335752 | 22:23 |
*** beddari has joined #puppet-openstack | 22:27 | |
*** cdelatte has joined #puppet-openstack | 22:27 | |
*** ntpttr has quit IRC | 22:31 | |
*** ntpttr has joined #puppet-openstack | 22:31 | |
openstackgerrit | Merged openstack/puppet-openstack-integration: scenario003: disable trove https://review.openstack.org/336131 | 22:32 |
*** beddari has quit IRC | 22:32 | |
*** beddari has joined #puppet-openstack | 22:37 | |
*** beddari has quit IRC | 22:42 | |
*** dfisher has quit IRC | 22:45 | |
*** beddari has joined #puppet-openstack | 22:46 | |
*** ducttape_ has quit IRC | 22:49 | |
*** cdelatte has quit IRC | 22:52 | |
*** beddari has quit IRC | 22:52 | |
*** mesamoo has quit IRC | 22:52 | |
iurygregory | richm, you around | 22:53 |
iurygregory | ? | 22:53 |
*** beddari has joined #puppet-openstack | 22:56 | |
*** beddari has quit IRC | 23:02 | |
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:03 |
iurygregory | sorry to bother you guys at this time XD | 23:04 |
*** beddari has joined #puppet-openstack | 23:07 | |
*** ayoung has joined #puppet-openstack | 23:09 | |
*** beddari has quit IRC | 23:12 | |
*** beddari has joined #puppet-openstack | 23:17 | |
*** markvoelker has quit IRC | 23:21 | |
*** saneax_AFK is now known as saneax | 23:22 | |
*** beddari has quit IRC | 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 has joined #puppet-openstack | 23:27 | |
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:29 |
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:30 |
*** beddari has quit IRC | 23:32 | |
iurygregory | humm i'll update the patch and ask about this to chem =) tks richm | 23:33 |
*** beddari has joined #puppet-openstack | 23:36 | |
*** cdelatte has joined #puppet-openstack | 23:44 | |
richm | iurygregory: ok | 23:46 |
*** beddari has quit IRC | 23:46 | |
*** beddari has joined #puppet-openstack | 23:47 | |
*** derekh has quit IRC | 23:47 | |
*** beddari has quit IRC | 23:52 | |
*** beddari has joined #puppet-openstack | 23:57 | |
*** hogepodge has joined #puppet-openstack | 23:59 | |
*** xarses has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!