Saturday, 2018-04-28

*** harlowja has quit IRC01:03
*** harlowja has joined #ara02:13
*** harlowja has quit IRC02:43
*** it_friday__ has joined #ara03:07
it_friday__THE L0DE RADIO HOUR IS LIVE HTTP://YOUTUBE.COM/L0DE/LIVE CALL IN NOW @ 315-505-4666 IRC.EFNET.ORG #LRH03:07
it_friday__ THE L0DE RADIO HOUR IS LIVE HTTP://YOUTUBE.COM/L0DE/LIVE CALL IN NOW @ 315-505-4666 IRC.EFNET.ORG #LRH03:07
it_friday__ THE L0DE RADIO HOUR IS LIVE HTTP://YOUTUBE.COM/L0DE/LIVE CALL IN NOW @ 315-505-4666 IRC.EFNET.ORG #LRH03:07
it_friday__ THE L0DE RADIO HOUR IS LIVE HTTP://YOUTUBE.COM/L0DE/LIVE CALL IN NOW @ 315-505-4666 IRC.EFNET.ORG #LRH03:07
it_friday__bcoca evrardjp hwoarang gvincent jmccrory jlozadad_ paulfantom spiette mhu tristanC jlozadad[m] DrWaluigi BlessJah jparrill_ JohnnyOSA openstackgerrit weshay sshnaidm|off ara-slack _KaszpiR_ jrist Verilium hughsaunders larsks SpamapS twouters boxrick dmsimard hiroki logan- berendt leifmadsen njohnston openstackstatus openstack mmercer myoung andymccr ssbarnea nhicher cliles mnaser miabbott Pies03:07
*** it_friday__ has left #ara03:07
*** bcoca has quit IRC03:36
*** gvincent_ has joined #ara05:38
*** gvincent has quit IRC05:38
*** sshnaidm|off has quit IRC06:17
*** gvincent_ has quit IRC07:12
*** dougbtv__ is now known as dougbtv12:22
*** sshnaidm has joined #ara12:42
*** jparrill_ has quit IRC15:24
*** jparrill_ has joined #ara15:24
hirokidmsimard: sorry, it was kingsday over here, so a bit AFK.. I've followed your instructions and tested out your code changes. However, it seems as though something goes wrong when requesting actual content stored in the SQLite database: https://ptpb.pw/AMsSt1E0zHV9-2al200FDr5-p-9m15:55
hirokiI've cleared the database and ran a playbook so it got populated with whatever database scheme is most current, just in case.15:56
ara-slack<dmsimard> @hiroki, that's an issue fixed yesterday due to a broken release of flask15:56
ara-slack<dmsimard> Let me rebase the patch so it contains the fix15:57
hirokiAh right ! I'll clear the environment for a new try-out then15:57
openstackgerritDavid Moreau Simard proposed openstack/ara master: Add support for custom context/application root  https://review.openstack.org/48437815:59
dmsimardhiroki: try with git fetch https://git.openstack.org/openstack/ara refs/changes/78/484378/11 && git checkout FETCH_HEAD15:59
dmsimardThat has the bad version blacklisted15:59
hirokiAlright, give me 10 minutes, the machine needs to be re-provisioned (2min)16:00
dmsimardHaha, no rush16:00
hirokilooks good so far, i'll do further testing and put the reverse proxy in place to see if that works as well16:10
hirokidmsimard: with the reverse proxy in front things still work great and as expected, so it seems that all is well with the code changes !16:16
*** DrWaluigi has quit IRC16:16
dmsimardNeat16:16
hirokiOne bug however remains, which occurs when clicking on a file reference in the 'Files' section16:16
hirokiINFO werkzeug: 127.0.0.1 - - [28/Apr/2018 16:15:31] "GET /ara/file/cf10a3ba-d543-4081-a398-da2fad6fb390 HTTP/1.1" 301 -16:16
hirokiThere is no trailing slash appended to the GET request, whereas with other requests (tasks, for instance), this issue does not occur16:17
dmsimard301 is a redirect16:17
dmsimardI'll take a look later perhaps16:17
dmsimardIn the meantime, I designed https://github.com/openstack/ansible-role-ara to be able to customize the root under embedded server, Apache and nginx16:18
hirokiYes it is a redirect, however, with a reverse proxy in between that speaks HTTPS front facing and proxying to ARA HTTP at the back, the file does not get loaded because ARA redirects to a HTTP URL, not the HTTPS url (which includes a non-default port BTW.)16:18
dmsimardOh, I see.16:19
hirokiThe port stays intact, but ont the protocol used HTTP vs. HTTPS16:19
hirokiGreat :) Seems like a good addition16:19
dmsimardI believe there is a flask setting for preserving the url scheme16:19
dmsimardhttp://flask.pocoo.org/docs/0.12/config/16:20
dmsimardPREFERRED_URL_SCHEME16:20
hirokiIt can be force with _scheme='https', but that's not a proper fix as it's hard coding it16:20
openstackgerritMerged openstack/ara master: Allow ARA module to be run  https://review.openstack.org/56356116:21
hirokiOh.. It defaults to HTTP16:21
dmsimardTry adding self.PREFERRED_URL_SCHEME = 'https' in config/base or something like that16:21
dmsimardJust to see if it works16:22
hirokiAlright16:22
hirokiAdded that in base.py under 'self.FREEZER_IGNORE_404_NOT_FOUND'. Did not work16:24
dmsimardHmmm, I'm not sure. Does Hiawatha have rewrite rules so you could rewrite http to https ?16:29
hirokiYes, but it does not work with a non-default HTTPS port -> https://www.hiawatha-webserver.org/forum/topic/272616:30
dmsimardThe url you are browsing doesn't have :9191 in it, does it ?16:32
*** DrWaluigi has joined #ara16:35
hirokidmsimard: no, just different URL scheme -> https://i.imgur.com/gngtwym.jpg16:37
*** jparrill_ has quit IRC16:40
*** jparrill_ has joined #ara16:51
hirokiWill the custom context/application root patch be merged with master or backported to 0.14.X ? Doesn't really matter to me, but then I know what to refer to in the ARA spec file16:53
*** jparrill_ has quit IRC17:51
*** jparrill_ has joined #ara18:16
dmsimardhiroki: it'll be part of the next release, 0.15.0, which is tagged from master19:35
hirokidmsimard: alright no worries, in that case i'll fix our spec file to that specific reference. FYI. -> https://ptpb.pw/ABf13R3lxQK_9iXd9crnhBhF6pp7/spec20:01
hirokiI also have a spec file for Hiawatha done, i'll ask our packager to send it upstream to Fedora. Let me know if you'd like a copy of it20:02
hirokiOops, old URL. Correct one -> https://ptpb.pw/ADCFlwu8BtjFVawYJBIFhBQx3lQY/spec20:03
hirokiIt is a requirement to have the logging.yml file in place ? We'd like to disable file based logging as we have systemd output stdout / stderr anyways and have that forwarded. I reckon we can change the logging.yml file to not log. It is possible to work around it by having ARA ignore the file in the first place ?20:06
dmsimardhiroki: fwiw ara is already packaged for rawhide: https://src.fedoraproject.org/cgit/rpms/ara.git/tree/20:07
dmsimardhiroki: yeah ARA will create a default logging file if there isn't one already20:08
dmsimardYou can configure it however you like though20:08
dmsimarddefault logging configuration file*20:08
hirokiGreat, haven't caught that ARA tree on Fedora yet. Thank you for the changes to the application root, all is working well now :)21:51
hirokidmsimard: could you blacklist Ansible 2.5.2 in the requirements ? There seems to be a bug with 2.5.2 that prevents the file module from being idempotent for symlinks. It always changes the file and therefore reports changed if it has to deal with a symlink. 2.5.1 is confirmed not to have the bug, 2.5.2 does. I'll ask for it to be reported on monday as I could not find any reported issues on Github22:31
hirokiTo try it out yourself: https://ptpb.pw/AM7aW5bxASXeMAM5aGp1zrqxmvJM/yaml22:32
dmsimardhiroki: if 2.5.2 doesn't work out for you, feel free to blacklist it locally (or pin to 2.5.1)22:37
hirokiWe did, but we reckon it will hurt more people then just us :)22:37

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