| *** ducttape_ has quit IRC | 00:00 | |
| *** Guest80615 is now known as saneax_AFK | 00:55 | |
| *** saneax_AFK is now known as saneax | 00:58 | |
| *** saneax is now known as Guest19379 | 00:58 | |
| *** ducttape_ has joined #puppet-openstack | 01:01 | |
| *** ducttape_ has quit IRC | 01:05 | |
| *** vinsh has joined #puppet-openstack | 01:22 | |
| *** Guest19379 is now known as saneax_AFK | 01:38 | |
| *** ilbot3 has quit IRC | 01:45 | |
| *** ducttape_ has joined #puppet-openstack | 01:46 | |
| *** ilbot3 has joined #puppet-openstack | 01:47 | |
| *** ducttape_ has quit IRC | 01:56 | |
| *** ducttape_ has joined #puppet-openstack | 02:32 | |
| *** ducttape_ has quit IRC | 03:02 | |
| *** ducttape_ has joined #puppet-openstack | 04:02 | |
| *** ducttape_ has quit IRC | 04:07 | |
| *** vinsh has quit IRC | 04:12 | |
| *** vinsh has joined #puppet-openstack | 04:13 | |
| *** vinsh has quit IRC | 04:13 | |
| *** vinsh has joined #puppet-openstack | 04:14 | |
| *** jaosorior has joined #puppet-openstack | 04:47 | |
| *** ducttape_ has joined #puppet-openstack | 05:04 | |
| *** ducttape_ has quit IRC | 05:08 | |
| *** dmsimard has quit IRC | 05:25 | |
| *** rcernin has joined #puppet-openstack | 05:25 | |
| *** dmsimard has joined #puppet-openstack | 05:26 | |
| *** ebalduf has quit IRC | 05:44 | |
| *** leanderthal|afk is now known as leanderthal | 05:56 | |
| *** ducttape_ has joined #puppet-openstack | 06:04 | |
| *** ducttape_ has quit IRC | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/355159 | 06:13 |
|---|---|---|
| *** rwsu has joined #puppet-openstack | 06:44 | |
| *** hjensas has joined #puppet-openstack | 06:58 | |
| *** hjensas has quit IRC | 06:58 | |
| *** hjensas has joined #puppet-openstack | 06:58 | |
| *** foxhunt_away is now known as foxhunt72 | 07:00 | |
| *** ducttape_ has joined #puppet-openstack | 07:05 | |
| *** ducttape_ has quit IRC | 07:09 | |
| *** ifarkas_afk is now known as ifarkas | 07:26 | |
| *** bkero-pto is now known as bkero | 07:56 | |
| *** paramite has joined #puppet-openstack | 08:03 | |
| *** ducttape_ has joined #puppet-openstack | 08:06 | |
| *** saneax_AFK is now known as saneax | 08:07 | |
| *** saneax is now known as Guest34773 | 08:07 | |
| *** ducttape_ has quit IRC | 08:11 | |
| *** dgurtner has joined #puppet-openstack | 08:15 | |
| *** dgurtner has quit IRC | 08:15 | |
| *** dgurtner has joined #puppet-openstack | 08:15 | |
| *** dtantsur|afk is now known as dtantsur | 08:18 | |
| *** Guest34773 is now known as saneax_AFK | 08:19 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options https://review.openstack.org/346685 | 08:20 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options https://review.openstack.org/346685 | 08:36 |
| *** markvoelker has joined #puppet-openstack | 08:41 | |
| *** markvoelker has quit IRC | 08:45 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint https://review.openstack.org/345183 | 08:47 |
| *** Jokke_ has joined #puppet-openstack | 09:00 | |
| Jokke_ | gDay | 09:00 |
| Jokke_ | anyone has idea why this https://review.openstack.org/#/c/353928/ fails with http://logs.openstack.org/28/353928/5/check/gate-puppet-manila-puppet-unit-3.6-centos-7/cbf7da9/console.html.gz#_2016-08-12_10_25_27_958013 ? | 09:01 |
| *** ducttape_ has joined #puppet-openstack | 09:07 | |
| *** ducttape_ has quit IRC | 09:11 | |
| *** jaosorior is now known as jaosorior_brb | 09:38 | |
| aderyugin | hi, Jokke_ as I can see you've created a define, but wrote a test for class, so you should rewrite your test for cephfsnative in a form like https://github.com/openstack/puppet-manila/blob/master/spec/defines/manila_backend_glusterfs_spec.rb and put it in folder defines instead of classes | 09:44 |
| Jokke_ | aderyugin: oh, thanks a million! ... trying to learn bit by bit. Had no idea. | 09:46 |
| openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 09:52 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 10:02 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Sorted variables alphabetically https://review.openstack.org/355400 | 10:02 |
| *** ducttape_ has joined #puppet-openstack | 10:08 | |
| *** ducttape_ has quit IRC | 10:12 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options https://review.openstack.org/346685 | 10:19 |
| openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 10:27 |
| *** jaosorior_brb is now known as jaosorior | 10:31 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class https://review.openstack.org/354125 | 10:32 |
| *** foxhunt72 has quit IRC | 10:32 | |
| *** markvoelker has joined #puppet-openstack | 10:42 | |
| *** markvoelker has quit IRC | 10:47 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-nova: Removing WARNING: line has more than 140 characters in puppet-nova profiles https://review.openstack.org/355409 | 10:48 |
| Jokke_ | degorenko: Thanks for your review ... this looks promising \\o \o/ o// o/7 | 10:53 |
| degorenko | Jokke_, for which patch? :D | 10:53 |
| Jokke_ | degorenko: the Manila CephFS | 10:54 |
| degorenko | ah | 10:54 |
| degorenko | :) | 10:54 |
| Jokke_ | thanks guys for bearing with me ... Puppet is really not the strong field for me ;) | 10:55 |
| degorenko | no problem | 10:58 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 11:00 |
| *** ducttape_ has joined #puppet-openstack | 11:08 | |
| *** ducttape_ has quit IRC | 11:13 | |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 11:25 |
| *** jaosorior has quit IRC | 11:34 | |
| *** jaosorior has joined #puppet-openstack | 11:35 | |
| EmilienM | good morning! | 11:42 |
| *** johnmilton has joined #puppet-openstack | 11:50 | |
| openstackgerrit | Merged openstack/puppet-cinder: Allow to manage Cinder Volume types from backends https://review.openstack.org/353775 | 11:51 |
| *** furlongm has joined #puppet-openstack | 11:51 | |
| *** furlongm_ has quit IRC | 11:51 | |
| *** dprince has joined #puppet-openstack | 11:52 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-ironic: Introduce ironic::ipxe class https://review.openstack.org/354125 | 11:52 |
| *** saneax_AFK is now known as saneax | 12:01 | |
| *** saneax is now known as Guest81752 | 12:01 | |
| *** ducttape_ has joined #puppet-openstack | 12:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: puppet4: fix ordering for compute agent https://review.openstack.org/354756 | 12:12 |
| *** ducttape_ has quit IRC | 12:12 | |
| *** jayg|g0n3 is now known as jayg | 12:13 | |
| *** ducttape_ has joined #puppet-openstack | 12:13 | |
| *** moravec has quit IRC | 12:13 | |
| *** moravec has joined #puppet-openstack | 12:14 | |
| *** moravec has quit IRC | 12:18 | |
| *** moravec has joined #puppet-openstack | 12:20 | |
| *** foxhunt72 has joined #puppet-openstack | 12:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: mitaka-only: force to clone poi on stable/mitaka https://review.openstack.org/355442 | 12:24 |
| *** Guest81752 is now known as saneax_AFK | 12:25 | |
| *** xarses has quit IRC | 12:28 | |
| *** markvoelker has joined #puppet-openstack | 12:30 | |
| iurygregory | morning! | 12:33 |
| *** ducttape_ has quit IRC | 12:35 | |
| iurygregory | hey EmilienM o/ | 12:46 |
| iurygregory | I've try to fix the nova_flavor type but with no luck https://review.openstack.org/#/c/337389/25/lib/puppet/type/nova_flavor.rb =( | 12:47 |
| openstackgerrit | Merged openstack/puppet-aodh: Move other-requirements.txt to bindep.txt https://review.openstack.org/354973 | 12:50 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/355159 | 12:50 |
| *** amoralej|off has quit IRC | 12:50 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Configure keystone authtoken options https://review.openstack.org/351272 | 12:51 |
| EmilienM | iurygregory: yeah, I noticed this morning too, I had no chance to find why yet | 12:52 |
| iurygregory | EmilienM, oh no problem o/ | 12:52 |
| *** amoralej has joined #puppet-openstack | 12:53 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Configure keystone authtoken options https://review.openstack.org/351272 | 12:55 |
| *** EmilienM sets mode: +o degorenko | 12:56 | |
| *** EmilienM sets mode: +o crinkle | 12:56 | |
| *** EmilienM sets mode: +o iberezovskiy|off | 12:56 | |
| *** EmilienM sets mode: +o dprince | 12:57 | |
| *** iberezovskiy|off is now known as iberezovskiy | 12:57 | |
| EmilienM | iberezovskiy, degorenko: can you please look https://review.openstack.org/#/c/355442/ and https://review.openstack.org/#/c/354837/ ? | 12:57 |
| EmilienM | I'm trying to bring puppet-ceph stable/hammer CI back | 12:58 |
| degorenko | ack +2 | 12:58 |
| EmilienM | thanks | 12:59 |
| EmilienM | also https://review.openstack.org/#/c/354762/ and https://review.openstack.org/#/c/354756/ please | 12:59 |
| EmilienM | i'm trying to bring all puppet4 jobs green | 12:59 |
| EmilienM | so we can enable voting on them :) | 13:00 |
| iurygregory | EmilienM, puppet-unit-latest-centos-7 is non-voting because of ceilometer only? | 13:07 |
| EmilienM | this job has never voted | 13:07 |
| EmilienM | http://docs.openstack.org/developer/puppet-openstack-guide/ci.html | 13:07 |
| iurygregory | lol i'm still as sleep hahah | 13:07 |
| iberezovskiy | reviewed | 13:07 |
| *** alee has quit IRC | 13:08 | |
| iberezovskiy | need to talk with packaging guys to fix 'user' issue | 13:08 |
| *** moravec has quit IRC | 13:08 | |
| *** moravec has joined #puppet-openstack | 13:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354835 | 13:10 |
| openstackgerrit | Merged openstack/puppet-ec2api: Remove S3 service https://review.openstack.org/354732 | 13:11 |
| EmilienM | iberezovskiy: which one? | 13:11 |
| iberezovskiy | EmilienM, in ceilometer | 13:11 |
| iberezovskiy | I'll talk to guys | 13:12 |
| *** ducttape_ has joined #puppet-openstack | 13:14 | |
| openstackgerrit | Merged openstack/puppet-openstack_spec_helper: mitaka-only: force to clone poi on stable/mitaka https://review.openstack.org/355442 | 13:16 |
| openstackgerrit | Michael Henkel proposed openstack/puppet-tripleo: Added Contrail Composable Services https://review.openstack.org/348905 | 13:16 |
| *** fultonj has joined #puppet-openstack | 13:22 | |
| iurygregory | EmilienM, https://github.com/openstack/puppet-cinder/blob/master/lib/puppet/provider/cinder.rb#L52 https://github.com/openstack/puppet-neutron/blob/436b01f5426470dc251b131e0fa957211b552471/lib/puppet/provider/neutron.rb#L32 Do you know why the difference in auth? | 13:24 |
| *** dmburmistrov has quit IRC | 13:26 | |
| EmilienM | iurygregory: no | 13:31 |
| iurygregory | skolekonov, ^ | 13:32 |
| iurygregory | do you know? | 13:32 |
| EmilienM | one needs auth_url other auth_uri | 13:32 |
| iurygregory | this is strange I think | 13:32 |
| *** ducttape_ has quit IRC | 13:33 | |
| EmilienM | degorenko, iberezovskiy: about https://review.openstack.org/#/c/354756/ - do we need for something? just to make sure | 13:40 |
| EmilienM | iurygregory: I'm looking nova today, trying to reproduce | 13:40 |
| iurygregory | EmilienM, ack | 13:40 |
| degorenko | EmilienM, +A, we +2'ed in one time | 13:41 |
| iurygregory | degorenko, i think iberezovskiy did not +A because need a fix in fuel no? | 13:41 |
| degorenko | iurygregory, for collector? | 13:42 |
| EmilienM | degorenko: thx sir | 13:42 |
| iurygregory | ' need to talk with packaging guys to fix 'user' issue' | 13:42 |
| degorenko | iurygregory, to fix it in proper way with dependency, no? | 13:43 |
| iurygregory | degorenko, oh yeah | 13:43 |
| iurygregory | sorry XD | 13:43 |
| degorenko | :) | 13:43 |
| iurygregory | degorenko, do you know why neutron use auth_url and other components use auth_uri? https://github.com/openstack/puppet-cinder/blob/master/lib/puppet/provider/cinder.rb#L52 https://github.com/openstack/puppet-neutron/blob/436b01f5426470dc251b131e0fa957211b552471/lib/puppet/provider/neutron.rb#L32 | 13:44 |
| degorenko | skolekonov, ^ | 13:44 |
| EmilienM | I think it just depends on the providers | 13:46 |
| EmilienM | like, cinder needs auth_uri and neutron auth_url | 13:46 |
| skolekonov | iurygregory, AFAIK auth_url is the replacement for identity_uri which was used before. auth_uri has a bit different meaning | 13:46 |
| skolekonov | but it seems they work both | 13:46 |
| skolekonov | not completely sure, however | 13:47 |
| iurygregory | skolekonov, yeah the replace for identity_uri is auth_url and the difference for auth_uri is the port (admin) =) | 13:48 |
| iurygregory | thanks god both work XD | 13:48 |
| skolekonov | :) | 13:48 |
| *** ayoung has joined #puppet-openstack | 13:55 | |
| *** dprince has quit IRC | 13:55 | |
| openstackgerrit | Merged openstack/puppet-gnocchi: Keystone authoken resource change https://review.openstack.org/347417 | 14:00 |
| openstackgerrit | Merged openstack/puppet-heat: Add log directory to .gitignore https://review.openstack.org/354581 | 14:00 |
| openstackgerrit | Merged openstack/puppet-ironic: Add log directory to .gitignore https://review.openstack.org/354583 | 14:00 |
| openstackgerrit | Merged openstack/puppet-mistral: Add log directory to .gitignore https://review.openstack.org/354794 | 14:00 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add ironic inspector as a terminated HAProxy endpoint https://review.openstack.org/345183 | 14:00 |
| skolekonov | iurygregory, btw, about project_domain_id and user_domain_id. They are not covered by new authtoken class, so I use service_default for them, just for compatibility | 14:04 |
| *** ducttape_ has joined #puppet-openstack | 14:04 | |
| iurygregory | skolekonov, yeah i notice, i comment about that we do not need both? | 14:05 |
| skolekonov | right, we don't them both set to smth, but it's possible that someone used only *_id options (without *_name) | 14:05 |
| skolekonov | *don't need | 14:05 |
| iurygregory | yeah =) | 14:06 |
| iurygregory | skolekonov, ack when you update i'll look =) | 14:06 |
| skolekonov | thanks for comments :) | 14:06 |
| iurygregory | np ^^ | 14:06 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Fix identation in docs for authtoken https://review.openstack.org/355480 | 14:07 |
| *** alee has joined #puppet-openstack | 14:08 | |
| *** ebalduf has joined #puppet-openstack | 14:09 | |
| *** devnull0 has joined #puppet-openstack | 14:14 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "run_tests: stop mistral testing" https://review.openstack.org/339813 | 14:15 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-manila: Add authtoken class https://review.openstack.org/351792 | 14:18 |
| *** jtomasek is now known as jtomasek|biab | 14:23 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: functions: clone poi mitaka when cloning ceph/hammer https://review.openstack.org/354837 | 14:26 |
| *** dprince has joined #puppet-openstack | 14:27 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Fix authtoken and metadata https://review.openstack.org/355480 | 14:27 |
| EmilienM | degorenko, iberezovskiy : https://review.openstack.org/#/c/354835/ | 14:28 |
| EmilienM | thanks | 14:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Update .gitreview to point at stable/hammer https://review.openstack.org/350511 | 14:29 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceph: Refresh keys if caps are changed https://review.openstack.org/350512 | 14:29 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Fix metadata to require keystone https://review.openstack.org/355494 | 14:30 |
| *** johnmilton has quit IRC | 14:37 | |
| EmilienM | iurygregory: ok I reproduced the nova/authtoken issue, now I'm trying something | 14:38 |
| iurygregory | EmilienM, you rock man \o/ thanks | 14:39 |
| EmilienM | well, I havent found it yet | 14:40 |
| iurygregory | but you have reproduced :P | 14:40 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Fix metadata for authtoken https://review.openstack.org/355505 | 14:42 |
| *** ducttape_ has quit IRC | 14:44 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: puppet4: ensure rabbitmq is ready before starting nova https://review.openstack.org/354762 | 14:44 |
| iurygregory | mjblack, you around? | 14:44 |
| *** xarses has joined #puppet-openstack | 14:53 | |
| *** dtantsur is now known as dtantsur|mtg | 14:57 | |
| *** xarses has quit IRC | 14:59 | |
| mjblack | iurygregory: am now | 15:03 |
| mjblack | whats up? | 15:03 |
| iurygregory | mjblack, hey o/ can I take the work in heat for authtoken? | 15:04 |
| mjblack | iurygregory: sure | 15:05 |
| iurygregory | mjblack, ack o/ | 15:05 |
| *** rcernin has quit IRC | 15:06 | |
| *** johnmilton has joined #puppet-openstack | 15:09 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: puppet4: fix ordering for compute agent https://review.openstack.org/354756 | 15:09 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge https://review.openstack.org/355519 | 15:11 |
| *** dprince has quit IRC | 15:12 | |
| *** johnmilton has quit IRC | 15:13 | |
| EmilienM | iurygregory: in self.get_auth_endpoint I changed auth_uri to be auth_url and it worked | 15:14 |
| EmilienM | let me double check | 15:14 |
| *** johnmilton has joined #puppet-openstack | 15:14 | |
| EmilienM | mhh it might be something else | 15:16 |
| *** jtomasek|biab is now known as jtomasek | 15:20 | |
| EmilienM | I think the latest release of puppet is breaking unit tests for puppet-nova | 15:23 |
| EmilienM | for nova::conductor and nova::scheduler | 15:24 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken https://review.openstack.org/337389 | 15:26 |
| EmilienM | iurygregory: let's try something^ | 15:26 |
| EmilienM | https://review.openstack.org/#/c/337389/25..26 | 15:26 |
| EmilienM | for unit tests failures, http://logs.openstack.org/89/337389/25/check/gate-puppet-nova-puppet-unit-latest-centos-7/b7844cd/console.html#_2016-08-14_14_32_49_099147 | 15:27 |
| *** foxhunt72 is now known as foxhunt_away | 15:34 | |
| EmilienM | I reported https://bugs.launchpad.net/puppet-neutron/+bug/1613329 | 15:36 |
| openstack | Launchpad bug 1613329 in puppet-nova "puppet 4.6.0 unit tests are broken" [Undecided,New] | 15:36 |
| EmilienM | I'll work on it today but feel free to help | 15:36 |
| *** haplo37__ has joined #puppet-openstack | 15:41 | |
| *** leanderthal is now known as leanderthal|afk | 15:41 | |
| *** devnull0 has quit IRC | 15:53 | |
| hogepodge | EmilienM: is there a publicly available repository of daily Newton RDO package builds that I can run the modules off of? | 15:57 |
| *** xarses has joined #puppet-openstack | 15:57 | |
| iurygregory | EmilienM, ok :D | 15:59 |
| *** ifarkas is now known as ifarkas_afk | 16:01 | |
| *** dtantsur|mtg is now known as dtantsur | 16:02 | |
| iurygregory | hogepodge, something like https://trunk.rdoproject.org/centos7-master/45/36/45369b608aea665e38ad203deb6936c1004e082d_6c60bd37/ ? | 16:06 |
| hogepodge | iurygregory: are there docs on how to activate it? I know that in the past I sometimes need to augment epel with newer packages | 16:07 |
| iurygregory | hogepodge, in integration we use repos.pp https://review.openstack.org/#/c/355159/3/manifests/repos.pp | 16:08 |
| *** cwolferh has joined #puppet-openstack | 16:09 | |
| iurygregory | i don't think there is doc about how activate | 16:09 |
| hogepodge | iurygregory: that should all be enough. I want to start working from master so when I have to do the newton release I'm not scrambling to figure out what broke for my modules in the previous cycle | 16:09 |
| *** jtomasek is now known as jtomasek|afk | 16:10 | |
| iurygregory | hogepodge, ok if you need help just ask here =) | 16:10 |
| *** dfisher has joined #puppet-openstack | 16:11 | |
| *** ducttape_ has joined #puppet-openstack | 16:18 | |
| *** dhollinger has joined #puppet-openstack | 16:22 | |
| *** dprince has joined #puppet-openstack | 16:24 | |
| *** xarses has quit IRC | 16:32 | |
| *** iberezovskiy is now known as iberezovskiy|off | 16:39 | |
| beagles | wow, CI seems cranky .. at least on mitaka | 16:42 |
| iurygregory | beagles wich patch? | 16:42 |
| beagles | iurygregory, https://review.openstack.org/#/c/355235/ | 16:43 |
| iurygregory | beagles, recheck is the only thing you can do | 16:45 |
| beagles | iurygregory, *shrug* ack | 16:45 |
| iurygregory | beagles, i have put recheck o/ | 16:45 |
| beagles | iurygregory, thanks!!! | 16:45 |
| iurygregory | beagles, np, i'm wondering if ubuntu nodes in infra are ok ... | 16:46 |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Disable barbican https://review.openstack.org/355221 | 16:56 |
| *** jaosorior has quit IRC | 17:06 | |
| *** dprince has quit IRC | 17:09 | |
| beagles | iurygregory, good question.. I wonder if I should pint openstack-infra | 17:16 |
| beagles | or ping rather.. they probably wouldn't mind pint either | 17:17 |
| *** ayoung has quit IRC | 17:25 | |
| *** dtantsur is now known as dtantsur|afk | 17:26 | |
| *** degorenko is now known as _degorenko|afk | 17:30 | |
| *** iurygregory has quit IRC | 17:36 | |
| *** dprince has joined #puppet-openstack | 17:36 | |
| *** rcernin has joined #puppet-openstack | 17:37 | |
| *** ayoung has joined #puppet-openstack | 17:41 | |
| EmilienM | beagles: did you investigate in logs why mitaka is failing on trusty? | 17:52 |
| EmilienM | looks like a problem with zuul-cloner and git servers | 17:52 |
| EmilienM | http://logs.openstack.org/35/355235/1/gate/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/aaf8252/console.html#_2016-08-15_13_04_26_268615 | 17:52 |
| *** iurygregory has joined #puppet-openstack | 17:53 | |
| beagles | EmilienM, that's about as far as I got, I haven't asked infra yet | 17:54 |
| EmilienM | ah :) | 17:54 |
| EmilienM | ask them | 17:54 |
| beagles | yep | 17:54 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken https://review.openstack.org/337389 | 18:13 |
| EmilienM | iurygregory: I think I found why | 18:21 |
| EmilienM | the nova thing | 18:21 |
| EmilienM | it sounds like an orchestration thing in poi | 18:21 |
| *** grimlock86 has joined #puppet-openstack | 18:29 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: provision: require admin user before creating nova flavor https://review.openstack.org/355592 | 18:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Move nova to authtoken https://review.openstack.org/337389 | 18:31 |
| *** ayoung has quit IRC | 18:37 | |
| iurygregory | EmilienM, oh the problem is in the poi? :O | 18:37 |
| iurygregory | sorry i was in a meeting | 18:37 |
| EmilienM | _ody: you around? need your help on some puppet4 things | 18:37 |
| *** rcernin has quit IRC | 18:42 | |
| *** haplo37__ has quit IRC | 18:50 | |
| dmsimard | EmilienM: hm, https://github.com/openstack/puppet-gnocchi/commit/7aa0c6052b76bb3be94d6a52ccba8f7d163fbee4 broke Packstack. | 18:52 |
| dmsimard | Probably packstack's fault though, just saying | 18:52 |
| EmilienM | weird, poi didnt break | 18:52 |
| dmsimard | Duplicate declaration: Gnocchi_config[keystone_authtoken/auth_version] is already declared in file /var/tmp/packstack/258ede9892104ffd980efac9019b6897/modules/packstack/manifests/gnocchi.pp:34; cannot redeclare at /var/tmp/packstack/258ede9892104ffd980efac9019b6897/modules/keystone/manifests/resource/authtoken.pp:326 on node n45.ci.centos.org | 18:52 |
| mwhahaha | yup that'll happen | 18:53 |
| iurygregory | lol | 18:53 |
| mwhahaha | need to remove the auth_version from packstack and use the new params available | 18:53 |
| mwhahaha | should get packstack visible in CI ;) | 18:53 |
| dmsimard | eh it's alright | 18:53 |
| dmsimard | nothing hard to fix | 18:53 |
| EmilienM | well, we have enough jobs I think | 18:53 |
| EmilienM | how many jobs have packstack, 3? | 18:54 |
| EmilienM | dmsimard: you think we could add them into check? | 18:54 |
| EmilienM | maybe as non voting | 18:54 |
| mwhahaha | i don't think we have any external coverage on gnocchi | 18:54 |
| dmsimard | I don't think it's worth it | 18:54 |
| dmsimard | In many ways p-o-i has better and more modern coverage than Packstack | 18:55 |
| mwhahaha | k | 18:55 |
| iurygregory | beagles, the CI does not like you XD | 18:55 |
| beagles | iurygregory, no it does not :) apparently there was an issue with zuul (or something). Infra says they have a patch pending | 18:56 |
| iurygregory | beagles, oh awesome :D now only 2 failling i put recheck again | 18:56 |
| beagles | iurygregory, :) thanks. It's like the "monte carlo" method of getting CI passing | 18:57 |
| *** johnmilton has quit IRC | 18:57 | |
| mwhahaha | looks like puppet-pacemaker is broken on xenial | 18:58 |
| EmilienM | so much things are broken | 18:59 |
| EmilienM | I'm working on puppet4 breakages this week, unit & functional | 18:59 |
| colonwq | mwhahaha, s/puppet-pacemaker/xenial/ :) | 18:59 |
| EmilienM | and I'm helping Iury with the nova/authtoken thing | 18:59 |
| * mwhahaha fast forwards to Friday | 19:00 | |
| EmilienM | lol if only | 19:01 |
| EmilienM | which reminds me: i'm on pto friday and monday, will be far in the middle of nowhere | 19:02 |
| colonwq | Have a good time away from everything. | 19:02 |
| mwhahaha | polarbear hunting again? | 19:03 |
| EmilienM | I won't tell anyone, it's a secret place in the world, where nobody goes :P | 19:03 |
| dfisher | Sun Microsystems! | 19:03 |
| * dfisher single tear | 19:03 | |
| EmilienM | lol ahahahah | 19:03 |
| *** haplo37__ has joined #puppet-openstack | 19:04 | |
| EmilienM | mwhahaha: https://goo.gl/maps/fX4bUGSm3q82 | 19:04 |
| mwhahaha | Good ol' maine | 19:05 |
| dfisher | "We're still a state!" - official Maine state motto | 19:05 |
| mwhahaha | north woods law, mildly amusing reality tv | 19:06 |
| colonwq | All I know about Maine I learned from reading Steven King. | 19:07 |
| EmilienM | are you scaring me? I'll lock the door. | 19:08 |
| dfisher | won't matter. | 19:09 |
| dfisher | U DED | 19:09 |
| colonwq | just bury the body in the old Indian cemetery up on the hill. | 19:10 |
| *** haplo37__ has quit IRC | 19:10 | |
| EmilienM | I knew I should have said no and stay in Canada | 19:10 |
| EmilienM | mwhahaha: do you remember Sofer's bug in puppet-keystone with ldap thing? | 19:12 |
| EmilienM | I think we're hitting that everywhere in our modules, with unit tests | 19:12 |
| mwhahaha | the bool2num thing | 19:12 |
| mwhahaha | ? | 19:12 |
| EmilienM | it seems like when you pick() parameters from a different class you need to include the class | 19:12 |
| EmilienM | see his patch https://review.openstack.org/#/c/354872/ | 19:12 |
| EmilienM | I have to do the same to fix puppet 4.6 unit tests or puppet-nova | 19:13 |
| mwhahaha | that's annoying | 19:14 |
| EmilienM | yes | 19:14 |
| mwhahaha | so is that a breaking change in the 4.5 to 4.6 path? | 19:14 |
| EmilienM | yeah | 19:14 |
| EmilienM | that's why any help from _ody on this thing would be great | 19:14 |
| EmilienM | I started to investigate the diff between 4.5 and 4.6 and I gave up | 19:14 |
| mwhahaha | does it just flat out error or are we getting weird data in places? | 19:15 |
| _ody | Sorry in a meeting all day. I'll try to hunt so,some down to help. | 19:16 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Fix nvoa::db and puppet 4.6 https://review.openstack.org/355609 | 19:16 |
| EmilienM | mwhahaha: ^ | 19:16 |
| EmilienM | _ody: please help | 19:16 |
| EmilienM | when you can | 19:16 |
| EmilienM | mwhahaha: no, unit tests fail | 19:17 |
| EmilienM | the pick() function doesn't work | 19:17 |
| mwhahaha | seems like maybe the fix is in the pick function then | 19:19 |
| *** johnmilton has joined #puppet-openstack | 19:21 | |
| mwhahaha | so it's the acceptance test right? where we're getting a "" from pick | 19:21 |
| mwhahaha | which should get cleaned up by https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/lib/puppet/parser/functions/pick.rb#L22 | 19:21 |
| *** haplo37__ has joined #puppet-openstack | 19:22 | |
| mwhahaha | wonder if an undeclared variable in 4.6 is ending up not being an empty string but some other non printable char or something | 19:22 |
| EmilienM | mwhahaha: no unit tests | 19:23 |
| mwhahaha | but i thought his bug was showing up in the xenial acceptance tests not the unit | 19:23 |
| EmilienM | mwhahaha: see http://logs.openstack.org/89/337389/25/check/gate-puppet-nova-puppet-unit-latest-centos-7/b7844cd/console.html#_2016-08-14_14_32_49_099016 | 19:23 |
| mwhahaha | http://logs.openstack.org/87/354087/3/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial/e89d782/console.html#_2016-08-11_23_56_40_060367 | 19:23 |
| EmilienM | mwhahaha: it's another problem | 19:23 |
| EmilienM | but maybe related | 19:24 |
| mwhahaha | oh right the other one where we determined it was probably parse order related | 19:24 |
| mwhahaha | where the p-o-i includes weren't necessarily being evaluated prior to the running of a resource | 19:25 |
| mwhahaha | maybe we need some contains or some thing | 19:25 |
| *** johnmilton has quit IRC | 19:26 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults 'undef' to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 19:29 |
| *** jonnytpuppet has joined #puppet-openstack | 19:32 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Fix nvoa::db and puppet 4.6 https://review.openstack.org/355609 | 19:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-nova: Fix nova::db and puppet 4.6 https://review.openstack.org/355609 | 19:34 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-neutron: Fix neutron::db and puppet 4.6 https://review.openstack.org/355617 | 19:35 |
| iurygregory | spam time? | 19:35 |
| EmilienM | so a lot of modules are affected | 19:37 |
| EmilienM | I'm going to wait for feedback on nova and neutron and I'll patch the rest later | 19:37 |
| EmilienM | I'm investigating a weird timeout issue on puppet4 and gnocchi http://logs.openstack.org/19/355519/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7-nv/e5807ba/console.html#_2016-08-15_15_37_05_598922 | 19:38 |
| mwhahaha | so the puppet pacemaker issue lead me to this bit of code, https://github.com/puppetlabs/puppet/blob/master/lib/puppet/property.rb#L374-L398 | 19:40 |
| mwhahaha | anytime i see 'Here be dragons' (╯°□°)╯︵ ┻━┻ | 19:41 |
| iurygregory | LOL | 19:41 |
| iurygregory | 'you need to be carreful?' | 19:42 |
| iurygregory | careful* | 19:42 |
| *** jonnytpuppet has quit IRC | 19:44 | |
| *** ayoung has joined #puppet-openstack | 19:48 | |
| EmilienM | mwhahaha: in 4.6? | 19:52 |
| EmilienM | I'm wondering what kind of QA puppetlabs is running really | 19:52 |
| mwhahaha | I'm sure its always been there but yea | 19:53 |
| mwhahaha | It's erroring on data comparisons, haven't gotten farther yey | 19:53 |
| EmilienM | I'm more talking about the bug Sofer and I have | 19:53 |
| EmilienM | it looks like a non backward compatible thing | 19:54 |
| EmilienM | my stuff sounds related to pick indeed | 19:54 |
| mwhahaha | oh yea in 4.6 | 19:59 |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script https://review.openstack.org/355629 | 20:08 |
| *** jayg is now known as jayg|g0n3 | 20:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs.sh: add support for Puppet4 modulepath https://review.openstack.org/355630 | 20:09 |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script https://review.openstack.org/355629 | 20:12 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ironic: conductor: add api_url parameter https://review.openstack.org/355632 | 20:13 |
| mwhahaha | http://logs.openstack.org/02/355202/1/check/gate-puppet-pacemaker-puppet-beaker-rspec-ubuntu-xenial/1879471/console.html#_2016-08-13_20_14_22_379463 and the other errors seems to point to some datatyping issue which i wonder if it's also the reason for the pick() issues | 20:14 |
| EmilienM | mwhahaha: ugly error | 20:18 |
| mwhahaha | pretty much | 20:18 |
| *** jonnytpuppet has joined #puppet-openstack | 20:20 | |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Fine tuning of stack script https://review.openstack.org/355629 | 20:21 |
| *** jonnytpuppet has quit IRC | 20:27 | |
| *** jonnytpuppet has joined #puppet-openstack | 20:30 | |
| Hunner | EmilienM: Hmm. Before https://review.openstack.org/#/c/355609/ how did ::nova get included? neither nova::deps nor nova::params declares the base class as far as I see | 20:37 |
| mwhahaha | it was included in the p-o-i manifest | 20:37 |
| mwhahaha | or not as the case may be | 20:37 |
| mwhahaha | the point is to try and maintain some type of backwards compatibility if that variable was set use it instead of the one to the class | 20:38 |
| EmilienM | Hunner: yeah, something broke the unit tests | 20:39 |
| EmilienM | and when I add ::nova in nova::db, it works again | 20:39 |
| Hunner | :/ | 20:39 |
| Hunner | EmilienM: but something IS declaring ::nova elsewhere? | 20:39 |
| Hunner | and you have to declare it there also? | 20:40 |
| EmilienM | in unit tests, yes | 20:40 |
| Hunner | I wonder if :pre_conditions broke or something... | 20:40 |
| EmilienM | Hunner: here https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_conductor_spec.rb#L61 | 20:40 |
| EmilienM | yeah | 20:40 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 20:57 |
| *** dprince has quit IRC | 20:59 | |
| iurygregory | see ya o/ i'll be back latter | 21:01 |
| *** haplo37__ has quit IRC | 21:03 | |
| openstackgerrit | Merged openstack/puppet-manila: Add authtoken class https://review.openstack.org/351792 | 21:05 |
| *** haplo37__ has joined #puppet-openstack | 21:18 | |
| *** grimlock86 has quit IRC | 21:19 | |
| *** grimlock86 has joined #puppet-openstack | 21:25 | |
| mwhahaha | stupid data types | 21:35 |
| * mwhahaha cries | 21:35 | |
| dmsimard | Are we super positive that this is_service_default method works well ? https://github.com/openstack/puppet-gnocchi/blob/7aa0c6052b76bb3be94d6a52ccba8f7d163fbee4/manifests/keystone/authtoken.pp#L226 | 21:41 |
| dmsimard | I'm passing a password to gnocchi::keystone::authtoken and it doesn't look like it's picking it up | 21:41 |
| dmsimard | i.e, in Packstack: https://review.openstack.org/#/c/355601/2/packstack/puppet/modules/packstack/manifests/gnocchi.pp | 21:42 |
| dmsimard | Pasted some error snippets in my last comment in https://review.openstack.org/#/c/355601/ .. | 21:42 |
| mwhahaha | dmsimard: works fine provided you have openstacklib | 22:00 |
| *** xarses has joined #puppet-openstack | 22:01 | |
| mwhahaha | dmsimard: do you have a debug log for the hiera loading? | 22:02 |
| *** saneax_AFK is now known as saneax | 22:05 | |
| *** saneax is now known as Guest16777 | 22:05 | |
| mwhahaha | dmsimard: you need to include the keystone::authtoken prior to the api | 22:05 |
| mwhahaha | i will comment on the review | 22:05 |
| *** beagles is now known as beagles_brb | 22:11 | |
| *** haplo37__ has quit IRC | 22:15 | |
| *** jistr has quit IRC | 22:18 | |
| *** jistr has joined #puppet-openstack | 22:19 | |
| *** dhollinger has quit IRC | 22:21 | |
| xarses | is there an etherpad for tomorrow? https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-20160816 I'd guess | 22:28 |
| bkero | sounds like you're making it | 22:29 |
| xarses | seems so | 22:39 |
| *** ducttape_ has quit IRC | 22:40 | |
| *** beagles_brb is now known as beagles | 22:42 | |
| *** grimlock86 has quit IRC | 23:01 | |
| *** markvoelker has quit IRC | 23:06 | |
| *** dfisher has quit IRC | 23:13 | |
| *** xarses has quit IRC | 23:22 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Defaults empty hash to kernel_modules and sysctl_settings https://review.openstack.org/355615 | 23:24 |
| *** xarses has joined #puppet-openstack | 23:34 | |
| *** dgurtner has quit IRC | 23:42 | |
| dmsimard | mwhahaha: ohhhhhh, thanks | 23:52 |
| dmsimard | <3 | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!