| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/104023 | 00:00 |
|---|---|---|
| vipul | so there is a 'default' set of capabilities that once you upgrade/or install trove new -- you get. Deployers from then on have the option of exposing additional conf values by using the mgmt api to add them as capabilities in the DB | 00:00 |
| amrith | 1. when code wants a configuration value it uses a single mechanism. let me call it GETCONF | 00:00 |
| amrith | 2. GETCONF first looks at a set of capability overrides | 00:00 |
| amrith | 3. GETCONF will look at a conf file if no capability overrides are found | 00:01 |
| amrith | 4. an operator can store their preferred values into a conf file | 00:01 |
| amrith | 5. a trove release will ship with an empty capabilities database ALWAYS, every time | 00:01 |
| amrith | 6. when a user updates a conf with the SETCONF call, it makes a capability override | 00:01 |
| amrith | 7. there is an optional migration script that will take a conf file and make it a capability set. | 00:02 |
| amrith | <over> | 00:02 |
| vipul | So one issue i see with that summary is.. how do we know that a CONF is meant to be exposed, if it doesn't exist as a capability and just as a conf option | 00:02 |
| vipul | I don't want to expose every single thing in trove.conf through the API | 00:03 |
| iccha | vipul: list capabilties as a config | 00:03 |
| SlickNik | vipul: Yes, youd have to have a list of confs that are "promoted". | 00:03 |
| iccha | capabiltities_list=volume_support,users, | 00:03 |
| vipul | iccha: a new cfg option.. that's a list? ok that could work | 00:03 |
| SlickNik | Now whether you want to expose that as a capability or not, is up to the deployer. :P | 00:03 |
| iccha | phew this sounds almost like a work around to override config values without service restarts | 00:04 |
| SlickNik | amrith: Summary looks okay to me up till 6. 7 doesn't seem all that important. | 00:04 |
| *** michael-yu has quit IRC | 00:04 | |
| openstackgerrit | Anna Shen proposed a change to openstack/trove-integration: Add support for a neutron-based install https://review.openstack.org/78123 | 00:04 |
| dougshelley66 | sorry all just catching up. | 00:05 |
| amrith | SlickNik, meet vipul. vipul meet SlickNik. | 00:05 |
| amrith | oh, I thought you were meeting with rob ford today? | 00:05 |
| esmute | lol | 00:05 |
| dougshelley66 | well i did that earlier | 00:05 |
| dougshelley66 | but he had to go out for some drinks | 00:05 |
| dougshelley66 | so i came home | 00:05 |
| amrith | he's back at work, very sad and all ... | 00:05 |
| esmute | SlickNik: +1. I dont know the need to do a migration script to create capabilities out of conf file | 00:05 |
| dougshelley66 | so after the discussions today, i think there are other gaps in the capabilities as spec'd | 00:06 |
| amrith | so SlickNik re: #7 that was an integral part of what vipul suggested, I though. | 00:06 |
| amrith | thought. | 00:06 |
| dougshelley66 | it seems like it makes sense to revist the spec and flesh out some more of these details. We can work on that with whomever would like to help out | 00:06 |
| vipul | amrith, SlickNik: Yea, i was in favor of seeding the database, but i wasn't thinking of them as purely overrides | 00:07 |
| esmute | unless we are planning to rid the conf file, i dont see the need to have default capabilities in DB. | 00:07 |
| *** amcrn has quit IRC | 00:07 | |
| vipul | if we go with iccha's proposal, then the API when someone installs trove would return at least something | 00:07 |
| vipul | (i was under the impression we would only return items in the DB, and have a visibility flag in the db) | 00:08 |
| *** saurabhs has quit IRC | 00:08 | |
| dougshelley66 | i would think the capabilities that are implemented should be put in the database via migration? | 00:08 |
| vipul | so instaed of seeding the DB, i think we woudl want to ensure that a value is set for capabiltities_list=volume_support,users, | 00:09 |
| iccha | i think we have some sort of general consensus . we have made progress on bringing up some of the gaps in capabilities. dougshelley66 since he has volunteered can maybe wikify a proposal we could look at next meeting? | 00:09 |
| *** saurabhs has joined #openstack-trove | 00:09 | |
| iccha | meanwhile if other features want to get started, they could start by adding config values since that does seem to fit into the current plans? | 00:10 |
| amrith | so, out of curiousity, is the capabilities feature even relevant? | 00:11 |
| amrith | any more, that is? | 00:11 |
| amrith | also, dougshelley66's question is still open ... | 00:12 |
| vipul | what i want is an API that can tell me how Trove is configured, so I can do special things based on what comes back. | 00:12 |
| *** yidclare has joined #openstack-trove | 00:12 | |
| vipul | I don't care if we call that capabilities, or read-only conf, or whatever | 00:12 |
| dougshelley66 | i have some thoughts on improvements to the previously laid out feature that could address what vipul is talking about | 00:13 |
| abramley | Vipul +1 | 00:13 |
| esmute | so that goes back to having a simplified version of capabilities (readonly). Which i think, based on the level of discussion and disagreement, we should do | 00:13 |
| esmute | Happy Canada Day dougshelley66, btw! | 00:13 |
| iccha | i dont mean to cut off this discussion, but i did have another question i wanted to bring up wrt volume support and the api responses we send | 00:14 |
| dougshelley66 | i think capabilities needs to support read-only (i.e. product implemented stuff) and read-write (i.e. operator modifiable) thngs | 00:14 |
| dougshelley66 | it doesn't now | 00:14 |
| amrith | iccha, before you do ... | 00:14 |
| dougshelley66 | i also think that we need to work in "datastore manager" capabilities vs datastore-version capabilities | 00:14 |
| amrith | do you have what you need in a decision | 00:14 |
| SlickNik | Okay so what's the main scenario to be able to override (set - read/write) CONF values on the fly like this? | 00:14 |
| dougshelley66 | i think the former are read only and the latter could be read-write | 00:15 |
| dougshelley66 | anyway, i think i would like to think a bit on the feedback and take a shot at writing it up | 00:15 |
| iccha | amrith: i think irrespective of how we decide to populate the capabilities db, looks like we re going to need config values | 00:15 |
| iccha | thanks dougshelley66 :) i think rueben would be interested in helping out as well | 00:16 |
| dougshelley66 | iccha ok sounds good - schang will also be involved | 00:16 |
| SlickNik | as far as I can tell, the requirements behind the initial capabilities BP was to only expose config values programmatically via an API (read-only). | 00:16 |
| dougshelley66 | well the thing that doesn't make sense is that the overrides are by datastore version | 00:17 |
| dougshelley66 | which could only be relevant at the operator level | 00:17 |
| amrith | SlickNik, dougshelley66, vipul, iccha, esmute ... thanks folks. good chatting. now have to go see about dinner. will be in and out. | 00:17 |
| *** annashen_ has quit IRC | 00:17 | |
| SlickNik | amrith: thanks! Bon apetit. | 00:17 |
| iccha | thanks amrith have a good dinner | 00:18 |
| esmute | amrith: Buen provecho | 00:18 |
| amrith | thx iccha | 00:18 |
| iccha | SlickNik: i wanted to bring up about api response as well. and right now volume in the repsonse is config driver | 00:18 |
| amrith | esmute, i no habla espanol | 00:18 |
| *** mattgriffin has joined #openstack-trove | 00:18 | |
| esmute | amrith: But you must speak French then | 00:18 |
| amrith | esmute, I no habla fracais ;) | 00:20 |
| SlickNik | iccha: Not sure I fully grokked what you meant by that. Can you elucidate? | 00:20 |
| iccha | sure SlickNik https://github.com/openstack/trove/blob/master/trove/common/apischema.py#L192 | 00:21 |
| iccha | and https://github.com/openstack/trove/blob/master/trove/flavor/views.py#L38 | 00:21 |
| iccha | and since we are moving to datastore specific configs and in the above two cases are indepedent of datastore stores | 00:22 |
| iccha | i think we should modify response to always show both volume and local_storage | 00:22 |
| *** yidclare has quit IRC | 00:25 | |
| SlickNik | iccha: That's a tricky issue. That's one possible solution, I think. Might need to mull that over for a bit. | 00:26 |
| iccha | okays thanks SlickNik for playing moderator today :) | 00:27 |
| SlickNik | iccha: Not a problem. Thank you! | 00:28 |
| *** saurabhs has left #openstack-trove | 00:38 | |
| *** ViswaV_ has joined #openstack-trove | 00:55 | |
| *** ViswaV has quit IRC | 00:58 | |
| *** nehav has joined #openstack-trove | 01:00 | |
| *** eghobo has quit IRC | 01:13 | |
| *** nosnos has joined #openstack-trove | 01:13 | |
| *** nosnos has quit IRC | 01:20 | |
| *** robertmyers has joined #openstack-trove | 01:30 | |
| *** mattgriffin has quit IRC | 01:42 | |
| *** annashen_ has joined #openstack-trove | 01:56 | |
| *** nehav has quit IRC | 02:27 | |
| *** amytron has joined #openstack-trove | 02:31 | |
| *** ramishra has joined #openstack-trove | 02:37 | |
| *** ViswaV_ has quit IRC | 02:42 | |
| *** ViswaV has joined #openstack-trove | 02:43 | |
| *** michael-yu has joined #openstack-trove | 02:51 | |
| *** yidclare has joined #openstack-trove | 02:52 | |
| *** mattgriffin has joined #openstack-trove | 02:56 | |
| *** michael-yu has quit IRC | 02:58 | |
| *** eghobo has joined #openstack-trove | 03:00 | |
| *** michael-yu has joined #openstack-trove | 03:01 | |
| *** michael-yu has quit IRC | 03:03 | |
| *** yidclare has quit IRC | 03:07 | |
| *** ramishra has quit IRC | 03:33 | |
| *** ramishra has joined #openstack-trove | 03:40 | |
| *** annashen_ has quit IRC | 03:47 | |
| *** annashen_ has joined #openstack-trove | 03:48 | |
| *** nehav has joined #openstack-trove | 03:48 | |
| *** nosnos has joined #openstack-trove | 03:52 | |
| *** ramishra has quit IRC | 03:52 | |
| *** ramishra has joined #openstack-trove | 03:53 | |
| *** annashen_ has quit IRC | 03:53 | |
| *** ViswaV_ has joined #openstack-trove | 03:56 | |
| *** ViswaV has quit IRC | 04:00 | |
| *** ViswaV_ has quit IRC | 04:00 | |
| *** ViswaV has joined #openstack-trove | 04:02 | |
| *** annashen_ has joined #openstack-trove | 04:02 | |
| *** eghobo has quit IRC | 04:05 | |
| *** nehav has quit IRC | 04:09 | |
| *** sgotliv has joined #openstack-trove | 04:09 | |
| *** ajc_ has joined #openstack-trove | 04:17 | |
| *** robertmyers has quit IRC | 04:18 | |
| *** sgotliv has quit IRC | 04:43 | |
| *** ramishra_ has joined #openstack-trove | 04:50 | |
| *** ramishr__ has joined #openstack-trove | 04:51 | |
| *** ramishra has quit IRC | 04:53 | |
| *** ViswaV has quit IRC | 04:54 | |
| *** SushillKM has joined #openstack-trove | 04:54 | |
| *** ramishra_ has quit IRC | 04:55 | |
| *** Longgeek has joined #openstack-trove | 04:55 | |
| *** ViswaV has joined #openstack-trove | 04:56 | |
| *** SushillKM has quit IRC | 05:06 | |
| *** ViswaV_ has joined #openstack-trove | 05:07 | |
| *** ramishr__ has quit IRC | 05:08 | |
| *** ramishra has joined #openstack-trove | 05:08 | |
| *** ViswaV has quit IRC | 05:11 | |
| *** ViswaV has joined #openstack-trove | 05:17 | |
| *** ViswaV_ has quit IRC | 05:19 | |
| *** achampion has joined #openstack-trove | 05:26 | |
| *** sgotliv has joined #openstack-trove | 05:34 | |
| *** juantwo has quit IRC | 05:44 | |
| *** Longgeek has quit IRC | 05:45 | |
| *** Longgeek has joined #openstack-trove | 05:45 | |
| *** Longgeek has quit IRC | 05:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex https://review.openstack.org/104074 | 06:06 |
| *** coolsvap|afk is now known as coolsvap | 06:09 | |
| *** sgotliv has quit IRC | 06:18 | |
| *** Longgeek has joined #openstack-trove | 06:27 | |
| *** michael-yu has joined #openstack-trove | 06:32 | |
| *** mattgriffin has quit IRC | 06:42 | |
| *** denis_makogon has joined #openstack-trove | 06:53 | |
| *** amytron has quit IRC | 06:59 | |
| openstackgerrit | lawrancejing proposed a change to openstack/trove: Add CONTRIBUTING.rst https://review.openstack.org/103787 | 07:00 |
| *** julienvey has joined #openstack-trove | 07:10 | |
| *** flaper87|afk is now known as flaper87 | 07:24 | |
| *** michael-yu has quit IRC | 07:28 | |
| *** karimb has joined #openstack-trove | 07:28 | |
| *** annashen_ has quit IRC | 07:38 | |
| *** annashen_ has joined #openstack-trove | 07:38 | |
| *** annashen_ has quit IRC | 07:43 | |
| *** boden has joined #openstack-trove | 07:58 | |
| *** boden has quit IRC | 08:04 | |
| *** nosnos has quit IRC | 08:05 | |
| *** nosnos has joined #openstack-trove | 08:05 | |
| *** denis_makogon has quit IRC | 08:37 | |
| openstackgerrit | A change was merged to openstack/trove: Imported Translations from Transifex https://review.openstack.org/104074 | 08:37 |
| *** Longgeek has quit IRC | 09:04 | |
| *** dmakogon_ is now known as denis_makogon | 09:38 | |
| denis_makogon | SlickNik, ping , need to talk to you about ceilometer BP and your -2 on stats patch | 09:38 |
| denis_makogon | SlickNik, also, your suggestion at https://review.openstack.org/#/c/96795/ generates PEP8 issues, because import line is too long (83 symb.) and suggestion to use "\" is more that weired, because it's 100% not pythonic way | 09:40 |
| denis_makogon | SlickNik, so, please re-review patchj | 09:40 |
| openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Fix signature for launching int-tests https://review.openstack.org/103201 | 09:42 |
| *** ramishra has quit IRC | 09:47 | |
| *** ramishra has joined #openstack-trove | 09:55 | |
| openstackgerrit | Denis M. proposed a change to openstack/python-troveclient: Add instance/backups stats API https://review.openstack.org/103853 | 09:59 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 10:00 |
| amrith | denis_makogon, you there? | 10:35 |
| amrith | denis_makogon, when you get back please see https://review.openstack.org/#/c/96795 | 10:44 |
| *** boden has joined #openstack-trove | 10:45 | |
| amrith | I'm almost positive (but I won't claim to speak for him) that what SlickNik is objecting to is that H301 does not like your importing more than one module in a single import statement. He is not objecting to the use of () to wrap the line, just that you changed the wrapped line to import three things. He's asking that you make this three wrapped lines. | 10:45 |
| *** Longgeek has joined #openstack-trove | 10:52 | |
| *** ramishra has quit IRC | 10:57 | |
| *** ramishra has joined #openstack-trove | 10:57 | |
| denis_makogon | amrith, it's hard to follow rules (hacking) that are not being tested, so, there's another issue, it's not enough PEP8 rules checks in Trove | 11:01 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Fix tracking of SG's provisioned by Heat https://review.openstack.org/96795 | 11:02 |
| denis_makogon | amrith, we don't have to be mediums to know what is being affected or not | 11:02 |
| amrith | denis_makogon, I don't understand either of your comments. please elaborate. I think what SlickNik is proposing is very concrete. he is objecting to the violation of H301 which very clearly states that you shouldn't import more than 1 module per line. you imported 3. | 11:05 |
| denis_makogon | amrith, i get that, but i don't see where hacking check in being performed, so, it's pretty interesting to see comments that are saying that given code affects something that never is being checked | 11:06 |
| amrith | denis_makogon, take a look at https://review.openstack.org/#/c/84258/ | 11:08 |
| amrith | maybe I missed some email on the thread that says a similar change will be rolled out more widely | 11:08 |
| amrith | also https://review.openstack.org/#/c/67147/ | 11:09 |
| denis_makogon | amrith https://github.com/openstack/trove/blob/master/tox.ini#L44 | 11:10 |
| amrith | denis_makogon, I get it. your code passed jenkins and reddwarf. so I realize that H301 wasn't being enforced. | 11:11 |
| denis_makogon | amrith, as i said, i don't have to be medium to know that something is failing, but "something" is neven been examined | 11:11 |
| denis_makogon | *never | 11:11 |
| denis_makogon | amrith, until H301 is ignored - comments about imports are nothing else than "wishlist" things | 11:12 |
| denis_makogon | that's all | 11:12 |
| denis_makogon | not even reason of -1, since check for certain rule is ignored | 11:13 |
| amrith | sorry denis_makogon I thought I'd help. my mistake. | 11:18 |
| denis_makogon | amrith, np, everything is ok, thanks =) | 11:20 |
| *** Longgeek has quit IRC | 11:22 | |
| *** Longgeek has joined #openstack-trove | 11:23 | |
| *** radez_g0n3 is now known as radez | 11:43 | |
| *** Longgeek_ has joined #openstack-trove | 11:43 | |
| *** Longgeek has quit IRC | 11:47 | |
| *** nosnos has quit IRC | 11:55 | |
| *** nosnos_ has joined #openstack-trove | 11:58 | |
| *** juantwo has joined #openstack-trove | 12:02 | |
| *** ramishra has quit IRC | 12:03 | |
| *** ramishra has joined #openstack-trove | 12:04 | |
| *** Longgeek_ has quit IRC | 12:04 | |
| *** Longgeek has joined #openstack-trove | 12:05 | |
| *** Longgeek_ has joined #openstack-trove | 12:06 | |
| *** ramishra has quit IRC | 12:08 | |
| *** Longgeek has quit IRC | 12:09 | |
| *** juantwo has quit IRC | 12:10 | |
| *** ramishra has joined #openstack-trove | 12:15 | |
| *** juantwo has joined #openstack-trove | 12:20 | |
| *** ajc_ has quit IRC | 12:30 | |
| *** coolsvap is now known as coolsvap|afk | 12:31 | |
| *** fifieldt has quit IRC | 12:35 | |
| *** zigo has quit IRC | 12:37 | |
| *** zigo has joined #openstack-trove | 12:37 | |
| *** zigo has quit IRC | 12:41 | |
| *** zigo has joined #openstack-trove | 12:42 | |
| *** zigo has quit IRC | 12:45 | |
| *** Longgeek_ has quit IRC | 12:51 | |
| *** demorris has joined #openstack-trove | 12:51 | |
| *** Longgeek has joined #openstack-trove | 12:52 | |
| *** miqui has joined #openstack-trove | 12:56 | |
| *** ramishra has quit IRC | 12:59 | |
| *** achampio1 has joined #openstack-trove | 13:01 | |
| *** achampion has quit IRC | 13:03 | |
| *** demorris_ has joined #openstack-trove | 13:05 | |
| *** demorris has quit IRC | 13:08 | |
| *** demorris_ is now known as demorris | 13:08 | |
| *** tvoran has joined #openstack-trove | 13:10 | |
| *** Longgeek_ has joined #openstack-trove | 13:12 | |
| *** Longgeek has quit IRC | 13:15 | |
| *** Longgeek has joined #openstack-trove | 13:21 | |
| *** Longgeek_ has quit IRC | 13:24 | |
| *** grapex has joined #openstack-trove | 13:30 | |
| *** grapex_ has joined #openstack-trove | 13:32 | |
| *** achampion has joined #openstack-trove | 13:36 | |
| *** grapex has quit IRC | 13:36 | |
| *** nosnos_ has quit IRC | 13:37 | |
| *** achampio1 has quit IRC | 13:38 | |
| *** amytron has joined #openstack-trove | 13:42 | |
| *** nehav has joined #openstack-trove | 13:46 | |
| *** nehav has quit IRC | 13:46 | |
| *** nehav has joined #openstack-trove | 13:46 | |
| *** grapex_ has quit IRC | 13:47 | |
| *** grapex has joined #openstack-trove | 13:47 | |
| *** jcru has joined #openstack-trove | 13:48 | |
| *** nosnos has joined #openstack-trove | 13:49 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 13:50 |
| *** annashen_ has joined #openstack-trove | 13:54 | |
| *** zigo has joined #openstack-trove | 13:58 | |
| *** nosnos has quit IRC | 13:59 | |
| *** dkehnx has quit IRC | 14:02 | |
| *** rhodgin has joined #openstack-trove | 14:03 | |
| *** amytron has quit IRC | 14:06 | |
| *** amytron has joined #openstack-trove | 14:07 | |
| *** yidclare has joined #openstack-trove | 14:08 | |
| *** yidclare has quit IRC | 14:09 | |
| *** nehav3 has joined #openstack-trove | 14:10 | |
| *** nehav has quit IRC | 14:11 | |
| *** ramishra has joined #openstack-trove | 14:14 | |
| *** sgotliv has joined #openstack-trove | 14:15 | |
| *** ramishra has quit IRC | 14:18 | |
| *** mattgriffin has joined #openstack-trove | 14:22 | |
| *** sgotliv has quit IRC | 14:23 | |
| *** dkehnx has joined #openstack-trove | 14:23 | |
| *** ramishra has joined #openstack-trove | 14:24 | |
| *** jcru has quit IRC | 14:25 | |
| *** jcru has joined #openstack-trove | 14:26 | |
| *** robertmyers has joined #openstack-trove | 14:26 | |
| *** kevinconway has joined #openstack-trove | 14:31 | |
| *** dkehn_ has joined #openstack-trove | 14:39 | |
| *** annashen_ has quit IRC | 14:39 | |
| *** annashen_ has joined #openstack-trove | 14:40 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:41 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Update backup state when guest error occures https://review.openstack.org/102262 | 14:41 |
| *** dkehn_ has quit IRC | 14:42 | |
| *** dkehn_ has joined #openstack-trove | 14:42 | |
| *** ramishra has quit IRC | 14:43 | |
| *** dkehnx has quit IRC | 14:43 | |
| *** dkehn_ is now known as dkehnx | 14:43 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:43 |
| *** annashen_ has quit IRC | 14:44 | |
| *** ramishra has joined #openstack-trove | 14:53 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:54 |
| *** coolsvap|afk is now known as coolsvap | 14:55 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:56 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:56 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 14:57 |
| *** thedodd has joined #openstack-trove | 15:00 | |
| *** kevinconway has quit IRC | 15:02 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 15:04 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Update backup state when guest error occures https://review.openstack.org/102262 | 15:04 |
| *** doddstack has joined #openstack-trove | 15:06 | |
| *** ramishra has quit IRC | 15:08 | |
| *** thedodd has quit IRC | 15:08 | |
| *** nehav3 has quit IRC | 15:15 | |
| *** nehav has joined #openstack-trove | 15:16 | |
| *** ViswaV has quit IRC | 15:17 | |
| *** ViswaV_ has joined #openstack-trove | 15:17 | |
| *** nehav has quit IRC | 15:17 | |
| *** nehav2 has joined #openstack-trove | 15:18 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.backup.* actions https://review.openstack.org/102206 | 15:18 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.instance.* actions https://review.openstack.org/101189 | 15:18 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.instance.resize.flavor.* actions https://review.openstack.org/104582 | 15:18 |
| *** nehav3 has joined #openstack-trove | 15:19 | |
| *** nehav2 has quit IRC | 15:19 | |
| *** radez is now known as radez_g0n3 | 15:20 | |
| *** nehav3 has quit IRC | 15:24 | |
| *** ViswaV_ has quit IRC | 15:40 | |
| *** ViswaV has joined #openstack-trove | 15:41 | |
| *** ViswaV_ has joined #openstack-trove | 15:42 | |
| *** ViswaV has quit IRC | 15:46 | |
| *** nehav has joined #openstack-trove | 15:48 | |
| *** rhodgin has quit IRC | 15:53 | |
| *** eghobo has joined #openstack-trove | 15:54 | |
| *** eghobo has quit IRC | 15:59 | |
| openstackgerrit | A change was merged to openstack/trove: Remove redundant heat templates from codebase https://review.openstack.org/99000 | 16:04 |
| *** eghobo has joined #openstack-trove | 16:04 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.instance.resize.flavor.* actions https://review.openstack.org/104582 | 16:04 |
| *** ramishra has joined #openstack-trove | 16:09 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.instance.resize.*.* actions https://review.openstack.org/104582 | 16:09 |
| *** flaper87 is now known as flaper87|afk | 16:12 | |
| *** ramishra has quit IRC | 16:13 | |
| *** michael-yu has joined #openstack-trove | 16:13 | |
| *** eguz has joined #openstack-trove | 16:15 | |
| anteaya | SlickNik: let me know your latest status and if there is anything i can do to help | 16:15 |
| *** eguz has quit IRC | 16:17 | |
| *** eghobo has quit IRC | 16:19 | |
| *** michael-yu has quit IRC | 16:22 | |
| *** karimb has quit IRC | 16:25 | |
| openstackgerrit | Greg Lucas proposed a change to openstack/trove-integration: Re-mount swift disk image in start-deps https://review.openstack.org/102642 | 16:30 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Enable H301 hacking rule https://review.openstack.org/104616 | 16:41 |
| *** Longgeek has quit IRC | 16:41 | |
| SlickNik | denis_makogon: It's fine to wrap the line using parens if it's longer than 80 chars; what the comment was mentioning was that we should try and not have multiple modules imported in the same line. | 16:49 |
| SlickNik | denis_makogon: Looks like you've fixed that in the new patchset that you resubmitted. | 16:50 |
| denis_makogon | SlickNik, i get and fixed that, but H301 is disabled for tox checks | 16:50 |
| denis_makogon | SlickNik, so, i submitted new patch that allows H301 to be examined over given code | 16:50 |
| SlickNik | denis_makogon: Okay. Sounds good. | 16:51 |
| denis_makogon | SlickNik, but there's a problem, when H301 gets enabled code is being examined with H302 too | 16:52 |
| denis_makogon | H302 - import only module | 16:52 |
| denis_makogon | so i added H302 to ignore list | 16:52 |
| SlickNik | I'd prefer to fix issues with H302 rather than add it to the import list. | 16:53 |
| denis_makogon | SlickNik, another question, about https://review.openstack.org/#/c/103239/, there's no changes for taskamanager service | 16:53 |
| SlickNik | But these are style issues, so I'd prioritize fixing other bugs over them. | 16:53 |
| denis_makogon | SlickNik, that's why i did only H301, because H302 will take too much efforts | 16:54 |
| denis_makogon | SlickNik, i do agree that H302 should be fixed, but not for now, i suppose | 16:55 |
| SlickNik | denis_makogon: If you don't have the bandwidth that's okay. Someone else might pick it up if they have the time / inclination. | 16:55 |
| SlickNik | denis_makogon: That's the good part about open source. :) | 16:56 |
| denis_makogon | SlickNik, yes, i up for giving this task to someone else | 16:56 |
| denis_makogon | *i'm | 16:56 |
| denis_makogon | SlickNik, can we talk about https://review.openstack.org/#/c/103853/? | 16:57 |
| SlickNik | Sure thing. | 16:57 |
| denis_makogon | SlickNik, i've read initial BP and convo | 16:58 |
| denis_makogon | and was made a decision to add new admin API calls | 16:59 |
| SlickNik | Decision by whom, can I ask? | 16:59 |
| denis_makogon | community, it was a regular weekly meeting | 16:59 |
| SlickNik | Can you point me to the notes? | 16:59 |
| SlickNik | And why is this information not in the BP? | 16:59 |
| denis_makogon | sure, let me find a link | 16:59 |
| denis_makogon | about what ? | 16:59 |
| denis_makogon | SlickNik, all notification types and mgmt API calls are there | 17:00 |
| SlickNik | About the stats API? I don't see any API changes for that bp | 17:00 |
| SlickNik | https://blueprints.launchpad.net/trove/+spec/ceilometer-integration | 17:00 |
| SlickNik | There aren't any details in that BP. | 17:01 |
| denis_makogon | SlickNik, New mgmt API routes for pollster should be added: | 17:02 |
| denis_makogon | SlickNik, right after notifications | 17:02 |
| SlickNik | The BP doesn't say anything about API routes for the pollster. | 17:04 |
| SlickNik | I'd suggest spending some time working on the design details and the BP first — and fleshing those out before pushing up code. | 17:04 |
| SlickNik | Since it came out of dormancy, I'd suggest using the BP template format to iron those out as well. | 17:05 |
| denis_makogon | ok | 17:06 |
| denis_makogon | SlickNik, last thing, https://review.openstack.org/#/c/97510/ | 17:07 |
| SlickNik | What about it? | 17:09 |
| *** ramishra has joined #openstack-trove | 17:10 | |
| *** michael-yu has joined #openstack-trove | 17:11 | |
| denis_makogon | SlickNik, everything was fixed according all comments, could this be approved? | 17:12 |
| *** ramishra has quit IRC | 17:14 | |
| SlickNik | denis_makogon: If everything has been fixed, I'm sure it'll get approved when folks have a chance to look at it. | 17:14 |
| SlickNik | denis_makogon: The latest patchset was only uploaded this morning. | 17:15 |
| denis_makogon | SlickNik, no, gerrit says Jun.27 | 17:15 |
| SlickNik | oh, sorry I was looking at the wrong change. My bad. | 17:16 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.instance.resize.*.* actions https://review.openstack.org/104582 | 17:16 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.backup.* actions https://review.openstack.org/102206 | 17:16 |
| denis_makogon | SlickNik, np | 17:16 |
| denis_makogon | i'm out, gonna be available at 4-6 PM | 17:16 |
| *** denis_makogon has quit IRC | 17:16 | |
| SlickNik | Sure thing. Will make sure to take a look at it when doing reviews. | 17:16 |
| johnma | I just wanted to bring up the issue I had yesterday with installing a different datastore using redstart kick-start command. So yesterday running this command didnt install a guest VM with mysql. I saw a note that mentioned SlickNik had a change go in for trove-integration last week and my setup was couple of weeks old so I got the latest one today and tried it again. But now it complains... | 17:18 |
| johnma | ...about cloud-init-datasources not found in certain directories. Does anyone know where I can find this or what this error means | 17:18 |
| johnma | Thank you | 17:18 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Enable H301 hacking rule https://review.openstack.org/104616 | 17:18 |
| SlickNik | johnma: You probably need to update the tripleo-image-elements repo as well. | 17:20 |
| SlickNik | should be at /opt/stack/tripleo-image-elements | 17:20 |
| johnma | ok. let me try that. Thanks SlickNik | 17:21 |
| *** tvoran has quit IRC | 17:28 | |
| *** radez_g0n3 is now known as radez | 17:33 | |
| *** demorris has quit IRC | 17:34 | |
| *** michael-yu has quit IRC | 17:45 | |
| *** dkehn_ has joined #openstack-trove | 17:52 | |
| *** dkehnx has quit IRC | 17:56 | |
| *** demorris has joined #openstack-trove | 18:00 | |
| *** amcrn has joined #openstack-trove | 18:05 | |
| *** coolsvap is now known as coolsvap|afk | 18:08 | |
| johnma | SlickNik: I got past that error. I actually had to update the diskimage-builder repo for it to work. But the kickstart still fails to get mysql. I can see an error message which says: 'chroot: failed to run command 'sh': Exec format error" | 18:14 |
| *** julienvey has quit IRC | 18:16 | |
| *** ViswaV_ has quit IRC | 18:16 | |
| *** ViswaV has joined #openstack-trove | 18:16 | |
| *** michael-yu has joined #openstack-trove | 18:18 | |
| *** doddstack has quit IRC | 18:19 | |
| johnma | I have pasted a snippet of the output at paste.openstack.org/show/85445. Thanks | 18:19 |
| *** tvoran has joined #openstack-trove | 18:28 | |
| *** tvoran has quit IRC | 18:33 | |
| *** yidclare has joined #openstack-trove | 18:34 | |
| *** ramishra has joined #openstack-trove | 18:39 | |
| *** eghobo has joined #openstack-trove | 18:39 | |
| *** ramishra has quit IRC | 18:44 | |
| *** yidclare has quit IRC | 18:48 | |
| openstackgerrit | A change was merged to openstack/trove: Updated from global requirements https://review.openstack.org/104023 | 19:01 |
| *** tvoran has joined #openstack-trove | 19:03 | |
| *** doddstack has joined #openstack-trove | 19:08 | |
| anteaya | SlickNik: any news? | 19:18 |
| *** amytron has left #openstack-trove | 19:27 | |
| *** julienvey has joined #openstack-trove | 19:37 | |
| *** nehav has quit IRC | 19:40 | |
| *** ramishra has joined #openstack-trove | 19:40 | |
| *** ramishra has quit IRC | 19:44 | |
| *** yidclare has joined #openstack-trove | 20:03 | |
| *** dkehn__ has joined #openstack-trove | 20:04 | |
| *** michael-yu has quit IRC | 20:04 | |
| *** dkehn_ has quit IRC | 20:07 | |
| *** dkehn__ is now known as dkehnx | 20:19 | |
| *** boden has quit IRC | 20:30 | |
| *** georgelorch has quit IRC | 20:33 | |
| *** denis_makogon has joined #openstack-trove | 20:33 | |
| *** julienvey has quit IRC | 20:40 | |
| *** mattgriffin has quit IRC | 20:41 | |
| *** ramishra has joined #openstack-trove | 20:41 | |
| *** mattgriffin has joined #openstack-trove | 20:45 | |
| *** mattgriffin has quit IRC | 20:46 | |
| *** ramishra has quit IRC | 20:46 | |
| *** juantwo has quit IRC | 20:58 | |
| *** annashen_ has joined #openstack-trove | 20:59 | |
| *** annashen_ has quit IRC | 21:00 | |
| *** annashen_ has joined #openstack-trove | 21:00 | |
| *** annashen_ has quit IRC | 21:04 | |
| *** michael-yu has joined #openstack-trove | 21:07 | |
| *** radez is now known as radez_g0n3 | 21:16 | |
| *** vgnbkr has quit IRC | 21:24 | |
| *** vgnbkr has joined #openstack-trove | 21:25 | |
| *** robertmyers has quit IRC | 21:25 | |
| *** doddstack has quit IRC | 21:44 | |
| *** michael-yu has quit IRC | 21:51 | |
| *** esp_ has joined #openstack-trove | 21:58 | |
| *** michael-yu has joined #openstack-trove | 21:59 | |
| *** michael-yu has quit IRC | 22:00 | |
| *** denis_makogon has quit IRC | 22:01 | |
| *** yidclare has quit IRC | 22:03 | |
| *** michael-yu has joined #openstack-trove | 22:07 | |
| *** esp_ has quit IRC | 22:13 | |
| *** mattgriffin has joined #openstack-trove | 22:14 | |
| *** michael-yu has quit IRC | 22:28 | |
| *** mattgriffin has quit IRC | 22:29 | |
| *** eghobo has quit IRC | 22:29 | |
| *** georgelorch has joined #openstack-trove | 22:29 | |
| *** jcru has quit IRC | 22:29 | |
| *** grapex has quit IRC | 22:33 | |
| *** tvoran has quit IRC | 22:34 | |
| *** michael-yu has joined #openstack-trove | 22:41 | |
| *** juantwo has joined #openstack-trove | 22:50 | |
| *** juantwo has quit IRC | 22:51 | |
| *** juantwo has joined #openstack-trove | 22:51 | |
| *** demorris has quit IRC | 22:57 | |
| *** michael-yu has quit IRC | 23:19 | |
| *** michael-yu has joined #openstack-trove | 23:19 | |
| *** ramishra has joined #openstack-trove | 23:43 | |
| *** annashen_ has joined #openstack-trove | 23:47 | |
| *** ramishra has quit IRC | 23:48 | |
| *** eghobo has joined #openstack-trove | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!