Tuesday, 2018-05-01

*** bcoca has quit IRC03:55
*** jlozadad_ has joined #ara06:46
*** jlozadad has quit IRC06:47
*** weshay is now known as weshay|ruck12:46
*** bcoca has joined #ara12:56
*** bcoca has joined #ara12:56
ara-slack<dbpiv> Dumb Question….  I have encountered the race condition again and I went about looking to see if I could install the dev version that had the fix (https://review.openstack.org/560998).  At this point all I can seem to install is ara-0.14.7.dev27 which upon comparison of ara/models.py it doesn’t seem to have the race condition fix.  I used pip install git+https://git.openstack.org/openstack/ara to install dev27, is there another option for ge12:56
ara-slackthe race fix?  I a manually pulled the models.py and replaced my /Library/Python/2.7/site-packages/ara/models.py and got no love.12:56
ara-slack<dbpiv> Of course if the answer is hold on and wait for 0.15.0, then I can do that as well.12:56
*** tbielawa has joined #ara13:08
*** jlozadad_ has quit IRC13:14
*** jlozadad_ has joined #ara13:19
*** openstackgerrit has quit IRC13:34
*** jlozadad has joined #ara13:34
*** jlozadad_ has quit IRC13:34
*** jlozadad has quit IRC13:39
*** jlozadad has joined #ara13:40
*** tbielawa is now known as tbielawa|appt14:06
*** Bakey has joined #ara14:26
ara-slack<dmsimard> @dbpiv hey, installing from master should definitely have the fix, it's merged in master15:09
ara-slack<dbpiv> hmmm15:09
ara-slack<dmsimard> Try installing in a fresh virtualenv to make sure ?15:09
ara-slack<dbpiv> let me try to pull it again and force pip to not use cache.15:10
ara-slack<dbpiv> same error, but now that I stare at the error it is different.  I wonder if I have a different error.    IntegrityError: (raised as a result of Query-invoked autoflush; consider using a session.no_autoflush block if this flush is occurring prematurely) (sqlite3.IntegrityError) UNIQUE constraint failed: files.playbook_id, files.path [SQL: u’INSERT INTO files (id, playbook_id, path, content_id, is_playbook) VALUES (?, ?, ?, ?, ?)’] [parameter15:12
ara-slack(‘833b6cc4-669d-4429-b554-3230788ee879’, u’dde7e91d-d13a-441a-95e2-7e55c45394b4', u’/Users/dbpiv/Documents/PSFT/Ansible/StopStack.yml’, None, 1)] (Background on this error at: http://sqlalche.me/e/gkpj)15:12
ara-slack<dbpiv> Looks like a different error.15:14
ara-slack<dbpiv> I guess I can go delete the offending row15:14
ara-slack<dmsimard> That's a unique constraint failure, huh, like it tried to add the same file twice for a playbook. That's odd.15:19
ara-slack<dbpiv> what is odd is that the id doesn’t exist15:20
ara-slack<dmsimard> Basically there shouldn't be two files with the same path for a playbook. I suppose we can try/catch this one so it's not fatal but I wonder how/where it happened15:20
ara-slack<dmsimard> Is it just a regular playbook you're running ? Any particular things like custom strategy or stuff like that ?15:21
ara-slack<dbpiv> regular playbook…. developing a playbook that will get a list of ids and then serially connect to the server and shutdown services owned by that id.  And thus far it has not been successful, but that shouldn’t matter.15:22
ara-slack<dmsimard> Ok I thought that perhaps there could be multithreaded confusion with the free strategy (for example)15:23
ara-slack<dmsimard> You're using sqlite, right ?15:23
ara-slack<dbpiv> yes on the sqlite15:23
ara-slack<dbpiv> so  is ara trying to some sort of cleanup on a failed run and thus it thinks it needs to insert this row post mortem?15:26
*** miabbott has quit IRC15:28
dmsimardunlikely, I'm not sure why it's stuck on that -- you run a new playbook and it fails like that ?15:31
*** miabbott has joined #ara15:32
ara-slack<dbpiv> if I run a different playbook, it runs as expected.  I have run a few this morning.   I had some early morning patching to do.15:34
*** tbielawa|appt is now known as tbielawa16:15
dmsimardSo it's only when browsing the interface then ?16:21
ara-slack<dbpiv> yes16:21
*** sshnaidm is now known as sshnaidm|afk17:17
*** tbielawa is now known as tbielawa|brb17:24
*** myoung is now known as myoung|biab18:06
*** tbielawa|brb is now known as tbielawa18:07
*** openstackgerrit has joined #ara18:35
openstackgerritDavid Moreau Simard proposed openstack/ansible-role-ara master: Fix the log location by using the log directory  https://review.openstack.org/56555818:35
dmsimardhiroki: I've just tried SSL with nginx reverse proxying to the embedded web server and it appears to be working fine18:44
dmsimardnginx config almost verbatim from ansible-role-ara: http://paste.openstack.org/show/720196/18:45
dmsimardso I'll merge the patch in it's current state -- I've also verified that it does not appear to negatively impact the other WSGI implementations18:46
dmsimardI've been delaying 0.15.0 for a while and it needs to ship, if there's something else to fix, we can land it in 0.15.118:46
dmsimardhiroki: https://i.imgur.com/8zHibie.png18:48
dmsimard0.15.0rc1 will be tagged as soon as https://review.openstack.org/#/c/484378/ lands18:50
ara-slack<dmsimard> ^ @harlowja18:51
dmsimardssbarnea: ^18:51
dmsimardI know you two have been waiting for it for a while, sorry about that18:51
ara-slack<harlowja> cool18:52
dmsimardWe're touching a couple of sensitive bits of code with 0.15.0 and I wanted to do my best to make sure we don't break anything18:53
dmsimardI'll write a blog post for the changelog.18:53
ssbarneadmsimard: in fact what I was looking for was the offline support (plain html version). not sure if this solves that case too, but i will check.18:53
dmsimardssbarnea: you mean viewing from a local filesystem without a web browser ?18:54
dmsimarder, web server18:54
ssbarneadmsimard: nope, i know this is broken by http safety design. but storing ara html output to a webserver.18:55
ssbarneanormally all paths should have being relative to the main index.html one, but in my case they were not. but i suspect something specific to my setup as i got different resoults in other cases.18:55
dmsimardssbarnea: oh, that was the issue with the "ara generate ." right ?18:55
ssbarneadmsimard: exactly, generate issue.18:56
dmsimardssbarnea: I was never able to reproduce that issue on my end, we can fix it in a 0.15.1 hotfix if we end up figuring out the problem18:56
ssbarneasure, no worry. i have other pressing things i need to fix now.18:56
dmsimardwfm18:56
dmsimardthanks for your patience and for your three patches :)18:57
openstackgerritMerged openstack/ara master: Add support for custom context/application root  https://review.openstack.org/48437819:06
*** myoung|biab is now known as myoung19:16
-dmsimard- 0.15.0rc1 is tagged and will be available on pypi shortly (install with pip install --pre ara), changelog: https://github.com/openstack/ara/releases/tag/0.15.0.0rc119:20
ara-slack<dmsimard> <here> ^19:21
*** tbielawa is now known as tbielawa|brb19:49
*** tbielawa|brb is now known as tbielawa20:19
*** tbielawa has quit IRC20:26
*** myoung is now known as myoung|biab20:36
openstackgerritDavid Moreau Simard proposed openstack/ansible-role-ara master: Fix the log location by using the log directory  https://review.openstack.org/56555821:06
*** jlozadad_ has joined #ara21:35
*** jlozadad has quit IRC21:36
*** jlozadad_ has quit IRC21:36
*** jlozadad has joined #ara21:36
*** myoung|biab is now known as myoung21:54
*** jparrill_ has quit IRC22:27
*** jparrill_ has joined #ara22:28
*** jparrill_ has quit IRC22:48
*** jparrill has joined #ara22:53
*** jparrill has quit IRC22:57
*** jparrill has joined #ara22:59
*** bcoca has quit IRC23:29

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