| *** themroc has joined #ara | 07:30 | |
| *** ssbarnea|bkp2 has quit IRC | 11:01 | |
| *** ssbarnea|rover has joined #ara | 11:02 | |
| *** herald85 has joined #ara | 13:36 | |
| openstackgerrit | Guillaume Vincent proposed openstack/ara-web master: Display results instead of task for a given playbook https://review.openstack.org/633749 | 14:34 |
|---|---|---|
| *** sshnaidm is now known as sshnaidm|mtg | 15:04 | |
| dmsimard | apollo13, gvincent: I'm questioning the way our repositories and projects are set up -- do you have time to chat ? I created an etherpad to start the discussion: https://etherpad.openstack.org/p/ara-1.0-repository-structure | 15:07 |
| gvincent | available in 30min | 15:07 |
| apollo13 | " We need to think about the migration path for users restricted to py2" <-- can't that just be "keep using ara 0?" | 15:08 |
| dmsimard | yes and no | 15:09 |
| gvincent | py2 is end of life at the end of the year | 15:09 |
| dmsimard | we're not going to maintain 0.x forever | 15:09 |
| dmsimard | 0.x works with py3 | 15:09 |
| apollo13 | okay, differently put; which environments are py2 only? not that we are talking about controller machines etc here | 15:09 |
| apollo13 | not target machines where you run ansible | 15:10 |
| apollo13 | expecting a user to use rhel 7 to get py3.6 from epel doesn't sound to bad | 15:10 |
| dmsimard | 3.6 isn't in epel | 15:10 |
| dmsimard | I think epel is 3.4 | 15:10 |
| apollo13 | epel __has__ python36 as package | 15:11 |
| apollo13 | unless you and I have a different EPEL *hmm* | 15:11 |
| dmsimard | hmmm | 15:11 |
| apollo13 | for a few months now I think | 15:11 |
| dmsimard | so the fact that epel has 3.4 and 3.6 doesn't really matter | 15:12 |
| dmsimard | I mean | 15:12 |
| dmsimard | it actually has both | 15:12 |
| dmsimard | so I stand corrected | 15:12 |
| dmsimard | I haven't even tried running 1.0 on centos | 15:13 |
| apollo13 | also note that rhel 8 will be released "soon", probably before ara 0 goes EOL and then you have a good platform with core support for py3 | 15:15 |
| dmsimard | apollo13: yes, I'm not too worried about that part | 15:18 |
| dmsimard | I guess the question I am trying to solve is: should we just merge everything back into the one "openstack/ara" repository or not ? | 15:18 |
| apollo13 | personally I feel it would make some stuff easier (ie linter config etc…) packaging might get fun though | 15:19 |
| dmsimard | I think ara-web is an exception and should stay separate | 15:19 |
| dmsimard | but I am really questioning myself about ara-plugins, ara-server and ara-clients | 15:19 |
| apollo13 | but you still want to be able to install separately I assume? | 15:19 |
| dmsimard | yes, I actually haven't gotten around to testing that "pip install ara.clients"-like syntax | 15:20 |
| apollo13 | there is no such thing as pip install ara.clients | 15:21 |
| apollo13 | "." gets normalized to "-", so it just installs ara-clients ™ | 15:21 |
| dmsimard | I remember reading about it but that's the extent of what I know | 15:21 |
| apollo13 | https://dpaste.de/HYzQ/raw | 15:21 |
| apollo13 | and off from work, bbl | 15:24 |
| *** sshnaidm|mtg is now known as sshnaidm | 15:33 | |
| dmsimard | gvincent: for the failure in https://review.openstack.org/#/c/633749/ I was looking at https://facebook.github.io/create-react-app/docs/troubleshooting#npm-run-build-fails-to-minify | 15:40 |
| dmsimard | so I updated react-scripts to >=2.0.0 and npm install/npm_start seems to work fine but it also updated package-lock.json quite a bit. Should we always commit package-lock.json ? | 15:41 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-web master: Update react-scripts to >=2.0.0 https://review.openstack.org/633764 | 15:44 |
| dmsimard | 16k lines changed in package-lock.json | 15:44 |
| dmsimard | if the jobs pass we can rebase your patch on top of it | 15:45 |
| *** openstackgerrit has quit IRC | 15:51 | |
| gvincent | @dmsimard, it will work | 15:51 |
| gvincent | we need react-scripts > 2 yeah | 15:52 |
| dmsimard | ok I'll rebase your patch | 15:55 |
| *** openstackgerrit has joined #ara | 15:55 | |
| openstackgerrit | David Moreau Simard proposed openstack/ara-web master: Display results instead of task for a given playbook https://review.openstack.org/633749 | 15:55 |
| gvincent | IMO mono repo is the best :D | 15:58 |
| dmsimard | gvincent, apollo13: I fleshed out the questions in https://etherpad.openstack.org/p/ara-1.0-repository-structure a bit | 15:59 |
| gvincent | available for a quick meeting if you want | 15:59 |
| dmsimard | gvincent: I don't really mind mono repo, it's more about how to make the components installable individually | 15:59 |
| gvincent | hum I don't have the solution | 16:04 |
| gvincent | containers and rpm/deb ? | 16:05 |
| dmsimard | I'll do some research | 16:11 |
| gvincent | for LessPass I choose to produce well tested container through docker hub | 16:21 |
| gvincent | but because we are Red Hat you will problably need to produce some rpms | 16:21 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-web master: Update react-scripts to >=2.0.0 https://review.openstack.org/633764 | 16:21 |
| openstackgerrit | David Moreau Simard proposed openstack/ara-web master: Ensure we run unit and integration tests for relevant files https://review.openstack.org/633777 | 16:21 |
| dmsimard | gvincent: ara isn't a product that red hat sells though :p | 16:22 |
| dmsimard | It's important to consider distribution packages, though | 16:23 |
| gvincent | yeah, I'm not relevant to answer those questions but I'm interested in the solution :D | 16:29 |
| dmsimard | gvincent: I had to do another rebase to fix an issue with the jobs -- can you review/approve https://review.openstack.org/#/c/633777/ and https://review.openstack.org/#/c/633764/ ? | 16:37 |
| dmsimard | for your patch, the draft is ready: http://logs.openstack.org/49/633749/2/check/ara-web-build-dashboard/2b3ae6a/npm/html/ | 16:38 |
| dmsimard | I like where this is going and see the need for improvements on the API | 16:40 |
| dmsimard | gvincent: should the dropdown be checkboxes for each status instead of a dropdown ? | 16:41 |
| dmsimard | in order to display, for example, failed and changed tasks simultaneously | 16:42 |
| *** themroc has quit IRC | 16:57 | |
| dmsimard | gvincent: left comments in https://review.openstack.org/#/c/633749/, I'll be back after lunch | 16:58 |
| *** herald85 has quit IRC | 17:57 | |
| *** harlowja has joined #ara | 19:44 | |
| *** gvincent has quit IRC | 21:52 | |
| *** gvincent has joined #ara | 21:52 | |
| -openstackstatus- NOTICE: http://zuul.openstack.org is not working. https://zuul.openstack.org does work. Please use that while we investigate. | 23:12 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!