Thursday, 2015-10-08

*** openstackgerrit has quit IRC00:01
*** openstackgerrit has joined #tripleo00:01
openstackgerritOpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements  https://review.openstack.org/23228200:09
*** sdake_ has joined #tripleo00:11
*** sdake has quit IRC00:11
*** zaneb has quit IRC00:22
*** spredzy has quit IRC00:22
*** spredzy has joined #tripleo00:23
*** shadower has joined #tripleo00:23
*** d0ugal has quit IRC00:24
*** d0ugal has joined #tripleo00:30
*** d0ugal is now known as Guest7455400:30
*** shivrao has quit IRC00:31
*** morazi has quit IRC00:34
*** zaneb has joined #tripleo00:34
*** rlandy has quit IRC00:50
*** sthillma has quit IRC00:51
*** weshay_xchat has joined #tripleo01:12
*** weshay_xchat is now known as weshay01:12
*** tiswanso has joined #tripleo01:22
*** tiswanso has quit IRC01:26
*** weshay has quit IRC01:48
*** tiswanso has joined #tripleo01:59
*** tiswanso_ has joined #tripleo02:01
*** tiswanso has quit IRC02:04
*** sdake_ is now known as sdake02:15
*** rhallisey has quit IRC02:24
*** sdake has quit IRC02:30
*** lazy_prince has joined #tripleo02:39
*** spzala has joined #tripleo02:40
*** spzala has quit IRC02:40
*** killer_prince has joined #tripleo02:41
*** lazy_prince has quit IRC02:44
*** untriaged-bot has joined #tripleo03:00
untriaged-botUntriaged bugs so far:03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150366903:00
openstackLaunchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa)03:00
*** untriaged-bot has quit IRC03:00
*** sdake has joined #tripleo03:02
*** tristanC has quit IRC03:10
*** tristanC has joined #tripleo03:12
*** tiswanso_ has quit IRC03:16
*** shivrao has joined #tripleo03:23
*** shivrao_ has joined #tripleo03:26
*** shivrao has quit IRC03:28
*** shivrao_ has quit IRC03:28
*** shivrao has joined #tripleo03:30
*** BharatK has joined #tripleo03:31
*** BharatK has quit IRC03:37
*** BharatK has joined #tripleo03:51
*** killer_prince has quit IRC04:23
*** lazy_prince has joined #tripleo04:39
*** jaosorior has joined #tripleo04:40
*** BharatK has quit IRC04:50
*** bvandenh has joined #tripleo04:54
*** masco has joined #tripleo05:01
*** lazy_prince has quit IRC05:02
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable TLS deployment for the Overcloud  https://review.openstack.org/23193005:03
*** aukhan has joined #tripleo05:12
openstackgerritTzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API  https://review.openstack.org/23043205:18
*** tzumainn has quit IRC05:23
*** matbu has quit IRC05:32
*** matbu has joined #tripleo05:34
*** shivrao has quit IRC05:36
*** Marga_ has joined #tripleo05:41
*** Marga_ has quit IRC05:42
*** Marga_ has joined #tripleo05:42
*** BharatK has joined #tripleo05:53
*** jaosorior has quit IRC05:54
*** matbu has quit IRC05:54
*** jaosorior has joined #tripleo05:54
*** matbu has joined #tripleo05:54
*** jaosorior has quit IRC06:13
*** rasca has joined #tripleo06:14
*** lazy_prince has joined #tripleo06:14
*** jaosorior has joined #tripleo06:14
*** adrianopetrich has quit IRC06:15
*** jaosorior has quit IRC06:16
*** jaosorior has joined #tripleo06:17
*** BharatK has quit IRC06:24
*** BharatK has joined #tripleo06:33
*** Ng has quit IRC06:34
*** adrianopetrich has joined #tripleo06:35
*** Ng has joined #tripleo06:38
*** ChanServ sets mode: +v Ng06:38
*** ukalifon1 has joined #tripleo06:42
*** yog_ has joined #tripleo06:43
*** jprovazn has joined #tripleo06:45
*** jcoufal has joined #tripleo06:49
*** mcornea has joined #tripleo07:01
openstackgerritxiexs proposed openstack/os-cloud-config: Add pxe_irmc driver support into os-cloud-config  https://review.openstack.org/18871707:06
*** yamahata has joined #tripleo07:11
*** aufi has joined #tripleo07:14
*** paramite has joined #tripleo07:21
*** Guest74554 is now known as d0ugal07:23
*** shardy has joined #tripleo07:23
*** d0ugal has quit IRC07:23
*** d0ugal has joined #tripleo07:23
openstackgerritJan Provaznik proposed openstack/tripleo-docs: Add a doc note about updating changed templates  https://review.openstack.org/22937307:23
*** ifarkas has joined #tripleo07:25
*** paramite is now known as paramite|afk07:34
*** sthillma has joined #tripleo07:34
*** sthillma_ has joined #tripleo07:37
*** sthillma has quit IRC07:39
*** sthillma_ is now known as sthillma07:39
*** paramite|afk is now known as paramite07:41
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates: Enable Manila integration  https://review.openstack.org/18813707:44
*** jaosorior has quit IRC07:45
*** dtantsur|afk is now known as dtantsur07:46
*** openstackstatus has quit IRC07:47
*** openstackstatus has joined #tripleo07:49
*** ChanServ sets mode: +v openstackstatus07:49
*** jaosorior has joined #tripleo07:49
d0ugalWhat's with all the ERROR: InterpreterNotFound: python3.4?07:50
jaosoriord0ugal: Haven't figured it out. That error happens quite sporadically07:51
d0ugalIt's an odd one.07:52
jaosoriorwaiting for an hour or so and then doing a recheck usually works. But it's pretty annoying07:53
*** adrianopetrich has quit IRC07:53
d0ugaljaosorior: heh, I keep doing rechecks too and felt guilty for it :)07:53
d0ugal(I should probably be hunting for the root of the problem)07:54
*** mbound has joined #tripleo07:59
*** persia has quit IRC07:59
*** persia has joined #tripleo08:00
*** derekh has joined #tripleo08:05
*** shardy_ has joined #tripleo08:08
*** akrivoka has joined #tripleo08:08
*** shardy has quit IRC08:10
*** shardy_ has quit IRC08:13
*** shardy has joined #tripleo08:14
*** cmyster has quit IRC08:18
*** sthillma has quit IRC08:22
*** cmyster has joined #tripleo08:24
*** cmyster has quit IRC08:24
*** cmyster has joined #tripleo08:24
*** lucasagomes has joined #tripleo08:26
*** mkovacik has joined #tripleo08:29
*** paramite is now known as paramite|afk08:37
*** jistr has joined #tripleo08:40
*** paramite|afk is now known as paramite08:45
*** rlandy has joined #tripleo08:48
*** gfidente has joined #tripleo08:49
*** kukacz has joined #tripleo08:51
*** athomas has quit IRC08:56
*** athomas has joined #tripleo08:59
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150366909:00
openstackLaunchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa)09:00
*** untriaged-bot has quit IRC09:00
*** olap has joined #tripleo09:02
*** devvesa has joined #tripleo09:02
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers  https://review.openstack.org/23205109:03
openstackgerritMerged openstack/tripleo-heat-templates: puppet: allow to configure any ceilometer param with Hiera  https://review.openstack.org/23053109:03
*** olap has quit IRC09:04
*** adrianopetrich has joined #tripleo09:06
openstackgerritYanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers  https://review.openstack.org/23207509:07
openstackgerritJan Provaznik proposed openstack/tripleo-specs: Spec for running overcloud nodes reboot  https://review.openstack.org/22448109:07
*** pelix has joined #tripleo09:11
devvesapino|work: I've -1'ed your yesterdays' patch with a humble request09:22
pino|workdevvesa: no, i won't add more magic envvars09:23
devvesaoh09:23
pino|workthey are bad, and can change behaviour without you noticing that09:23
pino|workparameters make invocations explicit, and you can easily spot the differences between invocations09:24
devvesaok. you are more experiencied than me on these executions, I'll remove the -109:24
devvesaI thought you missed it09:25
*** adrianopetrich has quit IRC09:30
*** adrianopetrich has joined #tripleo09:30
*** dtantsur is now known as dtantsur|brb09:36
*** adrianopetrich has quit IRC09:39
*** adrianopetrich has joined #tripleo09:39
*** adrianopetrich has quit IRC09:52
openstackgerritDerek Higgins proposed openstack/diskimage-builder: Avoid transcending /proc with find  https://review.openstack.org/23044809:54
*** adrianopetrich has joined #tripleo09:57
*** tosky has joined #tripleo09:58
*** kbyrne has quit IRC10:12
*** kbyrne has joined #tripleo10:14
shardydevvesa: Hey, context is we used to have a bunch of scripts (tripleo-incubator), where env vars were used heavily and it became quite hard to manage10:24
shardylately we've been trying to move more to a parameter driven CLI approach, via tripleoclient, but there are still some env vars in there as you've discovered10:24
*** shardy_ has joined #tripleo10:25
*** shardy has quit IRC10:25
devvesaIt would be more comfortable with me, since I don't need to modify tripleo.sh and I thought it was a common practice, but yeah, fair enough :)10:26
*** persia has quit IRC10:26
*** persia has joined #tripleo10:27
*** cody-somerville has quit IRC10:27
openstackgerritBharat Kumar Kobagana (BharatK) proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila  https://review.openstack.org/23093610:29
*** shardy_ has quit IRC10:30
openstackgerritYanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers  https://review.openstack.org/23207510:30
*** shardy has joined #tripleo10:31
*** adrianopetrich has quit IRC10:38
*** sdake has quit IRC10:41
*** trown|outttypeww is now known as trown10:42
*** dtantsur|brb is now known as dtantsur10:45
*** thrash|g0ne is now known as thrash10:49
spredzyshardy, question: I am trying to allow a user to specify multiple --ntp-server (https://review.openstack.org/#/c/232075/) in python-tripleoclient. For some reason no matter if I pass a single or several values it seems heat doesn't retrieved the value specified in there. Would you know why ?10:55
spredzyI always end up with ntp::servers: [] but if I hardcode a value in overcloud-without-mergepy I do get the expected value for ntp::servers10:56
jaosoriorspredzy11:02
*** aukhan has quit IRC11:02
jaosorioradded a response in your CR11:02
spredzyjaosorior, thx sorry was already thinking about the amend :p11:03
openstackgerritYanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers  https://review.openstack.org/23207511:03
spredzyjaosorior, done11:03
spredzyjaosorior, would you have any idea of the why of what I stated above ?11:04
jaosoriorspredzy: Not from the top of my head11:05
shardyspredzy: I'm not sure tbh - I'd run tripleoclient with --debug then you can at least prove it's the client that's at fault11:05
shardyspredzy: have you considered that the template change is backwards incompatible?11:05
shardyif we leave the parameter name alone it wouldn't be11:05
jistrspredzy: on line 264 https://review.openstack.org/#/c/232075/4/tripleoclient/v1/overcloud_deploy.py there's a method which creates the parameter env file, you could add a debug print to have the whole file printed on stdout too, to see what went wrong11:06
shardyspredzy: the other option would be to document --ntp-server as taking a comma delimited string11:06
shardyI don't know why action=append isn't working tbh11:07
jistrspredzy: and i think that what jaosorior could be the actual cause of the trouble11:07
jistr*what jaosorior wrote11:07
spredzyjistr, shadower it is actually working if I print parameters it is in there11:07
spredzybut it doesn't end up in the hiera down the line11:07
jistrspredzy: can you paste how the line in the environment file looks?11:08
spredzyjistr, yep typo on the patchset but I do have append on the test platform11:08
spredzyjistr, sure11:08
spredzyjistr, http://paste.openstack.org/show/475711/11:10
jistri think it should be 'NtpServers': '0.centos.pool.ntp.org,1.toto.com'11:10
jistrrather than 'NtpServers': ['0.centos.pool.ntp.org', '1.toto.com']11:11
jaosoriorjistr: Doesn't heat accept both representations?11:11
jaosoriorjistr: At least that's how it appears in the examples http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#parameters-section11:12
spredzyshardy, I take the point for backward incompatiblity will switch back to NtpServer in next review11:12
*** akrivoka has quit IRC11:13
*** akrivoka has joined #tripleo11:13
jistrjaosorior, spredzy: hmm if the array syntax is valid too, then i don't know11:13
jistrspredzy: if you do `heat stack-show overcloud | grep NtpServers`, do you see the correct value?11:15
openstackgerritDerek Higgins proposed openstack/diskimage-builder: Avoid transcending /proc with find  https://review.openstack.org/23044811:16
spredzyjistr, erf... I in fact do11:16
jistrhmm :/11:17
spredzyjistr, no wait... wrong pltfm11:17
spredzyjistr, it is empty11:19
spredzy "NtpServers": ""11:20
jistrspredzy: the paste you posted before is a print of a python data structure, right? (I don't see `parameters:` there)11:21
spredzyjistr, yes its the print of the parameters variable11:22
spredzyin python-tripleoclient11:22
jistrspredzy: if you can debug-print the actual env file as it gets written to tmp, it might uncover something11:22
spredzyjistr, how do I do that ?11:26
jistrspredzy: i was thinking here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L26811:28
jistrprint(json.dumps(parameter_defaults))11:28
jistrbut reading the code now, it seems pretty straightforward11:28
jistri mean, i don't know how something could go wrong if it just uses json.dumps()11:29
*** lucasagomes is now known as lucas-hungry11:30
jistri wonder if the other settings from that environment file take effect11:31
spredzyjistr, the value in paramter_defaults is correct11:31
*** rhallisey has joined #tripleo11:31
*** jcoufal_ has joined #tripleo11:32
jaosoriorspredzy: Was that working when you had the string delimited by commas?11:33
spredzyjaosorior, yes11:34
*** jcoufal has quit IRC11:34
openstackgerritRyan Hallisey proposed openstack/tripleo-heat-templates: Docker compute role configured via Puppet  https://review.openstack.org/20950511:34
spredzylet me redeploy with the comma to confirm and get the output of heat stack-show resoure11:35
openstackgerritGiulio Fidente proposed openstack/tripleo-heat-templates: Ensure Glance API reaches Registry using the service VIP  https://review.openstack.org/23248011:36
*** jcoufal has joined #tripleo11:40
*** jcoufal_ has quit IRC11:42
*** jcoufal has quit IRC11:46
*** jcoufal has joined #tripleo11:47
*** weshay_xchat has joined #tripleo11:47
gfidentespredzy, jistr so in the hierarchy, the topmost preveals on the others?11:54
jistrgfidente: yes11:54
gfidentetks ;)11:54
*** jprovazn has quit IRC12:03
openstackgerritMerged openstack/tripleo-docs: Add a doc note about updating changed templates  https://review.openstack.org/22937312:11
*** jayg|g0n3 is now known as jayg12:15
spredzyjistr, jaosorior ok I have a bit more of context. --ntp-server x,y,z is passed along fine while NtpServer is declared as a string. However as soon as NtpServer is changed from string to comma_delimited_list the value is not understood anymore resulting in ""12:15
jaosoriorspredzy: so, if it's a comma_delimited_list it doesn't take the value when in the tripleoclient it's passed as a list?12:18
*** lucas-hungry is now known as lucasagomes12:18
*** weshay_xchat is now known as weshay12:19
spredzyjaosorior, this is what I am observing at the moment yes12:19
jaosoriorfunky12:20
jaosoriorfair enough then12:20
spredzyjaosorior, which sounds like I am doing something wrong as https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L88-L89 are both declared as comma_delimited_list12:22
spredzyin the overcloud-without-mergepy12:22
jaosoriorspredzy: And you're sure you're using the correct template from your other commit and everything? (I mess that up often haha)12:24
spredzyI am using master + all NtpServer switched from string to comma_delimited_list12:25
*** jprovazn has joined #tripleo12:25
spredzyin overcloud-without-mergepy + puppet/X.yaml12:25
shardyspredzy: comma_delimited_list will accept both a comma delimited string, and an actual list, as in NeutronTunnelIdRanges etc12:25
shardyspredzy: I'd check via openstack --debug overcloud deploy to see exactly what is getting passed to heat12:26
shardyspredzy: I can also provide an example environment that allows deploying directly via heatclient (for comparison) if that helps12:26
*** dprince has joined #tripleo12:28
*** tzumainn has joined #tripleo12:29
spredzyshardy, that would help. I am also redeploying with --debug12:29
*** adrianopetrich has joined #tripleo12:32
*** olap has joined #tripleo12:36
spredzyshardy, "stack_status": "CREATE_IN_PROGRESS", "stack_owner": "admin", "parameters": {"NeutronPublicInterfaceRawDevice": "", "KeystoneSigningKey": "******", "NtpServer": "" this is what I see in --debug while few line above that one I have [...] -H 'X-Auth-Url: http://192.0.2.1:5000/v2.0' -d '{"stack_name": "overcloud", "environment": {"parameter_defaults": {"ControllerCount": 1, "NeutronNetworkVLANRanges": "datacentre:1:1000", "ComputeCount": 1, "NtpSer12:38
spredzyver": "0.centos.pool.ntp.org"12:38
shardy"NtpServer": ""12:39
shardyspredzy: its included twice12:40
shardyspredzy: Oh, I see it's two different log lines12:40
spredzythey are12:41
*** cody-somerville has joined #tripleo12:41
shardyStill, that "" looks wrong, is it included twice, in both parameters and parameter_defaults for example?12:41
*** julim has joined #tripleo12:42
spredzyshardy, no, just once Deploying environment: {u'parameter_defaults': {u'ControllerCount': 1, u'NeutronNetworkVLANRanges': u'datacentre:1:1000', u'ComputeCount': 1, u'NtpServer': u'0.centos.pool.ntp.org', [...]12:44
openstackgerritDougal Matthews proposed openstack/tripleo-common: Collect the common logic used when dealing with Heat  https://review.openstack.org/22899112:48
dprincedtantsur: hi, I've been poking at IPA a bit. What is your take on when TripleO should switch to IPA as a deployment ramdisk?12:54
dprincedtantsur: I'm keen on it because it means one less ramdisk to build (faster CI potentiall)12:54
dtantsurdprince, yesterday is the best time, right now works for me as well :)12:56
*** masterbound has joined #tripleo12:56
trowndprince: I am pretty sure lucasagomes has a patch up for that12:57
* lucasagomes reads12:57
lucasagomesdprince, ++ to switch to IPA12:57
dprincedtantsur: cool. most of my IP changes already landed... but I personally need this too: https://review.openstack.org/#/c/232157/12:57
dprincelucasagomes: BOOM, lets do it12:58
lucasagomesdprince, BOOM, yeah, we already deprecated the bash ramdisk in Ironic btw12:58
dprincedtantsur, lucasagomes: Yeah, I had several issues w/ IPA (testing on real baremetal) but I just got it working yesterday12:58
lucasagomesit was on my list to switch the TripleO CI to IPA (I think I had a patch for it, but needed to be rebased with another patch by trown )12:58
dprincetrown: which patch?12:59
*** mbound has quit IRC12:59
lucasagomesdprince, I had this https://review.openstack.org/#/c/210461/13:00
lucasagomesbut it was before because it was using devtest13:00
lucasagomesdprince, https://review.openstack.org/#/c/222231/ (trown's patch, already merged)13:01
dprincelucasagomes: gotcha, I would have just gone and landed that BTW but it is probably fine to leave it now13:01
lucasagomeswe perhaps just need to change it to build the IPA by default on tripleoclient13:01
trownlucasagomes: it is built by default now13:02
trownlucasagomes: I think we just need to configure Ironic to use it13:02
dprincelucasagomes: yep, got it. I'd rather see us rip all of the 'overcloud image build' out of tripleoclient and use a YAML/JSON config to control what images get built. But cool. I get where it is at13:03
lucasagomestrown, right, so we just need to set it to the nodes13:03
lucasagomesdprince, ++ yeah needs some refactor, but I think that mostly, if not all, the plumbing is done13:03
lucasagomesdprince, lemme know if you need some help setting it up13:03
dprincelucasagomes: cool, thanks. I'll actually have one more patch for the iboot PM to add a bit of sleep in the 'reboot' command.13:04
dprincelucasagomes: the agent deployment uses that power management command... and it fires off super fast13:04
dprincelucasagomes: I need a bit of sleep (probably a config setting too for it)13:05
lucasagomesdprince, cool, sure13:05
dprincelucasagomes: just a minor difference from how the old/new deployment styles work...13:05
dprincelucasagomes: anyway, it works... but that ramdisk is massive13:05
*** lazy_prince has quit IRC13:06
lucasagomesdprince, yeah... I hear ya, it can be shrunk we just need to work on that13:06
lucasagomes<sambetts> lucasagomes: Awesome :D I've managed to get a tiny core IPA image nearly working I think its just missing some dependecies13:07
lucasagomesdprince, ^13:07
lucasagomesit was today, so some people are working on slimmed versions of IPA ramdisks13:08
*** slagle changes topic to "TripleO | CI status: http://tripleo.org/cistatus.html | Docs http://docs.openstack.org/developer/tripleo-docs/"13:08
dprincelucasagomes: nice :)13:09
trownlucasagomes: what is tiny core? is it something that can be packaged downstream?13:09
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers  https://review.openstack.org/23205113:09
trowngoodsquishy now has its own url and owl nice :)13:09
lucasagomestrown, it's just a tiny linux distro13:10
lucasagomestoday we use a fedora cloud image to build IPA13:10
lucasagomesand it comes with a lot of stuff in it13:10
openstackgerritYanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers  https://review.openstack.org/23207513:10
trownlucasagomes: right, just wondering what the RDO implications of using a base other than fedora or centos are13:10
trownlucasagomes: I am not sure we can do that13:11
lucasagomestrown, yeah, probably we can't... we could clean up the fedora image tho13:11
trownya for sure13:11
lucasagomesthe problem I saw with fedora when I did some cleaning was the dependencies13:11
lucasagomese.g qemu-img depending on libX11 stuff for some reason13:11
*** lblanchard has joined #tripleo13:14
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers  https://review.openstack.org/23205113:14
*** shardy_ has joined #tripleo13:14
*** shardy has quit IRC13:15
slaglei started a high priorty review etherpad https://etherpad.openstack.org/p/tripleo-review-priorities13:18
dprincetrown: from an RDO standpoint we could still build a centos ramdisk... it'll just be bigger13:18
slaglefor my own sanity if nothing else :)13:18
*** cody-somerville has quit IRC13:18
*** shardy_ has quit IRC13:19
trownslagle: I like it... all my reviews are high priority :p13:20
*** shardy has joined #tripleo13:20
*** tiswanso has joined #tripleo13:22
trownifarkas: do you think you could look at https://review.openstack.org/221664 I would like to get something merged for tempest for the RDO test day13:23
*** pradk has joined #tripleo13:24
*** morazi has joined #tripleo13:32
*** regebro has quit IRC13:33
*** regebro has joined #tripleo13:34
spredzyjaosorior, shardy jistr I tried to sum up everything in here https://bugs.launchpad.net/tripleo/+bug/150413713:35
openstackLaunchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New]13:35
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Test a update in trunk repository version  https://review.openstack.org/22978913:35
*** masco has quit IRC13:40
*** cody-somerville has joined #tripleo13:42
*** olap has quit IRC13:44
*** olap has joined #tripleo13:45
openstackgerritMarek Aufart proposed openstack/tripleo-common: [WIP]Add resource capabilities  https://review.openstack.org/23253413:46
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Manage keystone related settings directly in t-h-t manifests  https://review.openstack.org/23035713:58
openstackgerritYanis Guenane proposed openstack/tripleo-heat-templates: Add more components virtual ip mapping into controller.  https://review.openstack.org/23136113:58
*** Goneri has joined #tripleo14:00
openstackgerritMarek Aufart proposed openstack/tripleo-common: [WIP]Add resource capabilities  https://review.openstack.org/23253414:00
*** regebro has quit IRC14:01
*** egafford has joined #tripleo14:03
dprinceshardy: this would be a nice add https://review.openstack.org/#/c/213902/114:03
*** cody-somerville has quit IRC14:10
openstackgerritJaume Devesa proposed openstack/tripleo-puppet-elements: Adding networking MidoNet element  https://review.openstack.org/23255114:12
openstackgerritJaume Devesa proposed openstack/tripleo-puppet-elements: Adding networking MidoNet element  https://review.openstack.org/23255114:15
therveHi! I have some issues with instack-virt-setup, was wondering if it was the right place to ask14:19
therveApparently my libvirt isn't configuring dnsmasq properly?14:19
*** jtomasek has quit IRC14:21
dprincetrown: to finish up the conversation earlier are you (or is anyone) working to finish the switch to IPA?14:22
*** lazy_prince has joined #tripleo14:24
trowndprince: I am not currently working on that14:25
trownI am pretty bogged down trying to get RDO liberty released14:25
dprincetrown: okay, thanks I may poke at it a bit if I can...14:26
*** spzala has joined #tripleo14:27
*** olap has quit IRC14:29
*** sdake has joined #tripleo14:29
*** lazy_prince has quit IRC14:29
*** penick has joined #tripleo14:30
*** lazy_prince has joined #tripleo14:31
*** jtomasek has joined #tripleo14:35
dprincethrash: around? I wanted to chat a bit about tripleoclient, and possibly using build-images/load-images from the incubator (or converting those to be part of tripleoclient)14:36
openstackgerritMerged openstack/python-tripleoclient: Update requirements  https://review.openstack.org/23196114:36
thrashdprince: here.14:37
thrashand yes. that's up for inclusion.14:37
thrashdprince: plan is to put them into tripelo-common14:37
dprincethrash: have you done work on it yet?14:37
thrashdprince: I was gonna start yesterday, but got side-tracked.14:37
thrashI think the biggest question I have about it... where should the default go?14:38
*** jaosorior has quit IRC14:38
thrashaka, the base tripleo yaml template.14:38
*** cody-somerville has joined #tripleo14:38
*** jaosorior has joined #tripleo14:38
thrashkinda bikeshedding but...14:38
dprincethrash: rather than just move them over I was thinking perhaps 'openstack diskimage build -c <path to YAML>'14:39
dprincethrash: and perhaps 'openstack diskimage load -c <path to YAML>'14:39
dprincethrash: as for the default YAML, I think tripleo-common would be fine. I want to see these split out into distro specific files though14:40
thrashdprince: I think the first step is to make it into somewhat of an api and put it in tripleo-common, then we can add the new commands to tripleoclient14:40
thrashdprince: I will work on it today.14:41
openstackgerritxiexs proposed openstack/tripleo-incubator: Replace the hard-code home directory with a tilde.  https://review.openstack.org/23256914:41
thrashdprince: work for you? :)14:42
dprincethrash: I'm still a bit fuzzy. Would you use something similar to this: http://git.openstack.org/cgit/openstack/tripleo-incubator/tree/scripts/overcloud_puppet_disk_images.yaml14:43
thrashthat's the idea. were you thinking something else?14:43
spredzydprince, If I want to retrieve the value of PublicVirtualIP in puppet/controller.yaml how should I proceed since https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/controller.yaml#L419 is deprecated. Also when I look at https://github.com/openstack/tripleo-heat-templates/blob/master/overcloud-without-mergepy.yaml#L567-L594 I see that they all point to internal/storage/ctrl not public. Where can I retrieve the public ip value per serv14:43
spredzyice ?14:43
dprincethrash: no, I think that is much better than hard coding things into tripleoclient python, although we could improve build-images I think too14:44
thrashdprince: i would like to see the ability to 'layer' the yamls14:44
dprincethrash: having build-images support overlays (a mechanism) to layer would be idea14:44
dprincethrash: exactly14:44
thrash:)14:44
thrashI think something very similar was done in khaleesi-settings.14:44
* dprince isn't familiar w/ khaleesi-settings14:45
thrashI will start with the naive port/implementation14:45
thrashdprince: it's really just about taking multiple yamls and merging them down14:45
dprincethrash: sure, that sounds simple enough14:45
thrashdprince: ok. cool.14:45
shardydprince: Hey!  Looking at that os-net-config patch14:46
openstackgerritxiexs proposed openstack/tripleo-incubator: Replace the hard-code home directory with a tilde.  https://review.openstack.org/23256914:46
dprincethrash: see also this docker patch I did a long time ago now https://review.openstack.org/#/c/199830/14:46
thrashdprince: will do..14:46
shardydprince: also, wanted to chat about the templates defaults stuff14:47
dprinceshardy: oh right, sorry I -1'd some of those14:47
shardydprince: my motivation is to align more closely with both the docs, and tripleo.sh (which is based on the docs)14:47
dprincethrash: thanks, press on. Very anxious to have this :)14:47
shardydprince: I know it won't work for everyone, but I've been using that environment (as have nearly all downstream users, and most developers), thus those defaults make more sense to me14:48
shardydprince: also, I really thing having hard-coded (different) defaults in the client sucks14:48
dprinceshardy: sure, my concern is mostly where those defaults don't align w/ upstream projects (something TripleO has traditionally strived for)14:48
dprinceshardy: lioadm isn't the cinder iscsi_helper default for example14:48
shardydprince: yeah, I was wondering why we choose lioadm in the client then14:49
dprinceshardy: also, I consider tripleoclient to be an extremely prescriptive tool... almost unusable for some people (like me) who often don't use those defaults14:49
shardydprince: the back-story to all this, is I've started deploying directly via heatclient14:49
*** adrianopetrich has quit IRC14:49
dprinceshardy: why not just a simple env file to set the tripleclient defaults for a distro (or product)14:49
shardyit actually works really nicely, but I need a huge environment file, which is inconsistent with the templates14:49
dprinceshardy: and those can live whereever14:49
thrashshardy: https://review.openstack.org/#/c/226790/14:49
shardydprince: sure, but I still think it makes more sense for the template defaults to align with our docs14:50
dprinceshardy: I deploy directly w/ heatclient sometimes too, I just have a custom.yaml where I set my defaults14:50
*** mkovacik has quit IRC14:50
shardydprince: If we had a "centos_defaults.yaml" in t-h-t, that would also be fine14:50
dprinceshardy: sure, I think some of your defaults probably go into there. lioadm could for example14:51
shardybut then, we should remove all the conflicting defaults in the overcloud-without-mergepy template, because they're not helpful14:51
dprinceshardy: but defaults for things like the images and flavors don't make sense to me I think14:51
thrashshardy: dprince that review is to remove all of those defaults...14:51
dprincethrash: ++14:51
thrashexcept for the images and flavors.14:51
shardydprince: but our docs define images, and flavors, which don't align *at all* with what's in the templates?14:52
slagleugh...we moved templates under os-apply-config/ ?14:52
slaglethat is horrible14:52
shardyI know it's historical, but how does that make sense?14:52
slagleconsidering puppet heavily relies on "os-apply-config"14:52
dprinceshardy: right, and what about other implementations like docker?14:52
shardydprince: that's fine, then we have an environments/docker.yaml14:52
dprinceslagle: what would you call it? we can move it again to wherever?14:53
shardydprince: but the template defaults, if we have any, should at least work with *a* documented implementation14:53
*** penick has quit IRC14:53
slagledprince: honestly, i'd delete. or at least deprecate14:53
shardyand currently, we only have one which is landed, CI'd and actually works14:53
dprinceshardy: they do, for flavors we just require that you set them14:53
slagledprince: if we want a name, how about bash-elements or bash14:53
slagledprince: that's the actual analogy to "puppet"14:53
shardydprince: but we know that every developer and downstream user, if they follow the docs, or use tripleo.sh will have "baremetal"14:53
dprinceslagle: I'm not sure it really matters, and I do think they are headed towards deprecation14:54
shardywhy would we make them write 5 lines in a yaml file, when we know that nearly all new users will need that setting?14:54
dprinceslagle: my goal was to leave them in place until we have at least one other implementation in t-h-t. So docker...14:54
slaglewell, it matters if the point of the change was to add clarity14:54
dprinceslagle: once we have that I think lets deprecate14:54
shardydprince: for folks such as yourself your parameters file still works exactly the same14:54
shardyit's just lowering the bar (slightly) for new folks14:54
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package  https://review.openstack.org/22815714:54
dprinceslagle: name for this is perhaps a bike shed man...14:54
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment  https://review.openstack.org/22815614:54
openstackgerritDerek Higgins proposed openstack-infra/tripleo-ci: Use tripleo.sh to build packages  https://review.openstack.org/22942514:55
dprinceshardy: I don't use baremetal :)14:55
shardydprince: e.g I had to cut/paste the parameters from a --debug from tripleoclient before I could get deployments via heat to work14:55
dprinceshardy: I have 2 flavors FWIW14:55
slagledprince: i'd wager it's not...but given you picked the name, i can see how you feel it might be14:55
shardydprince: *You* don't, and that's fine, but 99% of other folks are14:55
slaglejust the fact that we're saying os-apply-config templates are headed towards deprecation ought to set off alarm bells14:56
shardyreverse engineering from tripleoclient isn't a good experience, even for a developer14:56
slagleos-apply-config aint going anywhere, anytime soon14:56
dprinceshardy: w/ regards to the flavors I think I could be talked into 'baremetal'. FOr the image names though I'm less convinced there is a good default14:56
shardydprince: following our documentation results in exactly one image "openstack-full"14:56
shardywhy wouldn't we use that as a default?14:57
dprinceshardy: I agree, that is why I think a simple distro defaults heat environment is the best approach for these things14:57
dprinceshardy: those docs are still very new, and don't (yet) support docker14:57
shardydprince: that's fine, and setting these defaults has zero imact on docker14:57
shardydprince: if we're saying the defaults can't default to one implementation and actually work, we may as well remove all of them IMO14:58
shardyat least then it's less confusing14:58
dprinceslagle: sorry you don't like the name. I couldn't think of anything better... was just trying to clean house a bit14:58
dprinceslagle: because new users typically find looking at t-h-t to be really confusing14:59
dprinceshardy: For things like flavors, and images I'm not sure I like defaults... because nova doesn't have them14:59
dprinceshardy: those are just required settings14:59
shardydprince: Ok, we should remove all the defaults from the templates then15:00
*** untriaged-bot has joined #tripleo15:00
untriaged-botUntriaged bugs so far:15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150366915:00
openstackLaunchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa)15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150413715:00
openstackLaunchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New]15:00
*** untriaged-bot has quit IRC15:00
shardyand just maintain a defaults environment which is consumed by all CLI/UI tools15:00
dprinceshardy: I think that is taking it too far15:00
*** spzala has quit IRC15:00
shardydprince: the whole point of a "default" is that it works for the most common case15:00
shardywhich, in this case, is the only case we actually document, CI and support atm15:01
*** tiswanso has quit IRC15:01
shardyIn this case, the defaults work for exactly 0% of cases, because we do things like refer to legacy image names from old docs/scripts15:01
*** tiswanso has joined #tripleo15:02
dprinceshardy: so do you think a default for real hardware is prefered to virt testing then? Even though maybe 90% of the developers and users will try things out in virt land?15:02
shardydprince: I did a Heat/TripleO demo at a conference this week, and it was bad - it was like "look here are the 35 different parameters you have to set to deploy TripleO"15:03
dprinceshardy: I would argue there isn't a good virt default. I happen to like 'kvm' ... but in fact it will fail for most devs15:03
dprinceshardy: right, I think a simple heat environment to set those for tripleo-docs makes the most sense here15:03
shardydprince: Ok, maybe that's a bad setting - interestingly following the docs gets you kvm, even on VMs15:03
dprinceshardy: yeah, well perhaps most people use nested virt?15:04
dprinceshardy: anyways, just an example15:04
shardydprince: Ok, let me see if I can rebase things to get the ones we actually agree on landed15:04
dprinceshardy: I'm not -2 on these FWIW.... just a bit unconvinced that there *is* a good default for some of these things15:04
shardythen we can argue over the rest another day ;)15:04
dprinceshardy: half of them are a pretty easy +2 from me :)15:05
shardydprince: Yeah, I get why you say that, but I'm trying to improve the experience for new folks who just follow the docs15:05
openstackgerritThomas Herve proposed openstack/instack-undercloud: Don't use dnsmasq lease file to retrieve address  https://review.openstack.org/23258415:05
*** spzala has joined #tripleo15:05
therveYay first patch15:05
dprinceshardy: ++15:05
shardymore advanced folks can still set whatever they want, and they know how to do it :)15:05
shardytherve: Welcome! :)15:05
therveshardy, Thanks :)15:06
*** aufi has quit IRC15:09
*** masterbound has quit IRC15:12
openstackgerritDan Sneddon proposed openstack/tripleo-docs: Updates to network documentation for new features  https://review.openstack.org/22190815:14
kukaczhi all, second day I'm stucked running introspection with an strange error causing "dracut emergency"15:16
kukaczmay I ask for hints someone here?15:16
*** mbound has joined #tripleo15:20
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package  https://review.openstack.org/22815715:22
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment  https://review.openstack.org/22815615:22
*** david-ly_ has joined #tripleo15:22
*** david-lyle has quit IRC15:23
openstackgerritMerged openstack/instack-undercloud: Run db-sync as ironic-inspector user  https://review.openstack.org/23165615:24
*** david-ly_ is now known as david-lyle15:24
*** mbound has quit IRC15:24
*** alop has joined #tripleo15:26
*** mbound has joined #tripleo15:26
*** mcornea has quit IRC15:30
*** egafford has left #tripleo15:35
shardydprince: Do you think we can land these please?15:41
shardyhttps://review.openstack.org/#/q/status:open+project:openstack/tripleo-heat-templates+branch:master+topic:node_extraconfig,n,z15:41
shardyOn aggregate all the tests are passing between the two patches in the series15:41
openstackgerritThomas Herve proposed openstack/instack-undercloud: Don't use dnsmasq lease file to retrieve address  https://review.openstack.org/23258415:43
openstackgerritMerged openstack/os-net-config: Support multiple addresses assignment with ifcfg  https://review.openstack.org/21390215:49
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package  https://review.openstack.org/22815715:54
openstackgerritDerek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment  https://review.openstack.org/22815615:54
*** egafford has joined #tripleo15:59
*** paramite has quit IRC16:02
*** yog_ has quit IRC16:02
*** ukalifon1 has quit IRC16:03
openstackgerritRyan Brady proposed openstack/instack-undercloud: Adds Swift Versioning Support  https://review.openstack.org/23263416:10
*** mudassirlatif has joined #tripleo16:12
openstackgerritRyan Brady proposed openstack/instack-undercloud: Adds Swift Versioning Support  https://review.openstack.org/23263416:16
*** BharatK has quit IRC16:16
openstackgerritDougal Matthews proposed openstack/tripleo-common: Collect the common logic in tripleo-common  https://review.openstack.org/22899116:16
*** CheKoLyN has joined #tripleo16:17
*** ifarkas has quit IRC16:18
egaffordHi all. In my current dev flow, at `openstack overcloud image upload`, seeing 'ERROR: openstack Required file "./ironic-python-agent.initramfs" does not exist.' This is quite accurate; it definitely does not exist (and never has to date). Is this a known issue?16:19
egafford(Or is there a dependency chain among CRs that's partially merged?)16:20
*** w_ has joined #tripleo16:20
*** olaph has quit IRC16:21
*** regebro has joined #tripleo16:23
trownegafford: it should get built with `openstack overcloud image build` but probably doesnt due to docs16:23
egaffordtrown: Ah, so it's just not part of --all right now?16:24
trownhttps://review.openstack.org/#/c/232147/16:24
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Use same host for cinder volume active/passive  https://review.openstack.org/22532016:24
egaffordtrown: Ran `openstack overcloud image build --all 2>&1 | tee image-build.log`16:24
trownegafford: it is part of --all, but without the DIB_YUM_REPO_CONF in that doc patch the package can be found for it16:24
trownand the image build silently fails to build it16:25
egaffordtrown: Nice.16:25
trownya16:25
egaffordThanks; sounds like all will be well.16:25
trown`export DIB_YUM_REPO_CONF="/etc/yum.repos.d/delorean.repo /etc/yum.repos.d/delorean-deps.repo` is what is needed prior to image build16:25
*** w__ has joined #tripleo16:27
*** w_ has quit IRC16:31
openstackgerritMerged openstack/diskimage-builder: Avoid transcending /proc with find  https://review.openstack.org/23044816:32
*** mbound has quit IRC16:37
*** jistr has quit IRC16:38
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update README.rst  https://review.openstack.org/23067516:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update README.rst  https://review.openstack.org/23067516:41
*** Marga_ has quit IRC16:46
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: pacemaker: Set manage_service/enabled via Hiera  https://review.openstack.org/22188116:51
*** regebro has quit IRC16:51
*** jcoufal has quit IRC16:54
*** derekh has quit IRC16:55
*** sthillma has joined #tripleo17:02
*** kukacz has quit IRC17:02
*** jaosorior has quit IRC17:04
*** trown is now known as trown|lunch17:04
*** jaosorior has joined #tripleo17:04
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update README.rst  https://review.openstack.org/23067517:05
*** sthillma_ has joined #tripleo17:05
*** sthillma has quit IRC17:07
*** sthillma_ is now known as sthillma17:07
*** jaosorior has quit IRC17:08
*** mudassirlatif has quit IRC17:09
*** w__ is now known as olaph17:11
*** penick has joined #tripleo17:12
*** dtantsur is now known as dtantsur|afk17:14
*** lucasagomes has quit IRC17:19
*** Marga_ has joined #tripleo17:20
*** shivrao has joined #tripleo17:23
*** yamahata has quit IRC17:23
*** yamahata has joined #tripleo17:24
*** shadower has quit IRC17:32
*** lazy_prince has quit IRC17:33
*** athomas has quit IRC17:37
*** athomas has joined #tripleo17:37
*** athomas has quit IRC17:45
*** trown|lunch is now known as trown17:48
jpeelerdprince: just fyi, the instack kernel panic occurred again. so i'm disabling docker on the host and we'll see what happens then17:50
dprincejpeeler: sounds good17:57
*** kukacz has joined #tripleo17:57
dprincejpeeler: maybe even reboot to make sure all dm_thin enablement is gone17:57
jpeelerk17:58
dprincejpeeler: I can't remember if I just disabled docker, or if I disabled and rebooted17:58
*** kukacz has quit IRC18:00
slaglethrash: looks like this is needed for tripleoclient: https://review.openstack.org/23269418:00
slaglewe got called out on the ML :)18:00
*** jprovazn has quit IRC18:01
*** kukacz has joined #tripleo18:02
thrashslagle: ack18:08
thrashslagle: I had already handles that.18:08
thrash*handled18:08
thrash:)18:08
thrashhttps://review.openstack.org/#/c/232589/18:08
thrashalthough your commit message looks better. :)18:09
thrashslagle: I will abandon in favor of yours.18:09
thrashslagle: did that break for *everybody*?18:10
thrashif so, that seems bad on their side.18:10
openstackgerritDan Prince proposed openstack/tripleo-heat-templates: Update README.rst  https://review.openstack.org/23067518:12
slaglethrash: i think it's just a verification job they run18:15
thrashslagle: ok. good. :)18:16
*** tosky has quit IRC18:16
*** lblanchard has quit IRC18:16
openstackgerritBen Nemec proposed openstack/instack-undercloud: Nothing to see here  https://review.openstack.org/23270018:22
slaglerhallisey: i'm trying out the docker patch, and getting: http://fpaste.org/276568/44328569/ from Heat after a while18:23
slaglerhallisey: the controller finished deploying fine. have you ever seen this before?18:23
slagleit's not obvious to me why Heat would error like this. i had no stacks present prior to starting the deployment18:24
*** lblanchard has joined #tripleo18:25
*** devvesa has quit IRC18:28
*** pelix has quit IRC18:28
*** gchamoul has quit IRC18:29
*** spzala has quit IRC18:29
dprinceslagle: ive seen that once before I think. I don't think it is related to docker though18:30
dprinceslagle: would be nice to have debug logs on it...18:30
slaglewell, i have debug logs for heat, etc18:33
rhalliseyI wonder if one of the containers just died or something18:35
slaglei think this is on the heat side18:35
rhalliseyweird18:35
jpeelerdprince: got the panic again, no docker running on host and this was after a reboot. appears to be related to ironic since that's what i was doing18:35
slaglei have 2 cores on this undercloud, it's almost as if the request to create this nested stack got picked up by 2 different heat-engine processes18:35
rhalliseyjpeeler, oh really hmm18:35
dprincejpeeler: oh, interesting18:35
dprincejpeeler: perhaps it is just a matter of time for me to hit it again too18:36
dprincejpeeler: can you still login to the VM?18:36
jpeelernot this time18:36
slagledid these kernel panics start after enabling nested kvm?18:37
*** penick has quit IRC18:37
dprincejpeeler: so Ironic is doing qemu-img operations on the fedora/centos-atomic image. I wonder if one of those is firing off the dm_thin code...18:37
jpeelerslagle: although i believe my setup does have nested vm enabled, no nested vms are in use18:38
jpeeler(haven't been deploying with  --libvirt-type=qemu)18:38
dprincejpeeler: perhaps we need to understand whats sorts of IO operations activate this18:38
dprincejpeeler: sure, but if it is something within the VM that we think is causing it (ironic) I don't think nested virt matters18:39
dprincejpeeler: could be worth eliminating that as a concern though18:39
jpeeleri'll turn it off then18:40
trownso ... I think we need a to reboot the undercloud in CI18:40
rhalliseyslagle, the kernel panic is due to docker being present in the overcloud.  For me, I got around it by using fedora-atomic, but not sure if that worked for jpeeler18:40
rhalliseythere is an old kernel *we think* lying around18:40
trownrebooting the undercloud breaks OVS in my virt env18:41
dprincetrown: I think a reboot might be a bit expensive18:42
dprincetrown: timewise18:42
dprincetrown: probably good to track down why it isn't working though18:42
trowndprince: ya, but it is a pretty bad result to have rebooting the undercloud break18:43
trowndprince: but it maybe that is more of a downstream test18:43
dprincetrown: yeah, infra has been pinging us about how long our jobs take18:43
dprincetrown: especially the HA job18:43
rhalliseyslagle, let me know if that happens again. I haven't hit that issue before18:44
dprincetrown: perhaps when we get our walltime down we can entertain adding some coverage for undercloud, overcloud, and perhaps even tenant reboots18:44
trowndprince: makes sense18:44
slagledprince: it looks to me like 2 heat engines collided trying to create the same nested stack18:45
slaglehttps://bugs.launchpad.net/tripleo/+bug/150426618:45
openstackLaunchpad bug 1504266 in tripleo "Stack overcloud-ComputeNodesPostDeployment-dkq4fgg4lqk5-NovaComputeContainersDeployment-6zpj2rijh3lk already has an action (CREATE) in progress." [Medium,Triaged]18:45
dprincetrown: the inability to reboot an undercloud was the initial use case that drove the creation of os-net-config FWIW18:45
dprinceslagle: hmmm, interesting18:46
dprinceslagle: add heat to that bug too?18:46
slaglei did18:47
*** gchamoul has joined #tripleo18:51
*** gchamoul has quit IRC18:51
jpeeleris there a way to force delete an ironic node since delete doesn't work with provision state deploy failed?18:52
*** MasterPiece has joined #tripleo18:52
*** spzala has joined #tripleo18:53
*** gchamoul has joined #tripleo18:54
*** MasterPiece has quit IRC18:55
openstackgerritRyan Brady proposed openstack/tripleo-common: [WIP]Initial workflow implementation of tripleo-common  https://review.openstack.org/22729718:55
dprincejpeeler: this is the sledge hammer approach http://paste.openstack.org/show/475778/18:58
dprincejpeeler: I hit this the other day... and I think ironic has some state transition issues18:59
dprincejpeeler: it should always be possible to recover using only the CLI18:59
dprincejpeeler: probably worth filing an Ironic bug on this...18:59
slagleor just look up the one i filed over a year ago :)18:59
jpeelerer wow19:00
dprinceslagle: probably worth asking someone to have a closer look then19:00
dprinceif it is annoying enough we should fix it19:00
dprinceslagle: got a link?19:01
*** mkovacik has joined #tripleo19:02
slaglei did, a while back19:02
slaglelucas looked into it, but it was all kind of blocked on the breaking lock discussions in ironic19:03
slaglei'm not sure what the status of that is now19:03
*** penick has joined #tripleo19:05
*** rasca has quit IRC19:07
*** gfidente has quit IRC19:10
slagledprince: jpeeler : https://bugs.launchpad.net/ironic/+bug/146193719:10
openstackLaunchpad bug 1461937 in Ironic "Node stuck on DEPLOYING (potentially all *ING) state(s)" [High,Fix released] - Assigned to Lucas Alvares Gomes (lucasagomes)19:10
slagleif it's in deploying, supposedly there is a timeout19:11
dprinceslagle: I think I may be talking about a slightly different case19:14
*** sthillma has quit IRC19:14
dprinceslagle: which is sort of that I'd like the equivalent of a 'nova reset-state' command for ironic that actually works...19:14
dprinceslagle: to reset the power status to 'available'19:14
dprincejpeeler: it might be worth capturing exactly what you did to 'hang' the instance in a bug?19:15
jpeelerdprince: this was due to kernel panicking during a deploy19:16
jpeelerand then rebooting obviously19:16
jpeeleris that worth describing?19:17
rhalliseydprince, got the kernel panic19:17
dprincejpeeler: perhaps, only to the extend that ironic can get into a state that isn't recoverable without editing the database19:17
rhalliseyI just rapidly redeployed the overcloud and that triggered it19:17
jpeelerdprince: i'm sure it'll happen again and then i'll collect some exact real details19:18
dprincerhallisey: yeah, based on what jpeeler said earlier I was thinking maybe this is just local to the ironic VM19:19
dprincerhallisey: because the ironic vm does run qemu-img commands on the fedora-atomic image... perhaps something there is tripping up the dm_thin code in the instack VM kernel?19:20
dprincerhallisey: basically one of the local "volume" operations that ironic does is causing this kernel bug to manifest itself19:21
dprincerhallisey: perhaps?19:21
dprincerhallisey: could we try tailing the ironic conductor log, watching the commands it runs19:26
rhalliseydprince, possibly. I'm definetly looking at ironic for this19:26
dprincerhallisey: and continually deploying just the fedora-atomic image to a VM19:26
dprincerhallisey: I think that would narrow it down quite a bit...19:26
rhalliseykk19:26
dprincerhallisey: I'm trying to look at this too BTW a bit more closely19:29
rhalliseyok cool19:29
slaglerhallisey: so even though the compute node is fedora, i can use my centos7 image for the controller?19:34
slaglei dont see why not, just figured i'd confirm :)19:34
rhalliseyslagle, ya that should be fine19:35
dprincerhallisey, jpeeler: could we do the upcoming "config" rework for docker in a follow up patch19:36
rhalliseyya that's the plan19:36
dprincegreat, thanks19:36
jpeelerack19:36
rhalliseyI'd rather get the docker bits merged now and then move to stable liberty soon after19:36
*** dprince has quit IRC19:38
*** rbrady is now known as rbrady-run19:41
*** dsneddon is now known as dsneddon_lunch19:50
*** lblanchard has quit IRC19:55
*** dprince has joined #tripleo19:57
*** lblanchard has joined #tripleo20:04
egaffordEmilienM: ping (re: Trove).20:08
egaffordMay be too late; if it is please do disregard.20:08
*** rlandy has quit IRC20:11
slaglehe's on the west coast at puppetconf and drinking beer20:11
egaffordslagle: Ah, cool. Not too late in general, then, just too late right now. :)20:11
*** yamahata has quit IRC20:12
egaffordslagle: Was chatting with vkmc (Trove team) re: puppet Trove action items (changing rabbit namespace, etc.) The Trove integration card for director is a little light; is there an upstream card capturing the currently known action items?20:12
slagleegafford: not that i know of20:14
egaffordslagle: Ack. Any sense of when EmilienM might be back about?20:14
slagleegafford: no...next week probably20:16
egaffordslagle: Ok, next week isn't so far away. Thanks.20:17
*** sdake has quit IRC20:17
*** sdake has joined #tripleo20:23
*** tiswanso has quit IRC20:30
*** yamahata has joined #tripleo20:30
*** penick has quit IRC20:31
*** dprince has quit IRC20:34
*** jayg is now known as jayg|g0n320:37
*** penick has joined #tripleo20:38
*** yamahata has quit IRC20:39
*** yamahata has joined #tripleo20:47
EmilienMegafford: hey, sorry I was afk at puppetconf20:47
EmilienMslagle: not yet lol20:47
EmilienMegafford: there is not upstream card capturing the items, but we can create it on https://trello.com/b/4X3zxWRZ/on-going-effort20:48
EmilienMI would be happy to help20:48
*** dsneddon_lunch is now known as dsneddon20:48
egaffordEmilienM: No problem at all! Thanks. What I'm seeing thus far is that the current trove.conf has a disconnect with the puppet modules in re: Rabbit config (puppet want the keys to be in oslo_messaging_rabbit; trove isn't having any of it and is keeping those keys in DEFAULT.)20:55
EmilienMthat sucks20:56
egaffordI'm working on a patch now to fix it, actually; not a puppet expert, but that much is in my power. :)20:56
EmilienMlet me see though, I saw some patches like this20:56
egaffordAh, cool.20:56
*** lblanchard has quit IRC20:56
egaffordI didn't see anything in active review in puppet-trove.20:56
EmilienMegafford: wait, we already have oslo_messaging_rabbit stuff20:57
egaffordIt's also conceivable that it's the conf that's out of date; I have a bit more research to do before definitively defining the scope of the task (sahara guy, not trove guy, really.)20:57
egaffordYeah, you do, but trove doesn't, it seems.20:58
*** MasterPiece has joined #tripleo20:58
EmilienMhttps://github.com/openstack/puppet-trove/blob/master/manifests/api.pp#L344-L35020:58
EmilienMegafford: ^20:58
egaffordEmilienM: Yes, I know (I'm pulling overcloud modules from head of that repo.)20:59
EmilienMso what is missing?20:59
egaffordTrove itself has those keys under DEFAULT/rabbit_userid, etc. in our current packages.20:59
egaffordNot oslo_messaging_rabbit.20:59
EmilienMoh so you need to ask for an OPM update20:59
*** rbrady-run is now known as rbrady20:59
egaffordWhat's that three-letter acronym mean?21:00
EmilienMyou should ping imck8 (not on this channel, but on #rdo)21:00
EmilienMopenstack puppet modules21:00
EmilienMit's an rpm that contains our modules21:00
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150417921:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150366921:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150413721:00
egaffordAh, yes, got it.21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/150426321:00
openstackLaunchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve)21:00
openstackLaunchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa)21:00
*** untriaged-bot has quit IRC21:00
openstackLaunchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New]21:00
openstackLaunchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged]21:00
egaffordI don't think an opm update would help, though; that would pull master into OPM, which would still result in trove and puppet being out of sync, yes.21:01
openstackgerritRyan Brady proposed openstack/tripleo-common: [WIP]Initial workflow implementation of tripleo-common  https://review.openstack.org/22729721:01
*** spzala has quit IRC21:02
egaffordThe puppet modules are setting a fundamentally different config key than the packaged trove.conf, at master. Anyway, let me do a bit more investigation (genconfig on trove master) before I say firmly which one is out of date.21:03
*** trown is now known as trown|outttypeww21:05
openstackgerritTzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API  https://review.openstack.org/23043221:06
*** julim has quit IRC21:11
*** rhallisey has quit IRC21:15
*** sdake has quit IRC21:28
*** spzala has joined #tripleo21:30
*** MasterPiece has quit IRC21:31
*** Goneri has quit IRC21:37
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Drop fixtures  https://review.openstack.org/23278921:39
*** alop has quit IRC21:43
*** weshay has quit IRC21:47
*** ibravo has joined #tripleo21:50
*** thrash is now known as thrash|g0ne21:55
*** akrivoka has quit IRC21:56
*** kukacz has quit IRC22:00
*** adrianopetrich has joined #tripleo22:02
*** sthillma has joined #tripleo22:03
*** morazi has quit IRC22:06
*** pradk has quit IRC22:17
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/23278922:19
egaffordEmilienM: Figured out what's going on; it looks like we need to update trove.conf.sample in the trove package itself. Your team is off the hook entirely on this one. :)22:21
EmilienMegafford: ok22:21
egafford(Unless I'm wrong again, which one can never rule out.)22:21
openstackgerritBen Nemec proposed openstack/instack-undercloud: Update some discovery references to inspection  https://review.openstack.org/23280222:24
openstackgerritBen Nemec proposed openstack/instack-undercloud: Update some discovery references to inspection  https://review.openstack.org/23280222:25
*** ibravo has quit IRC22:31
*** mudassirlatif has joined #tripleo22:35
*** egafford has quit IRC22:35
mudassirlatifHi, what is a rough time estimate for how long it takes to setup an undercloud on a Virtual environment ( http://docs.openstack.org/developer/tripleo-docs/environments/virtual.html )?22:36
mudassirlatifa few hours?22:36
*** rhallisey has joined #tripleo23:06
*** CheKoLyN has quit IRC23:13
*** spzala has quit IRC23:13
*** apuimedo has quit IRC23:19
*** apuimedo has joined #tripleo23:20
*** sthillma_ has joined #tripleo23:29
*** sthillma has quit IRC23:30
*** sthillma_ is now known as sthillma23:30
*** mbound has joined #tripleo23:34
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme  https://review.openstack.org/22235123:51
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update apt-preferences element README from free text to table formatting  https://review.openstack.org/23221123:52
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update apt-preferences element README from free text to table formatting  https://review.openstack.org/23221123:55
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme  https://review.openstack.org/22235123:56
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update apt-conf elements README from free text to table formatting  https://review.openstack.org/23177023:57
openstackgerritAugustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme  https://review.openstack.org/22235123:59

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