*** sshnaidm is now known as sshnaidm|afk | 00:28 | |
openstackgerrit | Merged openstack/ara feature/1.0: Implement get_or_create for the File model serializer https://review.openstack.org/641140 | 01:25 |
---|---|---|
openstackgerrit | Merged openstack/ara feature/1.0: Rely on the API server for the get_or_create methods https://review.openstack.org/641141 | 01:25 |
openstackgerrit | Merged openstack/ara feature/1.0: Move fields out of serializers and into fields https://review.openstack.org/641134 | 01:33 |
openstackgerrit | Merged openstack/ara feature/1.0: Remove unused code and variables out of serializers https://review.openstack.org/641135 | 01:33 |
apollo13 | dmsimard: to be honest, since in 90% of the cases we actually want to serve static files I'd just add whitenoise and use that | 08:33 |
apollo13 | dmsimard: https://pypi.org/project/whitenoise/ we can configure it that it a) always serves static files and b) does reloading when debug=true afaik | 08:34 |
*** sshnaidm|afk has quit IRC | 09:26 | |
*** sshnaidm|afk has joined #ara | 10:54 | |
*** sshnaidm|afk has quit IRC | 10:57 | |
krion | hi | 11:08 |
krion | dmsimard: if you want/can quickfix there's a missing " in the ara_record module example at https://ara.readthedocs.io/en/stable/usage.html#using-the-ara-record-module | 11:08 |
krion | first example, last line, last char (missing ") | 11:09 |
larsks | krion: you can fix it yourself by changing doc/source/usage.rst in the sources and then submitting a review. | 11:59 |
*** sshnaidm|afk has joined #ara | 12:01 | |
dmsimard | apollo13 : yeah I saw whitenoise but I preferred avoiding a dependency if we only needed that route :p | 12:50 |
dmsimard | I think it's ok to encourage putting a web server in front. | 12:52 |
apollo13 | dmsimard: yeah but putting a webserver in front also means collecting static files etc… | 13:02 |
apollo13 | dmsimard: for simplicity etc whitenoise would be "nice" :D | 13:02 |
dmsimard | But using runserver or wsgi still requires collecting static files no ? | 13:02 |
dmsimard | Does whitenoise make it so it's no longer required ? | 13:03 |
apollo13 | runserver uses the file finders and serves it automatically without collecting | 13:03 |
apollo13 | whitenoise can be configured to reuse that | 13:03 |
apollo13 | I mean for more adanced stuff like hashed static files we'd probably still want collectstatic, I guess the main question is how much work you want to put on the user | 13:04 |
dmsimard | I'll give it a try | 13:08 |
*** sshnaidm|afk is now known as sshnaidm | 13:32 | |
dmsimard | apollo13: whitenoise worked for runserver out of the box but gunicorn doesn't like it. Troubleshooting now | 14:30 |
dmsimard | that was my fault, forgot to set ARA_SETTINGS and it was using the one from my home dir | 14:31 |
dmsimard | so hey, that works. | 14:31 |
dmsimard | just noticed something weird with timestamps, though. log timestamps are jumping timezones or something, see at the end: http://paste.openstack.org/show/747342/ | 14:32 |
apollo13 | that is interesting | 14:41 |
apollo13 | I am not sure how to explain the switch | 14:42 |
dmsimard | I tried playing with the USE_TZ parameter a bit but it was even weirder | 14:43 |
apollo13 | yeah it is probably something around USE_TZ and TIME_ZONE | 14:43 |
dmsimard | the time went back instead of ahead | 14:43 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Use whitenoise for serving static files https://review.openstack.org/641403 | 14:48 |
dmsimard | apollo13: ^ thanks, I'll abandon the other one | 14:48 |
apollo13 | mhm what would be the clone url? | 14:49 |
apollo13 | apollo13@review.openstack.org:29418/openstack/ara.git | 14:49 |
apollo13 | it is apparently not | 14:49 |
apollo13 | ah nevermind probably gotta reread the admin docs | 14:50 |
apollo13 | ssh://florian@review.openstack.org:29418/openstack/ara.git did not work. Description: florian@review.openstack.org: Permission denied (publickey). | 14:51 |
apollo13 | fatal: Could not read from remote repository. | 14:51 |
apollo13 | ah wait | 14:51 |
apollo13 | works, thanks | 14:51 |
apollo13 | dmsimard: do you have a mac or linux? | 14:52 |
dmsimard | I need to send a new patchset | 14:52 |
dmsimard | I run fedora | 14:52 |
apollo13 | mhm, I wonder what happens to the time, we didn't make USE_TZ/TIMEZONE configurable | 14:53 |
apollo13 | although | 14:53 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Use whitenoise for serving static files https://review.openstack.org/641403 | 14:54 |
apollo13 | it might make sense that the logging changes after django configured settings according to USE_TZ/TIMEZONE | 14:54 |
dmsimard | apollo13: pretty sure the USE_TZ/TIMEZONE settings were carried over from copypasta | 14:54 |
dmsimard | so I wouldn't put too much thought into their current values | 14:54 |
dmsimard | but yeah, I think it might be a roundabout dynaconf/loading settings thing | 14:55 |
apollo13 | yeah, in the long run we probably should get the writing of our yaml settings file out of settings | 14:55 |
apollo13 | no that cannot be | 14:55 |
dmsimard | apollo13: +1 on yaml settings | 14:55 |
apollo13 | because our settings.py is no longer affected by dynaconf weirdness | 14:55 |
apollo13 | but django operates like this: | 14:55 |
apollo13 | * configure minimal logging | 14:55 |
apollo13 | * load settings | 14:55 |
apollo13 | * configure timezone/etc (I guess) | 14:55 |
apollo13 | * reconfigure logging according to LOGGING in settings.py | 14:55 |
dmsimard | sort of makes sense I suppose | 14:56 |
apollo13 | now between logging during "* load settings" and "* reconfigure logging" the time can switch | 14:56 |
apollo13 | I'd expect it to be correct at the end though | 14:56 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Use whitenoise for serving static files https://review.openstack.org/641403 | 16:51 |
dmsimard | apollo13: I think ^ is good to go | 17:08 |
*** irclogbot_0 has joined #ara | 18:09 | |
*** irclogbot_0 has quit IRC | 18:36 | |
openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: Add a dedicated page (and URL) for community/help information https://review.openstack.org/641470 | 18:58 |
openstackgerrit | Merged openstack/ara feature/1.0: Add missing relationship from result to play https://review.openstack.org/641133 | 19:21 |
*** irclogbot_0 has joined #ara | 21:06 | |
*** zbr|ssbarnea has joined #ara | 21:08 | |
*** zbr has quit IRC | 21:10 | |
*** irclogbot_0 has quit IRC | 21:28 | |
openstackgerrit | David Moreau Simard proposed openstack/ara-infra master: Add a dedicated page (and URL) for community/help information https://review.openstack.org/641470 | 21:47 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: WIP: Refactor Serializers for different formats and request types https://review.openstack.org/641136 | 22:13 |
*** irclogbot_0 has joined #ara | 22:19 | |
*** irclogbot_0 has quit IRC | 22:34 | |
*** irclogbot_0 has joined #ara | 22:34 | |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Remove mentions of ara_wsgi_configure_service in the role https://review.openstack.org/641505 | 22:40 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Include ansible-role-ara's README in the docs https://review.openstack.org/641506 | 22:40 |
openstackgerrit | David Moreau Simard proposed openstack/ara feature/1.0: Include ansible-role-ara's README in the docs https://review.openstack.org/641506 | 22:44 |
*** irclogbot_0 has quit IRC | 22:44 | |
openstackgerrit | Merged openstack/ara-infra master: Add a dedicated page (and URL) for community/help information https://review.openstack.org/641470 | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!