Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-07-15

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

All times shown according to UTC.

Time Nick Message
00:25 openstackgerrit Merged openstack/puppet-openstacklib: Add include on ::mysql::server  https://review.openstack.org/199822
00:32 ducttape_ joined #puppet-openstack
00:41 xarses joined #puppet-openstack
00:50 xingchao joined #puppet-openstack
00:53 topshare joined #puppet-openstack
00:58 aimon joined #puppet-openstack
01:18 rcallawa joined #puppet-openstack
01:23 topshare joined #puppet-openstack
01:36 bklei joined #puppet-openstack
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now Puppet/OpenStack http://irclog.perlgeek.de/puppet-openstack/today - Weekly meeting (every Tuesday 13:00 UTC): https://wiki.openstack.org/w​iki/Meetings/PuppetOpenStack - On going efforts: https://trello.com/puppetopenstack
02:00 markvoelker joined #puppet-openstack
02:02 openstackgerrit Merged openstack/puppet-openstack-integration: puppetfile: Bump puppetlabs-inifile module to 1.4.x  https://review.openstack.org/199988
02:03 rcallawa_ joined #puppet-openstack
02:12 openstackgerrit Merged openstack/puppet-openstack_extras: Fixup README.md table of contents  https://review.openstack.org/198803
02:17 openstackgerrit Merged openstack/puppet-ceilometer: Simplify parameters for rpc_backend  https://review.openstack.org/196036
02:24 openstackgerrit Merged openstack/puppet-neutron: N1Kv:Adding params for IPv6 communication with VSM  https://review.openstack.org/200735
02:42 xarses_ joined #puppet-openstack
02:57 bklei joined #puppet-openstack
03:18 openstackgerrit Merged openstack/puppet-sahara: spec: Update deprecated rspec keywords  https://review.openstack.org/199620
03:20 ducttape_ joined #puppet-openstack
03:21 openstackgerrit Merged openstack/puppet-keystone: fix module install reference  https://review.openstack.org/199810
03:30 topshare joined #puppet-openstack
03:35 markvoelker joined #puppet-openstack
03:38 markvoelker_ joined #puppet-openstack
04:01 bklei joined #puppet-openstack
04:04 markvoelker joined #puppet-openstack
04:08 markvoelker_ joined #puppet-openstack
04:18 linkedinyou joined #puppet-openstack
04:26 rcallawa joined #puppet-openstack
04:31 topshare joined #puppet-openstack
04:36 danieru joined #puppet-openstack
04:41 gildub joined #puppet-openstack
04:57 francois joined #puppet-openstack
05:06 markvoelker joined #puppet-openstack
05:08 markvoelker_ joined #puppet-openstack
05:14 LimorStotland Hi all stupid questions... i have the puppet for mistral and i want to contribute it. where should i open the BP and to where should i contribute it?
05:50 bklei joined #puppet-openstack
05:51 topshare joined #puppet-openstack
05:53 dboik joined #puppet-openstack
06:02 paramite joined #puppet-openstack
06:18 stamak joined #puppet-openstack
06:33 topshare joined #puppet-openstack
06:39 saneax joined #puppet-openstack
07:06 samuelBartel joined #puppet-openstack
07:07 topshare_ joined #puppet-openstack
07:25 zigo joined #puppet-openstack
07:35 _stowa_ joined #puppet-openstack
07:37 topshare joined #puppet-openstack
07:39 bklei joined #puppet-openstack
07:45 dgurtner joined #puppet-openstack
07:45 dachary joined #puppet-openstack
07:54 andreww joined #puppet-openstack
07:56 sergmelikyan joined #puppet-openstack
07:56 topshare joined #puppet-openstack
08:00 jayg|g0n3 joined #puppet-openstack
08:08 dmellado left #puppet-openstack
08:12 jistr joined #puppet-openstack
08:12 fvollero joined #puppet-openstack
08:16 fvollero joined #puppet-openstack
08:25 fvollero joined #puppet-openstack
08:26 fvollero joined #puppet-openstack
08:34 stamak joined #puppet-openstack
08:34 topshare joined #puppet-openstack
08:35 derekh joined #puppet-openstack
08:36 markvoelker joined #puppet-openstack
08:43 LimorStotland_ joined #puppet-openstack
08:45 LimorStotland__ joined #puppet-openstack
08:51 markvoelker joined #puppet-openstack
08:53 LimorStotland joined #puppet-openstack
09:00 rcarrillocruz morning folks
09:00 rcarrillocruz what's the convention for documenting parameters on puppet classes
09:00 rcarrillocruz required params first, optional last
09:00 rcarrillocruz ?
09:06 markvoelker joined #puppet-openstack
09:12 dalgaaf joined #puppet-openstack
09:20 markvoelker joined #puppet-openstack
09:28 bklei joined #puppet-openstack
09:35 markvoelker joined #puppet-openstack
09:49 markvoelker joined #puppet-openstack
09:59 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
10:01 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
10:04 markvoelker joined #puppet-openstack
10:05 openstackgerrit Yanis Guenane proposed openstack/puppet-tripleo: Implement firewalling in tripleo::firewall  https://review.openstack.org/202027
10:06 sergmelikyan joined #puppet-openstack
10:10 rcallawa joined #puppet-openstack
10:12 gildub joined #puppet-openstack
10:18 markvoelker joined #puppet-openstack
10:20 openstackgerrit Ricardo Carrillo Cruz proposed openstack/puppet-ironic: Add bifrost manifest  https://review.openstack.org/202025
10:33 markvoelker joined #puppet-openstack
10:37 rcallawa_ joined #puppet-openstack
10:42 rcallawa joined #puppet-openstack
10:46 rcallawa_ joined #puppet-openstack
10:48 markvoelker joined #puppet-openstack
10:59 EmilienM good morning folks
10:59 EmilienM rcarrillocruz: you have good examples in our modules, let me know if you need some code highlight
11:02 markvoelker joined #puppet-openstack
11:07 sergmelikyan joined #puppet-openstack
11:09 rcarrillocruz EmilienM: hiya, upped https://review.openstack.org/202025 , created example and followed documentation as ::ironic manifest
11:09 rcarrillocruz thx
11:09 EmilienM rcarrillocruz: cool, let me look
11:09 rcarrillocruz EmilienM: where are you based btw? France or are you in the US
11:09 rcarrillocruz you appear to have a wide range of working hours
11:10 EmilienM rcarrillocruz: eastern
11:10 EmilienM rcarrillocruz: you?
11:10 rcarrillocruz CET, Spain
11:10 EmilienM cool :)
11:10 _nick EmilienM: i've got a couple of rdo-related bugs logged that are an easy fix - what's the etiquette as far as submitting a PR goes?  one per bug or a single PR that addresses both?
11:11 EmilienM _nick: the bugs need to be reported on Launchpad and submitted as a regular patch via Gerrit system with the tag Closes-bug: # - More doc here: http://docs.openstack.org/i​nfra/manual/developers.html
11:13 _nick yep, i've reported them both on launchpad - and thanks, just thought i'd double check and ask the obvious before making a fool of myself ;)
11:17 markvoelker joined #puppet-openstack
11:17 bklei joined #puppet-openstack
11:24 markvoelker joined #puppet-openstack
11:33 blkperl_ joined #puppet-openstack
11:33 gfidente joined #puppet-openstack
11:33 gfidente joined #puppet-openstack
11:37 stamak joined #puppet-openstack
11:38 openstack joined #puppet-openstack
11:39 markvoelker joined #puppet-openstack
11:51 xingchao joined #puppet-openstack
11:53 rcallawa joined #puppet-openstack
11:54 markvoelker joined #puppet-openstack
11:56 delattec joined #puppet-openstack
11:59 topshare joined #puppet-openstack
12:05 openstackgerrit Nick Jones proposed openstack/puppet-neutron: VPNaaS package name updates for RHEL / CentOS 7  https://review.openstack.org/202065
12:08 markvoelker joined #puppet-openstack
12:09 leeuwenrjj joined #puppet-openstack
12:10 ducttape_ joined #puppet-openstack
12:12 topshare joined #puppet-openstack
12:13 topshare joined #puppet-openstack
12:17 openstackgerrit Xiaohua Yuan proposed openstack/puppet-manila: Move qpid settings to oslo_messaging_qpid section  https://review.openstack.org/199374
12:17 bklei joined #puppet-openstack
12:18 morazi joined #puppet-openstack
12:23 markvoelker joined #puppet-openstack
12:31 magicboiz joined #puppet-openstack
12:32 magicboiz Hello there!
12:32 magicboiz Anyone smarter than me who can help me with this?:
12:32 magicboiz Error: Could not retrieve catalog from remote server: Error 400 on SERVER: $concat_basedir not defined. Try running again with pluginsync=true on the [master] and/or [main] section of your node's '/etc/puppet/puppet.conf'. at /etc/puppet/environments/production/​modules/concat/manifests/setup.pp:20 on node
12:33 magicboiz I have "pluginsync=true" enabled on master and main and agent sections of puppet.conf
12:33 magicboiz also I'm running foreman...
12:33 magicboiz puppet 3.8.1
12:34 magicboiz foreman 1.8.2
12:34 dprince joined #puppet-openstack
12:37 markvoelker joined #puppet-openstack
12:38 EmilienM magicboiz: have you tried  pluginsync=true on puppetmaster ?
12:40 magicboiz <EmilienM>: it's enabled on puppet.conf, in master section.... isn't it enough? I running apache/passenger btw...
12:41 EmilienM ah
12:41 EmilienM did not read correctly
12:43 magicboiz root@puppetmaster:/etc/puppet/environments# !facter
12:43 magicboiz facter concat_basedir
12:43 magicboiz Already evaluated rest at /var/lib/gems/1.9.1/gems/fact​er-2.4.4/lib/facter/gce.rb:4, reevaluating anyways
12:43 magicboiz Already evaluated rest at /var/lib/gems/1.9.1/gems/facte​r-2.4.4/lib/facter/ec2.rb:21, reevaluating anyways
12:43 magicboiz Already evaluated rest at /var/lib/gems/1.9.1/gems/facte​r-2.4.4/lib/facter/ec2.rb:38, reevaluating anyways
12:44 magicboiz facter doesn't return anything...
12:44 topshare joined #puppet-openstack
12:45 EmilienM magicboiz: maybe you would be luckier to ask on #puppet channel maybe
12:46 magicboiz ok, I'll try there, I asked here because I was trying the openstack module....
12:46 magicboiz thx
12:50 EmilienM magicboiz: which module?
12:50 EmilienM swift I would bet
12:52 markvoelker joined #puppet-openstack
12:54 ferest joined #puppet-openstack
13:01 markvoelker joined #puppet-openstack
13:05 sergmelikyan joined #puppet-openstack
13:09 markvoelker joined #puppet-openstack
13:10 markvoelker_ joined #puppet-openstack
13:14 bklei joined #puppet-openstack
13:15 richm joined #puppet-openstack
13:15 bklei joined #puppet-openstack
13:24 rcarrillocruz EmilienM: i haven't written beaker unit tests so far, do we have puppet-openstack specific guides?
13:24 rcarrillocruz before going to google and getting my head around official docs
13:25 dfisher joined #puppet-openstack
13:25 EmilienM rcarrillocruz: I don't ask for beaker tests, only rspec (see specs/classes)
13:26 rcarrillocruz erm, rspec
13:26 rcarrillocruz see? no idea on puppet unit testing :-)
13:27 EmilienM rcarrillocruz: I'm on a call right now, I'll help you today
13:27 EmilienM or anyone else here
13:28 rcarrillocruz no biggie, i'll dive in rspec tutorials, thx
13:30 derekh joined #puppet-openstack
13:30 _nick EmilienM: https://review.openstack.org/#/c/202065/ is mine, if anything looks / smells bad let me know
13:59 rhefner joined #puppet-openstack
14:03 dboik joined #puppet-openstack
14:04 ducttape_ joined #puppet-openstack
14:10 dboik EmilienM: hi
14:24 topshare joined #puppet-openstack
14:28 sergmelikyan joined #puppet-openstack
14:28 markvoelker joined #puppet-openstack
14:30 topshare joined #puppet-openstack
14:31 markvoel_ joined #puppet-openstack
14:31 rbrooker joined #puppet-openstack
14:31 topshare joined #puppet-openstack
14:34 xingchao joined #puppet-openstack
14:34 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Configure rpc options separately from ceilometer notifications  https://review.openstack.org/198744
14:34 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Add distribute mode for Sahara  https://review.openstack.org/192721
14:36 EmilienM dboik: hey
14:37 dboik EmilienM: I made some more changes to https://review.openstack.org/#/c/197181. Can you take another look and let me know what you think of the approach?
14:37 EmilienM dboik: sure! I was talking with jayg about that in our daily meeting
14:38 dboik ok cool, thanks!
14:49 jasondotstar joined #puppet-openstack
14:53 markvoelker joined #puppet-openstack
14:54 rcarrillocruz hmm
14:54 sergmelikyan joined #puppet-openstack
14:54 rcarrillocruz so, playing with rspec tests
14:54 rcarrillocruz i've done bundle install on puppet-ironic
14:54 rcarrillocruz and that seemd to have installed puppet 4.2.0 gem
14:54 EmilienM rcarrillocruz: yes it does
14:54 markvoelker_ joined #puppet-openstack
14:55 rcarrillocruz which breaks rake tests ( i get some error 'Could not autoload puppet/provider/package/pkgdmg' )
14:55 rcarrillocruz searching on irc history, crinkle suggested to other user to downgrade to puppet 3
14:55 rcarrillocruz is that accurate still?
14:55 rcarrillocruz i'll find out shortly
14:56 rcarrillocruz installing puppet gem v 3.8.1
14:56 EmilienM rcarrillocruz: AFIK, Puppet tests should pass 4.2: https://review.openstack.org/#/c/202025/
14:56 EmilienM maybe your test is not correct?
14:56 rcarrillocruz well, i get that error for already existing tests , have not added my tests yet
14:56 mdorman joined #puppet-openstack
14:57 EmilienM rcarrillocruz: why does it pass our CI now?
14:57 sergmelikyan joined #puppet-openstack
14:57 rcarrillocruz must be something on my env for sure
14:57 EmilienM rcarrillocruz: http://logs.openstack.org/25/202025/3/check​/gate-puppet-ironic-puppet-unit-latest/b2a3​5d0/console.html#_2015-07-15_10_26_23_701
14:58 EmilienM I guess it's in your env
14:58 crinkle rcarrillocruz: what version of rspec-puppet do you have?
14:58 crinkle earlier versions didn't work with puppet 4
14:58 EmilienM should be 2.1.0
14:58 EmilienM afik
14:58 rcarrillocruz i'll tell you in a  sec
14:58 EmilienM crinkle: o/
14:59 crinkle EmilienM: o/
14:59 topshare joined #puppet-openstack
15:00 rcarrillocruz rspec-puppet (2.2.0, 2.1.0)
15:00 rcarrillocruz is that ok?
15:01 crinkle should be
15:02 sergmelikyan joined #puppet-openstack
15:02 rcarrillocruz does that output means that i have two different versions of rspec-puppet?
15:02 EmilienM this is was we test in os infra
15:02 rcarrillocruz i'm completely new to the ruby ecosystem of tools
15:02 rcarrillocruz hmm
15:02 rcarrillocruz interesting
15:02 rcarrillocruz so
15:02 rcarrillocruz downgrading to puppet 3.8.1
15:02 rcarrillocruz makes  tests to pass
15:02 rcarrillocruz 140 examples, 0 failures
15:04 EmilienM mhh
15:04 EmilienM that's weird
15:04 EmilienM we also test rspec-puppet 2.1.0 with puppet 3.8.1 and 4.2
15:05 rcarrillocruz anyway, will dig into that later
15:05 rcarrillocruz quick 101 on rspec tests, i must add a class on /spec/classes for the bifrost stuff
15:05 rcarrillocruz anything else? fixtures, examples?
15:05 rcarrillocruz i take example is kind of actual data you pass to the tests to run
15:05 rcarrillocruz ?
15:06 crinkle examples would be nice but we're not actually great at keeping examples up to date
15:06 crinkle you can add to the acceptance tests
15:06 crinkle in spec/acceptance
15:06 rcarrillocruz ok
15:06 rcarrillocruz so spec/class
15:06 rcarrillocruz spec/acceptance
15:06 crinkle yep
15:06 rcarrillocruz let me see what's already in there for puppet-ironic
15:06 EmilienM acceptance is functional testing
15:07 sergmelikyan joined #puppet-openstack
15:07 * rcarrillocruz is happy to learn new things, not thrilled with ruby tho
15:20 topshare joined #puppet-openstack
15:24 saneax joined #puppet-openstack
15:54 stamak joined #puppet-openstack
16:05 andreww joined #puppet-openstack
16:09 cwolferh joined #puppet-openstack
16:14 xingchao joined #puppet-openstack
16:42 markvoelker joined #puppet-openstack
16:43 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Configure rpc options separately from ceilometer notifications  https://review.openstack.org/198744
16:43 aimon joined #puppet-openstack
16:43 EmilienM crinkle, nibalizer: I'm about being away for one week. I wanted to re-sync about openstack integration CI. 1/ I suggest to push another patchset on https://review.openstack.org/201584, regarding my reviews and have this job voting from start, so we can have functional testing from start for this repo at least - 2/ start a PoC of a minimal OpenStack installationg, using our work in beaker manifests (example with keystone+wsgi is a good start I
16:43 EmilienM think)
16:57 pabelanger joined #puppet-openstack
16:59 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Add distribute mode for Sahara  https://review.openstack.org/192721
17:01 EmilienM pabelanger: last comment
17:02 EmilienM pabelanger: on https://review.openstack.org/#/c/201584
17:05 stamak joined #puppet-openstack
17:10 logan2 joined #puppet-openstack
17:11 rcallawa joined #puppet-openstack
17:15 pabelanger EmilienM, Ah, learned something about zuul. fixing
17:16 pabelanger also, I think puppet-.*$ hits all of puppet-openstack-integration jobs
17:17 pabelanger well, it does I should say
17:25 guimaluf joined #puppet-openstack
17:26 guimaluf hey guys, when running puppet agent in my kesytone node, puppet hangs while executing Debug: Executing '/usr/bin/openstack project list --quiet --format csv --long'
17:27 guimaluf it executes right in the CLI but in puppet it hangs.... any idea?
17:28 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-federation  https://review.openstack.org/190361
17:32 mgarza_ joined #puppet-openstack
17:42 iurygregory hey EmilienM, i have moved the spec to liberty folder and changed the index =)
17:51 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module  https://review.openstack.org/200877
17:51 EmilienM iurygregory: cool, will look
17:52 openstackgerrit Andrey Shestakov proposed openstack/puppet-nova: Add ability to set scheduler_use_baremetal_filters  https://review.openstack.org/202215
17:54 guimaluf EmilienM, could you help me with this? when running puppet agent in my kesytone node, puppet hangs while executing Debug: Executing '/usr/bin/openstack project list --quiet --format csv --long'
17:54 EmilienM guimaluf: I can help if you provide us your manifest
17:55 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-federation  https://review.openstack.org/190361
17:56 iurygregory thanks EmilienM =)
17:57 guimaluf EmilienM, http://paste.openstack.org/show/378162/
17:59 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Run neutron db sync also for each neutron module  https://review.openstack.org/200877
18:00 social now it will magicly pass without duplicate declaration >.>
18:00 richm guimaluf: is this with the latest puppet-keystone from git?
18:00 guimaluf richm, 6.0.0
18:00 guimaluf not master
18:00 guimaluf but stable kilo
18:01 richm guimaluf: what happens if you run this from the command line:
18:01 EmilienM sync db is at false, why?
18:01 EmilienM keystone will fail to start I think if you don't run db sync
18:01 richm # OS_IDENTITY_API_VERSION=3 OS_TOKEN="your admin token" OS_URL=http://localhost:35357/v3 openstack project list --long
18:01 richm guimaluf: ^^^
18:02 EmilienM richm: see my comment, I don't think keystone can run if dbsync has not been run
18:02 richm ok - then ps -ef|grep keystone
18:07 guimaluf richm, I doesn't have v3 endpoint
18:07 guimaluf EmilienM, causa I usualy run mannualy
18:08 guimaluf EmilienM, and I'm testing on production... so I was afraid of massing the DB
18:08 EmilienM lol
18:08 EmilienM guimaluf: my guess is keystone does not start because you don't run db sync
18:09 rcallawa joined #puppet-openstack
18:09 EmilienM the use case of having this boolean is when running multi node or pacemaker, and you need to avoid race condition in running dbsync
18:12 guimaluf EmilienM, but keystone it's started!
18:12 EmilienM guimaluf: and logs?
18:12 EmilienM can you paste it?
18:16 mdorman joined #puppet-openstack
18:19 dalgaaf joined #puppet-openstack
18:21 richm guimaluf: what do you mean "doesn't have v3 endpoint"?
18:21 richm guimaluf: are you using Kilo version of Keystone?
18:22 richm guimaluf: if so, then yes, you have /v3 endpoint
18:23 richm unless that endpoint is created by db sync, and you haven't run db sync in a very long time
18:27 guimaluf richm, my db is synced, but I've not created the v3 endpoint in keystone....
18:29 EmilienM you should not have to
18:35 vinsh joined #puppet-openstack
18:39 richm guimaluf: you don't have to create the v3 endpoint - if you are using Juno or later, it should be there - and Icehouse too I believe
18:40 richm so try the command I pasted above
18:40 richm guimaluf: then, if that works, try https instead of http
18:41 guimaluf richm, i'm using kilo
18:42 guimaluf richm, can I check which OS_* vars it's been use?
18:42 richm guimaluf: not sure what you mean by check which OS_* vars it's been use
18:43 guimaluf richm, username, tenant, etc.
18:43 guimaluf richm, where puppet get this from? keystone.conf?
18:44 richm guimaluf: puppet runs the command exactly as I have pasted it above
18:45 richm guimaluf: it uses your admin_token from /etc/keystone/keystone.conf and constructs the url
18:45 richm guimaluf: which is why I'm asking what that command outputs for you
18:48 guimaluf richm, sorry I miss it
18:48 richm # OS_IDENTITY_API_VERSION=3 OS_TOKEN="your admin token" OS_URL=http://localhost:35357/v3 openstack project list --long
18:48 richm guimaluf: ^^^
18:49 guimaluf richm, it uses http right?
18:49 richm guimaluf: you specified to use https in your manifest - so try both "http:" and "https:"
18:52 guimaluf rich it hangs on https!
18:52 guimaluf INFO: urllib3.connectionpool Starting new HTTPS connection (1): localhost
18:52 guimaluf maybe certificate does not include localhost?
18:54 guimaluf richm, if I use the url included in certificate it works
18:54 guimaluf richm,  :) exactly that!
18:55 EmilienM disable SSL :)
18:56 guimaluf using http it doesn't work
19:02 ducttape_ joined #puppet-openstack
19:03 guimaluf richm, EmilienM  using http I got this NFO: urllib3.connectionpool Starting new HTTP connection (1): localhost
19:03 guimaluf DEBUG: urllib3.connectionpool Setting read timeout to None
19:07 guimaluf EmilienM, why use localhost when using ssl? it shouldn't use the AUTH_URL instead?
19:11 mgarza joined #puppet-openstack
19:13 dgurtner joined #puppet-openstack
19:13 dgurtner joined #puppet-openstack
19:13 guimaluf richm, there is any way to override this OS_URL var? any var in manifest that I could change it?
19:15 stamak joined #puppet-openstack
19:20 ashestakov joined #puppet-openstack
19:20 ashestakov hello
19:21 ashestakov review pls https://review.openstack.org/#/c/202215/
19:26 EmilienM ashestakov: done
19:27 ashestakov @EmilienM: should i add baremetal_scheduler_default_filters also?
19:27 rcallawa joined #puppet-openstack
19:27 EmilienM ashestakov: how would you like to configure it otherwise?
19:27 EmilienM ashestakov: are you using the default one?
19:28 ashestakov it has default value in nova, and it is ok for my case
19:28 ashestakov but i can add it same
19:30 rcallawa joined #puppet-openstack
19:30 EmilienM ashestakov: I think it makes sense though I don't know if it's useful to change it, is there any other filter?
19:30 EmilienM for baremetal I mean
19:31 ashestakov i don't know about non default filters
19:43 markvoelker joined #puppet-openstack
19:44 richm guimaluf: puppet-keystone contructs the URL to use from either 1) the DEFAULT admin_endpoint in keystone.conf or 2) http(s)://admin_bind_host:admin_port/v3
19:44 richm guimaluf: it uses http if [ssl] and [ssl] enable and [ssl] enable = True in keystone.conf
19:46 richm if there is no admin_bind_host in keystone.conf, it will use your ip address, which will cause https to fail
19:46 guimaluf richm, I've tested with http and https, boths hang.... I think my problem is 'localhost'
19:46 markvoelker joined #puppet-openstack
19:46 richm guimaluf: if keystone is listening to https, then http will hang
19:46 richm or fail in some odd way
19:47 richm guimaluf: so does it work if you use https://full.hostname.from.cert/v3 ?
19:47 guimaluf richm, it works!
19:47 guimaluf richm, it's the only way that works
19:48 guimaluf richm, localhost, IP address... nothing of this works
19:48 guimaluf richm, only full.hostname.from.cert
19:48 richm guimaluf: yes, that is as designed
19:48 richm guimaluf: that is the mitm attack prevention
19:49 richm guimaluf: Did your manifest work before upgrading to use puppet-keystone 6.0.0?  That is, is this a regression?
19:50 guimaluf richm, I'd installed keystone by hand, just now I'm trying to install via puppet-keystone
19:50 richm guimaluf: ok - so this is the first time you have tried it
19:50 guimaluf richm, exactly :)
19:50 richm guimaluf: so we need to figure out how you can specify the parameters to make https work
19:50 mgarza joined #puppet-openstack
19:50 guimaluf so I'm trying to adjust the puppet installation with the configs I've already have
19:51 guimaluf richm, thats the point
19:51 guimaluf i'm looking to puppet-keystone source code
19:51 guimaluf to figure out
19:51 richm guimaluf: looks like you should specify admin_endpoint to class ::keystone
19:52 richm specify it as 'https://full.hostname.from.cert:35357/'
19:53 guimaluf richm, without /v3 right?
19:55 richm guimaluf: right
19:57 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Add support for dedicated replication network.  https://review.openstack.org/177037
20:02 mgarza joined #puppet-openstack
20:04 guimaluf richm, aeeeeeeee!!! :D working!
20:05 richm guimaluf: excellent
20:08 guimaluf richm, thank you very very much :)
20:08 * EmilienM waves on richm
20:10 mgarza joined #puppet-openstack
20:12 EmilienM mgagne: can we have an URL that track our bugs in Puppet modules?
20:12 mgagne EmilienM: with project group yes :D
20:12 mgagne otherwise I guess we will have to find an other way
20:14 EmilienM mgagne: so today, we can't, right?
20:14 mgagne yea, I don't know how
20:15 mgagne I'm sure there are plenty of tools written by openstack people somewhere to deal with it
20:15 stamak joined #puppet-openstack
20:34 dgurtner joined #puppet-openstack
20:34 dgurtner joined #puppet-openstack
20:39 mdorman clayton:  there?
20:39 clayton ?
20:39 mdorman saw your comments on some of the db sync reviews, where it was set to notify the class instead of the exec
20:40 mdorman i was curious, what’s the behavior when something notifies a class?  does anything actually happen?  when i saw that code in the review i kinda just assumed that by notifying a class, it would send notifications to all resources in the class
20:40 mdorman but that is probably not what happens?
20:40 clayton it notifies everything in the class
20:40 _ody yep
20:40 mdorman ah,ok
20:40 clayton most of my comments were that he had changed the semantics
20:41 clayton in most cases packages were notifying the db sync exec and he changed them to just be done before
20:41 mdorman oh oh ok
20:41 clayton which means db syncs won't run on upgrades, which is a pretty huge issue imo
20:41 mdorman hyphens and tildes look almost exactly the same in gerrit to me, so i didn’t notice that
20:41 clayton he changed ~> to -> which is *really* hard to see in the font gerrit uses.
20:41 mdorman yup.
20:42 mdorman ok cool.  well i was just curious about the notify class thing.  thanks
20:42 mdorman another wrinkle in the brain
20:42 clayton I checked them all because I've seen it get accidently reverted before
20:42 ashestakov review pls https://review.openstack.org/#/c/202215/
20:46 jasondotstar joined #puppet-openstack
20:49 mgarza joined #puppet-openstack
20:50 EmilienM ashestakov: why false?
20:50 EmilienM ashestakov: it's not the default in OpenStack AFI
20:50 EmilienM AFIK
20:53 dboik joined #puppet-openstack
20:54 ashestakov it used only by ironic_host_manager
20:54 ashestakov https://github.com/openstack/nova/blob/maste​r/nova/scheduler/ironic_host_manager.py#L34
20:55 EmilienM ashestakov: yeah so I suggest you let the default like it's in OpenStack if the scheduler is enabled
20:56 ashestakov you mean if scheduler_use_baremetal_filter=true then baremetal_scheduler_default_filters=[array] ?
20:56 EmilienM ashestakov: I think yeah
20:57 mwhahaha or would undef be better as to not set it but provide the option to set it
20:57 mwhahaha to allow for the shipped defaults?
20:57 EmilienM that's a good option too
20:58 mwhahaha i think for optional stuff it might be best to use undef as to not force something that might change in code later
20:58 openstackgerrit Cody Herriges proposed openstack/puppet-nova: Enable nova service by default  https://review.openstack.org/184656
20:58 EmilienM mwhahaha: wfm
21:00 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: acceptance: validate Keystone with Tempest  https://review.openstack.org/198561
21:02 mgarza joined #puppet-openstack
21:04 rcallawa_ joined #puppet-openstack
21:06 openstackgerrit Andrey Shestakov proposed openstack/puppet-nova: Add ability to set default baremetal filters  https://review.openstack.org/202215
21:07 ashestakov mwhahaha: you mean undef for list of filters?
21:07 mwhahaha both
21:08 tdb joined #puppet-openstack
21:09 ashestakov but scheduler_use_baremetal_filters is boolean
21:10 mwhahaha i think that one would be ok
21:10 mwhahaha but the filters should be undef
21:10 EmilienM the idea is, when you enable it (boolean to true), you got the default filter
21:10 mwhahaha what's the default behaviour for nova_config with a value of undef?
21:11 clayton it won't add or remove the value
21:11 mwhahaha which is why i said undef, so it by default would do nothing and only if someone wants true/false would it do something
21:13 mgarza_ joined #puppet-openstack
21:15 ashestakov so scheduler_use_baremetal_filters=false and baremetal_scheduler_default_filters=undef ?
21:15 mwhahaha I think that would be ok
21:17 ashestakov @EmilienM: are you agree?
21:17 EmilienM ashestakov: yeah
21:19 ashestakov nova_config { 'DEFAULT/baremetal_scheduler_default_filters': value => join($baremetal_scheduler_default_filters,',')
21:19 ashestakov should it work if undef?
21:22 mwhahaha good question
21:23 ashestakov just tried, no
21:23 mwhahaha i'd wrap that in an if $baremetal_scheduler_default_filters then
21:24 mwhahaha rather than if $scheduler_use_baremetal_filters
21:24 mwhahaha similar to the isolated_images block on lines 112-120
21:25 openstackgerrit Andrey Shestakov proposed openstack/puppet-nova: Add ability to set default baremetal filters  https://review.openstack.org/202215
21:26 ashestakov check pls
21:26 dboik_ joined #puppet-openstack
21:26 cwolferh joined #puppet-openstack
21:28 mwhahaha looks ok to me, will need to see CI pass
21:30 EmilienM crinkle: I'm playing with zuul-clooner, and I'm looking for a way to find for a commit, it's latest stable branch (about upgrade job)
21:31 EmilienM like: zuul cloner to checkout the latest stable branch instead of master
21:32 EmilienM maybe a macro to export ZUUL_BRANCH with the latest stable branch of the commit I'm doing
21:34 * EmilienM asking on infra channel
21:44 mgarza joined #puppet-openstack
22:12 mgarza joined #puppet-openstack
22:21 aimon joined #puppet-openstack
22:25 topshare joined #puppet-openstack
22:38 pabelanger EmilienM, crinkle added comments to review.
22:39 EmilienM crinkle: how do you want to gate the repo?
22:40 EmilienM ie: how to test if manifest will actually work?
22:41 EmilienM I agree this is not a module, but still we are doing to write manifests, IMHO we need testing
22:41 crinkle but this is the thing doing the testing
22:41 pabelanger We'd create a new gate job that would run this code on a node
22:42 crinkle i don't see how adding unit tests and beaker tests on top of what we already have t othe individual modules could be helpful
22:42 crinkle pabelanger: right
22:42 pabelanger the validate is ran properly, either check puppet results from CLI and / or tempest jobs results
22:42 EmilienM crinkle: ok for not  having unit
22:42 EmilienM crinkle: but we can have beaker jobs that run the manifests
22:42 EmilienM and see if it works
22:43 EmilienM because if you break the integration manifest, it will break all modules CI
22:43 pabelanger but, if we run puppet apply on the actually scenario001.pp on the remote nodes, do we need beaker at all?
22:44 EmilienM before running scenario001.pp you need to push the file into the repo and for that we need to gate that
22:44 pabelanger right
22:44 crinkle can we just start with the puppet-check-jobs and as this evolves we can figure out what to do next?
22:44 pabelanger zuul can do that
22:44 pabelanger crinkle, ya, that's what I was hoping for too
22:44 EmilienM crinkle: I suggested to have beaker jobs from DAY1 so we can test what we want to actually deploy later
22:44 crinkle pabelanger: ++ sorry i'm picking this up a little late
22:45 pabelanger I think we each have ways of seeing the actual gate for this working, but think we need hammer it out in spec / etherpad
22:45 EmilienM crinkle: how do you test scenarios001.pp is not breaking puppet-keystone deployment?
22:45 EmilienM if you do a mistake in this file and it's merged, all modules CI will be broken
22:46 EmilienM that's the reason I wanted to start with beaker jobs to gate the repo itself
22:46 crinkle EmilienM: so you propose to run all the tests for all the modules on every patch to puppet-openstack-integration? because that is the only way to be sure that it won't break anything
22:46 pabelanger puppet-keystone will have a new gate called gate-openstack-integration-full-tempest-run right?
22:46 pabelanger or something like that
22:46 EmilienM pabelanger: yes something like this
22:47 EmilienM crinkle: I propose for now, we have one single job: gate-openstack-integration-full for example
22:47 pabelanger so that gate job, to me won't be using beaker.... but being deployed onto a bare node then simply calling puppet apply .pp
22:47 EmilienM and this job is run on all modules, included puppet-openstack-integration
22:47 EmilienM so puppet-openstack-integration can be tested itself
22:47 pabelanger right
22:48 pabelanger but I don't think beaker needed, since we have the infra CI tools to do it
22:48 EmilienM but how do you want to test it without beaker?
22:48 EmilienM openstack project have the same issue with tempest
22:48 crinkle i think whether we use beaker to do it or not is a later discussion
22:48 EmilienM they run a job and tempest needs to gate also this job
22:48 pabelanger for each new .pp file we add, we'd get a new jenkins node going to run the test
22:49 EmilienM crinkle: fine with me, I just wanted to start something that we can actually test
22:49 EmilienM and not some random manifests that we don't really test in our gate
22:49 crinkle EmilienM: we don't even have any tests yet
22:49 EmilienM crinkle: I'm used to work in TDD and to me this kind of repo needs to include manifest + test in one commit
22:50 pabelanger so, we are good for just puppet-check-jobs to start
22:50 crinkle pabelanger: i think emilien is arguing against that
22:50 EmilienM and how do you test the manifest?
22:50 crinkle i am +1 just puppet-check-jobs to start
22:50 EmilienM pabelanger: ?
22:50 saneax joined #puppet-openstack
22:50 pabelanger we need to agree on how we are actually going to test
22:51 crinkle i like the idea of one integration job that runs on all the modules and the integration repo itself, i don't care whether it uses beaker, but until we know what the test is going to look like we shouldn't add more jobs
22:51 pabelanger we don't have to merge the code right now
22:51 EmilienM so we push manifests and we wait to patch puppet-keystone later that'll run the job and hopefully works
22:51 pabelanger we can -1 workflow until CI part is done
22:51 EmilienM crinkle: I don't want more jobs now
22:51 EmilienM crinkle: just *one* for one repo
22:51 EmilienM I don't want to run the integration job for all modules now
22:51 crinkle EmilienM: so the puppet-check-job is okay?
22:51 EmilienM sure
22:51 crinkle okay
22:51 EmilienM but I also want beaker for the repo itself
22:52 crinkle let's do that later
22:52 pabelanger how does beaker scale with more then 1 host?
22:52 crinkle pabelanger: beaker can do multihost
22:52 EmilienM pabelanger: it's really early to talk about multi node
22:52 EmilienM we already talked about that yesterday
22:52 pabelanger right
22:52 EmilienM but to me, having tests for the manifest itself is not too early
22:52 pabelanger but for each .pp scenario we need a fresh node to test on right?
22:52 EmilienM it's part of TDD
22:52 pabelanger trying to understand on beaker does that
22:52 EmilienM yes
22:53 pabelanger with zuul / nodepool I know how to do it
22:53 EmilienM let's focus on one scenario, one .pp now
22:53 pabelanger just create a new job in jenkins
22:53 pabelanger ok
22:53 pabelanger redoing code
22:53 crinkle EmilienM: if we add a beaker job now that means we're committing ourselves to making the next commit add spec/spec_helper_acceptance.rb and a gemfile and rakefile and a test, so let's do that later
22:53 EmilienM pabelanger: that's the point, crinkle does not want beaker job now for the repo itself
22:54 EmilienM crinkle: I was thinking at 1/ push a patch for basic structure and make the job pass 2/ with gerrit dependency, push another patch on top of 1/ with a first manifest and the test
22:54 EmilienM and thanks to that ^ we can test what we actually want to deploy
22:55 pabelanger ya, need to learn more about beaker
22:55 crinkle EmilienM: my only point is that we don't have our plan ironed out yet, there seems to be disagreement over whether to even use beaker or not, so just for the moment let's hold off on adding that job until we have a spec ironed out
22:56 EmilienM ok
22:56 EmilienM unfortunately I have to go now for some personal reasons
22:56 EmilienM I'll catch-up next week. I'm happy if you can make progress on this work, feel free to do how you think it's the best. thanks crinkle and pabelanger
22:57 EmilienM o/
23:07 jasondotstar joined #puppet-openstack
23:09 topshare joined #puppet-openstack
23:11 xavpaice left #puppet-openstack
23:17 topshare joined #puppet-openstack
23:34 gildub joined #puppet-openstack
23:44 topshare joined #puppet-openstack
23:46 topshare_ joined #puppet-openstack
23:49 xingchao joined #puppet-openstack
23:58 xarses joined #puppet-openstack

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