Tuesday, 2017-05-02

openstackgerritDavid Moreau Simard proposed openstack/ara master: Make arguments for serve_static_packaged standard  https://review.openstack.org/46152600:04
openstackgerritDavid Moreau Simard proposed openstack/ara master: Tidy up HTML and get rid of needless whitespace  https://review.openstack.org/46155100:23
openstackgerritMerged openstack/ara master: Make arguments for serve_static_packaged standard  https://review.openstack.org/46152600:28
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add unit tests for CLI: playbook delete  https://review.openstack.org/46157500:46
openstackgerritDavid Moreau Simard proposed openstack/ara master: Remove dead functions/code  https://review.openstack.org/46157600:57
openstackgerritDavid Moreau Simard proposed openstack/ara master: Tidy up HTML and get rid of needless whitespace  https://review.openstack.org/46155100:58
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add unit tests for CLI: playbook delete  https://review.openstack.org/46157500:58
dmsimardjrist: still interested in a funny angry and sad versions of the ara logo for 404's and 500's btw :p00:59
*** iceyao has joined #ara01:08
iceyaomorning all01:20
dmsimardiceyao: I'm on the other end of the world I guess, good evening :p01:29
iceyaohi, I'm from china.01:31
dmsimardhey there01:34
jristdmsimard: haha like a frowny face?02:16
dmsimardjrist: you have creative freedom sir02:17
dmsimardjrist: something on the funny side of things that a bird could look like when it's angry or sad02:17
dmsimardangry could have red eyes with plucked feathers floating around or something02:17
jristwhat about a bunch of ara birds carrying a whale02:17
jrist:)02:17
dmsimardsad could just have T_T eyes with a tear02:17
dmsimardhum, that sounds a lot like the twitter fail whale :P02:18
dmsimardhttp://www.yiyinglu.com/wp-content/uploads/2013/11/lifting-a-dreamer-2009.jpg02:18
dmsimardjrist: I'm down for anything you come up with, just want to make custom 404/500 pages02:19
dmsimardjrist: the openstack.org is fun https://www.openstack.org/40402:21
iceyaooh, It's funny02:24
*** iceyao has quit IRC02:28
*** iceyao has joined #ara02:38
openstackgerritDavid Moreau Simard proposed openstack/ara master: Fix logic inside the treeview unit test  https://review.openstack.org/46159702:48
openstackgerritDavid Moreau Simard proposed openstack/ara master: Remove dead functions/code  https://review.openstack.org/46157602:49
openstackgerritDavid Moreau Simard proposed openstack/ara master: Tidy up HTML and get rid of needless whitespace  https://review.openstack.org/46155102:49
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add unit tests for CLI: playbook delete  https://review.openstack.org/46157502:49
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add unit tests for CLI: playbook delete  https://review.openstack.org/46157502:57
*** iceyao has quit IRC03:03
*** iceyao has joined #ara03:22
dmsimardI don't know who I was discussing this with03:25
dmsimardbut ARA has this problem where it didn't really save much for tasks with loops involved, it'd write "all items succeeded" and that was it03:25
*** iceyao has quit IRC03:26
openstackgerritMerged openstack/ara master: Fix logic inside the treeview unit test  https://review.openstack.org/46159703:28
openstackgerritMerged openstack/ara master: Remove dead functions/code  https://review.openstack.org/46157603:28
openstackgerritMerged openstack/ara master: Add unit tests for CLI: playbook delete  https://review.openstack.org/46157503:28
openstackgerritMerged openstack/ara master: Tidy up HTML and get rid of needless whitespace  https://review.openstack.org/46155103:29
openstackgerritDavid Moreau Simard proposed openstack/ara master: WIP: Properly store task results provided from loops  https://review.openstack.org/46161204:02
openstackgerritDavid Moreau Simard proposed openstack/ara master: WIP: Properly store task results provided from loops  https://review.openstack.org/46161204:03
*** iceyao has joined #ara05:06
*** iceyao has quit IRC05:17
*** iceyao has joined #ara05:33
*** iceyao has quit IRC05:37
*** iceyao has joined #ara06:13
*** iceyao_ has joined #ara06:22
*** iceyao has quit IRC06:23
*** iceyao_ has quit IRC08:09
*** iceyao has joined #ara08:17
*** iceyao has quit IRC09:57
*** iceyao has joined #ara10:20
*** iceyao has quit IRC10:24
*** iceyao has joined #ara11:21
dmsimardlarsks: found another obscure ansible bug with ara :(13:36
larsksdmsimard: awesome!13:36
larsksIs it my fault? :)13:36
larsksOh wait, you said in ansible.13:36
dmsimardlarsks: no, look at this and tell me what's the problem https://gist.github.com/dmsimard/cc39f4f5283ae31ce9e2ab0cb0b73fc013:36
larsksWhat exactly is the problem there?13:38
dmsimardlarsks: sometimes the task closes before the last item closes13:38
dmsimardin that gist, you see one output with two registered items, the other with three13:38
larsksRight. I saw that, but wanted to be sure I was looking at the right thing.13:39
larsksIs this with 2.3.0?13:39
dmsimardyeah13:39
dmsimardI got the expected "it's not a bug it's a feature" from upstream :)13:39
larsksDid they have any more details?  I mean, it looks like some sort of asynchronous execution happening, but you would think that the runner_on_ok method would only be called when the task had completed execution.13:41
dmsimardyeah, it's an async tradeoff13:41
larsksBut...I am confused as to how ansible can determine if the task was successful or not without waiting for all hosts to complete.13:42
*** iceyao has quit IRC13:44
dmsimardlarsks: the task status is determined upstream in the task engine, not from the callback13:46
dmsimardbut.. I don't know13:47
larsksRight, but it's calling the runner_on_ok method...which means it has already determined the task result.13:47
*** openstackgerrit has quit IRC13:48
dmsimardlarsks: shrug, I'll file a formal issue and see how we can work around it in ara -- https://review.openstack.org/#/c/461612/ simply won't work out of the box13:58
dmsimardhttp://logs.openstack.org/12/461612/2/check/gate-ara-integration-latest-centos-7/1d4a204/logs/build/result/ad6ef9ef-8d27-4386-9e31-006ef47ec8a1/ there should be 3 items here13:58
dmsimardfor callbacks that just print stuff, it's not so much an issue .. you just get lines that get printed a bit out of order13:58
dmsimardlarsks: created https://github.com/ansible/ansible/issues/2420714:08
*** iceyao has joined #ara14:11
*** iceyao has quit IRC14:16
*** openstackgerrit has joined #ara14:25
openstackgerritDavid Moreau Simard proposed openstack/ara master: Properly store task results provided from loops  https://review.openstack.org/46161214:25
*** iceyao has joined #ara14:30
*** iceyao has quit IRC14:41
*** hughsaunders has quit IRC14:43
*** dmsimard has quit IRC14:44
*** jmccrory has quit IRC14:45
*** dmsimard has joined #ara14:46
*** ChanServ sets mode: +o dmsimard14:48
*** hughsaunders has joined #ara14:50
*** jmccrory has joined #ara14:51
openstackgerritMerged openstack/ara master: Properly store task results provided from loops  https://review.openstack.org/46161215:00
*** iceyao has joined #ara16:40
*** los has joined #ara16:41
*** iceyao has quit IRC16:47
losHI @dmsimard, first of all, thanks for creating this project, it's been quite helpful for us.  As a side note, have you considered the ability to filter results based on tags and have the tasks stats change based on the filter?16:47
losI am thinking of the use case where I have a playbook that have tons of tasks, but there are a few critical ones.  I could tag the tasks in a way to label them critical and being able to narrow down the ARA report to only those16:49
dmsimardlos: hi o/16:51
dmsimardlos: so ARA does not (yet) do anything with tags in general right now -- there is https://storyboard.openstack.org/#!/story/2000747 that mentions ARA should at least save and display what tags were used for a playbook run16:52
dmsimardlos: you have a really good idea on the task tagging, though, I'll have to write it down16:53
*** femto_ has quit IRC16:53
dmsimardI'm not sure if the task's tags are made available in the context where ara is running, I'll need to check16:55
dmsimardtask tags are totally available inside the task metadata16:57
losI'm sure you have a tons of enhancements on the pipeline, this would be a pretty cool one from our point of view :)16:59
dmsimardI kind of want to do it now16:59
dmsimardcause it's an easy addition16:59
losoh man, I thought posing this enhancement was going to be somewhat of a long shot, thanks!17:01
*** harlowja has quit IRC17:04
dmsimardlos: ARA is mostly developed out of user feedback17:19
dmsimardwhether that my own feedback or others :p17:19
*** iceyao has joined #ara17:27
*** iceyao has quit IRC17:32
dmsimardlos: so it's a package deal, you'll get a bit more than what you asked for, apologies17:38
los@dmsimard: even better! what do you have in mind?17:46
dmsimardscreenshot soon17:47
dmsimardlos: just a poc implementation http://i.imgur.com/HCz8bT9.png17:52
dmsimardand task tags will be available and searchable17:53
dmsimardProbably going to give the playbook metadata it's own panel17:53
lossweet! that was fast, just made my day17:55
dmsimardIt will land in 0.13 (hopefully this week)17:56
dmsimard0.13 was supposed to be a small release but there's actually a bunch of stuff now17:57
losI'll keep an eye on it, thanks again!17:58
*** harlowja has joined #ara18:00
*** iceyao has joined #ara21:18
*** iceyao has quit IRC21:23
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add playbook parameters and metadata, handle task tags  https://review.openstack.org/46190721:27
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add playbook parameters and metadata, handle task tags  https://review.openstack.org/46190721:27
dmsimardlos: ^ I'll send you a link when there's a report generated in-gate21:28
dmsimardlos: http://logs.openstack.org/07/461907/2/check/gate-ara-integration-latest-centos-7/672e715/logs/build/reports/index.html21:32
dmsimardSee the parameter tab21:32
dmsimardand here: http://logs.openstack.org/07/461907/2/check/gate-ara-integration-latest-centos-7/672e715/logs/build/reports/9c23eba4-e5de-4543-822d-e43ab13f5b4a.html21:32
dmsimardIf you search for "tag" (or "untag" or "verytag") in the task search box you'll get the appropriate tasks -- tags can be seen in a tooltip if you hover the task name and inside the task result popup too.21:33
*** iceyao has joined #ara21:42
*** iceyao has quit IRC21:47
*** iceyao has joined #ara22:02
rattboioh dmsimard! Is that a permalink to a playbook now?22:04
rattboiI must have missed that22:04
*** iceyao has quit IRC22:06
dmsimardrattboi: yeah, permalinks are coming back in the next version22:17
rattboiawesome :)22:17
dmsimardPlaybooks have their permalink and anything in a popup too (file, task result, hosts, etc)22:17
rattboiis there a tl;dr version of what the solution was that didn't kill static compiling time?22:18
dmsimardtl;dr it's not bad at the scale at which the static generation is aimed at22:18
dmsimardIf you're conplaining that static generation takes too long and is too heavy and you're generating data for 20 000 playbooks I'll tell you to use the dynamic version22:19
rattboimakes sense to me22:20
dmsimardThere's data in the commit message here: https://github.com/openstack/ara/commit/8c5f84827541851c07f0e05ee3edd3216a61a57422:20
dmsimardI didn't include generation time because it was essentially the same, for that particular dataset it's around 45 seconds on my laptop22:21
dmsimardThe amount of permalinks in the previous version of the UI was just too much. The new UI works around that in different clever ways (popups, ajax, etc)22:22
*** iceyao has joined #ara23:07
*** iceyao has quit IRC23:12
*** myoung has joined #ara23:28
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add playbook parameters and metadata, handle task tags  https://review.openstack.org/46190723:50

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