| *** thrash is now known as thrash|g0ne | 00:36 | |
| *** chlong has quit IRC | 00:52 | |
| *** chlong has joined #tripleo | 00:54 | |
| openstackgerrit | Merged openstack/os-cloud-config: Don't register https endpoints for non-ssl services https://review.openstack.org/176900 | 01:10 |
|---|---|---|
| openstackgerrit | Merged openstack/os-cloud-config: Enable ssl registration for Heat https://review.openstack.org/176483 | 01:11 |
| *** alop has quit IRC | 01:37 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: puppet controller role: per service VIP settings https://review.openstack.org/187219 | 01:41 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set glance_api_servers for cinder https://review.openstack.org/187220 | 01:42 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: configure pacemaker VIPs for isolated networks https://review.openstack.org/187221 | 01:42 |
| *** al has quit IRC | 02:02 | |
| *** al has joined #tripleo | 02:03 | |
| *** yamahata has quit IRC | 02:04 | |
| *** radek_ has joined #tripleo | 02:05 | |
| *** radek_ has quit IRC | 02:17 | |
| *** daneyon_ has quit IRC | 02:29 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Add support for isolating swift storage nets https://review.openstack.org/188655 | 02:52 |
| *** untriaged-bot has joined #tripleo | 03:00 | |
| untriaged-bot | Untriaged bugs so far: | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461615 | 03:00 |
| openstack | Launchpad bug 1461615 in diskimage-builder "Yum vars directory doesn't exist" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461631 | 03:00 |
| openstack | Launchpad bug 1461631 in diskimage-builder "New Fedora 22 cloud images fail to build" [Undecided,New] | 03:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1459465 | 03:00 |
| openstack | Launchpad bug 1459465 in tripleo "devtest fails to build ramdisk on Fedora or CentOS due to Yum multilib error" [Undecided,Triaged] | 03:01 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1461984 | 03:01 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461601 | 03:01 |
| openstack | Launchpad bug 1461984 in tripleo "VM element: /usr/sbin/grub2-install: error: will not proceed with blocklists." [Undecided,New] | 03:01 |
| openstack | Launchpad bug 1461601 in diskimage-builder "CentOS 6 not resizing" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 03:01 |
| *** untriaged-bot has quit IRC | 03:01 | |
| openstackgerrit | Steve Baker proposed openstack/tripleo-heat-templates: Config & deployments to update overcloud packages https://review.openstack.org/179974 | 03:58 |
| *** masco has joined #tripleo | 04:12 | |
| *** cwolferh has quit IRC | 05:06 | |
| *** cwolferh has joined #tripleo | 05:08 | |
| *** aufi has joined #tripleo | 06:11 | |
| *** dtantsur|afk is now known as dtantsur | 06:16 | |
| *** yuanying has joined #tripleo | 06:19 | |
| *** AK has joined #tripleo | 06:21 | |
| *** ishant has joined #tripleo | 06:36 | |
| *** matbu has left #tripleo | 06:51 | |
| *** jistr has joined #tripleo | 06:57 | |
| *** cody-somerville has quit IRC | 07:09 | |
| *** cody-somerville has joined #tripleo | 07:10 | |
| *** regebro has joined #tripleo | 07:14 | |
| *** Goneri has joined #tripleo | 07:19 | |
| *** pino|work has joined #tripleo | 07:31 | |
| *** ifarkas has joined #tripleo | 07:41 | |
| *** hewbrocca has joined #tripleo | 07:46 | |
| *** jistr is now known as jistr|biab | 07:47 | |
| *** rwsu has joined #tripleo | 07:48 | |
| *** paramite has joined #tripleo | 07:48 | |
| *** dtantsur is now known as dtantsur|brb | 07:52 | |
| *** lucasagomes has joined #tripleo | 07:57 | |
| *** jprovazn has joined #tripleo | 08:04 | |
| *** gfidente has joined #tripleo | 08:08 | |
| *** gfidente has quit IRC | 08:08 | |
| *** gfidente has joined #tripleo | 08:08 | |
| *** athomas has joined #tripleo | 08:08 | |
| *** bvandenh has joined #tripleo | 08:11 | |
| *** stendulker has joined #tripleo | 08:15 | |
| *** yuanying has quit IRC | 08:17 | |
| *** sdake has quit IRC | 08:34 | |
| *** chlong has quit IRC | 08:34 | |
| *** rwsu has quit IRC | 08:36 | |
| *** pelix has joined #tripleo | 08:44 | |
| *** dsneddon has quit IRC | 08:47 | |
| *** jistr|biab is now known as jistr | 08:50 | |
| *** masco has quit IRC | 08:54 | |
| *** dasm has joined #tripleo | 08:57 | |
| dasm | o/ | 08:57 |
| *** pdb_ is now known as pdb | 08:58 | |
| jistr | o/ | 08:59 |
| * jistr reloc | 08:59 | |
| *** jistr has quit IRC | 08:59 | |
| *** untriaged-bot has joined #tripleo | 09:00 | |
| untriaged-bot | Untriaged bugs so far: | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461615 | 09:00 |
| openstack | Launchpad bug 1461615 in diskimage-builder "Yum vars directory doesn't exist" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461631 | 09:00 |
| openstack | Launchpad bug 1461631 in diskimage-builder "New Fedora 22 cloud images fail to build" [Undecided,New] | 09:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1459465 | 09:01 |
| openstack | Launchpad bug 1459465 in tripleo "devtest fails to build ramdisk on Fedora or CentOS due to Yum multilib error" [Undecided,Triaged] | 09:01 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1461984 | 09:01 |
| openstack | Launchpad bug 1461984 in tripleo "VM element: /usr/sbin/grub2-install: error: will not proceed with blocklists." [Undecided,New] | 09:01 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461601 | 09:01 |
| openstack | Launchpad bug 1461601 in diskimage-builder "CentOS 6 not resizing" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 09:01 |
| *** untriaged-bot has quit IRC | 09:01 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Adds horizon to pacemaker when puppet-pacemaker is enabled https://review.openstack.org/188526 | 09:03 |
| *** mcornea has joined #tripleo | 09:14 | |
| *** masco has joined #tripleo | 09:18 | |
| *** jistr has joined #tripleo | 09:19 | |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron VLAN ranges param as array to puppet https://review.openstack.org/187262 | 09:19 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Fix list of type_drivers for ML2 plugin https://review.openstack.org/187472 | 09:20 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron VLAN ranges param as array to puppet https://review.openstack.org/187262 | 09:27 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Isolate the Ceph storage and storage_mgmt nets https://review.openstack.org/188233 | 09:28 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Fix list of type_drivers for ML2 plugin https://review.openstack.org/187472 | 09:29 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: Wire Neutron ML2 plugin and OVS agent settings as arrays https://review.openstack.org/187506 | 09:39 |
| openstackgerrit | simen proposed openstack/os-cloud-config: fix bug1462260 https://review.openstack.org/188717 | 09:42 |
| gfidente | StevenK, regarding the ext-net type, this would work without changes to os_cloud_config https://review.openstack.org/#/c/188341/5 | 09:42 |
| openstackgerrit | simen proposed openstack/os-cloud-config: Add pxe_irmc driver support into os-cloud-config(Fix bug1462260) https://review.openstack.org/188717 | 09:47 |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates: Adds horizon to pacemaker when puppet-pacemaker is enabled https://review.openstack.org/188526 | 09:50 |
| gfidente | marios++ :) | 10:02 |
| *** aberdine has left #tripleo | 10:02 | |
| gfidente | marios, do you know if httpd is enabled by default though? | 10:04 |
| *** regebro has quit IRC | 10:06 | |
| *** regebro has joined #tripleo | 10:07 | |
| marios | gfidente: was mostly working out the config actually, the addition is very usual 2/3 lines | 10:08 |
| openstackgerrit | Jan Provaznik proposed openstack/tripleo-heat-templates: Config & deployments to update overcloud packages https://review.openstack.org/179974 | 10:11 |
| marios | gfidente: you mean horizon in general? i get green pcs status after runnin git | 10:14 |
| gfidente | marios, no the httpd service | 10:14 |
| gfidente | I am not sure if it is enabled by default or not | 10:15 |
| gfidente | so we might need to do the enable_service thing | 10:15 |
| gfidente | before pacemaker starts it | 10:15 |
| gfidente | we can let them race for now though, jistr ^^ | 10:15 |
| marios | are we using https://github.com/openstack-infra/puppet-httpd (not sure) | 10:16 |
| gfidente | I wouldn't worry for now | 10:17 |
| gfidente | let's merge this first | 10:17 |
| gfidente | and later ensure disable, as long as we track it | 10:17 |
| gfidente | I think it is not terrible to let them race initially | 10:17 |
| gfidente | btw, no I don't see puppet-httpd | 10:17 |
| marios | gfidente: yeah i wasn't sure either (the one remaining thing of config i need to check is #7 on the list on the review and i was poking at which puppet-httpd we use, if at all) | 10:19 |
| gfidente | pblaho, shameless request for +a https://review.openstack.org/#/c/184481/ ? | 10:21 |
| gfidente | jprovazn, ^^ ? :) | 10:25 |
| marios | jistr: | 10:25 |
| marios | shadower: | 10:25 |
| jprovazn | gfidente: hi, waht was the question? | 10:25 |
| gfidente | jprovazn, kisses | 10:26 |
| marios | we are pimping a review | 10:26 |
| jprovazn | heh | 10:26 |
| marios | https://review.openstack.org/#/c/184481/ | 10:26 |
| marios | this one | 10:26 |
| marios | it is a great review | 10:26 |
| marios | LOOK AT IT | 10:26 |
| marios | LOOK | 10:26 |
| marios | it is wonderful | 10:26 |
| jprovazn | ah, in that case.. sec | 10:26 |
| * marios pokerface | 10:26 | |
| marios | jprovazn: for what is worth, i verified on a live env multiple times | 10:29 |
| marios | jprovazn: and we just had reports that it has been seen in the wild now (to do with template length) ci i believe? hence wanting to land it | 10:29 |
| jprovazn | marios: you were right, the patch is epic, I feel much better today after reading ti! | 10:30 |
| marios | SEE | 10:30 |
| marios | I TOLD YOU | 10:30 |
| marios | jprovazn: thanks very much for your time man i'm sure i'll be paying for it later | 10:30 |
| jprovazn | marios: oh yea, you will, you will ;) (just kiddin) | 10:31 |
| gfidente | jprovazn, I read the 'scale from 0' bug report | 10:31 |
| gfidente | I am scared | 10:31 |
| gfidente | ideally we could pass the ceph params if count goes > 0 | 10:32 |
| jprovazn | gfidente: ah, I noticed soem notification mail, but forgot to get back to it, let me look at it | 10:32 |
| gfidente | jprovazn, for cinder, if we keep the lvm backend enabled, it might even work by adding the new ceph backend! | 10:32 |
| gfidente | but for glance/nova we'd be screwed | 10:32 |
| gfidente | we can't switch their backends | 10:33 |
| jprovazn | yes, right | 10:33 |
| gfidente | BUT BUT BUT | 10:33 |
| jprovazn | gfidente: so, if we want to support scaling from 0 | 10:33 |
| gfidente | it's not terrible, we have booleans which enable/disable glance/nova optionally | 10:33 |
| jprovazn | ewe could updated puppet modules | 10:33 |
| jprovazn | to check if nova/cinder was already set w/o ceph | 10:33 |
| gfidente | so we might put some logic, on update from 0, yeah to not change that | 10:33 |
| gfidente | because cinder has multibackend | 10:33 |
| jprovazn | right | 10:34 |
| jprovazn | so I guess it's possible, but out of scope for now | 10:34 |
| gfidente | so for cinder it'd be safe, for glance/nova it won't | 10:34 |
| gfidente | yeah | 10:34 |
| jprovazn | IO, doc text: "don't do it" would be a solution :) | 10:34 |
| jprovazn | *IOW | 10:34 |
| gfidente | sure for now seems best thing to me as well | 10:34 |
| openstackgerrit | Merged openstack/tuskar: Alter the type for the contents column of stored_file https://review.openstack.org/184481 | 10:35 |
| hewbrocca | woowoo | 10:43 |
| hewbrocca | marios: xlnt | 10:43 |
| marios | if you get this excited about all the things you're seriously going to be exhausted by the time we ship it hewbrocca i mean pace yrself man srsly | 10:47 |
| hewbrocca | You think I'm excited now... | 10:47 |
| *** stendulker has quit IRC | 10:49 | |
| *** larstobi has quit IRC | 10:50 | |
| *** larstobi has joined #tripleo | 11:01 | |
| *** julim has joined #tripleo | 11:01 | |
| openstackgerrit | Merged openstack/tripleo-image-elements: Migrate os-*-config to package-installs v2 https://review.openstack.org/163187 | 11:04 |
| *** ishant has quit IRC | 11:06 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/os-cloud-config: Show error to the user when registering node https://review.openstack.org/188745 | 11:10 |
| *** rhallisey has joined #tripleo | 11:11 | |
| *** dtantsur|brb is now known as dtantsur | 11:12 | |
| openstackgerrit | Merged openstack/tripleo-incubator: Enable Heat SSL support https://review.openstack.org/176484 | 11:20 |
| *** panda has quit IRC | 11:21 | |
| *** panda has joined #tripleo | 11:21 | |
| *** radez is now known as radez_g0n3 | 11:23 | |
| jistr | gfidente: just fyi what's the cause of yesterday's hiera madness -- it's not string vs. json what makes the difference | 11:29 |
| jistr | gfidente: it's whether the thing is multiline or not | 11:29 |
| jistr | gfidente: https://github.com/openstack/tripleo-puppet-elements/blob/1b6bfa4965c8332d5ad2fdd0f3fa29a4e55b4c06/elements/hiera/os-refresh-config/configure.d/40-hiera-datafiles#L42-L46 | 11:29 |
| jistr | i think that's wrong and can result in garbled YAML (e.g. if the multiline string contained the single quotes too) | 11:30 |
| jistr | basically we misuse the fact that jq is called on line 40 as `jq -r` instead of just `jq`, which omits quotes around strings | 11:32 |
| jistr | this enables us to do the hack with passing a "[a,b,c]" string to hiera and make hiera interpret it as an array | 11:32 |
| jistr | i'd fix that but we already depend on this behavior | 11:32 |
| *** pdb has quit IRC | 11:33 | |
| gfidente | jistr, gets even more interesting | 11:33 |
| jistr | to put it more clearly -- if i remove the `-r` from jq and remove the multiline quoting hack | 11:34 |
| gfidente | so the str_replace "[ is non multiline | 11:34 |
| jistr | then datatypes from heat would be interpreted correctly | 11:34 |
| jistr | but we already depend on them being interpreted incorrectly | 11:34 |
| jistr | gfidente: re str_replace in [ -- exactly :) | 11:34 |
| jistr | gfidente: if it contained newlines, it would break | 11:34 |
| gfidente | if it did | 11:34 |
| gfidente | where is that we depend on broken behaviour? | 11:35 |
| gfidente | *currently | 11:35 |
| jistr | gfidente: e.g. all of these depend on it https://github.com/openstack/tripleo-heat-templates/blob/4ae1b72673b519797ecb8a44cbe96089366777dc/puppet/all-nodes-config.yaml#L95-L230 | 11:36 |
| gfidente | jistr, wait I might miss something | 11:36 |
| gfidente | those are single line so don't get wrapped into '' no? | 11:37 |
| jistr | gfidente: yeah exactly, and that's broken, should i be a purist :) because from heat they leave as strings, but in hiera they are interpreted as arrays, because they're not quoted | 11:37 |
| jistr | the -r parameter on jq makes us lose the data type | 11:39 |
| gfidente | jistr, well there isn't a mapping between heat/hieradata so I think its more that we don't want the multiline wrapped in '' no? | 11:39 |
| jistr | there isn't but technically there could be | 11:40 |
| jistr | jq is capable of printing all correctly so that you can just feed it to yaml, but for that to work we'd also have to correctly output the parameters from heat. strings as strings, arrays as arrays, etc | 11:40 |
| jistr | e.g. | 11:41 |
| jistr | [root@ov-dhgp5je3laa-0-mrqlmzaxis2n-controller-t7pbrg5f6n44 configure.d]# jq -a '.["neutron_public_interface"]' <<<$HIERA_DATA | 11:41 |
| jistr | "eth0" | 11:41 |
| jistr | [root@ov-dhgp5je3laa-0-mrqlmzaxis2n-controller-t7pbrg5f6n44 configure.d]# jq -r -a '.["neutron_public_interface"]' <<<$HIERA_DATA | 11:41 |
| jistr | eth0 | 11:41 |
| jistr | i'm not pasting the json config because it's big | 11:41 |
| jistr | but | 11:41 |
| jistr | the output of `jq -r` and `jq` there is the same | 11:42 |
| jistr | so if we just use `jq`, we get quoted strings out from it, and unquoted arrays/hashes | 11:42 |
| *** lucasagomes is now known as lucas-hungry | 11:42 | |
| *** rwsu has joined #tripleo | 11:43 | |
| jistr | and i think json is a subset of yaml (would need to verify that), so we could just feed `jq` output to hiera directly | 11:43 |
| jistr | but we'd need to force heat templates to output correct data type in the first place | 11:43 |
| gfidente | so, isn't just about dropping the '' when multiline? | 11:44 |
| gfidente | L43 | 11:44 |
| jistr | gfidente: no because then you're broken if you really have a multiline string that you actually need quoted :) but i think i can fix it while preserving the behavior we need | 11:45 |
| jistr | it's still going to be broken, but less broken | 11:45 |
| gfidente | jistr, well in that case, if I have a multiline and want it quoted, I manipulate it from heat | 11:46 |
| gfidente | or at least, this I was expecting to be the current behaviour | 11:46 |
| gfidente | and it actually is, except for the little logic which wraps things in '' when multiline | 11:46 |
| *** dhellmann has joined #tripleo | 11:46 | |
| jistr | re manipulating that from heat -- you can, but that doesn't make it the right thing to do imho :) | 11:46 |
| jistr | i mean | 11:47 |
| jistr | we have data in heat and we have data in hiera | 11:47 |
| jistr | they both support the main data types we care about | 11:47 |
| jistr | so why lose the information about that datatype and do hacks with quotes and whatnot (which can break if the quoted string contains a quote too, because we do improper quoting atm) | 11:48 |
| *** thrash|g0ne is now known as thrash | 11:48 | |
| jistr | when we can actually just preserve the data types coming from heat in hiera | 11:49 |
| gfidente | jistr, wait it is two different issues really to me | 11:49 |
| gfidente | 1. we probably don't want the script to mess with data when multiline | 11:49 |
| gfidente | 2. we might want to remove -r to preserve type | 11:49 |
| jistr | gfidente: if you do 2., you automatically get 1. | 11:50 |
| jistr | gfidente: but if we do 2., tripleo will break | 11:51 |
| jistr | gfidente: i'm planning to go middle ground on that | 11:51 |
| jistr | gfidente: i'm going to do 2. with the exception that if the string begins with "[", i'm going to treat it as an array and not a string | 11:52 |
| jistr | gfidente: which is i think the only 'wrong' behavior we depend on | 11:52 |
| *** cody-somerville has quit IRC | 11:53 | |
| jistr | gfidente: and hopefully over time we can remove this re-typing too, and preserve the types between heat and hiera 1:1 | 11:53 |
| jistr | yeah googling says that json is indeed a subset of yaml, so all this should be possible | 11:55 |
| gfidente | jistr, so the problem you see with https://github.com/openstack/tripleo-heat-templates/blob/4ae1b72673b519797ecb8a44cbe96089366777dc/puppet/all-nodes-config.yaml#L95-L230 | 11:55 |
| gfidente | is they should not have an outer " | 11:55 |
| *** mcornea has quit IRC | 11:55 | |
| *** mcornea has joined #tripleo | 11:56 | |
| gfidente | but that is the only thing right? | 11:56 |
| jistr | you mean here? | 11:56 |
| jistr | template: "['SERVERS_LIST']" | 11:56 |
| gfidente | yes | 11:57 |
| gfidente | jq without -r should parse that as list | 11:57 |
| jistr | if we remove the quote it might not work, because then the template will parse the param as an array, not as a string, and the str_replace function probably expects the "template: " to be a string | 11:57 |
| jistr | and i think str_replace will always return a string | 11:58 |
| jistr | so jq would always parse any output from str_replace as a string | 11:58 |
| jistr | but | 11:58 |
| gfidente | but it is a string | 11:58 |
| gfidente | I mean, to jq the metadata doesn't have any type | 11:58 |
| gfidente | except the formatting so without the outer " it should be parsed as array | 11:59 |
| gfidente | str_replace will output a string which is valid jq array | 11:59 |
| jistr | "to jq the metadata doesn't have any type" -- i don't think that's the case actually, lemme try to verify | 11:59 |
| *** zoliXXL is now known as zoli|brb | 11:59 | |
| gfidente | so I'd be inclined to remove the '' for multiline first | 12:00 |
| gfidente | because I don't expect the hook to manipulate data | 12:01 |
| gfidente | and then, to remove -r to see if we produce valid output from heat | 12:01 |
| gfidente | which we might not | 12:01 |
| *** AK has quit IRC | 12:05 | |
| jistr | gfidente: so the metadata from heat is json formatted, and the type is preserved, so to jq there are differences between data types... I wonder if comma_delimited_list gets passed to that metadata json as an array or string though. I'm not sure if we pass it directly somewhere or if we process it through str_replace always | 12:05 |
| *** dprince has joined #tripleo | 12:06 | |
| jistr | i'll try to fix it as much as possible without breaking existing functionality :) we'll see | 12:06 |
| *** cody-somerville has joined #tripleo | 12:07 | |
| *** cody-somerville has joined #tripleo | 12:07 | |
| gfidente | is 'json formatted type is preserved' ? | 12:07 |
| gfidente | how can it preserve type? | 12:08 |
| gfidente | a json param is passed into metadata unchanged and a string is always quoted? | 12:09 |
| gfidente | I am waiting for some oc to come up ... | 12:10 |
| openstackgerrit | James Slagle proposed openstack/tripleo-image-elements: Use nagios-common instead of nagios-plugins-all on RHEL. https://review.openstack.org/145038 | 12:12 |
| jistr | gfidente: "a json param is passed into metadata unchanged and a string is always quoted" -- exactly :) | 12:14 |
| *** jayg|g0n3 is now known as jayg | 12:16 | |
| gfidente | and lists? | 12:16 |
| gfidente | like the comma_serparated_list thing? | 12:16 |
| jistr | gfidente: yeah that's what i mentioned above -- i didn't come across a sample for that. Do we somewhere pass comma_separated_list directly to metadata? | 12:17 |
| gfidente | if -r saves us from doing the str_replace ... | 12:18 |
| gfidente | ... I don't think we pass it unchanged anywhere yet though | 12:18 |
| * gfidente still waiting for OC to come up | 12:18 | |
| *** zoli|brb is now known as zoli | 12:20 | |
| jistr | gfidente: i'm trying to fix it in a way that would be backwards compatible -- at first we'll still treat single line strings as raw YAML, but handling of arrays would be fixed, so then we can teach t-h-t (or maybe Heat even, if it currently can't) to output arrays properly, and then we'd just remove the special case from the hiera yaml writer | 12:30 |
| *** trown|outttypeww is now known as trown | 12:30 | |
| jistr | so it should all be smooth ;) | 12:30 |
| *** shardy_ has joined #tripleo | 12:39 | |
| *** bvandenh_ has joined #tripleo | 12:39 | |
| *** hewbrocc` has joined #tripleo | 12:39 | |
| *** athomas_ has joined #tripleo | 12:39 | |
| *** bvandenh has quit IRC | 12:39 | |
| *** shardy has quit IRC | 12:39 | |
| *** hewbrocca has quit IRC | 12:40 | |
| gfidente | jistr, I think I still don't get it | 12:40 |
| *** masco has quit IRC | 12:40 | |
| gfidente | removing the '' around multiline seems the only thing needed to me | 12:40 |
| *** pdb has joined #tripleo | 12:41 | |
| jistr | gfidente: then you cannot process multiline strings though :) | 12:41 |
| jistr | so if we had a variable which actually is a multiline string | 12:41 |
| jistr | it would break the whole yaml | 12:42 |
| jistr | the correct solution there is to preserve the types, but i'll make an exception for single-line strings for now, to keep tripleo working | 12:42 |
| jistr | and then we can fix those as we have time | 12:43 |
| *** athomas has quit IRC | 12:43 | |
| *** hewbrocc` has quit IRC | 12:43 | |
| *** bvandenh_ has quit IRC | 12:43 | |
| *** athomas_ has quit IRC | 12:43 | |
| jistr | if i can force jq to cooperate, that is :D | 12:44 |
| *** shardy_ has quit IRC | 12:44 | |
| gfidente | you cannot process multiline strings, unless heat puts them in quotes | 12:44 |
| gfidente | and this seems valid to me | 12:44 |
| *** shardy has joined #tripleo | 12:44 | |
| *** hewbrocca has joined #tripleo | 12:46 | |
| gfidente | I think os-collect-config is already doing json.dump | 12:46 |
| jistr | gfidente: heat puts all strings in quotes | 12:47 |
| jistr | regardless if single or multi line | 12:47 |
| jistr | it's just t-p-e which removes them | 12:47 |
| jistr | Heat actually needs to format the metadata as valid json | 12:48 |
| hewbrocca | who can tell me what we are doing with swift right now? | 12:50 |
| jistr | hewbrocca: dprince has a patch which adds network separation for it and fixes swift in general https://review.openstack.org/#/c/188655/ | 12:51 |
| *** radez_g0n3 is now known as radez | 12:52 | |
| *** lucas-hungry is now known as lucasagomes | 12:52 | |
| *** rlandy has joined #tripleo | 12:52 | |
| *** lblanchard has joined #tripleo | 12:52 | |
| hewbrocca | NICE | 12:55 |
| *** bvandenh_ has joined #tripleo | 12:56 | |
| *** athomas_ has joined #tripleo | 12:57 | |
| gfidente | jistr, so you mean tpe removes the quotes because of -r, correct? | 12:59 |
| jistr | gfidente: yeah | 12:59 |
| jistr | gfidente: w00t i think it's fixed | 13:01 |
| jistr | gfidente: here's a diff of controller.yaml before and after http://paste.fedoraproject.org/229222/09237143/ | 13:01 |
| jistr | * single-line strings will continue to be treated as raw YAML for now | 13:01 |
| jistr | * multi-line strings will get escaped correctly and will not break yaml if they contain single quotes | 13:02 |
| jistr | * arrays and jsons will be treated as arrays and jsons | 13:02 |
| * jistr prepares a patch | 13:02 | |
| *** mestery is now known as mestery_afk | 13:16 | |
| *** bvandenh_ has quit IRC | 13:20 | |
| *** spredzy is now known as spredzy|afk | 13:25 | |
| *** hewbrocc` has joined #tripleo | 13:35 | |
| *** shardy_ has joined #tripleo | 13:36 | |
| *** athomas__ has joined #tripleo | 13:36 | |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-puppet-elements: Preserve data types in Hiera where possible https://review.openstack.org/188772 | 13:36 |
| *** shardy has quit IRC | 13:36 | |
| larsks | That seems like a good situation for using YAML syntax for multiline literal text, rather than quoting... | 13:37 |
| *** morazi has joined #tripleo | 13:37 | |
| *** hewbrocca has quit IRC | 13:37 | |
| *** morazi is now known as morazi-mtg | 13:37 | |
| *** athomas_ has quit IRC | 13:39 | |
| *** athomas__ has quit IRC | 13:41 | |
| *** shardy_ has quit IRC | 13:41 | |
| *** hewbrocc` has quit IRC | 13:41 | |
| *** shardy has joined #tripleo | 13:42 | |
| jistr | +A anyone? :) https://review.openstack.org/#/c/185597/ | 13:49 |
| *** athomas__ has joined #tripleo | 13:53 | |
| *** aufi has quit IRC | 14:02 | |
| jistr | paramite: any luck getting devtest to deploy so far? | 14:02 |
| jistr | need help? | 14:02 |
| *** spredzy|afk is now known as spredzy | 14:03 | |
| *** yamahata has joined #tripleo | 14:03 | |
| paramite | jistr, currently deploy is running | 14:05 |
| jistr | cool | 14:05 |
| paramite | jistr, no failure so far | 14:05 |
| dprince | bnemec, gfidente: I think we are good here https://review.openstack.org/#/c/186775/ | 14:08 |
| openstackgerrit | Ryan Hefner proposed openstack/tripleo-heat-templates: Enable NetApp Backends in Cinder https://review.openstack.org/187700 | 14:11 |
| openstackgerrit | Ryan Hefner proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 14:11 |
| *** shardy_ has joined #tripleo | 14:11 | |
| *** athomas_ has joined #tripleo | 14:11 | |
| *** shardy has quit IRC | 14:12 | |
| openstackgerrit | Ryan Hefner proposed openstack/tripleo-heat-templates: Enable Manila integration https://review.openstack.org/188137 | 14:13 |
| openstackgerrit | Ryan Hefner proposed openstack/tripleo-heat-templates: Add NetApp integration to Manila https://review.openstack.org/188138 | 14:13 |
| *** athomas__ has quit IRC | 14:14 | |
| gfidente | dprince, ack | 14:15 |
| *** athomas_ has quit IRC | 14:16 | |
| *** shardy_ has quit IRC | 14:16 | |
| *** athomas_ has joined #tripleo | 14:16 | |
| *** shardy has joined #tripleo | 14:17 | |
| openstackgerrit | Merged stackforge/puppet-tripleo: Use node IP lists for HA Proxy ipaddresses https://review.openstack.org/186775 | 14:19 |
| *** jprovazn has quit IRC | 14:32 | |
| *** hewbrocca has joined #tripleo | 14:33 | |
| dprince | gfidente, bnemec: also these two https://review.openstack.org/#/c/186776/ , https://review.openstack.org/#/c/186240 | 14:34 |
| *** mcornea has quit IRC | 14:36 | |
| *** mcornea has joined #tripleo | 14:36 | |
| *** yamahata has quit IRC | 14:43 | |
| *** hewbrocca has quit IRC | 14:43 | |
| *** rhallisey has quit IRC | 14:45 | |
| *** ifarkas has quit IRC | 14:47 | |
| *** rhallisey has joined #tripleo | 14:56 | |
| *** panda has quit IRC | 14:59 | |
| *** untriaged-bot has joined #tripleo | 15:00 | |
| untriaged-bot | Untriaged bugs so far: | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461615 | 15:00 |
| openstack | Launchpad bug 1461615 in diskimage-builder "Yum vars directory doesn't exist" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461631 | 15:00 |
| openstack | Launchpad bug 1461631 in diskimage-builder "New Fedora 22 cloud images fail to build" [Undecided,New] | 15:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1459465 | 15:01 |
| openstack | Launchpad bug 1459465 in tripleo "devtest fails to build ramdisk on Fedora or CentOS due to Yum multilib error" [Undecided,Triaged] | 15:01 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1461984 | 15:01 |
| openstack | Launchpad bug 1461984 in tripleo "VM element: /usr/sbin/grub2-install: error: will not proceed with blocklists." [Undecided,New] | 15:01 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461601 | 15:01 |
| openstack | Launchpad bug 1461601 in diskimage-builder "CentOS 6 not resizing" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 15:01 |
| *** untriaged-bot has quit IRC | 15:01 | |
| *** athomas_ has quit IRC | 15:02 | |
| *** akrivoka has quit IRC | 15:10 | |
| *** dsneddon has joined #tripleo | 15:11 | |
| *** lsmola has quit IRC | 15:12 | |
| *** dprince has quit IRC | 15:16 | |
| *** dtantsur is now known as dtantsur|afk | 15:22 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Wire Neutron VLAN ranges param as array to puppet https://review.openstack.org/187262 | 15:23 |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Fix list of type_drivers for ML2 plugin https://review.openstack.org/187472 | 15:24 |
| *** mcornea has quit IRC | 15:29 | |
| *** shardy_ has joined #tripleo | 15:30 | |
| *** shardy has quit IRC | 15:31 | |
| *** shardy_ has quit IRC | 15:35 | |
| *** shardy has joined #tripleo | 15:36 | |
| *** daneyon has joined #tripleo | 15:36 | |
| *** daneyon has quit IRC | 15:38 | |
| *** daneyon has joined #tripleo | 15:38 | |
| *** noslzzp has joined #tripleo | 15:42 | |
| *** mcornea has joined #tripleo | 15:43 | |
| *** alop has joined #tripleo | 15:45 | |
| *** david-lyle has quit IRC | 15:46 | |
| *** david-lyle has joined #tripleo | 15:46 | |
| *** pblaho has quit IRC | 15:47 | |
| *** alop has quit IRC | 15:48 | |
| *** mcornea_ has joined #tripleo | 15:51 | |
| *** mcornea has quit IRC | 15:54 | |
| *** alop has joined #tripleo | 15:54 | |
| *** regebro has quit IRC | 16:03 | |
| *** jistr has quit IRC | 16:03 | |
| *** shardy_ has joined #tripleo | 16:07 | |
| *** shardy has quit IRC | 16:08 | |
| *** shardy_ has quit IRC | 16:13 | |
| *** shardy has joined #tripleo | 16:13 | |
| *** trown is now known as trown|lunch | 16:24 | |
| *** dprince has joined #tripleo | 16:34 | |
| *** rlandy has quit IRC | 16:36 | |
| *** rlandy has joined #tripleo | 16:37 | |
| *** gfidente is now known as gfidente|afk | 16:37 | |
| *** sdake has joined #tripleo | 16:39 | |
| *** sdake_ has joined #tripleo | 16:44 | |
| *** sdake has quit IRC | 16:48 | |
| *** alop has quit IRC | 16:50 | |
| *** mcornea_ has quit IRC | 16:53 | |
| *** paramite has quit IRC | 16:55 | |
| *** rlandy has quit IRC | 17:11 | |
| *** rlandy has joined #tripleo | 17:12 | |
| *** lucasagomes is now known as lucas-beer | 17:13 | |
| *** dguerri has joined #tripleo | 17:17 | |
| *** pelix has quit IRC | 17:19 | |
| *** sthillma has joined #tripleo | 17:38 | |
| *** trown|lunch is now known as trown | 17:41 | |
| *** rlandy has quit IRC | 17:47 | |
| *** sthillma_ has joined #tripleo | 17:49 | |
| *** sdake has joined #tripleo | 17:50 | |
| *** sthillma has quit IRC | 17:51 | |
| *** sthillma_ is now known as sthillma | 17:51 | |
| *** sdake_ has quit IRC | 17:54 | |
| *** sdake_ has joined #tripleo | 17:56 | |
| *** penick has joined #tripleo | 17:57 | |
| *** sdake has quit IRC | 17:59 | |
| *** rwsu has quit IRC | 18:04 | |
| *** penick has quit IRC | 18:06 | |
| *** sdake_ is now known as sdake | 18:10 | |
| *** alop has joined #tripleo | 18:18 | |
| *** gfidente|afk has quit IRC | 18:33 | |
| *** penick has joined #tripleo | 18:45 | |
| *** penick has quit IRC | 18:46 | |
| *** rhefner has quit IRC | 18:54 | |
| *** victor_lowther has quit IRC | 18:54 | |
| *** penick has joined #tripleo | 18:55 | |
| *** sdake has quit IRC | 19:08 | |
| *** Goneri has quit IRC | 19:10 | |
| *** radez is now known as radez_g0n3 | 19:15 | |
| *** Goneri has joined #tripleo | 19:25 | |
| *** sthillma_ has joined #tripleo | 19:26 | |
| *** sthillma has quit IRC | 19:28 | |
| *** Goneri has quit IRC | 19:33 | |
| dprince | bnemec, slagle: mind a quick ruby review https://review.openstack.org/#/c/186776/ | 19:37 |
| greghaynes | ruby review!? | 19:39 |
| greghaynes | ;) | 19:39 |
| *** sthillma_ has quit IRC | 19:40 | |
| slagle | sure, looking | 19:41 |
| slagle | omg 33 lines. this is the most ruby i've ever looked at | 19:42 |
| *** sthillma has joined #tripleo | 19:46 | |
| crinkle | you poor souls | 19:46 |
| openstackgerrit | Merged stackforge/puppet-tripleo: Add interface_for_ip function https://review.openstack.org/186776 | 19:48 |
| openstackgerrit | Merged stackforge/puppet-tripleo: Configure virtual IPs for split out networks https://review.openstack.org/186240 | 19:48 |
| slagle | dprince: the logic seemed sound ^^. i did not review for code correctness :-) | 19:51 |
| dprince | slagle: thanks, it functionally works if that is any comfort | 19:52 |
| *** penick has quit IRC | 19:56 | |
| *** penick has joined #tripleo | 20:01 | |
| *** CaptTofu has quit IRC | 20:02 | |
| *** penick has quit IRC | 20:03 | |
| *** penick has joined #tripleo | 20:05 | |
| *** penick has quit IRC | 20:07 | |
| *** morazi-mtg is now known as morazi | 20:13 | |
| *** jayg is now known as jayg|g0n3 | 20:19 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Bump up Ceph job timeout for resource availability https://review.openstack.org/188213 | 20:37 |
| openstackgerrit | Merged openstack/tripleo-image-elements: Use nagios-common instead of nagios-plugins-all on RHEL. https://review.openstack.org/145038 | 20:38 |
| *** victor_lowther has joined #tripleo | 20:38 | |
| *** CaptTofu has joined #tripleo | 20:48 | |
| *** lblanchard has quit IRC | 20:50 | |
| *** dprince has quit IRC | 20:51 | |
| *** penick has joined #tripleo | 20:59 | |
| *** untriaged-bot has joined #tripleo | 21:00 | |
| untriaged-bot | Untriaged bugs so far: | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461615 | 21:00 |
| openstack | Launchpad bug 1461615 in diskimage-builder "Yum vars directory doesn't exist" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461631 | 21:00 |
| openstack | Launchpad bug 1461631 in diskimage-builder "New Fedora 22 cloud images fail to build" [Undecided,New] | 21:00 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1459465 | 21:00 |
| openstack | Launchpad bug 1459465 in tripleo "devtest fails to build ramdisk on Fedora or CentOS due to Yum multilib error" [Undecided,Triaged] | 21:01 |
| untriaged-bot | https://bugs.launchpad.net/tripleo/+bug/1461984 | 21:01 |
| openstack | Launchpad bug 1461984 in tripleo "VM element: /usr/sbin/grub2-install: error: will not proceed with blocklists." [Undecided,New] | 21:01 |
| untriaged-bot | https://bugs.launchpad.net/diskimage-builder/+bug/1461601 | 21:01 |
| openstack | Launchpad bug 1461601 in diskimage-builder "CentOS 6 not resizing" [Undecided,In progress] - Assigned to Abel Lopez (al592b) | 21:01 |
| *** untriaged-bot has quit IRC | 21:01 | |
| *** penick has quit IRC | 21:04 | |
| *** trown is now known as trown|outttypeww | 21:06 | |
| openstackgerrit | greghaynes proposed openstack/diskimage-builder: Update default Fedora to 22 https://review.openstack.org/185843 | 21:09 |
| *** rhefner has joined #tripleo | 21:13 | |
| *** sthillma has quit IRC | 21:30 | |
| *** sthillma has joined #tripleo | 21:32 | |
| *** Marga_ has joined #tripleo | 21:39 | |
| *** Marga_ has quit IRC | 21:39 | |
| *** Marga_ has joined #tripleo | 21:40 | |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Add Redis as a Pacemaker resource https://review.openstack.org/182617 | 21:40 |
| *** openstack has joined #tripleo | 21:43 | |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Enable use of coordination_url in ceilometer https://review.openstack.org/171996 | 21:55 |
| openstackgerrit | Yanis Guenane proposed openstack/tripleo-heat-templates: Update mongodb implementation to reflect master https://review.openstack.org/185628 | 21:58 |
| *** pino|work has quit IRC | 22:12 | |
| *** thrash is now known as thrash|wknd | 22:15 | |
| *** morazi has quit IRC | 22:30 | |
| *** daneyon has quit IRC | 22:33 | |
| *** daneyon has joined #tripleo | 22:33 | |
| *** daneyon has quit IRC | 22:33 | |
| *** daneyon has joined #tripleo | 22:34 | |
| *** openstackgerrit has quit IRC | 22:37 | |
| *** openstackgerrit has joined #tripleo | 22:37 | |
| *** cody-somerville has quit IRC | 22:42 | |
| *** Marga_ has quit IRC | 22:53 | |
| *** dsneddon has quit IRC | 23:01 | |
| *** dsneddon has joined #tripleo | 23:03 | |
| *** sdake has joined #tripleo | 23:26 | |
| *** daneyon has quit IRC | 23:34 | |
| *** jpeeler has quit IRC | 23:48 | |
| *** BadCub is now known as BadCub-Hiding | 23:55 | |
| *** sdake_ has joined #tripleo | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!