| *** dougbtv has quit IRC | 01:02 | |
| *** gvincent has joined #ara | 07:52 | |
| *** themroc has joined #ara | 08:11 | |
| *** ssbarnea|rover has joined #ara | 08:54 | |
| *** herald85 has joined #ara | 08:57 | |
| *** herald85 has quit IRC | 10:07 | |
| *** sshnaidm|off is now known as sshnaidm | 10:27 | |
| ara-slack | guillaume.lunik: @guillaume.lunik has joined the channel | 15:09 |
|---|---|---|
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: WIP Move playbook details page in a dedicated page https://review.openstack.org/628451 | 15:28 |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Fix 404 config.json issue https://review.openstack.org/630704 | 15:28 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 15:30 |
| *** herald85 has joined #ara | 15:39 | |
| openstackgerrit | Merged openstack/ara-plugins master: Implement support for naming playbooks https://review.openstack.org/629927 | 15:49 |
| openstackgerrit | Merged openstack/ara-plugins master: Implement support for setting host aliases https://review.openstack.org/629928 | 15:49 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 15:50 |
| openstackgerrit | Merged openstack/ara-web master: Fix 404 config.json issue https://review.openstack.org/630704 | 15:58 |
| *** etienne has joined #ara | 16:05 | |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Move playbook details page in a dedicated page https://review.openstack.org/628451 | 16:12 |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Move playbook details page in a dedicated page https://review.openstack.org/628451 | 16:19 |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Move playbook details page in a dedicated page https://review.openstack.org/628451 | 16:21 |
| *** themroc has quit IRC | 16:46 | |
| *** herald85 has quit IRC | 16:52 | |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Move playbook details page in a dedicated page https://review.openstack.org/628451 | 19:18 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 19:35 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-server master: Serialize the value provided by ara_record https://review.openstack.org/629946 | 19:38 |
| dmsimard | gvincent: added a comment in https://review.openstack.org/#/c/628451/ | 19:56 |
| gvincent | dmsimard, | 19:57 |
| gvincent | Looking at the API, I see that we don't currently have support for "/api/v1/results?task=<:id>" so I'll add that. | 19:57 |
| gvincent | this will not be interesting | 19:57 |
| gvincent | I need to get all tasks for a given playbook | 19:57 |
| dmsimard | gvincent: tasks and results already support a playbook param | 19:58 |
| dmsimard | do you need something else ? | 19:58 |
| gvincent | if for every task I need to get results, it will be too long to display some informations | 19:58 |
| dmsimard | yes, agree | 19:58 |
| gvincent | ok so I will get results?playbook=... then | 19:58 |
| dmsimard | results are likely the biggest possible calls | 19:59 |
| dmsimard | hmm | 19:59 |
| gvincent | I see that | 20:00 |
| dmsimard | if you do a call to get all tasks and then a call to get all results | 20:00 |
| dmsimard | you need something to associate the results back to the tasks ? | 20:00 |
| dmsimard | results are "children" of tasks | 20:00 |
| gvincent | there is one result per host right? | 20:01 |
| gvincent | for a given task | 20:01 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 20:01 |
| dmsimard | yes | 20:01 |
| dmsimard | one task that runs on one host: one result, one task taht runs on 10 hosts: 10 results | 20:01 |
| gvincent | in ara 0.x we get all results for a given playbook when we click on tasks | 20:01 |
| dmsimard | yes, it's loaded more or less asynchronously | 20:02 |
| gvincent | so I will do my POC on ara v1 with /results | 20:03 |
| gvincent | /results?playbook=:id&status=failed | 20:03 |
| dmsimard | yeah would work, both playbook and status are available for search | 20:03 |
| gvincent | IMO result.content should only be on results/:id | 20:04 |
| dmsimard | +1 | 20:04 |
| dmsimard | same thing for file content | 20:04 |
| gvincent | /results should also have nested host | 20:04 |
| gvincent | instead of host id | 20:05 |
| dmsimard | sure | 20:06 |
| gvincent | dmsimard, do you want me tomorrow to work on /playbooks on the api? | 20:07 |
| gvincent | to get the nb_hosts, nb_plays, nb_files, nb_tasks and nb_records | 20:07 |
| gvincent | and remove files arguments and hosts ? | 20:08 |
| gvincent | nb_arguments* | 20:08 |
| dmsimard | gvincent: I was writing it down but by all means feel free to pick that up -- I'm trying to standardize the integration tests between 0.x and 1.x because ara-plugins and ara-clients aren't tested right now -- it's also the same test suite I'll use to populate the demo that I also need to update automatically | 20:09 |
| gvincent | ack I will work on this, except if apollo13 want to work on this | 20:10 |
| * apollo13 is swamped with other work | 20:10 | |
| gvincent | np | 20:10 |
| apollo13 | but at least I am working with ansible :) | 20:11 |
| gvincent | I will do it | 20:11 |
| dmsimard | fwiw another thing I have to do is write a blog post for a general status update and formalize the short term backlog better than in an etherpad | 20:11 |
| dmsimard | we have other potential contributors that are interested but docs and the to-do list are lacking | 20:11 |
| dmsimard | good news is that integration test work has picked up an actual bug so that's great :p | 20:14 |
| gvincent | \o/ | 20:14 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-server master: Serialize the value provided by ara_record https://review.openstack.org/629946 | 20:15 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 20:18 |
| dmsimard | apollo13: no hard feelings <3 | 20:20 |
| dmsimard | In completely unrelated news, the fact that ara 0.x is helping me troubleshoot the 1.0 integration jobs is just awesome | 20:23 |
| apollo13 | haha | 20:24 |
| dmsimard | http://logs.openstack.org/03/630303/6/check/ara-integration-ubuntu-2.6.11/418bc1f/ara-report/ | 20:24 |
| dmsimard | That's a report of Zuul running 0.x integration tests on 1.0 (with a minimal delta) | 20:25 |
| dmsimard | oh yeah there's another bug I need to figure out.. gotta write it down | 20:26 |
| dmsimard | there's something weird when the callback posts stats at the end | 20:27 |
| apollo13 | apropros callbacks and so | 20:33 |
| apollo13 | is there a way to disable play recap? | 20:33 |
| apollo13 | also the unixy stdout_callback is nice | 20:34 |
| dmsimard | not sure | 20:34 |
| apollo13 | oh I just need to override https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/callback/unixy.py#L168 | 20:36 |
| dmsimard | You'd need to carry a patched version in tree | 20:43 |
| dmsimard | Overriding it in another callback isn't going to prevent that from running | 20:44 |
| dmsimard | Actually since it's a stdout callback, there can only be one | 20:44 |
| apollo13 | dmsimard: yes, I am fine with putting it into my source | 20:47 |
| apollo13 | dmsimard: that said, "patching" is relative, I can import it and just override that method, so it is rather clean | 20:48 |
| dmsimard | import ? | 20:55 |
| apollo13 | sure | 21:00 |
| apollo13 | from ansible.plugins.callback import unixy | 21:00 |
| apollo13 | and then class CallbackModule(unixy.CallbackModule): and override the functions as needed | 21:00 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 21:02 |
| dmsimard | apollo13: you can do that at runtime ? | 21:03 |
| dmsimard | I guess I never tried | 21:03 |
| apollo13 | well it is just python | 21:03 |
| dmsimard | yeah but by the time you try to patch it, is it not already loaded in another memory context ? | 21:04 |
| apollo13 | I am not trying to patch the unixy plugin | 21:04 |
| apollo13 | I set stdout_callback to my_plugin | 21:04 |
| dmsimard | ohhhhh | 21:04 |
| apollo13 | and then create callback_plugins/my_plugin.py in my playbook folder | 21:04 |
| dmsimard | sure | 21:04 |
| apollo13 | but it is "nice" in the sense that I do not have much duplication | 21:04 |
| openstackgerrit | Merged openstack/ara-server master: Serialize the value provided by ara_record https://review.openstack.org/629946 | 22:56 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: WIP: New ARA 1.0 integration jobs https://review.openstack.org/630303 | 23:02 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!