Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:03 cwolferh joined #puppet-openstack
00:03 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to latest trunk  https://review.openstack.org/328007
00:04 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/326142
00:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
00:10 iurygregory_ EmilienM, i think i have an account '-'
00:10 xarses joined #puppet-openstack
00:11 EmilienM iurygregory_: yeah I follow you now
00:11 iurygregory_ but i don't use because i can't run with my smartphone =X
00:11 EmilienM so i'll check on you that you do running :P
00:11 EmilienM buy a garmin!
00:11 iurygregory_ a what? O.o
00:11 EmilienM https://explore.garmin.com/en-US/forerunner/
00:11 iurygregory_ lol
00:12 iurygregory_ this is expensive i think O.o
00:13 iurygregory_ omg $220
00:13 EmilienM iurygregory_: you can have one for < 100 USD
00:16 iurygregory_ EmilienM, the reason to not use my smartphone is just because my city does not have security =X (many thieves)
00:16 iurygregory_ =P
00:16 iurygregory_ but $100 is a good price i think. maybe i can find in Barcelona :D
00:17 EmilienM iurygregory_: stay safe!
00:20 igorbelikov joined #puppet-openstack
00:20 aglarendil joined #puppet-openstack
00:20 iurygregory_ EmilienM, oh yeah! and this month is crazy because we have a traditional festival XD (30 days of party lol)
00:21 petems joined #puppet-openstack
00:22 patrickeast joined #puppet-openstack
00:22 kgalanov joined #puppet-openstack
00:31 jgrassler joined #puppet-openstack
00:48 iurygregory_ Thanks good your patch for octavia has passed EmilienM \o/
00:48 iurygregory_ god*
00:49 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: beaker: remove multi domain config resources output test  https://review.openstack.org/327894
00:49 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file  https://review.openstack.org/320554
00:50 richm EmilienM: can you confirm that you started seeing the failures after https://github.com/openstack/puppet​-openstack-integration/commit/427f6​dda0a45a0bc4f60a627921597ef1dc4c89f ?
00:50 richm that is, after updating to the latest version of centos7 rdo?
00:51 EmilienM richm: again, we update every day
00:51 EmilienM richm: but yes
00:51 EmilienM something in OpenStack broke us the last 2 days
00:51 EmilienM I'm investigating what we need to disable to make it work again
00:52 richm EmilienM: I think it has to do with this: https://github.com/openstack/keystone/commi​t/d6b016dd91c743a2f454a3b4f9d055510c2215ae
00:52 EmilienM all fails I think
00:53 EmilienM richm: but our integration tests work fine :/
00:53 richm EmilienM: in the last good run, I don't see the message Reset password for user admin.
00:53 richm EmilienM: in the bad run, the first auth operation after that message fails
00:53 EmilienM in keystone logs?
00:53 richm yes
00:53 EmilienM mhh
00:53 EmilienM damn
00:54 richm trying to find in the puppet-keystone code where keystone-manage bootstrap is run and, if so, what is the password argument
00:54 EmilienM richm: https://github.com/openstack/puppet-keyston​e/blob/master/manifests/init.pp#L1077-L1084
00:55 richm I guess you need to use bootstrap to create the default domain?  But now it nukes the admin password.
00:55 EmilienM richm: this is how we deploy keystone in our CI https://github.com/openstack/puppet-openstack-​integration/blob/master/manifests/keystone.pp
00:56 richm you don't specify the bootstrap argument, and the default is `true`
00:56 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
00:56 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
00:56 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
00:56 richm oh - you do pass the admin token as the password argument
00:56 EmilienM yes
00:57 richm assuming admin_token == password for the user 'admin'
00:57 richm but if you're using token auth everywhere, it should not matter what the password is
00:58 EmilienM richm: FYI https://github.com/openstack/puppet-openstack-in​tegration/blob/master/manifests/keystone.pp#L79
00:58 richm when you create the admin user, do you set the password to the admin_token?
00:59 EmilienM we run: keystone-manage bootstrap --bootstrap-password ${admin_token}
00:59 EmilienM and yes they are same passwords
01:00 EmilienM well wait
01:00 EmilienM admin_token = 'admin_token'
01:00 richm it may be a red herring, but it is the only clue I have so far
01:00 EmilienM and admin = 'a_big_secret'
01:00 EmilienM see https://github.com/openstack/puppet-openstack-​integration/blob/master/manifests/keystone.pp
01:03 zigo joined #puppet-openstack
01:03 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
01:04 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328015
01:04 EmilienM mwhahaha: backports ^ auto approved, so we can have logs in our stable CI
01:04 richm EmilienM: so after the bootstrap is run, the puppet-keystone openstack provider tries to do password auth with user 'admin' and password 'a_big_secret', but the password was changed to 'admin_token'?
01:05 richm so auth fails?
01:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs: grab syslog with journalctl on Ubuntu  https://review.openstack.org/328016
01:06 iurygregory_ lol we change the password is change? O.o
01:06 iurygregory_ we change the password for the admin user?*
01:07 richm iurygregory_: yes
01:07 iurygregory_ enr '-'
01:07 ayoung joined #puppet-openstack
01:07 richm I think this line in keystone/init.pp is the problem:
01:07 richm command     => "keystone-manage bootstrap --bootstrap-password ${admin_token}",
01:07 richm should use ${admin_password}
01:07 richm but we don't have the admin_password in this context :-(
01:10 iurygregory_ if we pass admin_token this is token to be used .-. if we change admin password but we try to use only the admin token this should work i think
01:11 richm the --bootstrap-password argument is used here: https://github.com/openstack/keystone​/blob/master/keystone/cmd/cli.py#L87
01:11 richm then here: https://github.com/openstack/keystone​/blob/master/keystone/cmd/cli.py#L143
01:11 richm then here: https://github.com/openstack/keystone​/blob/master/keystone/cmd/cli.py#L223
01:14 EmilienM richm: so admin token & admin password should be the same in our CI?
01:14 EmilienM one day I'll understand how keystone works I swear!
01:15 richm EmilienM: that would be the most immediate fix, yes
01:15 richm EmilienM: don't worry - no one else will understand it either :P
01:15 iurygregory_ ahsuahsaushaush
01:15 EmilienM ok
01:15 * richm blames ayoung . . .
01:15 EmilienM let's try it
01:15 EmilienM we always blame ayoung
01:15 EmilienM everybody blames ayoung
01:15 ayoung Yep
01:15 * bkero blames ayoung
01:15 iurygregory_ i don't D:
01:15 ayoung It's all my fault
01:16 EmilienM richm: can you patch puppet-keystone please? I dont even know what to write in commit message
01:16 EmilienM we should rewrite keystone in GO
01:16 iurygregory_ LOL
01:16 ayoung so..I missed the early part of this
01:16 EmilienM and this problem would be solved
01:16 * richm runs
01:16 EmilienM richm: no no stay here
01:16 nibalizer EmilienM: yep letsget started
01:16 * EmilienM locks the door
01:16 richm I'm running from go
01:16 nibalizer we can put raft in it and everything
01:16 ayoung admin_token is, I think, a thing of the past once we talk bootstrap.  What you wrote above looks sane
01:17 richm ayoung: the problem is this:       command     => "keystone-manage bootstrap --bootstrap-password ${admin_token}",
01:17 richm that should be $admin_password
01:17 richm the token and the password are different
01:18 EmilienM richm: it's a critical backward incompatible change
01:18 EmilienM IMHO
01:18 ayoung richm, EmilienM we should remove references to admin_token now that we are using bootstrap.
01:18 ayoung EmilienM, here is the short version
01:18 * EmilienM unlock the door and run
01:18 iurygregory_ lol
01:18 ayoung admin_token was a hacky, not very secure way to allow a user to do initiaol configuration of the system.  It was hard to disable.
01:18 EmilienM yeah, right
01:19 richm EmilienM: I think I need to change openstack_integration?
01:19 ayoung notmorgan (Fainberg) took exception to this and fixed it.  THe admin_password should be different from admin_token in a long lived system
01:19 EmilienM richm: yes sorry
01:19 ayoung if we were still using admin token, it could easiler be something like my old favorite FreeIPA4All, because, by the time the system is up and running, it should be disabled
01:20 ayoung where as admin_password should be `uuid-gen -r` and will be used by the admin user to authenticate after the system is in production
01:20 EmilienM ok now in french please
01:20 EmilienM well admin_password & admin_token are different in our CI
01:21 EmilienM but beaker tests do not work anymore
01:21 EmilienM maybe because we run bootstrap or something?
01:21 ayoung admin_token était un pas de manière très sécurisée hacky pour permettre à un utilisateur de faire la configuration initiaol du système. Il était difficile de désactiver. notmorgan (Fainberg) a pris exception et réparée. THe mot_de_passe_admin doit être différent de admin_token dans un système de longue durée. si nous utilisions encore administrateur jeton, il pourrait easiler être quelque chose comme mon vi
01:21 ayoung eux FreeIPA4All favori, parce que, au moment où le système est en marche, il doit être désactivé. où comme mot_de_passe_admin devrait être `uuid-gen -r` et sera utilisée par l'utilisateur admin pour authentifier après que le système est en production.
01:21 EmilienM ah, thanks, it's better now
01:21 ayoung Mon aéroglisseur est plein d'anguilles.
01:22 * iurygregory_ is not understanding anything
01:22 Topic for #puppet-openstack is now CI status: orange (centos7 beaker jobs failing on puppet-keystone) - Puppet OpenStack modules - http://docs.openstack.org/deve​loper/puppet-openstack-guide/
01:22 EmilienM ayoung: lol
01:23 ayoung what are the failing tests?
01:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/328005
01:24 EmilienM ayoung: http://logs.openstack.org/44/327844/1/check/gat​e-puppet-keystone-puppet-beaker-rspec-centos-7/​e7a05a5/console.html#_2016-06-09_18_36_55_349
01:25 iurygregory_ best way to get -1 in CI LOL
01:25 ayoung nothing resembling a stack trace?
01:26 EmilienM ayoung: maybe richm has more thoughts on the error, I havent spent time, I'm really multi tasking on multiple CI issues
01:28 iurygregory_ http://logs.openstack.org/44/327844/1/check/gate-​puppet-keystone-puppet-beaker-rspec-centos-7/e7a0​5a5/logs/apache/keystone_wsgi_admin_access.txt.gz
01:28 iurygregory_ python-openstackclient keystoneauth1/2.8.1 python-requests/2.10.0 CPython/2.7.5 rich told something about this if i'm right
01:28 openstackgerrit Richard Megginson proposed openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token  https://review.openstack.org/328026
01:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked  https://review.openstack.org/328028
01:30 richm This was the first error I found - looked like many other errors were caused by this one - I didn't look any further
01:30 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge  https://review.openstack.org/327844
01:30 EmilienM richm: let's see if beaker jobs work, I used depends-on ^
01:31 EmilienM richm: see my comment on your patch though
01:33 openstackgerrit Richard Megginson proposed openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token  https://review.openstack.org/328026
01:33 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge  https://review.openstack.org/327844
01:34 ducttape_ joined #puppet-openstack
01:39 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
01:39 openstackgerrit Merged openstack/puppet-openstack-integration: refacto: move run_puppet & catch_selinux_alerts into functions  https://review.openstack.org/327197
01:48 ilbot3 joined #puppet-openstack
01:48 Topic for #puppet-openstack is now CI status: orange (centos7 beaker jobs failing on puppet-keystone) - Puppet OpenStack modules - http://docs.openstack.org/deve​loper/puppet-openstack-guide/
01:50 richm EmilienM: Can we see the ci testing for https://review.openstack.org/#/c/327844/ in progress?  I'd like to see if it can at least get past the point of the first failure
01:51 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328015
01:52 EmilienM richm: zuul.openstack.org
01:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
01:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked  https://review.openstack.org/328028
01:56 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to latest trunk  https://review.openstack.org/328007
01:56 EmilienM richm: they are still running, watch 327844
01:56 EmilienM 2 min left
01:57 richm uh - telnet?
01:57 EmilienM zuul ui
01:57 richm ah, cool
01:58 EmilienM richm: status.openstack.org/zuul/
02:00 richm looks like it got past the point of the failure
02:04 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT  https://review.openstack.org/325982
02:04 EmilienM richm: success!
02:06 EmilienM richm: so yeah it works, that's cool. I just want to highlight we found a critical thing in our CI that might be a non  backward compatible change in keystone cc ayoung
02:08 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules - http://docs.openstack.org/deve​loper/puppet-openstack-guide/
02:10 richm it's not easy being green . . .
02:21 EmilienM lol
02:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked  https://review.openstack.org/328028
02:25 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
02:27 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/326027
02:28 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
02:30 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/326027
02:33 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: beaker: remove multi domain config resources output test  https://review.openstack.org/327894
02:33 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file  https://review.openstack.org/320554
02:45 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
02:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
02:58 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
03:03 openstackgerrit Merged openstack/puppet-openstack-integration: the admin_password is now the same as the admin_token  https://review.openstack.org/328026
03:12 Shaik joined #puppet-openstack
03:48 jaosorior joined #puppet-openstack
04:06 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint  https://review.openstack.org/324356
04:21 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint  https://review.openstack.org/324764
04:32 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
04:51 cwolferh joined #puppet-openstack
05:03 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint  https://review.openstack.org/324764
05:07 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint  https://review.openstack.org/324356
05:10 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
05:14 darkxploit joined #puppet-openstack
05:15 markvoelker joined #puppet-openstack
05:24 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
06:06 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/327029
06:06 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/327069
06:06 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for glance API and registry  https://review.openstack.org/327473
06:06 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for RabbitMQ  https://review.openstack.org/327482
06:06 yolanda joined #puppet-openstack
06:09 darkxploit joined #puppet-openstack
06:12 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328090
06:24 danpawlik joined #puppet-openstack
06:33 openstackgerrit joined #puppet-openstack
06:37 cwolferh joined #puppet-openstack
06:43 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
06:51 jmikolajczak joined #puppet-openstack
06:55 dgurtner joined #puppet-openstack
06:59 xarses joined #puppet-openstack
07:11 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Provide default service_name for keystone endpoint  https://review.openstack.org/326031
07:13 openstackgerrit Merged openstack/puppet-tripleo: Enable bulk delete in swift for gnocchi  https://review.openstack.org/327214
07:14 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files.  https://review.openstack.org/326593
07:15 permalac joined #puppet-openstack
07:16 markvoelker joined #puppet-openstack
07:17 rcernin joined #puppet-openstack
07:23 xarses_ joined #puppet-openstack
07:24 xarses_ joined #puppet-openstack
07:27 tesseract joined #puppet-openstack
07:29 aimon_ joined #puppet-openstack
07:33 openstackgerrit joined #puppet-openstack
07:38 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-trove: Add missing keystone service and endpoint tests  https://review.openstack.org/323960
07:49 darkxploit joined #puppet-openstack
07:56 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Add user & user_role parameters in auth manifest and related tests  https://review.openstack.org/324296
07:56 karimb joined #puppet-openstack
08:00 cwolferh joined #puppet-openstack
08:03 jtomasek_ joined #puppet-openstack
08:03 shardy joined #puppet-openstack
08:17 markvoelker joined #puppet-openstack
08:24 queretaro joined #puppet-openstack
08:26 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files.  https://review.openstack.org/326593
08:30 jaosorior joined #puppet-openstack
08:31 hjensas joined #puppet-openstack
08:37 ifarkas joined #puppet-openstack
08:45 hjensas joined #puppet-openstack
08:45 hjensas joined #puppet-openstack
08:55 darkxploit joined #puppet-openstack
09:16 jmikolajczak joined #puppet-openstack
09:21 dgurtner joined #puppet-openstack
09:21 dgurtner joined #puppet-openstack
09:25 dgurtner joined #puppet-openstack
09:32 evidex joined #puppet-openstack
09:35 todin joined #puppet-openstack
10:15 openstackgerrit Merged openstack/puppet-openstack-integration: copy_logs: grab syslog with journalctl on Ubuntu  https://review.openstack.org/328016
10:17 queretaro joined #puppet-openstack
10:18 markvoelker joined #puppet-openstack
10:24 openstackgerrit Oleksiy Molchanov proposed openstack/puppet-ceph: Fix activate check  https://review.openstack.org/310507
10:36 cwolferh joined #puppet-openstack
10:36 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
10:37 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
10:46 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-heat: Provide default service_name for keystone endpoint  https://review.openstack.org/324764
11:04 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-mistral: Provide default service_name for keystone endpoint  https://review.openstack.org/324356
11:10 dmk0202 joined #puppet-openstack
11:10 ansmith joined #puppet-openstack
11:13 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
11:16 johnmilton joined #puppet-openstack
11:21 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-designate: Provide default service_name for keystone endpoint  https://review.openstack.org/326031
11:36 dmk0202 joined #puppet-openstack
11:41 openstackgerrit Venkata Mahesh Jonnalagadda proposed openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
11:43 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
11:43 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
11:43 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
11:44 dprince joined #puppet-openstack
11:55 morazi joined #puppet-openstack
11:56 danpawlik joined #puppet-openstack
12:03 dgurtner joined #puppet-openstack
12:03 dgurtner joined #puppet-openstack
12:05 ducttape_ joined #puppet-openstack
12:08 karimb joined #puppet-openstack
12:18 markvoelker joined #puppet-openstack
12:28 EmilienM good morning
12:35 openstackgerrit Merged openstack/puppet-tripleo: Make neutron server/ml2 profiles match t-h-t  https://review.openstack.org/326119
12:39 dmk0202 joined #puppet-openstack
12:45 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
12:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: workaround to run beaker with puppet4  https://review.openstack.org/327725
12:52 EmilienM degorenko: hey
12:52 qba73 joined #puppet-openstack
12:52 EmilienM degorenko: can I have a second eye on https://review.openstack.org/#/c/327678 ? I'm struggling to see why second puppet run removes the file instead of first run
12:53 EmilienM maybe ordering
12:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
12:55 iurygregory morning =D
12:58 iberezovskiy hi
12:59 dmk0202 joined #puppet-openstack
12:59 chem joined #puppet-openstack
13:06 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
13:10 richm joined #puppet-openstack
13:11 derekh joined #puppet-openstack
13:12 cwolferh joined #puppet-openstack
13:15 degorenko EmilienM, hey
13:15 EmilienM degorenko: yoyoyo
13:15 degorenko i was afk
13:15 EmilienM no worries
13:15 degorenko looking at 327678
13:15 EmilienM if you have 5 min to look this patch
13:15 EmilienM ok
13:15 EmilienM degorenko: doing reviews now, anything I should look in prio?
13:15 degorenko EmilienM, no, i dont have prio reviews :)
13:16 ayoung joined #puppet-openstack
13:18 openstackgerrit Merged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/328090
13:19 EmilienM iberezovskiy: can you look https://review.openstack.org/#/c/328028/ ?
13:20 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/326142
13:27 iberezovskiy EmilienM, so you are actually check that mirror was changed?
13:27 iberezovskiy if mirror change wasn't applied it's not ok
13:27 EmilienM iberezovskiy: I check puppet installed the repo
13:27 EmilienM before we had set +2
13:27 EmilienM so puppet could have failed, we missed it
13:29 zhongshengping joined #puppet-openstack
13:29 iberezovskiy yep, just wanted to clarify. merged
13:33 EmilienM thx
13:33 EmilienM clayton: no -1 but still some question on https://review.openstack.org/327823 - please look when you have time
13:34 clayton read the comment above in deps.pp :)
13:34 clayton support-packages is used for things that don't require restarting services
13:34 clayton so the client would be a support package
13:34 EmilienM clayton: right
13:34 EmilienM good
13:34 clayton for neutron, keepalived and  dnsmasq is a support package
13:35 EmilienM clayton: I'm done for this review, +2
13:35 clayton awesome :)
13:35 clayton the same pattern is used for neutron and nova and keystone
13:35 clayton it wasn't needed for any modules before that
13:35 EmilienM cool, excellent work
13:35 clayton thanks :)
13:36 EmilienM iberezovskiy: damn, I broke something
13:36 arnewiebalck joined #puppet-openstack
13:37 iberezovskiy where?
13:37 EmilienM nothing critical
13:37 EmilienM I'm fixing it
13:39 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: copy_logs: use journalctl only if present  https://review.openstack.org/328303
13:39 EmilienM iberezovskiy: ^
13:39 iberezovskiy don't we want to have journalctl always available?
13:39 EmilienM iberezovskiy: not on trusty ;)
13:39 EmilienM it's a sysvinit system
13:39 iberezovskiy oh yeah :)
13:42 zhongshengping joined #puppet-openstack
13:44 zhongshengping joined #puppet-openstack
13:45 openstackgerrit Dmitry Tantsur proposed openstack/puppet-tripleo: Add ironic profiles  https://review.openstack.org/319297
13:47 zhongshengping joined #puppet-openstack
13:51 dfisher joined #puppet-openstack
13:57 EmilienM degorenko: can you look https://review.openstack.org/#/c/328303/ please?
13:57 EmilienM iberezovskiy: also, if you have time, I'm trying to address the ubuntu changes in https://review.openstack.org/#/c/327770/ but CI still fails, any idea why second puppet run is not idempotent?
13:58 iberezovskiy i"ll take a look in ~10mins
13:59 zhongshengping joined #puppet-openstack
14:05 EmilienM iberezovskiy, degorenko: also need review on https://review.openstack.org/#/c/328014/ to get logs back on mitaka jobs
14:09 openstackgerrit Merged openstack/puppet-openstack-integration: run_tests: fail if repos are not installed when asked  https://review.openstack.org/328028
14:11 ducttape_ joined #puppet-openstack
14:11 iurygregory only me in the mid-cycle etherpad lol
14:12 cwolferh joined #puppet-openstack
14:17 EmilienM iurygregory: you can do the midcycle yourself? :)
14:22 ducttape_ joined #puppet-openstack
14:26 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
14:26 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
14:26 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
14:27 EmilienM iberezovskiy: is Dmitry here? ^
14:27 iberezovskiy nope, I'll ping him
14:27 EmilienM mhh ok
14:31 mkarpin joined #puppet-openstack
14:32 ifarkas joined #puppet-openstack
14:32 openstackgerrit Merged openstack/puppet-tripleo: Import ::nova class from THT  https://review.openstack.org/325982
14:33 iurygregory EmilienM, yes i can =P
14:33 xarses_ joined #puppet-openstack
14:33 iberezovskiy EmilienM, about https://review.openstack.org/#/c/327770/: I see there problem related to rabbimq-server: ERROR: epmd error for host ubuntu-xenial-rax-iad-1576843: timeout (timed out)
14:33 openstackgerrit Dan Radez proposed openstack/puppet-congress: Updating code to new puppet module stucture  https://review.openstack.org/328322
14:34 zhongshengping joined #puppet-openstack
14:38 EmilienM iberezovskiy: I'll recheck it once https://review.openstack.org/328303 merge
14:38 iberezovskiy http://logs.openstack.org/70/327770/2/check/gate-​puppet-openstack-integration-3-scenario002-tempes​t-ubuntu-xenial/499b8da/logs/rabbitmq/startup_log
14:39 iurygregory EmilienM, since congress already have code , i think it's good use cookiecutter to generate a new structure for the module and then add the previous files. what you think?
14:39 EmilienM iurygregory: yes
14:39 iurygregory ok =D
14:39 iberezovskiy probably smth wrong with hostname, let's wait for your patch
14:40 zhongshengping joined #puppet-openstack
14:42 EmilienM iberezovskiy: have you thought about mu issue on xenial/newton thing with keystone?
14:43 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/304780
14:43 iberezovskiy EmilienM, about the idempotency?
14:43 EmilienM yes
14:43 iberezovskiy I've statrted tests locally
14:51 zhongshengping joined #puppet-openstack
14:57 dhollinger joined #puppet-openstack
14:58 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
15:01 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
15:01 EmilienM iberezovskiy: ^ see change, degorenko suggested something cool
15:04 iberezovskiy subscribe, cool
15:17 openstackgerrit Merged openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328014
15:22 zhongshengping joined #puppet-openstack
15:29 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/328347
15:29 degorenko EmilienM, typo here? https://review.openstack.org/327725
15:30 EmilienM degorenko: you're nitpicking me? :P
15:30 EmilienM it's Frriday!
15:31 degorenko EmilienM, nope :D
15:31 degorenko but can - can't
15:31 degorenko :)
15:37 openstackgerrit Merged openstack/puppet-openstack-integration: copy_logs: use journalctl only if present  https://review.openstack.org/328303
15:37 iurygregory degorenko, just give a +1and not +2 =P
15:39 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/326142
15:43 shaikapsar joined #puppet-openstack
15:45 shaikapsar I'm having an issue with https://github.com/openstack/puppet-magnum​/blob/master/manifests/keystone/domain.pp
15:46 shaikapsar I can;t use domain.pp with out openrc file
15:51 degorenko shaikapsar, probably you faced with this bug: https://github.com/openstack/puppet-magnum/b​lob/master/manifests/keystone/domain.pp#L69
15:53 iberezovskiy EmilienM, still not idempotent
15:53 iberezovskiy and I don't see the update for package resource
15:54 shaikapsar No, domain.pp is trying to create keystone_user, keystone_domain and keystone_user_role
15:54 EmilienM iberezovskiy: weird
15:54 shaikapsar but those types need openrc file access the keystone service
15:55 shaikapsar if i place openrc file then those resource getting created
15:56 EmilienM degorenko, iberezovskiy: can you guys took over this one? I'm already trying to migrate beaker jobs on xenial
15:56 shaikapsar if i run domain pp in Fuel controller node I see an other issue , /etc/magnum/manum.conf doesn;t exist
15:56 iberezovskiy hm, but locally it work
15:56 shaikapsar If i run magnum on controller node it will work
15:58 shaikapsar Can we remove https://github.com/openstack/puppet-magnum/blo​b/master/manifests/keystone/domain.pp#L72-L84 from domain pp and magnum_config  with domain details in an other manifest file?
15:59 iberezovskiy let me retry
15:59 degorenko shaikapsar, fuel doesn't support magnum
16:00 degorenko shaikapsar, you should ask vanditboy
16:00 degorenko but he is not here now
16:06 shaikapsar joined #puppet-openstack
16:07 shaikapsar I'm working on fuel plugin for magnum and i got the issue
16:12 degorenko then possibly you need manually put all needed configs in config file
16:12 degorenko but as i know fuel plugin starts installing of service when keystone already finished
16:12 degorenko just use openrc
16:16 shaikapsar I already implemented openrc config, but i want to check with you that https://github.com/openstack/puppet-magnum/blo​b/master/manifests/keystone/domain.pp#L72-L84 line are valid in domain.pp
16:19 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/304780
16:32 ducttape_ joined #puppet-openstack
16:34 iberezovskiy EmilienM, can I try to rework this subscribe a bit?
16:34 EmilienM iberezovskiy: please take over it
16:34 EmilienM I'm already busy on other tasks for today
16:35 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
16:35 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
16:35 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
16:36 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
16:38 iberezovskiy @bbs 30min, I think test will show smth
16:42 chem joined #puppet-openstack
16:48 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-tripleo: WIP: integration of the new puppet pacemaker.  https://review.openstack.org/309069
16:52 colonwq joined #puppet-openstack
16:56 openstackgerrit Merged openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/326021
17:11 aimon joined #puppet-openstack
17:14 ayoung joined #puppet-openstack
17:15 iberezovskiy EmilienM, I see only one solution here, remove target file. using define won't be idempotent
17:15 EmilienM iberezovskiy: how?
17:16 iberezovskiy I wann file { ensure => absent } with the hardcoded path
17:16 iberezovskiy I wanna try*
17:16 iberezovskiy but it's horrible
17:17 EmilienM iberezovskiy: I did it before
17:17 EmilienM I didn't work fine
17:17 iberezovskiy EmilienM, did you try with subscribe?
17:18 EmilienM iberezovskiy: no
17:18 iberezovskiy let's do it :)
17:19 iberezovskiy I'll upload
17:20 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
17:22 EmilienM iberezovskiy: I have some patches
17:22 EmilienM https://review.openstack.org/#/c/327894/
17:22 EmilienM and https://review.openstack.org/#/c/320554/
17:23 EmilienM I'll need to add the nodesets on all modules
17:23 EmilienM and then we can switch our beaker jobs to xenial, no more blockers
17:24 iberezovskiy I can add nodesets files but only later (after 2-3 hours)
17:24 * iberezovskiy reviewing patches
17:25 EmilienM iberezovskiy: I'm on it already
17:26 iberezovskiy EmilienM, nodesets for keystone could be merged, right?
17:27 iberezovskiy I'm wondering because gate-puppet-openstack-integration-4​-scenario001-tempest-ubuntu-trusty is red
17:29 openstackgerrit Dan Radez proposed openstack/puppet-congress: Updating code to new puppet module stucture  https://review.openstack.org/328322
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Add nodepool-xenial.yml file  https://review.openstack.org/328401
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-barbican: Add nodepool-xenial.yml file  https://review.openstack.org/328402
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Add nodepool-xenial.yml file  https://review.openstack.org/328403
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Add nodepool-xenial.yml file  https://review.openstack.org/328404
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-designate: Add nodepool-xenial.yml file  https://review.openstack.org/328405
17:30 openstackgerrit Martin Mágr proposed openstack/puppet-tripleo: New composable service: Sensu  https://review.openstack.org/323303
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-ec2api: Add nodepool-xenial.yml file  https://review.openstack.org/328406
17:30 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Add nodepool-xenial.yml file  https://review.openstack.org/328407
17:31 EmilienM iberezovskiy: it's a random error: http://logs.openstack.org/54/320554/17/exp​erimental/gate-puppet-openstack-integratio​n-4-scenario001-tempest-ubuntu-trusty/9eb1​19e/console.html#_2016-06-10_16_23_49_902
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Add nodepool-xenial.yml file  https://review.openstack.org/328408
17:31 EmilienM I think we can safely merge them
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Add nodepool-xenial.yml file  https://review.openstack.org/328409
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-horizon: Add nodepool-xenial.yml file  https://review.openstack.org/328410
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Add nodepool-xenial.yml file  https://review.openstack.org/328411
17:31 * iurygregory runs spam time
17:31 openstackgerrit Emilien Macchi proposed openstack/puppet-magnum: Add nodepool-xenial.yml file  https://review.openstack.org/328412
17:32 openstackgerrit Emilien Macchi proposed openstack/puppet-manila: Add nodepool-xenial.yml file  https://review.openstack.org/328413
17:32 openstackgerrit Emilien Macchi proposed openstack/puppet-mistral: Add nodepool-xenial.yml file  https://review.openstack.org/328414
17:32 openstackgerrit Emilien Macchi proposed openstack/puppet-murano: Add nodepool-xenial.yml file  https://review.openstack.org/328415
17:32 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Add nodepool-xenial.yml file  https://review.openstack.org/328416
17:32 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Add nodepool-xenial.yml file  https://review.openstack.org/328417
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Add nodepool-xenial.yml file  https://review.openstack.org/328418
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: Add nodepool-xenial.yml file  https://review.openstack.org/328419
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-openstacklib: Add nodepool-xenial.yml file  https://review.openstack.org/328420
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-oslo: Add nodepool-xenial.yml file  https://review.openstack.org/328421
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-ovn: Add nodepool-xenial.yml file  https://review.openstack.org/328422
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-sahara: Add nodepool-xenial.yml file  https://review.openstack.org/328423
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: Add nodepool-xenial.yml file  https://review.openstack.org/328424
17:33 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Add nodepool-xenial.yml file  https://review.openstack.org/328425
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Add nodepool-xenial.yml file  https://review.openstack.org/328426
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-vswitch: Add nodepool-xenial.yml file  https://review.openstack.org/328427
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-watcher: Add nodepool-xenial.yml file  https://review.openstack.org/328428
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-zaqar: Add nodepool-xenial.yml file  https://review.openstack.org/328429
17:35 dprince joined #puppet-openstack
17:36 iberezovskiy awesome :D
17:36 cwolferh joined #puppet-openstack
17:37 iberezovskiy probably tacker is left only
17:38 iberezovskiy EmilienM, idempotency doesn't like us today, still the same
17:38 EmilienM damn
17:39 EmilienM iberezovskiy: ack for tacker
17:39 EmilienM well, tacker needs spec/acceptance thing
17:40 iurygregory same for congress i think
17:41 iberezovskiy and what about puppet-ceph?
17:41 iurygregory iberezovskiy, https://github.com/openstack/puppet-cep​h/tree/master/spec/acceptance/nodesets
17:41 iurygregory already have
17:41 iberezovskiy great
17:42 iurygregory module sync too =)
17:42 iberezovskiy then monasca and rally
17:42 iurygregory lol monasca =P
17:43 iurygregory rally makes sense =)
17:43 iberezovskiy yeah yeah, my bad
17:43 iberezovskiy just walked through github
17:43 iurygregory congress/tacker will need to use cookiecutter so we don't need =D
17:44 iurygregory because they should use modulesync and cookiecutter :D
17:47 iberezovskiy EmilienM, you know, I see interesting thing. This file removal doesn't apply at first puppet run, only for second
17:48 iurygregory lol
17:50 EmilienM iberezovskiy: yeah, that's what I figured too
17:51 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
17:53 shaikapsar joined #puppet-openstack
18:00 iberezovskiy bbs ~1 hour
18:09 openstackgerrit Merged openstack/puppet-openstack-integration: workaround to run beaker with puppet4  https://review.openstack.org/327725
18:16 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Pacemaker: only create DB on pacemaker master  https://review.openstack.org/328446
18:19 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/328347
18:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: remove mariadb prior to run Puppet  https://review.openstack.org/328449
18:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Import copy_logs.sh script from openstack-infra/project-config  https://review.openstack.org/328015
18:48 dgurtner joined #puppet-openstack
19:01 dprince EmilienM: did you want me to update this? https://review.openstack.org/#/c/319464/2
19:01 EmilienM dprince: yes please so we're consistent with other modules
19:02 EmilienM oh wait
19:02 EmilienM rebase it
19:02 EmilienM it was adressed it
19:02 openstackgerrit Emilien Macchi proposed openstack/puppet-glance: Switch $service_name to 'glance' for keystone  https://review.openstack.org/319464
19:02 EmilienM dprince: good now ^
19:02 EmilienM dprince: +A
19:03 dprince EmilienM: cool with me :)
19:03 yolanda joined #puppet-openstack
19:16 shaikapsar joined #puppet-openstack
19:16 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Schema change for deterministic alarms defs  https://review.openstack.org/328460
19:17 shaikapsar Please review https://review.openstack.org/#/c/321154/
19:25 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: bump RDO to latest OpenStack commit  https://review.openstack.org/328463
19:29 openstackgerrit Merged openstack/puppet-watcher: Add nodepool-xenial.yml file  https://review.openstack.org/328428
19:30 openstackgerrit Merged openstack/puppet-barbican: Add nodepool-xenial.yml file  https://review.openstack.org/328402
19:30 openstackgerrit Merged openstack/puppet-designate: Add nodepool-xenial.yml file  https://review.openstack.org/328405
19:31 openstackgerrit Merged openstack/puppet-ovn: Add nodepool-xenial.yml file  https://review.openstack.org/328422
19:32 openstackgerrit Merged openstack/puppet-murano: Add nodepool-xenial.yml file  https://review.openstack.org/328415
19:33 openstackgerrit Merged openstack/puppet-magnum: Add nodepool-xenial.yml file  https://review.openstack.org/328412
19:34 openstackgerrit Merged openstack/puppet-octavia: Add nodepool-xenial.yml file  https://review.openstack.org/328418
19:34 openstackgerrit Merged openstack/puppet-manila: Add nodepool-xenial.yml file  https://review.openstack.org/328413
19:50 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
19:59 openstackgerrit Merged openstack/puppet-ironic: Change default service_name to 'ironic'  https://review.openstack.org/327117
19:59 openstackgerrit Merged openstack/puppet-ceilometer: Add nodepool-xenial.yml file  https://review.openstack.org/328403
19:59 openstackgerrit Merged openstack/puppet-swift: Add nodepool-xenial.yml file  https://review.openstack.org/328424
20:09 openstackgerrit Merged openstack/puppet-zaqar: Add nodepool-xenial.yml file  https://review.openstack.org/328429
20:17 openstackgerrit Merged openstack/puppet-cinder: Add nodepool-xenial.yml file  https://review.openstack.org/328404
20:29 johnmilton joined #puppet-openstack
20:30 openstackgerrit Merged openstack/puppet-gnocchi: Add nodepool-xenial.yml file  https://review.openstack.org/328408
20:30 openstackgerrit Merged openstack/puppet-horizon: Add nodepool-xenial.yml file  https://review.openstack.org/328410
20:31 dprince joined #puppet-openstack
20:33 openstackgerrit Merged openstack/puppet-oslo: Add nodepool-xenial.yml file  https://review.openstack.org/328421
20:34 openstackgerrit Merged openstack/puppet-aodh: Add nodepool-xenial.yml file  https://review.openstack.org/328401
20:38 openstackgerrit Merged openstack/puppet-openstacklib: Add nodepool-xenial.yml file  https://review.openstack.org/328420
20:38 openstackgerrit Merged openstack/puppet-keystone: beaker: remove multi domain config resources output test  https://review.openstack.org/327894
20:40 openstackgerrit Shaik Apsar proposed openstack/puppet-trove: Option to specify the trove guestagent log file.  https://review.openstack.org/321154
20:41 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: repos: deploy newton ppa on ubuntu xenial  https://review.openstack.org/327678
20:41 iurygregory bye people o/ have a great weekend
20:42 openstackgerrit Merged openstack/puppet-murano: Provide default service_name for keystone endpoint  https://review.openstack.org/324749
20:43 EmilienM iurygregory: o/
20:49 johnmilton joined #puppet-openstack
20:50 openstackgerrit Merged openstack/puppet-sahara: Add nodepool-xenial.yml file  https://review.openstack.org/328423
20:50 openstackgerrit Merged openstack/puppet-trove: Add nodepool-xenial.yml file  https://review.openstack.org/328426
20:51 EmilienM beagles: in octavia, do keystone options are required more than api service? ie: worker, health manager, etc
20:52 ducttape_ joined #puppet-openstack
20:53 openstackgerrit Merged openstack/puppet-tempest: Add nodepool-xenial.yml file  https://review.openstack.org/328425
20:54 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
20:56 openstackgerrit Merged openstack/puppet-ironic: Add nodepool-xenial.yml file  https://review.openstack.org/328411
20:56 EmilienM mwhahaha: omg I'm glad we have nova::flavor provider
20:56 EmilienM mwhahaha: octavia requires to create a flavor with a predictable id
20:56 EmilienM so we can reuse the provider in puppet octavia to manage the flavor
20:57 mwhahaha well that's good
20:57 EmilienM I want to buy a beer to denis now
20:57 EmilienM was it denis?
20:57 mwhahaha no
20:57 EmilienM ah no
20:57 EmilienM Trevor McCasland
20:57 EmilienM oops sorry
20:58 EmilienM I'll buy 2 beers!
21:03 EmilienM mwhahaha: wdyt
21:03 EmilienM should we manage the flavor by default or not?
21:03 mwhahaha for octavia?
21:03 mwhahaha yea probably
21:04 mwhahaha with a $configure_flavor option to disable
21:04 EmilienM yeah
21:08 openstackgerrit Merged openstack/puppet-openstack_extras: Add nodepool-xenial.yml file  https://review.openstack.org/328419
21:08 openstackgerrit Merged openstack/puppet-glance: Add nodepool-xenial.yml file  https://review.openstack.org/328407
21:15 mwhahaha EmilienM: looks like mistral packaging is broken, http://logs.openstack.org/14/328414/1/check/ga​te-puppet-mistral-puppet-beaker-rspec-centos-7​/ab5ac6d/console.html#_2016-06-10_20_44_19_280
21:16 mwhahaha not sure why mistral is requiring magnumclient
21:20 EmilienM mwhahaha: packaging is fixed
21:20 EmilienM recheck
21:21 EmilienM we fixed it in rdo today
21:21 EmilienM and I just promoted our CI
21:21 EmilienM mwhahaha: still in gate: https://review.openstack.org/#/c/328463/
21:21 mwhahaha k
21:22 EmilienM mwhahaha: in puppet, can I use an hash to deploy a provider?
21:23 EmilienM like $hash = { 'id' => '1' } and nova_flavor { id => $hash[id] } ?
21:23 mwhahaha wat
21:23 mwhahaha yea
21:23 EmilienM maybe in puppet parser or puppet4 only :/
21:23 EmilienM how?
21:23 mwhahaha with create resource
21:23 mwhahaha it takes a hash i think
21:24 mwhahaha so you could just create_resource('nova_flavor', $hash)
21:24 EmilienM ah right
21:24 EmilienM nice!!
21:24 mwhahaha brb
21:27 dgurtner joined #puppet-openstack
21:29 openstackgerrit Merged openstack/puppet-heat: Provide default service_name for keystone endpoint  https://review.openstack.org/324764
21:40 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
21:40 EmilienM mwhahaha: I need to write some missing tests, but you can already tell me what you think ^
21:40 mwhahaha sure
21:41 EmilienM well amp_flavor_id to os_service_default sucks
21:41 EmilienM because it will make catalog fails by default...
21:41 EmilienM maybe I can pick a random value, like 65
21:43 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
21:44 EmilienM mwhahaha: need you to review the hash thing, I'll finish the rest later
21:45 mwhahaha yea that'll work, minor note on the name string
21:46 EmilienM ok thx
21:46 EmilienM I don't see the note
21:46 mwhahaha just posted
21:46 EmilienM ah thanks
21:49 EmilienM mwhahaha: does my reply makes sense?
21:49 dhollinger joined #puppet-openstack
21:49 EmilienM except it's Nova_flavor
21:50 EmilienM not Nova::Flavor
21:50 mwhahaha yea but you get the idea :D
21:50 EmilienM yeah
21:50 EmilienM cool!
21:50 mwhahaha but you may want to tag to restrict it
21:50 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
21:50 mwhahaha but yea
21:51 Hunner joined #puppet-openstack
21:51 Hunner joined #puppet-openstack
21:51 openstackgerrit Merged openstack/puppet-keystone: ubuntu: do not attempt to start apache during keystone setup  https://review.openstack.org/327770
21:51 openstackgerrit Merged openstack/puppet-openstack-integration: bump RDO to latest OpenStack commit  https://review.openstack.org/328463
21:52 mentat joined #puppet-openstack
22:02 openstackgerrit Merged openstack/puppet-keystone: Fix facts for policy test  https://review.openstack.org/326642
22:05 openstackgerrit Merged openstack/puppet-vswitch: Add nodepool-xenial.yml file  https://review.openstack.org/328427
22:06 EmilienM mwhahaha: wait, can I test a provider in classes rspec?
22:06 EmilienM not sure is_expected.to contain_nova__flavor works
22:06 mwhahaha yea
22:06 mwhahaha it should
22:06 EmilienM mhh ok
22:06 mwhahaha contain_nova_flavor
22:06 mwhahaha contain_nova__flavor would be for a resource nova::flavor
22:06 mwhahaha but contain_nova_flavor is for the resource nova_flavor :D
22:06 EmilienM ah right
22:07 EmilienM damn
22:07 EmilienM I'm tired
22:07 EmilienM what day is it again? :)
22:07 mwhahaha friday i hope :D
22:09 EmilienM mwhahaha: thx for your last review, really awesome
22:09 mwhahaha sure
22:12 openstackgerrit Merged openstack/puppet-keystone: Add nodepool-xenial.yml file  https://review.openstack.org/320554
22:47 openstackgerrit Merged openstack/puppet-heat: Add nodepool-xenial.yml file  https://review.openstack.org/328409
22:47 openstackgerrit Merged openstack/puppet-ceph: Fix keystone part for non-pki tokens  https://review.openstack.org/322810
22:48 EmilienM mwhahaha: its weird
22:48 EmilienM tests pass for all distro except ubuntu:
22:48 EmilienM Evaluation Error: Error while evaluating a Function Call, Could not autoload puppet/type/nova_flavor: Could not autoload puppet/provider/nova_flavor/openstack: cannot load such file -- puppet/provider/nova at /home/emilien/git/openstack/puppet/​puppet-octavia/spec/fixtures/module​s/octavia/manifests/worker.pp:63:7 on node xps
22:48 EmilienM (unit tests)
22:49 EmilienM on ubuntu-14.04-x86_64
22:49 mwhahaha Do we need the require ../../.. hack thing
22:49 EmilienM ah
22:50 EmilienM but why only ubuntu?
22:50 EmilienM it's the first loaded?
22:50 EmilienM in the test, yes
22:51 mwhahaha I hate that error cause I've never really gotten to the bottom of it :D
22:51 EmilienM maybe we need require File.join(File.dirname(__FILE__), '..','..','..', 'puppet/provider/nova')
22:51 mwhahaha Yea that
22:54 EmilienM mwhahaha: I don't even know what to write in commit message. Do you see the picture dog? it's me now.
22:54 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: providers: add hack to require nova providers outside puppet-nova  https://review.openstack.org/328525
22:54 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
22:55 mwhahaha Fixing provider load issues when nova flavor is used outside of nova or something
22:55 mwhahaha :)
22:57 openstackgerrit Merged openstack/puppet-neutron: Add nodepool-xenial.yml file  https://review.openstack.org/328416
22:58 EmilienM mwhahaha: have a nice week-end I'm off a bit now, beer o clock
22:58 mwhahaha later
23:23 aimon joined #puppet-openstack
23:24 ducttape_ joined #puppet-openstack
23:25 ducttape_ joined #puppet-openstack

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