Tuesday, 2017-08-08

openstackgerritMerged openstack/puppet-designate master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49134800:04
openstackgerritMerged openstack/puppet-qdr master: Fix permissions on qdrouterd.sasldb  https://review.openstack.org/48925700:34
openstackgerritzhongshengping proposed openstack/puppet-sahara master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49136200:42
openstackgerritMerged openstack/puppet-manila master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49135100:50
openstackgerritMerged openstack/puppet-nova master: Fix unit test  https://review.openstack.org/49135300:50
*** markvoelker has joined #puppet-openstack00:55
*** markvoelker_ has joined #puppet-openstack00:57
*** markvoelker has quit IRC01:01
openstackgerritzhongshengping proposed openstack/puppet-aodh master: Fix rpc_backend option warning  https://review.openstack.org/49131201:18
openstackgerritzhongshengping proposed openstack/puppet-aodh master: Fix rpc_backend option warning  https://review.openstack.org/49131201:18
openstackgerritzhongshengping proposed openstack/puppet-cinder master: Fix rpc_backend option warning  https://review.openstack.org/49130901:20
*** vinsh_ has quit IRC01:24
*** vinsh has joined #puppet-openstack01:27
*** rwsu has quit IRC01:32
openstackgerritMerged openstack/puppet-octavia master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49135701:33
*** rwsu has joined #puppet-openstack01:44
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:52
openstackgerritMerged openstack/puppet-tacker master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49136501:56
openstackgerritMerged openstack/puppet-vitrage master: Fix rpc_backend option warning and test failed  https://review.openstack.org/49131401:56
openstackgerritMerged openstack/puppet-trove master: Fix rpc_backend option warning  https://review.openstack.org/49130801:56
openstackgerritMerged openstack/puppet-watcher master: Change tag link in README to the newest address  https://review.openstack.org/49030601:56
openstackgerritMerged openstack/puppet-magnum master: Deprecate oslo.messaging rpc_backend option parameter  https://review.openstack.org/49135002:12
*** mtreinish has quit IRC02:17
*** vinsh has quit IRC02:18
*** mtreinish has joined #puppet-openstack02:22
*** dixiaoli has joined #puppet-openstack03:39
*** udesale has joined #puppet-openstack03:45
*** ratailor has joined #puppet-openstack04:52
*** skramaja has joined #puppet-openstack05:03
*** chem has quit IRC05:05
*** yprokule has joined #puppet-openstack05:44
*** udesale__ has joined #puppet-openstack06:02
*** udesale has quit IRC06:03
*** indistylo has joined #puppet-openstack06:08
*** agurenko has joined #puppet-openstack06:10
*** rcernin has joined #puppet-openstack06:21
*** dixiaoli has quit IRC06:39
*** jaganathan has joined #puppet-openstack06:45
*** simon-AS559 has joined #puppet-openstack06:53
*** gfidente has joined #puppet-openstack06:55
*** simon-AS559 has quit IRC06:57
LarsErikP question; https://github.com/openstack/puppet-nova/blob/stable/newton/manifests/wsgi/apache_placement.pp this should _not_ create and manage a systemd unit for nova-placement-api, right?07:00
LarsErikP'cause.. i think it's somehow doing that...07:00
*** markvoelker_ has quit IRC07:01
LarsErikPand.. it really shouldn't beacause the placement api must run in wsgi - right?07:03
*** alee has quit IRC07:05
*** markvoelker has joined #puppet-openstack07:07
*** markvoelker has quit IRC07:08
*** markvoelker has joined #puppet-openstack07:08
*** aarefiev has joined #puppet-openstack07:10
*** tesseract has joined #puppet-openstack07:21
*** udesale has joined #puppet-openstack07:23
*** udesale__ has quit IRC07:24
*** udesale__ has joined #puppet-openstack07:26
*** udesale has quit IRC07:29
*** oidgar has joined #puppet-openstack07:31
*** amoralej|off is now known as amoralej07:36
*** duritong has quit IRC07:38
*** ccamacho has joined #puppet-openstack07:38
*** simon-AS559 has joined #puppet-openstack07:39
*** simon-AS5591 has joined #puppet-openstack07:40
*** simon-AS559 has quit IRC07:43
*** duritong has joined #puppet-openstack07:45
*** ratailor is now known as ratailor|Lunch07:45
*** duritong has quit IRC07:50
*** duritong has joined #puppet-openstack07:56
LarsErikPi can't really find the trick to run nova-api and nova-placement api in apache wsgi AND running the nova-api-metadata in eventlet (because metadata apparently can't be run in wsgi?)07:59
LarsErikP^ newton07:59
openstackgerritMerged openstack/puppet-barbican master: Fix unit test  https://review.openstack.org/49133107:59
*** shardy has joined #puppet-openstack08:06
*** oidgar has quit IRC08:14
*** oidgar has joined #puppet-openstack08:14
*** oidgar has quit IRC08:15
*** ratailor|Lunch is now known as ratailor08:16
*** mwarad has joined #puppet-openstack08:17
*** derekh has joined #puppet-openstack08:20
*** karthiks has joined #puppet-openstack08:30
*** saneax has left #puppet-openstack08:43
LarsErikPi really don't understand what's going on here. For every other puppet run, the nova-api.service gets "masked"08:44
LarsErikPhttps://github.com/ntnusky/profile/blob/placement/manifests/openstack/nova/api.pp08:44
*** mwarad has quit IRC08:59
jaosoriorLarsErikP: you need to add service_name = 'httpd' for the nova-api and the nova-placement resources09:06
jaosoriorthen you need to import the nova::wsgi::apache_api   and the nova::wsgi::apache_placement  resources09:06
*** salmankhan has joined #puppet-openstack09:07
*** salmankhan1 has joined #puppet-openstack09:09
*** salmankhan has quit IRC09:11
*** salmankhan1 is now known as salmankhan09:11
LarsErikPjaosorior: but the nova-placement don't have any service_name parameter?09:42
*** shardy has quit IRC09:43
jaosoriorLarsErikP: let me re-check. That does apply for nova-api though.09:43
LarsErikPjaosorior: yep, and for nova-api I have that :-) https://github.com/ntnusky/profile/blob/placement/manifests/openstack/nova/api.pp#L4509:44
jaosoriorLarsErikP: In tripleo we don't seem to have any special configuration for nova placement. Seems it expects you to use wsgi by default09:45
jaosoriorso it would be a matter of importing both the ::nova::placement and the ::nova::wsgi::apache_placement  resources09:45
LarsErikPyeah, and that's what I actually do. The problem is that the nova-api service needs to run both in wsgi and in eventlet, because metadata can't run in wsgi... aaand, now the nova-api service gets masked every other puppet-run...09:47
LarsErikPhttps://github.com/ntnusky/profile/blob/placement/manifests/openstack/nova/api.pp#L18 and https://github.com/ntnusky/profile/blob/placement/manifests/openstack/nova/placement.pp09:47
*** udesale__ has quit IRC09:49
LarsErikPI am assuimg this comment is true now.. https://github.com/openstack/puppet-nova/blob/stable/newton/manifests/api.pp#L40409:50
*** shardy has joined #puppet-openstack09:56
*** fxpester has quit IRC10:02
LarsErikPjaosorior: oh - it may be due to this? https://github.com/openstack/puppet-nova/blob/stable/newton/spec/acceptance/nova_wsgi_apache_spec.rb#L6610:02
*** fxpester has joined #puppet-openstack10:03
jaosoriorLarsErikP: aaah, we run on CentOS/RHEL. It works there :)10:09
jaosoriorbut yeah, metadata runs in eventlet.10:09
LarsErikPfine, but I don't have any plans on changing OS :p10:09
jaosorioroh, and enabled_apis needs to be ['metadata'] IIRC10:09
LarsErikPit defaults to ['osapi_compute', 'metadata'] but if service_name is httpd, it will change to only ['metadata'] ? https://github.com/openstack/puppet-nova/blob/stable/newton/manifests/api.pp#L41710:09
*** dtantsur|afk is now known as dtantsur10:09
LarsErikPbut it must be something related to placement. 'cause - 1st run: nova-api.service is enabled, nova-placement-api.service is masked. 2nd run nova-api.service becomes maskes, and nova-placement-api.service is enabled10:10
LarsErikPbut the nova-placement-api.service shouldn't really exist at all, since it runs in apache wsgi.....10:11
LarsErikPmnaser: do you know the story behind this comment? https://github.com/openstack/puppet-nova/blob/stable/newton/spec/acceptance/nova_wsgi_apache_spec.rb#L6610:12
*** mdavidson has quit IRC10:16
*** iurygregory has quit IRC10:16
*** foxhunt_away has quit IRC10:16
*** mdavidson has joined #puppet-openstack10:16
*** iurygregory has joined #puppet-openstack10:17
*** markvoelker has quit IRC10:17
*** agurenko has quit IRC10:17
*** mkarpin has quit IRC10:17
*** foxhunt7z has joined #puppet-openstack10:18
*** agurenko has joined #puppet-openstack10:18
*** mkarpin has joined #puppet-openstack10:19
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic master: Stop using SSH drivers in tests, they were removed from ironic  https://review.openstack.org/49174610:19
dtantsurthis should fix puppet-ironic CI ^^^10:19
*** colonwq has quit IRC10:19
*** indistylo has quit IRC10:19
*** colonwq has joined #puppet-openstack10:20
*** indistylo has joined #puppet-openstack10:32
*** udesale has joined #puppet-openstack10:53
LarsErikPjaosorior: well, I think I this bug: https://bugs.launchpad.net/cloud-archive/+bug/170067711:00
openstackLaunchpad bug 1700677 in nova (Ubuntu Yakkety) "nova-api conflicts with nova-placement-api in newton on xenial" [High,Triaged]11:00
*** chem has joined #puppet-openstack11:03
*** fultonj has joined #puppet-openstack11:12
*** fultonj has quit IRC11:12
*** fultonj has joined #puppet-openstack11:13
*** amoralej is now known as amoralej|lunch11:22
*** atoth has joined #puppet-openstack11:48
openstackgerritMerged openstack/puppet-magnum master: Allow setting of the DB sync command path  https://review.openstack.org/49039711:49
jaosoriorLarsErikP: right, that sounds like the issue11:51
dtantsurfolks, could you please check https://review.openstack.org/#/c/491746/ ? it's needed to fix the puppet-ironic CI11:52
mnaserLarsErikP i actually run centos as well12:02
mnaserdo you have logs of the puppet run that's failing?12:03
*** jpena|off is now known as jpena12:03
*** trown|outtypewww is now known as trown12:05
*** ratailor has quit IRC12:15
*** skramaja has quit IRC12:24
*** rwsu has quit IRC12:28
*** rwsu has joined #puppet-openstack12:29
*** jaganathan has quit IRC12:30
LarsErikPmnaser: well, I think it's because of this now https://bugs.launchpad.net/cloud-archive/+bug/170067712:39
openstackLaunchpad bug 1700677 in nova (Ubuntu Yakkety) "nova-api conflicts with nova-placement-api in newton on xenial" [High,Triaged]12:39
LarsErikPso, not directly related to the puppet-modules12:39
mnaserLarsErikP i remember reading about this on the ML at some point too12:41
mnaserbut yeah, that's a bit rough :\12:41
LarsErikPyeah, 'cause that basically means that you can't currently run nova-api in nova-placement-api on the same host with xenial...12:44
LarsErikPand.. that kinda' sucks :p12:44
*** dprince has joined #puppet-openstack12:45
*** jpena is now known as jpena|mtg12:46
*** amoralej|lunch is now known as amoralej12:49
*** mandre_away is now known as mandre_mtg12:51
*** vinsh has joined #puppet-openstack12:54
*** jpena|mtg is now known as jpena|off12:56
iurygregorydtantsur, +2 in the ironic patch12:58
dtantsurthanks!12:58
iurygregoryyw ;)12:59
*** ansmith has joined #puppet-openstack12:59
*** clayton has quit IRC13:03
*** clayton has joined #puppet-openstack13:05
*** markvoelker has joined #puppet-openstack13:12
*** alee has joined #puppet-openstack13:21
*** rmart04 has joined #puppet-openstack13:33
*** rmart04 has quit IRC13:38
openstackgerritRadomir Dopieralski proposed openstack/puppet-horizon stable/ocata: Horizon SSL handshake errors with apache  https://review.openstack.org/49181013:41
*** jtomasek has joined #puppet-openstack13:46
*** alee has quit IRC13:55
*** oidgar has joined #puppet-openstack13:56
*** jaganathan has joined #puppet-openstack14:03
*** jtomasek has quit IRC14:15
*** agurenko has quit IRC14:17
*** Guest13936 is now known as med_14:21
*** med_ has quit IRC14:21
*** med_ has joined #puppet-openstack14:21
*** med_ is now known as medberry14:21
*** jpena|off is now known as jpena14:25
*** jaganathan has quit IRC14:34
*** jpena is now known as jpena|off14:38
*** medberry is now known as med_14:43
*** dtantsur is now known as dtantsur|brb14:44
*** indistylo has quit IRC14:55
*** xarses_ has joined #puppet-openstack14:57
*** jaganathan has joined #puppet-openstack14:58
*** dmsimard is now known as dmsimard|afk15:00
*** yprokule has quit IRC15:03
*** martbhell has joined #puppet-openstack15:10
*** jaganathan has quit IRC15:14
*** dtantsur|brb is now known as dtantsur15:30
*** ccamacho has quit IRC15:38
*** ccamacho has joined #puppet-openstack15:38
*** ntpttr_laptop has joined #puppet-openstack15:50
mwhahahawhy are our lint jobs running rake spec now15:55
mwhahahahttp://logs.openstack.org/52/491352/1/gate/gate-puppet-mistral-puppet-lint/9b68f25/console.html#_2016-12-16_09_27_21_57327815:55
mwhahahawait a min15:58
*** agurenko has joined #puppet-openstack15:59
*** alee_ has joined #puppet-openstack16:01
*** simon-AS5591 has quit IRC16:07
*** ntpttr_laptop__ has joined #puppet-openstack16:17
*** ntpttr_laptop has quit IRC16:17
*** ccamacho has quit IRC16:24
*** vinsh has quit IRC16:26
*** rcernin has quit IRC16:31
*** tesseract has quit IRC16:31
*** indistylo has joined #puppet-openstack16:33
*** simon-AS559 has joined #puppet-openstack16:35
*** markvoelker has quit IRC16:36
*** simon-AS559 has quit IRC16:40
*** simon-AS559 has joined #puppet-openstack16:40
martbhellhello :) I'm having some trouble figuring out the right syntax for pci_alias in puppet-nova. Anybody got some idea? If I use the one in the example: https://github.com/openstack/puppet-nova/blob/stable/newton/manifests/api.pp#L81 puppet writes pci_alias= in nova.conf as an array, not a multistr16:40
*** dmsimard|afk is now known as dmsimard16:43
*** agurenko has quit IRC16:45
*** shardy has quit IRC16:46
*** simon-AS559 has quit IRC16:46
*** derekh has quit IRC16:52
mwhahahamartbhell: it looks like in newer versions it's a listopt16:53
mwhahahadid we improperly backport that for newton?16:53
mwhahahaor do you need to upgrade your version for newton?16:53
mwhahahahttps://review.openstack.org/#/c/472611/16:53
*** trown is now known as trown|lunch16:54
martbhellmwhahaha: do you have some hint how one can find which version of nova has it has listopt? https://github.com/openstack/nova/blob/master/nova/conf/pci.py says MultiStrOpt16:59
mwhahahaask #openstack-nova17:00
martbhellwe run nova newton and the puppet version is the latest for newton too, 9.6.017:00
mwhahahaowalsh: do you know anyting about pci_alias?17:00
owalshmwhahaha: err... a little17:00
martbhell:)17:00
mwhahahaowalsh: do you recall anything about that format change and puppet-nova change we did for it? -^ :D17:01
mwhahahai thought it was just an array of strings17:02
* mwhahaha shrugs17:02
martbhelli have been trying to figure out the correct syntax for a two days, i'm quite new to both openstack and puppet though. If I remove the join(any2array(check_array_of_hash( from api.pp and just send $pci_alias it works but I guess these join/any2array are so one can have multiple aliases. But from what I can understand nova.conf wants multiple pci_aliases and each should be a hash17:04
mwhahaharight but it might just be an array of hash strings17:04
*** rwsu has quit IRC17:04
mwhahahai don't know, i honestly hate that config style17:04
mwhahahafor precisely this reason17:05
owalshmwhahaha, martbhell: yea, IIRC multiple pci_aliases is correct, each is a hash17:05
mwhahahahttps://github.com/openstack/puppet-nova/blob/8938e4cff9fb7aa02f6b01037f3afef22792b8c8/lib/puppet/parser/functions/check_array_of_hash.rb17:06
mwhahahaso i think you want an array of json strings17:06
mwhahaha['{foo: 1}', '{bar: 2}']17:06
martbhellit needs to be a string too17:07
*** udesale has quit IRC17:07
mwhahahayea other than what's in the param comments i have no idea what is needed for that17:08
martbhellSo at something like "['{"foo": "1"}', '{"bar": "2"}']" to make it valid-ish json - but i have tried something like this and it then writes the whole thing into one pci_alias, including the [] brackets17:10
mwhahahahttps://github.com/openstack/puppet-nova/blob/ff924133a46d648c130bff26cb21e0240476c91c/spec/classes/nova_api_spec.rb#L12017:11
mwhahahathat's what we test it with17:11
martbhellohh17:11
mwhahahait's just a spec test so that may not be valid config but that seems to be the format17:11
martbhellokay that's a lot of escaping17:12
martbhell:)17:12
martbhelli have not tried that i think17:12
mwhahahacould be lest escaping if the outside quotes were '17:12
mwhahaha'[{"vendor_id":"1234", ...},{"vendor_id":"1235", ...}]'17:13
mwhahahasomething to that effect17:13
martbhellalso I haven't tried to actually enter multiple aliases, only one17:15
martbhellmaybe that could have some effect too17:15
*** dtantsur is now known as dtantsur|afk17:17
owalshso, the example from the spec tests results in the nova.conf pci_alias=[{"vendor_id":"8086","product_id":"0126","name":"graphic_card"},{"vendor_id":"9096","product_id":"1520","name":"network_card"}]17:19
martbhellowalsh: is that by puppeting like this:?  pci_alias => "[{\"vendor_id\":\"8086\",\"product_id\":\"0126\",\"name\":\"graphic_card\"},{\"vendor_id\":\"9096\",\"product_id\":\"1520\",\"name\":\"network_card\"}]"17:22
owalshmartbhell: yes17:23
martbhellokay, with only one entry in the list that doesn't work for me in newton. Is evening time here so I won't be able to play more with this until tomorrow17:24
owalshmartbhell: evening here too, but I'll try to take a look at this later17:26
martbhellmuch appreciated :)17:28
*** vinsh has joined #puppet-openstack17:36
*** markvoelker has joined #puppet-openstack17:37
*** oidgar has quit IRC17:39
*** markvoelker has quit IRC17:44
*** salmankhan1 has joined #puppet-openstack17:48
*** Mosibi has quit IRC17:49
*** salmankhan has quit IRC17:49
*** salmankhan1 is now known as salmankhan17:49
*** salmankhan has quit IRC17:56
*** amoralej is now known as amoralej|off18:00
*** makowals has quit IRC18:06
*** gfidente is now known as gfidente|afk18:08
*** makowals has joined #puppet-openstack18:15
*** vinsh has quit IRC18:23
*** vinsh has joined #puppet-openstack18:24
*** vinsh has quit IRC18:25
*** vinsh_ has joined #puppet-openstack18:25
*** trown|lunch is now known as trown18:30
*** dprince has quit IRC18:36
*** markvoelker has joined #puppet-openstack18:40
*** simon-AS559 has joined #puppet-openstack18:46
*** markvoelker has quit IRC18:48
*** oidgar has joined #puppet-openstack19:08
*** oidgar has quit IRC19:28
*** gfidente|afk is now known as gfidente19:28
*** markvoelker has joined #puppet-openstack19:44
*** markvoelker has quit IRC19:51
*** simon-AS559 has quit IRC20:08
openstackgerritMerged openstack/puppet-ironic master: Stop using SSH drivers in tests, they were removed from ironic  https://review.openstack.org/49174620:26
*** ansmith has quit IRC20:28
*** dprince has joined #puppet-openstack20:55
*** trown is now known as trown|outtypewww21:03
*** dprince has quit IRC21:05
*** markvoelker has joined #puppet-openstack21:37
*** markvoelker has quit IRC21:44
*** dprince has joined #puppet-openstack21:59
*** ntpttr_laptop__ is now known as ntpttr_laptop22:01
*** markvoelker has joined #puppet-openstack22:04
*** markvoelker_ has joined #puppet-openstack22:05
*** markvoelker has quit IRC22:09
*** indistylo has quit IRC22:10
*** alee_ has quit IRC22:18
*** dprince has quit IRC22:31
*** markvoelker has joined #puppet-openstack22:36
*** markvoel_ has joined #puppet-openstack22:37
*** markvoelker_ has quit IRC22:40
*** markvoelker has quit IRC22:41
*** markvoelker has joined #puppet-openstack23:01
*** markvoel_ has quit IRC23:03
*** xarses_ has quit IRC23:05
*** alee_ has joined #puppet-openstack23:09
*** markvoelker has quit IRC23:18
*** markvoelker has joined #puppet-openstack23:18
*** markvoelker has quit IRC23:23
*** gfidente is now known as gfidente|afk23:39

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!