| *** openstackgerrit has quit IRC | 01:30 | |
| *** openstackstatus has quit IRC | 04:35 | |
| *** openstackstatus has joined #ara | 04:35 | |
| *** ChanServ sets mode: +v openstackstatus | 04:35 | |
| *** sshnaidm|pto is now known as sshnaidm | 07:06 | |
| *** evrardjp has quit IRC | 08:18 | |
| *** evrardjp has joined #ara | 08:19 | |
| *** themroc has joined #ara | 08:29 | |
| *** gvincent has joined #ara | 08:32 | |
| *** irclogbot_3 has joined #ara | 13:02 | |
| *** altlogbot_2 has joined #ara | 13:06 | |
| *** openstackgerrit has joined #ara | 14:04 | |
| openstackgerrit | David Moreau Simard proposed openstack/ara master: Update integration tests to use the latest Ansible releases https://review.openstack.org/651541 | 14:04 |
|---|---|---|
| *** openstackgerrit has quit IRC | 14:14 | |
| *** gvincent has quit IRC | 14:17 | |
| *** etienne has joined #ara | 14:44 | |
| *** openstackgerrit has joined #ara | 14:50 | |
| openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Fix typos in the paths returned by ara.setup https://review.openstack.org/651568 | 14:50 |
| openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: WIP: Add support for API authentication in the client https://review.openstack.org/651569 | 14:50 |
| openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: WIP: Add support for API authentication in the client https://review.openstack.org/651569 | 14:50 |
| dmsimard | apollo13: ^ tests aren't written yet but tested it and it works | 14:51 |
| dmsimard | need to figure out the best approach for unit and integration tests (i.e, create a user first, set write/read required, etc.) | 14:51 |
| *** gvincent has joined #ara | 15:06 | |
| *** themroc has quit IRC | 15:15 | |
| openstackgerrit | David Moreau Simard proposed openstack/ara master: Update integration tests to use the latest Ansible releases https://review.openstack.org/651541 | 15:49 |
| *** mtf8 has left #ara | 15:54 | |
| *** altlogbot_2 has quit IRC | 16:45 | |
| *** spiette has quit IRC | 17:12 | |
| *** spiette has joined #ara | 17:18 | |
| openstackgerrit | Merged openstack/ara master: Update integration tests to use the latest Ansible releases https://review.openstack.org/651541 | 17:46 |
| openstackgerrit | Merged openstack/ara feature/1.0: Fix typos in the paths returned by ara.setup https://review.openstack.org/651568 | 17:46 |
| *** spiette has quit IRC | 17:50 | |
| *** spiette has joined #ara | 17:55 | |
| openstackgerrit | David Moreau Simard proposed openstack/ara-web master: Update dependencies and move from patternfly-next to patternfly https://review.openstack.org/651620 | 18:09 |
| -openstackstatus- NOTICE: Restarting Gerrit on review.openstack.org to pick up new configuration for the replication plugin | 19:03 | |
| ara-slack | bhuiyanallen: @bhuiyanallen has joined the channel | 19:48 |
| apollo13 | dmsimard: ah yes, those changes look, uhm nice, the implementation is rather hacky, but I guess that is what django allows for XD | 20:06 |
| apollo13 | dmsimard: re tests, is a good question -- I'll think about it (currently at djangocon) | 20:19 |
| dmsimard | apollo13: np, have fun ! django has facilities for testing with authentication with the test client -- the ara clients don't have tests right now so I'll try to come up with something at least basic | 20:22 |
| apollo13 | dmsimard: mhm, wondering how that will work for ansible-integration :/ | 20:23 |
| dmsimard | can have the integration tests create a user prior and then use it in authentication | 20:24 |
| apollo13 | yeah but how about cleanup and teardown | 20:24 |
| apollo13 | I think the best option (tm) would be to write a custom test subclass that internally then calls ansible-playbook | 20:24 |
| apollo13 | is there currently a api thing so one can call ansible programtically? | 20:25 |
| apollo13 | ansible-runner or so there was in the past? | 20:25 |
| dmsimard | ansible-runner | 20:25 |
| dmsimard | apollo13: right now there is this: https://github.com/openstack/ara/blob/feature/1.0/roles/ara_tests/tasks/main.yaml#L73 | 20:25 |
| dmsimard | ansible-runner doesn't work the same for callback plugins, it's something I want to look into at some point though | 20:26 |
| apollo13 | mhm okay, so just subprocess mit check_call then | 20:27 |
| apollo13 | something for the sprints during saturday/sunday | 20:27 |
| apollo13 | mit == with | 20:28 |
| apollo13 | my german leaks | 20:28 |
| apollo13 | left a few comments on the auth stuff, hope it makes sense | 20:29 |
| apollo13 | curious, is there a way to tell https://review.openstack.org/#/q/project:openstack/ara to show only open reviews by default? | 20:35 |
| apollo13 | guess basically everything which has status in open, pending, reviewed or draft | 20:36 |
| dmsimard | Add status:open to your query | 21:50 |
| dmsimard | There's a lot of different query examples in the dash files here: https://github.com/openstack/gerrit-dash-creator/tree/master/dashboards | 21:52 |
| *** etienne has quit IRC | 21:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!