*** openstackgerrit has quit IRC | 00:01 | |
*** openstackgerrit has joined #tripleo | 00:01 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/tripleo-common: Updated from global requirements https://review.openstack.org/232282 | 00:09 |
---|---|---|
*** sdake_ has joined #tripleo | 00:11 | |
*** sdake has quit IRC | 00:11 | |
*** zaneb has quit IRC | 00:22 | |
*** spredzy has quit IRC | 00:22 | |
*** spredzy has joined #tripleo | 00:23 | |
*** shadower has joined #tripleo | 00:23 | |
*** d0ugal has quit IRC | 00:24 | |
*** d0ugal has joined #tripleo | 00:30 | |
*** d0ugal is now known as Guest74554 | 00:30 | |
*** shivrao has quit IRC | 00:31 | |
*** morazi has quit IRC | 00:34 | |
*** zaneb has joined #tripleo | 00:34 | |
*** rlandy has quit IRC | 00:50 | |
*** sthillma has quit IRC | 00:51 | |
*** weshay_xchat has joined #tripleo | 01:12 | |
*** weshay_xchat is now known as weshay | 01:12 | |
*** tiswanso has joined #tripleo | 01:22 | |
*** tiswanso has quit IRC | 01:26 | |
*** weshay has quit IRC | 01:48 | |
*** tiswanso has joined #tripleo | 01:59 | |
*** tiswanso_ has joined #tripleo | 02:01 | |
*** tiswanso has quit IRC | 02:04 | |
*** sdake_ is now known as sdake | 02:15 | |
*** rhallisey has quit IRC | 02:24 | |
*** sdake has quit IRC | 02:30 | |
*** lazy_prince has joined #tripleo | 02:39 | |
*** spzala has joined #tripleo | 02:40 | |
*** spzala has quit IRC | 02:40 | |
*** killer_prince has joined #tripleo | 02:41 | |
*** lazy_prince has quit IRC | 02:44 | |
*** untriaged-bot has joined #tripleo | 03:00 | |
untriaged-bot | Untriaged bugs so far: | 03:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 03:00 |
openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 03:00 |
*** untriaged-bot has quit IRC | 03:00 | |
*** sdake has joined #tripleo | 03:02 | |
*** tristanC has quit IRC | 03:10 | |
*** tristanC has joined #tripleo | 03:12 | |
*** tiswanso_ has quit IRC | 03:16 | |
*** shivrao has joined #tripleo | 03:23 | |
*** shivrao_ has joined #tripleo | 03:26 | |
*** shivrao has quit IRC | 03:28 | |
*** shivrao_ has quit IRC | 03:28 | |
*** shivrao has joined #tripleo | 03:30 | |
*** BharatK has joined #tripleo | 03:31 | |
*** BharatK has quit IRC | 03:37 | |
*** BharatK has joined #tripleo | 03:51 | |
*** killer_prince has quit IRC | 04:23 | |
*** lazy_prince has joined #tripleo | 04:39 | |
*** jaosorior has joined #tripleo | 04:40 | |
*** BharatK has quit IRC | 04:50 | |
*** bvandenh has joined #tripleo | 04:54 | |
*** masco has joined #tripleo | 05:01 | |
*** lazy_prince has quit IRC | 05:02 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/tripleo-heat-templates: Enable TLS deployment for the Overcloud https://review.openstack.org/231930 | 05:03 |
*** aukhan has joined #tripleo | 05:12 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API https://review.openstack.org/230432 | 05:18 |
*** tzumainn has quit IRC | 05:23 | |
*** matbu has quit IRC | 05:32 | |
*** matbu has joined #tripleo | 05:34 | |
*** shivrao has quit IRC | 05:36 | |
*** Marga_ has joined #tripleo | 05:41 | |
*** Marga_ has quit IRC | 05:42 | |
*** Marga_ has joined #tripleo | 05:42 | |
*** BharatK has joined #tripleo | 05:53 | |
*** jaosorior has quit IRC | 05:54 | |
*** matbu has quit IRC | 05:54 | |
*** jaosorior has joined #tripleo | 05:54 | |
*** matbu has joined #tripleo | 05:54 | |
*** jaosorior has quit IRC | 06:13 | |
*** rasca has joined #tripleo | 06:14 | |
*** lazy_prince has joined #tripleo | 06:14 | |
*** jaosorior has joined #tripleo | 06:14 | |
*** adrianopetrich has quit IRC | 06:15 | |
*** jaosorior has quit IRC | 06:16 | |
*** jaosorior has joined #tripleo | 06:17 | |
*** BharatK has quit IRC | 06:24 | |
*** BharatK has joined #tripleo | 06:33 | |
*** Ng has quit IRC | 06:34 | |
*** adrianopetrich has joined #tripleo | 06:35 | |
*** Ng has joined #tripleo | 06:38 | |
*** ChanServ sets mode: +v Ng | 06:38 | |
*** ukalifon1 has joined #tripleo | 06:42 | |
*** yog_ has joined #tripleo | 06:43 | |
*** jprovazn has joined #tripleo | 06:45 | |
*** jcoufal has joined #tripleo | 06:49 | |
*** mcornea has joined #tripleo | 07:01 | |
openstackgerrit | xiexs proposed openstack/os-cloud-config: Add pxe_irmc driver support into os-cloud-config https://review.openstack.org/188717 | 07:06 |
*** yamahata has joined #tripleo | 07:11 | |
*** aufi has joined #tripleo | 07:14 | |
*** paramite has joined #tripleo | 07:21 | |
*** Guest74554 is now known as d0ugal | 07:23 | |
*** shardy has joined #tripleo | 07:23 | |
*** d0ugal has quit IRC | 07:23 | |
*** d0ugal has joined #tripleo | 07:23 | |
openstackgerrit | Jan Provaznik proposed openstack/tripleo-docs: Add a doc note about updating changed templates https://review.openstack.org/229373 | 07:23 |
*** ifarkas has joined #tripleo | 07:25 | |
*** paramite is now known as paramite|afk | 07:34 | |
*** sthillma has joined #tripleo | 07:34 | |
*** sthillma_ has joined #tripleo | 07:37 | |
*** sthillma has quit IRC | 07:39 | |
*** sthillma_ is now known as sthillma | 07:39 | |
*** paramite|afk is now known as paramite | 07:41 | |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 07:44 |
*** jaosorior has quit IRC | 07:45 | |
*** dtantsur|afk is now known as dtantsur | 07:46 | |
*** openstackstatus has quit IRC | 07:47 | |
*** openstackstatus has joined #tripleo | 07:49 | |
*** ChanServ sets mode: +v openstackstatus | 07:49 | |
*** jaosorior has joined #tripleo | 07:49 | |
d0ugal | What's with all the ERROR: InterpreterNotFound: python3.4? | 07:50 |
jaosorior | d0ugal: Haven't figured it out. That error happens quite sporadically | 07:51 |
d0ugal | It's an odd one. | 07:52 |
jaosorior | waiting for an hour or so and then doing a recheck usually works. But it's pretty annoying | 07:53 |
*** adrianopetrich has quit IRC | 07:53 | |
d0ugal | jaosorior: 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 #tripleo | 07:59 | |
*** persia has quit IRC | 07:59 | |
*** persia has joined #tripleo | 08:00 | |
*** derekh has joined #tripleo | 08:05 | |
*** shardy_ has joined #tripleo | 08:08 | |
*** akrivoka has joined #tripleo | 08:08 | |
*** shardy has quit IRC | 08:10 | |
*** shardy_ has quit IRC | 08:13 | |
*** shardy has joined #tripleo | 08:14 | |
*** cmyster has quit IRC | 08:18 | |
*** sthillma has quit IRC | 08:22 | |
*** cmyster has joined #tripleo | 08:24 | |
*** cmyster has quit IRC | 08:24 | |
*** cmyster has joined #tripleo | 08:24 | |
*** lucasagomes has joined #tripleo | 08:26 | |
*** mkovacik has joined #tripleo | 08:29 | |
*** paramite is now known as paramite|afk | 08:37 | |
*** jistr has joined #tripleo | 08:40 | |
*** paramite|afk is now known as paramite | 08:45 | |
*** rlandy has joined #tripleo | 08:48 | |
*** gfidente has joined #tripleo | 08:49 | |
*** kukacz has joined #tripleo | 08:51 | |
*** athomas has quit IRC | 08:56 | |
*** athomas has joined #tripleo | 08:59 | |
*** untriaged-bot has joined #tripleo | 09:00 | |
untriaged-bot | Untriaged bugs so far: | 09:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 09:00 |
openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 09:00 |
*** untriaged-bot has quit IRC | 09:00 | |
*** olap has joined #tripleo | 09:02 | |
*** devvesa has joined #tripleo | 09:02 | |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers https://review.openstack.org/232051 | 09:03 |
openstackgerrit | Merged openstack/tripleo-heat-templates: puppet: allow to configure any ceilometer param with Hiera https://review.openstack.org/230531 | 09:03 |
*** olap has quit IRC | 09:04 | |
*** adrianopetrich has joined #tripleo | 09:06 | |
openstackgerrit | Yanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers https://review.openstack.org/232075 | 09:07 |
openstackgerrit | Jan Provaznik proposed openstack/tripleo-specs: Spec for running overcloud nodes reboot https://review.openstack.org/224481 | 09:07 |
*** pelix has joined #tripleo | 09:11 | |
devvesa | pino|work: I've -1'ed your yesterdays' patch with a humble request | 09:22 |
pino|work | devvesa: no, i won't add more magic envvars | 09:23 |
devvesa | oh | 09:23 |
pino|work | they are bad, and can change behaviour without you noticing that | 09:23 |
pino|work | parameters make invocations explicit, and you can easily spot the differences between invocations | 09:24 |
devvesa | ok. you are more experiencied than me on these executions, I'll remove the -1 | 09:24 |
devvesa | I thought you missed it | 09:25 |
*** adrianopetrich has quit IRC | 09:30 | |
*** adrianopetrich has joined #tripleo | 09:30 | |
*** dtantsur is now known as dtantsur|brb | 09:36 | |
*** adrianopetrich has quit IRC | 09:39 | |
*** adrianopetrich has joined #tripleo | 09:39 | |
*** adrianopetrich has quit IRC | 09:52 | |
openstackgerrit | Derek Higgins proposed openstack/diskimage-builder: Avoid transcending /proc with find https://review.openstack.org/230448 | 09:54 |
*** adrianopetrich has joined #tripleo | 09:57 | |
*** tosky has joined #tripleo | 09:58 | |
*** kbyrne has quit IRC | 10:12 | |
*** kbyrne has joined #tripleo | 10:14 | |
shardy | devvesa: 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 manage | 10:24 |
shardy | lately 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 discovered | 10:24 |
*** shardy_ has joined #tripleo | 10:25 | |
*** shardy has quit IRC | 10:25 | |
devvesa | It 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 IRC | 10:26 | |
*** persia has joined #tripleo | 10:27 | |
*** cody-somerville has quit IRC | 10:27 | |
openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/tripleo-heat-templates: Add Gluster integration to Manila https://review.openstack.org/230936 | 10:29 |
*** shardy_ has quit IRC | 10:30 | |
openstackgerrit | Yanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers https://review.openstack.org/232075 | 10:30 |
*** shardy has joined #tripleo | 10:31 | |
*** adrianopetrich has quit IRC | 10:38 | |
*** sdake has quit IRC | 10:41 | |
*** trown|outttypeww is now known as trown | 10:42 | |
*** dtantsur|brb is now known as dtantsur | 10:45 | |
*** thrash|g0ne is now known as thrash | 10:49 | |
spredzy | shardy, 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 |
spredzy | I always end up with ntp::servers: [] but if I hardcode a value in overcloud-without-mergepy I do get the expected value for ntp::servers | 10:56 |
jaosorior | spredzy | 11:02 |
*** aukhan has quit IRC | 11:02 | |
jaosorior | added a response in your CR | 11:02 |
spredzy | jaosorior, thx sorry was already thinking about the amend :p | 11:03 |
openstackgerrit | Yanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers https://review.openstack.org/232075 | 11:03 |
spredzy | jaosorior, done | 11:03 |
spredzy | jaosorior, would you have any idea of the why of what I stated above ? | 11:04 |
jaosorior | spredzy: Not from the top of my head | 11:05 |
shardy | spredzy: I'm not sure tbh - I'd run tripleoclient with --debug then you can at least prove it's the client that's at fault | 11:05 |
shardy | spredzy: have you considered that the template change is backwards incompatible? | 11:05 |
shardy | if we leave the parameter name alone it wouldn't be | 11:05 |
jistr | spredzy: 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 wrong | 11:06 |
shardy | spredzy: the other option would be to document --ntp-server as taking a comma delimited string | 11:06 |
shardy | I don't know why action=append isn't working tbh | 11:07 |
jistr | spredzy: and i think that what jaosorior could be the actual cause of the trouble | 11:07 |
jistr | *what jaosorior wrote | 11:07 |
spredzy | jistr, shadower it is actually working if I print parameters it is in there | 11:07 |
spredzy | but it doesn't end up in the hiera down the line | 11:07 |
jistr | spredzy: can you paste how the line in the environment file looks? | 11:08 |
spredzy | jistr, yep typo on the patchset but I do have append on the test platform | 11:08 |
spredzy | jistr, sure | 11:08 |
spredzy | jistr, http://paste.openstack.org/show/475711/ | 11:10 |
jistr | i think it should be 'NtpServers': '0.centos.pool.ntp.org,1.toto.com' | 11:10 |
jistr | rather than 'NtpServers': ['0.centos.pool.ntp.org', '1.toto.com'] | 11:11 |
jaosorior | jistr: Doesn't heat accept both representations? | 11:11 |
jaosorior | jistr: At least that's how it appears in the examples http://docs.openstack.org/developer/heat/template_guide/hot_spec.html#parameters-section | 11:12 |
spredzy | shardy, I take the point for backward incompatiblity will switch back to NtpServer in next review | 11:12 |
*** akrivoka has quit IRC | 11:13 | |
*** akrivoka has joined #tripleo | 11:13 | |
jistr | jaosorior, spredzy: hmm if the array syntax is valid too, then i don't know | 11:13 |
jistr | spredzy: if you do `heat stack-show overcloud | grep NtpServers`, do you see the correct value? | 11:15 |
openstackgerrit | Derek Higgins proposed openstack/diskimage-builder: Avoid transcending /proc with find https://review.openstack.org/230448 | 11:16 |
spredzy | jistr, erf... I in fact do | 11:16 |
jistr | hmm :/ | 11:17 |
spredzy | jistr, no wait... wrong pltfm | 11:17 |
spredzy | jistr, it is empty | 11:19 |
spredzy | "NtpServers": "" | 11:20 |
jistr | spredzy: the paste you posted before is a print of a python data structure, right? (I don't see `parameters:` there) | 11:21 |
spredzy | jistr, yes its the print of the parameters variable | 11:22 |
spredzy | in python-tripleoclient | 11:22 |
jistr | spredzy: if you can debug-print the actual env file as it gets written to tmp, it might uncover something | 11:22 |
spredzy | jistr, how do I do that ? | 11:26 |
jistr | spredzy: i was thinking here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L268 | 11:28 |
jistr | print(json.dumps(parameter_defaults)) | 11:28 |
jistr | but reading the code now, it seems pretty straightforward | 11:28 |
jistr | i mean, i don't know how something could go wrong if it just uses json.dumps() | 11:29 |
*** lucasagomes is now known as lucas-hungry | 11:30 | |
jistr | i wonder if the other settings from that environment file take effect | 11:31 |
spredzy | jistr, the value in paramter_defaults is correct | 11:31 |
*** rhallisey has joined #tripleo | 11:31 | |
*** jcoufal_ has joined #tripleo | 11:32 | |
jaosorior | spredzy: Was that working when you had the string delimited by commas? | 11:33 |
spredzy | jaosorior, yes | 11:34 |
*** jcoufal has quit IRC | 11:34 | |
openstackgerrit | Ryan Hallisey proposed openstack/tripleo-heat-templates: Docker compute role configured via Puppet https://review.openstack.org/209505 | 11:34 |
spredzy | let me redeploy with the comma to confirm and get the output of heat stack-show resoure | 11:35 |
openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Ensure Glance API reaches Registry using the service VIP https://review.openstack.org/232480 | 11:36 |
*** jcoufal has joined #tripleo | 11:40 | |
*** jcoufal_ has quit IRC | 11:42 | |
*** jcoufal has quit IRC | 11:46 | |
*** jcoufal has joined #tripleo | 11:47 | |
*** weshay_xchat has joined #tripleo | 11:47 | |
gfidente | spredzy, jistr so in the hierarchy, the topmost preveals on the others? | 11:54 |
jistr | gfidente: yes | 11:54 |
gfidente | tks ;) | 11:54 |
*** jprovazn has quit IRC | 12:03 | |
openstackgerrit | Merged openstack/tripleo-docs: Add a doc note about updating changed templates https://review.openstack.org/229373 | 12:11 |
*** jayg|g0n3 is now known as jayg | 12:15 | |
spredzy | jistr, 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 |
jaosorior | spredzy: 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 lucasagomes | 12:18 | |
*** weshay_xchat is now known as weshay | 12:19 | |
spredzy | jaosorior, this is what I am observing at the moment yes | 12:19 |
jaosorior | funky | 12:20 |
jaosorior | fair enough then | 12:20 |
spredzy | jaosorior, 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_list | 12:22 |
spredzy | in the overcloud-without-mergepy | 12:22 |
jaosorior | spredzy: And you're sure you're using the correct template from your other commit and everything? (I mess that up often haha) | 12:24 |
spredzy | I am using master + all NtpServer switched from string to comma_delimited_list | 12:25 |
*** jprovazn has joined #tripleo | 12:25 | |
spredzy | in overcloud-without-mergepy + puppet/X.yaml | 12:25 |
shardy | spredzy: comma_delimited_list will accept both a comma delimited string, and an actual list, as in NeutronTunnelIdRanges etc | 12:25 |
shardy | spredzy: I'd check via openstack --debug overcloud deploy to see exactly what is getting passed to heat | 12:26 |
shardy | spredzy: I can also provide an example environment that allows deploying directly via heatclient (for comparison) if that helps | 12:26 |
*** dprince has joined #tripleo | 12:28 | |
*** tzumainn has joined #tripleo | 12:29 | |
spredzy | shardy, that would help. I am also redeploying with --debug | 12:29 |
*** adrianopetrich has joined #tripleo | 12:32 | |
*** olap has joined #tripleo | 12:36 | |
spredzy | shardy, "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, "NtpSer | 12:38 |
spredzy | ver": "0.centos.pool.ntp.org" | 12:38 |
shardy | "NtpServer": "" | 12:39 |
shardy | spredzy: its included twice | 12:40 |
shardy | spredzy: Oh, I see it's two different log lines | 12:40 |
spredzy | they are | 12:41 |
*** cody-somerville has joined #tripleo | 12:41 | |
shardy | Still, that "" looks wrong, is it included twice, in both parameters and parameter_defaults for example? | 12:41 |
*** julim has joined #tripleo | 12:42 | |
spredzy | shardy, 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 |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Collect the common logic used when dealing with Heat https://review.openstack.org/228991 | 12:48 |
dprince | dtantsur: 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 |
dprince | dtantsur: I'm keen on it because it means one less ramdisk to build (faster CI potentiall) | 12:54 |
dtantsur | dprince, yesterday is the best time, right now works for me as well :) | 12:56 |
*** masterbound has joined #tripleo | 12:56 | |
trown | dprince: I am pretty sure lucasagomes has a patch up for that | 12:57 |
* lucasagomes reads | 12:57 | |
lucasagomes | dprince, ++ to switch to IPA | 12:57 |
dprince | dtantsur: cool. most of my IP changes already landed... but I personally need this too: https://review.openstack.org/#/c/232157/ | 12:57 |
dprince | lucasagomes: BOOM, lets do it | 12:58 |
lucasagomes | dprince, BOOM, yeah, we already deprecated the bash ramdisk in Ironic btw | 12:58 |
dprince | dtantsur, lucasagomes: Yeah, I had several issues w/ IPA (testing on real baremetal) but I just got it working yesterday | 12:58 |
lucasagomes | it 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 |
dprince | trown: which patch? | 12:59 |
*** mbound has quit IRC | 12:59 | |
lucasagomes | dprince, I had this https://review.openstack.org/#/c/210461/ | 13:00 |
lucasagomes | but it was before because it was using devtest | 13:00 |
lucasagomes | dprince, https://review.openstack.org/#/c/222231/ (trown's patch, already merged) | 13:01 |
dprince | lucasagomes: gotcha, I would have just gone and landed that BTW but it is probably fine to leave it now | 13:01 |
lucasagomes | we perhaps just need to change it to build the IPA by default on tripleoclient | 13:01 |
trown | lucasagomes: it is built by default now | 13:02 |
trown | lucasagomes: I think we just need to configure Ironic to use it | 13:02 |
dprince | lucasagomes: 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 at | 13:03 |
lucasagomes | trown, right, so we just need to set it to the nodes | 13:03 |
lucasagomes | dprince, ++ yeah needs some refactor, but I think that mostly, if not all, the plumbing is done | 13:03 |
lucasagomes | dprince, lemme know if you need some help setting it up | 13:03 |
dprince | lucasagomes: 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 |
dprince | lucasagomes: the agent deployment uses that power management command... and it fires off super fast | 13:04 |
dprince | lucasagomes: I need a bit of sleep (probably a config setting too for it) | 13:05 |
lucasagomes | dprince, cool, sure | 13:05 |
dprince | lucasagomes: just a minor difference from how the old/new deployment styles work... | 13:05 |
dprince | lucasagomes: anyway, it works... but that ramdisk is massive | 13:05 |
*** lazy_prince has quit IRC | 13:06 | |
lucasagomes | dprince, yeah... I hear ya, it can be shrunk we just need to work on that | 13:06 |
lucasagomes | <sambetts> lucasagomes: Awesome :D I've managed to get a tiny core IPA image nearly working I think its just missing some dependecies | 13:07 |
lucasagomes | dprince, ^ | 13:07 |
lucasagomes | it was today, so some people are working on slimmed versions of IPA ramdisks | 13:08 |
*** slagle changes topic to "TripleO | CI status: http://tripleo.org/cistatus.html | Docs http://docs.openstack.org/developer/tripleo-docs/" | 13:08 | |
dprince | lucasagomes: nice :) | 13:09 |
trown | lucasagomes: what is tiny core? is it something that can be packaged downstream? | 13:09 |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers https://review.openstack.org/232051 | 13:09 |
trown | goodsquishy now has its own url and owl nice :) | 13:09 |
lucasagomes | trown, it's just a tiny linux distro | 13:10 |
lucasagomes | today we use a fedora cloud image to build IPA | 13:10 |
lucasagomes | and it comes with a lot of stuff in it | 13:10 |
openstackgerrit | Yanis Guenane proposed openstack/python-tripleoclient: Amend --ntp-server help message to add support for multiple servers https://review.openstack.org/232075 | 13:10 |
trown | lucasagomes: right, just wondering what the RDO implications of using a base other than fedora or centos are | 13:10 |
trown | lucasagomes: I am not sure we can do that | 13:11 |
lucasagomes | trown, yeah, probably we can't... we could clean up the fedora image tho | 13:11 |
trown | ya for sure | 13:11 |
lucasagomes | the problem I saw with fedora when I did some cleaning was the dependencies | 13:11 |
lucasagomes | e.g qemu-img depending on libX11 stuff for some reason | 13:11 |
*** lblanchard has joined #tripleo | 13:14 | |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Allow a user to specify a comma separated list of ntp servers https://review.openstack.org/232051 | 13:14 |
*** shardy_ has joined #tripleo | 13:14 | |
*** shardy has quit IRC | 13:15 | |
slagle | i started a high priorty review etherpad https://etherpad.openstack.org/p/tripleo-review-priorities | 13:18 |
dprince | trown: from an RDO standpoint we could still build a centos ramdisk... it'll just be bigger | 13:18 |
slagle | for my own sanity if nothing else :) | 13:18 |
*** cody-somerville has quit IRC | 13:18 | |
*** shardy_ has quit IRC | 13:19 | |
trown | slagle: I like it... all my reviews are high priority :p | 13:20 |
*** shardy has joined #tripleo | 13:20 | |
*** tiswanso has joined #tripleo | 13:22 | |
trown | ifarkas: 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 day | 13:23 |
*** pradk has joined #tripleo | 13:24 | |
*** morazi has joined #tripleo | 13:32 | |
*** regebro has quit IRC | 13:33 | |
*** regebro has joined #tripleo | 13:34 | |
spredzy | jaosorior, shardy jistr I tried to sum up everything in here https://bugs.launchpad.net/tripleo/+bug/1504137 | 13:35 |
openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 13:35 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Test a update in trunk repository version https://review.openstack.org/229789 | 13:35 |
*** masco has quit IRC | 13:40 | |
*** cody-somerville has joined #tripleo | 13:42 | |
*** olap has quit IRC | 13:44 | |
*** olap has joined #tripleo | 13:45 | |
openstackgerrit | Marek Aufart proposed openstack/tripleo-common: [WIP]Add resource capabilities https://review.openstack.org/232534 | 13:46 |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Manage keystone related settings directly in t-h-t manifests https://review.openstack.org/230357 | 13:58 |
openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Add more components virtual ip mapping into controller. https://review.openstack.org/231361 | 13:58 |
*** Goneri has joined #tripleo | 14:00 | |
openstackgerrit | Marek Aufart proposed openstack/tripleo-common: [WIP]Add resource capabilities https://review.openstack.org/232534 | 14:00 |
*** regebro has quit IRC | 14:01 | |
*** egafford has joined #tripleo | 14:03 | |
dprince | shardy: this would be a nice add https://review.openstack.org/#/c/213902/1 | 14:03 |
*** cody-somerville has quit IRC | 14:10 | |
openstackgerrit | Jaume Devesa proposed openstack/tripleo-puppet-elements: Adding networking MidoNet element https://review.openstack.org/232551 | 14:12 |
openstackgerrit | Jaume Devesa proposed openstack/tripleo-puppet-elements: Adding networking MidoNet element https://review.openstack.org/232551 | 14:15 |
therve | Hi! I have some issues with instack-virt-setup, was wondering if it was the right place to ask | 14:19 |
therve | Apparently my libvirt isn't configuring dnsmasq properly? | 14:19 |
*** jtomasek has quit IRC | 14:21 | |
dprince | trown: to finish up the conversation earlier are you (or is anyone) working to finish the switch to IPA? | 14:22 |
*** lazy_prince has joined #tripleo | 14:24 | |
trown | dprince: I am not currently working on that | 14:25 |
trown | I am pretty bogged down trying to get RDO liberty released | 14:25 |
dprince | trown: okay, thanks I may poke at it a bit if I can... | 14:26 |
*** spzala has joined #tripleo | 14:27 | |
*** olap has quit IRC | 14:29 | |
*** sdake has joined #tripleo | 14:29 | |
*** lazy_prince has quit IRC | 14:29 | |
*** penick has joined #tripleo | 14:30 | |
*** lazy_prince has joined #tripleo | 14:31 | |
*** jtomasek has joined #tripleo | 14:35 | |
dprince | thrash: 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 |
openstackgerrit | Merged openstack/python-tripleoclient: Update requirements https://review.openstack.org/231961 | 14:36 |
thrash | dprince: here. | 14:37 |
thrash | and yes. that's up for inclusion. | 14:37 |
thrash | dprince: plan is to put them into tripelo-common | 14:37 |
dprince | thrash: have you done work on it yet? | 14:37 |
thrash | dprince: I was gonna start yesterday, but got side-tracked. | 14:37 |
thrash | I think the biggest question I have about it... where should the default go? | 14:38 |
*** jaosorior has quit IRC | 14:38 | |
thrash | aka, the base tripleo yaml template. | 14:38 |
*** cody-somerville has joined #tripleo | 14:38 | |
*** jaosorior has joined #tripleo | 14:38 | |
thrash | kinda bikeshedding but... | 14:38 |
dprince | thrash: rather than just move them over I was thinking perhaps 'openstack diskimage build -c <path to YAML>' | 14:39 |
dprince | thrash: and perhaps 'openstack diskimage load -c <path to YAML>' | 14:39 |
dprince | thrash: as for the default YAML, I think tripleo-common would be fine. I want to see these split out into distro specific files though | 14:40 |
thrash | dprince: 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 tripleoclient | 14:40 |
thrash | dprince: I will work on it today. | 14:41 |
openstackgerrit | xiexs proposed openstack/tripleo-incubator: Replace the hard-code home directory with a tilde. https://review.openstack.org/232569 | 14:41 |
thrash | dprince: work for you? :) | 14:42 |
dprince | thrash: 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.yaml | 14:43 |
thrash | that's the idea. were you thinking something else? | 14:43 |
spredzy | dprince, 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 serv | 14:43 |
spredzy | ice ? | 14:43 |
dprince | thrash: no, I think that is much better than hard coding things into tripleoclient python, although we could improve build-images I think too | 14:44 |
thrash | dprince: i would like to see the ability to 'layer' the yamls | 14:44 |
dprince | thrash: having build-images support overlays (a mechanism) to layer would be idea | 14:44 |
dprince | thrash: exactly | 14:44 |
thrash | :) | 14:44 |
thrash | I think something very similar was done in khaleesi-settings. | 14:44 |
* dprince isn't familiar w/ khaleesi-settings | 14:45 | |
thrash | I will start with the naive port/implementation | 14:45 |
thrash | dprince: it's really just about taking multiple yamls and merging them down | 14:45 |
dprince | thrash: sure, that sounds simple enough | 14:45 |
thrash | dprince: ok. cool. | 14:45 |
shardy | dprince: Hey! Looking at that os-net-config patch | 14:46 |
openstackgerrit | xiexs proposed openstack/tripleo-incubator: Replace the hard-code home directory with a tilde. https://review.openstack.org/232569 | 14:46 |
dprince | thrash: see also this docker patch I did a long time ago now https://review.openstack.org/#/c/199830/ | 14:46 |
thrash | dprince: will do.. | 14:46 |
shardy | dprince: also, wanted to chat about the templates defaults stuff | 14:47 |
dprince | shardy: oh right, sorry I -1'd some of those | 14:47 |
shardy | dprince: my motivation is to align more closely with both the docs, and tripleo.sh (which is based on the docs) | 14:47 |
dprince | thrash: thanks, press on. Very anxious to have this :) | 14:47 |
shardy | dprince: 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 me | 14:48 |
shardy | dprince: also, I really thing having hard-coded (different) defaults in the client sucks | 14:48 |
dprince | shardy: sure, my concern is mostly where those defaults don't align w/ upstream projects (something TripleO has traditionally strived for) | 14:48 |
dprince | shardy: lioadm isn't the cinder iscsi_helper default for example | 14:48 |
shardy | dprince: yeah, I was wondering why we choose lioadm in the client then | 14:49 |
dprince | shardy: also, I consider tripleoclient to be an extremely prescriptive tool... almost unusable for some people (like me) who often don't use those defaults | 14:49 |
shardy | dprince: the back-story to all this, is I've started deploying directly via heatclient | 14:49 |
*** adrianopetrich has quit IRC | 14:49 | |
dprince | shardy: why not just a simple env file to set the tripleclient defaults for a distro (or product) | 14:49 |
shardy | it actually works really nicely, but I need a huge environment file, which is inconsistent with the templates | 14:49 |
dprince | shardy: and those can live whereever | 14:49 |
thrash | shardy: https://review.openstack.org/#/c/226790/ | 14:49 |
shardy | dprince: sure, but I still think it makes more sense for the template defaults to align with our docs | 14:50 |
dprince | shardy: I deploy directly w/ heatclient sometimes too, I just have a custom.yaml where I set my defaults | 14:50 |
*** mkovacik has quit IRC | 14:50 | |
shardy | dprince: If we had a "centos_defaults.yaml" in t-h-t, that would also be fine | 14:50 |
dprince | shardy: sure, I think some of your defaults probably go into there. lioadm could for example | 14:51 |
shardy | but then, we should remove all the conflicting defaults in the overcloud-without-mergepy template, because they're not helpful | 14:51 |
dprince | shardy: but defaults for things like the images and flavors don't make sense to me I think | 14:51 |
thrash | shardy: dprince that review is to remove all of those defaults... | 14:51 |
dprince | thrash: ++ | 14:51 |
thrash | except for the images and flavors. | 14:51 |
shardy | dprince: but our docs define images, and flavors, which don't align *at all* with what's in the templates? | 14:52 |
slagle | ugh...we moved templates under os-apply-config/ ? | 14:52 |
slagle | that is horrible | 14:52 |
shardy | I know it's historical, but how does that make sense? | 14:52 |
slagle | considering puppet heavily relies on "os-apply-config" | 14:52 |
dprince | shardy: right, and what about other implementations like docker? | 14:52 |
shardy | dprince: that's fine, then we have an environments/docker.yaml | 14:52 |
dprince | slagle: what would you call it? we can move it again to wherever? | 14:53 |
shardy | dprince: but the template defaults, if we have any, should at least work with *a* documented implementation | 14:53 |
*** penick has quit IRC | 14:53 | |
slagle | dprince: honestly, i'd delete. or at least deprecate | 14:53 |
shardy | and currently, we only have one which is landed, CI'd and actually works | 14:53 |
dprince | shardy: they do, for flavors we just require that you set them | 14:53 |
slagle | dprince: if we want a name, how about bash-elements or bash | 14:53 |
slagle | dprince: that's the actual analogy to "puppet" | 14:53 |
shardy | dprince: but we know that every developer and downstream user, if they follow the docs, or use tripleo.sh will have "baremetal" | 14:53 |
dprince | slagle: I'm not sure it really matters, and I do think they are headed towards deprecation | 14:54 |
shardy | why 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 |
dprince | slagle: my goal was to leave them in place until we have at least one other implementation in t-h-t. So docker... | 14:54 |
slagle | well, it matters if the point of the change was to add clarity | 14:54 |
dprince | slagle: once we have that I think lets deprecate | 14:54 |
shardy | dprince: for folks such as yourself your parameters file still works exactly the same | 14:54 |
shardy | it's just lowering the bar (slightly) for new folks | 14:54 |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package https://review.openstack.org/228157 | 14:54 |
dprince | slagle: name for this is perhaps a bike shed man... | 14:54 |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment https://review.openstack.org/228156 | 14:54 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Use tripleo.sh to build packages https://review.openstack.org/229425 | 14:55 |
dprince | shardy: I don't use baremetal :) | 14:55 |
shardy | dprince: e.g I had to cut/paste the parameters from a --debug from tripleoclient before I could get deployments via heat to work | 14:55 |
dprince | shardy: I have 2 flavors FWIW | 14:55 |
slagle | dprince: i'd wager it's not...but given you picked the name, i can see how you feel it might be | 14:55 |
shardy | dprince: *You* don't, and that's fine, but 99% of other folks are | 14:55 |
slagle | just the fact that we're saying os-apply-config templates are headed towards deprecation ought to set off alarm bells | 14:56 |
shardy | reverse engineering from tripleoclient isn't a good experience, even for a developer | 14:56 |
slagle | os-apply-config aint going anywhere, anytime soon | 14:56 |
dprince | shardy: 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 default | 14:56 |
shardy | dprince: following our documentation results in exactly one image "openstack-full" | 14:56 |
shardy | why wouldn't we use that as a default? | 14:57 |
dprince | shardy: I agree, that is why I think a simple distro defaults heat environment is the best approach for these things | 14:57 |
dprince | shardy: those docs are still very new, and don't (yet) support docker | 14:57 |
shardy | dprince: that's fine, and setting these defaults has zero imact on docker | 14:57 |
shardy | dprince: if we're saying the defaults can't default to one implementation and actually work, we may as well remove all of them IMO | 14:58 |
shardy | at least then it's less confusing | 14:58 |
dprince | slagle: sorry you don't like the name. I couldn't think of anything better... was just trying to clean house a bit | 14:58 |
dprince | slagle: because new users typically find looking at t-h-t to be really confusing | 14:59 |
dprince | shardy: For things like flavors, and images I'm not sure I like defaults... because nova doesn't have them | 14:59 |
dprince | shardy: those are just required settings | 14:59 |
shardy | dprince: Ok, we should remove all the defaults from the templates then | 15:00 |
*** untriaged-bot has joined #tripleo | 15:00 | |
untriaged-bot | Untriaged bugs so far: | 15:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 15:00 |
openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 15:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 15:00 |
openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 15:00 |
*** untriaged-bot has quit IRC | 15:00 | |
shardy | and just maintain a defaults environment which is consumed by all CLI/UI tools | 15:00 |
dprince | shardy: I think that is taking it too far | 15:00 |
*** spzala has quit IRC | 15:00 | |
shardy | dprince: the whole point of a "default" is that it works for the most common case | 15:00 |
shardy | which, in this case, is the only case we actually document, CI and support atm | 15:01 |
*** tiswanso has quit IRC | 15:01 | |
shardy | In this case, the defaults work for exactly 0% of cases, because we do things like refer to legacy image names from old docs/scripts | 15:01 |
*** tiswanso has joined #tripleo | 15:02 | |
dprince | shardy: 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 |
shardy | dprince: 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 |
dprince | shardy: I would argue there isn't a good virt default. I happen to like 'kvm' ... but in fact it will fail for most devs | 15:03 |
dprince | shardy: right, I think a simple heat environment to set those for tripleo-docs makes the most sense here | 15:03 |
shardy | dprince: Ok, maybe that's a bad setting - interestingly following the docs gets you kvm, even on VMs | 15:03 |
dprince | shardy: yeah, well perhaps most people use nested virt? | 15:04 |
dprince | shardy: anyways, just an example | 15:04 |
shardy | dprince: Ok, let me see if I can rebase things to get the ones we actually agree on landed | 15:04 |
dprince | shardy: I'm not -2 on these FWIW.... just a bit unconvinced that there *is* a good default for some of these things | 15:04 |
shardy | then we can argue over the rest another day ;) | 15:04 |
dprince | shardy: half of them are a pretty easy +2 from me :) | 15:05 |
shardy | dprince: Yeah, I get why you say that, but I'm trying to improve the experience for new folks who just follow the docs | 15:05 |
openstackgerrit | Thomas Herve proposed openstack/instack-undercloud: Don't use dnsmasq lease file to retrieve address https://review.openstack.org/232584 | 15:05 |
*** spzala has joined #tripleo | 15:05 | |
therve | Yay first patch | 15:05 |
dprince | shardy: ++ | 15:05 |
shardy | more advanced folks can still set whatever they want, and they know how to do it :) | 15:05 |
shardy | therve: Welcome! :) | 15:05 |
therve | shardy, Thanks :) | 15:06 |
*** aufi has quit IRC | 15:09 | |
*** masterbound has quit IRC | 15:12 | |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-docs: Updates to network documentation for new features https://review.openstack.org/221908 | 15:14 |
kukacz | hi all, second day I'm stucked running introspection with an strange error causing "dracut emergency" | 15:16 |
kukacz | may I ask for hints someone here? | 15:16 |
*** mbound has joined #tripleo | 15:20 | |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package https://review.openstack.org/228157 | 15:22 |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment https://review.openstack.org/228156 | 15:22 |
*** david-ly_ has joined #tripleo | 15:22 | |
*** david-lyle has quit IRC | 15:23 | |
openstackgerrit | Merged openstack/instack-undercloud: Run db-sync as ironic-inspector user https://review.openstack.org/231656 | 15:24 |
*** david-ly_ is now known as david-lyle | 15:24 | |
*** mbound has quit IRC | 15:24 | |
*** alop has joined #tripleo | 15:26 | |
*** mbound has joined #tripleo | 15:26 | |
*** mcornea has quit IRC | 15:30 | |
*** egafford has left #tripleo | 15:35 | |
shardy | dprince: Do you think we can land these please? | 15:41 |
shardy | https://review.openstack.org/#/q/status:open+project:openstack/tripleo-heat-templates+branch:master+topic:node_extraconfig,n,z | 15:41 |
shardy | On aggregate all the tests are passing between the two patches in the series | 15:41 |
openstackgerrit | Thomas Herve proposed openstack/instack-undercloud: Don't use dnsmasq lease file to retrieve address https://review.openstack.org/232584 | 15:43 |
openstackgerrit | Merged openstack/os-net-config: Support multiple addresses assignment with ifcfg https://review.openstack.org/213902 | 15:49 |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to locally build delorean package https://review.openstack.org/228157 | 15:54 |
openstackgerrit | Derek Higgins proposed openstack/tripleo-common: tripleo.sh Add option to create a local Delorean environment https://review.openstack.org/228156 | 15:54 |
*** egafford has joined #tripleo | 15:59 | |
*** paramite has quit IRC | 16:02 | |
*** yog_ has quit IRC | 16:02 | |
*** ukalifon1 has quit IRC | 16:03 | |
openstackgerrit | Ryan Brady proposed openstack/instack-undercloud: Adds Swift Versioning Support https://review.openstack.org/232634 | 16:10 |
*** mudassirlatif has joined #tripleo | 16:12 | |
openstackgerrit | Ryan Brady proposed openstack/instack-undercloud: Adds Swift Versioning Support https://review.openstack.org/232634 | 16:16 |
*** BharatK has quit IRC | 16:16 | |
openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: Collect the common logic in tripleo-common https://review.openstack.org/228991 | 16:16 |
*** CheKoLyN has joined #tripleo | 16:17 | |
*** ifarkas has quit IRC | 16:18 | |
egafford | Hi 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 #tripleo | 16:20 | |
*** olaph has quit IRC | 16:21 | |
*** regebro has joined #tripleo | 16:23 | |
trown | egafford: it should get built with `openstack overcloud image build` but probably doesnt due to docs | 16:23 |
egafford | trown: Ah, so it's just not part of --all right now? | 16:24 |
trown | https://review.openstack.org/#/c/232147/ | 16:24 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Use same host for cinder volume active/passive https://review.openstack.org/225320 | 16:24 |
egafford | trown: Ran `openstack overcloud image build --all 2>&1 | tee image-build.log` | 16:24 |
trown | egafford: it is part of --all, but without the DIB_YUM_REPO_CONF in that doc patch the package can be found for it | 16:24 |
trown | and the image build silently fails to build it | 16:25 |
egafford | trown: Nice. | 16:25 |
trown | ya | 16:25 |
egafford | Thanks; 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 build | 16:25 |
*** w__ has joined #tripleo | 16:27 | |
*** w_ has quit IRC | 16:31 | |
openstackgerrit | Merged openstack/diskimage-builder: Avoid transcending /proc with find https://review.openstack.org/230448 | 16:32 |
*** mbound has quit IRC | 16:37 | |
*** jistr has quit IRC | 16:38 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Update README.rst https://review.openstack.org/230675 | 16:39 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Update README.rst https://review.openstack.org/230675 | 16:41 |
*** Marga_ has quit IRC | 16:46 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: pacemaker: Set manage_service/enabled via Hiera https://review.openstack.org/221881 | 16:51 |
*** regebro has quit IRC | 16:51 | |
*** jcoufal has quit IRC | 16:54 | |
*** derekh has quit IRC | 16:55 | |
*** sthillma has joined #tripleo | 17:02 | |
*** kukacz has quit IRC | 17:02 | |
*** jaosorior has quit IRC | 17:04 | |
*** trown is now known as trown|lunch | 17:04 | |
*** jaosorior has joined #tripleo | 17:04 | |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Update README.rst https://review.openstack.org/230675 | 17:05 |
*** sthillma_ has joined #tripleo | 17:05 | |
*** sthillma has quit IRC | 17:07 | |
*** sthillma_ is now known as sthillma | 17:07 | |
*** jaosorior has quit IRC | 17:08 | |
*** mudassirlatif has quit IRC | 17:09 | |
*** w__ is now known as olaph | 17:11 | |
*** penick has joined #tripleo | 17:12 | |
*** dtantsur is now known as dtantsur|afk | 17:14 | |
*** lucasagomes has quit IRC | 17:19 | |
*** Marga_ has joined #tripleo | 17:20 | |
*** shivrao has joined #tripleo | 17:23 | |
*** yamahata has quit IRC | 17:23 | |
*** yamahata has joined #tripleo | 17:24 | |
*** shadower has quit IRC | 17:32 | |
*** lazy_prince has quit IRC | 17:33 | |
*** athomas has quit IRC | 17:37 | |
*** athomas has joined #tripleo | 17:37 | |
*** athomas has quit IRC | 17:45 | |
*** trown|lunch is now known as trown | 17:48 | |
jpeeler | dprince: just fyi, the instack kernel panic occurred again. so i'm disabling docker on the host and we'll see what happens then | 17:50 |
dprince | jpeeler: sounds good | 17:57 |
*** kukacz has joined #tripleo | 17:57 | |
dprince | jpeeler: maybe even reboot to make sure all dm_thin enablement is gone | 17:57 |
jpeeler | k | 17:58 |
dprince | jpeeler: I can't remember if I just disabled docker, or if I disabled and rebooted | 17:58 |
*** kukacz has quit IRC | 18:00 | |
slagle | thrash: looks like this is needed for tripleoclient: https://review.openstack.org/232694 | 18:00 |
slagle | we got called out on the ML :) | 18:00 |
*** jprovazn has quit IRC | 18:01 | |
*** kukacz has joined #tripleo | 18:02 | |
thrash | slagle: ack | 18:08 |
thrash | slagle: I had already handles that. | 18:08 |
thrash | *handled | 18:08 |
thrash | :) | 18:08 |
thrash | https://review.openstack.org/#/c/232589/ | 18:08 |
thrash | although your commit message looks better. :) | 18:09 |
thrash | slagle: I will abandon in favor of yours. | 18:09 |
thrash | slagle: did that break for *everybody*? | 18:10 |
thrash | if so, that seems bad on their side. | 18:10 |
openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Update README.rst https://review.openstack.org/230675 | 18:12 |
slagle | thrash: i think it's just a verification job they run | 18:15 |
thrash | slagle: ok. good. :) | 18:16 |
*** tosky has quit IRC | 18:16 | |
*** lblanchard has quit IRC | 18:16 | |
openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Nothing to see here https://review.openstack.org/232700 | 18:22 |
slagle | rhallisey: i'm trying out the docker patch, and getting: http://fpaste.org/276568/44328569/ from Heat after a while | 18:23 |
slagle | rhallisey: the controller finished deploying fine. have you ever seen this before? | 18:23 |
slagle | it's not obvious to me why Heat would error like this. i had no stacks present prior to starting the deployment | 18:24 |
*** lblanchard has joined #tripleo | 18:25 | |
*** devvesa has quit IRC | 18:28 | |
*** pelix has quit IRC | 18:28 | |
*** gchamoul has quit IRC | 18:29 | |
*** spzala has quit IRC | 18:29 | |
dprince | slagle: ive seen that once before I think. I don't think it is related to docker though | 18:30 |
dprince | slagle: would be nice to have debug logs on it... | 18:30 |
slagle | well, i have debug logs for heat, etc | 18:33 |
rhallisey | I wonder if one of the containers just died or something | 18:35 |
slagle | i think this is on the heat side | 18:35 |
rhallisey | weird | 18:35 |
jpeeler | dprince: 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 doing | 18:35 |
slagle | i 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 processes | 18:35 |
rhallisey | jpeeler, oh really hmm | 18:35 |
dprince | jpeeler: oh, interesting | 18:35 |
dprince | jpeeler: perhaps it is just a matter of time for me to hit it again too | 18:36 |
dprince | jpeeler: can you still login to the VM? | 18:36 |
jpeeler | not this time | 18:36 |
slagle | did these kernel panics start after enabling nested kvm? | 18:37 |
*** penick has quit IRC | 18:37 | |
dprince | jpeeler: 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 |
jpeeler | slagle: although i believe my setup does have nested vm enabled, no nested vms are in use | 18:38 |
jpeeler | (haven't been deploying with --libvirt-type=qemu) | 18:38 |
dprince | jpeeler: perhaps we need to understand whats sorts of IO operations activate this | 18:38 |
dprince | jpeeler: sure, but if it is something within the VM that we think is causing it (ironic) I don't think nested virt matters | 18:39 |
dprince | jpeeler: could be worth eliminating that as a concern though | 18:39 |
jpeeler | i'll turn it off then | 18:40 |
trown | so ... I think we need a to reboot the undercloud in CI | 18:40 |
rhallisey | slagle, 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 jpeeler | 18:40 |
rhallisey | there is an old kernel *we think* lying around | 18:40 |
trown | rebooting the undercloud breaks OVS in my virt env | 18:41 |
dprince | trown: I think a reboot might be a bit expensive | 18:42 |
dprince | trown: timewise | 18:42 |
dprince | trown: probably good to track down why it isn't working though | 18:42 |
trown | dprince: ya, but it is a pretty bad result to have rebooting the undercloud break | 18:43 |
trown | dprince: but it maybe that is more of a downstream test | 18:43 |
dprince | trown: yeah, infra has been pinging us about how long our jobs take | 18:43 |
dprince | trown: especially the HA job | 18:43 |
rhallisey | slagle, let me know if that happens again. I haven't hit that issue before | 18:44 |
dprince | trown: perhaps when we get our walltime down we can entertain adding some coverage for undercloud, overcloud, and perhaps even tenant reboots | 18:44 |
trown | dprince: makes sense | 18:44 |
slagle | dprince: it looks to me like 2 heat engines collided trying to create the same nested stack | 18:45 |
slagle | https://bugs.launchpad.net/tripleo/+bug/1504266 | 18:45 |
openstack | Launchpad bug 1504266 in tripleo "Stack overcloud-ComputeNodesPostDeployment-dkq4fgg4lqk5-NovaComputeContainersDeployment-6zpj2rijh3lk already has an action (CREATE) in progress." [Medium,Triaged] | 18:45 |
dprince | trown: the inability to reboot an undercloud was the initial use case that drove the creation of os-net-config FWIW | 18:45 |
dprince | slagle: hmmm, interesting | 18:46 |
dprince | slagle: add heat to that bug too? | 18:46 |
slagle | i did | 18:47 |
*** gchamoul has joined #tripleo | 18:51 | |
*** gchamoul has quit IRC | 18:51 | |
jpeeler | is there a way to force delete an ironic node since delete doesn't work with provision state deploy failed? | 18:52 |
*** MasterPiece has joined #tripleo | 18:52 | |
*** spzala has joined #tripleo | 18:53 | |
*** gchamoul has joined #tripleo | 18:54 | |
*** MasterPiece has quit IRC | 18:55 | |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: [WIP]Initial workflow implementation of tripleo-common https://review.openstack.org/227297 | 18:55 |
dprince | jpeeler: this is the sledge hammer approach http://paste.openstack.org/show/475778/ | 18:58 |
dprince | jpeeler: I hit this the other day... and I think ironic has some state transition issues | 18:59 |
dprince | jpeeler: it should always be possible to recover using only the CLI | 18:59 |
dprince | jpeeler: probably worth filing an Ironic bug on this... | 18:59 |
slagle | or just look up the one i filed over a year ago :) | 18:59 |
jpeeler | er wow | 19:00 |
dprince | slagle: probably worth asking someone to have a closer look then | 19:00 |
dprince | if it is annoying enough we should fix it | 19:00 |
dprince | slagle: got a link? | 19:01 |
*** mkovacik has joined #tripleo | 19:02 | |
slagle | i did, a while back | 19:02 |
slagle | lucas looked into it, but it was all kind of blocked on the breaking lock discussions in ironic | 19:03 |
slagle | i'm not sure what the status of that is now | 19:03 |
*** penick has joined #tripleo | 19:05 | |
*** rasca has quit IRC | 19:07 | |
*** gfidente has quit IRC | 19:10 | |
slagle | dprince: jpeeler : https://bugs.launchpad.net/ironic/+bug/1461937 | 19:10 |
openstack | Launchpad bug 1461937 in Ironic "Node stuck on DEPLOYING (potentially all *ING) state(s)" [High,Fix released] - Assigned to Lucas Alvares Gomes (lucasagomes) | 19:10 |
slagle | if it's in deploying, supposedly there is a timeout | 19:11 |
dprince | slagle: I think I may be talking about a slightly different case | 19:14 |
*** sthillma has quit IRC | 19:14 | |
dprince | slagle: which is sort of that I'd like the equivalent of a 'nova reset-state' command for ironic that actually works... | 19:14 |
dprince | slagle: to reset the power status to 'available' | 19:14 |
dprince | jpeeler: it might be worth capturing exactly what you did to 'hang' the instance in a bug? | 19:15 |
jpeeler | dprince: this was due to kernel panicking during a deploy | 19:16 |
jpeeler | and then rebooting obviously | 19:16 |
jpeeler | is that worth describing? | 19:17 |
rhallisey | dprince, got the kernel panic | 19:17 |
dprince | jpeeler: perhaps, only to the extend that ironic can get into a state that isn't recoverable without editing the database | 19:17 |
rhallisey | I just rapidly redeployed the overcloud and that triggered it | 19:17 |
jpeeler | dprince: i'm sure it'll happen again and then i'll collect some exact real details | 19:18 |
dprince | rhallisey: yeah, based on what jpeeler said earlier I was thinking maybe this is just local to the ironic VM | 19:19 |
dprince | rhallisey: 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 |
dprince | rhallisey: basically one of the local "volume" operations that ironic does is causing this kernel bug to manifest itself | 19:21 |
dprince | rhallisey: perhaps? | 19:21 |
dprince | rhallisey: could we try tailing the ironic conductor log, watching the commands it runs | 19:26 |
rhallisey | dprince, possibly. I'm definetly looking at ironic for this | 19:26 |
dprince | rhallisey: and continually deploying just the fedora-atomic image to a VM | 19:26 |
dprince | rhallisey: I think that would narrow it down quite a bit... | 19:26 |
rhallisey | kk | 19:26 |
dprince | rhallisey: I'm trying to look at this too BTW a bit more closely | 19:29 |
rhallisey | ok cool | 19:29 |
slagle | rhallisey: so even though the compute node is fedora, i can use my centos7 image for the controller? | 19:34 |
slagle | i dont see why not, just figured i'd confirm :) | 19:34 |
rhallisey | slagle, ya that should be fine | 19:35 |
dprince | rhallisey, jpeeler: could we do the upcoming "config" rework for docker in a follow up patch | 19:36 |
rhallisey | ya that's the plan | 19:36 |
dprince | great, thanks | 19:36 |
jpeeler | ack | 19:36 |
rhallisey | I'd rather get the docker bits merged now and then move to stable liberty soon after | 19:36 |
*** dprince has quit IRC | 19:38 | |
*** rbrady is now known as rbrady-run | 19:41 | |
*** dsneddon is now known as dsneddon_lunch | 19:50 | |
*** lblanchard has quit IRC | 19:55 | |
*** dprince has joined #tripleo | 19:57 | |
*** lblanchard has joined #tripleo | 20:04 | |
egafford | EmilienM: ping (re: Trove). | 20:08 |
egafford | May be too late; if it is please do disregard. | 20:08 |
*** rlandy has quit IRC | 20:11 | |
slagle | he's on the west coast at puppetconf and drinking beer | 20:11 |
egafford | slagle: Ah, cool. Not too late in general, then, just too late right now. :) | 20:11 |
*** yamahata has quit IRC | 20:12 | |
egafford | slagle: 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 |
slagle | egafford: not that i know of | 20:14 |
egafford | slagle: Ack. Any sense of when EmilienM might be back about? | 20:14 |
slagle | egafford: no...next week probably | 20:16 |
egafford | slagle: Ok, next week isn't so far away. Thanks. | 20:17 |
*** sdake has quit IRC | 20:17 | |
*** sdake has joined #tripleo | 20:23 | |
*** tiswanso has quit IRC | 20:30 | |
*** yamahata has joined #tripleo | 20:30 | |
*** penick has quit IRC | 20:31 | |
*** dprince has quit IRC | 20:34 | |
*** jayg is now known as jayg|g0n3 | 20:37 | |
*** penick has joined #tripleo | 20:38 | |
*** yamahata has quit IRC | 20:39 | |
*** yamahata has joined #tripleo | 20:47 | |
EmilienM | egafford: hey, sorry I was afk at puppetconf | 20:47 |
EmilienM | slagle: not yet lol | 20:47 |
EmilienM | egafford: there is not upstream card capturing the items, but we can create it on https://trello.com/b/4X3zxWRZ/on-going-effort | 20:48 |
EmilienM | I would be happy to help | 20:48 |
*** dsneddon_lunch is now known as dsneddon | 20:48 | |
egafford | EmilienM: 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 |
EmilienM | that sucks | 20:56 |
egafford | I'm working on a patch now to fix it, actually; not a puppet expert, but that much is in my power. :) | 20:56 |
EmilienM | let me see though, I saw some patches like this | 20:56 |
egafford | Ah, cool. | 20:56 |
*** lblanchard has quit IRC | 20:56 | |
egafford | I didn't see anything in active review in puppet-trove. | 20:56 |
EmilienM | egafford: wait, we already have oslo_messaging_rabbit stuff | 20:57 |
egafford | It'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 |
egafford | Yeah, you do, but trove doesn't, it seems. | 20:58 |
*** MasterPiece has joined #tripleo | 20:58 | |
EmilienM | https://github.com/openstack/puppet-trove/blob/master/manifests/api.pp#L344-L350 | 20:58 |
EmilienM | egafford: ^ | 20:58 |
egafford | EmilienM: Yes, I know (I'm pulling overcloud modules from head of that repo.) | 20:59 |
EmilienM | so what is missing? | 20:59 |
egafford | Trove itself has those keys under DEFAULT/rabbit_userid, etc. in our current packages. | 20:59 |
egafford | Not oslo_messaging_rabbit. | 20:59 |
EmilienM | oh so you need to ask for an OPM update | 20:59 |
*** rbrady-run is now known as rbrady | 20:59 | |
egafford | What's that three-letter acronym mean? | 21:00 |
EmilienM | you should ping imck8 (not on this channel, but on #rdo) | 21:00 |
EmilienM | openstack puppet modules | 21:00 |
EmilienM | it's an rpm that contains our modules | 21:00 |
*** untriaged-bot has joined #tripleo | 21:00 | |
untriaged-bot | Untriaged bugs so far: | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504179 | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1503669 | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504137 | 21:00 |
egafford | Ah, yes, got it. | 21:00 |
untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1504263 | 21:00 |
openstack | Launchpad bug 1504179 in tripleo "instack-virt-setup can't use dnsmasq lease file anymore" [Undecided,In progress] - Assigned to Thomas Herve (therve) | 21:00 |
openstack | Launchpad bug 1503669 in tripleo "Including an additional element is inconvenient" [Undecided,New] - Assigned to Jaume Devesa (devvesa) | 21:00 |
*** untriaged-bot has quit IRC | 21:00 | |
openstack | Launchpad bug 1504137 in tripleo "Parameter from tripleoclient not passed to Heat template" [Undecided,New] | 21:00 |
openstack | Launchpad bug 1504263 in tripleo "os-net-config mapping by MAC fails if trying to map in alternate order" [Undecided,Triaged] | 21:00 |
egafford | I 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 |
openstackgerrit | Ryan Brady proposed openstack/tripleo-common: [WIP]Initial workflow implementation of tripleo-common https://review.openstack.org/227297 | 21:01 |
*** spzala has quit IRC | 21:02 | |
egafford | The 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|outttypeww | 21:05 | |
openstackgerrit | Tzu-Mainn Chen proposed openstack/tripleo-specs: Spec for TripleO overcloud deployment REST API https://review.openstack.org/230432 | 21:06 |
*** julim has quit IRC | 21:11 | |
*** rhallisey has quit IRC | 21:15 | |
*** sdake has quit IRC | 21:28 | |
*** spzala has joined #tripleo | 21:30 | |
*** MasterPiece has quit IRC | 21:31 | |
*** Goneri has quit IRC | 21:37 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Drop fixtures https://review.openstack.org/232789 | 21:39 |
*** alop has quit IRC | 21:43 | |
*** weshay has quit IRC | 21:47 | |
*** ibravo has joined #tripleo | 21:50 | |
*** thrash is now known as thrash|g0ne | 21:55 | |
*** akrivoka has quit IRC | 21:56 | |
*** kukacz has quit IRC | 22:00 | |
*** adrianopetrich has joined #tripleo | 22:02 | |
*** sthillma has joined #tripleo | 22:03 | |
*** morazi has quit IRC | 22:06 | |
*** pradk has quit IRC | 22:17 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/232789 | 22:19 |
egafford | EmilienM: 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 |
EmilienM | egafford: ok | 22:21 |
egafford | (Unless I'm wrong again, which one can never rule out.) | 22:21 |
openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Update some discovery references to inspection https://review.openstack.org/232802 | 22:24 |
openstackgerrit | Ben Nemec proposed openstack/instack-undercloud: Update some discovery references to inspection https://review.openstack.org/232802 | 22:25 |
*** ibravo has quit IRC | 22:31 | |
*** mudassirlatif has joined #tripleo | 22:35 | |
*** egafford has quit IRC | 22:35 | |
mudassirlatif | Hi, 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 |
mudassirlatif | a few hours? | 22:36 |
*** rhallisey has joined #tripleo | 23:06 | |
*** CheKoLyN has quit IRC | 23:13 | |
*** spzala has quit IRC | 23:13 | |
*** apuimedo has quit IRC | 23:19 | |
*** apuimedo has joined #tripleo | 23:20 | |
*** sthillma_ has joined #tripleo | 23:29 | |
*** sthillma has quit IRC | 23:30 | |
*** sthillma_ is now known as sthillma | 23:30 | |
*** mbound has joined #tripleo | 23:34 | |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme https://review.openstack.org/222351 | 23:51 |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update apt-preferences element README from free text to table formatting https://review.openstack.org/232211 | 23:52 |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update apt-preferences element README from free text to table formatting https://review.openstack.org/232211 | 23:55 |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme https://review.openstack.org/222351 | 23:56 |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update apt-conf elements README from free text to table formatting https://review.openstack.org/231770 | 23:57 |
openstackgerrit | Augustina Ragwitz proposed openstack/diskimage-builder: Update rhel7 element readme https://review.openstack.org/222351 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!