| *** Goneri has joined #tripleo | 00:32 | |
| openstackgerrit | Feng Pan proposed openstack/tripleo-heat-templates: Change ping wait flag https://review.openstack.org/314344 | 00:32 |
|---|---|---|
| *** Goneri has quit IRC | 00:39 | |
| *** pabelanger has quit IRC | 00:42 | |
| *** jpena has joined #tripleo | 00:44 | |
| *** mburned_out has joined #tripleo | 00:44 | |
| *** pabelanger has joined #tripleo | 00:45 | |
| *** hewbrocca-afk has joined #tripleo | 00:46 | |
| *** mbound has joined #tripleo | 00:47 | |
| *** mburned_out is now known as mburned | 00:50 | |
| *** mbound has quit IRC | 00:52 | |
| *** dmacpher-afk has joined #tripleo | 00:52 | |
| *** al has quit IRC | 00:59 | |
| *** al has joined #tripleo | 01:01 | |
| *** trozet has quit IRC | 01:07 | |
| *** rhallisey has quit IRC | 01:48 | |
| *** saneax is now known as saneax_AFK | 01:51 | |
| *** yamahata has joined #tripleo | 02:03 | |
| *** ayoung has quit IRC | 02:17 | |
| *** saneax_AFK is now known as saneax | 02:40 | |
| *** coolsvap has joined #tripleo | 02:41 | |
| *** rajinir has joined #tripleo | 02:44 | |
| *** jokke_ has quit IRC | 02:45 | |
| *** jokke_ has joined #tripleo | 02:45 | |
| *** yuanying has quit IRC | 02:51 | |
| *** stendulker has joined #tripleo | 03:04 | |
| *** shivrao has quit IRC | 03:12 | |
| *** akshai has joined #tripleo | 03:18 | |
| *** masco has joined #tripleo | 03:22 | |
| *** links has joined #tripleo | 03:29 | |
| *** ebalduf has joined #tripleo | 03:34 | |
| *** ccamacho has quit IRC | 03:36 | |
| *** ccamacho has joined #tripleo | 03:39 | |
| *** ayoung has joined #tripleo | 03:42 | |
| *** panda has quit IRC | 03:42 | |
| *** panda has joined #tripleo | 03:42 | |
| *** shivrao has joined #tripleo | 03:43 | |
| *** shivrao has quit IRC | 03:43 | |
| *** tzumainn has quit IRC | 03:48 | |
| *** yuanying has joined #tripleo | 03:48 | |
| *** sirushti has quit IRC | 03:57 | |
| *** sirushti has joined #tripleo | 04:06 | |
| *** saneax is now known as saneax_AFK | 04:11 | |
| *** stendulker_ has joined #tripleo | 04:19 | |
| *** stendulker has quit IRC | 04:23 | |
| *** rlandy has quit IRC | 04:24 | |
| *** stendulker has joined #tripleo | 04:27 | |
| *** stendulker_ has quit IRC | 04:31 | |
| *** ramishra has joined #tripleo | 04:33 | |
| *** veteran has joined #tripleo | 04:42 | |
| *** shivrao has joined #tripleo | 04:43 | |
| *** shivrao_ has joined #tripleo | 04:43 | |
| *** rajinir has quit IRC | 04:46 | |
| *** saneax_AFK is now known as saneax | 04:47 | |
| *** shivrao has quit IRC | 04:47 | |
| *** shivrao_ is now known as shivrao | 04:47 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile https://review.openstack.org/304125 | 04:54 |
| *** trozet has joined #tripleo | 04:56 | |
| *** fragatina has quit IRC | 04:59 | |
| *** akshai has quit IRC | 05:00 | |
| *** trozet has quit IRC | 05:16 | |
| *** fragatina has joined #tripleo | 05:17 | |
| *** fragatina has quit IRC | 05:22 | |
| *** coolsvap has quit IRC | 05:25 | |
| *** itamarl has joined #tripleo | 05:31 | |
| *** fragatina has joined #tripleo | 05:33 | |
| *** fragatina has quit IRC | 05:33 | |
| *** fragatina has joined #tripleo | 05:34 | |
| *** saneax is now known as saneax_AFK | 05:37 | |
| openstackgerrit | Merged openstack/tripleo-ui: Remove module cache files from git https://review.openstack.org/313546 | 05:37 |
| openstackgerrit | Jiri Tomasek proposed openstack/tripleo-ui: Tracking nodes operations in Nodes Table https://review.openstack.org/314233 | 05:51 |
| *** d0ugal has joined #tripleo | 05:52 | |
| *** itamarl has quit IRC | 05:53 | |
| *** oshvartz has joined #tripleo | 06:05 | |
| *** itamarl has joined #tripleo | 06:06 | |
| *** apetrich has joined #tripleo | 06:24 | |
| *** rcernin has joined #tripleo | 06:26 | |
| *** itamarl has quit IRC | 06:26 | |
| *** apetrich has quit IRC | 06:27 | |
| *** apetrich has joined #tripleo | 06:28 | |
| *** dmacpher-afk has quit IRC | 06:31 | |
| *** bvandenh has joined #tripleo | 06:32 | |
| *** cwolferh has quit IRC | 06:34 | |
| *** jprovazn has joined #tripleo | 06:37 | |
| *** tremble has joined #tripleo | 06:38 | |
| *** shivrao has quit IRC | 06:39 | |
| *** mcornea has joined #tripleo | 06:44 | |
| *** itamarl has joined #tripleo | 06:45 | |
| *** florianf has joined #tripleo | 06:46 | |
| *** tesseract has joined #tripleo | 07:03 | |
| *** sshnaidm has quit IRC | 07:03 | |
| *** shivrao has joined #tripleo | 07:07 | |
| *** dsariel has joined #tripleo | 07:11 | |
| *** hewbrocca-afk is now known as hewbrocca | 07:14 | |
| *** zoli_gone-proxy is now known as zoliXXL | 07:15 | |
| *** zoliXXL is now known as zoli|wfh | 07:15 | |
| *** d0ugal has quit IRC | 07:23 | |
| *** d0ugal has joined #tripleo | 07:26 | |
| *** d0ugal has quit IRC | 07:26 | |
| *** d0ugal has joined #tripleo | 07:26 | |
| zoli|wfh | good morning | 07:28 |
| *** paramite has joined #tripleo | 07:30 | |
| *** shivrao has quit IRC | 07:30 | |
| *** mikelk has joined #tripleo | 07:34 | |
| *** olap has joined #tripleo | 07:36 | |
| *** ifarkas has joined #tripleo | 07:43 | |
| *** aufi has joined #tripleo | 07:48 | |
| *** paramite is now known as paramite|afk | 07:51 | |
| *** liverpooler has joined #tripleo | 07:52 | |
| *** shardy has joined #tripleo | 07:56 | |
| *** paramite|afk is now known as paramite | 07:59 | |
| ansiwen | morning | 08:00 |
| *** jaosorior has joined #tripleo | 08:01 | |
| ccamacho | o/ | 08:04 |
| jaosorior | hey dude | 08:05 |
| shardy | 'morning folks | 08:05 |
| *** gfidente has joined #tripleo | 08:08 | |
| *** SlickNik has quit IRC | 08:10 | |
| *** SlickNik has joined #tripleo | 08:11 | |
| bandini | morning * | 08:11 |
| *** ohamada has joined #tripleo | 08:12 | |
| *** athomas has joined #tripleo | 08:14 | |
| *** dtantsur|afk is now known as dtantsur | 08:16 | |
| ccamacho | Guys, just to recheck is anyone working in a AllNodesServices, like the ControllerServices resource for composable THT? | 08:17 |
| *** yamahata has quit IRC | 08:20 | |
| *** SlickNik has quit IRC | 08:22 | |
| *** lucas-dinner is now known as lucasagomes | 08:22 | |
| *** SlickNik has joined #tripleo | 08:22 | |
| shardy | ccamacho: You mean like OS::TripleO::AllNodesExtraConfig ? | 08:25 |
| shardy | ccamacho: we'll have to maintain all the existing ExtraConfig interfaces, even if we do implement an alternative method via the composable services model | 08:26 |
| gfidente | so I opened a LP for the issue which is causing upgrades job to fail | 08:26 |
| gfidente | https://bugs.launchpad.net/tripleo/+bug/1580076 | 08:26 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 08:26 |
| gfidente | but I am still investigating why that happens | 08:26 |
| shardy | gfidente: nice, thanks for looking into it! | 08:26 |
| gfidente | looks like computes can't reach keystone and the pingtest vm can't be scheduled | 08:26 |
| *** devvesa has joined #tripleo | 08:27 | |
| gfidente | but I haven't figured why that is, so I am trying to reproduce locally | 08:27 |
| shardy | ccamacho: Oh, do you mean a parameter like AllNodesServices, that puts a service on all roles? | 08:27 |
| shardy | personally I would prefer we didn't do that, as IMHO it's clearer to just add a service to the list for all roles | 08:28 |
| shardy | and also it's going to be very difficult to do when we enable arbitrary custom roles | 08:28 |
| shardy | The first priority is adding a $RoleServices parameter for all existing roles, and decomposing all the services on the roles we already have | 08:30 |
| shardy | then we need to enable arbitrary custom roles, which will basically repeat the same pattern for user provided additional roles | 08:30 |
| *** cmyster has joined #tripleo | 08:32 | |
| *** cmyster has quit IRC | 08:32 | |
| *** cmyster has joined #tripleo | 08:32 | |
| ccamacho | shardy, like ControllerServiceChain for the controller services (https://review.openstack.org/#/c/259568/) but for all nodes | 08:33 |
| shardy | ccamacho: What advantage would that have, over say adding the service explicitly to each role's list of services? | 08:35 |
| shardy | ccamacho: I'm thinking that although that's a slightly more verbose interface, it's more explicit (easier for e.g UI to show what services are running on each role), and also potentially cleaner when the implementation needs to be adapted to support custom user-defined roles (which I'm currently working on based on the jinja template approach we talked about at summit) | 08:37 |
| *** jistr has joined #tripleo | 08:37 | |
| openstackgerrit | Giulio Fidente proposed openstack/puppet-tripleo: Add Cinder API/Scheduler/Volume roles https://review.openstack.org/310939 | 08:39 |
| *** sshnaidm has joined #tripleo | 08:40 | |
| ccamacho | shardy, ok, is that I have finished the NTP service with composable roles but, wanted to keep working to land it sooner as it applies to all roles | 08:44 |
| shardy | ccamacho: Sure, in that case, I'd suggest writing patches that wire in parameters for all existing roles, just like ControllerServices | 08:45 |
| shardy | that will help because it's a prerequisite to decomposing all the existing roles | 08:45 |
| shardy | and it will enable you to wire in your NTP patch to all roles | 08:45 |
| shardy | we can revisit the AllNodes thing later, but I think we should land the per-role interfaces first | 08:45 |
| ccamacho | sure, thanks! | 08:46 |
| *** derekh has joined #tripleo | 08:47 | |
| shardy | https://etherpad.openstack.org/p/tripleo-composable-roles-work | 08:49 |
| shardy | ccamacho: I added a section for per-role prerequistes there, feel free to update | 08:49 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Install virtualenv for tempest https://review.openstack.org/314462 | 08:50 |
| *** anshul has joined #tripleo | 08:53 | |
| ccamacho | shardy, nice! | 08:53 |
| *** anshul is now known as Guest64153 | 08:54 | |
| *** itamarl_ has joined #tripleo | 09:06 | |
| *** itamarl has quit IRC | 09:08 | |
| *** itamarl_ is now known as itamarl | 09:09 | |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Add timestamps to postci log https://review.openstack.org/314468 | 09:11 |
| *** ebalduf has quit IRC | 09:13 | |
| *** saneax_AFK is now known as saneax | 09:20 | |
| *** milan has quit IRC | 09:28 | |
| *** chem has joined #tripleo | 09:29 | |
| *** chlong has quit IRC | 09:30 | |
| *** mgould has joined #tripleo | 09:34 | |
| *** gfidente has quit IRC | 09:40 | |
| *** tosky has joined #tripleo | 09:45 | |
| shadower | mandre: ping | 09:53 |
| shardy | dtantsur: just FYI - I think it was you who complained re the annoying SIGNAL_COMPLETE: Unknown events, this is the fix https://review.openstack.org/#/c/314226/ | 09:58 |
| derekh | Is anybody looking into whats up with the upgrades job? | 09:58 |
| *** coolsvap has joined #tripleo | 09:58 | |
| dtantsur | awesome! | 09:58 |
| shardy | derekh: I think gfidente is | 09:58 |
| derekh | shardy: ack, will chat to him once he comes online | 10:00 |
| shardy | he was around earlier, hopeully will show up again soon :) | 10:01 |
| *** milan has joined #tripleo | 10:01 | |
| openstackgerrit | Marios Andreou proposed openstack/tripleo-common: Adds a simple 'overcloud_shell' mistral worfklow https://review.openstack.org/313084 | 10:02 |
| *** jaosorior has quit IRC | 10:09 | |
| *** jaosorior has joined #tripleo | 10:09 | |
| *** ooolpbot has joined #tripleo | 10:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 10:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 10:10 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 10:10 |
| *** ooolpbot has quit IRC | 10:10 | |
| *** liverpooler has quit IRC | 10:26 | |
| *** liverpooler has joined #tripleo | 10:26 | |
| *** chem has quit IRC | 10:45 | |
| *** rodrigods has quit IRC | 10:49 | |
| *** rodrigods has joined #tripleo | 10:49 | |
| *** chem has joined #tripleo | 10:49 | |
| *** hewbrocca is now known as hewbrocca-afk | 10:51 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-pacemaker: WIP: integrate PCS provider in the merge. https://review.openstack.org/310713 | 10:53 |
| *** dtantsur is now known as dtantsur|bbl | 10:55 | |
| *** jpena is now known as jpena|lunch | 10:55 | |
| mandre | shadower: pong | 10:56 |
| shadower | mandre: so the only validanion not working under Ansible 2 is the mysql file limits one | 10:57 |
| shadower | mandre: I just looked it and unfortunately, the mysql_variables module we use there has incompatible output between 1.9 and 2.0 | 10:57 |
| shadower | mandre: so instead of hacking around that, I'd rather we just switched to 2.0 wholesale | 10:58 |
| shadower | what do you think? | 10:58 |
| mandre | shadower: seems good to me | 10:58 |
| shadower | cool, I'll send a patch | 10:59 |
| shadower | of course, if the deployers need 1.9 support, we can add it later, but I'd like 2.0 be the default | 10:59 |
| mandre | excellent, looking forward to try it out | 10:59 |
| mandre | agreed, 2.0 is a better default | 11:00 |
| ccamacho | derekh, gfidente I pasted some more info about (https://bugs.launchpad.net/tripleo/+bug/1580076) on the ticket | 11:02 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 11:02 |
| derekh | ccamacho: thanks, The message you posted "Message: No valid host was found. There are not enough hosts available." comes from nova any time it fails to schedule a VM, | 11:05 |
| *** stendulker has quit IRC | 11:06 | |
| derekh | ccamacho: it looks like the reason it can't start the VM is because the nova compute node is trying to talk to http://[2001:db8:fd00:1000::10]:5000/v3/auth/tokens but its can't for some reason | 11:06 |
| *** rhallisey has joined #tripleo | 11:06 | |
| ccamacho | derekh: Im trying to reproduce it again (I did that yesterday) to get more nova details on my lab, the thing is that this behavior does not happen all the time, In my case only when creating/deleting the overcloud some times. | 11:08 |
| derekh | ccamacho: ok, I'm trying to do the same, hopfully between us we'll figure it out | 11:08 |
| ccamacho | sure :) | 11:09 |
| *** ooolpbot has joined #tripleo | 11:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 11:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 11:10 |
| *** ooolpbot has quit IRC | 11:10 | |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 11:10 |
| *** jaosorior has quit IRC | 11:20 | |
| *** jaosorior has joined #tripleo | 11:20 | |
| openstackgerrit | Derek Higgins proposed openstack/tripleo-heat-templates: Nothing to see here https://review.openstack.org/314510 | 11:22 |
| derekh | ccamacho: testing a hunch ^ seeing if setting neutron_auth_type back to v3password helps (undoing https://review.openstack.org/#/c/312300/1) | 11:24 |
| *** mgould is now known as mgould|afk | 11:26 | |
| *** thrash|g0ne is now known as thrash | 11:35 | |
| jistr | derekh: yeah i saw the patch changing that value going to puppet-nova. We seem to be having a failure when establishing the connection though, so as long as IP and port are unaffected by the 'password' vs. 'v3password' change, it should at least establish the connection, regardless of what the rest of the URL looks like. But it's probably worth a try nevertheless. | 11:41 |
| *** panda has quit IRC | 11:42 | |
| *** panda has joined #tripleo | 11:42 | |
| *** jaosorior has quit IRC | 11:42 | |
| *** jaosorior has joined #tripleo | 11:43 | |
| openstackgerrit | Dougal Matthews proposed openstack/tripleo-common: WIP Testing and tweaks ontop of https://review.openstack.org/#/c/282366 https://review.openstack.org/314520 | 11:45 |
| d0ugal | rbrady: https://review.openstack.org/#/c/314520/ | 11:45 |
| d0ugal | rbrady: I'd just pull those changes into yours, I wasn't sure if I should add a new patchset or not - so take what you find useful from that. | 11:46 |
| *** ccamacho is now known as ccamacho|lunch | 11:46 | |
| *** weshay has joined #tripleo | 11:48 | |
| rbrady | d0ugal: wow. thanks! | 11:49 |
| *** lucasagomes is now known as lucas-hungry | 11:52 | |
| *** karthiks has quit IRC | 11:52 | |
| *** saneax is now known as saneax_AFK | 11:56 | |
| *** gfidente has joined #tripleo | 11:57 | |
| *** jtomasek_ has joined #tripleo | 11:58 | |
| *** pkovar has joined #tripleo | 12:00 | |
| derekh | jistr: I've a feeling that maybe v3password choose the internalurl endpoint and for some reason the password plugin is using publicurl | 12:01 |
| derekh | jistr: but its just a hunch, we'll see what happens | 12:01 |
| d0ugal | rbrady: np, hopefully it helps. | 12:02 |
| *** social has joined #tripleo | 12:03 | |
| *** karthiks has joined #tripleo | 12:04 | |
| *** jcoufal has joined #tripleo | 12:04 | |
| jistr | derekh: yea that could be it perhaps. Also, the time when the change was made in puppet-nova is near the time the upgrade jobs started failing. | 12:05 |
| derekh | jistr: yup | 12:05 |
| *** saneax_AFK is now known as saneax | 12:05 | |
| derekh | <openstackgerrit> Derek Higgins proposed openstack/tripleo-heat-templates: Nothing to see here https://review.openstack.org/314510 | 12:05 |
| derekh | <derekh> ccamacho: testing a hunch ^ seeing if setting neutron_auth_type back to v3password helps (undoing https://review.openstack.org/#/c/312300/1) | 12:05 |
| derekh | gfidente: ^ | 12:05 |
| shardy | jtomasek_: Can you by any chance provide me with a tripleo-ui screenshot, showing where we prompt for per-service configuration parameters? | 12:08 |
| *** jtomasek_ has quit IRC | 12:08 | |
| shardy | In particular if you can show anything related to configuring Ceph or CephStorage roles that would be very helpful | 12:08 |
| jtomasek | shardy: sure | 12:09 |
| shardy | thanks! | 12:09 |
| * shardy really must get it running himself soon | 12:09 | |
| jtomasek | shardy: actually, we're not doing it per service yet, we just list all the parameters in one modal | 12:10 |
| *** ooolpbot has joined #tripleo | 12:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 12:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 12:10 |
| *** ooolpbot has quit IRC | 12:10 | |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 12:10 |
| openstackgerrit | Jiri Stransky proposed openstack/tripleo-docs: Upgrade documentation https://review.openstack.org/308985 | 12:10 |
| shardy | jtomasek: Ok, that's fine - just anywhere we surface any parameters related to Ceph derived from the templates will be fine | 12:11 |
| jtomasek | shardy: ok, give me a while, I'll get back to you | 12:11 |
| shardy | jtomasek: thanks! | 12:11 |
| *** pradk has joined #tripleo | 12:16 | |
| *** dprince has joined #tripleo | 12:16 | |
| *** jayg|g0n3 is now known as jayg | 12:16 | |
| *** jcoufal has quit IRC | 12:21 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Initial Integration of Mistral into TripleO https://review.openstack.org/282366 | 12:21 |
| *** jpena|lunch is now known as jpena | 12:21 | |
| *** jcoufal has joined #tripleo | 12:22 | |
| rbrady | d0ugal: I have pulled your change into the patch and added you as a co-author | 12:22 |
| *** colonwq has quit IRC | 12:23 | |
| d0ugal | rbrady: cool, thanks! | 12:23 |
| *** dkehn has quit IRC | 12:23 | |
| *** dkehn has joined #tripleo | 12:24 | |
| *** dmacpher has joined #tripleo | 12:26 | |
| *** morazi has joined #tripleo | 12:27 | |
| *** aufi has quit IRC | 12:28 | |
| *** colonwq has joined #tripleo | 12:28 | |
| *** rlandy has joined #tripleo | 12:29 | |
| jtomasek | shardy: https://goo.gl/photos/NFKeoQ2ZTxePetqeA | 12:29 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Add timestamps to postci log https://review.openstack.org/314468 | 12:30 |
| *** MaxPC has joined #tripleo | 12:31 | |
| shardy | jtomasek: thanks! | 12:31 |
| *** ramishra has quit IRC | 12:34 | |
| rbrady | d0ugal: re latest feedback on https://review.openstack.org/#/c/282366/ . How strongly do you feel about the change of container to overcloud_name? | 12:34 |
| openstackgerrit | Merged openstack/tripleo-common: StackAction parameter to distinguish top-level create vs. update https://review.openstack.org/306392 | 12:36 |
| dprince | d0ugal: hi, yeah I had an issue with your comment from patchset 22. To me "process_templates" has nothing to do with the overcloud_name I think | 12:36 |
| dprince | d0ugal: process_templates is just a utility function that could be used for a variety of things, validating stacks, creating stacks, updating stacks. etc | 12:37 |
| *** dtantsur|bbl is now known as dtantsur | 12:37 | |
| dprince | d0ugal: and as such I would prefer more generic names for the parameters | 12:37 |
| *** liverpooler has quit IRC | 12:38 | |
| dprince | rbrady: another question. Why are we passing in the swift base URL now again? | 12:38 |
| dprince | rbrady: Initially I was getting that from the swift endpoint directly | 12:39 |
| *** sshnaidm has quit IRC | 12:39 | |
| d0ugal | rbrady, dprince - sure, then it shouldn't set stack_name :) or it should have a more generic name than container | 12:39 |
| d0ugal | since container clear maps to swift IMO | 12:39 |
| dprince | d0ugal: given the function requires the use of swift I would argue 'contianer' is the most right thing | 12:39 |
| d0ugal | alright, I don't really mind that much but it still seems wrong to me | 12:40 |
| dprince | d0ugal: we aren't abstracting this like the old API did | 12:40 |
| dprince | d0ugal: swift *is* our storage mechanism I think | 12:40 |
| d0ugal | Yup, that's fair | 12:40 |
| d0ugal | but that function also sets the stack_name, so it steps beyond just dealing with containers | 12:41 |
| rbrady | dprince: your original code had this call to process_multiple_environments_and_files in an action which had access to the endpoint via the base action | 12:41 |
| rbrady | dprince: when I moved this out to a utils method, I needed to pass in either the endpoint or the url, since I didn't use anything else from the endpoint I chose the url | 12:41 |
| *** fultonj has joined #tripleo | 12:41 | |
| * shardy predicts folks will want git as an alternative to swift at some point | 12:41 | |
| dprince | shardy: git checkout, tarball, swift post. DONE! | 12:42 |
| dprince | shardy: please... lets not mention that again :) | 12:42 |
| shardy | heh, fair enough ;) | 12:42 |
| rbrady | drpince: you'll notice I had to do the same with the auth_token as well because the utils method doesn't have direct access to the mistral context | 12:43 |
| dprince | shardy: and if they did want to do it differently I would argue the code goes into python-heatclient anyways. So think about reviewing a git patch there... | 12:43 |
| *** ccamacho|lunch is now known as ccamacho | 12:44 | |
| *** karthiks has quit IRC | 12:44 | |
| *** saneax is now known as saneax_AFK | 12:44 | |
| *** coolsvap has quit IRC | 12:47 | |
| dprince | rbrady: http://paste.openstack.org/show/496567/ | 12:48 |
| dprince | rbrady: we already use the Mistral ctx in the base class. So why not that ^^^ | 12:48 |
| *** coolsvap has joined #tripleo | 12:48 | |
| dprince | rbrady: sorry if I seem nit-picky. My review is very much comparing my prototype code to what has evolved. I'm just trying to see things line up | 12:49 |
| *** coolsvap has quit IRC | 12:50 | |
| *** coolsvap has joined #tripleo | 12:50 | |
| *** saneax_AFK is now known as saneax | 12:51 | |
| *** lucas-hungry is now known as lucasagomes | 12:51 | |
| EmilienM | jistr: I'm reading your comment on https://review.openstack.org/#/c/314253/ - I'm going to move things to step5 but I still think step 6 is useless | 12:51 |
| jtomasek | dprince: what is the zaqar status right now, Is it already possible to listen to zaqar queue with current undercloud? | 12:51 |
| rbrady | dprince: I'm not getting the value of adding that code for the swift url. I can already get the url by calling self.get_object_client().url. I still have to pass it to the utils method. | 12:51 |
| EmilienM | jistr: our CI is currently failing a lot of times, and upgade job is broken, because it takes too much time, we need to do something | 12:52 |
| jtomasek | dprince: I see you are using zaqar action here https://review.openstack.org/#/c/313953/2/workbooks/deployment.yaml | 12:52 |
| dprince | jtomasek: yes. I posted a fix of the weekend to fix an issue with subscribing to a queue though | 12:52 |
| dprince | jtomasek: https://review.openstack.org/#/c/313950/ | 12:52 |
| jistr | EmilienM: yea i see that but i'm not sure removing step 6 is the way, at least not until we can say "yea we can do overcloud init via puppet even without step 6" /cc jaosorior | 12:52 |
| dprince | jtomasek: with that fix it all works for me :) | 12:52 |
| jtomasek | dprince: how can I apply that change to my undercloud? | 12:53 |
| EmilienM | jistr: at least we can try | 12:53 |
| *** sshnaidm has joined #tripleo | 12:53 | |
| dprince | jtomasek: edit /etc/zaqar/zaqar.conf directly and restart openstack-zaqar and openstack-zaqar@1 | 12:53 |
| EmilienM | jistr: do you have another solution to reduce CI jobs duration? | 12:54 |
| jtomasek | dprince: thanks! | 12:54 |
| dprince | rbrady: when you say utils which line are you referring to? | 12:54 |
| rbrady | dprince line 99 in /utils/templates.py | 12:55 |
| jistr | EmilienM: image caching should help a ton i think. | 12:56 |
| *** karthiks has joined #tripleo | 12:57 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: keystone: drop usage of step 6 https://review.openstack.org/314543 | 12:57 |
| *** mbound has joined #tripleo | 12:57 | |
| dprince | rbrady: okay, I think it is fine then. I got confused and thought it was the action, I see your point. It is just the utils class | 12:58 |
| dprince | rbrady: going to try it out now | 12:58 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: deployment: drop step6 https://review.openstack.org/314253 | 12:58 |
| *** julim has joined #tripleo | 12:58 | |
| EmilienM | jistr: I've looked at results, it helps but not "a ton". Dropping steps will help, every Puppet run on a controller is 7 minutes, that you can multiply by number of nodes and number of steps | 12:59 |
| EmilienM | slagle: can we land https://review.openstack.org/#/c/314242/ even if not passing upgrade job? It reduce (a little) the jobs) | 13:01 |
| *** Goneri has joined #tripleo | 13:02 | |
| jaosorior | EmilienM: If we figure out that we can consistently deploy the keystone endpoints on step 5 (meaning that keystone will be up) Then I'm fine removing step 6 | 13:02 |
| dprince | EmilienM: I would defer to jistr and marios for that one | 13:02 |
| *** zoli|wfh is now known as zoli|lunch | 13:02 | |
| dprince | jistr/marios: Was step 7 never used for pacemaker fencing (settling) or something? | 13:02 |
| *** mbound has quit IRC | 13:02 | |
| EmilienM | if I manage to make my patch passing CI consistently, I don't see the point of keeping step 6 | 13:02 |
| jaosorior | however, the t-h-t cr that removes step 6, in it's current state, seems to break things | 13:02 |
| jaosorior | EmilienM: I agree; having less steps is better | 13:03 |
| EmilienM | step 6 + step 7 deletion can save us 25 min, + overcloud caching, we can save 35 min | 13:03 |
| dprince | EmilienM: my concern is there is some case not covered by our CI related to multi-node HA upgrades or something | 13:03 |
| *** mgould|afk is now known as mgould | 13:03 | |
| EmilienM | jaosorior: yeah, see my patch in puppet-tripleo ^^^^ | 13:03 |
| jaosorior | EmilienM: I think step 7 is safer to remove, though | 13:03 |
| jaosorior | EmilienM: Ah, just noticed it | 13:03 |
| jistr | i think step 7 used to be for heat domain stuff, but got unnecessary after a few improvements to puppet-heat | 13:04 |
| EmilienM | gfidente: the point if removing steps is having less timeouts | 13:04 |
| jistr | step 7 is probably safe to remove, step 6 not (yet) | 13:04 |
| EmilienM | gfidente: look http://tripleo.org/cistatus.html and all the red | 13:04 |
| EmilienM | gfidente: 7 steps is too much, I'm sure we can do everything in 5 or 4 steps later | 13:04 |
| *** fpan has quit IRC | 13:05 | |
| EmilienM | we can do better at orchestration with resource ordering, tags, etc | 13:05 |
| *** pkovar has quit IRC | 13:05 | |
| EmilienM | dprince: for step7? | 13:05 |
| jaosorior | EmilienM: Have you checked why the upgrades gate is failing on the CR that removes step 7? Is it always timeouts? | 13:05 |
| dprince | EmilienM: I would like to modify the commit message a bit to avoid confusiong here. Perhaps referencing why step7 was added. | 13:05 |
| gfidente | though ordering/tagging in the manifests makes harder to split per-service | 13:06 |
| dprince | EmilienM: otherwise +2 on removing it from me | 13:06 |
| dprince | EmilienM: good detective work on this | 13:06 |
| EmilienM | jaosorior: I almost spent all my day yesterday man | 13:06 |
| EmilienM | jaosorior: I figured some steps are running for almost nothing, and take ~25 min (step 6 and 7) | 13:06 |
| EmilienM | jaosorior: jobs are timeouting a lot for different reasons | 13:07 |
| jaosorior | EmilienM: I haven't had much time to check, that's why I asked you. If it's failing the upgrades gate just because of timeouts... then screw it, lets merge that :P | 13:07 |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: composable heat services https://review.openstack.org/313577 | 13:07 |
| gfidente | so it's not that I don't want to remove unneeded steps, but I'd be careful about merging in same step stuff which isn't today in same step | 13:07 |
| gfidente | given the roles are stressing the ordering already | 13:07 |
| EmilienM | we're not merging anything, right now I'm playing detective and trying to run 5 steps. If it works, maybe we can look at merging it. Right now I'm not sure yet | 13:08 |
| EmilienM | though step7 can imho be removed safely | 13:08 |
| *** akshai has joined #tripleo | 13:08 | |
| EmilienM | but we can't merge patches anymore until upgrade jobs are failing | 13:08 |
| gfidente | yeah step 7 is fine to remove for sure | 13:09 |
| gfidente | I'll go back looking at the upgrades job issue | 13:10 |
| *** ooolpbot has joined #tripleo | 13:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 13:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 13:10 |
| *** ooolpbot has quit IRC | 13:10 | |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 13:10 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-tripleoclient: Remove the orchestration property from tripleoclient class https://review.openstack.org/308552 | 13:10 |
| dprince | jistr: so Step7 used to be Step6 before I incrented the Step counts for the composable services. | 13:11 |
| dprince | jistr: Step6 was added here: https://review.openstack.org/#/c/188415/ | 13:11 |
| EmilienM | dprince: right, that's what I found | 13:12 |
| EmilienM | and I think fencing can be done in step5, no need of step6 for that. | 13:12 |
| jistr | dprince: yea i found that too.. but why did we add step7 here? https://github.com/openstack/tripleo-heat-templates/commit/9988bd25aa4bac1375ef4783d636c7adecedee92 | 13:12 |
| dprince | EmilienM: I would like to mention these details in the commit message I think | 13:12 |
| EmilienM | dprince: yeah I got it man | 13:13 |
| dprince | jistr: because it existed. And all I did was increment it | 13:13 |
| EmilienM | dprince: I'm just waiting for CI before updating it | 13:13 |
| dprince | EmilienM: cool. I won't touch it then | 13:13 |
| gfidente | dprince, jistr I think that step got merged with one of the patches meant to do keystone users from puppet | 13:13 |
| gfidente | then we reverted the change doing the users via puppet but not the one adding the step | 13:13 |
| slagle | EmilienM: imo, we can't land that w/o passing upgrades | 13:14 |
| EmilienM | dprince: oh, for step7, ok sorry | 13:14 |
| jistr | gfidente: ah right that could be it... | 13:14 |
| jistr | slagle: +1 | 13:14 |
| gfidente | keystone endpoints actually, not keystone users | 13:14 |
| slagle | it's a template change removing a step, while we fully expect that to work on stack-update, we need to prove it :) | 13:14 |
| marios | dprince: jistr: reading back | 13:14 |
| slagle | EmilienM: ^^ | 13:14 |
| EmilienM | slagle: the thing is, I think upgrade job will come back with different fixes: reduce steps, caching, etc. And we need to manually evaluate every patch if it could break upgrade or not | 13:14 |
| dprince | jistr: I know why Step7 exists (I incremented the step counts). That was a simple thing and applied to all of the steps after ringbuilder so it doesn't concern me. My real question about this goes back to the addition of Step6 in https://review.openstack.org/#/c/188415/4 I think | 13:15 |
| jistr | dprince: so this is confusing and i had to read it a few times, but i think this change added one more step by mistake https://github.com/openstack/tripleo-heat-templates/commit/9988bd25aa4bac1375ef4783d636c7adecedee92 | 13:16 |
| slagle | EmilienM: well we could make a case for landing it w/ failing upgrades if it's been manualy tested i suppose | 13:16 |
| jistr | dprince: what we should have done is renumber those steps, but not add additional ones, no? | 13:16 |
| slagle | EmilienM: as long as that is captured in the review. so someone deploys with tht w/o the patch, then stack-updates to tht with the patch | 13:16 |
| dprince | jistr: no, that was correct. The ringbuilder step wasn't in the "count" before | 13:16 |
| dprince | jistr: all I did was add it into the step numbering sequence so it could be de-composed with all the other services | 13:17 |
| *** yamahata has joined #tripleo | 13:18 | |
| jistr | dprince: yes but the patch made it to be step 3, and bumped the step numbers on the consecutive steps. So Step6 had indeed hiera('step') == 6 instead of hiera('step') == 5 as it was before. And that was all that needed to be done in that patch imo. Unless i'm missing something, addition of step 7 is superfluous. | 13:18 |
| dprince | jistr: I changed the manifests as well though | 13:20 |
| *** pkovar has joined #tripleo | 13:20 | |
| dprince | jistr: in the increment steps patch I changed it to: | 13:21 |
| dprince | $enable_fencing = str2bool(hiera('enable_fencing', false)) and hiera('step') >= 6 | 13:21 |
| jistr | dprince: yes, but also, hiera('step') == 5 was the maximum before, and after your patch hiera('step') == 6 was the maximum in the manifests | 13:21 |
| dprince | jistr: where it used to be 5. So it was all fine | 13:21 |
| dprince | jistr: the fencing step always used >= | 13:22 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: deployment: remove Step7 https://review.openstack.org/314242 | 13:22 |
| jistr | dprince: so we went from Step6 = hiera5 to Step6 = hiera6, both in templates and in puppet manifests, which is correct, but still no need for step 7 imo. | 13:23 |
| *** links has quit IRC | 13:23 | |
| dprince | jistr: right. The rename didn't effect anything. But my real question then is back to this: https://review.openstack.org/#/c/188415/4/puppet/manifests/overcloud_controller_pacemaker.pp | 13:23 |
| dprince | jistr: that added the extra step6 at the time (which is now called step7) | 13:24 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: deployment: drop step6 https://review.openstack.org/314253 | 13:25 |
| EmilienM | jistr: why don't we enable fencing by default in our CI btw? | 13:26 |
| jistr | dprince: hehe i still think we don't understand each other. Step6 is *still* called Step6, but it now also uses hiera value 6. | 13:26 |
| marios | jistr: so the last step (i would have said it was 6, i don't recall 7) was for heat/keystone domain stuff... am looking at the review comments from jaosorior at https://review.openstack.org/#/c/314253 - have a call starting in 5 mins which I need to prepare for - will try and catchup later | 13:26 |
| *** egafford has joined #tripleo | 13:26 | |
| jistr | so: Step6 -- added because of fencing, might not be needed for fencing with EmilienM's patch, but still is used for keystone init | 13:27 |
| *** lblanchard has joined #tripleo | 13:27 | |
| jistr | Step7 -- never used, looks like it was added mistakenly during a refactoring of step numbers | 13:27 |
| jistr | (the step numbers should only be bumped on puppet side, to sync the Heat Step numbering with hiera step numbering, but no extra step addition was needed) | 13:28 |
| *** mgould has quit IRC | 13:29 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Add loadbalancer profile for ha & non-ha https://review.openstack.org/313026 | 13:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Add loadbalancer profile for ha & non-ha https://review.openstack.org/313026 | 13:30 |
| *** ramishra has joined #tripleo | 13:31 | |
| *** mgould has joined #tripleo | 13:31 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Deploy Loadbalancer as a composable role https://review.openstack.org/313029 | 13:32 |
| *** ramishra has quit IRC | 13:35 | |
| dprince | jistr: I disagree with your assessment that an extra step wasn't required for the step incrementing patch. Note that I removed ControllerRingbuilderPuppetConfig, and then added Step 7. The delta is 0. | 13:36 |
| jistr | dprince: well you removed the *config*, but not the *deployment*. The deployment was only renamed, so i think the delta is +1, as shown here: https://paste.fedoraproject.org/364649/87220146/ | 13:37 |
| * jistr double checks | 13:37 | |
| EmilienM | slagle: thx fo the co-authoring on https://review.openstack.org/#/c/309408/ ! but no worries for next times, you can skip it | 13:38 |
| *** sshnaidm has quit IRC | 13:39 | |
| dprince | jistr: if that is the case then I think we list that as the root cause here and move on :) | 13:39 |
| jistr | dprince: yea for Step7 indeed -- can be safely removed, IMO. Step6 is still used though in puppet-tripleo for Keystone init. | 13:41 |
| jistr | EmilienM: ^ | 13:41 |
| dprince | jistr: agree | 13:41 |
| *** sshnaidm has joined #tripleo | 13:41 | |
| EmilienM | jistr: and I dropped it | 13:41 |
| EmilienM | https://review.openstack.org/314543 | 13:41 |
| dprince | jistr: thanks for talking through this :) | 13:41 |
| *** bvandenh has quit IRC | 13:42 | |
| jistr | dprince: sure thing, thank you :) | 13:42 |
| *** bvandenh has joined #tripleo | 13:42 | |
| dprince | EmilienM: so with regards to the removal of step6 you would be looking at https://review.openstack.org/#/c/188415/4 | 13:43 |
| dprince | EmilienM: so if we can remove that too then just reference it. :) | 13:43 |
| *** jobewan has joined #tripleo | 13:43 | |
| EmilienM | dprince: did you see https://review.openstack.org/#/c/314253/ ? | 13:43 |
| dprince | guys, I think I win the award for wasting the most CI time this year. Step 7!! | 13:43 |
| EmilienM | dprince: congrats man | 13:44 |
| dprince | oh the public shame man! | 13:44 |
| jistr | EmilienM, dprince: and jaosorior's https://review.openstack.org/#/c/304125/ should be thought about too when removing step 6, but that might "just work" with what EmilienM did in https://review.openstack.org/314543 | 13:44 |
| dprince | EmilienM: cool, I will follow up on that | 13:44 |
| slagle | we've got to come up with a $$$ value on this wasted time | 13:46 |
| slagle | how much did the power cost? :) | 13:47 |
| *** tzumainn has joined #tripleo | 13:47 | |
| gfidente | naaah | 13:48 |
| gfidente | I bet money on step6 and maybe even step7 coming back as soon as we get more into the roles stuff | 13:48 |
| openstackgerrit | Merged openstack/puppet-tripleo: Add tripleo::selinux https://review.openstack.org/309408 | 13:49 |
| *** veteran has quit IRC | 13:49 | |
| slagle | EmilienM: i had to give you and yanis co-author on that ^...i just basically copied it :) | 13:49 |
| EmilienM | cool | 13:50 |
| slagle | plus i dont want anyone to think i wrote that much puppet | 13:50 |
| EmilienM | I think we stole it too lol | 13:50 |
| EmilienM | slagle: you can add "Puppet" in your Linkedin | 13:50 |
| EmilienM | I'll recommend you | 13:50 |
| *** dsariel has quit IRC | 13:54 | |
| *** coolsvap has quit IRC | 13:54 | |
| *** ebalduf has joined #tripleo | 13:55 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: Adds a tarball upload option to plan creation https://review.openstack.org/314582 | 13:55 |
| pabelanger | derekh: thanks for the review. dprince: EmilienM: do you mind looking at https://review.openstack.org/#/c/312725/ that's the current patchset to support centos-7 jenkins slaves | 13:56 |
| *** tosky has quit IRC | 13:57 | |
| *** yamahata has quit IRC | 13:57 | |
| *** yamahata has joined #tripleo | 13:57 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: Adds a tarball upload option to plan creation https://review.openstack.org/314582 | 13:57 |
| EmilienM | pabelanger: thing is, upgrade jobs are failing, and we decided to stop merging patches (except if upgrades were manually tested) | 13:58 |
| EmilienM | I would like to merge it, but I'm afraid we need to fix our upgrade jobs first | 13:59 |
| beagles | quit | 13:59 |
| beagles | arg | 13:59 |
| shardy | Hey folks, meeting time | 14:00 |
| dprince | pabelanger: ack, I will look at it | 14:00 |
| *** afazekas has quit IRC | 14:01 | |
| *** afazekas has joined #tripleo | 14:01 | |
| *** dsariel has joined #tripleo | 14:09 | |
| *** ooolpbot has joined #tripleo | 14:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 14:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 14:10 |
| *** ooolpbot has quit IRC | 14:10 | |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 14:10 |
| *** julim has quit IRC | 14:11 | |
| *** zoli|lunch is now known as zoli|wfh | 14:11 | |
| *** julim has joined #tripleo | 14:12 | |
| *** oshvartz has quit IRC | 14:13 | |
| *** ppradhan has joined #tripleo | 14:13 | |
| *** rajinir has joined #tripleo | 14:20 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: composable heat services https://review.openstack.org/313577 | 14:22 |
| *** jaosorior has quit IRC | 14:24 | |
| *** saneax is now known as saneax_AFK | 14:33 | |
| *** saneax_AFK is now known as saneax | 14:33 | |
| *** dc has joined #tripleo | 14:40 | |
| *** aufi has joined #tripleo | 14:41 | |
| *** jprovazn has quit IRC | 14:43 | |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/tripleo-quickstart: Move image cache dir to a different location https://review.openstack.org/313553 | 14:43 |
| dc | g | 14:44 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-puppet-elements: bump puppet-ceph to stable/hammer https://review.openstack.org/314311 | 14:44 |
| *** ramishra_ has quit IRC | 14:49 | |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Initial Integration of Mistral into TripleO https://review.openstack.org/282366 | 14:51 |
| *** pcaruana has joined #tripleo | 14:53 | |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-pacemaker: WIP: integrate PCS provider in the merge. https://review.openstack.org/310713 | 14:57 |
| *** dtrainor has joined #tripleo | 14:57 | |
| *** stendulker_ has joined #tripleo | 14:58 | |
| *** dsariel has quit IRC | 14:58 | |
| dprince | so who created this https://etherpad.openstack.org/p/tripleo-composable-roles-work | 14:59 |
| EmilienM | shardy: I think it's my fault | 15:00 |
| dprince | all of that goes away I think :) | 15:00 |
| EmilienM | your etherpad is much older | 15:00 |
| dprince | shardy: I like my etherpad into the BP you created | 15:00 |
| dprince | shardy: FWIW my style for spec light stuff is to link to etherpads in the LP specs | 15:00 |
| *** masco has quit IRC | 15:01 | |
| dprince | shardy: keeping the 'implements: blueprint' stuff in the commit messages is fine too but I find etherpads help organize the sequencing, etc. | 15:01 |
| *** milan has quit IRC | 15:04 | |
| *** tosky has joined #tripleo | 15:05 | |
| *** saneax is now known as saneax_AFK | 15:05 | |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: [WIP] Adds a tarball upload option to plan creation https://review.openstack.org/314582 | 15:06 |
| openstackgerrit | Florian Fuchs proposed openstack/tripleo-ui: [WIP] Adds a tarball upload option to plan creation https://review.openstack.org/314582 | 15:06 |
| *** trozet has joined #tripleo | 15:07 | |
| *** ooolpbot has joined #tripleo | 15:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 15:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 15:10 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 15:10 |
| *** ooolpbot has quit IRC | 15:10 | |
| *** dsariel has joined #tripleo | 15:12 | |
| pabelanger | doh, missed the weekly meeting | 15:13 |
| *** r-mibu has quit IRC | 15:14 | |
| *** r-mibu has joined #tripleo | 15:14 | |
| beagles | shardy, EmilienM: I'm cool with "workaround" for a bug tag as discussed. Should I just proceed with that? | 15:15 |
| EmilienM | currently in irc puppet meeting | 15:15 |
| EmilienM | but yeah wfm | 15:15 |
| beagles | EmilienM: thx | 15:15 |
| *** Guest64153 has quit IRC | 15:16 | |
| dprince | EmilienM: I'm moving stuff from your etherpad into mine BTW. | 15:19 |
| *** paramite is now known as paramite|afk | 15:19 | |
| dprince | EmilienM: seriously how did I not know about your etherpad man!? | 15:19 |
| EmilienM | dprince: you serious? | 15:19 |
| dprince | EmilienM: no idea. Never seen it | 15:19 |
| EmilienM | dprince: I sent an email to tripleo ML after summit | 15:19 |
| EmilienM | dprince: FWIW, mine has much more links & people connected. | 15:20 |
| EmilienM | but feel free to move stuffs, as long people use it and it's useful. | 15:20 |
| dprince | EmilienM: sorry, I totally missed that. Perhapd the URL was similar enough that I skipped it | 15:20 |
| EmilienM | np | 15:21 |
| *** aufi has quit IRC | 15:22 | |
| *** kbyrne has quit IRC | 15:22 | |
| *** hewbrocca-afk is now known as hewbrocca | 15:24 | |
| dprince | EmilienM: for every new services that we would have deployed with Pacemaker, do not deploy it with pacemaker anymore because of the pacemaker-lite spec. | 15:24 |
| EmilienM | dprince: I disagree | 15:24 |
| dprince | EmilienM: I'm removing that because incorrect I think. For now we do buth | 15:24 |
| dprince | EmilienM: that was in the other etherpad | 15:24 |
| dprince | EmilienM: I'm not copying it... | 15:24 |
| *** rhallisey has quit IRC | 15:24 | |
| EmilienM | I'm in a meeting... | 15:24 |
| EmilienM | brb ~15 min | 15:24 |
| *** rhallisey has joined #tripleo | 15:25 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: composable heat services https://review.openstack.org/313577 | 15:26 |
| sshnaidm | bnemec, what do you think? http://logs.openstack.org/68/314468/2/check-tripleo/gate-tripleo-ci-f22-nonha/af112b8/logs/postci.txt.gz | 15:29 |
| bnemec | sshnaidm: About what? | 15:34 |
| sshnaidm | bnemec, about adding timestamps to postci log to index it in logstash | 15:35 |
| bnemec | sshnaidm: Oh, yeah I'm all for that if it gets us the ability to logstash postci. | 15:35 |
| *** kbyrne has joined #tripleo | 15:36 | |
| sshnaidm | bnemec, great, so you can vote :) https://review.openstack.org/#/c/314468/ | 15:36 |
| *** milan has joined #tripleo | 15:39 | |
| *** rbrady has quit IRC | 15:39 | |
| *** rhallisey has quit IRC | 15:39 | |
| *** rbrady has joined #tripleo | 15:40 | |
| *** kbyrne has quit IRC | 15:40 | |
| *** rhallisey has joined #tripleo | 15:40 | |
| *** itamarl has quit IRC | 15:40 | |
| *** panda has quit IRC | 15:42 | |
| *** panda has joined #tripleo | 15:43 | |
| *** paramite|afk is now known as paramite | 15:44 | |
| *** mikelk has quit IRC | 15:45 | |
| dprince | EmilienM: no rush. But I don't see the email to openstack-dev about the composable services etherpad. I was going to reply with the new link... | 15:45 |
| shardy | beagles: sorry, also in mtg, +1 on workaround | 15:46 |
| *** yamahata has quit IRC | 15:47 | |
| *** paramite has quit IRC | 15:48 | |
| dprince | EmilienM: no, I got it. Sorting issue | 15:48 |
| *** dsariel has quit IRC | 15:48 | |
| EmilienM | dprince: http://lists.openstack.org/pipermail/openstack-dev/2016-May/093566.html | 15:48 |
| dprince | EmilienM: got it. replied. Really sorry I missed that one. Timing/travel :/ | 15:51 |
| dprince | EmilienM: anyways the etherpads are combined and I made some comments on the old with with respect to the pacemaker requirements | 15:51 |
| EmilienM | dprince: no worries I don't care. | 15:51 |
| *** tesseract has quit IRC | 15:52 | |
| *** myoung has joined #tripleo | 15:54 | |
| *** sshnaidm has quit IRC | 15:57 | |
| *** tremble has quit IRC | 15:58 | |
| *** dprince has quit IRC | 15:58 | |
| *** cwolferh has joined #tripleo | 16:00 | |
| beagles | shardy: cool, thx | 16:01 |
| *** penick has joined #tripleo | 16:04 | |
| EmilienM | slagle: ha job looks faster, isn't? https://review.openstack.org/#/c/314543/ | 16:06 |
| EmilienM | 1h 56m 19s | 16:06 |
| EmilienM | oh nevermind, I found some jobs where is was 2h00 | 16:07 |
| *** eil397 has joined #tripleo | 16:08 | |
| derekh | EmilienM: 1h 44 https://review.openstack.org/#/c/312397/ | 16:09 |
| EmilienM | yeah... nevermind | 16:09 |
| EmilienM | the THT patch I did will tell us | 16:09 |
| EmilienM | https://review.openstack.org/314253 | 16:09 |
| *** ooolpbot has joined #tripleo | 16:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 16:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 16:10 |
| *** ooolpbot has quit IRC | 16:10 | |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,Triaged] | 16:10 |
| *** ramishra has joined #tripleo | 16:11 | |
| *** dprince has joined #tripleo | 16:14 | |
| *** bvandenh has quit IRC | 16:20 | |
| *** ifarkas has quit IRC | 16:28 | |
| EmilienM | dprince: is https://review.openstack.org/#/c/314311/ fine for you now? | 16:31 |
| *** electrofelix has joined #tripleo | 16:36 | |
| *** yamahata has joined #tripleo | 16:36 | |
| *** ohamada has quit IRC | 16:38 | |
| openstackgerrit | Brad P. Crochet proposed openstack/tripleo-heat-templates: composable heat services https://review.openstack.org/313577 | 16:42 |
| *** gfidente has quit IRC | 16:48 | |
| *** gfidente has joined #tripleo | 16:48 | |
| *** derekh has quit IRC | 16:50 | |
| *** sshnaidm has joined #tripleo | 16:53 | |
| *** stendulker_ has quit IRC | 16:53 | |
| *** tserong has quit IRC | 16:54 | |
| *** mgould has quit IRC | 16:55 | |
| *** eil397 has quit IRC | 16:55 | |
| *** eil397 has joined #tripleo | 16:56 | |
| *** mgould has joined #tripleo | 16:58 | |
| *** mgould is now known as mgould|afk | 16:59 | |
| *** tserong has joined #tripleo | 16:59 | |
| *** penick has quit IRC | 16:59 | |
| dprince | EmilienM: Just so I'm clear puppet-ceph master is targetted for jewel then? | 16:59 |
| EmilienM | dprince: yes | 17:00 |
| jdob | if i post a patch against heat and then add a depends-on to it from the tripleo repo, will tripleo's CI pick up that patch? gfidente and I were talking about it yesterday but we weren't 100% sure | 17:02 |
| shardy | jdob: yes, it should | 17:03 |
| jdob | awesome, thanks shardy | 17:03 |
| shardy | Or at lest, it did last time I tried it :) | 17:03 |
| jdob | shardy: and I can WIP the heat one and CI won't ignore it, right? | 17:03 |
| gfidente | shardy jdob yeah this is useful for the get_addr issue testing | 17:04 |
| shardy | jdob: yup - the only gotcha is, when the heat patch actually merges, tripleo-ci will still use the pinned current-tripleo version | 17:04 |
| jdob | ya, i'm about to vomit log statements all over heat, so i dont want it to merge | 17:04 |
| jdob | yet at least :) | 17:04 |
| shardy | jdob: ack, AFAIK it should work | 17:05 |
| openstackgerrit | Giulio Fidente proposed openstack/tripleo-heat-templates: WIP: Force Nova to the Keystone internal endpoint https://review.openstack.org/314684 | 17:07 |
| *** shivrao has joined #tripleo | 17:07 | |
| gfidente | there is no support in puppet-nova for service_auth yet ^^ | 17:07 |
| gfidente | but I think that might fix the problem we're seeing in upgrades | 17:07 |
| gfidente | if it does, we can probably work an actual addition to the puppet module | 17:07 |
| *** devvesa has quit IRC | 17:09 | |
| jdob | gfidente: also, I'm gonna dork with the THT patch a bit too and mark that as WIP | 17:09 |
| gfidente | jdob sure | 17:09 |
| *** shivrao_ has joined #tripleo | 17:10 | |
| jdob | the next step after this is to sacrifice a goat to see if I can rid the evil spirits from heat | 17:10 |
| *** ooolpbot has joined #tripleo | 17:10 | |
| ooolpbot | URGENT TRIPLEO TASKS NEED ATTENTION | 17:10 |
| ooolpbot | https://bugs.launchpad.net/tripleo/+bug/1580076 | 17:10 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,In progress] - Assigned to Giulio Fidente (gfidente) | 17:10 |
| *** ooolpbot has quit IRC | 17:10 | |
| jdob | wow, that bot really is a seagull | 17:11 |
| *** tserong has quit IRC | 17:12 | |
| *** shivrao has quit IRC | 17:12 | |
| *** shivrao_ is now known as shivrao | 17:12 | |
| *** jpena is now known as jpena|off | 17:13 | |
| *** tserong has joined #tripleo | 17:13 | |
| openstackgerrit | Jay Dobies proposed openstack/tripleo-heat-templates: [WIP] Pass MysqlVirtualIP via EndpointMap https://review.openstack.org/305751 | 17:15 |
| *** athomas has quit IRC | 17:17 | |
| *** jistr has quit IRC | 17:21 | |
| *** MaxPC has quit IRC | 17:21 | |
| gfidente | jdob the depends-on | 17:22 |
| jdob | still working on the heat part | 17:22 |
| jdob | but i wanted to kick that off | 17:22 |
| gfidente | ack | 17:22 |
| gfidente | tty tomorrow! | 17:22 |
| jdob | though in hindsight, i probably should have waited | 17:22 |
| gfidente | thanks1 | 17:22 |
| jdob | night :) | 17:22 |
| *** gfidente has quit IRC | 17:22 | |
| *** zoli|wfh is now known as zoli|gone | 17:23 | |
| *** zoli|gone is now known as zoli_gone-proxy | 17:25 | |
| openstackgerrit | Brad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles https://review.openstack.org/313019 | 17:25 |
| dprince | shardy, slagle: this passes the upgrades job https://review.openstack.org/#/c/314510/1/puppet/hieradata/compute.yaml | 17:27 |
| dprince | EmilienM: ^^^ | 17:27 |
| dprince | Does that mean the puppet-keystone patch was perhaps the root cause of our issues? | 17:28 |
| dprince | Sorry, puppet-nove | 17:29 |
| dprince | http://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=271a2587a38c6a396213c95aaa8a690fcc283091 | 17:29 |
| slagle | shall we update the commit msg and merge it? | 17:30 |
| *** penick has joined #tripleo | 17:30 | |
| dprince | slagle: I think so. Probably worth investigating a revert in puppet-nova as well. | 17:30 |
| *** mcornea has quit IRC | 17:31 | |
| dprince | slagle: I will push a new patch and +2 it. No changes other than commit message and a comment maybe. | 17:31 |
| shardy | Has anyone verified that switching those settings manually in the nova conf breaks auth? | 17:31 |
| shardy | +1 on merging tho | 17:31 |
| dprince | shardy: I've not. But we've not had upgrades passing for days man | 17:31 |
| shardy | I had some weird issues trying to use the generic password vs v3password plugin in heat recently | 17:31 |
| *** TSCHAK_ has joined #tripleo | 17:32 | |
| shardy | dprince: Yeah, sure, I'm just thinking that should be a pretty quick confirmation, as those auth plugin settings generally either work or they don't | 17:32 |
| shardy | in particular the puppet stuff looks suspect because it mixes a generic (unversioned) password plugin with a v3 suffixed auth url | 17:32 |
| * dprince finds bug reference | 17:33 | |
| *** sambetts is now known as sambetts|afk | 17:33 | |
| dprince | https://bugs.launchpad.net/tripleo/+bug/1580076 | 17:33 |
| openstack | Launchpad bug 1580076 in tripleo "Upgrades job failing pingtest with "Message: No valid host was found."" [Critical,In progress] - Assigned to Giulio Fidente (gfidente) | 17:33 |
| *** rooooook has quit IRC | 17:34 | |
| *** rooooook has joined #tripleo | 17:34 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: Set nova neutron auth back to 'v3password'. https://review.openstack.org/314510 | 17:35 |
| dprince | slagle, shardy; I'd say we +A now | 17:36 |
| *** dmacpher is now known as dmacpher-afk | 17:36 | |
| slagle | Working ... | 17:37 |
| slagle | Working ... | 17:37 |
| slagle | there it goes | 17:37 |
| slagle | gerrit is slow today | 17:38 |
| slagle | dprince: i wonder if there is another bug in puppet somehow where it doesn't apply the config the first time, then on stack-update it does | 17:38 |
| dprince | slagle: could be | 17:39 |
| dprince | slagle: https://review.openstack.org/314695 | 17:40 |
| dprince | slagle: I'll ask Emilien and Colleen | 17:41 |
| *** shardy has quit IRC | 17:41 | |
| EmilienM | dprince: +2 | 17:44 |
| *** lucasagomes is now known as lucas-afk | 17:49 | |
| openstackgerrit | Merged openstack/tripleo-heat-templates: Set nova neutron auth back to 'v3password'. https://review.openstack.org/314510 | 17:50 |
| *** absubram has joined #tripleo | 17:50 | |
| *** bvandenh has joined #tripleo | 17:51 | |
| EmilienM | dprince, slagle: wdyt if we run upgrade jobs in puppet CI beside non-ha? would it be too much? | 17:51 |
| dprince | EmilienM: we could switch to just the upgrades job then? | 17:51 |
| EmilienM | yes | 17:52 |
| dprince | EmilienM: so long as we run a single job for each puppet patch I think it would be fine | 17:52 |
| dprince | EmilienM: plus, now that you've gotten rid of a step who knows. Maybe it'll even run fast :) | 17:53 |
| EmilienM | dprince: ok i'm submitting a patch. | 17:53 |
| dprince | gotta watch that dprince, that guys always added un-needed code everywhere | 17:53 |
| EmilienM | dprince: upgrade job is HA, right? | 17:53 |
| *** tosky has quit IRC | 17:53 | |
| EmilienM | ahahah no worries | 17:53 |
| crinkle | o/ | 17:54 |
| *** larstobi has joined #tripleo | 17:54 | |
| dprince | EmilienM: upgrades job is only single controller HA, but it does use pacemaker | 17:54 |
| *** pkovar has quit IRC | 17:55 | |
| EmilienM | oh ok | 17:55 |
| *** lucas-afk is now known as lucasagomes | 17:55 | |
| EmilienM | well, let's do it | 17:55 |
| openstackgerrit | Jay Dobies proposed openstack/tripleo-heat-templates: [WIP] Pass MysqlVirtualIP via EndpointMap https://review.openstack.org/305751 | 17:55 |
| * EmilienM preparing a patch | 17:55 | |
| dprince | crinkle: hi, a rather cryptic bug we seem to have traced back to that puppet-nova patch. I proposed a revert without spending too much time on it but perhaps you can double check it? | 17:56 |
| openstackgerrit | Sagi Shnaidman proposed openstack-infra/tripleo-ci: Add timestamps to postci log https://review.openstack.org/314468 | 17:56 |
| *** lucasagomes is now known as lucas-dinner | 17:57 | |
| crinkle | dprince: very strange, i'll approve after i dig through the logs for 313650 for a bit | 17:58 |
| dprince | crinkle: sure, np. We've just landed a fix in TripleO to avoid the issue so I think we aren't in a rush now | 17:59 |
| EmilienM | dprince: for puppet-pacemaker, I would make an exception and have both ha & upgrade jobs | 18:00 |
| EmilienM | this module is very critical and does not have much contributions, so we can have 2 jobs. | 18:00 |
| dprince | EmilienM: agree | 18:00 |
| dprince | good exception to make | 18:00 |
| dprince | EmilienM: lets loop derekh in on this too in case he has some resource concern I'm unaware of. I think it'll be fine though | 18:01 |
| EmilienM | sure, I'll mention it in commit message | 18:02 |
| EmilienM | dprince: https://review.openstack.org/314708 | 18:06 |
| *** rajinir has quit IRC | 18:06 | |
| *** mcornea has joined #tripleo | 18:08 | |
| *** rcernin has quit IRC | 18:13 | |
| *** florianf has quit IRC | 18:19 | |
| *** electrofelix has quit IRC | 18:19 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Composable role for RabbitMQ https://review.openstack.org/311849 | 18:19 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Deploy RabbitMQ as a composable role https://review.openstack.org/311863 | 18:20 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Add loadbalancer profile for ha & non-ha https://review.openstack.org/313026 | 18:20 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Deploy Loadbalancer as a composable role https://review.openstack.org/313029 | 18:21 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: deployment: remove Step7 https://review.openstack.org/314242 | 18:24 |
| *** dtantsur is now known as dtantsur|afk | 18:36 | |
| *** mcornea has quit IRC | 18:38 | |
| *** penick has quit IRC | 18:41 | |
| *** penick has joined #tripleo | 18:41 | |
| *** rcernin has joined #tripleo | 18:42 | |
| *** pkovar has joined #tripleo | 18:43 | |
| EmilienM | slagle, dprince: is there some WIP to have an upgrade job that test mitaka -> newton already? | 18:44 |
| slagle | EmilienM: the first step is to get the job times down | 18:45 |
| *** bvandenh has quit IRC | 18:45 | |
| slagle | we're hoping the caching plus the hw ram/ssd's will buy us enough time so we can run such an upgrades job | 18:45 |
| slagle | as is, there is no reason to add one, because it would never finish within the gate timeout | 18:46 |
| *** jcoufal has quit IRC | 18:47 | |
| EmilienM | slagle: it makes sense | 18:49 |
| dprince | yep | 18:50 |
| dprince | rbrady: hi, two very minor comments. https://review.openstack.org/#/c/282366 | 18:50 |
| *** olap has quit IRC | 18:50 | |
| dprince | rbrady: I've just tested your patch so with those fixed I'm +2 | 18:51 |
| dprince | rbrady: want me to fix them? | 18:52 |
| EmilienM | slagle, dprince: another option for later (maybe one day) when we'll have sorted things out around CI & hw & etc: having already-deployed undercloud/overcloud in a pool of resources, and use them for upgrade jobs | 18:52 |
| *** rajinir has joined #tripleo | 18:53 | |
| rbrady | dprince: wrt to the comment about heat and auth token, it's used in line 144, which calls into heat at line 148 and from looking at python-heatclient it seems it is used | 18:54 |
| rbrady | dprince: I can fix the setup.cfg and add "keystone auth token for interacting with heat and swift". would that suffice? | 18:57 |
| dprince | rbrady: yes. sorry the auth_token was is totally invalid | 18:58 |
| *** fpan has joined #tripleo | 18:58 | |
| dprince | rbrady: I see it now :) | 18:58 |
| rbrady | dprince: okay, removed the line in setup.cfg | 19:01 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Initial Integration of Mistral into TripleO https://review.openstack.org/282366 | 19:01 |
| *** fragatina has quit IRC | 19:02 | |
| dprince | rbrady: so the auth_token suggestion I had was actually right. What is going on is python-heatclient calls into Swift though. So the comment is fine, but it does actually eventually call Swift | 19:04 |
| dprince | rbrady: oh, and can we add Depends-On: Ibb98013d6ef283bd454f958bb036557d2b02997f | 19:04 |
| dprince | rbrady: that way the jenkins test should pass I think | 19:04 |
| dprince | rbrady: anyways, +2. Thanks man | 19:05 |
| openstackgerrit | Ryan Brady proposed openstack/tripleo-common: Initial Integration of Mistral into TripleO https://review.openstack.org/282366 | 19:06 |
| rbrady | dprince: done. thanks for all of the help | 19:07 |
| dprince | rbrady: cool, +2 again | 19:07 |
| dprince | rbrady: so I'd like to enable automatic cleanup of old executions | 19:08 |
| dprince | rbrady: Mistral already supports this, but its off by default | 19:08 |
| openstackgerrit | Merged openstack/tripleo-quickstart: Move image cache dir to a different location https://review.openstack.org/313553 | 19:08 |
| dprince | rbrady: does 48 hours seem like a reasonable time to keep old executions? | 19:08 |
| rbrady | dprince: I think that's a question best solved by asking an operator (or advocate). 48 hours seems reasonable to me, but I know I can turn that off if debugging something more long term | 19:09 |
| *** fragatina has joined #tripleo | 19:09 | |
| dprince | rbrady: sure, I'll throw up a patch and ask around I guess | 19:10 |
| rbrady | dprince: maybe we add the feature and document a note on disabling in case of need | 19:10 |
| dprince | rbrady: having no cleanup seems bad though | 19:10 |
| rbrady | dprince: agreed | 19:10 |
| *** fpan has quit IRC | 19:12 | |
| *** pcaruana has quit IRC | 19:12 | |
| openstackgerrit | Dan Prince proposed openstack/instack-undercloud: Enable Mistral execution cleanup https://review.openstack.org/314733 | 19:16 |
| openstackgerrit | Dimitri Savineau proposed openstack/tripleo-heat-templates: Remove duplicate neutron_mechanism_drivers https://review.openstack.org/314734 | 19:17 |
| rbrady | dprince: do those settings show up in /etc/mistral/mistral.conf? | 19:19 |
| dprince | rbrady: yeah. one sec | 19:19 |
| dprince | rbrady: for a really short cleanup (2 hours) | 19:20 |
| dprince | [execution_expiration_policy] | 19:20 |
| dprince | evaluation_interval=2 | 19:20 |
| dprince | older_than=60 | 19:20 |
| dprince | rbrady: that would let you test it out if you want to tinker with it | 19:20 |
| rbrady | dprince: thanks...adding that info to the notes for docs | 19:21 |
| dprince | rbrady: then restart mistral-engine | 19:21 |
| dprince | rbrady: cool | 19:21 |
| slagle | in which i find myself needing a yaml equivalent of jq | 19:22 |
| *** MaxPC has joined #tripleo | 19:22 | |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: composable neutron ml2 and ovs agent https://review.openstack.org/308514 | 19:23 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: composable neutron dhcp service https://review.openstack.org/303386 | 19:23 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: composable neutron metadata service https://review.openstack.org/303618 | 19:23 |
| openstackgerrit | Dan Prince proposed openstack/tripleo-heat-templates: composable neutron l3 service https://review.openstack.org/303562 | 19:23 |
| dprince | slagle: maybe not what you are asking for but the yaql stuff is pretty nice | 19:23 |
| slagle | is there a cli? | 19:24 |
| dprince | yeah | 19:24 |
| dprince | yaql | 19:24 |
| slagle | what's the package name? | 19:24 |
| dprince | I guess it loads json files though | 19:24 |
| slagle | hmm, can't find it on f22 | 19:24 |
| dprince | python2-yaql | 19:24 |
| dprince | slagle: install python2-yaql on your undercloud (Centos) | 19:25 |
| *** julim has quit IRC | 19:26 | |
| dprince | slagle: sadness it only loads json ATM | 19:26 |
| slagle | yea i see | 19:27 |
| *** paramite has joined #tripleo | 19:27 | |
| slagle | i want to be able to do something like "jq .parameters overcloud.yaml" and get just the parameters from overcloud.yaml in yaml format | 19:29 |
| dprince | slagle: heat stack-validate? | 19:29 |
| slagle | no, for any yaml | 19:29 |
| dprince | yeah, I get you | 19:29 |
| dprince | slagle: just create a simple python class man | 19:30 |
| *** paramite has quit IRC | 19:40 | |
| *** panda has quit IRC | 19:41 | |
| rbrady | dtrainor: what do you think of the 48 hour lifespan of old mistral executions? Does that sound reasonable enough from an operators perspective? should the time be shorter or longer? | 19:42 |
| dtrainor | i think it sounds rather long | 19:42 |
| dtrainor | is this a hard default of some type? | 19:43 |
| *** panda has joined #tripleo | 19:43 | |
| bnemec | It sounds short to me, coming from a customer who just left their overcloud in a bad state for weeks before getting back to it. | 19:44 |
| rbrady | dtrainor: it's just a config setting. it can be changed in a file and a service restarted to take effect. | 19:44 |
| dtrainor | as i understand executions, it doesn't sound like any of them should take that long | 19:44 |
| rbrady | bnemec: thanks, this is valuable feedback to get some live operations info | 19:44 |
| dtrainor | Unless it's in a Waiting or Idle state, no? | 19:45 |
| rbrady | dtrainor: it's information about past executions....history | 19:45 |
| rbrady | dtrainor: e.g. you run validations that fail and you want to look at the output gathered | 19:45 |
| dtrainor | oh, that was not clear. if that's the case then yes I'd agree with bnemec, I'd want as much of that as reasonably possible. | 19:45 |
| dtrainor | i think 7d would be a reasonable but aggressive default | 19:46 |
| openstackgerrit | Ethan Gafford proposed openstack/tripleo-heat-templates: Pass MysqlVirtualIP via EndpointMap https://review.openstack.org/305751 | 19:49 |
| openstackgerrit | Ethan Gafford proposed openstack/tripleo-heat-templates: Trove Integration https://review.openstack.org/233240 | 19:49 |
| dtrainor | Would that be do-able? | 19:50 |
| dprince | dtrainor: a deployment execution may take hours | 19:51 |
| dprince | dtrainor: so you prefer a week? | 19:51 |
| *** rcernin has quit IRC | 19:51 | |
| dtrainor | dprince, i understand, but rbrady was talking about historical data on completed tasks if I understand correctly | 19:52 |
| dtrainor | I'd want the historical data around for a pretty long time | 19:52 |
| dprince | dtrainor: https://review.openstack.org/#/c/314733/ | 19:53 |
| bnemec | We're not talking about an enormous amount of data here, are we? | 19:53 |
| dprince | bnemec: it all depends on what the workflows "publish" | 19:53 |
| bnemec | In some cases it should just be a boolean that says validation passed/failed. | 19:53 |
| dtrainor | that's why i cautioned with the word "reasonable", I'm not sure how much data there would be | 19:53 |
| dprince | bnemec: if they publish lots of data it could add up I think | 19:53 |
| dprince | I expect there will be so many executions that at some point it'll be hard to keep track of them. Thus I'd like to see us clean things up | 19:54 |
| dtrainor | it would be a lot of records, sure | 19:55 |
| dprince | in my case, if a workflow isn't running after a few minutes I never check it again | 19:55 |
| dprince | in most of my cases so far that is | 19:55 |
| dtrainor | and that's fine, but i'd also say that's because you're dprince | 19:55 |
| bnemec | :-) | 19:55 |
| dtrainor | others not as familiar with this might need more information | 19:56 |
| dprince | lets put it this way. If we have long lived data it needs to come out of executions and quickly move into something else (swift or something) | 19:56 |
| dprince | that is my take anyway | 19:56 |
| dprince | dtrainor: and yes, I have been creating lots of ad-hoc executions | 19:57 |
| dtrainor | i agree in some capacity, but i wouldn't want to switch tools to view that "old old" data | 19:57 |
| dtrainor | that's where the happy medium comes in to play i suppose | 19:57 |
| dprince | dtrainor: comment on that patch with your feedback. We can also start an email thread if that works | 19:57 |
| dtrainor | will do. thanks for running it past me rbrady | 19:57 |
| *** rcernin has joined #tripleo | 19:58 | |
| jrist | hey rbrady on this patch https://review.openstack.org/#/c/282366/ why is it saying jenkins -1 even though all the gates passed? | 20:00 |
| rbrady | jrist: because jenkins is a greedy bastard who stopped accepting my bribes to +1 patches | 20:01 |
| dprince | jrist: the requirements issue | 20:01 |
| dprince | jrist: seems it didn't pickup the Depends-On then? | 20:01 |
| rbrady | jrist: what dprince said | 20:01 |
| jrist | I see | 20:02 |
| *** dprince has quit IRC | 20:05 | |
| *** absubram has quit IRC | 20:06 | |
| *** akshai has quit IRC | 20:08 | |
| *** Goneri has quit IRC | 20:12 | |
| rbrady | jrist: did you see https://review.openstack.org/#/c/314638/ ? | 20:26 |
| *** fragatina has quit IRC | 20:27 | |
| *** ebalduf has quit IRC | 20:27 | |
| *** dc has quit IRC | 20:29 | |
| *** absubram has joined #tripleo | 20:33 | |
| bnemec | slagle: Regarding parameter editing in YAML, I've got two pieces of code that together would probably do what you need: | 20:39 |
| bnemec | Line 54 of https://review.openstack.org/#/c/253638/7/tripleo_heat_templates/environment_generator.py | 20:39 |
| bnemec | and https://github.com/cybertron/tripleo-scripts/blob/master/net_processing.py#L176 | 20:39 |
| *** ccamacho has quit IRC | 20:42 | |
| slagle | bnemec: cool, my example however was arbitrary and isn't what i was actually doing :) | 20:44 |
| slagle | i literally want a "yq" interface that works for anything. there appear to be some random ones on github | 20:45 |
| slagle | b/c ya know, if i'm going to spend my days coding in yaml... | 20:46 |
| slagle | (finish however you like i guess) | 20:46 |
| jrist | Yet Another YAML Implementation | 20:46 |
| jrist | YAYI | 20:46 |
| jrist | ? | 20:46 |
| * bnemec dies a little inside | 20:47 | |
| *** apetrich has quit IRC | 20:47 | |
| jrist | bnemec: :) | 20:48 |
| *** lucas-dinner has quit IRC | 20:51 | |
| *** ccamacho has joined #tripleo | 20:54 | |
| *** lucasagomes has joined #tripleo | 20:58 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: loadbalancer: add Aodh API support https://review.openstack.org/288122 | 21:05 |
| *** ebalduf has joined #tripleo | 21:16 | |
| *** lblanchard has quit IRC | 21:16 | |
| *** rhallisey has quit IRC | 21:17 | |
| *** jayg is now known as jayg|g0n3 | 21:22 | |
| *** hjensas has quit IRC | 21:23 | |
| *** yamahata has quit IRC | 21:23 | |
| *** MaxPC has quit IRC | 21:25 | |
| *** egafford has quit IRC | 21:26 | |
| *** hjensas has joined #tripleo | 21:27 | |
| *** yamahata has joined #tripleo | 21:29 | |
| *** mbound has joined #tripleo | 21:30 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/python-tripleoclient: Aodh integration to tripleoclient https://review.openstack.org/314784 | 21:41 |
| *** milan has quit IRC | 21:44 | |
| *** milan has joined #tripleo | 21:44 | |
| *** pkovar has quit IRC | 21:49 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: keystone: drop usage of step 6 https://review.openstack.org/314543 | 21:49 |
| openstackgerrit | Pradeep Kilambi proposed openstack/python-tripleoclient: Aodh integration to tripleoclient https://review.openstack.org/314784 | 21:53 |
| *** fultonj has quit IRC | 21:56 | |
| EmilienM | HA jobs randomly fail because of sahara db sync that runs before galera ready | 21:56 |
| *** fragatina has joined #tripleo | 21:57 | |
| *** rbrady has quit IRC | 21:59 | |
| *** ccamacho has quit IRC | 22:06 | |
| *** ccamacho has joined #tripleo | 22:09 | |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: Ensure db-sync Execs are run when Galera is ready https://review.openstack.org/314792 | 22:10 |
| EmilienM | slagle: ^ and https://review.openstack.org/#/c/314791/ will maybe help us too, so we can run dbsync in a previous step | 22:11 |
| EmilienM | slagle: thanks to resource tagging | 22:11 |
| openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates: deployment: drop step6 https://review.openstack.org/314253 | 22:12 |
| EmilienM | all upgade jobs are failing, and it seems https://review.openstack.org/#/c/282366/ should help | 22:15 |
| * EmilienM will continue later /me afk | 22:17 | |
| *** morazi has quit IRC | 22:43 | |
| *** Goneri has joined #tripleo | 23:03 | |
| *** jobewan has quit IRC | 23:07 | |
| *** saneax_AFK is now known as saneax | 23:20 | |
| *** d0ugal has quit IRC | 23:21 | |
| *** absubram has quit IRC | 23:26 | |
| *** rcernin has quit IRC | 23:28 | |
| *** Goneri has quit IRC | 23:30 | |
| *** d0ugal has joined #tripleo | 23:37 | |
| *** trozet has quit IRC | 23:38 | |
| *** rooooook has quit IRC | 23:39 | |
| *** rooooook has joined #tripleo | 23:43 | |
| *** rooooook has joined #tripleo | 23:44 | |
| *** crinkle has quit IRC | 23:55 | |
| *** chlong has joined #tripleo | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!