| *** VW has joined #craton | 00:08 | |
| *** VW has quit IRC | 01:25 | |
| *** VW_ has joined #craton | 01:25 | |
| *** VW_ has quit IRC | 01:26 | |
| *** VW has joined #craton | 01:26 | |
| *** VW has quit IRC | 09:31 | |
| *** VW has joined #craton | 09:32 | |
| *** VW has quit IRC | 11:58 | |
| *** VW has joined #craton | 11:59 | |
| *** VW has quit IRC | 12:47 | |
| *** VW has joined #craton | 12:47 | |
| -openstackstatus- NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks, and should return to an operable state momentarily | 12:50 | |
| *** VW_ has joined #craton | 12:50 | |
| *** VW has quit IRC | 12:52 | |
| *** VW_ has quit IRC | 13:26 | |
| *** VW has joined #craton | 13:26 | |
| *** VW has quit IRC | 14:02 | |
| *** VW has joined #craton | 14:29 | |
| *** openstackgerrit has joined #craton | 14:31 | |
| openstackgerrit | Michael Porras proposed openstack/python-cratonclient master: Add fields selector to <resource> show and list https://review.openstack.org/451932 | 14:31 |
|---|---|---|
| jimbaker | anonymike, thanks, i'm going to take a look at that patch | 15:06 |
| anonymike | jimbaker: Thank you! | 15:07 |
| jimbaker | anonymike, looks like some pretty easy fixes in the tox output, http://logs.openstack.org/32/451932/2/check/gate-python-cratonclient-python27-ubuntu-xenial/97523df/testr_results.html.gz | 15:19 |
| jimbaker | fixes to be done | 15:19 |
| anonymike | yeah, that's what I've been working on :/ but I think the main functionality is good | 15:19 |
| jimbaker | anonymike, what is the syntax for specifying a specific variable to be included? | 15:27 |
| jimbaker | craton host-list --fields=id,name works | 15:27 |
| jimbaker | was trying stuff like # craton host-list --fields=id,name,var:cell_capabilities | 15:27 |
| jimbaker | or will that be another patch? | 15:28 |
| anonymike | i thought we were holding off on that | 15:28 |
| jimbaker | anonymike, ahh, ok | 15:28 |
| anonymike | yeah :/ was the other json path stuff figured out? | 15:28 |
| jimbaker | so let's do that as a separate patch then | 15:28 |
| jimbaker | makes sense to separate to me | 15:28 |
| jimbaker | anonymike, one thing: i suggest you add a note in your commit message on usage | 15:29 |
| jimbaker | examples | 15:29 |
| jimbaker | can be a short example of what would go in the docs as well | 15:29 |
| anonymike | jimbaker: great idea | 15:29 |
| jimbaker | i also like how this works $ craton --format=json host-list --fields=id,name,variables | 15:31 |
| anonymike | yeah the json response is really slick | 15:34 |
| anonymike | also the double fields flag works --fields=name --fields=id,variables | 15:35 |
| anonymike | argparse ++ | 15:35 |
| jimbaker | yeah, argparse is pretty awesome. almost always you can make it do what you want with little fuss | 15:36 |
| jimbaker | i think the one thing it could really improve upon is built-in support for shell completion | 15:36 |
| jimbaker | surely someone has written that package for python... | 15:36 |
| jimbaker | while we are at it, better man page integration :) | 15:37 |
| *** VW_ has joined #craton | 16:01 | |
| *** VW_ has quit IRC | 16:03 | |
| *** VW_ has joined #craton | 16:03 | |
| *** VW has quit IRC | 16:05 | |
| *** zz_pwnall1337 is now known as pwnall1337 | 16:14 | |
| pwnall1337 | jimbaker: i pulled craton master | 16:14 |
| pwnall1337 | my db reset script is failing now | 16:14 |
| pwnall1337 | ERROR 1364 (HY000) at line 1: Field 'variable_association_id' doesn't have a default value | 16:14 |
| pwnall1337 | https://gist.github.com/pwnall1337/9a532fb224d2fbe881948cb0a80a6649 | 16:15 |
| pwnall1337 | not sure what changed | 16:15 |
| farid | anonymike: you ran into that at all ? ^ | 16:19 |
| anonymike | looking | 16:20 |
| anonymike | pretty sure I didn't change anything there | 16:21 |
| pwnall1337 | i ran the reset script cleaned the db then unintalled/reinstalled craton and then ran it again but the default data fails | 16:22 |
| pwnall1337 | the creation of project/users | 16:22 |
| jimbaker | pwnall1337, are you using dbsync bootstrap, plus the rest api for creation of projects/users? | 16:25 |
| jimbaker | you do not want to use mysql directly at all in craton | 16:26 |
| pwnall1337 | i guess things changed | 16:28 |
| pwnall1337 | my script is a month old | 16:28 |
| pwnall1337 | was the getting started guide updated? | 16:29 |
| anonymike | yes | 16:29 |
| anonymike | but nothing substantial on the back end was changed | 16:29 |
| anonymike | at least not by me | 16:29 |
| jimbaker | anonymike, pwnall1337 saw the latest version of the alembic migration now enforcing this constraint | 16:30 |
| jimbaker | it is for good :) | 16:30 |
| jimbaker | otherwise inscrutable errors | 16:30 |
| jimbaker | and to be clear, one can use mysql directly in a pinch. but with this constraint added, we now enforce what the models have been assuming | 16:31 |
| jimbaker | (the inscrutability occurs when sqlalchemy models diverge from the underlying data - something i might talk about in the forthcoming osic talk i'm doing on sqlalchemy) | 16:33 |
| jimbaker | https://github.com/openstack/craton/commit/555d0c885692200b6ab3dbb3adf780784e389183 | 16:33 |
| *** VW_ has quit IRC | 16:40 | |
| *** VW has joined #craton | 16:41 | |
| anonymike | pwnall1337: Im still working on getting some tests to pass but if you'd like to demo the ability to request specific fields for host-list you can run this version of the cratonclient https://review.openstack.org/#/c/451932/ | 16:46 |
| anonymike | Or I can since I have it running lol just not sure how it'd fit into the demo | 17:08 |
| pwnall1337 | sounds good | 17:10 |
| pwnall1337 | eating lunch atm | 17:10 |
| pwnall1337 | bb in 20 | 17:10 |
| pwnall1337 | i need to get my db bootstrap working | 17:11 |
| pwnall1337 | k im back | 17:17 |
| pwnall1337 | i can't bootstrap anonymike | 17:22 |
| pwnall1337 | https://gist.github.com/pwnall1337/72a97c7fa5fcd508705e3973f09505cc | 17:22 |
| pwnall1337 | ahh | 17:23 |
| pwnall1337 | it's formatting issue pasting from docs | 17:23 |
| anonymike | pwnall1337: did you get it? | 17:47 |
| pwnall1337 | yes | 17:47 |
| pwnall1337 | can you check pm | 17:47 |
| *** pwnall1337 is now known as zz_pwnall1337 | 17:54 | |
| *** VW has quit IRC | 17:59 | |
| *** VW has joined #craton | 18:00 | |
| *** harlowja has quit IRC | 18:33 | |
| *** VW has quit IRC | 19:02 | |
| *** VW has joined #craton | 19:03 | |
| *** harlowja has joined #craton | 19:58 | |
| *** VW has quit IRC | 20:41 | |
| *** VW has joined #craton | 20:41 | |
| *** VW_ has joined #craton | 21:18 | |
| *** VW has quit IRC | 21:18 | |
| *** harlowja has quit IRC | 22:05 | |
| *** VW_ has quit IRC | 22:06 | |
| *** harlowja has joined #craton | 23:25 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!