| *** dannywilson has quit IRC | 00:01 | |
| *** akerr has joined #openstack-cinder | 00:07 | |
| *** akerr has quit IRC | 00:14 | |
| *** jwang_ has joined #openstack-cinder | 00:16 | |
| openstackgerrit | Michael Price proposed openstack/cinder: Fix netapp_enable_multiattach default for E-Series https://review.openstack.org/223698 | 00:16 |
|---|---|---|
| *** tsekiyama has quit IRC | 00:18 | |
| *** cbader_ has joined #openstack-cinder | 00:18 | |
| *** devlaps has quit IRC | 00:18 | |
| *** smoriya has joined #openstack-cinder | 00:20 | |
| *** dims_ has quit IRC | 00:20 | |
| *** bapalm_ has joined #openstack-cinder | 00:20 | |
| *** cburgess_ has joined #openstack-cinder | 00:21 | |
| *** jwang has quit IRC | 00:21 | |
| *** cburgess has quit IRC | 00:21 | |
| *** cbader has quit IRC | 00:21 | |
| *** bapalm has quit IRC | 00:21 | |
| *** mtanino has quit IRC | 00:22 | |
| hodos | hi did any1 have a problem with test_volume_boot_pattern with devstack | 00:23 |
| *** p0rtal has joined #openstack-cinder | 00:24 | |
| *** esker has quit IRC | 00:24 | |
| *** esker has joined #openstack-cinder | 00:25 | |
| *** scottda_ has joined #openstack-cinder | 00:30 | |
| winston-d | harlowja: ping | 00:31 |
| *** mtaylor has joined #openstack-cinder | 00:31 | |
| harlowja | winston-d pong | 00:32 |
| thingee | leeantho: you think we'll have a patch for this tomorrow? https://bugs.launchpad.net/cinder/+bug/1486286 | 00:33 |
| openstack | Launchpad bug 1486286 in Cinder "'qos_specs' does not get populated by Cinder during scheduler volume placement" [Undecided,In progress] - Assigned to Anthony Lee (anthony-mic-lee) | 00:33 |
| thingee | leeantho: running out of time in this release | 00:33 |
| thingee | hemnafk: ^ | 00:33 |
| winston-d | harlowja: have a naive question about taskflow, http://docs.openstack.org/developer/taskflow/inputs_and_outputs.html said flow inputs are task arguments and outputs are task results. | 00:34 |
| *** patrickeast_ has joined #openstack-cinder | 00:34 | |
| *** rex_lee_ has joined #openstack-cinder | 00:34 | |
| winston-d | harlowja: if flow A is added before flow B, does that mean flow B's input is flow A's output? | 00:34 |
| leeantho | thingee, there is a patch for it here: https://review.openstack.org/#/c/214826/ | 00:35 |
| *** sweston_ has joined #openstack-cinder | 00:35 | |
| winston-d | leeantho: got ur message about this change, will review it today. | 00:35 |
| *** jdurgin has quit IRC | 00:35 | |
| *** eduardo has quit IRC | 00:35 | |
| *** jaypipes has quit IRC | 00:35 | |
| *** lennyb has quit IRC | 00:35 | |
| *** theanalyst has quit IRC | 00:35 | |
| *** mordred has quit IRC | 00:36 | |
| *** sweston has quit IRC | 00:36 | |
| *** rex_lee has quit IRC | 00:36 | |
| *** patrickeast has quit IRC | 00:36 | |
| *** scottda has quit IRC | 00:36 | |
| thingee | leeantho: thanks | 00:36 |
| *** diogogmt has quit IRC | 00:36 | |
| leeantho | winston-d, okay, thanks | 00:36 |
| *** scottda_ is now known as scottda | 00:36 | |
| harlowja | winston-d right, the tasks in flow-A will be used (if they provide outputs) by the tasks in flow-B (if any in flow-B require things that things from flow-A can provide) | 00:36 |
| thingee | leeantho: thanks | 00:36 |
| *** sweston_ is now known as sweston | 00:37 | |
| harlowja | winston-d a graph flow though can alter this to be something different, in that u can affect the ordering to be different by associating links yourself | 00:37 |
| *** patrickeast_ is now known as patrickeast | 00:38 | |
| winston-d | harlowja: then if flow A can be either put before & after flow B, how should tasks in flow A/B get input and generate output? | 00:38 |
| harlowja | woah, no doing that, lol | 00:39 |
| harlowja | flow B should fail to validate | 00:39 |
| harlowja | the combination of those should fail when things in flow B can't find needed inputs | 00:39 |
| *** jaypipes has joined #openstack-cinder | 00:39 | |
| harlowja | * http://docs.openstack.org/developer/taskflow/engines.html#validation | 00:40 |
| harlowja | that engine stage validates needed input/output 'finding' | 00:40 |
| winston-d | harlowja: then affect ordering means A->B->C or A->C, but C never comes before B and A and B never comes before A? | 00:41 |
| *** eduardo has joined #openstack-cinder | 00:41 | |
| harlowja | A and B will always be before C in that case | 00:42 |
| *** lennyb has joined #openstack-cinder | 00:42 | |
| harlowja | and A and B may provide outputs to C (and outputs of A may be used in B) | 00:42 |
| *** theanalyst has joined #openstack-cinder | 00:43 | |
| *** _cjones_ has quit IRC | 00:43 | |
| winston-d | so when creating flow/tasks, ordering is part of the task logic. | 00:44 |
| harlowja | yup | 00:45 |
| harlowja | u can 'view' this by http://paste.openstack.org/show/463773/ | 00:45 |
| harlowja | u can even make diagrams from similar pformat() methods... | 00:46 |
| winston-d | harlowja: nice! thx | 00:46 |
| harlowja | np | 00:47 |
| harlowja | >>> print(e.compilation.execution_graph.export_to_dot()) | 00:47 |
| harlowja | strict digraph "work" { | 00:47 |
| harlowja | "b==1.0"; | 00:47 |
| harlowja | "a==1.0"; | 00:47 |
| harlowja | "a==1.0" -> "b==1.0" [invariant=True]; | 00:47 |
| harlowja | } | 00:47 |
| harlowja | (which then can be piped into graphviz binaries) to get an image... | 00:47 |
| harlowja | for example | 00:47 |
| harlowja | and magic, lol | 00:48 |
| winston-d | harlowja: btw, is there a way to enforce the ordering between tasks? | 00:48 |
| harlowja | so thats basically what the patterns in taskflow do https://github.com/openstack/taskflow/tree/master/taskflow/patterns | 00:49 |
| harlowja | those patterns really only exist to enforce orderings :) | 00:49 |
| harlowja | with the graph_flow allowing for arbitrary dag(s) | 00:49 |
| harlowja | ^ which is the closest 'pattern' to the internal structure used by an engine | 00:50 |
| harlowja | https://github.com/openstack/taskflow/blob/master/taskflow/patterns/graph_flow.py#L76 can be used to do weirdo orderings if u want, lol | 00:50 |
| winston-d | like a manifest for ordering, so when somebody tries to change the ordering, simple unit test would be able to detect the violation. | 00:51 |
| harlowja | u could valiadate 'e.compilation.execution_graph' ? | 00:51 |
| harlowja | or do some kind of validation on it? | 00:51 |
| harlowja | (its basically https://github.com/networkx/networkx/blob/master/networkx/classes/digraph.py#L17 ) | 00:51 |
| harlowja | and maybe some routine in https://github.com/networkx/networkx/tree/master/networkx can help | 00:52 |
| harlowja | or compare it with manifest made using one of https://github.com/networkx/networkx/tree/master/networkx/readwrite ? | 00:52 |
| *** jdurgin has joined #openstack-cinder | 00:52 | |
| harlowja | (just an idear) | 00:52 |
| *** salv-orl_ has joined #openstack-cinder | 00:53 | |
| harlowja | * https://github.com/networkx/networkx#networkx (what networkx is) | 00:53 |
| openstackgerrit | Li Yingjun proposed openstack/python-cinderclient: Updating volume type 'is_public' status support https://review.openstack.org/221564 | 00:53 |
| harlowja | gotta be something in https://github.com/networkx/networkx/tree/master/networkx/algorithms to do maybe what u want :-P | 00:54 |
| harlowja | but maybe overkill, idk | 00:55 |
| harlowja | ha | 00:55 |
| *** salv-orlando has quit IRC | 00:56 | |
| winston-d | IIUC, the ordering is already there when creating task ('cos it's part of task logic), but it's only explictly declared when put into flows. I wonder if there is a way to construct a task ordering manifest so that taskflow can use to validate when creating the flow: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L783 | 00:56 |
| harlowja | seems like something that could be built-in or built in cinder, sounds simple to traverse the graph and verify ordering | 00:57 |
| *** salv-orl_ has quit IRC | 00:57 | |
| winston-d | meaning is there a way to explictly declare ordering rather than looking at source code of each task, so that somebody won't accidentally change the ordering here, for example: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/api/create_volume.py#L783 | 00:58 |
| harlowja | using the link method of graph flow (vs letting the linear flow do linear ordering explicitly) | 00:59 |
| *** leeantho has quit IRC | 00:59 | |
| harlowja | http://paste.openstack.org/show/463794/ | 00:59 |
| harlowja | or ^ to verify yourself | 01:00 |
| *** DericHorn-HP has joined #openstack-cinder | 01:00 | |
| harlowja | using 'has_edge' or other various methods on graphs | 01:00 |
| harlowja | but am thinking u want a kind of dict manifest? | 01:01 |
| harlowja | or something that doesn't care about patterns? | 01:01 |
| harlowja | just provide it, and do it, lol | 01:01 |
| harlowja | "a->b" means a->b damnit, lol | 01:01 |
| harlowja | and thats it, lol | 01:01 |
| * harlowja wonders what that manifest structure would look like | 01:02 | |
| harlowja | (likely not strings, ha) | 01:02 |
| winston-d | harlowja: y, i was thinking about taskflow.validate_flow_ordering(flow_chart) | 01:02 |
| harlowja | ya, that could do all that has_edge checks and stuff | 01:02 |
| harlowja | doesn't seem to hard, depending on what flow_chart is ,ha | 01:03 |
| harlowja | does not read visio files, lol | 01:03 |
| winston-d | lol, how about SVG | 01:03 |
| harlowja | ;-/ | 01:03 |
| harlowja | :-/ i mean | 01:03 |
| harlowja | https://github.com/networkx/networkx/tree/master/networkx/readwrite/json_graph is somewhat used i think | 01:03 |
| harlowja | then u can use the same manifests in horizon (somehow) | 01:03 |
| harlowja | i think that might be https://github.com/jsongraph/json-graph-specification ? | 01:04 |
| harlowja | maybe can use networkx ability to read that, then ask taskflow to check it or something | 01:04 |
| harlowja | gotta be some overlap algorithm that can do it, lol | 01:04 |
| *** dims_ has joined #openstack-cinder | 01:05 | |
| winston-d | yeah, not easy when you think about how weird a flow chart can be | 01:05 |
| harlowja | it'd be neat to someday give the json_graph form of engine.compilation.exection_graph to horizon, then be able to show the work happening visually | 01:06 |
| harlowja | someday man someday | 01:06 |
| winston-d | :) | 01:06 |
| harlowja | but i think something like it is possible, at a basic level its not hard to validate edge existence and all that | 01:07 |
| harlowja | depends on how u determine equality (which can be a hard problem i think) | 01:07 |
| winston-d | hmm, true | 01:08 |
| harlowja | https://github.com/networkx/networkx/blob/master/networkx/algorithms/isomorphism/isomorphvf2.py#L2 gets into the compilcated area, ha | 01:09 |
| harlowja | 'formal notion of "isomorphism", e.g., of "graph isomorphism", captures the informal notion that some objects have "the same structure"' (for what isomorphism is) | 01:09 |
| harlowja | * https://en.wikipedia.org/wiki/Graph_isomorphism#Motivation | 01:09 |
| harlowja | but ya, anyway | 01:10 |
| harlowja | ha | 01:10 |
| harlowja | enough theory for today folks | 01:10 |
| harlowja | lol | 01:10 |
| harlowja | homework due at 10am | 01:10 |
| winston-d | ;) i will skip tomrrow's class for sure | 01:10 |
| openstackgerrit | xing-yang proposed openstack/cinder: Extra specs may not be in volume types https://review.openstack.org/223393 | 01:10 |
| harlowja | : | 01:10 |
| harlowja | :) | 01:10 |
| harlowja | winston-d but maybe we can start small, see how it goes | 01:11 |
| harlowja | see if we can avoid turning this into a NP complete equivalence problem | 01:12 |
| harlowja | lol | 01:12 |
| harlowja | (which i think the generic case is, lol) | 01:12 |
| harlowja | *and not read visio files* | 01:12 |
| harlowja | lol | 01:12 |
| winston-d | right, we can take baby steps. | 01:13 |
| harlowja | ya | 01:13 |
| harlowja | alright, bbl | 01:14 |
| harlowja | please read pages 1-1000 by 10am to | 01:14 |
| winston-d | harlowja: thx man, learnt a lot already, need time to digest | 01:14 |
| harlowja | thx | 01:14 |
| harlowja | :) | 01:14 |
| harlowja | pages 1-1000 or else | 01:14 |
| harlowja | lol | 01:14 |
| *** baojg has joined #openstack-cinder | 01:15 | |
| *** crose has joined #openstack-cinder | 01:16 | |
| *** r-daneel has quit IRC | 01:17 | |
| *** cdelatte has joined #openstack-cinder | 01:18 | |
| *** stevemar has joined #openstack-cinder | 01:19 | |
| *** garthb has quit IRC | 01:19 | |
| *** baojg has quit IRC | 01:20 | |
| *** kaisers has quit IRC | 01:29 | |
| *** stevemar has quit IRC | 01:29 | |
| *** stevemar has joined #openstack-cinder | 01:30 | |
| *** dims_ has quit IRC | 01:34 | |
| *** ebalduf has joined #openstack-cinder | 01:37 | |
| *** baojg has joined #openstack-cinder | 01:38 | |
| *** kaisers has joined #openstack-cinder | 01:38 | |
| *** Apoorva_ has joined #openstack-cinder | 01:40 | |
| *** ebalduf has quit IRC | 01:41 | |
| *** Lee1092 has joined #openstack-cinder | 01:43 | |
| *** Apoorva has quit IRC | 01:44 | |
| *** Apoorva_ has quit IRC | 01:45 | |
| *** lkong has joined #openstack-cinder | 01:46 | |
| *** zhenguo has joined #openstack-cinder | 01:50 | |
| *** xyang1 has quit IRC | 01:52 | |
| *** cdelatte has quit IRC | 01:56 | |
| *** erhudy1 has quit IRC | 01:57 | |
| *** sasukeh has joined #openstack-cinder | 02:07 | |
| *** sasukeh has quit IRC | 02:14 | |
| *** haomaiwang has joined #openstack-cinder | 02:14 | |
| *** DericHorn-HP has quit IRC | 02:17 | |
| *** stevemar_ has joined #openstack-cinder | 02:20 | |
| *** hodos|2 has joined #openstack-cinder | 02:25 | |
| *** hodos has quit IRC | 02:27 | |
| *** kjelly has quit IRC | 02:27 | |
| *** zhenguo has quit IRC | 02:28 | |
| *** baojg has quit IRC | 02:28 | |
| *** stevemar has quit IRC | 02:28 | |
| *** theanalyst has quit IRC | 02:28 | |
| *** lennyb has quit IRC | 02:28 | |
| *** p0rtal_ has joined #openstack-cinder | 02:29 | |
| *** baojg has joined #openstack-cinder | 02:29 | |
| *** lennyb has joined #openstack-cinder | 02:29 | |
| *** crose has quit IRC | 02:31 | |
| *** p0rtal has quit IRC | 02:32 | |
| *** zhenguo has joined #openstack-cinder | 02:33 | |
| *** theanalyst has joined #openstack-cinder | 02:34 | |
| *** haomaiwang has quit IRC | 02:37 | |
| *** bkopilov has quit IRC | 02:38 | |
| *** mdbooth has quit IRC | 02:40 | |
| *** haomaiwang has joined #openstack-cinder | 02:43 | |
| *** mdbooth has joined #openstack-cinder | 02:46 | |
| *** haomaiwang has quit IRC | 02:50 | |
| *** xiaohui has quit IRC | 02:51 | |
| *** haomaiwang has joined #openstack-cinder | 02:52 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** DericHorn-HP has joined #openstack-cinder | 03:01 | |
| *** leeantho has joined #openstack-cinder | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** baojg has quit IRC | 03:07 | |
| *** leeantho_ has joined #openstack-cinder | 03:08 | |
| *** leeantho has quit IRC | 03:12 | |
| *** DericHorn-HP has quit IRC | 03:12 | |
| *** DericHorn-HP has joined #openstack-cinder | 03:15 | |
| *** fifieldt has joined #openstack-cinder | 03:18 | |
| *** baojg has joined #openstack-cinder | 03:22 | |
| *** mragupat has joined #openstack-cinder | 03:24 | |
| *** gouthamr has joined #openstack-cinder | 03:25 | |
| *** mragupat_ has joined #openstack-cinder | 03:25 | |
| *** mragupat has quit IRC | 03:29 | |
| *** gouthamr has quit IRC | 03:29 | |
| *** boris-42 has quit IRC | 03:30 | |
| *** gouthamr has joined #openstack-cinder | 03:32 | |
| *** gouthamr_ has joined #openstack-cinder | 03:34 | |
| *** rushiagr_away is now known as rushiagr | 03:35 | |
| jgriffith | anybody know what's up with the keystone install stuff? | 03:36 |
| *** gouthamr has quit IRC | 03:37 | |
| *** mudassirlatif has quit IRC | 03:37 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Attaching enhancement for EMC VNX driver https://review.openstack.org/223908 | 03:39 |
| *** david-lyle has joined #openstack-cinder | 03:41 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Attaching enhancement for EMC VNX driver https://review.openstack.org/223908 | 03:41 |
| *** jungleboyj has joined #openstack-cinder | 03:44 | |
| stevemar_ | jgriffith: care to elaborate? | 03:45 |
| jgriffith | stevemar_: https://bugs.launchpad.net/keystone/+bug/1496222 | 03:45 |
| openstack | Launchpad bug 1496222 in Keystone "Requirements update breaks keystone install on 3'rd party CI systems" [Undecided,New] | 03:45 |
| stevemar_ | jgriffith: oh i tohught bknudson or david-chen fixed this, it was a pbr thing | 03:47 |
| stevemar_ | lifeless: ^ | 03:47 |
| jgriffith | stevemar_: working on a fix now, but it will take a while to test it out | 03:47 |
| jgriffith | stevemar_: wonder if it just hasn't merged yet maybe? | 03:47 |
| jgriffith | stevemar_: maybe I will call it a night after all :) | 03:48 |
| lifeless | stevemar_: fixed in pbr 1.8 | 03:48 |
| lifeless | jgriffith: ^ | 03:48 |
| lifeless | ah | 03:49 |
| lifeless | upper-constraints.txt has pbr 1.6.0 in it | 03:49 |
| *** boris-42 has joined #openstack-cinder | 03:49 | |
| *** boris-42 has quit IRC | 03:49 | |
| *** boris-42 has joined #openstack-cinder | 03:49 | |
| lifeless | jgriffith: / stevemar_: propose a change to openstack/requirements to have pbr 1.8.0 in upper-constraints.txt | 03:49 |
| jgriffith | lifeless: cool... although until that propagates we're kinda hosed | 03:49 |
| jgriffith | lifeless: yeah, and that's the problem | 03:49 |
| jgriffith | well... I didn't realize it was a pbr thing, but you know what I mean | 03:49 |
| lifeless | no propogation needed | 03:50 |
| lifeless | its instant once we land it | 03:50 |
| jgriffith | lifeless: cool | 03:50 |
| jgriffith | lifeless: stevemar_ thank you | 03:50 |
| lifeless | because its centralised | 03:50 |
| jgriffith | lifeless: yep | 03:50 |
| jgriffith | makes life easier | 03:50 |
| stevemar_ | should still up the upper-constraints i think | 03:50 |
| stevemar_ | jgriffith: gonna mark your bug invalid, cool | 03:50 |
| stevemar_ | ? | 03:50 |
| jgriffith | stevemar_: Actually I was going to mark it as fixed with the review if you don't mind? | 03:51 |
| jgriffith | stevemar_: for people like me that run into this and try and search on it :) | 03:51 |
| stevemar_ | jgriffith: ah cool | 03:51 |
| jgriffith | or at least invalid with a link to the review and the bug you used (because surely you filed a bug) :) | 03:52 |
| stevemar_ | looks like someone else just opened it too: https://bugs.launchpad.net/keystone/+bug/1496220 | 03:53 |
| openstack | Launchpad bug 1496220 in Keystone "error in setup command: Invalid environment marker: (python_version=='2.7' # MPL)" [Undecided,New] | 03:53 |
| jgriffith | stevemar_: is this the change you were talking about? https://review.openstack.org/#/c/221157/7/upper-constraints.txt | 03:53 |
| jgriffith | stevemar_: looks like that's stuck :( | 03:54 |
| jgriffith | hehe.. well look at that :) | 03:54 |
| lifeless | ok, I have to run out in a minute | 03:55 |
| lifeless | if you can push up the openstack/requirements change, I can +2A it in about 45-50m, after its passed CI | 03:55 |
| lifeless | just edit upper-constraints.txt by hand, pbr line only, 1.6 -> 1.8 | 03:55 |
| stevemar_ | lifeless: will do | 03:56 |
| *** zerda has joined #openstack-cinder | 03:57 | |
| jgriffith | lifeless: will do | 03:57 |
| jgriffith | stevemar_: I marked my bug as a dup, Sam beat me by 9 minutes :) | 03:57 |
| jgriffith | stevemar_: you want to do it, or shall I? | 03:57 |
| *** rushiagr is now known as rushiagr_away | 03:57 | |
| stevemar_ | jgriffith: i'll do it, already started | 03:57 |
| jgriffith | too late :) | 03:57 |
| jgriffith | Just kidding | 03:58 |
| jgriffith | I'll leave it to you | 03:58 |
| *** p0rtal_ has quit IRC | 03:58 | |
| *** haomaiwang has quit IRC | 03:58 | |
| * jgriffith changes back to his other repo and continues what he was doing :) | 03:58 | |
| jgriffith | thanks again stevemar_ and lifeless | 03:58 |
| *** haomaiwang has joined #openstack-cinder | 03:58 | |
| *** p0rtal has joined #openstack-cinder | 03:58 | |
| *** DericHorn-HP has quit IRC | 03:59 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 04:01 | |
| *** einarr has quit IRC | 04:02 | |
| *** rick-chen has joined #openstack-cinder | 04:02 | |
| *** p0rtal has quit IRC | 04:03 | |
| stevemar_ | jgriffith: lifeless: https://review.openstack.org/#/c/223914/ | 04:03 |
| *** changbl has joined #openstack-cinder | 04:04 | |
| *** bkopilov has joined #openstack-cinder | 04:04 | |
| stevemar_ | hmm, this change clearly goes against the guidelines here: https://github.com/openstack/requirements/blob/master/README.rst#for-upper-constraintstxt-changes but i think the proposal bot is stuck | 04:05 |
| *** sgotliv_ has joined #openstack-cinder | 04:06 | |
| jgriffith | stevemar_: indeed | 04:07 |
| stevemar_ | jgriffith: oops, looks like you have the same patch to requirements | 04:07 |
| jgriffith | stevemar_: ? | 04:07 |
| jgriffith | Oh crap | 04:07 |
| jgriffith | stevemar_: sorry! | 04:07 |
| stevemar_ | haha | 04:07 |
| stevemar_ | https://review.openstack.org/#/c/223913/ | 04:07 |
| stevemar_ | its all good | 04:08 |
| jgriffith | I didn't mean to push that after you said you had it | 04:08 |
| stevemar_ | i will abandon mine, yours came in a minute earlier :) | 04:08 |
| jgriffith | I had "git review" when I said, "joking and ok" | 04:09 |
| jgriffith | didn't mean to hit enter | 04:09 |
| jgriffith | stevemar_: nah | 04:09 |
| jgriffith | I'm going to see if I can fix the stuck patch and request it speeded up | 04:09 |
| jgriffith | stevemar_: there, mines abandoned :) | 04:09 |
| stevemar_ | jgriffith: LOL | 04:09 |
| stevemar_ | i just .. haha | 04:10 |
| jgriffith | stevemar_: don't tell me you just | 04:10 |
| jgriffith | NOOOO | 04:10 |
| jgriffith | LOL | 04:10 |
| jgriffith | too funny | 04:10 |
| stevemar_ | oh thats good | 04:10 |
| stevemar_ | this is the last time lifeless leaves us unsupervised | 04:10 |
| stevemar_ | jgriffith: please go ahead and restore yours, i'm heading off soon anyway | 04:11 |
| jgriffith | stevemar_: ok, thanks... sorry for all the confusion :( | 04:12 |
| stevemar_ | jgriffith: its all good, i had a laugh | 04:12 |
| jgriffith | have a good night | 04:13 |
| *** sasukeh has joined #openstack-cinder | 04:16 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:21 | |
| *** sasukeh has quit IRC | 04:21 | |
| *** DericHorn-HP has joined #openstack-cinder | 04:26 | |
| *** sasukeh_ has joined #openstack-cinder | 04:26 | |
| *** sasukeh_ has quit IRC | 04:27 | |
| *** coolsvap has joined #openstack-cinder | 04:28 | |
| *** jungleboyj has quit IRC | 04:29 | |
| *** baojg has quit IRC | 04:33 | |
| *** stevemar_ has quit IRC | 04:38 | |
| *** baojg has joined #openstack-cinder | 04:40 | |
| *** DericHorn-HP has quit IRC | 04:40 | |
| *** jasl8r has quit IRC | 04:42 | |
| *** david-lyle has quit IRC | 04:42 | |
| *** jasl8r has joined #openstack-cinder | 04:43 | |
| *** rushiagr_away is now known as rushiagr | 04:43 | |
| *** david-lyle has joined #openstack-cinder | 04:45 | |
| *** jasl8r has quit IRC | 04:46 | |
| *** dannywilson has joined #openstack-cinder | 04:46 | |
| *** sasukeh has joined #openstack-cinder | 04:52 | |
| *** sasukeh has quit IRC | 04:57 | |
| *** amit213 has quit IRC | 04:59 | |
| *** amit213 has joined #openstack-cinder | 04:59 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** lkong has quit IRC | 05:13 | |
| *** lkong has joined #openstack-cinder | 05:13 | |
| *** garthb has joined #openstack-cinder | 05:14 | |
| *** leeantho_ has quit IRC | 05:15 | |
| *** gouthamr_ has quit IRC | 05:17 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Attaching enhancement for EMC VNX driver https://review.openstack.org/223908 | 05:23 |
| *** mragupat_ has quit IRC | 05:24 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Retype enhancement for EMC VNX cinder driver https://review.openstack.org/223924 | 05:27 |
| *** rick-chen has quit IRC | 05:27 | |
| *** baojg has quit IRC | 05:28 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Attaching enhancement for EMC VNX driver https://review.openstack.org/223908 | 05:31 |
| *** ankit_ag has joined #openstack-cinder | 05:38 | |
| *** yrabl has quit IRC | 05:38 | |
| *** harlowja_at_home has quit IRC | 05:41 | |
| *** vgridnev has joined #openstack-cinder | 05:45 | |
| *** p0rtal has joined #openstack-cinder | 05:49 | |
| lkong | Hi may i ask a question, what's the different between v1 and v2 | 05:54 |
| lifeless | strictlyb: jgriffith ? | 05:56 |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:02 | |
| *** EinstCrazy has joined #openstack-cinder | 06:06 | |
| *** _cjones_ has joined #openstack-cinder | 06:09 | |
| *** _cjones_ has quit IRC | 06:09 | |
| *** _cjones_ has joined #openstack-cinder | 06:10 | |
| *** deepakcs has joined #openstack-cinder | 06:11 | |
| *** uberjay has quit IRC | 06:17 | |
| *** uberjay has joined #openstack-cinder | 06:17 | |
| wanghao | ping winston-d | 06:21 |
| wanghao | hi here? | 06:21 |
| wanghao | 'Improve performance listing detail for volumes' patch has been updated for adding some new tests, would you check it if you have time. | 06:23 |
| *** markvoelker has quit IRC | 06:25 | |
| *** sgotliv_ has quit IRC | 06:26 | |
| *** e0ne has joined #openstack-cinder | 06:37 | |
| *** stevemar has joined #openstack-cinder | 06:38 | |
| *** p0rtal has quit IRC | 06:39 | |
| *** e0ne has quit IRC | 06:39 | |
| *** p0rtal has joined #openstack-cinder | 06:39 | |
| *** drjones has joined #openstack-cinder | 06:40 | |
| *** stevemar has quit IRC | 06:42 | |
| *** sasukeh has joined #openstack-cinder | 06:44 | |
| *** p0rtal has quit IRC | 06:44 | |
| *** _cjones_ has quit IRC | 06:44 | |
| *** ronis has joined #openstack-cinder | 06:53 | |
| *** sasukeh has quit IRC | 06:55 | |
| *** vgridnev has quit IRC | 06:57 | |
| *** sasukeh has joined #openstack-cinder | 06:58 | |
| *** yrabl has joined #openstack-cinder | 07:00 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:02 | |
| *** sasukeh has quit IRC | 07:03 | |
| *** rushiagr is now known as rushiagr_away | 07:03 | |
| *** vgridnev has joined #openstack-cinder | 07:11 | |
| winston-d | wanghao: sure | 07:12 |
| *** garthb has quit IRC | 07:12 | |
| *** anshul has joined #openstack-cinder | 07:15 | |
| *** boris-42 has quit IRC | 07:19 | |
| *** theanalyst has quit IRC | 07:19 | |
| *** zhenguo has quit IRC | 07:19 | |
| *** abehl has joined #openstack-cinder | 07:24 | |
| *** zhenguo has joined #openstack-cinder | 07:24 | |
| *** abehl has quit IRC | 07:24 | |
| *** abehl has joined #openstack-cinder | 07:24 | |
| *** boris-42 has joined #openstack-cinder | 07:25 | |
| *** theanalyst has joined #openstack-cinder | 07:25 | |
| *** markvoelker has joined #openstack-cinder | 07:26 | |
| *** EinstCrazy has quit IRC | 07:27 | |
| *** dannywilson has quit IRC | 07:30 | |
| *** markvoelker has quit IRC | 07:30 | |
| *** abehl has quit IRC | 07:33 | |
| wanghao | :winston-d: Thanks :) | 07:40 |
| *** sgotliv_ has joined #openstack-cinder | 07:46 | |
| *** jordanP has joined #openstack-cinder | 07:53 | |
| *** pschaef has joined #openstack-cinder | 07:55 | |
| *** drjones has quit IRC | 07:57 | |
| *** _cjones_ has joined #openstack-cinder | 07:58 | |
| *** vgridnev has quit IRC | 07:58 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** e0ne has joined #openstack-cinder | 08:03 | |
| openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 08:03 |
| *** haomaiwa_ has joined #openstack-cinder | 08:04 | |
| *** jistr has joined #openstack-cinder | 08:19 | |
| *** e0ne has quit IRC | 08:24 | |
| *** alexpilotti has joined #openstack-cinder | 08:35 | |
| *** ndipanov has quit IRC | 08:35 | |
| *** haomaiwa_ has quit IRC | 08:39 | |
| *** haomaiwang has joined #openstack-cinder | 08:40 | |
| *** ndipanov has joined #openstack-cinder | 08:43 | |
| *** _cjones_ has quit IRC | 08:45 | |
| *** e0ne has joined #openstack-cinder | 08:46 | |
| *** kbyrne has joined #openstack-cinder | 08:53 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:02 | |
| *** kbader has joined #openstack-cinder | 09:02 | |
| *** lpetrut has joined #openstack-cinder | 09:05 | |
| *** deepakcs has quit IRC | 09:08 | |
| *** aix has joined #openstack-cinder | 09:18 | |
| *** markvoelker has joined #openstack-cinder | 09:26 | |
| *** markvoelker has quit IRC | 09:31 | |
| *** dims_ has joined #openstack-cinder | 09:35 | |
| *** vgridnev has joined #openstack-cinder | 09:39 | |
| *** stevemar has joined #openstack-cinder | 09:40 | |
| *** stevemar has quit IRC | 09:44 | |
| *** _cjones_ has joined #openstack-cinder | 09:47 | |
| *** baojg has joined #openstack-cinder | 09:48 | |
| *** _cjones_ has quit IRC | 09:52 | |
| openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 09:53 |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** IanGovett has joined #openstack-cinder | 10:02 | |
| *** IanGovett has quit IRC | 10:18 | |
| *** jkraj has joined #openstack-cinder | 10:19 | |
| *** baojg has quit IRC | 10:30 | |
| *** baojg has joined #openstack-cinder | 10:31 | |
| *** ociuhandu has quit IRC | 10:31 | |
| *** lkong has quit IRC | 10:32 | |
| *** baojg has quit IRC | 10:35 | |
| *** bluex-pl has joined #openstack-cinder | 10:36 | |
| *** bluex-pl has quit IRC | 10:37 | |
| *** bluex-pl has joined #openstack-cinder | 10:37 | |
| *** kbader has joined #openstack-cinder | 10:49 | |
| *** kbader has quit IRC | 10:49 | |
| *** aix has quit IRC | 10:50 | |
| *** IanGovett has joined #openstack-cinder | 10:51 | |
| *** e0ne has quit IRC | 10:56 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** aix has joined #openstack-cinder | 11:03 | |
| *** mtaylor is now known as mordred | 11:12 | |
| *** e0ne has joined #openstack-cinder | 11:13 | |
| *** cdelatte has joined #openstack-cinder | 11:15 | |
| *** _cjones_ has joined #openstack-cinder | 11:17 | |
| *** ociuhandu has joined #openstack-cinder | 11:21 | |
| *** _cjones_ has quit IRC | 11:22 | |
| *** bkopilov has quit IRC | 11:26 | |
| *** markvoelker has joined #openstack-cinder | 11:27 | |
| *** julim has joined #openstack-cinder | 11:28 | |
| *** vgridnev has quit IRC | 11:30 | |
| *** markvoelker has quit IRC | 11:32 | |
| *** DericHorn-HP has joined #openstack-cinder | 11:34 | |
| *** haomaiwang has quit IRC | 11:35 | |
| *** vgridnev has joined #openstack-cinder | 11:37 | |
| *** haomaiwang has joined #openstack-cinder | 11:38 | |
| *** boris-42 has quit IRC | 11:50 | |
| *** ankit_ag has quit IRC | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:01 | |
| *** raildo-afk is now known as raildo | 12:02 | |
| *** zerda has quit IRC | 12:03 | |
| DuncanT | e0ne: Ping re quota get | 12:07 |
| e0ne | DuncanT: hi | 12:07 |
| DuncanT | e0ne: Just read your mail about fetching quota details from cinder | 12:08 |
| e0ne | DuncanT: yes. I would try to use admin context there | 12:09 |
| DuncanT | e0ne: I'm tempted to propose a revert of the keystone validation code in that code path - if the default rules in keystone mean a call taht was available (and needed by) normal users is now admin only, that's an API breakage, and a fairly serious one | 12:09 |
| DuncanT | e0ne: We can't just use an admin context, we need admin credentials | 12:09 |
| DuncanT | e0ne: Or an admin token | 12:10 |
| e0ne | DuncanT: +2 it it breaks existing api | 12:10 |
| DuncanT | Ok, I'll propose the revert and let anybody who really wants the validation propose something better. It isn't a trivial API break, it's a serious usability issue for people not to be able to tell how much quota they have | 12:10 |
| e0ne | let me check how it works in Kilo | 12:11 |
| *** haomaiwa_ has quit IRC | 12:11 | |
| *** markvoelker has joined #openstack-cinder | 12:12 | |
| *** tpsilva has joined #openstack-cinder | 12:13 | |
| *** akerr has joined #openstack-cinder | 12:15 | |
| *** akerr has quit IRC | 12:16 | |
| e0ne | DuncanT: looking thow the code - it should work in Kilo | 12:17 |
| *** akerr has joined #openstack-cinder | 12:17 | |
| DuncanT | e0ne: Yup, in kilo a tenant could get their own quota. The API was a bit poor, but it worked | 12:17 |
| *** _cjones_ has joined #openstack-cinder | 12:19 | |
| e0ne | DuncanT: so, what is the plan? try to revert commit? | 12:19 |
| DuncanT | e0ne: Yeah, I'm just putting together a revert commit now | 12:19 |
| DuncanT | e0ne: It isn't quite clean, so I'll have to do a bit of testing | 12:20 |
| *** EinstCrazy has joined #openstack-cinder | 12:23 | |
| *** _cjones_ has quit IRC | 12:23 | |
| *** jaypipes has quit IRC | 12:26 | |
| *** jaypipes_ has joined #openstack-cinder | 12:26 | |
| *** manas has joined #openstack-cinder | 12:28 | |
| *** timcl has joined #openstack-cinder | 12:32 | |
| *** jasl8r has joined #openstack-cinder | 12:33 | |
| *** edmondsw has joined #openstack-cinder | 12:33 | |
| *** e0ne has quit IRC | 12:38 | |
| *** EinstCrazy has quit IRC | 12:40 | |
| *** porrua has joined #openstack-cinder | 12:41 | |
| *** sage has quit IRC | 12:42 | |
| *** sage has joined #openstack-cinder | 12:43 | |
| * DuncanT is adding a few comments to the summit etherpad now, if anybody thinks that's a terrible idea, let me know | 12:43 | |
| *** bkopilov has joined #openstack-cinder | 12:45 | |
| *** raildo is now known as raildo-afk | 12:45 | |
| *** martyturner has joined #openstack-cinder | 12:46 | |
| *** dims_ has quit IRC | 12:47 | |
| *** sage has quit IRC | 12:49 | |
| *** DericHorn-HP has quit IRC | 12:50 | |
| *** bkopilov has quit IRC | 12:51 | |
| *** sage has joined #openstack-cinder | 12:51 | |
| *** bkopilov has joined #openstack-cinder | 12:51 | |
| *** annegentle has joined #openstack-cinder | 12:54 | |
| *** bill_az has joined #openstack-cinder | 12:55 | |
| *** smoriya has quit IRC | 12:55 | |
| *** aix has quit IRC | 12:56 | |
| *** xyang1 has joined #openstack-cinder | 12:57 | |
| *** dims_ has joined #openstack-cinder | 12:58 | |
| *** martyturner has quit IRC | 12:58 | |
| *** timcl1 has joined #openstack-cinder | 13:01 | |
| *** eharney has joined #openstack-cinder | 13:02 | |
| *** martyturner has joined #openstack-cinder | 13:04 | |
| *** erlon has joined #openstack-cinder | 13:04 | |
| *** timcl has quit IRC | 13:05 | |
| *** DericHorn-HP has joined #openstack-cinder | 13:05 | |
| *** esker has quit IRC | 13:05 | |
| *** sage has quit IRC | 13:07 | |
| *** gouthamr has joined #openstack-cinder | 13:07 | |
| *** raildo-afk is now known as raildo | 13:08 | |
| *** mriedem has joined #openstack-cinder | 13:08 | |
| *** e0ne has joined #openstack-cinder | 13:09 | |
| *** jkraj has quit IRC | 13:10 | |
| *** aix has joined #openstack-cinder | 13:10 | |
| *** sage has joined #openstack-cinder | 13:11 | |
| *** _cjones_ has joined #openstack-cinder | 13:19 | |
| *** krtaylor has quit IRC | 13:20 | |
| *** dustins has joined #openstack-cinder | 13:21 | |
| *** Yogi1 has joined #openstack-cinder | 13:23 | |
| *** gouthamr has quit IRC | 13:23 | |
| *** _cjones_ has quit IRC | 13:24 | |
| *** gouthamr has joined #openstack-cinder | 13:26 | |
| *** annegentle has quit IRC | 13:30 | |
| *** annegentle has joined #openstack-cinder | 13:30 | |
| *** salv-orlando has joined #openstack-cinder | 13:31 | |
| *** dustins has quit IRC | 13:32 | |
| *** sgotliv_ has quit IRC | 13:35 | |
| *** kbader has joined #openstack-cinder | 13:37 | |
| *** dustins has joined #openstack-cinder | 13:39 | |
| *** kbader1 has joined #openstack-cinder | 13:40 | |
| *** asselin_ has joined #openstack-cinder | 13:41 | |
| *** kbader has quit IRC | 13:41 | |
| *** rushiagr_away is now known as rushiagr | 13:42 | |
| *** mtanino has joined #openstack-cinder | 13:42 | |
| *** ebalduf has joined #openstack-cinder | 13:43 | |
| *** zhenguo has quit IRC | 13:43 | |
| *** DericHorn-HP has quit IRC | 13:45 | |
| *** rushiagr is now known as rushiagr_away | 13:48 | |
| *** sasukeh has joined #openstack-cinder | 13:49 | |
| *** sgotliv_ has joined #openstack-cinder | 13:49 | |
| *** Zhongjun has joined #openstack-cinder | 13:50 | |
| *** ebalduf has quit IRC | 13:50 | |
| *** merooney has joined #openstack-cinder | 13:53 | |
| *** erhudy has joined #openstack-cinder | 13:54 | |
| *** merooney has quit IRC | 14:00 | |
| *** Yogi11 has joined #openstack-cinder | 14:00 | |
| *** Yogi1 has quit IRC | 14:01 | |
| *** sayali has quit IRC | 14:02 | |
| *** merooney has joined #openstack-cinder | 14:02 | |
| *** annasort has quit IRC | 14:03 | |
| *** ericksonsantos has quit IRC | 14:04 | |
| *** mragupat has joined #openstack-cinder | 14:09 | |
| *** nikeshm has quit IRC | 14:10 | |
| *** mragupat has quit IRC | 14:10 | |
| kbader1 | boot from volume aside, is there a way to influence whether a cinder volume uses virtio-scsi or virtio-blk? | 14:10 |
| *** aix has quit IRC | 14:11 | |
| *** nikeshm has joined #openstack-cinder | 14:12 | |
| *** r-daneel has joined #openstack-cinder | 14:12 | |
| *** julim has quit IRC | 14:12 | |
| *** sayali has joined #openstack-cinder | 14:14 | |
| *** _cjones_ has joined #openstack-cinder | 14:21 | |
| *** DericHorn-HP has joined #openstack-cinder | 14:22 | |
| DuncanT | kbader1 I think the glance metadata influences it, not sure though | 14:22 |
| *** aix has joined #openstack-cinder | 14:22 | |
| *** _cjones_ has quit IRC | 14:25 | |
| *** DericHorn-HP has quit IRC | 14:26 | |
| *** jasl8r has quit IRC | 14:26 | |
| *** thangp has joined #openstack-cinder | 14:27 | |
| *** crose has joined #openstack-cinder | 14:28 | |
| *** sayali has quit IRC | 14:28 | |
| *** anshul has quit IRC | 14:28 | |
| *** alexpilotti has quit IRC | 14:29 | |
| *** rick-chen has joined #openstack-cinder | 14:30 | |
| *** DericHorn-HP has joined #openstack-cinder | 14:30 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova (draft) https://review.openstack.org/224124 | 14:31 |
| *** garthb has joined #openstack-cinder | 14:31 | |
| *** openstackgerrit has quit IRC | 14:31 | |
| *** openstackgerrit has joined #openstack-cinder | 14:32 | |
| *** DericHorn-HP has quit IRC | 14:32 | |
| *** annasort has joined #openstack-cinder | 14:34 | |
| *** kbader1 has quit IRC | 14:36 | |
| *** kbader has joined #openstack-cinder | 14:36 | |
| *** julim has joined #openstack-cinder | 14:37 | |
| *** david-lyle has quit IRC | 14:37 | |
| *** haomaiwang has joined #openstack-cinder | 14:39 | |
| *** kbader1 has joined #openstack-cinder | 14:39 | |
| *** kbader1 has quit IRC | 14:40 | |
| *** lcurtis has joined #openstack-cinder | 14:40 | |
| *** sayali has joined #openstack-cinder | 14:41 | |
| *** kbader has quit IRC | 14:41 | |
| *** raildo is now known as raildo-afk | 14:41 | |
| *** raildo-afk is now known as raildo | 14:41 | |
| *** ebalduf has joined #openstack-cinder | 14:42 | |
| *** diogogmt has joined #openstack-cinder | 14:42 | |
| *** nikeshm has quit IRC | 14:42 | |
| *** kbader has joined #openstack-cinder | 14:46 | |
| *** charlesr has joined #openstack-cinder | 14:46 | |
| *** ganv_ has joined #openstack-cinder | 14:47 | |
| ganv_ | Hi. I am trying to deploy openstack swift as a backend storage to backing up cinder volumes. Is it possible to do this without keystone in the picture? | 14:47 |
| *** kbader1 has joined #openstack-cinder | 14:50 | |
| *** deepakcs has joined #openstack-cinder | 14:51 | |
| *** kbader has quit IRC | 14:51 | |
| *** edtubill has joined #openstack-cinder | 14:52 | |
| *** edtubill has quit IRC | 14:52 | |
| *** yrabl has quit IRC | 14:52 | |
| *** annegent_ has joined #openstack-cinder | 14:54 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Extra specs may not be in volume types https://review.openstack.org/223393 | 14:54 |
| *** annegentle has quit IRC | 14:54 | |
| *** edtubill has joined #openstack-cinder | 14:55 | |
| *** mc_nair has joined #openstack-cinder | 14:56 | |
| *** haomaiwang has quit IRC | 14:56 | |
| *** dims_ has quit IRC | 14:58 | |
| *** stevemar has joined #openstack-cinder | 14:58 | |
| *** dims_ has joined #openstack-cinder | 14:58 | |
| *** jistr is now known as jistr|call | 14:58 | |
| *** boris-42 has joined #openstack-cinder | 15:01 | |
| *** openstackgerrit has quit IRC | 15:01 | |
| *** openstackgerrit has joined #openstack-cinder | 15:02 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Extra specs may not be in volume types https://review.openstack.org/223393 | 15:04 |
| *** hemnafk is now known as hemna | 15:08 | |
| *** kbader1 has quit IRC | 15:11 | |
| *** thangp has quit IRC | 15:13 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:13 | |
| *** jungleboyj has joined #openstack-cinder | 15:14 | |
| *** jgregor has joined #openstack-cinder | 15:14 | |
| *** thangp has joined #openstack-cinder | 15:14 | |
| DuncanT | e0ne: Just added some very minor comments to your bare metal spec. If you get any pushback in general, let me know - I've a customer who wants this in upstream - we have a PoC with a hacked client already done a few months back | 15:16 |
| e0ne | DuncanT: thanks for feedback! it's very important to get it early | 15:17 |
| jgriffith | DuncanT: I have customers that want a pony | 15:17 |
| jgriffith | DuncanT: a few want two AND a Unicorn | 15:17 |
| e0ne | DuncanT: I'm going to update spec with PoC | 15:17 |
| e0ne | jgriffith: it's easier than have an unicorn | 15:17 |
| e0ne | :) | 15:17 |
| jgriffith | e0ne: :) | 15:17 |
| DuncanT | jgriffith: I thought you bred ponies? Glue a horn on one and make sure the contract is well written.... | 15:18 |
| e0ne | DuncanT: :) | 15:18 |
| jgriffith | DuncanT: perhaps I'll hire an attorney and make a fortune | 15:18 |
| DuncanT | e0ne: Yup. The host field is the only snag I've come across so far - that and the fact our API sucks, but it can be made to work easily enough | 15:18 |
| jgriffith | after I pay a fortune | 15:18 |
| DuncanT | jgriffith: The only people getting rich out of that one are the attornies, I suspect | 15:19 |
| e0ne | we've got a joke in Russian: giraffe is a horse, which is correspond to all customer's requirements | 15:19 |
| e0ne | DuncanT: I'm using fake host or local hostname in my poc | 15:20 |
| jgriffith | e0ne: :) I like it | 15:20 |
| DuncanT | e0ne: Yeah, I just had a placeholder value too | 15:21 |
| e0ne | DuncanT: our customers want this feature in upstream in scope of ironic integration | 15:21 |
| *** ronis has quit IRC | 15:21 | |
| *** asselin_ has quit IRC | 15:23 | |
| *** manas has quit IRC | 15:23 | |
| DuncanT | e0ne: Walt has a bunch of those customers too I think. Plenty of interest, and at least 3 of the interested parties are core, should help I hope | 15:23 |
| *** jaypipes_ is now known as jaypipes | 15:23 | |
| e0ne | :) | 15:23 |
| *** ganv_ has quit IRC | 15:23 | |
| *** rick-chen has quit IRC | 15:23 | |
| e0ne | DuncanT: it will be implemented. it's all about timeframe: Mikata or later | 15:24 |
| hemna | what did I do now? | 15:24 |
| *** deepakcs has quit IRC | 15:24 | |
| e0ne | hemna: evening:) | 15:24 |
| jgriffith | plans for Cinder world domination are finally coming together... mua-ha-mua-hahaa | 15:24 |
| DuncanT | hemna: Hi. You had an interest in bare-metal cinder attach :-) | 15:24 |
| hemna | DuncanT, yes :) | 15:24 |
| hemna | it's been an interest of mine for a while | 15:24 |
| hemna | I think we should iron this out in Tokyo | 15:25 |
| e0ne | jgriffith: we need to add it to Cinder Wiki | 15:25 |
| hemna | there have been thoughts of putting some aspect of it, or all of it in cinderclient | 15:25 |
| jgriffith | e0ne: World domination :) | 15:25 |
| hemna | and there have been serious concerns about it | 15:25 |
| jgriffith | hemna: there's already a well written spec and even a poc on the way | 15:25 |
| e0ne | jgriffith: it's a good mission for Cinder, I guess | 15:25 |
| jgriffith | e0ne: well.. somebody has to do it :) | 15:25 |
| hemna | jgriffith, yah I've looked at e0ne's poc | 15:25 |
| DuncanT | hemna: There's a PoC up... I'm against adding it to cinder client though - eone is talking about a cinder-baremetal client instead, which is good for me | 15:26 |
| jgriffith | DuncanT: +1 | 15:26 |
| e0ne | DuncanT: poc based on cinderclient is the easiest way to implement it | 15:26 |
| xyang1 | jgriffith: winston-d can you take a look of the extra spec patch again? https://review.openstack.org/#/c/223393/. Still waiting for Jenkins, but let me know if the approach looks ok | 15:26 |
| hemna | yah I'm ok with that, as long as cinder-baremetal is a project under cinder that's installable via pip | 15:26 |
| xyang1 | jgriffith: winston-d I removed the changes in scheduler | 15:26 |
| *** nkrinner is now known as nkrinner_afk | 15:26 | |
| e0ne | DuncanT: I don't want to have cookiecutter hell in such early stage | 15:27 |
| DuncanT | hemna: It should install as part of cinder-client IMO | 15:27 |
| e0ne | hemna: +1 | 15:27 |
| xyang1 | jgriffith: winston-d all changes are in volume api now | 15:27 |
| DuncanT | e0ne: For PoC that's fine, just don't merge it | 15:27 |
| *** sasukeh has quit IRC | 15:27 | |
| e0ne | DuncanT: sure. I don't want to mark it with WIP or -2 to get more feedback | 15:27 |
| hemna | DuncanT, as a separate bin? that sounds good | 15:27 |
| e0ne | DuncanT: it should depends on cinderclient and brick | 15:27 |
| jgriffith | xyang1: we've kind of ran you back and forth on this I fear | 15:28 |
| xyang1 | jgriffith: yes:) | 15:28 |
| jgriffith | xyang1: sorry :( | 15:28 |
| xyang1 | jgriffith: no problem:) | 15:28 |
| jgriffith | xyang1: I'm ok with what you have for sure. I do still think that in M we should treat them as two entities | 15:28 |
| jgriffith | :) | 15:28 |
| xyang1 | jgriffith: sure:) | 15:29 |
| DuncanT | hemna: Yes, or even an argv[0] check in the same binary and some python packaging magic... I just don't want the command line interface to the 'cinder' binary to get even more confusing | 15:29 |
| hemna | DuncanT, so if we make it part of the cinderclient codebase, then cinderclient requires os-brick | 15:29 |
| hemna | DuncanT, +1 | 15:29 |
| xyang1 | jgriffith: volume has a qos spec id, but not a extra spec id | 15:29 |
| jgriffith | xyang1: yeah..... it's sort of goofy | 15:29 |
| e0ne | DuncanT: +2 | 15:29 |
| jgriffith | xyang1: we use the type as the ID | 15:29 |
| DuncanT | hemna: If that's seen as a bad thing, I've no problem with it being packaged separately... whatever makes more sense | 15:29 |
| xyang1 | jgriffith: if we separate them, we should handle it the same way | 15:30 |
| jgriffith | xyang1: +100 | 15:30 |
| *** jistr|call is now known as jistr | 15:30 | |
| hemna | DuncanT, I don't think it's a bad thing, just waited to raise it. | 15:30 |
| e0ne | DuncanT: imo, we need to get as short list of dependencies for it, as we can | 15:30 |
| DuncanT | hemna: I'm easy either way - the command line usability is my sole concern - packaging good taste we have a reasonable amount of | 15:31 |
| hemna | sure, totally agree. | 15:31 |
| *** vgridnev has quit IRC | 15:31 | |
| hemna | I think this would confuse the API for the cinder client for sure, which is already a confusing API to begin with. | 15:32 |
| *** pots has joined #openstack-cinder | 15:32 | |
| jgriffith | hemna: curious what's confusing about it? Maybe we can make it better? | 15:32 |
| *** annegent_ has quit IRC | 15:32 | |
| hemna | jgriffith, well, I think we started working on that a bit in Fort Collins. | 15:32 |
| jgriffith | create, delete, list, snapshot-create, snapshot-delete, snapshot-list.... seems straight forward | 15:33 |
| hemna | yah that stuff is fine | 15:33 |
| hemna | it's the attachment process | 15:33 |
| *** annegentle has joined #openstack-cinder | 15:33 | |
| jgriffith | hemna: oh... that again :) | 15:33 |
| hemna | which the baremetal attach process would confuse even more if it was added to cinderclient itself. | 15:33 |
| *** haomaiwa_ has joined #openstack-cinder | 15:33 | |
| hemna | nm. I'm agreeing with you guys :) | 15:33 |
| hemna | ignore me. | 15:34 |
| jgriffith | not confusing to me: http://j-griffith.github.io/2015/07/16/volume-attach-code-flow-in-cinder/ :) | 15:34 |
| *** sasukeh has joined #openstack-cinder | 15:34 | |
| jgriffith | Just kidding | 15:34 |
| jgriffith | I'll shut up now | 15:34 |
| hemna | :) | 15:34 |
| *** david-ly_ has joined #openstack-cinder | 15:35 | |
| guitarzan | jgriffith: nice writeup! where's the detach one? :D | 15:35 |
| *** rick-chen has joined #openstack-cinder | 15:36 | |
| scottda | actually, you beat me to it. I was wondering if jgriffith would document the detach flow at some point as well. Please? | 15:37 |
| jgriffith | guitarzan: I really should do that | 15:38 |
| jgriffith | guitarzan: scottda I'll try and write it up tonight or tomorrow | 15:38 |
| jgriffith | thanks for the reminder | 15:38 |
| scottda | That'd be great. | 15:38 |
| *** gouthamr has quit IRC | 15:39 | |
| guitarzan | jgriffith: then blow people's minds and write up nova's block migrate | 15:39 |
| * guitarzan runs | 15:39 | |
| *** harlowja_at_home has quit IRC | 15:39 | |
| e0ne | :) | 15:41 |
| hemna | lol | 15:41 |
| hemna | guitarzan, only if we get a live migration writeup as well. | 15:41 |
| hemna | :P | 15:41 |
| *** david-ly_ is now known as david-lyle | 15:41 | |
| guitarzan | hemna: yeah, that's what I was going for | 15:41 |
| guitarzan | it's the same thing really | 15:41 |
| jgriffith | guitarzan: +1 | 15:42 |
| *** pots1 has joined #openstack-cinder | 15:42 | |
| guitarzan | that does expose a ton of issues with "what should initialize_connection" do | 15:43 |
| hemna | yah it is. | 15:43 |
| hemna | heh, we tried to iron that out in fort collins a bit | 15:43 |
| hemna | as well as create_export fun | 15:43 |
| guitarzan | yeah, your herculean multiple attach is another that's doing a lot of work on the status quo :) | 15:44 |
| hemna | guitarzan, I think we need to hook up with the nova folks in Tokyo and iron out the multi-attach issues. | 15:44 |
| *** pots has quit IRC | 15:44 | |
| hemna | and see if we can get it to finally land. | 15:44 |
| guitarzan | multiple attach / initiatlize_connection/ create export/ live migrate all get really conflated | 15:45 |
| *** jordanP has quit IRC | 15:45 | |
| hemna | yah, but I think we've started fixing some of that in L. | 15:47 |
| hemna | lots of drivers, including mine, were operating under incorrect assumptions at initialize_connection time. | 15:47 |
| hemna | that cause live migration problems :( | 15:47 |
| *** pots has joined #openstack-cinder | 15:47 | |
| guitarzan | hemna: yeah, those assumptions need to be made a lot more clear | 15:48 |
| *** pots1 has quit IRC | 15:49 | |
| hemna | jgriffith's writeup was a great start. I think it'd be nice to encapsulate that into a driver developer's howto on the wiki. | 15:50 |
| jgriffith | hemna: wiki's suck | 15:50 |
| jgriffith | hemna: dev docs in treee maybe? | 15:50 |
| hemna | but not having any howto sucks more ? | 15:50 |
| jgriffith | hemna: IMO we really should be using those | 15:51 |
| hemna | yah, no disagreement there. | 15:51 |
| *** krtaylor has joined #openstack-cinder | 15:51 | |
| guitarzan | anti-wiki? blasphemer | 15:51 |
| hemna | just having something somewhere that goes over some of the need to know stuff and gotchas | 15:51 |
| jgriffith | hemna: the trick is will anybody update it? | 15:51 |
| jgriffith | guitarzan: proudly!! | 15:51 |
| e0ne | jgriffith, hemna: implement it in a fake driver? | 15:51 |
| guitarzan | considering how much pain it is getting any patch through, I'd prefer a wiki in our case :) | 15:52 |
| *** ByPasS has quit IRC | 15:52 | |
| jgriffith | guitarzan: those and etherpads are the most abused and incorrectly utilized tools I know of | 15:52 |
| hemna | e0ne, well, I think that's kinda what the 'reference driver' is supposed to do | 15:52 |
| *** tsekiyama has joined #openstack-cinder | 15:52 | |
| jgriffith | guitarzan: it is painful... ideally we'd fix that :) | 15:52 |
| *** pots1 has joined #openstack-cinder | 15:52 | |
| hemna | e0ne, but the code doesn't really explain why you do things a certain way, or why you don't. | 15:52 |
| e0ne | hemna: agree | 15:52 |
| jgriffith | e0ne: well it is implemented, the problem is our shit's impossible to figure out now by code inspection | 15:53 |
| *** merooney has quit IRC | 15:53 | |
| hemna | e0ne, and it doesn't cover some of the FC related driver things. initiator_target_map, fczm_utils decorators, etc. | 15:53 |
| jgriffith | e0ne: and throw in the mess with 12 subclasses for the base driver and... well; GOOD LUCK!!! | 15:53 |
| hemna | :) | 15:53 |
| *** merooney has joined #openstack-cinder | 15:53 | |
| e0ne | Ж) | 15:53 |
| e0ne | :) | 15:53 |
| hemna | and the VDs | 15:53 |
| jgriffith | hemna: sighhhhh | 15:53 |
| hemna | :P | 15:54 |
| *** jistr has quit IRC | 15:54 | |
| *** ByPasS has joined #openstack-cinder | 15:54 | |
| jgriffith | I'm hoping to propose an alternative (again) in Tokyo | 15:54 |
| *** pots has quit IRC | 15:54 | |
| jgriffith | one using a sane impl of ABC, and the other using Zope interfaces | 15:54 |
| hemna | jgriffith, +1 | 15:54 |
| *** devlaps has joined #openstack-cinder | 15:54 | |
| e0ne | hemna: will we have shared session with ironic again? | 15:54 |
| hemna | I like what the ABC stuff we have provides, it's just a mess. | 15:54 |
| jgriffith | but I'm sure to be shot down *again* | 15:54 |
| smcginnis | Ooh, zope. :) | 15:55 |
| jgriffith | smcginnis: :) | 15:55 |
| jgriffith | smcginnis: It's a pretty clean way to do it so far it seems | 15:55 |
| *** garthb has quit IRC | 15:55 | |
| smcginnis | jgriffith: I had briefly looked at it a ways back and forgot about it. | 15:55 |
| jgriffith | smcginnis: at least it's readable :) | 15:55 |
| smcginnis | jgriffith: It does look cleaner. | 15:56 |
| smcginnis | jgriffith: Yesm and more readable! | 15:56 |
| jgriffith | smcginnis: anyway, I hope to have a couple of proposals | 15:56 |
| smcginnis | +1 | 15:56 |
| jgriffith | and I can take the abuse again | 15:56 |
| jgriffith | I'm used to it :) | 15:56 |
| hemna | :) | 15:56 |
| hemna | jgriffith, you already have my +1 on it :) | 15:56 |
| jgriffith | hemna: I've kinda realized the way it rolls lately.... | 15:57 |
| jgriffith | I propose something... it gets shot down | 15:57 |
| hemna | just don't couple that with pulling drivers from tree, and you'll be fine :) | 15:57 |
| jgriffith | somebody else proposes the same thing a few months later... they're a hero | 15:57 |
| jgriffith | hemna: LOL | 15:57 |
| jgriffith | hemna: I've given up on that one | 15:57 |
| smcginnis | jgriffith: I've been wanting to spend a little more time on the inheritance thing too. If you want help or someone to bounce ideas of, would love to help out. | 15:57 |
| hemna | pulling drivers from tree is the new 'standardize capabilities' | 15:57 |
| guitarzan | aww | 15:57 |
| jgriffith | hemna: although... I was thinking.. you're half way there already :) | 15:57 |
| jgriffith | hemna: having external libs is 1/2 way there | 15:58 |
| jgriffith | guitarzan: oh be quiet! | 15:58 |
| jgriffith | :) | 15:58 |
| hemna | yah there have been benefits for us for doing that | 15:58 |
| jgriffith | hemna: for sure! | 15:58 |
| jgriffith | it's a mixed bag | 15:58 |
| hemna | re: fixing issues in our lib post OS release | 15:58 |
| jgriffith | and it might be the best answer | 15:58 |
| jgriffith | hemna: I mean the model of libs and driver in tree might be the best conpromise/answer | 15:58 |
| *** timcl has joined #openstack-cinder | 15:58 | |
| *** timcl1 has quit IRC | 15:59 | |
| guitarzan | and all the drivers turn into driverlib.do_something() | 15:59 |
| hemna | yah | 15:59 |
| hemna | if you go to the extreme, then the drivers are nothing but shims to vendor client lib. | 15:59 |
| DuncanT | guitarzan: I'll be anti that - I already hate the drivers that do that | 15:59 |
| hemna | not that I'm advocating that | 15:59 |
| flip214 | meeting! | 15:59 |
| guitarzan | DuncanT: I don't like the idea either, I'm just tossing it out there :) | 15:59 |
| guitarzan | that would defeat the point of a driver imo | 16:00 |
| *** zhangjn has joined #openstack-cinder | 16:00 | |
| *** haomaiwa_ has quit IRC | 16:01 | |
| *** 16WAAAFWE has joined #openstack-cinder | 16:01 | |
| *** DericHorn-HP has joined #openstack-cinder | 16:01 | |
| *** p0rtal has joined #openstack-cinder | 16:04 | |
| *** pots1 has quit IRC | 16:04 | |
| *** EinstCrazy has joined #openstack-cinder | 16:04 | |
| *** yuriy_n17 has quit IRC | 16:05 | |
| *** p0rtal has quit IRC | 16:05 | |
| *** EinstCrazy has quit IRC | 16:06 | |
| *** p0rtal has joined #openstack-cinder | 16:06 | |
| *** pots1 has joined #openstack-cinder | 16:06 | |
| *** rajinir has joined #openstack-cinder | 16:08 | |
| *** EinstCrazy has joined #openstack-cinder | 16:08 | |
| *** EinstCrazy has quit IRC | 16:08 | |
| *** EinstCrazy has joined #openstack-cinder | 16:09 | |
| *** merooney has quit IRC | 16:09 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: Fix issue of volume after host-assisted migration https://review.openstack.org/217522 | 16:11 |
| openstackgerrit | Peter Wang proposed openstack/cinder: Fix issue of volume after host-assisted migration https://review.openstack.org/217522 | 16:13 |
| *** leeantho has joined #openstack-cinder | 16:15 | |
| *** pots1 has quit IRC | 16:17 | |
| *** pots1 has joined #openstack-cinder | 16:19 | |
| *** gman-tx has joined #openstack-cinder | 16:19 | |
| *** annegentle has quit IRC | 16:20 | |
| *** annegentle has joined #openstack-cinder | 16:21 | |
| *** EinstCrazy has quit IRC | 16:22 | |
| *** geguileo has joined #openstack-cinder | 16:24 | |
| *** sasukeh has quit IRC | 16:26 | |
| *** pots1 has quit IRC | 16:27 | |
| *** garthb has joined #openstack-cinder | 16:28 | |
| *** pots1 has joined #openstack-cinder | 16:29 | |
| *** rick-chen has quit IRC | 16:30 | |
| *** porrua has quit IRC | 16:30 | |
| *** mudassirlatif has joined #openstack-cinder | 16:31 | |
| *** kbader has joined #openstack-cinder | 16:32 | |
| *** breitz has joined #openstack-cinder | 16:33 | |
| *** sasukeh has joined #openstack-cinder | 16:34 | |
| *** ebalduf has quit IRC | 16:35 | |
| *** ebalduf has joined #openstack-cinder | 16:35 | |
| *** ronis has joined #openstack-cinder | 16:38 | |
| *** Apoorva has joined #openstack-cinder | 16:39 | |
| *** vgridnev has joined #openstack-cinder | 16:41 | |
| *** dannywilson has joined #openstack-cinder | 16:42 | |
| *** earlephilhower has joined #openstack-cinder | 16:43 | |
| *** sasukeh has quit IRC | 16:45 | |
| *** DericHorn-HP has quit IRC | 16:45 | |
| *** porrua has joined #openstack-cinder | 16:48 | |
| *** dannywilson has quit IRC | 16:49 | |
| *** sasukeh has joined #openstack-cinder | 16:52 | |
| *** merooney has joined #openstack-cinder | 16:52 | |
| *** pots1 has quit IRC | 16:53 | |
| *** bill_az_ has joined #openstack-cinder | 16:54 | |
| *** sasukeh has quit IRC | 16:55 | |
| *** diogogmt_ has joined #openstack-cinder | 16:56 | |
| *** diogogmt has quit IRC | 16:56 | |
| *** diogogmt_ is now known as diogogmt | 16:56 | |
| *** p0rtal has quit IRC | 16:57 | |
| *** kbader has quit IRC | 16:57 | |
| *** _cjones_ has joined #openstack-cinder | 16:58 | |
| *** annegentle has quit IRC | 16:59 | |
| *** kbader has joined #openstack-cinder | 16:59 | |
| *** annegentle has joined #openstack-cinder | 17:00 | |
| *** 16WAAAFWE has quit IRC | 17:01 | |
| thingee | ok people had blockers for me? | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| thingee | DuncanT: dulek ^ | 17:01 |
| dulek | thingee: https://review.openstack.org/#/c/223602/ | 17:02 |
| *** sasukeh has joined #openstack-cinder | 17:02 | |
| dulek | thingee: Not sure if this is really critical, but when someone issues cinder service-list before a service got its first update, then we have 500 from the API. | 17:03 |
| dulek | I mean before a service send its first update. | 17:03 |
| thingee | dulek: I can just push the fix through | 17:04 |
| *** jgregor has quit IRC | 17:04 | |
| thingee | dulek: alex did leave a comment | 17:04 |
| *** kbader has quit IRC | 17:04 | |
| dulek | Yeah, I see, I can address that if you think its necessary or do another commit with that fix. | 17:04 |
| *** kbader has joined #openstack-cinder | 17:05 | |
| thingee | dulek: do another push and we can merge it today. | 17:05 |
| thingee | dulek: mind pinging me on it? | 17:05 |
| dulek | I will then. Thanks! | 17:05 |
| dulek | thingee: Sure. | 17:05 |
| *** kbader has quit IRC | 17:06 | |
| *** sasukeh has quit IRC | 17:07 | |
| *** mriedem has quit IRC | 17:09 | |
| *** haomaiwang has quit IRC | 17:12 | |
| *** sasukeh has joined #openstack-cinder | 17:12 | |
| *** edtubill has quit IRC | 17:13 | |
| *** pschaef has quit IRC | 17:13 | |
| *** anshul has joined #openstack-cinder | 17:15 | |
| *** e0ne has quit IRC | 17:19 | |
| *** kbader has joined #openstack-cinder | 17:19 | |
| *** dannywilson has joined #openstack-cinder | 17:22 | |
| *** sgotliv_ has quit IRC | 17:23 | |
| *** mriedem has joined #openstack-cinder | 17:24 | |
| *** kbader has quit IRC | 17:27 | |
| *** sasukeh has quit IRC | 17:27 | |
| *** sasukeh has joined #openstack-cinder | 17:30 | |
| hemna | jgriffith, ping | 17:31 |
| *** diogogmt has quit IRC | 17:34 | |
| *** sasukeh has quit IRC | 17:35 | |
| *** p0rtal has joined #openstack-cinder | 17:35 | |
| jgriffith | hemna: hey | 17:36 |
| hemna | http://54.164.167.86/solidfire-ci-logs//refs-changes-26-214826-8 | 17:37 |
| hemna | jgriffith, from this patch https://review.openstack.org/#/c/214826/ | 17:37 |
| hemna | just wanted to make sure solidfire was cool w/ that patch before +A'ing it | 17:37 |
| jgriffith | hemna: hmm... wtf? | 17:37 |
| jgriffith | hemna: so there was a problem with a requirements patch in Keystone last night | 17:39 |
| jgriffith | hemna: but that wasn't breaking the posting of logs | 17:39 |
| jgriffith | hemna: lemme try a recheck | 17:39 |
| hemna | ok coolio. | 17:39 |
| jgriffith | thanks for pointing it out | 17:39 |
| hemna | np | 17:40 |
| hemna | I thought you might want to know :P | 17:40 |
| jgriffith | hemna: indeed :) | 17:40 |
| *** p0rtal_ has joined #openstack-cinder | 17:40 | |
| jgriffith | hemna: ok, well it's running... let's see what happens :) | 17:41 |
| hemna | sweet | 17:41 |
| *** pots1 has joined #openstack-cinder | 17:43 | |
| *** p0rtal has quit IRC | 17:43 | |
| *** p0rtal_ has quit IRC | 17:44 | |
| *** tpsilva has quit IRC | 17:49 | |
| *** ociuhandu has quit IRC | 17:52 | |
| *** charlesr has quit IRC | 17:52 | |
| *** crose has quit IRC | 17:52 | |
| *** crose has joined #openstack-cinder | 17:52 | |
| *** tpsilva has joined #openstack-cinder | 17:52 | |
| *** ociuhandu has joined #openstack-cinder | 17:53 | |
| *** diablo_rojo has quit IRC | 17:54 | |
| *** diogogmt has joined #openstack-cinder | 17:55 | |
| *** jgregor has joined #openstack-cinder | 17:56 | |
| *** bluex-pl has quit IRC | 17:56 | |
| *** ociuhandu has quit IRC | 17:57 | |
| *** Zhongjun has quit IRC | 17:57 | |
| *** e0ne has joined #openstack-cinder | 17:58 | |
| smcginnis | hemna, DuncanT, e0ne: Glad to see your names out there too. I'll be happy no matter what the result. :) | 17:59 |
| hemna | we have a good group of folks. Cinder will be in good hands either way | 17:59 |
| smcginnis | hemna: +1 | 17:59 |
| *** DericHorn-HP has joined #openstack-cinder | 17:59 | |
| smcginnis | And a good bench if we start rotating. | 18:00 |
| *** setmason has joined #openstack-cinder | 18:00 | |
| *** setmason has left #openstack-cinder | 18:01 | |
| *** setmason has joined #openstack-cinder | 18:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/224229 | 18:01 |
| hemna | yup. I think it's a good idea to have rotation | 18:02 |
| *** p0rtal has joined #openstack-cinder | 18:02 | |
| smcginnis | Doesn't hurt to at least have more folks that know how things work. | 18:02 |
| smcginnis | I've been thinking we should propose a Core Bootcamp or something like and advanced course for what they do for the pre-Summit training. | 18:02 |
| hemna | that would be pretty cool | 18:03 |
| *** diablo_rojo has joined #openstack-cinder | 18:03 | |
| hemna | even a cinder driver bootcamp might be cool. | 18:03 |
| smcginnis | hemna: Yeah, actually that would be awesome. | 18:03 |
| *** lpetrut has quit IRC | 18:03 | |
| hemna | mriedem, do you guys have an etherpad for Nova Mitaka summit session topics? | 18:04 |
| mriedem | hemna: it's a google form this time around | 18:04 |
| mriedem | sec | 18:04 |
| mriedem | hemna: https://docs.google.com/spreadsheets/d/1MZVwxv8t6sM15Kct5i7yOS-8-NCvqsW742s5Y_LDyjg/edit#gid=816860825 | 18:05 |
| mriedem | new topics are posted via form https://docs.google.com/forms/d/1Ozh18Uo2svZMFff1YttjcBdVqMQ3gxGl_Sjsvo-tWh4/viewform?c=0&w=1 | 18:05 |
| hemna | mriedem, thanks | 18:05 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Check for None on service's updated_at https://review.openstack.org/223602 | 18:05 |
| e0ne | smcginnis, hemna: thanks, +1 | 18:06 |
| *** annegentle has quit IRC | 18:12 | |
| *** annegentle has joined #openstack-cinder | 18:12 | |
| dulek | smcginnis: Ha, I've got a new patchset uploaded just a minute after you gave +2. It's a shame I've wasted it! ;) | 18:14 |
| smcginnis | dulek: Hah, no problem, I'll take a look again. ;) | 18:15 |
| diablo_rojo | thingee: The bug you emailed me about is actually covered by my genconfig patch thats out for review. I will add that to the commit message so that its more obvious. Is it still possible to get that in before the cutoff? | 18:15 |
| hemna | anyone restack today? | 18:15 |
| hemna | I'm getting an odd keystone failure | 18:15 |
| smcginnis | hemna: Had an issue yesterday, but stacked today fine. | 18:16 |
| *** diogogmt has quit IRC | 18:16 | |
| smcginnis | hemna: What's the error. | 18:16 |
| *** openstackgerrit has quit IRC | 18:16 | |
| hemna | http://paste.openstack.org/show/465026/ | 18:16 |
| hemna | I JUST updated devstack and all my /opt/stack/* repos, then that happend | 18:16 |
| *** openstackgerrit has joined #openstack-cinder | 18:17 | |
| smcginnis | hemna: jgriffith was burning the midnight oil last night fixing a pbr issue. | 18:17 |
| *** jdandrea has joined #openstack-cinder | 18:17 | |
| smcginnis | hemna: Looks like this might be it. | 18:17 |
| smcginnis | Try blowing away your venv. | 18:17 |
| e0ne | hemna: did you update /opt/requirements? | 18:17 |
| *** diogogmt has joined #openstack-cinder | 18:17 | |
| *** e0ne has quit IRC | 18:18 | |
| jdandrea | Is it typical for the volume (not volumev2) service be set to use v2 (not v1) endpoints in keystone? | 18:18 |
| hemna | I have a script that I run and it updates every valid git repo in /opt/stack/ | 18:18 |
| smcginnis | hemna: This ones a little different because it's outside of the projects. | 18:19 |
| smcginnis | If I remeber right 1.6 or something of pbr wasn't parsing python versions correctly. | 18:19 |
| hemna | http://paste.openstack.org/show/465027/ | 18:19 |
| *** pots has joined #openstack-cinder | 18:19 | |
| *** pots1 has quit IRC | 18:20 | |
| hemna | smcginnis, ok, updated pbr to 1.8.0 | 18:20 |
| hemna | was at 1.5.0 | 18:20 |
| smcginnis | hemna: Oh, also make sure to get the lastest devstack too. I think that's what did it for me - updating devstack and redoing my venv. | 18:20 |
| *** e0ne has joined #openstack-cinder | 18:20 | |
| *** yrabl has joined #openstack-cinder | 18:21 | |
| * smcginnis crosses fingers | 18:21 | |
| jdandrea | Answered my own question: http://docs.openstack.org/kilo/install-guide/install/apt/content/cinder-install-controller-node.html (step 4) | 18:21 |
| e0ne | even more: /opt/stack/requirements and it's venv | 18:21 |
| smcginnis | jdandrea: ;) Glad you got it. | 18:22 |
| hemna | stacking..... | 18:22 |
| smcginnis | v1 will be going away in the very near future. | 18:22 |
| hemna | pewp | 18:22 |
| jdandrea | smcginnis: Thx. Now I'm wrestling with a python-cinderclient issue regarding v2 and being told there are multiple matching endpoints. | 18:22 |
| smcginnis | hemna: Same error? | 18:23 |
| hemna | I'll try nuking /opt/stack/requirements/.venv | 18:23 |
| hemna | yah same here | 18:23 |
| *** changbl has quit IRC | 18:23 | |
| thingee | diablo_rojo: possibly, but yes please edit the commit message, thanks | 18:23 |
| smcginnis | jdandrea: Hmm, I know we had some ripple effects from trying to do aut-discovery. Maybe thingee is around and knows more. | 18:23 |
| * smcginnis didn't completely follow | 18:23 | |
| jdandrea | smcginnis: Gathering more intel on this end. | 18:24 |
| diablo_rojo | thingee: I will add that now and get a patch pushed up immediately. Its ready to be reviewed and it would be good to get it in if possible. | 18:24 |
| hemna | crap | 18:25 |
| hemna | same issue | 18:25 |
| hemna | :( | 18:25 |
| *** yrabl has quit IRC | 18:26 | |
| guitarzan | jdandrea: https://review.openstack.org/#/c/214802/ ? | 18:26 |
| *** yrabl has joined #openstack-cinder | 18:26 | |
| smcginnis | hemna: Try rm -fr / ? :] | 18:26 |
| smcginnis | Dang, I get nervous even typing that in an irc window. | 18:27 |
| hemna | ooh yah, haven't tried that one yet | 18:27 |
| e0ne | smcginnis: you forgot fdisk call | 18:27 |
| hemna | lol | 18:27 |
| thingee | smcginnis: they were reverted | 18:27 |
| * hemna fires up a new vagrant.... | 18:27 | |
| smcginnis | thingee: I seem to remember some after affects from that. Maybe not. | 18:27 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Dynamically create cinder.conf.sample https://review.openstack.org/219700 | 18:28 |
| *** merooney has quit IRC | 18:29 | |
| *** david-lyle has quit IRC | 18:30 | |
| *** boris-42 has quit IRC | 18:30 | |
| asselin | smcginnis, you also forgot the sudo | 18:34 |
| e0ne | hemna: please, let me know if you'll get any issue with c-all patch. I'll be available ~2 hours more today | 18:34 |
| *** pots3 has joined #openstack-cinder | 18:34 | |
| smcginnis | asselin: Pfft, I just run as root. :P | 18:34 |
| smcginnis | hah | 18:34 |
| *** pots has quit IRC | 18:34 | |
| hodos|2 | hi, could someone explain how to turn on/off multipath for iscsi? | 18:36 |
| *** vgridnev has quit IRC | 18:38 | |
| hemna | hodos|2, for Nova, you can remove the iscsi_use_multipath = True, or set it to False in /etc/nova/nova.conf | 18:42 |
| hodos|2 | hemna: what about cinder ? | 18:43 |
| hemna | use_multipath_for_image_xfer = False | 18:45 |
| hemna | in /etc/cinder/cinder.conf | 18:45 |
| e0ne | hodos|2: enforce_multipath_for_image_xfer conf param | 18:45 |
| hemna | e0ne, that's for only enforcing that mulipath daemon must be running to cause an exception | 18:46 |
| *** aix has quit IRC | 18:46 | |
| *** dannywilson has quit IRC | 18:46 | |
| hodos|2 | hemna: thanks | 18:46 |
| e0ne | hemna: thanks. I need go deeper in multipath topic | 18:47 |
| smcginnis | Anyone with drivers that use the volume id for your array identification: | 18:48 |
| smcginnis | Have you tried to do a retype with migrate lately? | 18:48 |
| smcginnis | Seeing something different. This had been working. | 18:49 |
| hemna | smcginnis, I recently fixed an issue with our driver in a migration into our array from an external | 18:49 |
| hemna | re: _name_id garbage | 18:49 |
| smcginnis | Now it creates the new vol, migrates, updates the new volume with the old id, then tries to delete the temp id leaving the original volume. | 18:49 |
| smcginnis | hemna: Looks like we're doing something similar other than provider_location. | 18:50 |
| hemna | smcginnis, I don't think our driver even uses provider_location at all | 18:51 |
| hemna | fwiw | 18:51 |
| hemna | also, smcginnis https://review.openstack.org/#/c/217522/ | 18:51 |
| hemna | that's sitting out there. haven't had time to play with it yet though | 18:51 |
| *** pots has joined #openstack-cinder | 18:52 | |
| *** pots3 has quit IRC | 18:52 | |
| smcginnis | hemna: Thanks. Not sure if that will do it, but maybe I'll give it a shot and see. | 18:53 |
| hemna | it's on my list of stuff to play with...I just have a big backlog today | 18:53 |
| smcginnis | Apparently I now do too. :) | 18:54 |
| hemna | :) | 18:54 |
| *** martyturner has quit IRC | 18:55 | |
| *** merooney has joined #openstack-cinder | 18:56 | |
| *** timcl1 has joined #openstack-cinder | 18:57 | |
| *** Yogi1 has joined #openstack-cinder | 18:58 | |
| *** Yogi11 has quit IRC | 18:58 | |
| *** timcl has quit IRC | 19:00 | |
| jdandrea | smcginnis: Here's what I understand to be happening (as reported by a colleague). I asked them for steps to reproduce so that I'm absolutely sure. http://paste.openstack.org/show/465066/ | 19:00 |
| *** vgridnev has joined #openstack-cinder | 19:01 | |
| jdandrea | smcginnis: I should also mention 1.0.8 is being run from within a VM but otherwise the same rc (environment) file. | 19:01 |
| *** yrabl has quit IRC | 19:02 | |
| smcginnis | e0ne: Any idea if that's related to https://github.com/openstack/python-cinderclient/commit/04caf88307235afd6cb34cd369cc4ef983665457 | 19:03 |
| e0ne | smcginnis: what is the issue? | 19:04 |
| *** esker has joined #openstack-cinder | 19:04 | |
| smcginnis | jdandrea: Mind summarizing? Here's the results: https://github.com/openstack/python-cinderclient/commit/04caf88307235afd6cb34cd369cc4ef983665457 | 19:05 |
| smcginnis | Oops, not that one. | 19:05 |
| smcginnis | http://paste.openstack.org/show/465066/ | 19:05 |
| hemna | ok new vagrant, new stack problems. *sigh* | 19:05 |
| e0ne | smcginnis: I'm sorry. I still don't understand what went wrong | 19:07 |
| smcginnis | e0ne: Sorry, jdandrea must be busy now. Apparently they are getting an error from keystone of multiple matching endpoints. | 19:08 |
| jdandrea | smcginnis: I'm here. | 19:08 |
| smcginnis | e0ne: Lemme see if I can capture what was said exactly. | 19:08 |
| smcginnis | jdandrea: Oh good. Can you summarize for e0ne what you're seeing? | 19:08 |
| e0ne | smcginnis, jdandrea : stacktrace will ba helpful in any case | 19:08 |
| jdandrea | smcginnis, e0ne sure. I've asked for exact steps to reproduce (someone else here is seeing it). | 19:09 |
| jdandrea | e0ne: Per http://docs.openstack.org/kilo/install-guide/install/apt/content/cinder-install-controller-node.html the volume and volumev2 endpoints both point to the v2 API endpoint. | 19:09 |
| e0ne | hm... | 19:10 |
| hemna | smcginnis, http://paste.openstack.org/show/465091/ | 19:10 |
| jdandrea | Someone was running python-cinderclient 1.0.8 in a VM and noticed they couldn't specify API version 2 (using "2") whether via command line option or exported env var. | 19:10 |
| dulek | thingee: https://review.openstack.org/#/c/223602/ just passed Jenkins. | 19:11 |
| smcginnis | hemna: Doh! | 19:11 |
| jdandrea | e0ne: The same one then tried python-cinderclient 1.1.1 from the controller node. The command line option still didn't fly (told there's more than one matching v2 endpoint), but the exported environment variable worked. | 19:11 |
| thingee | dulek: ack | 19:11 |
| smcginnis | hemna: Looks vaguely familiar like I've run into something similar, but I don't think I've seen that exact error. | 19:11 |
| jdandrea | e0ne: There was no stack trace, just an error. | 19:11 |
| e0ne | jdandrea: I never tested case when volume and volumev2 endpoints are both for api v2 | 19:12 |
| jdandrea | e0ne: (which is what I'm trying to get a hold of, the exact response they got). | 19:12 |
| thingee | dulek: +2/A | 19:12 |
| hemna | arg | 19:12 |
| jdandrea | e0ne: *nods* exactly, and we were unsure about the docs mentioning this note: "The Block Storage service requires both the volume and volumev2 services. However, both services use the same API endpoint that references the Block Storage version 2 API." | 19:12 |
| jdandrea | They were wondering if volume should reference v1 (but I know v1 is going away too, so ...). | 19:13 |
| e0ne | jdandrea: i'm confused | 19:13 |
| guitarzan | jdandrea: it seems like that error message is just telling the truth if there are two v2 endpoints in your catalog | 19:14 |
| e0ne | jdandrea: could you try to setup 'volume' endpoint for api v1? | 19:14 |
| jdandrea | e0ne: I could but the admin insists that it should be setup with both for v2 because of what's in the docs: http://docs.openstack.org/kilo/install-guide/install/apt/content/cinder-install-controller-node.html (see step 4). | 19:14 |
| e0ne | I'm not sure where the bug is: cinder, keystone, docs | 19:14 |
| *** tpsilva has quit IRC | 19:14 | |
| smcginnis | hemna: Based on the error it's not passing a name to the --user arg. I just stacked about an hour ago and I did notice it creating new roles. :/ | 19:14 |
| guitarzan | a long time ago there was complaints about calling the v2 endpoint volume2 and some people? wanted it to just be called volume | 19:15 |
| e0ne | jdandrea: about 4 years ago, my lesson #1 was: don't believe in openstack docs | 19:15 |
| smcginnis | e0ne: ;) | 19:15 |
| hemna | smcginnis, yah It looks like it doesn't have the admin_user for whatever reason :( | 19:15 |
| e0ne | jdandrea: I need to check in on my env | 19:15 |
| jdandrea | e0ne: It seems that, when both are v2 (as instructed in the installation guide), the 1.1.1 client works if OS_VOLUME_API_VERSION is exported as 2, but not if --os-volume-api-version 2 is used. | 19:15 |
| guitarzan | jdandrea: *that* certainly sounds like a bug | 19:16 |
| jgriffith | hemna: smcginnis that's an "old" problem, not sure why you're seeing it now | 19:16 |
| jdandrea | e0ne: As for not believing in docs, try telling that to folks who depend upon them as canonical. But I get your point. ;) | 19:16 |
| jgriffith | hemna: smcginnis I don't remember the bug exactly, but my CI hit it one night before node-pool units were updated | 19:16 |
| jgriffith | hemna: smcginnis and a fix was pushed overnight while I slept :) | 19:16 |
| jgriffith | hemna: any chance your bits are stale? | 19:17 |
| jdandrea | jgriffith: This is using cinder-kilo, yea. | 19:17 |
| e0ne | jdandrea: it was kind of a joke | 19:17 |
| *** jungleboyj has quit IRC | 19:17 | |
| smcginnis | jgriffith, hemna: I don't see anything in the recent devstack commits that looks related. | 19:17 |
| hemna | I just updated my /opt/stack/ repos | 19:17 |
| jdandrea | e0ne: Yup, we use the same joke. ;D | 19:17 |
| hemna | I'll nuke em again and start over | 19:17 |
| e0ne | :) | 19:17 |
| smcginnis | I love devstack. | 19:17 |
| e0ne | hemna: did you try PIP_UPGRADE=True? | 19:17 |
| jgriffith | hemna: so the problem was in a lib dependency | 19:17 |
| jgriffith | hemna: It may or may not be what you're seeing, but I suspect it is the same thing | 19:18 |
| hemna | e0ne, env var prior to running stack.sh ? | 19:18 |
| jgriffith | e0ne: +1 | 19:18 |
| hemna | jgriffith, ok | 19:18 |
| e0ne | hemna: in local rc | 19:18 |
| hemna | e0ne, ok added to my local.conf | 19:18 |
| smcginnis | jgriffith: You use volume['id'] for array identity, right? Have you done a migrate lately? | 19:19 |
| smcginnis | jgriffith: Or are you tracking by provider_id now? | 19:19 |
| e0ne | PIP_UPGRADE=True (copy-pasted version) | 19:19 |
| *** DericHorn-HP has quit IRC | 19:19 | |
| jgriffith | smcginnis: I still use volume['id'] | 19:19 |
| openstackgerrit | Michael Price proposed openstack/cinder: Fix netapp_enable_multiattach default for E-Series https://review.openstack.org/223698 | 19:19 |
| jgriffith | smcginnis: no, I haven't tried it lately | 19:19 |
| jgriffith | smcginnis: do I need to ? :) | 19:19 |
| smcginnis | jgriffith: I'm seeing upgrade_migrated_volume called to be able to rename the temp name to the original name, but then for the delete of the original volume it is sending in the temp id. | 19:19 |
| jgriffith | good lord! | 19:20 |
| smcginnis | jgriffith: So of course I don't find it because it never existed with that id on the original backend. | 19:20 |
| jgriffith | sigh | 19:20 |
| jgriffith | train-wreck | 19:20 |
| smcginnis | jgriffith: So it doesn't like it should only affect me, so just curious if anyone else has seen that. | 19:20 |
| jdandrea | smcginnis: Ok, so I got an account over there and tried python-cinderclient 1.1.1 and I do NOT have that issue. I will still ask them for steps to reproduce. As for client 1.0.8, is it more a case of "don't use that version?" | 19:20 |
| hodos|2 | jgriffith: hi, for some reason i get iscsi devices sometimes mounted to /dev/mapper/... but it's not using multipath, do you have ideahow can this happen ? | 19:20 |
| jgriffith | smcginnis: indeed, should impact quite a few people. I'll look into it on my side. Thx for the heads up | 19:21 |
| smcginnis | jdandrea: Oh good. Well, maybe not for you. But if it's just a specific environment it's better than total breakage. :) | 19:21 |
| *** p0rtal_ has joined #openstack-cinder | 19:21 | |
| smcginnis | jgriffith: No problem. Would be good to have someone else confirm the issue. | 19:21 |
| hemna | hodos|2, if the multipath daemon is running, it will create /dev/mapper/<WWN> devices automatically | 19:21 |
| smcginnis | I'm looking through Vincent's changes again, but I don't think they are the cause. But that area has been touched recently. | 19:22 |
| hodos|2 | hemna: i thought that it shouldn't do this, if connection properties provide only single iqn, portal, etc | 19:22 |
| hemna | hodos|2, the multipath daemon itself is independent of all of that. if it's running, it will create multipath devices regardless of Nova/Cinder | 19:23 |
| hemna | depending on the multipath.conf settings (blacklist/whitelist) of course | 19:23 |
| *** lpetrut has joined #openstack-cinder | 19:23 | |
| hemna | hodos|2, id you don't want multipath devices created, disable the daemon :) | 19:24 |
| *** p0rtal has quit IRC | 19:24 | |
| hemna | or blacklist the backend in multipath.conf | 19:24 |
| hodos|2 | hemna: when i type 'multipath' on console it says that it's not installed | 19:25 |
| *** aix has joined #openstack-cinder | 19:25 | |
| *** edtubill has joined #openstack-cinder | 19:25 | |
| hodos|2 | hemna: maybe tempest installs it in venv ? | 19:25 |
| hemna | nah | 19:25 |
| hemna | mutlipath -ll ? | 19:25 |
| e0ne | jdandrea: I'll try to reproduce this issu on my env | 19:25 |
| hemna | hodos|2, also I believe some udev rules can create /dev/mapper/ entires. ymmv | 19:26 |
| hemna | entries | 19:26 |
| hemna | ok I'm out for a bit....lunch | 19:26 |
| *** hemna is now known as hemnafk | 19:26 | |
| *** Apoorva has quit IRC | 19:27 | |
| *** bill_az_ has quit IRC | 19:31 | |
| diablo_rojo | jgriffith: When you get a sec you should take a look at the genconfig patch and see if I get your seal of approval. I think it's ready to go! :) https://review.openstack.org/#/c/219700/ | 19:31 |
| e0ne | jdandrea: 'cinder list' works for me with the latest client from master | 19:32 |
| jgriffith | diablo_rojo: sure, I'll look at it now | 19:32 |
| *** jungleboyj has joined #openstack-cinder | 19:32 | |
| e0ne | jdandrea: 'volume' and 'volumev2' endpoints are configured to use api v2 | 19:32 |
| diablo_rojo | jgriffith: Thank you :) | 19:33 |
| *** yrabl has joined #openstack-cinder | 19:33 | |
| * e0ne bb in ~12 hours | 19:34 | |
| *** thangp_ has joined #openstack-cinder | 19:34 | |
| jdandrea | e0ne: Yep. They're using whatever client ubuntu has for kilo (which I *think* is 1.1.1). It works for me too. I asked for their OS_ prefixed environment variables and a capture of what they're seeing. | 19:34 |
| *** thangp has quit IRC | 19:34 | |
| jdandrea | e0ne: As for 1.0.8, which they'd also like to use, they're not sure what to do there. Neither method works for them. I've asked for the environment there as well. | 19:35 |
| jdandrea | (I encouraged 1.1.1.) | 19:35 |
| *** timcl1 has quit IRC | 19:35 | |
| *** bill_az has quit IRC | 19:35 | |
| openstackgerrit | Mudassir Latif proposed openstack/cinder: Allow replicated volumes to be recoverable https://review.openstack.org/219900 | 19:35 |
| *** e0ne has quit IRC | 19:35 | |
| *** ebalduf has quit IRC | 19:36 | |
| *** timcl has joined #openstack-cinder | 19:36 | |
| *** Apoorva has joined #openstack-cinder | 19:37 | |
| jgriffith | diablo_rojo: one complaint! | 19:38 |
| diablo_rojo | jgriffith: Yes? | 19:38 |
| jgriffith | diablo_rojo: why do your patches always ask me if I want to translate from Norwegian? | 19:38 |
| jgriffith | :) | 19:38 |
| *** thangp_ has quit IRC | 19:38 | |
| jgriffith | diablo_rojo: https://www.dropbox.com/s/v7xebngtpn5uue6/Screenshot%202015-09-16%2013.38.42.png?dl=0 | 19:38 |
| diablo_rojo | jgriffith: I totally don't have dropbox setup on this computer :) | 19:40 |
| jgriffith | diablo_rojo: haha.. no worries | 19:40 |
| jgriffith | diablo_rojo: you shouldn't need it by the way, should just be a url | 19:40 |
| jgriffith | diablo_rojo: it was kind of a joke | 19:40 |
| jgriffith | diablo_rojo: for some reason chrome always asks me if I want to translate from norweigan for your patches | 19:41 |
| diablo_rojo | jgriffith: Its funny though because I am actually half norwegian :) | 19:41 |
| jgriffith | diablo_rojo: yeah... that and the whole Minnesota thing | 19:42 |
| jgriffith | diablo_rojo: anyway.. nice work on that IMO | 19:42 |
| diablo_rojo | jgriffith: That too :) The american scandanavia :) | 19:42 |
| jgriffith | :) | 19:43 |
| diablo_rojo | jgriffith: Aww :) Thank you. | 19:43 |
| *** ebalduf has joined #openstack-cinder | 19:46 | |
| *** yrabl has quit IRC | 19:46 | |
| *** kbader has joined #openstack-cinder | 19:48 | |
| *** timcl has quit IRC | 19:48 | |
| *** lpetrut has quit IRC | 19:50 | |
| *** nikeshm has joined #openstack-cinder | 19:50 | |
| *** bill_az has joined #openstack-cinder | 19:51 | |
| *** thangp has joined #openstack-cinder | 20:01 | |
| *** openstackstatus has quit IRC | 20:02 | |
| *** openstackstatus has joined #openstack-cinder | 20:03 | |
| *** timcl has joined #openstack-cinder | 20:03 | |
| *** ChanServ sets mode: +v openstackstatus | 20:03 | |
| *** stevemar has quit IRC | 20:08 | |
| *** annasort has quit IRC | 20:08 | |
| *** stevemar has joined #openstack-cinder | 20:09 | |
| *** sgotliv_ has joined #openstack-cinder | 20:10 | |
| *** timcl1 has joined #openstack-cinder | 20:10 | |
| *** jmenbo has joined #openstack-cinder | 20:10 | |
| *** merooney has quit IRC | 20:11 | |
| *** timcl has quit IRC | 20:11 | |
| *** hemnafk is now known as hemna | 20:11 | |
| *** stevemar has quit IRC | 20:11 | |
| *** akerr has quit IRC | 20:13 | |
| openstackgerrit | Michael Price proposed openstack/cinder: NetApp: Fix volume extend with E-Series https://review.openstack.org/224285 | 20:15 |
| hemna | diablo_rojo, http://paste.openstack.org/show/465193/ | 20:19 |
| hemna | diablo_rojo, the generate_cinder_opts.py pukes if you aren't in the cinder root dir fwiw | 20:20 |
| hemna | diablo_rojo, only seems to run via tools/config/generate_sample.sh | 20:20 |
| hemna | diablo_rojo, if you cd into tools/config, then run it, it vomits | 20:20 |
| scottda | Is there a plan to deprecate xml support for the Cinder API? | 20:21 |
| hemna | scottda, heh | 20:21 |
| scottda | What's so funny? | 20:21 |
| hemna | I don't think there is a plan yet. | 20:21 |
| scottda | OK. | 20:21 |
| hemna | and I don't think we can just nuke it. but hey, microversions..... | 20:21 |
| scottda | yep. | 20:21 |
| smcginnis | diablo_rojo: Hmm, maybe check somehow that it is being called from tox -e genconfig? Pass in a special key? Or check that pwd is what's expected? | 20:22 |
| scottda | I'm just seeing it while porting microversions and wondering if there's a plan, that's all. | 20:22 |
| *** jmenbo has quit IRC | 20:22 | |
| *** jmenbo has joined #openstack-cinder | 20:23 | |
| *** p0rtal_ has quit IRC | 20:28 | |
| *** kbader1 has joined #openstack-cinder | 20:28 | |
| *** p0rtal has joined #openstack-cinder | 20:28 | |
| diablo_rojo | hemna: smcginnis: Okay I will address that and get a new patchset up soon. | 20:29 |
| diablo_rojo | thanks for the heads up and advice | 20:29 |
| *** kbader has quit IRC | 20:29 | |
| hemna | diablo_rojo, it could be as simple as detecting that it's not running from the cinder root and vomit with a good message ? | 20:29 |
| hemna | the script used to have the ability to set the basedir | 20:30 |
| diablo_rojo | hemna: Yeah I will add that check in since I have the basedir still being set in generate_sample.sh | 20:32 |
| diablo_rojo | hemna: Thanks :) | 20:32 |
| *** p0rtal has quit IRC | 20:32 | |
| *** annasort has joined #openstack-cinder | 20:34 | |
| *** dustins has quit IRC | 20:35 | |
| *** sgotliv_ has quit IRC | 20:36 | |
| *** timcl1 has quit IRC | 20:36 | |
| *** ndipanov has quit IRC | 20:37 | |
| *** hodos|2 has quit IRC | 20:37 | |
| diablo_rojo | hemna: How are you running it? When I run tox -e genconfig from in tools/config it works. | 20:38 |
| diablo_rojo | hemna: and when I run it inside the cinder dir inside the base dir it works too. | 20:38 |
| hemna | diablo_rojo, http://paste.openstack.org/show/465193/ | 20:39 |
| hemna | it's a shell script, so I tried to run it manually. | 20:39 |
| hemna | :) | 20:39 |
| diablo_rojo | hemna: Ha ha sorry I looked but didn't see :) So just for grins. If you run it in tox does it work? | 20:40 |
| diablo_rojo | I will address that problem though | 20:40 |
| hemna | I was able to run it tools/config/generate_sample.sh from the cinder root dir | 20:41 |
| hemna | it didn't puke | 20:41 |
| diablo_rojo | hemna: Hm. Okay thanks for the heads up :) | 20:42 |
| hemna | np | 20:44 |
| jgriffith | guitarzan: just for the record, I'm proposing that the setting of the quota values should be done ONLY by the quota-class-update API call | 20:46 |
| jgriffith | guitarzan: and that the conf file should not be needed to bootstrap the values | 20:46 |
| hemna | diablo_rojo, so running tox -egenconfig didn't puke :) | 20:46 |
| jgriffith | guitarzan: we should either just have a default set that admin can update/modify using the API | 20:47 |
| jgriffith | guitarzan: well.. no "either/or" that's just what we should have | 20:47 |
| jgriffith | guitarzan: my gripe is that the db migration relies on / needs a conf entry to pull that in in the first place. That's all | 20:47 |
| *** porrua has quit IRC | 20:48 | |
| jgriffith | guitarzan: but I kinda don't know what all is being discussed now, so I don't want to post anything in the bug that might add confusion | 20:48 |
| guitarzan | jgriffith: I'm in 100% agreement on everything you just said | 20:49 |
| guitarzan | but it is not possible to retroactively fix that old migration with a new migration | 20:49 |
| jgriffith | guitarzan: phewww | 20:49 |
| guitarzan | a new migration will just break other users... like me for one | 20:49 |
| guitarzan | that's why I did suggest the taboo of "edit the existing migration" | 20:50 |
| jgriffith | guitarzan: well... couldn't a new migration check if the column is "not null" ? | 20:50 |
| guitarzan | because the only people that need fixed are the ones that have never run this new migration | 20:50 |
| jgriffith | guitarzan: I don't see why that's a challenge? | 20:50 |
| jgriffith | guitarzan: it's similar to how we moved nova-volumes around way back when | 20:50 |
| diablo_rojo | hemna: I thoroughly tested that from a bunch of different places :) I should hope not. | 20:50 |
| jgriffith | guitarzan: if you "had them" we move the info around, if you don't we just do "our thing" | 20:50 |
| guitarzan | this proposed migration just deletes the quota class | 20:51 |
| guitarzan | end of story | 20:51 |
| jgriffith | guitarzan: That's wrong :) | 20:51 |
| jgriffith | end of story | 20:51 |
| jgriffith | :) | 20:51 |
| guitarzan | maybe I need to reread it | 20:51 |
| jgriffith | guitarzan: I mean, if that's what it does it's wrong | 20:51 |
| guitarzan | https://review.openstack.org/#/c/223300/2/cinder/db/sqlalchemy/migrate_repo/versions/056_remove_default_quota_values_from_table.py | 20:51 |
| guitarzan | I still think the best thing to do is edit the old migration | 20:51 |
| guitarzan | and blank it | 20:52 |
| jgriffith | WTF? | 20:52 |
| guitarzan | anyone that is currently having problems can just delete their own default quota class | 20:52 |
| guitarzan | this doesn't even seem to remove the default class, but all of them | 20:53 |
| *** sgotliv has joined #openstack-cinder | 20:53 | |
| guitarzan | I'm not sure why table.delete() seemed like a good idea | 20:53 |
| jgriffith | guitarzan: don't know if that comment will help or hurt: https://review.openstack.org/#/c/223300/ | 20:55 |
| jgriffith | guitarzan: but as you say, regardless... blowing away an existing deployments quota settings is not the right answer | 20:56 |
| jgriffith | and just putting another quota read from the conf file just completely misses the entire point of the bug | 20:56 |
| guitarzan | jgriffith: one thing about your proposal is that it'll never be null, since it already got messed up with the previous migration | 20:57 |
| jgriffith | guitarzan: My vote is modify the migration | 20:57 |
| *** pots2 has joined #openstack-cinder | 20:57 | |
| *** kbader1 has quit IRC | 20:57 | |
| patrickeast | hemna: hey, i've got a multipath question for you | 20:58 |
| jgriffith | guitarzan: whether that be cheating and fixing it directly, or as a side effect of a db compaction | 20:58 |
| *** crose has quit IRC | 20:58 | |
| jgriffith | which we've never done and sorely need | 20:58 |
| hemna | patrickeast, yough | 20:58 |
| guitarzan | jgriffith: I think migration 21 can be basically blanked out | 20:58 |
| patrickeast | hemna: actually two things, first an easy one... do you know whats up with https://review.openstack.org/#/c/196482/ ? | 20:58 |
| patrickeast | hemna: second, i'm looking at https://bugs.launchpad.net/cinder/+bug/1495701 | 20:59 |
| openstack | Launchpad bug 1495701 in OpenStack Compute (nova) "Sometimes Cinder volumes fail to attach with error "The device is not writable: Permission denied"" [Undecided,New] | 20:59 |
| patrickeast | i found out that my CI system stops doing it if I disable multipathing | 20:59 |
| patrickeast | so something weird is happening with the multipathing symlinks i think | 20:59 |
| patrickeast | it repros in the gate on encryption tests sometimes too | 20:59 |
| hemna | patrickeast, I was JUST talking to tsekiyama about that first one. he's looking into changing it | 20:59 |
| patrickeast | ah ok awesome | 21:00 |
| hemna | patrickeast, as it stands that patch would fail with FC volumes | 21:00 |
| *** pots has quit IRC | 21:00 | |
| patrickeast | ouch, thats not good | 21:00 |
| patrickeast | so for that bug, any ideas why a multipath device would sometimes be giving permission denied? | 21:01 |
| hemna | so for that bug | 21:01 |
| hemna | the log entry seems to indicate that the volume is iSCSI | 21:01 |
| patrickeast | yea when it happens in the gate it is | 21:02 |
| patrickeast | iscsi w/ lvm i think | 21:02 |
| hemna | I don't think I've seen that one yet | 21:02 |
| hemna | what test ? | 21:02 |
| patrickeast | its pretty rare http://logstash.openstack.org/#eyJzZWFyY2giOiJcImxpYnZpcnRFcnJvcjogb3BlcmF0aW9uIGZhaWxlZDogb3BlbiBkaXNrIGltYWdlIGZpbGUgZmFpbGVkXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjYwNDgwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0NDIyNjY3MDU1NzZ9 | 21:02 |
| hemna | do you have an example Pure CI log failure ? | 21:02 |
| patrickeast | for my CI system it varies, i've got a list of maybe half a dozen tests | 21:02 |
| patrickeast | http://ec2-54-67-51-189.us-west-1.compute.amazonaws.com/00/219700/9/check/PureFCDriver-tempest-dsvm-volume-multipath/1979276/logs/screen-c-vol.txt.gz? | 21:02 |
| patrickeast | different test | 21:02 |
| patrickeast | same problem | 21:03 |
| hemna | that looks like a failed encrypted volume test ? | 21:03 |
| patrickeast | sometimes in libvirt barfing that it can't open the disk, sometimes qemu-img, sometimes c-bak trying to copy data | 21:03 |
| patrickeast | yea for the gate ones that seems to be a favorite | 21:03 |
| *** p0rtal has joined #openstack-cinder | 21:03 | |
| hemna | device is busy kinda makes me think that it's trying to open a raw device where that device is part of a multipath device | 21:04 |
| hemna | where the multipath daemon itself has it open | 21:04 |
| hemna | and hence can't read from it. | 21:04 |
| patrickeast | ooh interesting | 21:04 |
| hemna | might have failed to detect the multipath device in time, and then fell back on the raw device path | 21:05 |
| hemna | and prior to trying to do cryptsetup the mpath device was created | 21:05 |
| hemna | bleh | 21:05 |
| hemna | I'm kinda guessing there though | 21:05 |
| patrickeast | well so for the gate its not using multipath and still gets errors like this http://logs.openstack.org/22/222922/2/check/gate-tempest-dsvm-full-lio/550be5e/logs/libvirt/libvirtd.txt.gz#_2015-09-13_17_34_07_499 | 21:05 |
| patrickeast | so its just a normal iscsi single path kind of deal | 21:06 |
| patrickeast | which is weird that single-path FC for my CI system works just fine : / | 21:06 |
| * hemna waits for the log to load in chrome..... | 21:06 | |
| hemna | goosh, that log file is giant | 21:07 |
| hemna | heh | 21:07 |
| patrickeast | haha yea | 21:07 |
| hemna | have a timestamp in that file for me ? | 21:07 |
| hemna | I'm not sure what error I'm looking for in that one | 21:07 |
| patrickeast | oh, should have been in the link | 21:07 |
| patrickeast | _2015-09-13_17_34_07_499 | 21:07 |
| hemna | I just get the top of the file | 21:08 |
| patrickeast | if you leave it long enough it *should* jump to it | 21:08 |
| patrickeast | once the file loads : / | 21:08 |
| hemna | hrmm | 21:08 |
| hemna | chrome seems to be trying to be "smart" | 21:08 |
| hemna | and not loading the entire file | 21:08 |
| hemna | wtf | 21:08 |
| hemna | phew | 21:09 |
| hemna | ok now it worked | 21:09 |
| hemna | only had to refresh 10 times | 21:09 |
| patrickeast | lol | 21:09 |
| patrickeast | yea those libvirt log files are brutal | 21:09 |
| hemna | ok, so that log files is a non multipath test ? | 21:09 |
| patrickeast | yea | 21:09 |
| hemna | multipath disabled and not running? | 21:09 |
| *** raildo is now known as raildo-afk | 21:10 | |
| hemna | if the daemon is running, that might cause it | 21:10 |
| patrickeast | its from a official gate job | 21:10 |
| patrickeast | i dont think they have any multipathing | 21:10 |
| patrickeast | sry, afk for a couple min | 21:10 |
| hemna | ok bbiab | 21:11 |
| *** hemna is now known as hemnafk | 21:11 | |
| *** ociuhandu has joined #openstack-cinder | 21:12 | |
| *** vgridnev has quit IRC | 21:13 | |
| *** jwcroppe has joined #openstack-cinder | 21:14 | |
| *** crose has joined #openstack-cinder | 21:16 | |
| *** thangp has quit IRC | 21:18 | |
| *** IanGovett has quit IRC | 21:20 | |
| smcginnis | geguileo: ping | 21:21 |
| *** dustins has joined #openstack-cinder | 21:24 | |
| *** boris-42 has joined #openstack-cinder | 21:26 | |
| *** cdelatte has quit IRC | 21:27 | |
| *** changbl has joined #openstack-cinder | 21:30 | |
| *** Yogi1 has quit IRC | 21:31 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Dynamically create cinder.conf.sample https://review.openstack.org/219700 | 21:32 |
| diablo_rojo | hemnafk: jgriffith smcginnis ^^^ | 21:33 |
| *** diablo_rojo has left #openstack-cinder | 21:33 | |
| DuncanT | thingee: https://bugs.launchpad.net/cinder/+bug/1491495 is the bug I thought should be a blocker. Submitting a patch to fix shortly | 21:35 |
| openstack | Launchpad bug 1491495 in Cinder "cinderclient.quotas.get() fails against Cinder service in a Devstack" [Undecided,Fix committed] - Assigned to Ivan Kolodyazhny (e0ne) | 21:35 |
| *** julim has quit IRC | 21:35 | |
| *** ebalduf has quit IRC | 21:40 | |
| *** kbader has joined #openstack-cinder | 21:44 | |
| *** edtubill has left #openstack-cinder | 21:44 | |
| *** anshul has quit IRC | 21:45 | |
| *** sgotliv has quit IRC | 21:45 | |
| *** krtaylor has quit IRC | 21:46 | |
| *** hemnafk is now known as hemna | 21:46 | |
| *** kbader1 has joined #openstack-cinder | 21:47 | |
| *** kbader has quit IRC | 21:48 | |
| *** mdenny has quit IRC | 21:50 | |
| *** gman-tx has quit IRC | 21:50 | |
| thingee | DuncanT: ack | 21:52 |
| *** xyang1 has quit IRC | 21:52 | |
| *** xyang1 has joined #openstack-cinder | 21:52 | |
| *** changbl has quit IRC | 21:55 | |
| *** david-lyle has joined #openstack-cinder | 21:56 | |
| *** hemna is now known as hemnafk | 22:00 | |
| smcginnis | hemnafk, jgriffith: In reference to my earlier retype issue, https://review.openstack.org/#/c/217522/ appears to fix it. | 22:03 |
| smcginnis | Not sure if it is/was an issue for you as well. | 22:03 |
| *** setmason has quit IRC | 22:04 | |
| *** setmason has joined #openstack-cinder | 22:04 | |
| *** jgregor has quit IRC | 22:05 | |
| *** jgregor has joined #openstack-cinder | 22:06 | |
| *** dustins has quit IRC | 22:07 | |
| *** jungleboyj has quit IRC | 22:09 | |
| *** jgregor has quit IRC | 22:10 | |
| *** stevemar has joined #openstack-cinder | 22:10 | |
| *** mriedem has quit IRC | 22:11 | |
| *** jwcroppe has quit IRC | 22:11 | |
| *** stevemar has quit IRC | 22:13 | |
| patrickeast | hemnafk: asselin: did you guys ever have any luck with the SR-IOV stuff with your FC HBAs? | 22:14 |
| *** takedakn has joined #openstack-cinder | 22:14 | |
| asselin | patrickeast, hemnafk and cbader_ played around with it a bit, but I don't remember hearing any successful result | 22:15 |
| patrickeast | bummer | 22:15 |
| *** hemnafk is now known as hemna | 22:16 | |
| *** annegentle has quit IRC | 22:16 | |
| hemna | patrickeast, so the SR-IOV works great | 22:16 |
| hemna | it's super awesome | 22:16 |
| patrickeast | \o/ | 22:16 |
| hemna | for ethernet interfaces | 22:16 |
| patrickeast | womp womp | 22:16 |
| hemna | I got so far that the HBA said it supports it | 22:17 |
| hemna | but the number of devs maxes out at 0 | 22:17 |
| hemna | which means that it's not supported | 22:17 |
| hemna | emulex's documentation was a mess | 22:17 |
| patrickeast | dang | 22:17 |
| cbader_ | patrickeast, I couldn't get much info on HBA support for SR-IOV plenty for nic. | 22:17 |
| hemna | so we are in contact with emulex to find out if any FC ports are SR-IOV'able | 22:17 |
| hemna | from what I could tell, only ethernet is really supported. | 22:18 |
| cbader_ | patrickeast, I have email into emulex engineer for info no response yet | 22:18 |
| hemna | :( | 22:18 |
| *** cbader_ has quit IRC | 22:18 | |
| patrickeast | hmm alright | 22:18 |
| patrickeast | thats a bummer | 22:18 |
| patrickeast | hopefully its just a documentation thing or driver update kind of deal | 22:18 |
| hemna | patrickeast, yah that would be nice | 22:19 |
| hemna | I spent a few days on it and got stuck | 22:19 |
| *** takedakn has quit IRC | 22:22 | |
| *** takedakn has joined #openstack-cinder | 22:23 | |
| *** pots2 has quit IRC | 22:24 | |
| patrickeast | hemna: one thing i do see on this page is that they require a firmware update for both emulex and qlogic https://blogs.oracle.com/raghuram/entry/fibre_channel_sr_iov | 22:26 |
| patrickeast | hemna: although its off in solaris land so i dunno how much carries over to what we would be trying | 22:26 |
| *** pots has joined #openstack-cinder | 22:27 | |
| hemna | heh, yah I saw that when I was hacking around as well | 22:27 |
| hemna | heh solaris | 22:28 |
| hemna | haven't been on a solaris box in 15 years | 22:28 |
| patrickeast | i'm trying to find at least a little bit of hope so i can back up my request for new 16Gb cards with (potentially) SR-IOV | 22:28 |
| patrickeast | haha yea | 22:28 |
| *** david-lyle has quit IRC | 22:29 | |
| *** esker has quit IRC | 22:29 | |
| hemna | patrickeast, http://paste.openstack.org/show/465303/ | 22:30 |
| hemna | so that's our ethernet nic | 22:31 |
| hemna | you can see that it has SR-IOV on, and lists 8 VFs | 22:31 |
| hemna | if that Total VGs is non zero, then SR-IOV is on and capable | 22:31 |
| *** ronis has quit IRC | 22:32 | |
| *** krtaylor has joined #openstack-cinder | 22:32 | |
| patrickeast | sweet | 22:32 |
| hemna | patrickeast, http://paste.openstack.org/show/465311/ | 22:33 |
| hemna | that's our FC HBA | 22:33 |
| hemna | hey look SR-IOV! | 22:33 |
| hemna | Initial VFs: 0 | 22:33 |
| hemna | wah wah wahhhhh | 22:33 |
| *** setmason has quit IRC | 22:34 | |
| *** p0rtal has quit IRC | 22:34 | |
| *** kbader1 has quit IRC | 22:34 | |
| *** p0rtal has joined #openstack-cinder | 22:34 | |
| rhedlind | patrickeast: Is there documentation somewhere on how to configure the image cache? | 22:35 |
| patrickeast | rhedlind: almost, https://review.openstack.org/#/c/221548/ | 22:35 |
| patrickeast | hemna: gah that sucks | 22:36 |
| patrickeast | hemna: teases with the feature supported... and then no VF's | 22:36 |
| rhedlind | patrickeast: thanks! I guess I will review that change while I am trying it out. :) | 22:37 |
| scottda | What is the schedule for removing support for Cinder API v1? Is there one? | 22:37 |
| *** ebalduf has joined #openstack-cinder | 22:38 | |
| *** setmason has joined #openstack-cinder | 22:38 | |
| *** p0rtal has quit IRC | 22:39 | |
| hemna | patrickeast, http://www-dl.emulex.com/support/elx/rt99/b15.1/docs/linux/linux_manual_elx.pdf | 22:40 |
| hemna | patrickeast, thats the magic doc that I found that got me real close. | 22:40 |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Fix cinder-all binary https://review.openstack.org/223851 | 22:40 |
| *** cdelatte has joined #openstack-cinder | 22:42 | |
| *** lcurtis has quit IRC | 22:42 | |
| hemna | patrickeast, page 35 talks about what to look for in the lspci output to see if it's enabled, etc. | 22:42 |
| *** p0rtal has joined #openstack-cinder | 22:43 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Fix MITM vulnerability for Brocade FC SAN lookup https://review.openstack.org/218341 | 22:43 |
| *** ebalduf has quit IRC | 22:44 | |
| patrickeast | mm looks like qlogic might be out, " Even though the SRIOV specification covers both storage and networking, the initial implementation is for networking only, similar to all other vendors." | 22:45 |
| patrickeast | from http://www.qlogic.com/Resources/Documents/FAQs/Adapters/QLogic_FlexSuite_Adapters.pdf | 22:46 |
| *** jungleboyj has joined #openstack-cinder | 22:46 | |
| hemna | boo | 22:46 |
| patrickeast | also implies emulex won't support it yet either | 22:46 |
| hemna | l a m e | 22:46 |
| patrickeast | since there are like 2 vendors >.< | 22:46 |
| jungleboyj | hemna: jgriffith smcginnis Kendall forgot to run Pep8 on her last patch. I am fixing the failures. Sorry, she was trying to catch her carpool and forgot. | 22:48 |
| hemna | jungleboyj, ok coolio. :) | 22:48 |
| *** dims_ has quit IRC | 22:48 | |
| hemna | alrighty, off to cat herding practice | 22:49 |
| *** hemna is now known as hemnafk | 22:49 | |
| jungleboyj | hemna: Sorry about that. Good thing she has such an team lead to save her. ;-) | 22:49 |
| jungleboyj | hemnafk: Have fun. | 22:49 |
| kmartin | jungleboyj, hemnafk, smcginnis can you check this one later https://review.openstack.org/#/c/223851/ (eOne should be in bed and it required a simple rebase), it will need a +A | 22:52 |
| jungleboyj | kmartin: Sure. | 22:52 |
| *** IanGovett has joined #openstack-cinder | 22:55 | |
| *** bill_az_ has joined #openstack-cinder | 22:55 | |
| *** setmason has quit IRC | 22:56 | |
| *** bill_az has quit IRC | 22:56 | |
| *** bill_az has joined #openstack-cinder | 22:57 | |
| *** bill_az_ has quit IRC | 22:57 | |
| *** setmason has joined #openstack-cinder | 23:02 | |
| *** IanGovett has quit IRC | 23:03 | |
| *** markvoelker has quit IRC | 23:05 | |
| *** mc_nair has quit IRC | 23:09 | |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Dynamically create cinder.conf.sample https://review.openstack.org/219700 | 23:12 |
| jungleboyj | hemnafk: smcginnis jgriffith thingee ^^^ That should hopefully be the real last patch. Sorry about that. | 23:12 |
| thingee | jungleboyj: ack | 23:13 |
| jgriffith | jungleboyj: I already had a fix for it | 23:13 |
| jgriffith | jungleboyj: but I'm also fixing some other things since this keeps dragging on | 23:13 |
| jgriffith | rsending my +2 | 23:13 |
| jgriffith | in other words, taking away my +2 | 23:13 |
| jgriffith | but if others happen to merge it before I get to it, that's fine | 23:13 |
| *** takedakn has quit IRC | 23:17 | |
| *** Zhongjun has joined #openstack-cinder | 23:20 | |
| *** setmason has quit IRC | 23:31 | |
| jungleboyj | What other stuff? | 23:32 |
| *** setmason has joined #openstack-cinder | 23:32 | |
| jungleboyj | jgriffith: ^^^ | 23:32 |
| *** dims_ has joined #openstack-cinder | 23:32 | |
| *** pots has quit IRC | 23:35 | |
| *** pots1 has joined #openstack-cinder | 23:38 | |
| jungleboyj | Oh, see your comments. | 23:40 |
| *** chlong has quit IRC | 23:41 | |
| jgriffith | jungleboyj: sorry, stepped away | 23:47 |
| jgriffith | jungleboyj: make sense? | 23:48 |
| jgriffith | jungleboyj: the other thing is the business of the whole commont_string, cmd_opts grep etc is kinda fugly | 23:48 |
| jgriffith | not a deal breaker but... | 23:48 |
| *** salv-orlando has quit IRC | 23:48 | |
| jgriffith | jungleboyj: i can condense all of that down to like 4 lines of code | 23:49 |
| jgriffith | that's actually readable :) | 23:49 |
| *** erhudy has quit IRC | 23:49 | |
| jgriffith | jungleboyj: also if it's going to require running in the base repo directory, that should probably be enforced/checked don't ya think? | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
