*** SergeyLukjanov is now known as SergeyLukjanov_ | 00:00 | |
*** jmontemayor has quit IRC | 00:04 | |
*** amytron has quit IRC | 00:16 | |
*** Ranjitha has joined #openstack-trove | 00:18 | |
*** yogesh has quit IRC | 00:22 | |
*** matsuhashi has joined #openstack-trove | 00:24 | |
*** plodronio has joined #openstack-trove | 00:28 | |
*** thedodd has quit IRC | 00:29 | |
hub_cap | juice: awesome | 00:31 |
---|---|---|
*** rnirmal has quit IRC | 00:35 | |
openstackgerrit | A change was merged to openstack/trove: Fix default_datastore migration script https://review.openstack.org/69537 | 00:35 |
*** rnirmal has joined #openstack-trove | 00:35 | |
*** rnirmal has quit IRC | 00:35 | |
*** yidclare has quit IRC | 00:37 | |
*** harlowja is now known as harlowja_away | 01:04 | |
*** michael-yu has quit IRC | 01:07 | |
*** jimbobhickville has quit IRC | 01:13 | |
*** plodronio has quit IRC | 01:13 | |
*** Ranjitha has quit IRC | 01:22 | |
*** harlowja_away is now known as harlowja | 01:26 | |
*** michael-yu has joined #openstack-trove | 01:27 | |
*** ViswaV_ has quit IRC | 01:29 | |
*** haomaiwang has joined #openstack-trove | 01:30 | |
*** yogesh has joined #openstack-trove | 01:33 | |
*** haomaiwa_ has quit IRC | 01:34 | |
*** yogesh has quit IRC | 01:38 | |
*** nosnos has joined #openstack-trove | 01:41 | |
*** grapex has quit IRC | 01:49 | |
*** michael-yu has quit IRC | 01:57 | |
*** Barker has joined #openstack-trove | 02:01 | |
*** mattgriffin has quit IRC | 02:11 | |
*** amytron has joined #openstack-trove | 02:14 | |
*** khyati has quit IRC | 02:15 | |
*** thedodd has joined #openstack-trove | 02:18 | |
*** robertmyers has joined #openstack-trove | 02:19 | |
*** matsuhashi has quit IRC | 02:34 | |
*** matsuhas_ has joined #openstack-trove | 02:37 | |
*** erkules_ has joined #openstack-trove | 02:39 | |
*** erkules has quit IRC | 02:41 | |
*** thedodd has quit IRC | 02:44 | |
*** matsuhashi has joined #openstack-trove | 03:07 | |
*** matsuhas_ has quit IRC | 03:08 | |
*** matsuhashi has quit IRC | 03:08 | |
hub_cap | cp16net: around? | 03:10 |
*** plodronio has joined #openstack-trove | 03:16 | |
*** matsuhashi has joined #openstack-trove | 03:20 | |
cp16net | what up | 03:20 |
cp16net | hub_cap: i'm here | 03:21 |
hub_cap | sweet, i -2d yer shiz | 03:22 |
hub_cap | id like to discuss why its in a json file and not the db (The config defaults) | 03:22 |
hub_cap | and im not opposed to, if we come to an understanding and things change, to do it in another review | 03:22 |
hub_cap | and attach to this blueprint | 03:22 |
hub_cap | fwiw | 03:22 |
hub_cap | or u can do nothing if it makes sense to me and amcrn :) | 03:22 |
cp16net | hub_cap: thats a great question | 03:23 |
cp16net | i think it would really make sense in the db | 03:23 |
hub_cap | cp16net: i thought so myself lol | 03:23 |
cp16net | its easier for now to modify | 03:23 |
hub_cap | well FUDGE | 03:23 |
hub_cap | i have to go on a walk, baby is being cranky | 03:23 |
cp16net | but i think its fine for the first pass | 03:24 |
hub_cap | ill be back in like ~20 min if yer still around | 03:24 |
hub_cap | ok so ill take off the +2, but i cant let the BP be implemented unitl the 2nd part is done | 03:24 |
hub_cap | err off the -2 | 03:24 |
cp16net | ok i'll be online sorta i'm playing fifa atm as well | 03:24 |
hub_cap | tight | 03:24 |
cp16net | word | 03:24 |
hub_cap | one other thing to think about | 03:24 |
hub_cap | can u get the 2nd part done by the i3 cutoff? | 03:24 |
cp16net | 2nd part being? | 03:25 |
hub_cap | i can do a FFE for it, since i dotn think itll be a complete feature in icehouse | 03:25 |
cp16net | parameters in the db? | 03:25 |
hub_cap | moving that json to the db | 03:25 |
hub_cap | ya | 03:25 |
cp16net | or the enum stuff? | 03:25 |
hub_cap | db | 03:25 |
cp16net | because thats another bp | 03:25 |
hub_cap | which is? | 03:25 |
hub_cap | the db stuff? | 03:25 |
hub_cap | i dont think so, i dont like this feature the way it is | 03:25 |
cp16net | making like booleans and other strings in to a sorta enum style for validation | 03:25 |
hub_cap | so im not marking that bp as fixed | 03:25 |
hub_cap | ohhh no not htat | 03:25 |
cp16net | ok | 03:26 |
hub_cap | moving from taht funky json file to the db | 03:26 |
hub_cap | ok so think about if thats feasible for i3 | 03:27 |
hub_cap | or if i need to chat w/ ttx about a FFE for it | 03:27 |
hub_cap | and ill finish reviewing thsi as-is | 03:27 |
hub_cap | cool amcrn ? | 03:27 |
amcrn | i think everyone agrees that it should be in the database, but the loading portion of it is something i question | 03:27 |
cp16net | ok it should work fine | 03:27 |
cp16net | i hadnt seen any issues | 03:27 |
cp16net | and amcrn checked it out | 03:27 |
cp16net | not sense the changes tho | 03:27 |
amcrn | i mean, what is the "happy path" on loading all of your configuration parameters in the database? | 03:27 |
amcrn | we write some sql script and check it in to the repo? | 03:28 |
cp16net | that would be part of it | 03:28 |
hub_cap | why not put it in the migration? | 03:28 |
amcrn | ok, just making sure the cloud operator wouldn't get an empty table and we say "yeah, import all that stuff somehow" | 03:29 |
hub_cap | HAH ya | 03:29 |
hub_cap | i vote for that amcrn ;) | 03:29 |
amcrn | lol | 03:29 |
hub_cap | ok shoes on, g2g | 03:29 |
hub_cap | ill bb in a bit | 03:29 |
hub_cap | amcrn: ponder this, is it just me, or was it actually _nice_ that today ended up being chilly :) | 03:29 |
amcrn | cp16net: mysql has ENUM as well, so that additional bp can be solved at the same time | 03:30 |
amcrn | using the database itself, and not additional code, which turns out nicely | 03:30 |
*** plodronio has quit IRC | 03:33 | |
*** matsuhashi has quit IRC | 03:34 | |
*** matsuhashi has joined #openstack-trove | 03:35 | |
*** robertmyers has quit IRC | 03:36 | |
*** robertmyers has joined #openstack-trove | 03:36 | |
*** thedodd has joined #openstack-trove | 03:38 | |
*** jasonb365 has joined #openstack-trove | 03:39 | |
*** matsuhashi has quit IRC | 03:43 | |
*** khyati has joined #openstack-trove | 03:47 | |
*** matsuhashi has joined #openstack-trove | 03:50 | |
hub_cap | i didnt miss anything :) | 03:57 |
hub_cap | so cp16net u good w/ the work? we can figure out the best way to prepopulate it (maybe there is some prior art w/ nova) | 03:57 |
amcrn | hub_cap: you don't take issue with solving the validation bp by means of schema do you? | 03:58 |
amcrn | i.e. use ENUM where appropriate, etc. | 03:58 |
hub_cap | not @all | 03:58 |
hub_cap | i assume its not only a mysql thing tho eh? | 03:59 |
hub_cap | cp16net: already i see inserting in the nova sql | 03:59 |
hub_cap | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/216_havana.py#L90 | 04:00 |
amcrn | hub_cap: fair point. i suppose at some point when this hits the mgmt api, maybe some basic enum validation can be added to the command | 04:01 |
amcrn | then again, no coddling exists for using garbage CONF values, so perhaps i'm trying to build bumper-lanes where they're not needed | 04:02 |
* amcrn shrugs | 04:03 | |
* hub_cap shrugs back | 04:03 | |
hub_cap | i think its fair to see what it looks like | 04:03 |
hub_cap | and then tell cp16net all that work he did we dislike :D | 04:03 |
hub_cap | but i assume that the allowed values table will have a constraint on a version_id right? | 04:04 |
amcrn | you're referring to a datastore-version-id, yes? | 04:04 |
hub_cap | ya | 04:05 |
amcrn | it should be directly tied to one, yes | 04:05 |
hub_cap | yar | 04:05 |
hub_cap | oh man im hungr now :) | 04:05 |
amcrn | well, actually, a manager | 04:05 |
hub_cap | yea | 04:05 |
hub_cap | ok time for me to dissapear again | 04:06 |
hub_cap | kroll show ftw | 04:06 |
amcrn | well, the user creates configuration-groups that are tied to instances with specific datastore-versions, but the list of parameters and such is tied to a manager | 04:06 |
hub_cap | :) | 04:06 |
hub_cap | fair point. that igth require a bit of rework w/ the current model | 04:06 |
amcrn | bah, dvr it, i wanted to quickly chat about something MongoDB related | 04:06 |
amcrn | ;p | 04:06 |
hub_cap | ok hit me | 04:06 |
*** thedodd has quit IRC | 04:18 | |
* cp16net catches up | 04:22 | |
hub_cap | :) | 04:23 |
*** Barker has quit IRC | 04:25 | |
cp16net | cool yeah i see the datastore version id being tied to every parameter | 04:25 |
cp16net | and i could see a mgmt api for CRAPing them | 04:26 |
*** harlowja is now known as harlowja_away | 04:26 | |
hub_cap | and be sure to index the name field (especially if you are doing selects on it to check validation against) | 04:32 |
*** robertmyers has quit IRC | 04:32 | |
*** amcrn has quit IRC | 04:34 | |
cp16net | it will mostly be selecting by name and version id | 04:35 |
cp16net | or just version id | 04:35 |
hub_cap | ya dependingon where it is in de codes | 04:37 |
hub_cap | cool, so ill finish w/ this, and ill chat ttx up w/ a ffe | 04:37 |
hub_cap | can u change the bp to like good progress or something cp16net | 04:37 |
hub_cap | and maybe put a note on it | 04:37 |
cp16net | hmm it was put to needs review | 04:50 |
cp16net | meaning code complete | 04:50 |
cp16net | but i moved it back to good progess | 04:51 |
openstackgerrit | A change was merged to openstack/trove: Replaces local generate_uuid with common.utils https://review.openstack.org/69129 | 04:55 |
cp16net | ok i fixed up the client | 04:57 |
openstackgerrit | Craig Vyvial proposed a change to openstack/python-troveclient: adds support for configurations management https://review.openstack.org/53169 | 04:58 |
openstackgerrit | Craig Vyvial proposed a change to openstack/python-troveclient: adds support for configurations management https://review.openstack.org/53169 | 05:00 |
openstackgerrit | Paul Lodronio proposed a change to openstack/trove: adding additional datastore tests https://review.openstack.org/70296 | 05:04 |
*** haomaiwang has quit IRC | 05:09 | |
*** haomaiwang has joined #openstack-trove | 05:10 | |
*** jasonb365_ has joined #openstack-trove | 05:18 | |
*** jasonb365 has quit IRC | 05:18 | |
*** jasonb365_ is now known as jasonb365 | 05:18 | |
*** demorris has joined #openstack-trove | 05:27 | |
*** jasonb365 has quit IRC | 05:29 | |
*** rwsu has quit IRC | 05:59 | |
*** michael-yu has joined #openstack-trove | 06:44 | |
*** khyati has quit IRC | 06:51 | |
*** haomai___ has joined #openstack-trove | 07:01 | |
*** haomaiwang has quit IRC | 07:01 | |
*** denis_makogon has joined #openstack-trove | 08:01 | |
*** michael-yu has quit IRC | 08:07 | |
*** igor has joined #openstack-trove | 08:10 | |
*** michael-yu has joined #openstack-trove | 08:20 | |
*** matsuhashi has quit IRC | 08:37 | |
*** michael-yu has quit IRC | 08:41 | |
openstackgerrit | Justin Hopper proposed a change to openstack/trove: Fixed Instance Status usage https://review.openstack.org/66063 | 08:44 |
*** ashestakov has joined #openstack-trove | 08:46 | |
*** haomai___ has quit IRC | 08:47 | |
*** haomaiwang has joined #openstack-trove | 08:48 | |
openstackgerrit | Justin Hopper proposed a change to openstack/trove: Fix Timestamp diff in Instance and Notification https://review.openstack.org/70313 | 09:06 |
*** ashestakov has quit IRC | 09:07 | |
*** igor has quit IRC | 09:23 | |
*** igor has joined #openstack-trove | 09:24 | |
*** tenaglia has quit IRC | 09:26 | |
*** flaper87|afk is now known as flaper87 | 09:35 | |
*** erkules_ is now known as erkules | 09:40 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 09:43 | |
*** matsuhashi has joined #openstack-trove | 09:44 | |
*** SnowDust has joined #openstack-trove | 09:45 | |
*** matsuhashi has quit IRC | 10:05 | |
*** denis_makogon has quit IRC | 10:07 | |
*** matsuhashi has joined #openstack-trove | 10:16 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 10:20 | |
*** ashestakov has joined #openstack-trove | 10:21 | |
openstackgerrit | shivam shukla proposed a change to openstack/trove: Added check for not use heat in security group delete https://review.openstack.org/69176 | 10:33 |
*** haomaiwang has quit IRC | 10:39 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 10:50 | |
*** flaper87 is now known as flaper87|afk | 10:52 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 10:53 | |
*** tenaglia has joined #openstack-trove | 10:53 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 10:54 | |
*** flaper87|afk is now known as flaper87 | 10:55 | |
*** Ranjitha has joined #openstack-trove | 10:57 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 10:59 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 11:11 | |
*** haomaiwa_ has joined #openstack-trove | 11:20 | |
*** Ranjitha_ has joined #openstack-trove | 11:21 | |
*** Ranjitha has quit IRC | 11:21 | |
*** Ranjitha_ is now known as Ranjitha | 11:21 | |
*** jaishanker has quit IRC | 12:15 | |
*** boden has joined #openstack-trove | 12:41 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 12:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 12:54 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov | 12:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 12:55 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov | 12:56 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 13:00 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 13:03 | |
*** jimbobhickville has joined #openstack-trove | 13:07 | |
*** Barker has joined #openstack-trove | 13:27 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 13:28 | |
*** haomaiwa_ has quit IRC | 13:31 | |
*** haomaiwang has joined #openstack-trove | 13:32 | |
*** matsuhashi has quit IRC | 13:41 | |
*** matsuhashi has joined #openstack-trove | 13:50 | |
*** matsuhashi has quit IRC | 13:57 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 14:00 | |
*** demorris has quit IRC | 14:02 | |
*** matsuhashi has joined #openstack-trove | 14:04 | |
*** haomai___ has joined #openstack-trove | 14:14 | |
*** haomaiwang has quit IRC | 14:17 | |
*** Ranjitha has quit IRC | 14:18 | |
*** radez_g0n3 is now known as radez | 14:21 | |
*** jcru has joined #openstack-trove | 14:23 | |
*** demorris has joined #openstack-trove | 14:24 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 14:26 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 14:26 | |
*** matsuhashi has quit IRC | 14:28 | |
*** mrsnivvel has quit IRC | 14:29 | |
*** amytron has quit IRC | 14:29 | |
openstackgerrit | Denis M. proposed a change to openstack/trove: Update delete method for heat flow https://review.openstack.org/70355 | 14:31 |
openstackgerrit | Denis M. proposed a change to openstack/trove: Add stack_id for future usage https://review.openstack.org/67873 | 14:31 |
*** SnowDust has quit IRC | 14:34 | |
*** nosnos has quit IRC | 14:37 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 14:45 | |
*** amytron has joined #openstack-trove | 14:54 | |
*** robertmyers has joined #openstack-trove | 14:58 | |
*** robertmyers has quit IRC | 15:00 | |
*** robertmyers has joined #openstack-trove | 15:01 | |
*** robertmyers has quit IRC | 15:02 | |
*** russellb is now known as rustlebee | 15:02 | |
*** robertmyers has joined #openstack-trove | 15:02 | |
*** robertmy_ has joined #openstack-trove | 15:03 | |
*** robertmyers has quit IRC | 15:03 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 15:04 | |
*** robertmy_ has quit IRC | 15:04 | |
*** robertmy_ has joined #openstack-trove | 15:06 | |
*** robertm__ has joined #openstack-trove | 15:07 | |
*** robertmy_ has quit IRC | 15:07 | |
*** robertm__ has quit IRC | 15:08 | |
*** robertmy_ has joined #openstack-trove | 15:09 | |
*** robertmyers has joined #openstack-trove | 15:10 | |
*** robertmyers has quit IRC | 15:11 | |
*** robertmy_ has quit IRC | 15:11 | |
*** robertmyers has joined #openstack-trove | 15:11 | |
*** kevinconway has joined #openstack-trove | 15:12 | |
*** robertmyers has quit IRC | 15:12 | |
*** robertmyers has joined #openstack-trove | 15:13 | |
*** rwsu has joined #openstack-trove | 15:13 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:14 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 15:15 | |
*** grapex has joined #openstack-trove | 15:16 | |
*** grapex has quit IRC | 15:20 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:20 | |
*** grapex has joined #openstack-trove | 15:21 | |
*** mattgriffin has joined #openstack-trove | 15:22 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 15:29 | |
*** datsun180b has joined #openstack-trove | 15:33 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:34 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 15:43 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 15:44 | |
*** flaper87 is now known as flaper87|afk | 15:47 | |
*** igor has quit IRC | 15:48 | |
*** thedodd has joined #openstack-trove | 15:48 | |
*** ViswaV has joined #openstack-trove | 15:52 | |
*** ViswaV_ has joined #openstack-trove | 15:54 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 15:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 15:56 | |
*** ViswaV has quit IRC | 15:58 | |
*** doddstack has joined #openstack-trove | 16:02 | |
*** demorris_ has joined #openstack-trove | 16:03 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:04 | |
*** thedodd has quit IRC | 16:05 | |
*** demorris has quit IRC | 16:06 | |
*** demorris_ is now known as demorris | 16:06 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 16:19 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 16:20 | |
openstackgerrit | Andrey Shestakov proposed a change to openstack/trove: Initial ceilometer integration https://review.openstack.org/70369 | 16:24 |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:24 | |
*** jmontemayor has joined #openstack-trove | 16:30 | |
*** jasonb365 has joined #openstack-trove | 16:32 | |
cp16net | hub_cap: i've added a bp for the db stuff https://blueprints.launchpad.net/trove/+spec/configuration-parameters-in-db | 16:33 |
*** ramashri has joined #openstack-trove | 16:37 | |
*** ramashri has quit IRC | 16:37 | |
*** SergeyLukjanov is now known as SergeyLukjanov_a | 16:39 | |
*** SergeyLukjanov_a is now known as SergeyLukjanov_ | 16:40 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:41 | |
*** michael-yu has joined #openstack-trove | 16:46 | |
*** kevinconway has quit IRC | 16:49 | |
*** kevinconway has joined #openstack-trove | 16:50 | |
*** flaper87|afk is now known as flaper87 | 16:50 | |
*** jcru has quit IRC | 16:50 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 16:51 | |
*** SergeyLukjanov_ is now known as SergeyLukjanov | 16:54 | |
*** SergeyLukjanov is now known as SergeyLukjanov_ | 16:55 | |
*** michael-yu has quit IRC | 16:56 | |
*** flaper87 is now known as flaper87|afk | 16:58 | |
*** robertmy_ has joined #openstack-trove | 17:00 | |
*** jcru has joined #openstack-trove | 17:03 | |
*** robertmyers has quit IRC | 17:04 | |
*** datsun180b has quit IRC | 17:05 | |
*** jasonb365_ has joined #openstack-trove | 17:07 | |
*** kevinconway has quit IRC | 17:07 | |
*** jasonb365 has quit IRC | 17:07 | |
*** jasonb365_ is now known as jasonb365 | 17:07 | |
*** robertmy_ has quit IRC | 17:13 | |
*** robertmyers has joined #openstack-trove | 17:13 | |
*** shakayumi has joined #openstack-trove | 17:18 | |
*** ViswaV_ has left #openstack-trove | 17:20 | |
*** ViswaV_ has joined #openstack-trove | 17:20 | |
*** ViswaV_ has quit IRC | 17:20 | |
*** shakayum_ has joined #openstack-trove | 17:21 | |
*** ViswaV has joined #openstack-trove | 17:21 | |
*** shakayumi has quit IRC | 17:24 | |
openstackgerrit | Mat Lowery proposed a change to openstack/trove: Get service endpoints from catalog https://review.openstack.org/68015 | 17:41 |
*** ViswaV has quit IRC | 17:44 | |
*** kevinconway has joined #openstack-trove | 17:45 | |
hub_cap | cp16net: k | 17:45 |
*** demorris has quit IRC | 17:48 | |
*** jimbobhickville has quit IRC | 17:50 | |
openstackgerrit | Mat Lowery proposed a change to openstack/trove: Get service endpoints from catalog https://review.openstack.org/68015 | 17:53 |
*** harlowja_away is now known as harlowja | 17:55 | |
*** kevinconway has quit IRC | 17:56 | |
*** diakunchikov has quit IRC | 17:59 | |
*** diakunchikov has joined #openstack-trove | 18:00 | |
*** grapex_ has joined #openstack-trove | 18:00 | |
*** grapex has quit IRC | 18:02 | |
*** kevinconway has joined #openstack-trove | 18:04 | |
*** Barker has quit IRC | 18:05 | |
ashestakov | https://review.openstack.org/#/c/70369/ review pls | 18:05 |
*** shakayum_ has quit IRC | 18:12 | |
*** ViswaV has joined #openstack-trove | 18:14 | |
*** shakayumi has joined #openstack-trove | 18:16 | |
openstackgerrit | Denis M. proposed a change to openstack/trove: Update delete method for heat flow https://review.openstack.org/70355 | 18:17 |
openstackgerrit | Justin Hopper proposed a change to openstack/trove: Fixed Instance Status usage https://review.openstack.org/66063 | 18:17 |
*** doddstack has quit IRC | 18:20 | |
*** ViswaV has quit IRC | 18:24 | |
*** SnowDust has joined #openstack-trove | 18:25 | |
*** amcrn has joined #openstack-trove | 18:29 | |
*** datsun180b has joined #openstack-trove | 18:40 | |
*** yogesh has joined #openstack-trove | 18:41 | |
hub_cap | juice: :) | 18:42 |
juice | don't get too happy yet, I think it failed the integration tests | 18:42 |
juice | but it's close | 18:42 |
*** michael-yu has joined #openstack-trove | 18:43 | |
hub_cap | oh poop | 18:43 |
* hub_cap gets angry | 18:43 | |
* juice covers his head | 18:43 | |
*** demorris has joined #openstack-trove | 18:44 | |
* hub_cap flails arms around, missing juice | 18:44 | |
hub_cap | oh this is like dnd | 18:44 |
* hub_cap rolls a 19 1/2 sided dice, rolls 11 | 18:44 | |
hub_cap | juice: have u seen the key&peele d&d skit? omg | 18:45 |
* juice counters with a 20d rolls a 3, stumbles backwards and falls into trap | 18:45 | |
juice | no but that sounds awesome | 18:45 |
juice | wait maybe I did...better check | 18:45 |
hub_cap | find it right now and watch it :) | 18:45 |
SnowDust | is it abt size ? then i will say 34dd ! | 18:45 |
SnowDust | oops .. | 18:45 |
hub_cap | um not appropriate | 18:45 |
SnowDust | oh its numbers .. | 18:46 |
juice | jeesh SnowDust, this was totally PG | 18:46 |
hub_cap | dungeons and dragons | 18:46 |
juice | short hand for 20 sided die | 18:46 |
SnowDust | anyway expunge .. | 18:46 |
*** ashestakov has quit IRC | 18:46 | |
juice | i haven't even watched it yet but I am laughing already | 18:46 |
hub_cap | juice: go peep it | 18:47 |
hub_cap | all i have to say | 18:47 |
hub_cap | kanye the giant | 18:47 |
*** demorris_ has joined #openstack-trove | 18:49 | |
*** demorris has quit IRC | 18:50 | |
*** demorris_ is now known as demorris | 18:51 | |
SnowDust | i am bored .. | 18:52 |
SnowDust | suggest me some learning .. | 18:52 |
*** igor___ has joined #openstack-trove | 18:53 | |
*** yogesh_ has joined #openstack-trove | 18:55 | |
*** yogesh has quit IRC | 18:55 | |
hub_cap | docker! | 18:58 |
hub_cap | everyone likes docker | 18:58 |
hub_cap | its hotter than hansel (zoolander reference) | 18:59 |
juice | nothing is hotter than hansel | 19:01 |
* amcrn does "blue steel" | 19:01 | |
hub_cap | i have a pic of my son when he was really young doin a kissy face | 19:02 |
juice | I am pretty sure that was "blue magnum" amcrn | 19:02 |
hub_cap | i sent it around saying it was his blue steel (or maybe i said magnum) | 19:02 |
juice | question, question for the room | 19:02 |
juice | what is a generic name for the database service that the guest agent is monitoring? | 19:03 |
hub_cap | larry? | 19:03 |
hub_cap | no srsly im not sure i understand what u mean :) | 19:03 |
hub_cap | u mean datastore_type? | 19:04 |
juice | so the guest agent is monitoring mysql, redis, mongo processes | 19:04 |
juice | and it reports via conductor its status | 19:04 |
hub_cap | sure | 19:04 |
hub_cap | so u wanna know what the json reporting looks like? | 19:04 |
hub_cap | so u know what the value returned (err name) is for the datstore | 19:04 |
*** SnowDust has left #openstack-trove | 19:05 | |
juice | not really | 19:05 |
juice | I was speaking more of in vocabulary terms | 19:05 |
*** SnowDust has joined #openstack-trove | 19:05 | |
hub_cap | ah icic | 19:05 |
hub_cap | i mean, id always call it the datastore type | 19:06 |
hub_cap | no matter where it was in the process | 19:06 |
juice | so you would say 'the datastore type is running"? | 19:06 |
*** thedodd has joined #openstack-trove | 19:06 | |
hub_cap | the datstore is running? | 19:06 |
amcrn | if i were to give it a label, i'd say the "process monitor" is running, because ultimately it's asserting the status of a process. | 19:06 |
juice | great we are on the same page | 19:06 |
juice | end conversation :) | 19:06 |
hub_cap | i mean, the customer is passin gin datastore_type potentiall, and is calling it a datstore with list datastores | 19:06 |
hub_cap | :) | 19:06 |
hub_cap | BREAK | 19:06 |
amcrn | actually, let's make it more Java, call it the ProcessMonitorFactoryProxyBeanSingleton | 19:07 |
*** ViswaV has joined #openstack-trove | 19:07 | |
hub_cap | Dont forget the FactoryFactory amcrn | 19:07 |
juice | amcrn I am not speaking of the agent but the datastore process that the agent is monitoring | 19:07 |
amcrn | feels enterprisey | 19:07 |
amcrn | juice: ah, then i agree with hub_cap | 19:07 |
hub_cap | that spawns singleton factory instances of factories to spin up process monitor beans | 19:07 |
SnowDust | i dont know java :( | 19:08 |
juice | ok let's back up | 19:08 |
juice | you guys are moving too fast for me | 19:08 |
juice | let's start with the factory proxy | 19:08 |
hub_cap | is is templated? | 19:08 |
*** ViswaV_ has joined #openstack-trove | 19:08 | |
hub_cap | can u make any factroy from the factory proxy? | 19:09 |
hub_cap | sorry, a factory factory from the factory proxy? | 19:09 |
hub_cap | god i love java | 19:09 |
juice | I think we should throw a bridge in there just to be safe | 19:09 |
* hub_cap sings "take em to the bridge" | 19:09 | |
hub_cap | cmon, anyone, anyone? | 19:09 |
juice | git-up | 19:09 |
vgnbkr | +1 java. A type, a type, my kingdom for a type. | 19:10 |
hub_cap | LOL | 19:10 |
juice | before we get too far off topic what did hub_cap and amcrn "agree" on | 19:10 |
hub_cap | factory^factory | 19:11 |
*** demorris has quit IRC | 19:11 | |
hub_cap | juice: the conversation was already over when amcrn rolled the java sided dice | 19:11 |
*** ViswaV has quit IRC | 19:12 | |
juice | holy crap | 19:12 |
juice | it is friday isn't it | 19:12 |
hub_cap | :D | 19:12 |
juice | actually a factory is a perfect solution for dice | 19:12 |
*** igor___ has quit IRC | 19:12 | |
juice | dnd dice factory | 19:13 |
juice | c and c music factory | 19:13 |
juice | it just all works | 19:13 |
*** igor__ has joined #openstack-trove | 19:13 | |
juice | is_datastore_running? | 19:14 |
openstackgerrit | Andreas Jaeger proposed a change to openstack/database-api: Add xml:ids to resources https://review.openstack.org/70407 | 19:15 |
*** SnowDust has quit IRC | 19:15 | |
*** jasonb365 has quit IRC | 19:16 | |
juice | aren't we getting rid of xml? | 19:16 |
hub_cap | juice: yes | 19:17 |
vgnbkr | Testing question: when you run int-tests, where does stdout get redirected to? Is there a log file somewhere? I can't find the output that should be being generated. | 19:17 |
hub_cap | yea | 19:17 |
hub_cap | vgnbkr: ^ ^ | 19:17 |
*** demorris has joined #openstack-trove | 19:17 | |
hub_cap | let me find it | 19:17 |
vgnbkr | (the result of "print" statements) | 19:17 |
hub_cap | its like rdtest.log or something | 19:17 |
hub_cap | find . -name '*.log' | 19:17 |
hub_cap | sry | 19:17 |
hub_cap | find ../ -name '*.log' | 19:17 |
*** igor__ has quit IRC | 19:17 | |
amcrn | vgnbkr: trovetest.log i believe | 19:18 |
hub_cap | cuz the other logs are 1 dir up from -integration/scripts | 19:18 |
vgnbkr | Yeah, I got report.log and the logging dir, but I'm not seeing the output I expect. | 19:18 |
hub_cap | but juice what andres is doign is for the database-api | 19:18 |
hub_cap | aka, its all xml | 19:18 |
hub_cap | cuz its our docs :) | 19:18 |
vgnbkr | Ah, haven't come across trovetest.log - let me look. | 19:18 |
hub_cap | hopefully it got renamed amcrn to trovetest from rdtest :P | 19:18 |
hub_cap | u know i dont see that file anymore | 19:19 |
hub_cap | i only see the stuff in ../report/logs | 19:19 |
hub_cap | and at that its only trove* logs | 19:19 |
hub_cap | i think somethign si broken | 19:19 |
vgnbkr | No trovetest.log. Specifically, it's the print statements in initialize.py:Daemon.run() that I'm trying to find. | 19:20 |
hub_cap | yea they used to be logged vgnbkr, and grapex_ did that work (But i think hes afk) | 19:21 |
hub_cap | oh wait i havnet run int-tests eithre ;) | 19:21 |
hub_cap | LOL | 19:21 |
hub_cap | so | 19:21 |
hub_cap | ../report/report.log | 19:21 |
hub_cap | lol is not it | 19:21 |
ViswaV_ | hub_cap: juice: wouldn't you want to call it 'datastore instance' because that mysql (or redis or mongodb) process on the guestagent VM is an 'instance' , datastore_type points to "what kind of datastore this is an instance of" … in fact that process is an amalgamation of "a specific version of a specific datastore_type" ….although 'instance' may mean to represent the whole guest VM… | 19:21 |
hub_cap | but i do see it tyr to log to /var/log/syslog... | 19:22 |
juice | viswav_: hub_cap: this is within the context of a trove instance...so it feels to me adding instance to datastore is redundant, no? | 19:23 |
hub_cap | ViswaV_: so yer saying 'datastore instance' and not just datastore? | 19:24 |
juice | instance.datastore_instance or simply instance.datastore | 19:24 |
hub_cap | yes | 19:24 |
hub_cap | instance.datastore | 19:24 |
hub_cap | man im hungry! | 19:24 |
* hub_cap is random today | 19:24 | |
juice | come on up to seattle, we are eating chinese food today | 19:24 |
juice | happy lunar new year for those of you that celebrate | 19:24 |
vgnbkr | Thanks. Something must be redirecting to/dev/null. int-tests can't start trove-api, but I can start it manually no problem. | 19:25 |
ViswaV_ | juice: agree….as I mentioned instance might already mean the whole VM. instance.datastore sounds appropriate. | 19:25 |
*** ViswaV_ is now known as ViswaV | 19:25 | |
juice | there is definitely a hierarchy that needs to be noted | 19:25 |
hub_cap | juice: ++ yup | 19:26 |
hub_cap | yum* | 19:26 |
juice | currently in trove an instance encapsulates the nova compute instance and a datastore running on that compute instance | 19:26 |
hub_cap | that and saying database instead of instance people might think they get 1 database per server (not one database _server_) so to speak | 19:27 |
*** SnowDust has joined #openstack-trove | 19:28 | |
juice | good point, how this might change when we add clustering? fairly big implications for the code base and status reporting no? unless we plan on treating each node as a separate trove instance (datastore + compute) | 19:28 |
vgnbkr | Not sure it it's relevant here, but I believe that going forward a trove database would consist of a set of nova instances with a datastore implemented across them. | 19:29 |
hub_cap | ugh i wonder how random people from places get my phone # | 19:30 |
juice | vgnbkr: its is not immediately relevant but you are answering what will soon become a reality in trove | 19:30 |
juice | Facebook, linkedin? | 19:30 |
hub_cap | yea in some aspect | 19:30 |
hub_cap | nope i dont list i | 19:30 |
hub_cap | i wonder if percona conf gives it out | 19:31 |
hub_cap | i think i put my actual # last yr on the eventbrite like a moron | 19:31 |
juice | I don't want to propose some burdensome effort but would it be wise to have a "dictionary" for trove...perhaps in the wiki? | 19:31 |
juice | there maybe only a collection of 10 terms or so but it might help us better communicate in the chat room and write more succinct code | 19:32 |
*** datsun180b_ has joined #openstack-trove | 19:32 | |
*** datsun180b has quit IRC | 19:33 | |
*** datsun180b_ is now known as datsun180b | 19:33 | |
hub_cap | juice: u woudlnt be the first to propse it | 19:38 |
hub_cap | but im not sure it ever got done | 19:38 |
hub_cap | id prefer it in the code somewhere | 19:38 |
hub_cap | as well as in database-api | 19:38 |
hub_cap | kevinconway: hey did u | 19:39 |
hub_cap | trove stop | 19:39 |
hub_cap | trove hilighting | 19:39 |
hub_cap | trove on | 19:39 |
hub_cap | trove? | 19:39 |
ViswaV | "dictionary" for trove +1 | 19:45 |
*** denis_makogon has joined #openstack-trove | 19:46 | |
openstackgerrit | Mat Lowery proposed a change to openstack/trove: Add clearer help kwarg for max_volumes_per_user https://review.openstack.org/70205 | 19:47 |
juice | In the code is a good idea - at the primary class level is appropriate | 19:48 |
hub_cap | +1 juice | 19:48 |
denis_makogon | hi, guys | 19:49 |
hub_cap | howdy | 19:49 |
denis_makogon | i've got one question to all | 19:49 |
denis_makogon | integration uses precise-cloud-init | 19:50 |
denis_makogon | any objections of moving to quantal ? | 19:50 |
denis_makogon | and also, is dib broken or kick-start ? | 19:52 |
*** flaper87|afk is now known as flaper87 | 20:01 | |
*** ViswaV has quit IRC | 20:02 | |
SnowDust | denis_makogon is quantal a LTS ? | 20:07 |
*** flaper87 is now known as flaper87|afk | 20:10 | |
*** datsun180b has quit IRC | 20:10 | |
denis_makogon | SnowDust, https://wiki.ubuntu.com/ServerTeam/CloudArchive?action=AttachFile&do=get&target=plan-revised.png | 20:12 |
denis_makogon | 14.04 would LTS | 20:12 |
denis_makogon | *would be | 20:13 |
SnowDust | denis_makogon, short ans pls :) | 20:13 |
SnowDust | hmm .. | 20:13 |
SnowDust | so we should only support LTS | 20:13 |
denis_makogon | redstack told me "not tested" | 20:13 |
SnowDust | same is my view with fedora( it should be centos instead ) | 20:14 |
denis_makogon | so, if tests would pass on quantal, it means that we could add quatal as possible variant | 20:14 |
SnowDust | fedora has 6 months release cycle .. compared to EL distros which have 2 years | 20:14 |
denis_makogon | also there's problem in 12.04 LTS | 20:15 |
SnowDust | denis_makogon, if a pluggable logic is possible then custom is king ( here the cloud providers ) | 20:15 |
SnowDust | customer* | 20:15 |
denis_makogon | start-stop-deamon doesn't work with some services | 20:16 |
SnowDust | u mean .. start/stop trove-guest ? | 20:16 |
*** jasonb365 has joined #openstack-trove | 20:16 | |
*** jimbobhickville has joined #openstack-trove | 20:18 | |
denis_makogon | no | 20:22 |
denis_makogon | services which installed from packages | 20:22 |
*** khyati_ has joined #openstack-trove | 20:23 | |
juice | @all - the gate looks hosed right now due to glance image upload issues | 20:26 |
juice | please standby while troubleshoot what is going on | 20:26 |
hub_cap | denis_makogon its not up to us to decide what distros openstack uses | 20:29 |
hub_cap | if they are now installing tests and claiming to work w/ quantal, we will use it | 20:29 |
denis_makogon | ubuntu blog says that quantal is safe for have release | 20:30 |
denis_makogon | *havanna | 20:30 |
*** grapex_ has quit IRC | 20:37 | |
*** ViswaV has joined #openstack-trove | 20:37 | |
*** grapex has joined #openstack-trove | 20:37 | |
*** datsun180b has joined #openstack-trove | 20:40 | |
*** ashestakov_phone has joined #openstack-trove | 20:44 | |
*** flaper87|afk is now known as flaper87 | 20:45 | |
*** SnowDust has quit IRC | 20:45 | |
hub_cap | ubuntu blog != openstack | 20:47 |
hub_cap | go into opesntack-infra and ask them | 20:47 |
*** flaper87 is now known as flaper87|afk | 20:48 | |
hub_cap | denis_makogon: plz report back when finished so the channel knows what then answer is | 20:51 |
denis_makogon | hub_cap, of course | 20:52 |
hub_cap | <3 | 20:53 |
datsun180b | oh man 019_datastore_fix has been my bane all day | 20:56 |
datsun180b | talking with grapex i think i know what's wrong now | 20:56 |
hub_cap | ? | 20:56 |
denis_makogon | Asnwer: so for a variety of reasons, we decided to stick with lts releases | 20:56 |
grapex | hub_cap vipul SlickNik: There's a bug in 019_datastore_fix | 20:57 |
datsun180b | so if i try to add my conductor_lastseen table as migration 20, 19 fails because orm.mappers makes reference to the table before it's been created | 20:57 |
vipul | crap... what is the issue | 20:57 |
grapex | It configures the database to work with the data, basically connecting to it, which initializes all models | 20:57 |
grapex | The problem is, datsun180b wants to add a table | 20:57 |
grapex | So his new table, defined by a 020 migration file, is not ready by the time the 019 migration file runs "configure_db" | 20:57 |
* datsun180b used BATON PASS! Return, datsun180b! Go, grapex! | 20:57 | |
jimbobhickville | isn't this already being worked on? or is this a new different one? | 20:57 |
grapex | We need to change 019. Possibly it should be added to contrib if we can't figure out a way to add a default datastore without the full availability of the sqlalchemy models | 20:58 |
jimbobhickville | cp16net and amcrn had something in the works | 20:58 |
vipul | so there was a fix merged yesterday | 20:58 |
datsun180b | adding migration 020 (my table) is no problem, but migrate fails if i add the mapping to session.py | 20:58 |
datsun180b | well let me make sure i'm absolutely using the latest and greates | 20:58 |
datsun180b | t | 20:58 |
vipul | https://github.com/openstack/trove/blob/master/trove/db/sqlalchemy/migrate_repo/versions/019_datastore_fix.py | 20:59 |
vipul | we removed the use of the db models | 20:59 |
jimbobhickville | yeah, they removed the reliance on the session and used sqlalchemy directly to do the queries | 20:59 |
datsun180b | well poop | 20:59 |
jimbobhickville | and I learned that I know jack-all about how sqlalchemy works, because I find this whole thing insane | 20:59 |
grapex | WELL SHEESH DATSUN180B! | 21:00 |
datsun180b | hey hey i'm just calling it like i'm seeing it | 21:00 |
grapex | vipul: Sorry, I looked at it and didn't notice the difference between what I'd looked at yesterday | 21:00 |
datsun180b | same, i thought i already _had_ the fix what wit the religious use of git fetch --all and pull --rebase | 21:01 |
grapex | datsun180b: You must have missed your morning sacrament. | 21:01 |
vipul | no worries, let's just hope this new version works :) | 21:01 |
kevinconway | yeah, you don't want to use the session and models in your DDL | 21:01 |
datsun180b | well if it doesn't you'll hear about it from me | 21:01 |
jimbobhickville | and let the fight for the 020 filename begin! I lost the battle for 017 and 018 :( | 21:02 |
*** demorris_ has joined #openstack-trove | 21:02 | |
datsun180b | You merely adopted the rebase. I was born in it! | 21:05 |
*** demorris__ has joined #openstack-trove | 21:05 | |
*** demorris has quit IRC | 21:05 | |
*** demorris__ is now known as demorris | 21:05 | |
*** jasonb365 has quit IRC | 21:06 | |
*** demorris_ has quit IRC | 21:06 | |
datsun180b | $ git rebase --continue | 21:07 |
datsun180b | Applying: adding lastseen table | 21:07 |
datsun180b | Applying: argh what is this | 21:07 |
*** jimbobhickville has quit IRC | 21:07 | |
grapex | datsun180b: Thank you Bane | 21:08 |
openstackgerrit | A change was merged to openstack/python-troveclient: Add Neutron support https://review.openstack.org/61481 | 21:11 |
cp16net | lol | 21:11 |
hub_cap | grapex: sweet on the approves there (and validation) | 21:12 |
hub_cap | aka making sure the actual int-tests pass | 21:13 |
cp16net | amcrn: it makes me laugh a little that you use Rainicorn for the name of your test instance :-P | 21:16 |
juice | datsun180b: nice | 21:16 |
*** mmcdaris has joined #openstack-trove | 21:19 | |
datsun180b | i hope i just never slip up and find myself on @CLFLN | 21:21 |
datsun180b | oh uh work/language warning on that | 21:21 |
hub_cap | amcrn: WIN!!! https://github.com/hub-cap/lady-rainicorn | 21:21 |
cp16net | hub_cap: vipul SlickNik grapex: can you weigh in on the first comment on this review plz? https://review.openstack.org/#/c/53168/20/trove/configuration/service.py | 21:22 |
hub_cap | juice: youve submitted obth of those eh? | 21:22 |
hub_cap | the one from amcrn | 21:22 |
grapex | cp16net: in a bit | 21:22 |
cp16net | thx | 21:22 |
hub_cap | cp16net: the one from amcrn? | 21:23 |
cp16net | yes | 21:23 |
hub_cap | kk | 21:23 |
cp16net | its a small ask | 21:23 |
*** boden has quit IRC | 21:23 | |
juice | hub_cap: which both of those are you referring to | 21:24 |
hub_cap | this goes back to a question i had but never vinished on your review cp16net | 21:24 |
hub_cap | why do we have an option to turn off auto apply to dynamics | 21:24 |
juice | if someone wants to double up and take a look at why our devstack image upload no longer works that would be great | 21:24 |
hub_cap | if we have auto apply to dynamics, _no matter what_, then we never have a scenario where it looks like we need a restart for a dynamic var | 21:25 |
juice | I am in the process of building a fresh devstack in hp cloud | 21:25 |
hub_cap | juice: yea i just refreshed the revie page and saw the 2nd patch and the dependent one | 21:25 |
juice | yes | 21:25 |
juice | I haven't rebased the second one off of the pep8 fix that was broken in the first | 21:26 |
juice | however, I won't submit anything else until the gate is functioning again | 21:26 |
hub_cap | sure juice | 21:27 |
hub_cap | i can wait :) | 21:27 |
hub_cap | weve waited this long!!! | 21:27 |
hub_cap | and even after we +2 it heh, it wil take ~16 hrs to merge | 21:27 |
juice | ha ha - yes i know | 21:27 |
* juice lowers his head in shame | 21:28 | |
hub_cap | oh its not your fault | 21:28 |
juice | ribbons of shame | 21:28 |
hub_cap | i blame vipul | 21:28 |
hub_cap | ;) | 21:28 |
ViswaV | "devstack image upload no longer works" is this what broke the gate? | 21:28 |
juice | did anyone see "gung-ho"? | 21:28 |
vipul | the gate gods is not I :p | 21:28 |
juice | viswav: yes where did you see that? | 21:28 |
ViswaV | juice: lifted from hub_cap comment above.. | 21:30 |
juice | oh - he he | 21:30 |
juice | tee hee | 21:30 |
ViswaV | btw, what exactly does that mean…? image upload to glance in devstack (via dib) is broken? | 21:31 |
juice | well after dib completes but yes | 21:31 |
juice | cannot connect to host is what it gets | 21:32 |
datsun180b | can't upload an image until it's built | 21:32 |
denis_makogon | hub_cap, could you please approve this one https://blueprints.launchpad.net/trove/+spec/stack-id ? | 21:36 |
denis_makogon | hub_cap, i took into account your comments | 21:36 |
hub_cap | denis_makogon: this bp for i3 you are saying? | 21:37 |
denis_makogon | hub_cap, trove next | 21:37 |
hub_cap | this is the new smaller blueprint, right? | 21:37 |
denis_makogon | yes | 21:37 |
hub_cap | so u dont want this in for icehouse? | 21:37 |
denis_makogon | only stack id | 21:37 |
hub_cap | cuz trove next does _not_ mean icehouse | 21:37 |
denis_makogon | hub_cap, i think it will be in I3 | 21:37 |
hub_cap | ive changed to i3 | 21:38 |
denis_makogon | thanks | 21:38 |
hub_cap | np | 21:38 |
denis_makogon | could you please tell when juno opens ? | 21:38 |
datsun180b | is that what we're calling j? | 21:39 |
vipul | :q | 21:39 |
vipul | whoops | 21:39 |
hub_cap | datsun180b: yes a while ago | 21:39 |
hub_cap | vipul: irc != vimi | 21:39 |
hub_cap | *vim | 21:39 |
hub_cap | denis_makogon: its on the wiki page somewhere, id have to search it like you | 21:39 |
datsun180b | well how about that | 21:39 |
hub_cap | search for icehouse release | 21:40 |
hub_cap | there is a nice graph | 21:40 |
denis_makogon | hub_cap, ok, i'll search | 21:40 |
denis_makogon | guys, since you are all here, could you please review this one https://review.openstack.org/#/c/67873/ | 21:40 |
datsun180b | am i the only one that sees the wiki as some kind of giant magic 8-ball that can answer short queries but only after a moderate amount of shaking | 21:40 |
denis_makogon | hub_cap, https://review.openstack.org/#/c/63789/ could you take a look at updated patch ? | 21:41 |
denis_makogon | datsun180b, wiki its something like Hidden Room (from Harry Potter books) | 21:42 |
datsun180b | but that room "only appears when a person is in great need of it." | 21:44 |
denis_makogon | we want it, in any cases | 21:44 |
hub_cap | denis_makogon: plz update your commit msgs to refer to the new blueprint | 21:45 |
hub_cap | (stack-id one) | 21:45 |
denis_makogon | hub_cap, of course | 21:45 |
hub_cap | cuz u just told a bunch of people to review code i -2'd | 21:45 |
datsun180b | what the wiki needs is some governance and structure enforcement | 21:45 |
denis_makogon | guys, this patches been hanging almost two month (or less) https://review.openstack.org/#/c/64302/ https://review.openstack.org/#/c/64303/ | 21:46 |
hub_cap | im not sure that people will want to review the code if they see a -2 denis_makogon | 21:46 |
datsun180b | and i don't say that just because i know a taxonomist or two | 21:46 |
denis_makogon | hub_cap, i hope that they would read comments below | 21:46 |
hub_cap | and why not just ask me to remove the -2 first and fix the comment? | 21:47 |
hub_cap | therefore removing any assumption that they might do what u say | 21:47 |
hub_cap | :) | 21:47 |
vipul | SlickNik: https://github.com/openstack-dev/devstack/commit/465653e25f0379891c30c5ef2eb8ed8c286f972a | 21:47 |
denis_makogon | ok, i'll do that | 21:47 |
vipul | SlickNik: that's the culprit | 21:47 |
hub_cap | cp16net: ^^ | 21:47 |
SlickNik | Ah, good call vipul | 21:48 |
hub_cap | cp16net: has been having issues all day too | 21:48 |
SlickNik | I'm looking at why we're using localhost instead of the HOST_IP in redstack | 21:48 |
datsun180b | man, whenever i updated and rebase i feel like the kid who got all the good video games five years out of date and is only now getting all the funny jokes | 21:48 |
vipul | keystone/keystone.conf:admin_bind_host = 10.6.25.41 | 21:50 |
vipul | glance/glance-api.conf:bind_host = 0.0.0.0 | 21:50 |
vipul | so it seems like keystone is the only one that actually inisets that | 21:50 |
SlickNik | vipul: https://github.com/openstack/trove-integration/blob/master/scripts/functions#L55-56 | 21:51 |
SlickNik | ^^it's hardcoded to localhost instead of using the actual host ip | 21:51 |
SlickNik | let me check the fix of using the actual host IP | 21:51 |
juice | so locally i am getting a host not found for keystone | 21:52 |
juice | http://localhost:35357/v2.0/tokens | 21:52 |
hub_cap | blame me ;) | 21:52 |
vipul | juice: yea, it's actually binding itself to the private ip instead of localhost | 21:53 |
*** openstackgerrit has quit IRC | 21:53 | |
*** openstackgerrit has joined #openstack-trove | 21:53 | |
hub_cap | good thing we already have a function to get the ip addr | 21:55 |
hub_cap | so it should be a easy fix | 21:55 |
juice | yep that will do it - good catch vipul | 21:55 |
openstackgerrit | SlickNik proposed a change to openstack/trove-integration: Fixed retrieve_token function to use KEYSTONE_AUTH_HOST https://review.openstack.org/70451 | 21:55 |
SlickNik | ^^ easier done than said. :) | 21:56 |
hub_cap | sweet | 21:56 |
hub_cap | im going to manually verify | 21:57 |
juice | slicknik: there are a couple of other spots that need to change | 21:57 |
hub_cap | and then just +2approe | 21:57 |
juice | nova-client uses localhost for the os-auth-url | 21:57 |
SlickNik | what port? | 21:58 |
juice | same port | 21:58 |
juice | different host | 21:58 |
SlickNik | gotcha | 21:58 |
SlickNik | yeah I see it in redstack | 21:58 |
SlickNik | good call juice | 21:58 |
hub_cap | issue another one SlickNik im still unstacking | 21:59 |
hub_cap | juice: i assume if i get a created instance im good to go? | 21:59 |
juice | and try nova-client | 21:59 |
hub_cap | u mean redstack nova-client | 21:59 |
juice | right | 21:59 |
hub_cap | lets just remove all that shit | 21:59 |
SlickNik | yes, ./redstack nova-client is busted because of this too. | 22:00 |
juice | the passthroughs? | 22:00 |
hub_cap | there is no reason for us to have that | 22:00 |
hub_cap | yes | 22:00 |
hub_cap | . ~/devstack/openrc | 22:00 |
hub_cap | nova blah | 22:00 |
hub_cap | trove blah | 22:00 |
hub_cap | glance blah | 22:00 |
hub_cap | cinder blah | 22:00 |
hub_cap | win blah | 22:00 |
SlickNik | +2 approved. | 22:00 |
SlickNik | to removing all the passthroughs | 22:00 |
SlickNik | lol | 22:00 |
datsun180b | hub_cap: why unstack when you can pgrep -fl python | awk '{print $1;}' | xargs kill | 22:00 |
hub_cap | ya ill push a review if none else does | 22:00 |
juice | what about swift blah hub_cap? | 22:01 |
datsun180b | i don't trust pkill | 22:01 |
hub_cap | does it also restart tgt and clean all my voluemes datsun180b ?? ;) | 22:01 |
SlickNik | I'll push one up. I'm already there now. | 22:01 |
hub_cap | SlickNik: <3 | 22:01 |
hub_cap | juice: swift blah | 22:01 |
datsun180b | hub_cap: well if you want to get the job done right you do it from orbit anyway | 22:01 |
juice | thanks | 22:01 |
hub_cap | SlickNik: u doin the 2nd patch and the removal together? | 22:03 |
hub_cap | or u dont need a 2nd patch snce we ar enuking it? | 22:03 |
*** demorris has quit IRC | 22:03 | |
SlickNik | hub_cap: Gonna push the first patch as is to unblock the build. | 22:04 |
SlickNik | 2nd patch is gonna nuke the pass throughs; and since the code is gone, we don't need the keystone URL cleanup in that case. | 22:04 |
SlickNik | sound good juice / hub_cap? | 22:04 |
juice | sounds good to me | 22:05 |
*** mmcdaris has left #openstack-trove | 22:06 | |
juice | slicknik - will it get through the tests with just that one change? | 22:09 |
SlickNik | yes, I think it should. I tested that the image is built successfully. | 22:09 |
SlickNik | (and uploaded) | 22:09 |
hub_cap | SlickNik: ++ | 22:10 |
hub_cap | i figured as much, and was already testing the first patch | 22:10 |
*** kevinconway has quit IRC | 22:13 | |
*** shakayumi has quit IRC | 22:14 | |
hub_cap | SlickNik: | 9486081a-0ce1-4792-8f83-cd0d5b00996e | ubuntu_mysql | qcow2 | ovf | 431411200 | active | | 22:14 |
hub_cap | good to go | 22:14 |
*** shakayumi has joined #openstack-trove | 22:14 | |
hub_cap | +2 aproved | 22:14 |
openstackgerrit | A change was merged to openstack/trove-integration: Fixed retrieve_token function to use KEYSTONE_AUTH_HOST https://review.openstack.org/70451 | 22:14 |
juice | I may have done something wrong to corrupt my environment but I am not able to get tests to pass | 22:16 |
juice | there are similar issues within trove-api when it tries to use localhost for keystone | 22:16 |
hub_cap | well poopooo | 22:17 |
juice | so somewhere in either devstack or trove-integration that service url is getting set incorrectly | 22:17 |
juice | will continue to look | 22:17 |
juice | current symptoms are that flavors tests fail and then others later | 22:18 |
hub_cap | dropdown dates for calendar = fail | 22:18 |
juice | yes a major f'ing fail | 22:18 |
juice | so much easier than typing | 22:18 |
juice | I am going to create a drop down that has day/month/year all combined as choices | 22:19 |
juice | that is going to be - THE bomb | 22:19 |
hub_cap | um woah | 22:20 |
SlickNik | yup, I think there's more stuff that needs to be fixed in the trove integration tests. | 22:22 |
SlickNik | juice: are you seeing "Unauthorized" errors in the flavor-api tests? | 22:23 |
juice | in the api log (screen) | 22:23 |
juice | cannot connect to keystone | 22:23 |
SlickNik | "2014-01-31 22:21:34.072 TRACE keystoneclient.middleware.auth_token NetworkError: Unable to communicate with keystone" | 22:24 |
juice | I wonder because I made the correction too late in the process | 22:27 |
juice | is this test.conf used at all? | 22:28 |
esp | +1 dropdown :) | 22:28 |
juice | in conf directory | 22:28 |
juice | weird - my endpoints according to nova are correct | 22:30 |
SlickNik | juice: what does the [filter:authtoken] section in /etc/trove/api-paste.ini say? | 22:31 |
ViswaV | troveclient is using localhost at lot of places.. for the auth endpoint | 22:31 |
ViswaV | troveclient/compat/auth.py | 22:31 |
ViswaV | 176: return "%s/%s" % ('http://localhost:8779/v1.0', | 22:31 |
ViswaV | troveclient/compat/common.py | 22:31 |
ViswaV | 161: "version. Default: http://localhost:5000/v2.0") | 22:31 |
ViswaV | just some e.g.… may be something similar is affecting the api tests? | 22:32 |
ViswaV | my trove cli commands are failing with 'Unauthorized' error... | 22:32 |
juice | slicknik - oh 127.0.0.1 | 22:33 |
juice | yours? | 22:33 |
SlickNik | ViswaV: I believe those are defaults as examples. The actual value should be governed by what is set in the api-paste.ini in /etc/trove | 22:33 |
juice | that is for auth token | 22:33 |
SlickNik | yeah juice. Same. That's the issue. | 22:33 |
juice | filter:tokenauth :) is correct | 22:33 |
SlickNik | Well _one_ issue at least. | 22:33 |
SlickNik | juice: can you gist me your api-paste.ini? | 22:34 |
SlickNik | Yeah, devstack is writing the correct value to filter:tokenauth. | 22:34 |
datsun180b | i'm getting the same Unauthorized as ViswaV at the moment, was about to dig into my configs for other localhost instances too | 22:35 |
juice | that fixed the initial set of tests | 22:35 |
juice | changing that token | 22:36 |
*** jmontemayor has quit IRC | 22:36 | |
SlickNik | datsun180b: change [filter:authtoken] section in /etc/trove/api-paste.ini to use the actual host IP instead of 127.0.0.1 | 22:37 |
juice | slicknik https://gist.github.com/juicegit/8e623bff788600a0b176 | 22:37 |
datsun180b | yeah i'm following along | 22:37 |
ViswaV | SlickNik, that trove/api-paste.ini change does not seem to be enough | 22:42 |
ViswaV | I actually noticed that the trove-api process fails during startup | 22:42 |
ViswaV | keystone middleware failure… probably some other localhost/127.0.0.1 entry in keystone client/api related conf files? | 22:43 |
SlickNik | yeah ViswaV looking to see what else may be the issue. | 22:44 |
SlickNik | juice: https://github.com/openstack/trove/commit/cb58f8ca0d807cb2ece78afc923f4b2a1d344c2e may explain why we have 2 filters 'authtoken' and 'tokenauth' | 22:44 |
ViswaV | Ah… had to go into the screen session for tr-api, CTRL-C and then start fresh screen session for tr-api. Now my "trove list" is working | 22:45 |
ViswaV | May be the tr-api just got hung? | 22:46 |
hub_cap | i thought we fixed that? | 22:46 |
SlickNik | ViswaV: protip: use "./redstack stop" to stop all trove services, and "./redstack start" to start all trove services | 22:46 |
hub_cap | aka, removed the filter that wasnt proper named | 22:46 |
juice | slicknik so something is dynamically updating the configs for the new authtoken but leaving our hardcoded one behind | 22:46 |
ViswaV | SlickNik: Thx for the tip ! | 22:47 |
SlickNik | hub_cap: we fixed it in trove. The devstack scripts are still writing the correct IPs to the old filters (i.e. to tokenauth) | 22:47 |
*** robertmyers has quit IRC | 22:48 | |
SlickNik | see juice's gist for example: https://gist.github.com/juicegit/8e623bff788600a0b176 | 22:48 |
*** ashestakov has joined #openstack-trove | 22:48 | |
SlickNik | ViswaV: if you want to stop / start all services ( nova, cinder, glance, etc) use "./redstack stop_deps" and "./redstack start_deps" instead | 22:48 |
SlickNik | ViswaV: the api-paste.ini settings get read and set only on startup, so it makes sense that you had to restart. | 22:49 |
SlickNik | hub_cap: https://github.com/openstack-dev/devstack/blob/master/lib/trove#L132-139 | 22:50 |
ashestakov | hey guys, what with neutron stuff? | 22:50 |
ViswaV | SlickNik: I did attempt to restart (using a screen killing/starting handy function I had)….but for some reason it did not work properly. I noticed it only after attaching to screen and still seeing the process hanging... | 22:51 |
SlickNik | ViswaV: ah, gotcha | 22:51 |
juice | slicknik: are you submitting the patch for devstack? | 22:51 |
juice | how long does that take to get approved...based on previous history | 22:51 |
SlickNik | juice: yeah. | 22:51 |
SlickNik | It should be a small change, so hopefully not too long. | 22:52 |
SlickNik | testing it out now. | 22:52 |
*** ramashri has joined #openstack-trove | 22:53 | |
*** michael-yu has quit IRC | 22:55 | |
hub_cap | <3 SlickNik | 23:00 |
hub_cap | ashestakov: trove is failing int-test right now, so everything else is 2nd | 23:00 |
hub_cap | but iirc, we merged the client so we can test the trove review | 23:00 |
hub_cap | and then grapex was going to approve it | 23:00 |
*** ashestakov_phone has quit IRC | 23:00 | |
grapex | hub_cap: Should I not have merged that? | 23:02 |
hub_cap | no its fine grapex | 23:02 |
hub_cap | that, in no way, broke int-tests | 23:02 |
hub_cap | ashestakov: was just asking about its progress i think grapex | 23:03 |
grapex | hub_cap: Ok... I have to head out now. Talk to you all later! | 23:03 |
hub_cap | grapex: im goin to approve that review ok | 23:03 |
*** jcru has quit IRC | 23:03 | |
hub_cap | once int-tetss pass on it | 23:03 |
grapex | Ok | 23:03 |
hub_cap | if u arent around this eve | 23:03 |
grapex | hub_cap: Thanks | 23:03 |
hub_cap | <3 grapex | 23:03 |
hub_cap | cu | 23:03 |
* grapex blushes | 23:03 | |
hub_cap | :P | 23:03 |
*** yogesh_ has quit IRC | 23:03 | |
* grapex giggles in a high pitched voice | 23:03 | |
openstackgerrit | Michael Basnight proposed a change to openstack/trove-integration: Migrating trove to entry points https://review.openstack.org/64924 | 23:05 |
amcrn | lol @ you two | 23:07 |
SlickNik | heh | 23:09 |
SlickNik | fyi: https://review.openstack.org/#/c/70466/ | 23:09 |
*** radez is now known as radez_g0n3 | 23:09 | |
SlickNik | hub_cap, juice, ViswaV, vipul, et al. | 23:09 |
ViswaV | Yes SlickNik .. | 23:11 |
ViswaV | thx. | 23:11 |
SlickNik | np, asking for reviews in openstack-qa to see if anyone is around to unblock us. | 23:12 |
*** michael-yu has joined #openstack-trove | 23:14 | |
hub_cap | oh mang | 23:14 |
ViswaV | That change is required on top of the api-paste.ini auth_host setting change? Or does that code take care of that? | 23:14 |
ViswaV | SlickNik: ^^ | 23:15 |
ViswaV | Seems like code change alone would be enough to take care of it, right? | 23:16 |
SlickNik | ViswaV: that change should take care of writing the correct host IP value to the api-paste.ini auth_host setting. | 23:16 |
SlickNik | Yes, it should take care of it so that you don't have to do anything. | 23:17 |
hub_cap | dear new rackspace CP, where the hell did u put my api token | 23:18 |
ViswaV | SlickNik: do I have to unstack/stack after cherry picking your change? | 23:22 |
ashestakov | token.. how about add possibility to troveclient for auth by token? | 23:22 |
SlickNik | ViswaV: If you changed the api-paste by hand, you don't need the devstack change. | 23:23 |
ViswaV | Got it. | 23:23 |
*** amytron has quit IRC | 23:24 | |
*** ramashri has quit IRC | 23:29 | |
hub_cap | why has it taken me so long to get hip to virtualenvwrapper??? | 23:34 |
hub_cap | oh the days of little venvs everywhere, gone! | 23:35 |
*** thedodd has quit IRC | 23:37 | |
*** ashestakov has quit IRC | 23:38 | |
*** ramashri has joined #openstack-trove | 23:39 | |
*** ramashri has quit IRC | 23:42 | |
juice | hub_cap: now you have me interested | 23:47 |
openstackgerrit | Tim Simpson proposed a change to openstack/trove: Changing DNS to pass string to driver https://review.openstack.org/69480 | 23:51 |
*** grapex has quit IRC | 23:52 | |
hub_cap | SlickNik: any luck w/ the trove stuff in -qa? | 23:52 |
hub_cap | hub_cap: dude if u havent mucked w/ it yet, get it | 23:53 |
*** datsun180b has quit IRC | 23:53 | |
hub_cap | it puts your venvs in .virtaul.... or something | 23:53 |
hub_cap | and changes your paths like virtualenv does | 23:53 |
hub_cap | but u never have to cd to a dir | 23:53 |
hub_cap | :) | 23:53 |
SlickNik | hub_cap, you might be typing into the wrong window. | 23:53 |
hub_cap | or source anything | 23:53 |
hub_cap | naw im talking to juice a bout venvwrapper | 23:53 |
SlickNik | ahhhh | 23:54 |
SlickNik | missed that. | 23:54 |
SlickNik | Just saw your question, and a bunch of venv stuff. :) | 23:54 |
juice | that is pretty sweet - sounds much more maintainable than the directory based one | 23:54 |
juice | I always forget where I stashed certain envs | 23:54 |
hub_cap | LOL SlickNik but since i have your attn i do want to know if youve gotten trction | 23:54 |
juice | though that might be a personal problem | 23:55 |
hub_cap | juice: me too, and i have a venv for interacting w/ cloud servers in rax | 23:55 |
hub_cap | so its much nicer to just use this | 23:55 |
SlickNik | hub_cap: we now have a non-voting tempest job running in openstack-ci | 23:55 |
juice | kinda like supernova | 23:55 |
SlickNik | have you seen that? | 23:55 |
SlickNik | I'm working on adding trove tests to that. | 23:56 |
SlickNik | https://review.openstack.org/#/c/69501/ is the first in the series of tests. | 23:56 |
*** denis_makogon has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!