Thursday, 2017-01-26

*** karimb has quit IRC00:03
*** permalac has quit IRC09:12
*** permalac has joined #ara09:14
*** karimb has joined #ara09:43
*** karimb has quit IRC09:43
*** permalac has quit IRC09:45
*** permalac has joined #ara09:47
*** karimb has joined #ara09:52
*** karimb has quit IRC10:31
*** karimb has joined #ara10:50
*** karimb has quit IRC11:12
*** karimb has joined #ara11:27
*** karimb has quit IRC11:28
*** karimb has joined #ara11:30
*** karimb has quit IRC12:16
*** karimb has joined #ara12:19
*** karimb has quit IRC12:39
*** karimb has joined #ara13:20
*** jrist has quit IRC14:56
*** jrist has joined #ara15:10
*** karimb has quit IRC15:44
*** karimb has joined #ara15:52
*** jrist has quit IRC16:10
*** karimb has quit IRC16:15
*** jrist has joined #ara16:22
*** karimb has joined #ara16:27
*** karimb has quit IRC17:33
*** karimb has joined #ara17:36
*** karimb has quit IRC17:46
berendt19:03:28 OperationalError: (sqlite3.OperationalError) table data already exists [SQL: u'\nCREATE TABLE data (\n\tid VARCHAR(36) NOT NULL, \n\tplaybook_id VARCHAR(36), \n\t"key" VARCHAR(255), \n\tvalue BLOB, \n\tPRIMARY KEY (id), \n\tFOREIGN KEY(playbook_id) REFERENCES playbooks (id) ON DELETE RESTRICT, \n\tUNIQUE (playbook_id, "key")\n)\n\n']19:04
berendti got this issue when trying to initiate a new sqlite database with current release19:04
berendtis this issue already known?19:04
berendtit is related to the ara_record data table: INFO  [alembic.runtime.migration] Running upgrade da9459a1f71c -> e8e78fd08bf2, ara_record data19:10
berendtthe upgrade is called twice19:10
berendtthat's interesting, i set database = sqlite:///ara.sqlite to generate the file in the current directory, when i do not set this parameter everything is working like expected19:13
berendtabsolute path (sqlite:////tmp/ara.sqlite) is also working like expected19:13
dmsimardberendt: Haven't seen that happen19:20
dmsimardberendt: can you let me know if you figure out how to reproduce the issue ?19:20
berendtgive me a second19:20
*** karimb has joined #ara19:26
berendtdmsimard: https://gist.github.com/berendt/5a661fdf0711009eb60ccfd72edab44119:30
berendtaccording to http://docs.sqlalchemy.org/en/latest/core/engines.html#sqlite this connection url should be possible19:31
*** karimb has quit IRC19:45
dmsimardberendt: hm, but current working directory relative to what ?20:35
dmsimardberendt: to where you are running ansible? to where ara is installed?20:35
berendtdmsimard: i think sqlite:///ara.sqlite means: use ara.sqlite in cwd20:35
berendtara/ansible installed inside a venv20:35
dmsimardberendt: behavior for that is not clear to me, I mean I undertand that if you have a .py file in cwd and you execute it from there it'll properly use it in cwd20:36
dmsimardbut when used from ansible/ara /me shrugs20:36
berendtas a workaround i set the ara_database env var in my jenkins job to an absolute path pointing to a dir inside the workspace20:37
dmsimardokay20:38
berendti thought that it is possible to say: create the sqlite file in the directory where i am calling ansible-playbook from20:38
berendtthis way it would be possible to set the parameter inside ansible.cfg without using the env vars20:38
dmsimardYou could probably create your ansible.cfg dynamically ?20:39
berendtyes, this would be an other way to workaround this issue20:39
dmsimardsort of like how it's shown here http://ara.readthedocs.io/en/latest/configuration.html#using-ansible-cfg20:39
berendti try to recude as many dynamic generated files as possible20:40
berendtan other issue: 20:27:07 UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 139: ordinal not in range(128)20:40
berendti used some german characters inside a playbook20:40
berendtgot this error after calling ara generate20:40
dmsimardhmm20:56
dmsimardI've seen that happen before on breaking whitespace characters20:56
dmsimardI have an idea on how to fix it, hang on.20:58
*** openstackgerrit has joined #ara21:32
openstackgerritDavid Moreau Simard proposed openstack/ara: Ensure we're dealing with decoded utf-8 strings for file formatting  https://review.openstack.org/42593221:32
dmsimardberendt: that should do it ^21:33
dmsimardI was able to reproduce the issue with a breaking whitespace character, applied the fix and then tried again and it worked21:33
berendtcan you test with ä, ö, ü21:33
dmsimardberendt: sure, hang on21:34
dmsimardberendt: funny story, it's damn near impossible to test encoding related issue in py.test unit tests :/21:35
dmsimardoh, I just had an idea on how to test it .. I'll try after21:35
dmsimardberendt: yeah those characters are fine21:36
berendtnice, thanks :)21:37
dmsimardberendt: there's one last thing I want to merge and I'll cut a release with that bugfix included21:38
berendtit is not urgent, i simply removed all special chars for the moment21:38
berendtit is possible to write ae instead of ä21:38
dmsimardbah, there's a couple bugfixes due for release21:39
openstackgerritDavid Moreau Simard proposed openstack/ara: Ensure we're dealing with decoded utf-8 strings for file formatting  https://review.openstack.org/42593221:40
openstackgerritDavid Moreau Simard proposed openstack/ara: Ensure we're dealing with decoded utf-8 strings for file formatting  https://review.openstack.org/42593221:41
dmsimardberendt: there ^ unit testing for encoding issue sucks but I added it to an integration test.21:41
dmsimardthanks for poking me about it, I've been meaning to fix it for a while21:42
dmsimardMy keyboard layout has me typing breaking whitespace characters all the time :(21:43
openstackgerritDavid Moreau Simard proposed openstack/ara: Ensure we're dealing with decoded utf-8 strings for file formatting  https://review.openstack.org/42593221:44
berendtdmsimard: do you know/use molecule?21:46
dmsimardberendt: I've heard a lot about it but I haven't had the opportunity of using it yet21:47
dmsimardI happen to already have tooling to re-run things in ephemeral environments so I need another excuse to spend time with it :)21:49
berendtdmsimard: https://github.com/metacloud/molecule/issues/72821:52
berendt;)21:52
dmsimardhah21:52
dmsimardthanks for the advertising :)21:52
berendti use molecule for the test of my ansible roles21:52
berendti think custom integration of ara by using a custom ansible cfg is already possible21:53
berendtbut would be nice to have a native integration21:53
dmsimardI need to revive my patch to add it in Zuul natively :)21:53
berendtis it possible to only output relative paths in the visualisation? e.g. playbook.yml instead of /var/jenkins_home/workspace/aio-reset-node/playbook.yml21:57
dmsimardberendt: not currently22:03
dmsimardberendt: I think I had an idea around that, I forgot22:04
dmsimardberendt: ah there you got https://storyboard.openstack.org/#!/story/200081722:04
dmsimardI'll take it into account in the upcoming UI refactor22:04
dmsimardmnaser: fyi I know you use molecule too https://github.com/metacloud/molecule/issues/728 :)22:08
mnaseroh we already do this dmsimard22:09
mnaseri'll find time for a comment22:09
dmsimardI think it's about adding it natively or something to that effect22:09
* dmsimard needs to find time to try molecule one day22:09
mnaserdmsimard the time you spend to get it running will result in many hours saved22:09
mnasertho i think molecule is becoming *kinda* useless with molecule v222:10
mnaserit pretty much becomes a scaffolding tool and uses the openstack modules to create servers etc22:10
*** themurph has quit IRC23:20
*** themurph has joined #ara23:28

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