| *** sshnaidm has quit IRC | 00:44 | |
| *** sshnaidm has joined #ara | 01:50 | |
| *** altlogbot_1 has quit IRC | 03:44 | |
| *** altlogbot_1 has joined #ara | 03:44 | |
| *** altlogbot_1 has quit IRC | 04:38 | |
| *** altlogbot_3 has joined #ara | 04:38 | |
| *** altlogbot_3 has quit IRC | 04:40 | |
| *** altlogbot_2 has joined #ara | 04:40 | |
| *** herald85 has joined #ara | 07:04 | |
| *** gvincent has quit IRC | 07:44 | |
| *** idir__ has joined #ara | 08:10 | |
| *** idir__ has quit IRC | 08:10 | |
| *** sshnaidm has quit IRC | 08:30 | |
| *** sshnaidm has joined #ara | 08:31 | |
| *** jungleslow has joined #ara | 09:24 | |
| *** gvincent has joined #ara | 09:29 | |
| *** gvincent has quit IRC | 09:30 | |
| *** gvincent has joined #ara | 09:31 | |
| *** vincent-- has joined #ara | 09:56 | |
| *** vincent- has quit IRC | 09:59 | |
| *** idir__ has joined #ara | 10:26 | |
| idir__ | hi, i setup authentication for ara-api (ara version 1.0.0.0b2), and my problem is: the interface does not get data from the api because of authentication. how can we solve this? | 10:28 |
|---|---|---|
| *** idir__ has quit IRC | 10:37 | |
| openstackgerrit | Guillaume Vincent proposed recordsansible/ara-web master: Differentiate between no playbooks from api server unreachable https://review.opendev.org/661940 | 11:06 |
| gvincent | dmsimard, https://review.opendev.org/#/c/661940/ | 11:07 |
| gvincent | dmsimard, CORS is still not allowed on https://api.demo.recordsansible.org/ | 11:08 |
| *** dbpiv has quit IRC | 12:08 | |
| *** idir__ has joined #ara | 12:19 | |
| idir__ | hi, i setup ara version 1.0.0.0 and activate the authentication in ara-api, so the problem is : the interface of ara (ara-web) can not get data (401), how can i solve this ? thank you | 12:25 |
| idir__ | is there any configuration in ara-web to solve this problem ? | 12:36 |
| dmsimard | idir__: authentication is not yet supported in ara-web | 12:41 |
| dmsimard | https://github.com/ansible-community/ara-web/issues/1. | 12:41 |
| openstackgerrit | David Moreau Simard proposed recordsansible/ara-infra master: Allow all CORS origins for the API live demo https://review.opendev.org/661969 | 12:48 |
| dmsimard | gvincent: fixed ^ sorry | 12:48 |
| idir__ | dmsimard: ok thank you :) | 12:53 |
| idir__ | i added a task in database.potgresql to automate the creation of the django surper user, and also some variables for that and i can share it with you if you whant | 12:56 |
| gvincent | dmsimard, how can I know if the server is protected? | 12:56 |
| idir__ | want | 12:56 |
| dmsimard | gvincent: protected ? by a password ? | 12:57 |
| gvincent | yes for the ui do you want me to catch any 401 api error and redirect to a login page? | 12:57 |
| dmsimard | idir__: would definitely appreciate if you could send those patches for the roles ! | 12:58 |
| dmsimard | gvincent: that would make sense I guess. Only show a login page if the remote server requires authentication ? | 12:58 |
| dmsimard | I don't know if django returns any headers or something, let me set it up | 12:59 |
| gvincent | yes but there is no way for me to check if the api is protected except try and fail right? | 12:59 |
| dmsimard | gvincent: that's what I'm going to double check | 12:59 |
| gvincent | dmsimard, I was more thinking about a variable in the config | 13:00 |
| dmsimard | gvincent: a variable for what, showing the login screen ? | 13:00 |
| gvincent | no variable saying this ara-web is for a protected api | 13:01 |
| idir__ | if you set up the authentication django will send 401 error | 13:01 |
| gvincent | apiUrl: ..., isApiProtected: true | 13:01 |
| gvincent | but let's make the interface generic | 13:01 |
| gvincent | dmsimard, is /api/v1/ protected when the server is protected? | 13:03 |
| gvincent | idir__, ^ | 13:03 |
| dmsimard | gvincent: http://api.trunk.demo.recordsansible.org && http://web.trunk.demo.recordsansible.org | 13:04 |
| dmsimard | set up with read_protected and write_protected | 13:04 |
| idir__ | yes it is | 13:04 |
| dmsimard | haven't set up a user yet, hang on | 13:04 |
| gvincent | perfect | 13:04 |
| dmsimard | gvincent: "/" could probably be not protected | 13:04 |
| dmsimard | would that help ? | 13:04 |
| gvincent | dmsimard, I will check at startup /api/v1/ if 401 I will redirect the user to the login page | 13:05 |
| gvincent | it helps thank you | 13:05 |
| dmsimard | also btw I added a small comment in https://review.opendev.org/#/c/661940/ | 13:05 |
| idir__ | when you set to true the to variables i think all is protected | 13:05 |
| gvincent | I can work on this | 13:05 |
| gvincent | dmsimard, I didn't want to include html in the page to avoid xss issues | 13:06 |
| gvincent | html in the message | 13:06 |
| openstackgerrit | Merged recordsansible/ara-web master: Differentiate between no playbooks from api server unreachable https://review.opendev.org/661940 | 13:09 |
| idir__ | dmsimard: how can i sen patches for the roles | 13:12 |
| idir__ | ? | 13:12 |
| dmsimard | idir__: I hope to be able to open up pull requests in the near future but in the meantime you need to send your patches to gerrit which involves setting up a few things that I'm happy to help with | 13:13 |
| dmsimard | There are (now slightly outdated) docs here https://ara.readthedocs.io/en/stable/contributing.html but I had made a tl;dr version here https://github.com/ansible-community/ara/pull/12#issuecomment-433119539 | 13:14 |
| gvincent | dmsimard, I would like to rename container to page in the js app | 13:19 |
| gvincent | is it something I can do ? | 13:19 |
| idir__ | @dmsimard i'll try gerrit, I will write to you later. it's ok for you ? | 13:21 |
| dmsimard | idir__: sure thing | 13:21 |
| dmsimard | gvincent: you mean like the file names ? | 13:21 |
| gvincent | yes | 13:21 |
| dmsimard | sure, go for it | 13:21 |
| dmsimard | do what you gotta do | 13:21 |
| idir__ | dmsimard: you sed "Clone the repo" which repo ? the one in github ? | 13:49 |
| idir__ | https://github.com/ansible-community/ara | 13:49 |
| idir__ | this one https://github.com/ansible-community/ara/tree/1.0.0b2 | 13:50 |
| dmsimard | idir__: that's the same repo but different branches | 13:52 |
| dmsimard | idir__: if you want to send a patch on top of 1.0, you need to checkout the feature/1.0 branch | 13:53 |
| idir__ | yes ^^ | 13:53 |
| dmsimard | git clone https://github.com/ansible-community/ara; cd ara; git checkout feature/1.0 | 13:53 |
| idir__ | after chnages i have to commit and git review | 13:54 |
| dmsimard | idir__: yup, that's it | 13:55 |
| dmsimard | if you've set up your ssh key and username in gerrit | 13:55 |
| idir__ | yes i do | 13:55 |
| dmsimard | the thing that tends to cause issues is if your local ssh user is different than the one set up in gerrit | 13:55 |
| dmsimard | for example if your local username is "user" and the gerrit user is "foo", it will try to authenticate as user@review.opendev.org and that wouldn't work since it needs to be foo@review.opendev.org | 13:55 |
| dmsimard | if it's different, you can set it up in your .ssh/config file like this: http://paste.openstack.org/show/752247/ | 13:57 |
| idir__ | yes it's normal, know i'm not on my personal machine, i will create another key from mine and add it to opendev | 13:57 |
| idir__ | i added Redhat version in vars with some changes in tasks i will add them | 14:00 |
| idir__ | i'm very excited to contribute to this project :) | 14:00 |
| dmsimard | idir__: and I'm very happy that you are able to contribute :D | 14:01 |
| openstackgerrit | David Moreau Simard proposed recordsansible/ara feature/1.0: Port contribution docs from 0.x and update for 1.0 https://review.opendev.org/661992 | 14:03 |
| dmsimard | idir__: ^ a first iteration of the contribution docs updated for 1.0 | 14:04 |
| idir__ | i've never used opendev, so i'll take the time to see how it works :p | 14:11 |
| *** etienne has quit IRC | 14:38 | |
| *** dbpiv has joined #ara | 14:42 | |
| openstackgerrit | David Moreau Simard proposed recordsansible/ara feature/1.0: Port FAQ docs from 0.x and update for 1.0 https://review.opendev.org/662010 | 14:54 |
| gvincent | David is there an endpoint for the api to get a short lived token? | 14:58 |
| gvincent | or should I store the user credentials locally? | 14:58 |
| dmsimard | apollo13: ^ | 14:59 |
| dmsimard | gvincent: asking the expert | 14:59 |
| dmsimard | I don't think there's anything of the like right now but perhaps https://www.django-rest-framework.org/api-guide/authentication/#tokenauthentication could help | 15:00 |
| dmsimard | storing the credentials locally sounds like a good first iteration | 15:01 |
| gvincent | in term of security this is crapy :D | 15:01 |
| gvincent | JWT on DRF is one module to add :D | 15:01 |
| dmsimard | gvincent: this one ? https://github.com/davesque/django-rest-framework-simplejwt | 15:04 |
| dmsimard | looks simple enough but out of scope for 1.0, can be 1.1 :D | 15:06 |
| openstackgerrit | Guillaume Vincent proposed recordsansible/ara-web master: Fix error handler in Playbooks page https://review.opendev.org/662016 | 15:06 |
| openstackgerrit | Guillaume Vincent proposed recordsansible/ara-web master: WIP Support authentication https://review.opendev.org/662017 | 15:06 |
| gvincent | dmsimard, WIP https://review.opendev.org/#/c/662017/ | 15:07 |
| gvincent | can you send me textContent= in https://review.opendev.org/#/c/662017/1/src/login/LoginContainer.js | 15:07 |
| gvincent | I need to go | 15:07 |
| gvincent | for know I will sotre in the localStorage password and username | 15:08 |
| gvincent | and add basic auth on every request | 15:08 |
| gvincent | tomorrow is public holiday in France dmsimard | 15:08 |
| gvincent | I will finish it on friday | 15:08 |
| gvincent | cheers! | 15:08 |
| dmsimard | gvincent: no stress, I'll try it out and review in the meantime | 15:09 |
| dmsimard | good job :) | 15:09 |
| gvincent | https://i.imgur.com/rK1aEpp.png | 15:09 |
| *** herald85 has quit IRC | 15:09 | |
| dmsimard | gvincent: very cool | 15:11 |
| dmsimard | it doesn't seem to work locally yet | 15:11 |
| openstackgerrit | Merged recordsansible/ara feature/1.0: Port contribution docs from 0.x and update for 1.0 https://review.opendev.org/661992 | 15:14 |
| dmsimard | gvincent: added comments on https://review.opendev.org/#/c/662017/ | 15:19 |
| openstackgerrit | David Moreau Simard proposed recordsansible/ara feature/1.0: Port FAQ docs from 0.x and update for 1.0 https://review.opendev.org/662010 | 15:26 |
| *** jungleslow has quit IRC | 15:27 | |
| openstackgerrit | Merged recordsansible/ara-infra master: Allow all CORS origins for the API live demo https://review.opendev.org/661969 | 15:32 |
| *** altlogbot_2 has quit IRC | 15:35 | |
| *** altlogbot_2 has joined #ara | 15:35 | |
| *** irclogbot_3 has quit IRC | 15:36 | |
| *** irclogbot_3 has joined #ara | 15:36 | |
| idir__ | dmsimard: gvincent Hi, I'll test my configuration of gerrit. Please ignore it | 15:37 |
| dmsimard | sure | 15:38 |
| openstackgerrit | Merged recordsansible/ara-web master: Fix error handler in Playbooks page https://review.opendev.org/662016 | 15:51 |
| idir__ | dmsimard: I can't use SSH for push my commit. I try to use http connection. But the url from the documentation for the generation of change-id (http://review.opendev.org/tools/hook/commit-msg) doesn't work | 15:57 |
| idir__ | oups missing s in hooks :p | 15:59 |
| openstackgerrit | djemaoune idir proposed recordsansible/ara feature/1.0: test gerrit https://review.opendev.org/662026 | 16:00 |
| openstackgerrit | djemaoune idir proposed recordsansible/ara feature/1.0: check topic fix https://review.opendev.org/662027 | 16:02 |
| dmsimard | idir__: congrats :) | 16:07 |
| dmsimard | Topics can also be set manually in the gerrit web UI after the patch was sent but not after it merged | 16:07 |
| *** ktibi has joined #ara | 16:18 | |
| ktibi | Hi guys, I works with idir__ . thx for fix in beta2, all works with ansible 2.8 | 16:18 |
| *** idir__ has quit IRC | 16:26 | |
| dmsimard | ktibi: ohai o/ | 16:26 |
| ktibi | dmsimard, We'll use the 1.0 version even if this beta doesn't have all feature for the web interface. | 16:27 |
| ktibi | I can see you work very fast so it's ok if we are beta users | 16:28 |
| ktibi | We'll try to contribute to the ansible part, but we don't have JS devs sorry ^^ | 16:29 |
| dmsimard | ktibi: I'm also not a very javascript person :p | 16:34 |
| dmsimard | 1.0 is pretty much ready for release, I've been wrapping up missing docs and writing a blog post | 16:34 |
| dmsimard | The current state of things is not perfect but it will be a good foundation that we can iterate on for future dot releases :) | 16:35 |
| ktibi | dmsimard, Yep maybe need some fix to have a web interface more friendly | 16:54 |
| ktibi | like a filter or a button for sort each line | 16:55 |
| ktibi | because I need to scroll for find my last playbook (default make the oldest in the first position) | 16:56 |
| openstackgerrit | Merged recordsansible/ara feature/1.0: Port FAQ docs from 0.x and update for 1.0 https://review.opendev.org/662010 | 17:13 |
| *** sshnaidm is now known as sshnaidm|off | 17:33 | |
| dmsimard | ktibi: some of the high level items are documented as issues in https://github.com/ansible-community/ara-web/issues | 17:55 |
| dmsimard | Feel free to weigh in if you have opinions or create new issues if you have ideas | 17:55 |
| ktibi | ok, we'll try to make that ) | 17:56 |
| *** ktibi has quit IRC | 17:57 | |
| dmsimard | Right now we are working to implement data from the API in the right places at the right time, usability and prettiness will come naturally as we iterate | 17:58 |
| *** dbpiv has quit IRC | 19:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
