richm | nope | 00:00 |
---|---|---|
richm | whiskey tango foxtrot | 00:00 |
*** aimon_ has quit IRC | 00:01 | |
*** xarses has joined #puppet-openstack | 00:09 | |
*** dgurtner has quit IRC | 00:09 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Fix module install in beaker https://review.openstack.org/198928 | 00:17 |
crinkle | richm: ^ i have a theory | 00:17 |
richm | I'm all ears | 00:22 |
crinkle | richm: see https://review.openstack.org/198928 | 00:24 |
*** ducttape_ has joined #puppet-openstack | 00:31 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Fix module install in beaker https://review.openstack.org/198928 | 00:32 |
*** aimon has joined #puppet-openstack | 00:33 | |
*** ducttape_ has quit IRC | 00:40 | |
*** morazi has quit IRC | 00:45 | |
*** saneax has quit IRC | 00:55 | |
*** jasondotstar has quit IRC | 01:12 | |
*** ducttape_ has joined #puppet-openstack | 01:14 | |
*** bklei has joined #puppet-openstack | 01:15 | |
*** jasondotstar has joined #puppet-openstack | 01:25 | |
*** ducttape_ has quit IRC | 01:32 | |
openstackgerrit | Saksham Varma proposed openstack/puppet-neutron: N1Kv: Adding nexus1000v puppet class https://review.openstack.org/192944 | 01:33 |
crinkle | EmilienM: there's an issue with the upgrade tests, it looks like if HEAD~1 is already broken then a commit to fix it won't pass http://logs.openstack.org/28/198928/2/check/gate-puppet-keystone-puppet-beaker-rspec-upgrade-dsvm-trusty/1f4ceb0/console.html | 01:39 |
*** stevemar has joined #puppet-openstack | 01:41 | |
*** bklei has quit IRC | 01:54 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-heat: Support setting instance_user to an empty string https://review.openstack.org/197147 | 02:01 |
*** xingchao has joined #puppet-openstack | 02:04 | |
*** xingchao has quit IRC | 02:13 | |
mfisch | awesome to see all these mirantis/fuel patches coming in | 02:13 |
*** richm has quit IRC | 02:33 | |
openstackgerrit | Steve Baker proposed openstack/puppet-heat: Allow empty string instance_user https://review.openstack.org/198942 | 02:40 |
*** stevemar has quit IRC | 02:54 | |
*** stevemar has joined #puppet-openstack | 02:54 | |
openstackgerrit | Matt Fischer proposed openstack/puppet-heat: Support setting instance_user to an empty string https://review.openstack.org/197147 | 03:13 |
*** arnaud_orange1 has joined #puppet-openstack | 03:30 | |
*** arnaud_orange has quit IRC | 03:31 | |
openstackgerrit | gaoshilin proposed openstack/puppet-ironic: fix log_dir in ironic.conf https://review.openstack.org/198225 | 03:52 |
*** ianw has quit IRC | 03:56 | |
*** ianw has joined #puppet-openstack | 03:56 | |
EmilienM | crinkle: yes, if master is broken, upgrade flow will never pass CI | 04:07 |
EmilienM | this is why it's not voting now | 04:08 |
*** bklei has joined #puppet-openstack | 04:15 | |
*** bklei has quit IRC | 04:19 | |
openstackgerrit | Merged openstack/puppet-keystone: Fix module install in beaker https://review.openstack.org/198928 | 04:39 |
*** vinsh has quit IRC | 04:51 | |
*** saneax has joined #puppet-openstack | 05:06 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 05:08 |
*** gildub has joined #puppet-openstack | 05:15 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 05:24 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: fix pep8 errors https://review.openstack.org/197998 | 05:28 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 05:37 |
openstackgerrit | Merged openstack/puppet-swift: Add max_header_size field for PKI tokens https://review.openstack.org/198057 | 05:39 |
*** stamak has joined #puppet-openstack | 05:56 | |
*** stamak has quit IRC | 06:01 | |
*** bklei has joined #puppet-openstack | 06:04 | |
*** vinsh has joined #puppet-openstack | 06:08 | |
*** bklei has quit IRC | 06:08 | |
*** vinsh has quit IRC | 06:13 | |
*** LimorStotland has joined #puppet-openstack | 06:13 | |
*** tdb has quit IRC | 06:26 | |
*** tdb has joined #puppet-openstack | 06:26 | |
*** LimorStotland has quit IRC | 06:35 | |
*** jtomasek has joined #puppet-openstack | 06:50 | |
*** stevemar has quit IRC | 06:59 | |
*** stevemar has joined #puppet-openstack | 07:00 | |
*** jpena has joined #puppet-openstack | 07:08 | |
*** stevemar has quit IRC | 07:11 | |
*** dgurtner has joined #puppet-openstack | 07:18 | |
*** dgurtner has quit IRC | 07:18 | |
*** dgurtner has joined #puppet-openstack | 07:18 | |
*** jistr has joined #puppet-openstack | 07:37 | |
*** LimorStotland has joined #puppet-openstack | 07:44 | |
*** stamak has joined #puppet-openstack | 07:45 | |
*** paramite has joined #puppet-openstack | 07:51 | |
*** bklei has joined #puppet-openstack | 07:52 | |
*** bklei has quit IRC | 07:57 | |
*** stamak has quit IRC | 08:02 | |
*** LimorStotland has quit IRC | 08:06 | |
*** vinsh has joined #puppet-openstack | 08:10 | |
*** ToMiles has quit IRC | 08:10 | |
*** imcsk8 is now known as imcsk8|zZz | 08:10 | |
*** stevemar has joined #puppet-openstack | 08:12 | |
*** vinsh has quit IRC | 08:14 | |
*** stevemar has quit IRC | 08:16 | |
*** derekh has joined #puppet-openstack | 08:17 | |
*** gildub has quit IRC | 08:20 | |
*** LimorStotland has joined #puppet-openstack | 08:25 | |
*** LimorStotland has quit IRC | 08:44 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-cinder: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197015 | 08:49 |
*** LimorStotland has joined #puppet-openstack | 08:51 | |
*** gfidente has joined #puppet-openstack | 08:52 | |
*** gfidente is now known as gfidente|afk | 08:52 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ironic: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197016 | 08:56 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-neutron: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197054 | 09:00 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 09:01 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 09:02 |
trash | haha, nova list does "X-Auth-Project-Id: admin" on the curl. ID... "admin"... ;) | 09:02 |
trash | seems like any openstack command line client does stuff differently currently | 09:02 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-sahara: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197023 | 09:04 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-glance: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197024 | 09:06 |
*** cooldharma06 has joined #puppet-openstack | 09:06 | |
cooldharma06 | how to install puppet-openstack any reference guide or links.? | 09:07 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-keystone: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197028 | 09:08 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-heat: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197031 | 09:09 |
*** jpena is now known as jpena|doctor | 09:10 | |
*** LimorStotland has quit IRC | 09:22 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-glance: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197024 | 09:22 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 09:25 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-neutron: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197054 | 09:30 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-nova: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197013 | 09:31 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 09:33 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-cinder: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197015 | 09:35 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-glance: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197024 | 09:36 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-heat: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197031 | 09:38 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ironic: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197016 | 09:39 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-keystone: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197028 | 09:40 |
*** bklei has joined #puppet-openstack | 09:41 | |
*** bklei has quit IRC | 09:46 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-neutron: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197054 | 09:47 |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-sahara: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197023 | 09:48 |
*** _joes_ has quit IRC | 09:52 | |
*** paramite is now known as paramite|afk | 09:57 | |
*** _joes_ has joined #puppet-openstack | 09:59 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 10:04 |
*** gchamoul is now known as gchamoul|afk|lun | 10:04 | |
*** gchamoul|afk|lun is now known as gchamoul|afk | 10:05 | |
*** _joes_ has quit IRC | 10:09 | |
*** _joes_ has joined #puppet-openstack | 10:11 | |
*** Nick_joes has joined #puppet-openstack | 10:12 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-cinder: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197015 | 10:14 |
*** _joes_ has quit IRC | 10:16 | |
*** stamak has joined #puppet-openstack | 10:20 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 10:25 |
*** gfidente|afk has quit IRC | 10:28 | |
openstackgerrit | Timur Nurlygayanov proposed openstack/puppet-ceilometer: Fixed issue with rabbit_ha_queues parameter https://review.openstack.org/197020 | 10:51 |
*** paramite|afk is now known as paramite | 10:56 | |
*** Nick_joes_ has joined #puppet-openstack | 11:07 | |
*** dalgaaf has joined #puppet-openstack | 11:08 | |
*** Nick_joes_ has quit IRC | 11:10 | |
*** Nick_joes_ has joined #puppet-openstack | 11:10 | |
*** Nick_joes has quit IRC | 11:10 | |
*** delatte has quit IRC | 11:11 | |
*** jpena|doctor is now known as jpena | 11:22 | |
*** Nick_joes_ has quit IRC | 11:26 | |
*** bklei has joined #puppet-openstack | 11:30 | |
*** delatte has joined #puppet-openstack | 11:31 | |
*** bklei has quit IRC | 11:35 | |
*** paramite is now known as paramite|afk | 11:44 | |
*** paramite|afk is now known as paramite | 11:44 | |
*** prad has joined #puppet-openstack | 11:45 | |
*** stevemar has joined #puppet-openstack | 11:50 | |
*** Nick_joes_ has joined #puppet-openstack | 11:50 | |
*** stevemar has quit IRC | 11:54 | |
*** jasondotstar has quit IRC | 11:57 | |
*** markvoelker has quit IRC | 11:59 | |
*** markvoelker has joined #puppet-openstack | 11:59 | |
*** derekh has quit IRC | 12:01 | |
*** derekh has joined #puppet-openstack | 12:02 | |
*** jayg|g0n3 is now known as jayg | 12:03 | |
*** ferest has joined #puppet-openstack | 12:08 | |
*** jpena is now known as jpena|lunch | 12:15 | |
*** dprince has joined #puppet-openstack | 12:15 | |
*** ferest has quit IRC | 12:15 | |
*** samuelBartel has joined #puppet-openstack | 12:23 | |
iurygregory | good morning people | 12:23 |
*** LimorStotland has joined #puppet-openstack | 12:26 | |
*** Nick_joes has joined #puppet-openstack | 12:29 | |
*** markvoelker has quit IRC | 12:31 | |
*** Nick_joes_ has quit IRC | 12:33 | |
*** gchamoul|afk is now known as gchamoul | 12:34 | |
*** vinsh has joined #puppet-openstack | 12:39 | |
*** markvoelker has joined #puppet-openstack | 12:40 | |
*** bklei has joined #puppet-openstack | 12:46 | |
*** bklei has quit IRC | 12:50 | |
*** morazi has joined #puppet-openstack | 12:55 | |
EmilienM | good morning | 12:57 |
EmilienM | cooldharma06: we have this wiki page which is a good start: https://wiki.openstack.org/wiki/Puppet | 12:58 |
iurygregory | morning | 13:00 |
*** saneax has quit IRC | 13:01 | |
*** gfidente has joined #puppet-openstack | 13:05 | |
*** jasondotstar has joined #puppet-openstack | 13:09 | |
*** jasondot_ has joined #puppet-openstack | 13:10 | |
*** jasondot_ has quit IRC | 13:13 | |
*** jasondotstar has quit IRC | 13:14 | |
*** jasondotstar has joined #puppet-openstack | 13:14 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tempest: Implement more Tempest options https://review.openstack.org/198572 | 13:15 |
*** dfisher has joined #puppet-openstack | 13:16 | |
*** skolekonov has quit IRC | 13:16 | |
*** bklei has joined #puppet-openstack | 13:20 | |
*** jpena|lunch is now known as jpena | 13:22 | |
cooldharma06 | EmilienM i tried to install openstack from puppet modules | 13:24 |
cooldharma06 | i am facing the following error -> Error: MySQL setup failed. The inferred location of the database based on the | 13:25 |
cooldharma06 | openstack::network::management hiera value is . The | 13:25 |
cooldharma06 | explicit address from openstack::controller::address::management | 13:25 |
cooldharma06 | is 172.16.33.4 and inferred address is . Please correct this difference. at /etc/puppet/modules/openstack/manifests/profile/mysql.pp:9 on node puppet-opens.chennai.cdac.in | 13:25 |
cooldharma06 | sorry for flooding | 13:25 |
cooldharma06 | http://pastebin.com/KGDUntMA | 13:27 |
openstackgerrit | Merged openstack/puppet-monasca: Update mysql schema definitions, fix thresh startup script dependency https://review.openstack.org/198838 | 13:27 |
EmilienM | cooldharma06: I have no idea which module are you using | 13:33 |
*** richm has joined #puppet-openstack | 13:38 | |
cooldharma06 | EmilienM i am using puppetlabs-openstack | 13:38 |
EmilienM | cooldharma06: this module is not developed by Puppet OpenStack group, but though we would be happy to help you! | 13:38 |
*** stevemar has joined #puppet-openstack | 13:39 | |
*** jasondotstar has quit IRC | 13:40 | |
EmilienM | richm: good morning | 13:41 |
EmilienM | richm: I drop some comments in our patches, but if you prefer we can discuss here, much easier | 13:41 |
*** stevemar has quit IRC | 13:42 | |
cooldharma06 | EmilienM thanks and also i saw this one -> https://ask.puppetlabs.com/questions/6567/revisions/ if your free time have a look on this and suggest some ideas to solve this | 13:48 |
cooldharma06 | *in | 13:48 |
EmilienM | richm: last night, I spent time to read code and understand why you did the ugly restart in the beaker patch | 13:50 |
EmilienM | richm: tell me if I'm wrong but, in fact in lib/puppet/provider/keystone_domain/openstack.rb when you update the default domain id you should restart keystone, right? | 13:50 |
EmilienM | richm: if so, maybe we could run a : puppet('resource', 'service', resource[:service_name], 'restart') | 13:51 |
EmilienM | richm: but only when updating the ID | 13:51 |
richm | EmilienM: right | 13:52 |
richm | I'm not familiar with that puppet command | 13:53 |
EmilienM | no prob | 13:53 |
richm | The other alternative is | 13:53 |
richm | use sql to write to the database directly | 13:53 |
openstackgerrit | Drew Fisher proposed openstack/puppet-designate: Remove POSIX users, groups, and file modes. https://review.openstack.org/197336 | 13:53 |
EmilienM | richm: no way | 13:53 |
richm | yeah - ugly - but a (remotely) possible alternative | 13:54 |
EmilienM | no | 13:54 |
EmilienM | we won't do that | 13:54 |
richm | ok | 13:54 |
richm | If there is a puppet-ly way to restart keystone, that would be best | 13:54 |
EmilienM | richm: let me try my alternative first and see how it behaves. Can you just confirm the cases we need to restart ? | 13:54 |
richm | afaik, only writing the default_domain_id | 13:55 |
EmilienM | richm: L89 and L97 ? | 13:55 |
richm | https://review.openstack.org/#/c/174439/37/lib/puppet/provider/keystone_domain/openstack.rb | 13:55 |
EmilienM | richm: we won't do SQL because we want to be storage backend agnostic. People might not use MySQL or PGSQL here | 13:56 |
EmilienM | richm: yes this file, but the lines? | 13:56 |
richm | After line 101 | 13:56 |
richm | when the default_domain_id change is written to the file | 13:56 |
EmilienM | richm: perfect | 13:56 |
richm | self.class.keystone_file.store | 13:56 |
EmilienM | richm: I'm testing it locally and I push | 13:56 |
richm | after that line, restart keystone | 13:56 |
richm | excellent | 13:56 |
EmilienM | ack | 13:56 |
EmilienM | richm: also: https://review.openstack.org/#/c/178828/55..56/spec/acceptance/basic_keystone_spec.rb,cm | 13:57 |
EmilienM | richm: is my comment valid (on keystone::roles::admin) | 13:57 |
EmilienM | ? | 13:57 |
richm | looking | 13:57 |
richm | EmilienM: You are correct - using a different service_user_domain and service_tenant_domain breaks the ability to authenticate with the service account with v2 | 14:00 |
EmilienM | richm: ok, I don't think it's a blocker | 14:00 |
richm | But I didn't think that mattered with the test - we're just trying to see if v3 auth is working? | 14:01 |
EmilienM | richm: brb | 14:01 |
richm | not to give an example of how this should be actually deployed? | 14:01 |
EmilienM | richm: yes but I also want to check if people still using v2 can use the same manifests as before | 14:01 |
richm | hmm | 14:01 |
richm | ok | 14:01 |
richm | then we'll have to redesign that test | 14:01 |
openstackgerrit | Gael Chamoulaud proposed openstack/puppet-neutron: Sync DB when enabling FWAAS https://review.openstack.org/196939 | 14:02 |
EmilienM | probably | 14:02 |
richm | ok | 14:03 |
richm | The goal of this first phase of v3 was to ensure that no existing manifests/composition layers had to change | 14:03 |
richm | Which means they don't know anything about domains | 14:03 |
richm | which means they always use the default domain | 14:04 |
richm | which means we should just use that to do v3 testing | 14:04 |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add distribute mode for Sahara https://review.openstack.org/192721 | 14:04 |
richm | that is, our test should _not_ specify any domains, if we want to simulate how existing code will work | 14:04 |
richm | that is, should not specify any domain parameters to any classes or resources | 14:04 |
richm | however, we can explicitly specify the domain arguments to openstack e.g. --os-user-domain-name Default --os-auth-url ..../v3 --os-identity-api-version 3 | 14:05 |
richm | in order to test that v3 auth/commands are working | 14:05 |
*** xingchao has joined #puppet-openstack | 14:07 | |
EmilienM | richm: yes but you brought a change in the composition layer, since you add a new param (see my comment) | 14:09 |
EmilienM | richm: the service_tenant_domain parameter | 14:09 |
EmilienM | is that a big deal if users don't have it set if they don't run v3 API ? | 14:09 |
EmilienM | richm: the problem I have with my proposal (restart thing) is that keystone will be restarted two times in the catalog | 14:13 |
EmilienM | richm: can't we pull out this code and configure the parameter using keystone_config *before* keystone service starts? | 14:13 |
richm | EmilienM: Do you mean the default_domain_id parameter? | 14:15 |
EmilienM | richm: yes | 14:15 |
richm | EmilienM: https://bugs.launchpad.net/keystone/+bug/1472285 | 14:27 |
openstack | Launchpad bug 1472285 in Keystone "set default domain dynamically" [Undecided,New] | 14:27 |
EmilienM | richm: nice | 14:27 |
*** ducttape_ has joined #puppet-openstack | 14:27 | |
EmilienM | richm: it won't stop us \o/ | 14:27 |
richm | EmilienM: you figured out how to do the restart? cool | 14:31 |
EmilienM | richm: yes, I'm testing it atm | 14:31 |
openstackgerrit | Andrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/197181 | 14:35 |
openstackgerrit | Andrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/197181 | 14:40 |
*** vinsh has quit IRC | 14:41 | |
openstackgerrit | Andrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/197181 | 14:42 |
*** arnaud_orange1 has quit IRC | 14:46 | |
EmilienM | our weekly meeting starts in 13 minutes! our agenda is here: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20150707 | 14:48 |
*** mdorman has joined #puppet-openstack | 14:52 | |
*** ducttape_ has quit IRC | 14:53 | |
EmilienM | mdorman: hey, please look at sileht's reply on the ML | 14:53 |
mdorman | yup, justlooking at that now | 14:54 |
openstackgerrit | Merged openstack/puppet-keystone: fix pep8 errors https://review.openstack.org/197998 | 14:57 |
*** Nick_joes_ has joined #puppet-openstack | 14:58 | |
*** vinsh has joined #puppet-openstack | 15:01 | |
sbadia | (weekly meeting on #openstack-meeting-4) | 15:01 |
sbadia | (hi) | 15:01 |
*** Nick_joes has quit IRC | 15:01 | |
*** sbadia sets mode: +oooo clayton dprince EmilienM francois | 15:03 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add distribute mode for Sahara https://review.openstack.org/192721 | 15:06 |
*** topshare has joined #puppet-openstack | 15:08 | |
*** topshare has quit IRC | 15:08 | |
*** topshare has joined #puppet-openstack | 15:09 | |
*** topshare has quit IRC | 15:09 | |
*** topshare has joined #puppet-openstack | 15:09 | |
*** imcsk8|zZz is now known as imcsk8 | 15:18 | |
*** saneax has joined #puppet-openstack | 15:21 | |
*** gchamoul is now known as gchamoul|AFK | 15:21 | |
*** stevemar has joined #puppet-openstack | 15:22 | |
*** xingchao has quit IRC | 15:23 | |
*** LimorStotland has quit IRC | 15:26 | |
*** cwolferh has quit IRC | 15:26 | |
openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Add distribute mode for Sahara https://review.openstack.org/192721 | 15:34 |
openstackgerrit | Andrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/197181 | 15:34 |
openstackgerrit | craig delatte proposed openstack/puppet-cinder: WIP - module to set quotas per volume type https://review.openstack.org/197572 | 15:35 |
openstackgerrit | Cody Herriges proposed openstack/puppet-nova: Enable nova service by default https://review.openstack.org/184656 | 15:36 |
*** xingchao has joined #puppet-openstack | 15:36 | |
*** spredzy is now known as spredzy|afk | 15:45 | |
*** bklei_ has joined #puppet-openstack | 15:46 | |
*** bklei has quit IRC | 15:47 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-ceilometer: Sync up module with deedb1e of puppet-modulesync-configs https://review.openstack.org/189862 | 15:54 |
*** paramite is now known as paramite|afk | 15:56 | |
openstackgerrit | Cody Herriges proposed openstack/puppet-ceilometer: Sync up module with deedb1e of puppet-modulesync-configs https://review.openstack.org/189862 | 15:59 |
*** jistr has quit IRC | 16:00 | |
*** mgarza_ has joined #puppet-openstack | 16:00 | |
*** jtomasek has quit IRC | 16:01 | |
*** xingchao has quit IRC | 16:03 | |
*** jistr has joined #puppet-openstack | 16:07 | |
*** aimon has quit IRC | 16:08 | |
*** cwolferh has joined #puppet-openstack | 16:13 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 16:14 |
EmilienM | richm: ^ | 16:14 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 16:14 |
EmilienM | richm, crinkle : https://review.openstack.org/#/c/175060/38..40/manifests/init.pp,cm | 16:15 |
*** bklei_ has quit IRC | 16:15 | |
*** bklei has joined #puppet-openstack | 16:16 | |
*** stamak has quit IRC | 16:18 | |
*** hogepodge has quit IRC | 16:21 | |
*** jpena has quit IRC | 16:21 | |
*** jpena has joined #puppet-openstack | 16:33 | |
degorenko | hi :) cores, please take a look on: https://review.openstack.org/#/c/198744 https://review.openstack.org/#/c/192721 | 16:34 |
openstackgerrit | Andrew Boik proposed openstack/puppet-neutron: Update Cisco ML2 classes and templates https://review.openstack.org/197181 | 16:34 |
*** jistr has quit IRC | 16:35 | |
*** aimon has joined #puppet-openstack | 16:47 | |
openstackgerrit | Ryan Brown proposed openstack/puppet-heat: Expose RPC response timeout as a puppet parameter. https://review.openstack.org/197964 | 16:54 |
_ody | EmilienM: https://review.openstack.org/#/c/189862 and its log, http://logs.openstack.org/62/189862/7/check/gate-puppet-ceilometer-puppet-beaker-rspec-upgrade-dsvm-trusty/922d026/console.html. Bundler issue? Looks like jenkins is doing the "bundle install" based on the patchset but then we manually checkout the previous commit. This causes issues with rspec and our Rakefile since it attempts to ru | 16:55 |
_ody | n "bundle install" with, in this case a different rspec-puppet version. | 16:55 |
EmilienM | _ody: it should pass, isn't? | 16:56 |
EmilienM | _ody: it pass for me on keystone https://review.openstack.org/#/c/175060 | 16:56 |
_ody | EmilienM: Should only happen with pinned gems change between commits. | 16:57 |
_ody | ...because we do "install" instead of "update" | 16:57 |
EmilienM | ah | 16:58 |
EmilienM | _ody: or we should drop the Gemlock ? | 16:58 |
_ody | EmilienM: You mean to delete it as part of the job? | 17:00 |
EmilienM | _ody: not yet I think | 17:02 |
EmilienM | _ody: I'll catchup after lunch, I'm afk now | 17:02 |
_ody | ok. | 17:02 |
*** jpena has quit IRC | 17:04 | |
*** ducttape_ has joined #puppet-openstack | 17:04 | |
*** xingchao has joined #puppet-openstack | 17:04 | |
*** gfidente has quit IRC | 17:11 | |
*** derekh has quit IRC | 17:12 | |
_ody | EmilienM: When you get back...or just move the "bundle install" http://bit.ly/1IHbD6r | 17:18 |
openstackgerrit | Mike Dorman proposed openstack/puppet-keystone: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194399 | 17:19 |
*** jtomasek has joined #puppet-openstack | 17:21 | |
*** stamak has joined #puppet-openstack | 17:22 | |
openstackgerrit | Mike Dorman proposed openstack/puppet-neutron: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194381 | 17:23 |
dfisher | erm. | 17:25 |
dfisher | so. | 17:25 |
*** xingchao has quit IRC | 17:25 | |
dfisher | the changes made to nova and cinder (so far -haven't checked others) to move rabbitmq settings to [oslo_messaging_rabbit] | 17:26 |
dfisher | that's no bueno for Juno | 17:26 |
openstackgerrit | Mike Dorman proposed openstack/puppet-heat: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194369 | 17:26 |
openstackgerrit | Mike Dorman proposed openstack/puppet-glance: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194354 | 17:27 |
mdorman | dfisher: correct. the heartbeating feature only applies to kilo | 17:28 |
dfisher | no, not heartbeating. | 17:28 |
mdorman | so these would not be backported to juno | 17:28 |
dfisher | moving rabbitmq settings to the [oslo_messaging_rabbit] inifile section | 17:28 |
dfisher | [oslo_messaging_rabbit] | 17:29 |
dfisher | rabbit_userid=guest | 17:29 |
dfisher | rabbit_password=guest | 17:29 |
dfisher | rabbit_ha_queues=False | 17:29 |
dfisher | rabbit_virtual_host=/ | 17:29 |
dfisher | rabbit_port=5672 | 17:29 |
dfisher | rabbit_host=myhost | 17:29 |
mdorman | iirc that stuff is only in master, not in the juno branch | 17:29 |
*** ducttape_ has quit IRC | 17:29 | |
dfisher | ^ no bueno for juno | 17:29 |
dfisher | i grabbed the 5.1.0 tagged csets ... | 17:29 |
dfisher | hrm. | 17:29 |
openstackgerrit | Mike Dorman proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194343 | 17:30 |
dfisher | commit 68c0cfe93572d4a7cdfa7bd68c4abd7904bba9fd | 17:30 |
dfisher | Merge: 81525ed 3a24742 | 17:30 |
dfisher | Author: Jenkins <jenkins@review.openstack.org> | 17:30 |
dfisher | Date: Mon Jun 8 21:51:17 2015 +0000 | 17:30 |
dfisher | Merge "Revert "Revert "Move rabbit/kombu settings to oslo_messaging_rabbit section""" | 17:30 |
dfisher | that one. | 17:30 |
dfisher | blah. will deal with it after lunch... | 17:30 |
openstackgerrit | Mike Dorman proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194336 | 17:30 |
mdorman | hmm. unsure about that. i know it had been merged and then reverted… surprised there is a reversion of the revert, heh. | 17:33 |
crinkle | that's not in 5.1.0 or stable/juno | 17:34 |
mdorman | that’s waht i thought.. | 17:39 |
openstackgerrit | Mike Dorman proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat https://review.openstack.org/194315 | 17:39 |
*** xarses has quit IRC | 17:43 | |
*** stamak has quit IRC | 17:48 | |
*** ducttape_ has joined #puppet-openstack | 17:53 | |
*** ddmitriev has joined #puppet-openstack | 17:56 | |
*** stamak has joined #puppet-openstack | 17:57 | |
*** reed_ has quit IRC | 17:59 | |
EmilienM | _ody: have you submitted the patch in gerrit? | 18:00 |
*** mgarza_ has quit IRC | 18:00 | |
*** xarses has joined #puppet-openstack | 18:01 | |
*** mgarza_ has joined #puppet-openstack | 18:06 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - the keystone_service resource https://review.openstack.org/184195 | 18:09 |
*** stevemar has left #puppet-openstack | 18:09 | |
*** hogepodge has joined #puppet-openstack | 18:11 | |
*** dgurtner has quit IRC | 18:14 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 18:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 18:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 18:16 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 18:16 |
*** hogepodge has quit IRC | 18:20 | |
*** stamak has quit IRC | 18:20 | |
*** paramite|afk is now known as paramite | 18:28 | |
EmilienM | _ody, crinkle: richm: the Exec works and beaker tests pass on v3 thing now :) | 18:33 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 18:38 |
*** mgarza_ has quit IRC | 18:38 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 18:38 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 18:39 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 18:39 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 18:39 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - examples https://review.openstack.org/197678 | 18:39 |
*** mgarza_ has joined #puppet-openstack | 18:45 | |
dfisher | crinkle/mdorman: thanks for pointing out that the rabbitmq goop isn't actually in the stable/juno branch. I was looking in the wrong place :( | 18:45 |
*** hogepodge has joined #puppet-openstack | 19:00 | |
xarses | EmilienM: so, https://review.openstack.org/#/c/197332/ fails upgrade still, but https://review.openstack.org/#/c/198925/ works with out yours | 19:00 |
xarses | (and others) | 19:01 |
EmilienM | xarses: ok | 19:01 |
*** stamak has joined #puppet-openstack | 19:04 | |
richm | I think I still may need to make a change to user, tenant, and user_role | 19:05 |
EmilienM | richm: ack | 19:05 |
richm | The way I implemented self.instances for those, it returns resources with both the short name 'admin' and the long name 'admin::admin_domain' - duplicates | 19:05 |
EmilienM | richm: is it related to crinkle's comment? | 19:05 |
richm | yes | 19:05 |
EmilienM | ok cool | 19:06 |
richm | I thought it might be useful if puppet resource showed both the short name and the long name | 19:06 |
richm | sort of like how a hostname query shows you the fqdn and the aliases | 19:06 |
richm | I'm also concerned because self.instances is the method that generates the resource names for comparison with the puppet catalog | 19:07 |
crinkle | i don't think that's how puppet resource should behave, and it also brings up questions of how prefetching works if there are duplicates | 19:07 |
richm | and I have no idea if users will use keystone_user { 'admin' } or keystone_user { 'admin::admin_domain' } | 19:07 |
richm | I guess if we force manifest writers/installers to know absolutely that the use of short names or long names for a resource means they must be used consistently, everywhere | 19:08 |
richm | then self.instances can skip adding the long name resource if the short name already exists | 19:08 |
richm | that is, if someone uses keystone_user { 'admin' } somewhere, then they cannot also refer to that same resource using keystone_user { 'admin::domain' } somewhere else | 19:09 |
*** xingchao has joined #puppet-openstack | 19:10 | |
richm | they must always and forever use 'admin' as the name of that resource | 19:10 |
crinkle | i think that's fine, same as we say package { 'httpd': name => 'apache2' } we still refer to it as 'httpd' | 19:11 |
crinkle | if there isn't a domain specified, why don't we just assume the domain is Default? | 19:12 |
richm | ok - I just checked the bp and it looks like no changes will be required there | 19:12 |
richm | crinkle: we do - we try to use the default_domain_id in the config, or 'Default' | 19:12 |
crinkle | okay | 19:12 |
*** xingchao has quit IRC | 19:14 | |
*** mgarza_ has quit IRC | 19:14 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 19:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 19:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 19:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 19:15 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 19:16 |
*** stamak has quit IRC | 19:18 | |
*** topshare has quit IRC | 19:19 | |
richm | ok - new v3 patches forthcoming for user, tenant, and user_role | 19:20 |
*** cwolferh has quit IRC | 19:20 | |
*** topshare has joined #puppet-openstack | 19:21 | |
*** mgarza has joined #puppet-openstack | 19:24 | |
mdorman | clayton: you around? | 19:25 |
*** linkedinyou has joined #puppet-openstack | 19:30 | |
mgagne | so glorious to see all those deprecation warnings | 19:41 |
mdorman | haha | 19:41 |
*** jtomasek has quit IRC | 20:00 | |
*** mgarza has quit IRC | 20:00 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 20:02 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 20:02 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 20:02 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 20:03 |
*** mgarza_ has joined #puppet-openstack | 20:03 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 20:04 |
*** jtomasek has joined #puppet-openstack | 20:06 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-ceilometer: Use module install script for acceptance tests https://review.openstack.org/199240 | 20:08 |
*** jtomasek has quit IRC | 20:09 | |
*** jtomasek has joined #puppet-openstack | 20:10 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-cinder: Use module install script for acceptance tests https://review.openstack.org/199241 | 20:10 |
*** cwolferh has joined #puppet-openstack | 20:15 | |
EmilienM | richm: regular beaker job pass now (with the exec). I'm digging the upgrade failures now | 20:17 |
*** jtomasek has quit IRC | 20:19 | |
richm | ok | 20:22 |
EmilienM | richm: it's pretty clear switching from v2 to v3 does not work with Puppet | 20:22 |
EmilienM | I wonder if it's a blocker | 20:22 |
EmilienM | I'm interested to know from mfisch, mfisch, crinkle about it ^ | 20:23 |
richm | I would say yes until we know more | 20:23 |
mfisch | whats the context for "switching"? | 20:24 |
mfisch | I think right now I have both APIs up | 20:24 |
mfisch | but nobody using V3 | 20:24 |
*** xingchao has joined #puppet-openstack | 20:24 | |
richm | if you use the puppet-keystone code from the kilo candidate, it will use the v3 api with openstack | 20:25 |
mfisch | so puppet will make v3 api calls, ok | 20:25 |
mfisch | that should be okay for us as long as its not adding domains | 20:25 |
EmilienM | see logs here: https://jenkins02.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-upgrade-dsvm-centos7/34/console | 20:25 |
richm | it doesn't add domains by default | 20:26 |
EmilienM | you'll end up with this error | 20:26 |
EmilienM | mfisch: no it doesn't | 20:26 |
EmilienM | but when you try to add, it fails if you're already have the resources | 20:26 |
*** paramite has quit IRC | 20:26 | |
openstackgerrit | Mathieu Gagné proposed openstack/puppet-nova: Use proper spelling in nova::compute::ironic parameters https://review.openstack.org/199245 | 20:27 |
EmilienM | the "openstack project set: error: unrecognized arguments: --domain service_domain" thing is weird | 20:27 |
mfisch | it is trying to use domains then | 20:27 |
EmilienM | at second run | 20:27 |
richm | to me it means that we are somehow trying to use the v2 api, which should not be possible | 20:27 |
EmilienM | yes, because we updated the composition layer | 20:27 |
EmilienM | if you don't, everything is fine | 20:27 |
richm | it's trying to use domains in the spec acceptance test because we are forcing it to | 20:27 |
EmilienM | mfisch: look openstack/puppet-keystone: | 20:27 |
EmilienM | oops | 20:27 |
richm | if you reuse existing manifests, it will not add domains | 20:27 |
EmilienM | https://review.openstack.org/#/c/193359/ | 20:28 |
EmilienM | mfisch: upgrade works when we don't touch our composition layer | 20:28 |
EmilienM | everything is backward compatible | 20:28 |
EmilienM | it starts to fail when you enable domains, etc | 20:28 |
mfisch | curious where --domain service_domain came from | 20:28 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-designate: Use module install script for acceptance tests https://review.openstack.org/199247 | 20:28 |
*** xingchao has quit IRC | 20:29 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 20:29 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 20:30 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 20:30 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 20:30 |
*** ducttape_ has quit IRC | 20:30 | |
mfisch | EmilienM: okay can you give me the exact issue? When you have a system thats been using v2 and you upgrade puppet-keystone, stuff breaks? | 20:30 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-glance: Use module install script for acceptance tests https://review.openstack.org/199250 | 20:30 |
EmilienM | mfisch: I can help you to reproduce | 20:31 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 20:31 |
EmilienM | mfisch: no it does not break if you don't change your composition layer | 20:31 |
EmilienM | but if you do, like in https://review.openstack.org/#/c/178828/61/spec/acceptance/basic_keystone_spec.rb,cm | 20:31 |
EmilienM | it fails | 20:31 |
openstackgerrit | Merged openstack/puppet-neutron: Add ability to specify auth_region https://review.openstack.org/198133 | 20:32 |
EmilienM | richm: I think it's because we add service_tenant_domain to the admin user | 20:32 |
mfisch | okay let me look | 20:32 |
mfisch | I'm rebuilding a few boxes might be slow response | 20:32 |
EmilienM | do you have an openstack cloud up and running? | 20:33 |
mfisch | yep no dev cloud atm | 20:33 |
mfisch | but I can bring one up | 20:33 |
EmilienM | mfisch: i can give up a tips to try a patch with vagrant | 20:33 |
*** dprince has quit IRC | 20:34 | |
mfisch | I can bring up a dev cloud pretty quickly | 20:34 |
EmilienM | mfisch: http://my1.fr/blog/puppet-module-functional-testing-with-vagrant-openstack-and-beaker/ | 20:34 |
mfisch | we use vagrant-openstack | 20:34 |
EmilienM | same | 20:34 |
EmilienM | the script checkout the patch though | 20:34 |
mfisch | mine are just dirty atm so I need to rebuild | 20:34 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-glance: Use module install script for acceptance tests https://review.openstack.org/199250 | 20:35 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-glance: Correct `puppet module list` check in beaker https://review.openstack.org/199253 | 20:35 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-designate: Correct `puppet module list` check in beaker https://review.openstack.org/199254 | 20:37 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-cinder: Correct `puppet module list` check in beaker https://review.openstack.org/199255 | 20:38 |
richm | EmilienM: ok | 20:38 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-ceilometer: Correct `puppet module list` check in beaker https://review.openstack.org/199256 | 20:38 |
EmilienM | richm: I'm trying to reproduce & dig why it fails - though it should be a blocker IMHO, because we're not breaking backward compatibility | 20:39 |
richm | you mean, it should not be a blocker | 20:40 |
EmilienM | richm: sorry yes | 20:41 |
richm | ok | 20:41 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-gnocchi: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199259 | 20:42 |
EmilienM | richm: on my laptop when I test upgrade: 19 examples, 0 failures | 20:43 |
EmilienM | wtf | 20:43 |
EmilienM | something in CI | 20:43 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-gnocchi: Correct `puppet module list` check in beaker https://review.openstack.org/199261 | 20:44 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-gnocchi: Use module install script for acceptance tests https://review.openstack.org/199262 | 20:44 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-heat: Correct `puppet module list` check in beaker https://review.openstack.org/199264 | 20:46 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-heat: Use module install script for acceptance tests https://review.openstack.org/199265 | 20:46 |
EmilienM | _ody: re- bundle install issue, have you submitted a patch in project-config? | 20:47 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-horizon: Correct `puppet module list` check in beaker https://review.openstack.org/199268 | 20:47 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-horizon: Use module install script for acceptance tests https://review.openstack.org/199269 | 20:47 |
EmilienM | mfisch: I think I found why | 20:49 |
*** jasondotstar has joined #puppet-openstack | 20:50 | |
*** ducttape_ has joined #puppet-openstack | 20:51 | |
*** jayg is now known as jayg|g0n3 | 20:52 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: support for keystone v3 api - examples https://review.openstack.org/197678 | 20:53 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-ironic: Correct `puppet module list` check in beaker https://review.openstack.org/199275 | 20:54 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-ironic: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199276 | 20:54 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-ironic: Use module install script for acceptance tests https://review.openstack.org/199277 | 20:54 |
mfisch | EmilienM: let me know I'm bringing up my env | 20:54 |
EmilienM | mfisch: it might be related to _ody remark lately | 20:54 |
EmilienM | about the way we update gems | 20:54 |
EmilienM | but I'm still digging | 20:54 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-cinder: Correct `puppet module list` check in beaker https://review.openstack.org/199255 | 20:56 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-cinder: Use module install script for acceptance tests https://review.openstack.org/199279 | 20:56 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-designate: Correct `puppet module list` check in beaker https://review.openstack.org/199254 | 20:57 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-designate: Use module install script for acceptance tests https://review.openstack.org/199280 | 20:57 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-glance: Correct `puppet module list` check in beaker https://review.openstack.org/199253 | 20:58 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-glance: Use module install script for acceptance tests https://review.openstack.org/199281 | 20:58 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-horizon: Use module install script for acceptance tests https://review.openstack.org/199282 | 20:58 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-heat: Use module install script for acceptance tests https://review.openstack.org/199283 | 20:59 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-gnocchi: Use module install script for acceptance tests https://review.openstack.org/199284 | 21:00 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-manila: Correct `puppet module list` check in beaker https://review.openstack.org/199287 | 21:03 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-manila: Use module install script for acceptance tests https://review.openstack.org/199288 | 21:03 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-neutron: Correct `puppet module list` check in beaker https://review.openstack.org/199289 | 21:05 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-neutron: Use module install script for acceptance tests https://review.openstack.org/199290 | 21:05 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-nova: Correct `puppet module list` check in beaker https://review.openstack.org/199292 | 21:07 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-nova: Use module install script for acceptance tests https://review.openstack.org/199293 | 21:07 |
EmilienM | crinkle: upgrade job was really broken | 21:09 |
EmilienM | crinkle: thanks to _ody, it should be better: https://review.openstack.org/#/c/199291 | 21:09 |
crinkle | yay \o/ | 21:11 |
EmilienM | and this is (I hope) why upgrade job is failing on keystone v3 | 21:12 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstacklib: Correct `puppet module list` check in beaker https://review.openstack.org/199294 | 21:12 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstacklib: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199295 | 21:13 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstacklib: Use module install script for acceptance tests https://review.openstack.org/199296 | 21:13 |
EmilienM | richm: I confirm! I reproduced the exact same bug | 21:13 |
EmilienM | and with _ody patch, upgrade is working! | 21:13 |
EmilienM | mfisch: give up your testing | 21:14 |
richm | excellent | 21:14 |
EmilienM | yeah ! | 21:14 |
*** delatte has quit IRC | 21:14 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-sahara: Correct `puppet module list` check in beaker https://review.openstack.org/199298 | 21:14 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-sahara: Use module install script for acceptance tests https://review.openstack.org/199299 | 21:14 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-swift: Correct `puppet module list` check in beaker https://review.openstack.org/199300 | 21:15 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-swift: Use module install script for acceptance tests https://review.openstack.org/199301 | 21:15 |
*** jasondotstar has quit IRC | 21:16 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-tempest: Correct `puppet module list` check in beaker https://review.openstack.org/199302 | 21:17 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-tempest: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199303 | 21:17 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-tempest: Use module install script for acceptance tests https://review.openstack.org/199304 | 21:17 |
fvollero | gosh, colleen is on fire today :/ | 21:19 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-trove: Correct `puppet module list` check in beaker https://review.openstack.org/199305 | 21:19 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-trove: Use module install script for acceptance tests https://review.openstack.org/199306 | 21:19 |
*** iurygregory has left #puppet-openstack | 21:20 | |
dfisher | you know, crinkle, if you like doing testing stuff this much … I have like 25 new resource types for Solaris that I need unit tests written for. ;) | 21:21 |
crinkle | >.< | 21:21 |
dfisher | we finally hired somebody to do that work. | 21:21 |
dfisher | so, you might see a PR … soon? | 21:22 |
dfisher | eventually? | 21:22 |
EmilienM | dfisher: go ahead | 21:22 |
dfisher | ? | 21:22 |
EmilienM | dfisher: with PR | 21:22 |
dfisher | for Solaris stuffs? | 21:23 |
dfisher | not ready yet :) | 21:23 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-vswitch: Correct `puppet module list` check in beaker https://review.openstack.org/199308 | 21:23 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-vswitch: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199309 | 21:23 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-vswitch: Use module install script for acceptance tests https://review.openstack.org/199310 | 21:23 |
dfisher | I really hope to get it upstream "soon". I have to do all the legal stuff and that can take awhile :/ | 21:23 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-keystone: Correct `puppet module list` check in beaker https://review.openstack.org/199311 | 21:24 |
fvollero | crinkle: tell me, i need to remove gerrit notifications email?? :) | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - keystone::endpoint https://review.openstack.org/193359 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - the user_role resource https://review.openstack.org/174999 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - resource::service_identity https://review.openstack.org/193357 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - keystone::resource::authconfig https://review.openstack.org/193356 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - keystone and keystone::roles::admin https://review.openstack.org/175060 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - examples https://review.openstack.org/197678 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - the user resource https://review.openstack.org/174976 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - the project resource https://review.openstack.org/174468 | 21:27 |
openstackgerrit | Richard Megginson proposed openstack/puppet-keystone: support for keystone v3 api - beaker tests https://review.openstack.org/178828 | 21:27 |
fvollero | here we go... | 21:27 |
EmilienM | boom | 21:28 |
crinkle | fvollero: you read gerrit notifications email? :P | 21:28 |
EmilienM | ahah | 21:29 |
fvollero | crinkle: sometimes but i've them with the company phone that keep ringing hahaah | 21:33 |
EmilienM | o_O | 21:33 |
crinkle | @.@ | 21:34 |
crinkle | i wouldn't survive | 21:34 |
fvollero | EmilienM: crinkle: problem solved, I just added a filter (we all love zimbra) to be in a separated folder (noisy puppet-openstack) :) | 21:35 |
crinkle | \o/ | 21:36 |
EmilienM | fvollero: sorry to work! | 21:36 |
dfisher | zimbra … that's an oracle product, right? | 21:36 |
fvollero | dfisher: no... opensource | 21:37 |
dfisher | k | 21:37 |
dfisher | because we (here at Oracle) use zimbra and omg I hate it. | 21:37 |
dfisher | hey, not that anybody in here will care too much but Solaris 11.3 beta was released today. you can go d/l it for free. | 21:38 |
mfisch | EmilienM: okay I got side tracked, working on A10 LBs now | 21:47 |
EmilienM | mfisch: what's that? | 21:48 |
mfisch | an expensive and hard to use hardware load balancer | 21:48 |
EmilienM | can you review again https://review.openstack.org/#/c/198572/ please ? | 21:49 |
EmilienM | mfisch: ^ | 21:49 |
mfisch | yes I need to drive back to FoCo first so tonight | 21:50 |
*** mgarza_ has quit IRC | 21:57 | |
*** ducttape_ has quit IRC | 21:58 | |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstacklib: Use module install script for acceptance tests https://review.openstack.org/199296 | 21:58 |
openstackgerrit | Colleen Murphy proposed openstack/puppet-openstacklib: Use beaker-puppet_install_helper to install puppet https://review.openstack.org/199295 | 21:58 |
*** mgarza has joined #puppet-openstack | 21:59 | |
mgagne | EmilienM: I see keystone_auth_* are not "properly" deprecated: https://github.com/openstack/puppet-cinder/blob/master/manifests/api.pp furthermore I see authtoken configs are configured in paste.ini, shouldn't it be in cinder.conf instead? | 22:00 |
EmilienM | yeah | 22:00 |
EmilienM | think | 22:00 |
EmilienM | I think | 22:00 |
mgagne | I'm updating my manifests and it bothers me :P | 22:01 |
richm | yes, it should be in $component.conf, not the paste ini | 22:09 |
*** morazi has quit IRC | 22:10 | |
*** xingchao has joined #puppet-openstack | 22:14 | |
*** prad has quit IRC | 22:16 | |
fvollero | EmilienM: rofl :) | 22:19 |
fvollero | dfisher: +1000 | 22:19 |
fvollero | xingchao: Ni haoo :) how are you doing ? it's been a while :) | 22:19 |
dfisher | crap - what did I do now? :) | 22:20 |
fvollero | dfisher: 23:37:44 < dfisher> because we (here at Oracle) use zimbra and omg I hate it. | 22:20 |
dfisher | ah. yes. | 22:20 |
dfisher | the hate. | 22:20 |
fvollero | dfisher: love and peace :) | 22:21 |
dfisher | tequila and lime | 22:21 |
*** xingchao has quit IRC | 22:24 | |
*** linkedinyou has quit IRC | 22:24 | |
fvollero | dfisher: yeap! | 22:26 |
*** linkedinyou has joined #puppet-openstack | 22:28 | |
_ody | dfisher: Cool. Wish we had a better NFS/ZFS HA story on our Solaris 11 filers. Would sure make upgrades easier. | 22:32 |
_ody | RIght now we upgrade once a year at best and only accomplish it by having VMware move all the data to a different filer for us...and will likely do the same for OpenStack this fall. | 22:34 |
dfisher | ugh. | 22:34 |
fvollero | dfisher: so there you're using the solaris zones to deploy openstack? :) | 22:55 |
dfisher | yep. | 22:55 |
dfisher | including the new Kernel Zones. | 22:55 |
dfisher | full VMs | 22:55 |
dfisher | not just OS containers | 22:56 |
fvollero | dfisher: coolio :) | 22:56 |
fvollero | dfisher: Sun developed an awesome product :) | 22:56 |
dfisher | Sun is still developing an awesome product. | 22:56 |
dfisher | it's just called Oracle now. | 22:56 |
dfisher | :( | 22:56 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: This is a merge of 2 commits to pass gate testing https://review.openstack.org/199336 | 22:57 |
openstackgerrit | Mike Dorman proposed openstack/puppet-neutron: WIP: Restart Neutron agents if package updated https://review.openstack.org/199337 | 22:58 |
mgagne | now that's a funny thing: nova::vncproxy::common will always complain (warnings) because it tries to read variables defined in different non-included/defined classes. :-/ | 22:59 |
dfisher | yeah … how the hell do you fix that :( | 22:59 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: This is a merge of 2 commits to pass gate testing https://review.openstack.org/199336 | 23:00 |
mgagne | I don't know tbh... | 23:00 |
mgagne | unless you have a version of pick which allows you to pass the name of the variable (a string) to check and it checks for you if the variable is defined and avoid throwing warnings | 23:01 |
*** aimon has quit IRC | 23:02 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: This is a merge of 2 commits to pass gate testing https://review.openstack.org/199336 | 23:03 |
mgagne | something like that: $vncproxy_port_real = pick($vncproxy_port, '$::nova::compute::vncproxy_port', '$::nova::vncproxy::port', 6080) | 23:04 |
dfisher | my head hurts. | 23:06 |
openstackgerrit | Mike Dorman proposed openstack/puppet-neutron: WIP: Restart Neutron agents if package updated https://review.openstack.org/199337 | 23:07 |
*** dfisher has quit IRC | 23:07 | |
openstackgerrit | Merged openstack/puppet-tempest: Implement more Tempest options https://review.openstack.org/198572 | 23:09 |
mfisch | mgagne: can I get a +2 on this comment only change? https://review.openstack.org/#/c/198508/ | 23:14 |
mgagne | mfisch: checking | 23:14 |
mfisch | thanks | 23:15 |
mfisch | trying to close some of these out | 23:15 |
mgagne | I think we only mark parameters as required if the actual parameter is required by the puppet class. no? | 23:16 |
mfisch | out of the box, it is required | 23:19 |
mfisch | it is implicitly required | 23:20 |
*** bklei has quit IRC | 23:20 | |
mfisch | i'd be okay with (Optional) but leaving my sentence | 23:20 |
*** bklei has joined #puppet-openstack | 23:20 | |
*** linkedinyou has quit IRC | 23:23 | |
*** bklei has quit IRC | 23:25 | |
*** gildub has joined #puppet-openstack | 23:26 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: define ceph::rgw::keystone. https://review.openstack.org/117422 | 23:28 |
mgagne | I'm ok with it | 23:28 |
mgagne | we can revisit later if I can no longer sleep at night | 23:28 |
mgagne | mfisch: ^ | 23:29 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: define ceph::rgw::keystone. https://review.openstack.org/117422 | 23:32 |
openstackgerrit | Merged openstack/puppet-cinder: Fix comments to note rabbit_password is required https://review.openstack.org/198508 | 23:36 |
*** aimon has joined #puppet-openstack | 23:46 | |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone to use correct URL/API string https://review.openstack.org/153783 | 23:47 |
openstackgerrit | David Gurtner proposed stackforge/puppet-ceph: Fix rgw keystone to use correct URL/API string https://review.openstack.org/153783 | 23:48 |
*** xingchao has joined #puppet-openstack | 23:49 | |
xingchao | fvollero: Hi, sorry for replying late. I was absent from irc at that time. | 23:51 |
*** jasondotstar has joined #puppet-openstack | 23:51 | |
*** mdorman has quit IRC | 23:57 | |
*** dalgaaf has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!