*** haomaiwang has quit IRC | 00:01 | |
*** haomaiwang has joined #openstack-cinder | 00:01 | |
*** IlyaG has quit IRC | 00:04 | |
*** IlyaG has joined #openstack-cinder | 00:05 | |
*** mylu has quit IRC | 00:09 | |
openstackgerrit | Scott DAngelo proposed openstack/cinder: Volume create fails with FakeISCSIDriver https://review.openstack.org/277619 | 00:11 |
---|---|---|
*** IlyaG has quit IRC | 00:12 | |
jgriffith | patrickeast: ping | 00:13 |
*** IlyaG has joined #openstack-cinder | 00:14 | |
*** IlyaG has quit IRC | 00:14 | |
patrickeast | jgriffith: pong | 00:15 |
jgriffith | patrickeast: hey.. I just wanted to confirm input from others on cast vs call for the failover call | 00:15 |
jgriffith | patrickeast: I can argue either way in my mind | 00:16 |
jgriffith | patrickeast: it's a critical enough op that blocking may not be a bad idea IMHO... but on the other hand it could be a long process? | 00:16 |
*** angela-s has quit IRC | 00:16 | |
patrickeast | jgriffith: hmm it seems like blocking would make sense | 00:16 |
patrickeast | jgriffith: but we don't really have like a return value or something | 00:17 |
jgriffith | patrickeast: well | 00:17 |
jgriffith | patrickeast: I just put a return of the "active_backend_id" in the api layer | 00:17 |
patrickeast | jgriffith: oh | 00:17 |
patrickeast | jgriffith: well then | 00:17 |
jgriffith | patrickeast: now whether I want to return that in the json call or not I dunno... vs just updating the db at that point | 00:17 |
*** mylu has joined #openstack-cinder | 00:18 | |
patrickeast | jgriffith: sry got side tracked looking at the code | 00:20 |
patrickeast | jgriffith: so uh, i like the idea of returning the id from the api... makes automating this much nicer | 00:20 |
jgriffith | no worries, I've been side-tracked for a week now :) | 00:20 |
patrickeast | jgriffith: and i know for like the pure backend the failover is quick, just a handful of rest api calls | 00:21 |
jgriffith | patrickeast: yeah, the initial reason for me was that I didn't have a place to pop it in the service data :) | 00:21 |
patrickeast | jgriffith: but i don't know enough about other backends to say for sure if they could handle doing it in a reasonable amount of time | 00:21 |
jgriffith | patrickeast: and I don't like messing with the view-builder :) | 00:21 |
patrickeast | jgriffith: haha yea | 00:22 |
*** cknight has quit IRC | 00:23 | |
jgriffith | patrickeast: I guess I'll move forward with it and see if somebody screams :) | 00:23 |
patrickeast | jgriffith: sounds good to me | 00:23 |
patrickeast | jgriffith: can always fix it with microversions on the api later :D | 00:24 |
jgriffith | patrickeast: LOL | 00:24 |
jgriffith | patrickeast: well, I was thinking worse case, we can break the call into two pieces at the manager->driver layer. But sure... let's just say "micro-versions", that's the one size fits all answer these days :) | 00:24 |
patrickeast | jgriffith: haha, exactly, a one-size-fits-all answer | 00:25 |
patrickeast | jgriffith: but i guess my point was its easy enough to just do the split later, and if anyone complains that its a behavior change.. boom increment the api number and call it a day | 00:25 |
jgriffith | patrickeast: +1 | 00:26 |
*** dramakri has quit IRC | 00:27 | |
jgriffith | ok, have a note in the code that hopefully vendors will read and provide input if needed | 00:29 |
jgriffith | thanks patrickeast | 00:29 |
patrickeast | np | 00:29 |
*** bill_az has quit IRC | 00:35 | |
*** tpsilva has quit IRC | 00:36 | |
*** arch-nemesis has quit IRC | 00:39 | |
*** jwang has quit IRC | 00:39 | |
*** jwang_ has joined #openstack-cinder | 00:39 | |
*** jwang_ has quit IRC | 00:43 | |
*** jwang has joined #openstack-cinder | 00:49 | |
*** itlinux has joined #openstack-cinder | 00:52 | |
*** jwang_ has joined #openstack-cinder | 00:53 | |
*** jwang has quit IRC | 00:53 | |
*** haomaiwang has quit IRC | 01:01 | |
*** haomaiwang has joined #openstack-cinder | 01:01 | |
*** garthb has quit IRC | 01:03 | |
openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector api in VMEM cinder driver https://review.openstack.org/277644 | 01:15 |
*** funeutron has joined #openstack-cinder | 01:18 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Trying py27/34 with oslo-master https://review.openstack.org/277653 | 01:27 |
*** mtanino has quit IRC | 01:34 | |
*** cknight has joined #openstack-cinder | 01:40 | |
*** cknight1 has joined #openstack-cinder | 01:42 | |
*** cknight has quit IRC | 01:45 | |
fernnest__ | smcginnis, is it reasonable to break a large commit into smaller filesets that don't get called (until a later commit)? | 01:54 |
*** apoorvad has quit IRC | 01:54 | |
*** itzdilip has quit IRC | 01:55 | |
*** itzdilip has joined #openstack-cinder | 01:56 | |
*** ChubYann has quit IRC | 01:56 | |
openstackgerrit | Merged openstack/cinder: IBM XIV/DS8K: Implements Replication V2 https://review.openstack.org/269490 | 01:57 |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-cinder | 02:01 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: Implement v2 replication https://review.openstack.org/237387 | 02:03 |
*** leeantho has quit IRC | 02:12 | |
*** csky has quit IRC | 02:20 | |
*** csky has joined #openstack-cinder | 02:21 | |
*** mudassirlatif has quit IRC | 02:21 | |
*** thurloat is now known as thurloat_isgone | 02:21 | |
*** xyang1 has quit IRC | 02:26 | |
*** erlon has quit IRC | 02:26 | |
*** mylu has quit IRC | 02:32 | |
*** mylu has joined #openstack-cinder | 02:35 | |
*** edmondsw has quit IRC | 02:41 | |
*** dan_nguyen has quit IRC | 02:55 | |
*** haomaiwang has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** EinstCrazy has joined #openstack-cinder | 03:02 | |
*** EinstCrazy has quit IRC | 03:07 | |
*** bardia has quit IRC | 03:07 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 03:10 |
*** mylu has quit IRC | 03:14 | |
*** dims has joined #openstack-cinder | 03:27 | |
*** dims_ has quit IRC | 03:28 | |
openstackgerrit | Merged openstack/cinder: Port objects unit tests to Python 3 https://review.openstack.org/272576 | 03:29 |
*** dims has quit IRC | 03:30 | |
*** mylu has joined #openstack-cinder | 03:33 | |
*** links has joined #openstack-cinder | 03:41 | |
*** dims has joined #openstack-cinder | 03:43 | |
*** mylu has quit IRC | 03:45 | |
*** mylu has joined #openstack-cinder | 03:46 | |
*** dims has quit IRC | 03:47 | |
*** mylu has quit IRC | 03:49 | |
*** mylu has joined #openstack-cinder | 03:49 | |
*** Poornima has joined #openstack-cinder | 03:50 | |
openstackgerrit | Merged openstack/cinder: ScaleIO QoS Support https://review.openstack.org/260719 | 03:56 |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwa_ has joined #openstack-cinder | 04:01 | |
*** apoorvad has joined #openstack-cinder | 04:13 | |
apoorvad | xyang1 EMC XIO CI is voting multiple times on https://review.openstack.org/#/c/261126/, is there any known issue? | 04:14 |
*** apoorvad has quit IRC | 04:19 | |
*** mylu has quit IRC | 04:19 | |
*** mylu has joined #openstack-cinder | 04:20 | |
openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 04:24 |
*** mylu has quit IRC | 04:38 | |
*** shyama has joined #openstack-cinder | 04:40 | |
*** salv-orl_ has joined #openstack-cinder | 04:41 | |
*** mylu has joined #openstack-cinder | 04:42 | |
*** salv-orlando has quit IRC | 04:43 | |
*** shausy has joined #openstack-cinder | 04:52 | |
*** mylu has quit IRC | 04:55 | |
*** dan_nguyen has joined #openstack-cinder | 04:56 | |
*** dulek has joined #openstack-cinder | 05:00 | |
*** haomaiwa_ has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** cknight1 has quit IRC | 05:02 | |
*** funeutron has quit IRC | 05:03 | |
*** mylu has joined #openstack-cinder | 05:05 | |
*** jgriffith is now known as jgriffith_away | 05:07 | |
*** cfriesen has quit IRC | 05:11 | |
*** itzdilip has quit IRC | 05:11 | |
*** mylu has quit IRC | 05:13 | |
*** nikeshm has quit IRC | 05:14 | |
*** chhavi has joined #openstack-cinder | 05:17 | |
*** mylu has joined #openstack-cinder | 05:20 | |
*** shausy has quit IRC | 05:37 | |
*** shausy has joined #openstack-cinder | 05:38 | |
*** dan_nguyen has quit IRC | 05:40 | |
*** mylu has quit IRC | 05:45 | |
*** mylu has joined #openstack-cinder | 05:46 | |
*** mylu has quit IRC | 05:49 | |
*** mylu has joined #openstack-cinder | 05:53 | |
*** mylu has quit IRC | 05:55 | |
*** Poornima has quit IRC | 05:56 | |
*** haomaiwang has quit IRC | 06:01 | |
*** Poornima has joined #openstack-cinder | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** shausy has quit IRC | 06:07 | |
*** shausy has joined #openstack-cinder | 06:07 | |
*** Poornima has quit IRC | 06:08 | |
*** savihou has joined #openstack-cinder | 06:11 | |
*** Poornima has joined #openstack-cinder | 06:13 | |
*** Poornima has quit IRC | 06:13 | |
*** Poornima has joined #openstack-cinder | 06:14 | |
*** sheel has joined #openstack-cinder | 06:16 | |
*** avishay has joined #openstack-cinder | 06:31 | |
*** avishay has quit IRC | 06:51 | |
*** avishay has joined #openstack-cinder | 06:53 | |
*** leeantho has joined #openstack-cinder | 06:55 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** avishay has quit IRC | 07:02 | |
*** nkrinner has joined #openstack-cinder | 07:03 | |
*** lpetrut has joined #openstack-cinder | 07:03 | |
*** anshul has joined #openstack-cinder | 07:06 | |
*** isaacb has joined #openstack-cinder | 07:21 | |
*** savihou has quit IRC | 07:28 | |
*** sgotliv has joined #openstack-cinder | 07:30 | |
*** lpetrut has quit IRC | 07:31 | |
*** avishay has joined #openstack-cinder | 07:34 | |
*** alyson_ has quit IRC | 07:37 | |
*** alyson_ has joined #openstack-cinder | 07:37 | |
*** belmoreira has joined #openstack-cinder | 07:41 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwa_ has joined #openstack-cinder | 08:01 | |
*** liverpooler has joined #openstack-cinder | 08:03 | |
*** mvk has joined #openstack-cinder | 08:04 | |
*** savihou has joined #openstack-cinder | 08:05 | |
*** leeantho has quit IRC | 08:11 | |
*** alonma has joined #openstack-cinder | 08:16 | |
*** sinese_ has joined #openstack-cinder | 08:24 | |
*** deepakcs has joined #openstack-cinder | 08:27 | |
*** sgotliv has quit IRC | 08:34 | |
*** e0ne has joined #openstack-cinder | 08:49 | |
*** mvk has quit IRC | 08:49 | |
*** ndipanov has joined #openstack-cinder | 08:55 | |
*** bkumar has joined #openstack-cinder | 08:59 | |
*** haomaiwa_ has quit IRC | 09:01 | |
*** geguileo has joined #openstack-cinder | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
*** arecknag has joined #openstack-cinder | 09:03 | |
*** shausy has quit IRC | 09:08 | |
*** arecknag has quit IRC | 09:08 | |
*** shausy has joined #openstack-cinder | 09:09 | |
*** arecknag has joined #openstack-cinder | 09:09 | |
*** sheel has quit IRC | 09:17 | |
*** mvk has joined #openstack-cinder | 09:17 | |
*** e0ne has quit IRC | 09:19 | |
*** alonma has quit IRC | 09:25 | |
*** alonma has joined #openstack-cinder | 09:25 | |
*** arecknag has quit IRC | 09:26 | |
*** arecknag has joined #openstack-cinder | 09:28 | |
*** mvk has quit IRC | 09:29 | |
*** alonma has quit IRC | 09:29 | |
*** mvk has joined #openstack-cinder | 09:30 | |
*** alonma has joined #openstack-cinder | 09:31 | |
*** jwcroppe has joined #openstack-cinder | 09:33 | |
*** shausy has quit IRC | 09:33 | |
*** alonma has quit IRC | 09:35 | |
*** jwcroppe has quit IRC | 09:36 | |
*** jwcroppe has joined #openstack-cinder | 09:37 | |
*** alonma has joined #openstack-cinder | 09:37 | |
*** ociuhandu has quit IRC | 09:38 | |
ntt | Hi, I'd like to use nfs as a cinder backend but I have a question: Compute nodes will have a local mountpoint or traffic passes through the cinder-volume node? | 09:40 |
*** jwcroppe has quit IRC | 09:41 | |
*** lpetrut has joined #openstack-cinder | 09:51 | |
*** yhayashi has quit IRC | 09:59 | |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwa_ has joined #openstack-cinder | 10:01 | |
*** arecknag has quit IRC | 10:05 | |
*** arecknag has joined #openstack-cinder | 10:06 | |
*** e0ne has joined #openstack-cinder | 10:21 | |
dulek | ntt: Trafic will not pass through c-vols | 10:22 |
ntt | dulek: thank you. So each compute node will have N mountpoints, where N = number of volume used by that compute node. Is this correct? | 10:24 |
*** Roamer`_ is now known as Roamer` | 10:28 | |
dulek | ntt: I believe so, but it may be more complicated when taking multiattach into account. | 10:28 |
*** esp has joined #openstack-cinder | 10:30 | |
*** esp has quit IRC | 10:37 | |
*** salv-orlando has joined #openstack-cinder | 10:40 | |
*** salv-orl_ has quit IRC | 10:43 | |
*** dims has joined #openstack-cinder | 10:45 | |
*** shyama has quit IRC | 10:47 | |
*** alonma has quit IRC | 10:47 | |
*** dims has quit IRC | 10:49 | |
*** dims has joined #openstack-cinder | 10:50 | |
*** deepakcs has quit IRC | 10:52 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port backup drivers to Python 3 https://review.openstack.org/277778 | 10:53 |
*** alonma has joined #openstack-cinder | 10:54 | |
*** tpsilva has joined #openstack-cinder | 10:55 | |
*** erlon has joined #openstack-cinder | 10:56 | |
*** alonma has quit IRC | 10:58 | |
openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 10:59 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 10:59 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 10:59 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 10:59 |
*** alonma has joined #openstack-cinder | 11:00 | |
*** haomaiwa_ has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** sgotliv has joined #openstack-cinder | 11:04 | |
*** alonma has quit IRC | 11:05 | |
*** alonma has joined #openstack-cinder | 11:06 | |
*** mvk has quit IRC | 11:07 | |
*** mvk has joined #openstack-cinder | 11:08 | |
*** chhavi has quit IRC | 11:08 | |
*** chhavi has joined #openstack-cinder | 11:08 | |
*** alonma has quit IRC | 11:11 | |
*** alonma has joined #openstack-cinder | 11:12 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Implement disconnect_volume for RemoteFsConnector https://review.openstack.org/277788 | 11:14 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Implement disconnect_volume for RemoteFsConnector (NFS only) https://review.openstack.org/277788 | 11:14 |
*** alonma has quit IRC | 11:17 | |
*** alonma has joined #openstack-cinder | 11:17 | |
*** bkumar has quit IRC | 11:32 | |
*** dims_ has joined #openstack-cinder | 11:40 | |
*** dims has quit IRC | 11:40 | |
*** boris-42 has quit IRC | 11:43 | |
*** arecknag has quit IRC | 11:44 | |
*** arecknag has joined #openstack-cinder | 11:45 | |
*** xek_ is now known as xek | 11:51 | |
*** lprice has joined #openstack-cinder | 11:53 | |
*** shyama has joined #openstack-cinder | 11:53 | |
*** lprice1 has quit IRC | 11:55 | |
openstackgerrit | wanghao proposed openstack/cinder: Clean up image tmp file if c-vol gets restarted https://review.openstack.org/225081 | 11:55 |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
*** dims_ has quit IRC | 12:11 | |
*** Poornima has quit IRC | 12:15 | |
*** sgotliv has quit IRC | 12:16 | |
*** dims_ has joined #openstack-cinder | 12:19 | |
*** thurloat_isgone is now known as thurloat | 12:22 | |
*** avishay has quit IRC | 12:22 | |
*** zhangjn has quit IRC | 12:24 | |
*** zhangjn has joined #openstack-cinder | 12:27 | |
*** mtanino has joined #openstack-cinder | 12:28 | |
openstackgerrit | Szymon Borkowski proposed openstack/cinder: Fixed logging for oslo versioned objects https://review.openstack.org/266289 | 12:31 |
openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS driver: Fix SSH and cluster_admin_ip0 bug https://review.openstack.org/277495 | 12:37 |
*** openstackgerrit_ has joined #openstack-cinder | 12:39 | |
*** marcusvrn_ has joined #openstack-cinder | 12:44 | |
*** alonma has quit IRC | 12:53 | |
*** alonma has joined #openstack-cinder | 12:54 | |
*** alonma has quit IRC | 12:58 | |
*** alonma has joined #openstack-cinder | 13:00 | |
*** porrua has joined #openstack-cinder | 13:00 | |
*** haomaiwang has quit IRC | 13:01 | |
*** haomaiwang has joined #openstack-cinder | 13:01 | |
*** alonma has quit IRC | 13:03 | |
*** alonma has joined #openstack-cinder | 13:04 | |
*** ildikov has quit IRC | 13:06 | |
*** avishay has joined #openstack-cinder | 13:13 | |
*** pots has joined #openstack-cinder | 13:14 | |
smcginnis | fernnest__: Saw your question. Yes, that would be reasonable. | 13:15 |
*** links has quit IRC | 13:17 | |
*** smoriya_ has quit IRC | 13:18 | |
e0ne | smcginnis: hi. could you please review https://review.openstack.org/#/c/267801/? | 13:19 |
smcginnis | e0ne: Sure, I'll take a look. | 13:20 |
e0ne | smcginnis: thanks | 13:20 |
smcginnis | Any time | 13:21 |
*** deepakcs has joined #openstack-cinder | 13:24 | |
*** bill_az has joined #openstack-cinder | 13:25 | |
swamireddy | Hi | 13:31 |
swamireddy | cinder quota-show and quota-usage - CLIs are not showing the backup related details? | 13:31 |
swamireddy | is it expected? | 13:31 |
swamireddy | I used cinder quota-show <tenant-id> | 13:32 |
*** avishay has quit IRC | 13:34 | |
*** diablo_rojo has joined #openstack-cinder | 13:38 | |
*** mriedem has joined #openstack-cinder | 13:41 | |
mtanino | swamireddy: I think these commands show backup_gigabytes and backups like in this report. https://bugzilla.redhat.com/show_bug.cgi?id=1289681 | 13:41 |
openstack | bugzilla.redhat.com bug 1289681 in openstack-cinder "VolumeLimitExceeded error even though quota is updated to larger value" [High,Closed: worksforme] - Assigned to eharney | 13:41 |
*** jordanP has joined #openstack-cinder | 13:41 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API v1 to Python 3 https://review.openstack.org/277835 | 13:42 |
*** alyson_ has quit IRC | 13:42 | |
*** eharney has joined #openstack-cinder | 13:45 | |
*** haomaiwang has quit IRC | 13:46 | |
*** haomaiwa_ has joined #openstack-cinder | 13:47 | |
swamireddy | mtanino: hm....I use cinder 1.1.1 - its not showing the backup details...(backup limits and usage) | 13:47 |
*** crose has joined #openstack-cinder | 13:48 | |
mtanino | swamireddy: I tried Liberty base version and it | 13:48 |
*** akerr has joined #openstack-cinder | 13:49 | |
mtanino | supports backup related details. | 13:49 |
*** alonma has quit IRC | 13:52 | |
*** anshul has quit IRC | 13:53 | |
*** avishay has joined #openstack-cinder | 13:54 | |
*** cbader has joined #openstack-cinder | 13:55 | |
*** edmondsw has joined #openstack-cinder | 13:56 | |
*** openstackgerrit_ has quit IRC | 13:56 | |
*** alonma has joined #openstack-cinder | 13:57 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API v1 and v2 to Python 3 https://review.openstack.org/277835 | 14:00 |
openstackgerrit | Victor Stinner proposed openstack/cinder: hacking: Fix false positive in C302 check https://review.openstack.org/277841 | 14:00 |
*** vgridnev has joined #openstack-cinder | 14:00 | |
*** alonma has quit IRC | 14:00 | |
*** haomaiwa_ has quit IRC | 14:01 | |
*** alonma has joined #openstack-cinder | 14:01 | |
*** haomaiwa_ has joined #openstack-cinder | 14:01 | |
*** akerr has quit IRC | 14:01 | |
*** xyang1 has joined #openstack-cinder | 14:02 | |
*** ildikov has joined #openstack-cinder | 14:02 | |
*** dims_ has quit IRC | 14:03 | |
*** alonma has quit IRC | 14:05 | |
*** akerr has joined #openstack-cinder | 14:05 | |
*** cknight has joined #openstack-cinder | 14:05 | |
*** jungleboyj has joined #openstack-cinder | 14:05 | |
*** deepakcs has quit IRC | 14:07 | |
*** mtanino has quit IRC | 14:07 | |
*** haomaiwa_ has quit IRC | 14:08 | |
*** haypo has joined #openstack-cinder | 14:09 | |
*** alonma has joined #openstack-cinder | 14:09 | |
*** dims has joined #openstack-cinder | 14:10 | |
haypo | eharney: hey! it's really cool to get a +2 in less than 6 months :-) i appreciate your reactivity ;) | 14:11 |
eharney | :) | 14:11 |
smcginnis | haypo: I thought we've been pretty good about getting those through faster? | 14:13 |
haypo | smcginnis: it's a pleasure to contribute to cinder, you are usually quite fast compared to other openstack services | 14:14 |
smcginnis | haypo: Haha, good to hear. :) | 14:14 |
jungleboyj | That is a good way to start Tuesday. :-) | 14:14 |
smcginnis | Well, for us. Not for you with the other services. | 14:14 |
smcginnis | :) | 14:14 |
*** mylu has joined #openstack-cinder | 14:14 | |
*** knikolla has joined #openstack-cinder | 14:15 | |
jungleboyj | Other services? ;-) | 14:15 |
smcginnis | jungleboyj: ;) | 14:15 |
haypo | eharney, smcginnis : see my grumpy email sent yesterday about a different service, almost no review after 4 months: http://lists.openstack.org/pipermail/openstack-dev/2016-February/086060.html | 14:16 |
smcginnis | haypo: Ooo, sadness. | 14:16 |
*** links has joined #openstack-cinder | 14:16 | |
smcginnis | haypo: I think you are justified though. | 14:16 |
smcginnis | haypo: You can't keep rebasing if no one's going to ever look at it. | 14:17 |
smcginnis | It's unfortunate though, for sure. | 14:17 |
*** rlrossit has joined #openstack-cinder | 14:17 | |
*** alonma has quit IRC | 14:17 | |
haypo | smcginnis: i'm annoyed because we agreed on a plan, but then, nothing | 14:17 |
smcginnis | haypo: Yeah, just reading through that all the way. Sounded like it was on track. :/ | 14:18 |
jungleboyj | smcginnis: Have you seen these start popping up? https://bugs.launchpad.net/cinder/+bug/1543407 | 14:18 |
openstack | Launchpad bug 1543407 in Cinder " ScaleIO QoS Support" [Undecided,New] | 14:18 |
smcginnis | haypo: Well, I can say for me at least, I really appreciate the work you've done in Cinder for py3. | 14:18 |
haypo | well, i also understand that reviewing my boring patches is less exciting than working on new stuff | 14:18 |
smcginnis | haypo: It's a huge undertaking but very necessary. | 14:19 |
haypo | but i'm no more interested to ping dev forever | 14:19 |
haypo | smcginnis: we are closer everyday to get all unit tests running on py3, for cinder | 14:19 |
*** alonma has joined #openstack-cinder | 14:19 | |
smcginnis | haypo: +1 :) | 14:19 |
smcginnis | jungleboyj: No, what's the deal there? | 14:19 |
*** eharney has quit IRC | 14:19 | |
smcginnis | jungleboyj: Oh, those. Yeah. | 14:20 |
jungleboyj | smcginnis: I don't know and the message that comes with it doesn't totally make sense to me. | 14:20 |
smcginnis | jungleboyj: I have that on the agenda for tomorrow. | 14:20 |
jungleboyj | smcginnis: Ok, cool. | 14:20 |
smcginnis | jungleboyj: Basically there's been a change in how DocImpact works. | 14:20 |
*** jgriffith_away is now known as jgriffith | 14:21 | |
smcginnis | It's now on us (who actually know what changed) to provide the documentation update details. | 14:21 |
*** esp has joined #openstack-cinder | 14:21 | |
smcginnis | jungleboyj: Before DocImpact would open bugs against the doc team, then they would have to go figure out what they were supposed to do. | 14:21 |
jungleboyj | Ahhh, so do we need to write the documentation, or just put the information in the bug? | 14:22 |
smcginnis | jungleboyj: From what I understand, we just need to put the information in the bug, then pass it on to them. | 14:22 |
smcginnis | jungleboyj: Although I'm sure that team would be open to getting patches directly to update the docs. ;) | 14:23 |
openstackgerrit | Merged openstack/cinder: Support cinder_img_volume_type in image metadata https://review.openstack.org/258649 | 14:24 |
haypo | smcginnis: what's the plan for XML? deprecated ok, but do you plan to drop code to handle XML? | 14:24 |
jungleboyj | smcginnis: Ok. Thanks for the info. | 14:24 |
smcginnis | haypo: Yes. I wouldn't bother with updating that. | 14:24 |
e0ne | haypo: we're going to drop XML API in Newton | 14:25 |
*** alonma has quit IRC | 14:25 | |
haypo | smcginnis: when i port code, i don't really care if it's deprecated or not. i just port it :) it was a general question | 14:25 |
haypo | e0ne: ok, so next cycle | 14:25 |
smcginnis | :) | 14:25 |
*** jgregor has joined #openstack-cinder | 14:25 | |
*** alonma has joined #openstack-cinder | 14:26 | |
*** esp has quit IRC | 14:29 | |
*** akshai has joined #openstack-cinder | 14:30 | |
*** alonma has quit IRC | 14:30 | |
flip214 | I'd like to ask for another small review: https://review.openstack.org/#/c/277314/2 | 14:33 |
*** mylu has quit IRC | 14:34 | |
*** dave-mccowan has joined #openstack-cinder | 14:34 | |
*** jgriffith is now known as jgriffith_away | 14:35 | |
*** mylu has joined #openstack-cinder | 14:35 | |
*** akerr has quit IRC | 14:36 | |
*** gouthamr has joined #openstack-cinder | 14:37 | |
*** akerr has joined #openstack-cinder | 14:37 | |
*** jordanP has quit IRC | 14:39 | |
*** tongli has joined #openstack-cinder | 14:40 | |
*** jgriffith_away is now known as jgriffith | 14:43 | |
*** ociuhandu has joined #openstack-cinder | 14:43 | |
*** alonma has joined #openstack-cinder | 14:43 | |
*** ociuhandu has quit IRC | 14:46 | |
*** dustins has joined #openstack-cinder | 14:47 | |
*** alonma has quit IRC | 14:48 | |
*** julim_ has quit IRC | 14:49 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port volume_type_encryption to Python 3 https://review.openstack.org/277861 | 14:49 |
*** cfriesen has joined #openstack-cinder | 14:49 | |
*** alonma has joined #openstack-cinder | 14:49 | |
*** amit213 has quit IRC | 14:51 | |
*** tpatzig has quit IRC | 14:53 | |
*** mtanino has joined #openstack-cinder | 14:54 | |
*** alonma has quit IRC | 14:54 | |
*** laughterwym has joined #openstack-cinder | 14:54 | |
*** links has quit IRC | 14:54 | |
*** mkoderer has quit IRC | 14:55 | |
*** alonma has joined #openstack-cinder | 14:55 | |
*** mkoderer has joined #openstack-cinder | 14:56 | |
*** tpatzig has joined #openstack-cinder | 14:56 | |
*** mylu has quit IRC | 14:56 | |
*** mylu has joined #openstack-cinder | 14:57 | |
*** ociuhandu has joined #openstack-cinder | 14:58 | |
diablo_rojo | smcginnis: hemna e0ne patrickeast DuncanT jungleboyj jgriffith xyang1 geguileo In the cross project meeting today we will be discussing these two reviews, if there are any opinions you would like me to bring forward please let me know as soon as you can. https://review.openstack.org/#/c/242852/ https://review.openstack.org/#/c/245629/5 Thanks! | 14:59 |
jgriffith | diablo_rojo: they both look good to me | 14:59 |
*** alonma has quit IRC | 14:59 | |
jgriffith | diablo_rojo: I'll do a proper review and add comments before you're meeting | 15:00 |
diablo_rojo | jgriffith: Cool. Thanks John :) | 15:00 |
jgriffith | diablo_rojo: thanks for pointing them out | 15:00 |
diablo_rojo | jgriffith: No problem. | 15:00 |
*** laughterwym has quit IRC | 15:00 | |
smcginnis | diablo_rojo: Yeah, looks fine to me. Thanks! | 15:01 |
diablo_rojo | smcginnis: Excellent :) | 15:01 |
*** laughterwym has joined #openstack-cinder | 15:04 | |
*** crose has quit IRC | 15:05 | |
*** lprice1 has joined #openstack-cinder | 15:05 | |
e0ne | diablo_rojo: looks like a right direction. reading | 15:07 |
e0ne | jgriffith: btw, please take a look on https://review.openstack.org/#/c/267801/ - functional tests for cinder | 15:08 |
*** sheel has joined #openstack-cinder | 15:08 | |
*** lprice has quit IRC | 15:09 | |
*** alonma has joined #openstack-cinder | 15:09 | |
*** baumann has joined #openstack-cinder | 15:09 | |
diablo_rojo | e0ne: Thanks for checking it out :) | 15:10 |
*** edtubill has joined #openstack-cinder | 15:11 | |
jgriffith | diablo_rojo: so I did have some objections on this one: https://review.openstack.org/#/c/245629/5 Noted in comments | 15:11 |
jgriffith | diablo_rojo: let me know if there's any questions that come up I can help with. | 15:11 |
jgriffith | diablo_rojo: I'm certainly in the minority on that one :) | 15:12 |
jgriffith | but that's where I like to be usually | 15:12 |
diablo_rojo | jgriffith: I definitely will and I will call you into the meeting if they have any questions about what you've said . | 15:12 |
e0ne | diablo_rojo: should it https://review.openstack.org/#/c/242852/ be re-writed to be a cross-project? | 15:12 |
e0ne | for now, it's about keystone | 15:12 |
diablo_rojo | jgriffith: It is more fun to play devils advocate :) | 15:12 |
jgriffith | diablo_rojo: :) | 15:12 |
diablo_rojo | jgriffith: Not even trying to be clever there with my nick :) | 15:13 |
jgriffith | diablo_rojo: uh-huh :) | 15:13 |
jgriffith | diablo_rojo: I thought the same thing when you typed that | 15:13 |
smcginnis | Hah | 15:14 |
*** alonma has quit IRC | 15:14 | |
diablo_rojo | e0ne: It was originally for keystone, but I think they are just looking at it like a cross project spec | 15:14 |
diablo_rojo | jgriffith: You can be my advocate any time you want though :) | 15:14 |
*** cdelatte has joined #openstack-cinder | 15:14 | |
jgriffith | haha... "devils advocate" I like it | 15:14 |
jgriffith | diablo_rojo: I see what you did there | 15:15 |
*** alonma has joined #openstack-cinder | 15:15 | |
diablo_rojo | jgriffith: :D | 15:15 |
diablo_rojo | e0ne: It looks like the spec is in the correct repo, so I guess we are just assuming it would be good for all projects and Keystone is just one application for the spec. | 15:17 |
e0ne | diablo_rojo: as I understood, the same should be implemented in each project, right? | 15:18 |
diablo_rojo | e0ne: That is correct. | 15:19 |
*** ildikov has quit IRC | 15:19 | |
e0ne | diablo_rojo: good. I'll -1 on it. it woould be good to have more general spec to not confuse anybody | 15:19 |
*** ildikov has joined #openstack-cinder | 15:19 | |
*** alonma has quit IRC | 15:19 | |
smcginnis | Do users have problems with spec files now? | 15:20 |
smcginnis | I never heard that feedback? | 15:20 |
jgriffith | smcginnis: users and spec files? | 15:20 |
diablo_rojo | e0ne: I was confused at first too till I reread the commit message. | 15:20 |
e0ne | smcginnis: what problems you you mean? | 15:20 |
jgriffith | smcginnis: or do you mean policy files? | 15:20 |
smcginnis | I'm not against formalizing some current conventions, but just surprised that it seems to be such a big issue for others. | 15:21 |
smcginnis | jgriffith: policy files. | 15:21 |
jgriffith | smcginnis: ahhh | 15:21 |
jgriffith | smcginnis: yeah, so Brick exposed a big problem | 15:21 |
smcginnis | Oops, I see my typo. | 15:21 |
e0ne | diablo_rojo: yeah, commit message is good | 15:21 |
jgriffith | smcginnis: the problem is when you have things like libs that are written such that they need policy files... making them all "line up" | 15:21 |
jgriffith | smcginnis: some like decoupling.. some don' t :) | 15:22 |
smcginnis | jgriffith: Yeah, true. Same issue with rootwrap. | 15:22 |
jgriffith | smcginnis: exactly | 15:22 |
*** knomura has joined #openstack-cinder | 15:22 | |
jgriffith | smcginnis: my fear is what's proposed there has the potential to be WAAAAY more complex than I would prefer | 15:22 |
smcginnis | jgriffith: Kind of what I'm thinking. | 15:23 |
smcginnis | There's definitely some caveats as it is today. | 15:23 |
jgriffith | smcginnis: I'm also somewhat unclear on how that file *looks* when it's done... like "10K lines of json" Puke | 15:23 |
smcginnis | But when it comes down to it, it's fairly simple really. | 15:23 |
smcginnis | Hah, yeah, that will be a good end user experience. | 15:23 |
jgriffith | :) | 15:23 |
jgriffith | getting breakfast, back in a bit | 15:24 |
*** alonma has joined #openstack-cinder | 15:25 | |
*** dustins has quit IRC | 15:26 | |
smcginnis | Gaahh! Love how gerrit randomly jumps around as you scroll. :| | 15:26 |
diablo_rojo | smcginnis: Is it time for gerrit complaints already today? | 15:27 |
diablo_rojo | Where is hemna? I bet he has a thing or two to say here :) | 15:27 |
smcginnis | diablo_rojo: Kind of early I guess. hemna's not even here to join in. | 15:27 |
*** timcl has joined #openstack-cinder | 15:27 | |
diablo_rojo | smcginnis: Hold that thought for an hour or two, make a list even, then we can go for the gold. | 15:28 |
smcginnis | ;) | 15:28 |
diablo_rojo | smcginnis: We really should have spent all friday of the meetup filing gerrit bugs and fixing them | 15:28 |
*** dustins has joined #openstack-cinder | 15:28 | |
smcginnis | Hah, that would have kept us busy. | 15:29 |
diablo_rojo | smcginnis: No doubt. | 15:29 |
*** alonma has quit IRC | 15:30 | |
*** dustins has quit IRC | 15:33 | |
*** dustins has joined #openstack-cinder | 15:36 | |
*** laughterwym has quit IRC | 15:36 | |
*** mragupat has joined #openstack-cinder | 15:38 | |
*** krtaylor has quit IRC | 15:42 | |
jungleboyj | diablo_rojo: Ok, I think e0ne is right that the first one needs to be generalized. | 15:44 |
jungleboyj | diablo_rojo: Looks like jgriffith has made some good comments on the second one that should be addressed. | 15:44 |
jungleboyj | diablo_rojo: Thanks for bringing those up. Good to know that those items are being discussed. | 15:45 |
*** julim has joined #openstack-cinder | 15:45 | |
diablo_rojo | jungleboyj: No problem. I will bring all of that up in the meeting this afternoon. | 15:45 |
openstackgerrit | Shilpa Jagannath proposed openstack/python-cinderclient: Allow "cinder backup-delete" to delete multiple backups in one request https://review.openstack.org/277895 | 15:46 |
*** alonma has joined #openstack-cinder | 15:47 | |
jungleboyj | diablo_rojo: Good deal. Thanks! | 15:47 |
*** marcusvrn_ has quit IRC | 15:47 | |
sheel | jungleboyj: smcginnis: I too want to mention 2 points where we should think about | 15:49 |
sheel | 1. limiting size of patch to around 1000 LOC : may be all guys find it difficult to review large patchsets..so it would be good if we can force this policy.. | 15:50 |
sheel | 2. review requests : Is it possible to find some other way of requesting for reviews? | 15:51 |
smcginnis | sheel: That's a hard policy to enforce. Sometimes it is hard to break something down into smaller chunks. | 15:51 |
smcginnis | sheel: It's definitely the recommended way, and folks will have a lot better chance of getting timely reviews if their patches aren't so large. | 15:52 |
*** alonma has quit IRC | 15:52 | |
smcginnis | sheel: But I don't think we can have a hard fast rule about it. | 15:52 |
sheel | smcginnis: yes T&C applies... some exception would be there... | 15:52 |
smcginnis | sheel: As for review requests, I think that's something folks have struggled with a long time. | 15:52 |
*** liverpooler has quit IRC | 15:53 | |
smcginnis | Not sure if anyone has come up with a good solution, other than to wait for folks to get the time to review | 15:53 |
*** alonma has joined #openstack-cinder | 15:53 | |
akerr | bribes help | 15:53 |
smcginnis | I'm not against getting requests for reviews (reasonably) but I know others definitely do not like it. | 15:53 |
smcginnis | Hah! | 15:53 |
sheel | smcginnis: yes, time factor matters most | 15:53 |
smcginnis | sheel: Yeah, it is unfortunate when I see fairly trivial patches out there for weeks. | 15:54 |
*** isaacb has quit IRC | 15:54 | |
smcginnis | So I can state here what I try to tell everyone when asked - if anyone is looking at getting involved but not sure where to start - code reviews are always welcome. | 15:55 |
sheel | smcginnis: agreed with every word you said | 15:55 |
sheel | smcginnis: actually why I raised this point because sometimes its demotivating for new guys to get their things go ahead... | 15:55 |
smcginnis | sheel: Yeah, I know. :{ | 15:56 |
sheel | smcginnis: so, I was just looking emphatically and was trying if anything is possible to mobilize same... | 15:56 |
smcginnis | sheel: If you come up with a solution, let me know. ;) | 15:56 |
sheel | smcginnis: will see if anything is possible for sure and will discuss with you guys | 15:57 |
*** alonma has quit IRC | 15:57 | |
sheel | smcginnis: thanks for discussion ... :) | 15:58 |
smcginnis | sheel: Thanks for spending time on it. Always worth considering. | 15:58 |
sheel | :) | 15:58 |
*** jdurgin1 has joined #openstack-cinder | 15:58 | |
hemna | mornin | 16:01 |
*** esker has joined #openstack-cinder | 16:01 | |
smcginnis | hemna: howdy | 16:02 |
diablo_rojo | hemna: Morning :) | 16:02 |
*** eharney has joined #openstack-cinder | 16:02 | |
*** garthb has joined #openstack-cinder | 16:02 | |
*** arch-nemesis has joined #openstack-cinder | 16:02 | |
*** sinese_ has quit IRC | 16:04 | |
*** lpetrut has quit IRC | 16:04 | |
Swanson | hemna: happy National Home Warranty Day Eve. | 16:04 |
*** alonma has joined #openstack-cinder | 16:04 | |
smcginnis | Wuh | 16:04 |
hemna | ?! | 16:04 |
smcginnis | Swanson: You suck for making me waste time on googling this: http://www.2-10.com/blog/announcing-the-first-annual-national-home-warranty-day/ | 16:05 |
Swanson | Tomorrow is National Home Warranty Day so today is National Home Warranty Day Eve. It's a greeting. | 16:05 |
smcginnis | :) | 16:05 |
smcginnis | hemna: I'll walk over and smack him for you. | 16:06 |
*** JoseMello has joined #openstack-cinder | 16:06 | |
Swanson | Today is something like national bagel day. Of which I'm seeing none around. | 16:06 |
Swanson | So obviously we don't celebrate that here. | 16:06 |
smcginnis | Swanson: You should fix that. | 16:07 |
Swanson | smcginnis, I'll just submit a national no bagels day. Doesn't seem too hard to get one in the old registry. | 16:07 |
*** mvk has quit IRC | 16:07 | |
openstackgerrit | Merged openstack/cinder: Volume create fails with FakeISCSIDriver https://review.openstack.org/277619 | 16:07 |
hemna | :) | 16:09 |
*** boris-42 has joined #openstack-cinder | 16:11 | |
jgriffith | Swanson: problem solved on my end | 16:14 |
*** krtaylor has joined #openstack-cinder | 16:14 | |
jgriffith | Swanson: ham, egg and cheese on an everything!! Mmmmmmmm | 16:14 |
*** laughterwym has joined #openstack-cinder | 16:14 | |
*** jungleboyj has quit IRC | 16:14 | |
smcginnis | :) | 16:16 |
Swanson | jgriffith, I hope you enjoyed that. Off to the vending machine, I guess. | 16:18 |
*** crose has joined #openstack-cinder | 16:18 | |
jgriffith | Swanson: yuck :( | 16:19 |
openstackgerrit | Chaozhe Chen(ccz) proposed openstack/python-cinderclient: Fix some flake8 violations https://review.openstack.org/277925 | 16:19 |
*** isaacb has joined #openstack-cinder | 16:20 | |
*** belmoreira has quit IRC | 16:21 | |
*** alonma has quit IRC | 16:23 | |
*** ntpttr has joined #openstack-cinder | 16:25 | |
*** knikolla has quit IRC | 16:27 | |
*** knikolla has joined #openstack-cinder | 16:27 | |
*** vgridnev has quit IRC | 16:27 | |
*** jungleboyj has joined #openstack-cinder | 16:29 | |
*** alonma has joined #openstack-cinder | 16:30 | |
*** avihou has joined #openstack-cinder | 16:32 | |
*** savihou has quit IRC | 16:34 | |
*** laughterwym has quit IRC | 16:36 | |
*** knomura has quit IRC | 16:36 | |
*** laughterwym has joined #openstack-cinder | 16:37 | |
*** savihou has joined #openstack-cinder | 16:38 | |
*** avihou has quit IRC | 16:40 | |
*** salv-orl_ has joined #openstack-cinder | 16:41 | |
*** laughterwym has quit IRC | 16:43 | |
*** krtaylor has quit IRC | 16:44 | |
*** salv-orlando has quit IRC | 16:44 | |
scottda | If my 'git review' of a patch has picked up a bunch of new commits from master, I can just squash them and submit, right? It's just rebasing against master? | 16:45 |
smcginnis | scottda: Doesn't it just merge your code with master? | 16:45 |
smcginnis | scottda: I thought squash was only needed if you locally had several commits that you wanted to combine into one. | 16:45 |
scottda | 16:46 | |
scottda | Yes, that's when I normally need to squash. But somehow I've picked up a few of the latest patches from master... | 16:46 |
smcginnis | Hmm, something doesn't seem right there. | 16:46 |
scottda | I agree. | 16:46 |
scottda | That's why I'm not pushing the button... | 16:47 |
scottda | https://www.irccloud.com/pastebin/3aKpwyRt/ | 16:48 |
*** esker has quit IRC | 16:49 | |
scottda | smcginnis: But those patches are on master, so it seems that if I squash them and submit, it will have no net effect | 16:50 |
*** knikolla has quit IRC | 16:50 | |
smcginnis | scottda: If you do git log does it look right? | 16:51 |
smcginnis | scottda: I thought the squash would try to put all of the changes into one commit. | 16:51 |
smcginnis | scottda: Did you have a rebase that didn't complete? | 16:51 |
scottda | I had a rebase, but I don't think it failed to complete | 16:52 |
scottda | You are right, squashing is wrong, that will lose the history | 16:52 |
smcginnis | All: BTW, eventlet release got yanked, so CI's might start failing now trying to stack and not being able to find the package. | 16:52 |
smcginnis | scottda: Maybe try git rebase --continue? | 16:52 |
smcginnis | scottda: Just to see what happens. | 16:52 |
*** jungleboyj has quit IRC | 16:52 | |
scottda | No rebase in progress? | 16:53 |
scottda | That's the message ^^^ | 16:53 |
smcginnis | patrickeast is a git pro. Maybe he knows? | 16:53 |
smcginnis | scottda: Otherwise I've redownloaded my patches in the past with git review -d [number], then tried rebasing again. | 16:54 |
scottda | Maybe the answer is "yes", submit the above commits. Which should preserve them as separate | 16:54 |
smcginnis | scottda: Maybe make sure your local master is up to date before trying to rebase on it? | 16:54 |
smcginnis | I think git review does an implicit rebase on to remote master, so maybe that's why? | 16:55 |
* smcginnis is grasping at straws. | 16:55 | |
*** knomura has joined #openstack-cinder | 16:55 | |
scottda | Thanks for the grasping :) | 16:55 |
*** krtaylor has joined #openstack-cinder | 16:55 | |
smcginnis | scottda: Or yeah, you could just answer Y and if it messes anything up just clean up after. :] | 16:56 |
scottda | What could go wrong? | 16:56 |
*** apoorvad has joined #openstack-cinder | 16:56 | |
smcginnis | Hehe, "what does this button do?" - BOOM | 16:56 |
scottda | "failed to push" | 16:57 |
scottda | OK, this seems familiar. I'll dig a bit more. Thanks smcginnis | 16:57 |
*** alonma has quit IRC | 16:57 | |
smcginnis | scottda: Leave your local branch, git checkout master; git pull; git review -d [patchid]; git rebase master | 16:58 |
*** alonma has joined #openstack-cinder | 16:58 | |
*** jgriffith is now known as jgriffith_away | 16:58 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Log stack trace for middleware faults https://review.openstack.org/277963 | 16:58 |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 16:59 |
sheel | dulek: hi | 16:59 |
hemna | CI = busted | 16:59 |
scottda | BOOM | 16:59 |
hemna | someone pushed a new version of eventlet out and remove the older version from pypi | 17:00 |
hemna | global requirements lists the older version as the upper limit. | 17:00 |
hemna | oops. | 17:00 |
smcginnis | hemna: Yep, actually deleted the old version of eventlet off of pypi. | 17:00 |
*** mragupat has quit IRC | 17:00 | |
sheel | dulek: could we discuss your comment on https://review.openstack.org/#/c/274589/? | 17:00 |
smcginnis | hemna: Bad form. | 17:00 |
*** timcl has quit IRC | 17:00 | |
hemna | smcginnis, save space = break the world. | 17:00 |
hemna | wee! | 17:00 |
smcginnis | hemna: Watch it all burn. | 17:00 |
*** mragupat has joined #openstack-cinder | 17:01 | |
hemna | kick back, drink your coffee and watch it all.... | 17:01 |
smcginnis | Well, if anyone was waiting for a window to do some CI maintenance, nows your time. :) | 17:01 |
scottda | smcginnis: Actually, I think that lost all my changes from today | 17:01 |
hemna | :) | 17:01 |
smcginnis | scottda: You didn't delete your original branch, did you? | 17:01 |
scottda | I was on master | 17:02 |
*** alonma has quit IRC | 17:02 | |
smcginnis | scottda: As hemna would say: pewp | 17:02 |
scottda | ha | 17:02 |
scottda | No big. I can fix | 17:03 |
smcginnis | scottda: Hopefully it wasn't too much. | 17:03 |
* patrickeast catching up | 17:03 | |
*** raildo-afk is now known as raildo | 17:03 | |
*** alonma has joined #openstack-cinder | 17:04 | |
scottda | no, not to bad | 17:04 |
patrickeast | We're they committed at some point? | 17:04 |
scottda | patrickeast: locally | 17:04 |
scottda | But not on a branch. on master | 17:04 |
smcginnis | Pro tip: work on branches. :P | 17:04 |
patrickeast | That's fine, look at git reflog | 17:04 |
patrickeast | And check-out the sha id of when they were on master | 17:05 |
scottda | OH yeah! reflog magic. | 17:05 |
*** savihou has quit IRC | 17:06 | |
smcginnis | scottda: See, I told you patrickeast was a git pro. ;) | 17:06 |
*** knomura has quit IRC | 17:07 | |
*** isaacb has quit IRC | 17:07 | |
*** alonma has quit IRC | 17:08 | |
*** dan_nguyen has joined #openstack-cinder | 17:08 | |
openstackgerrit | Raildo Mascena proposed openstack/cinder: Include allocated quota value in the quota reserve https://review.openstack.org/240228 | 17:08 |
scottda | patrickeast: That did the trick. Thanks! | 17:09 |
*** chris_morrell has joined #openstack-cinder | 17:09 | |
patrickeast | np | 17:09 |
*** mylu has quit IRC | 17:10 | |
scottda | I shall send you a fruit log this Christmas | 17:10 |
*** alonma has joined #openstack-cinder | 17:11 | |
patrickeast | haha | 17:12 |
*** arecknag has quit IRC | 17:14 | |
*** jungleboyj has joined #openstack-cinder | 17:15 | |
*** leeantho has joined #openstack-cinder | 17:15 | |
*** dramakri has joined #openstack-cinder | 17:15 | |
*** alonma has quit IRC | 17:16 | |
*** martyturner has joined #openstack-cinder | 17:18 | |
*** mudassirlatif has joined #openstack-cinder | 17:19 | |
*** alonma has joined #openstack-cinder | 17:20 | |
openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector api in VMEM cinder driver https://review.openstack.org/277644 | 17:20 |
-openstackstatus- NOTICE: Gerrit is restarting now, to alleviate current performance impact and WebUI errors. | 17:23 | |
diablo_rojo | ^^awesome | 17:24 |
*** knomura has joined #openstack-cinder | 17:24 | |
*** alonma has quit IRC | 17:24 | |
*** alonma has joined #openstack-cinder | 17:26 | |
*** amoturi has joined #openstack-cinder | 17:26 | |
*** mylu has joined #openstack-cinder | 17:27 | |
*** knomura has quit IRC | 17:28 | |
*** bardia has joined #openstack-cinder | 17:29 | |
*** alonma has quit IRC | 17:30 | |
*** jdurgin1 has quit IRC | 17:30 | |
openstackgerrit | Ryan McNair proposed openstack/cinder: Split out NestedQuotas into a separate driver https://review.openstack.org/274825 | 17:31 |
*** e0ne has quit IRC | 17:32 | |
*** mragupat has quit IRC | 17:32 | |
*** mragupat has joined #openstack-cinder | 17:33 | |
*** knikolla has joined #openstack-cinder | 17:33 | |
amoturi | Hi Sean, could u take a look at https://review.openstack.org/#/c/273741/ The CI results are now posted. | 17:33 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3 https://review.openstack.org/277983 | 17:33 |
openstackgerrit | Scott DAngelo proposed openstack/cinder: cinder-api-microversions code https://review.openstack.org/224910 | 17:34 |
*** mylu has quit IRC | 17:34 | |
*** alonma has joined #openstack-cinder | 17:34 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3 https://review.openstack.org/277983 | 17:34 |
*** jdurgin1 has joined #openstack-cinder | 17:36 | |
*** jdurgin1 has left #openstack-cinder | 17:36 | |
*** timcl has joined #openstack-cinder | 17:36 | |
*** mylu has joined #openstack-cinder | 17:36 | |
*** mragupat has quit IRC | 17:36 | |
*** mragupat has joined #openstack-cinder | 17:37 | |
*** Vasanthi has joined #openstack-cinder | 17:38 | |
*** alonma has quit IRC | 17:38 | |
*** laughterwym has joined #openstack-cinder | 17:39 | |
*** dustins_ has joined #openstack-cinder | 17:40 | |
*** dustins has quit IRC | 17:41 | |
jungleboyj | smcginnis: When you have a chance can you put this in? https://review.openstack.org/#/c/275927/ :-) Thanks. | 17:43 |
openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Add an implementation for validate_connector https://review.openstack.org/277644 | 17:43 |
*** laughterwym has quit IRC | 17:44 | |
*** sinese_ has joined #openstack-cinder | 17:46 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API contribs to Python 3 https://review.openstack.org/277861 | 17:50 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port utils.safe_minidom_parse_string() to Python 3 https://review.openstack.org/277983 | 17:50 |
openstackgerrit | Victor Stinner proposed openstack/cinder: py3: Fix usage of JSON in API contrib https://review.openstack.org/277989 | 17:50 |
*** ntt has quit IRC | 17:50 | |
*** ildikov has quit IRC | 17:53 | |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API v1 and v2 to Python 3 https://review.openstack.org/277835 | 17:53 |
*** vgridnev has joined #openstack-cinder | 17:53 | |
*** timcl1 has joined #openstack-cinder | 17:53 | |
*** alkhodos has joined #openstack-cinder | 17:53 | |
*** knikolla has quit IRC | 17:54 | |
*** lpetrut has joined #openstack-cinder | 17:55 | |
*** jordanP has joined #openstack-cinder | 17:55 | |
*** timcl has quit IRC | 17:56 | |
*** bardia has quit IRC | 17:56 | |
*** mylu has quit IRC | 17:56 | |
*** mylu has joined #openstack-cinder | 17:57 | |
*** knikolla has joined #openstack-cinder | 17:58 | |
*** mylu has quit IRC | 17:58 | |
*** jwang has joined #openstack-cinder | 18:00 | |
*** jwang_ has quit IRC | 18:00 | |
*** akshai has quit IRC | 18:00 | |
*** mylu has joined #openstack-cinder | 18:00 | |
kmartin__ | jungleboyj, do you want to add a release note for this? https://review.openstack.org/#/c/275927/ :) | 18:00 |
*** akshai has joined #openstack-cinder | 18:02 | |
*** jordanP has quit IRC | 18:03 | |
*** mylu has quit IRC | 18:03 | |
*** ntpttr has quit IRC | 18:06 | |
*** dramakri has quit IRC | 18:06 | |
openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 18:07 |
*** mylu has joined #openstack-cinder | 18:08 | |
*** alonma has joined #openstack-cinder | 18:10 | |
*** dustins_ has quit IRC | 18:11 | |
*** alonma has quit IRC | 18:15 | |
*** dustins has joined #openstack-cinder | 18:16 | |
*** e0ne has joined #openstack-cinder | 18:17 | |
*** alonma has joined #openstack-cinder | 18:18 | |
*** martyturner has quit IRC | 18:22 | |
*** alonma has quit IRC | 18:22 | |
*** martyturner has joined #openstack-cinder | 18:22 | |
*** timcl1 has quit IRC | 18:23 | |
*** delattec has joined #openstack-cinder | 18:23 | |
*** cdelatte has quit IRC | 18:26 | |
*** alonma has joined #openstack-cinder | 18:28 | |
smcginnis | jungleboyj: At least you name yourself in those statements. :) | 18:28 |
*** dustins has quit IRC | 18:28 | |
*** dustins has joined #openstack-cinder | 18:28 | |
*** dustins has quit IRC | 18:29 | |
jungleboyj | kmartin__: You are poking a bear today my friend. | 18:29 |
kmartin__ | jungleboyj, :) | 18:30 |
jungleboyj | kmartin__: :-) | 18:30 |
*** esker has joined #openstack-cinder | 18:30 | |
jungleboyj | smcginnis: Yes, easy to figure out who did it that way. | 18:30 |
smcginnis | jungleboyj: Hey, if you're consistent on that, we really could add a hacking check. | 18:31 |
jungleboyj | :-) I will write that up. | 18:32 |
*** mragupat_ has joined #openstack-cinder | 18:32 | |
*** alonma has quit IRC | 18:32 | |
*** jgriffith_away is now known as jgriffith | 18:34 | |
*** mragupat has quit IRC | 18:36 | |
*** ildikov has joined #openstack-cinder | 18:36 | |
*** akshai has quit IRC | 18:36 | |
e0ne | hemna: hi. are you around? I've got a question about brickclient | 18:37 |
*** akshai has joined #openstack-cinder | 18:38 | |
*** avishay has quit IRC | 18:39 | |
*** ociuhandu has quit IRC | 18:39 | |
jungleboyj | patrickeast: kmartin__ What a bunch of wise guys! | 18:41 |
hemna | e0ne, hey | 18:42 |
e0ne | hemna: according to my patch with attach/dettach | 18:43 |
e0ne | 1) https://review.openstack.org/277788 - my patch for NFS connector | 18:44 |
e0ne | and 2) are we going to add rootwrap for brickclient for Mitaka release? | 18:44 |
hemna | e0ne, https://review.openstack.org/#/c/277224 | 18:44 |
e0ne | w/o rootwrap I need to do few hacks and it will be very ugly UX | 18:44 |
hemna | I'm working on helping that | 18:44 |
*** ChubYann has joined #openstack-cinder | 18:44 | |
hemna | but it's currently not working :( | 18:44 |
e0ne | :( | 18:45 |
*** mylu has quit IRC | 18:45 | |
e0ne | I can take a look on it tomorrow | 18:45 |
e0ne | but in case, if we'll not merge it in Mitaka | 18:45 |
hemna | https://review.openstack.org/#/c/277957/ | 18:45 |
hemna | I had to post that patch against privsep fwiw | 18:45 |
e0ne | what are we going to to do with attach/detach stuff in brickclient? | 18:45 |
hemna | to fix one of it's problems | 18:46 |
hemna | what do you mean? | 18:46 |
hemna | the cinder client extension is what you mean I presume ? | 18:46 |
*** rlrossit has quit IRC | 18:46 | |
e0ne | yes | 18:46 |
e0ne | python-brick-cinderclient-ext | 18:46 |
Vasanthi | Any idea why all CIs are failing with this error? 2016-02-09 16:25:43.641 | Could not find a version that satisfies the requirement eventlet===0.18.1 (from -c /opt/stack/new/requirements/upper-constraints.txt (line 122)) (from versions: 0.2, 0.5.3, 0.6.1, 0.7, 0.8, 0.8.16, 0.9.17, 0.10.0, 0.11.0, 0.12.1, 0.13.0, 0.14.0, 0.15.2, 0.16.1, 0.17.4, 0.18.2) 2016-02-09 16:25:43.662 | No matching distribution found for eventl | 18:46 |
*** chhavi has quit IRC | 18:47 | |
*** bardia has joined #openstack-cinder | 18:47 | |
hemna | Vasanthi, there is a fix working it's way through the gate | 18:47 |
e0ne | Vasanthi: there was an issue with eventlet==0.18. maybe 0.18.1 it not available on all pypi mirrors | 18:47 |
hemna | someone released a new version of eventlet 0.18.2, and then deleted 0.18.1 | 18:47 |
hemna | which broke the world. | 18:47 |
e0ne | :( | 18:47 |
*** delatte has joined #openstack-cinder | 18:48 | |
diablo_rojo | patrickeast: How did testing go of the multipath brick patch go yesterday? :) | 18:49 |
*** mriedem has quit IRC | 18:49 | |
*** wN has quit IRC | 18:50 | |
kmartin__ | Vasanthi, https://review.openstack.org/#/c/277912/ | 18:51 |
*** delattec has quit IRC | 18:51 | |
*** mriedem has joined #openstack-cinder | 18:51 | |
e0ne | hemna: https://review.openstack.org/#/c/263744/9/brick_cinderclient_ext/brick_utils.py - line #34 | 18:51 |
hemna | e0ne, ew that is ugly | 18:52 |
hemna | so....yah | 18:52 |
hemna | :P | 18:52 |
e0ne | rootwrap:( | 18:52 |
*** timcl has joined #openstack-cinder | 18:52 | |
hemna | I'm not sure how else to do that | 18:52 |
hemna | w/o privsep fix | 18:52 |
e0ne | the same for me | 18:53 |
hemna | http://paste.openstack.org/show/486460/ | 18:53 |
hemna | so that's the latest w/ the privsep patch | 18:53 |
e0ne | I'll try to take a look on privsep patch tomorrow. | 18:53 |
hemna | I guess I can try and debug it and see if I can offer up a fix | 18:53 |
hemna | I'd like to get this working ASAP | 18:53 |
e0ne | probably I will help somehow | 18:53 |
hemna | not sure what timezone gus is in | 18:53 |
*** rlrossit has joined #openstack-cinder | 18:54 | |
e0ne | hemna: the error means that you try access to the file w/o sudo - but I'm sure you know it | 18:54 |
*** wN has joined #openstack-cinder | 18:55 | |
*** wN has joined #openstack-cinder | 18:55 | |
*** martyturner has quit IRC | 18:56 | |
*** krtaylor has quit IRC | 18:57 | |
*** cknight has quit IRC | 19:00 | |
hemna | well if you cat the file w/o sudo it says permission denied | 19:00 |
hemna | not file not found | 19:00 |
hemna | cat /etc/iscsi/initiatorname.iscsi | 19:00 |
hemna | cat: /etc/iscsi/initiatorname.iscsi: Permission denied | 19:00 |
*** akshai has quit IRC | 19:01 | |
*** akshai has joined #openstack-cinder | 19:02 | |
*** martyturner has joined #openstack-cinder | 19:02 | |
*** raildo is now known as raildo-afk | 19:02 | |
e0ne | hemna: please, share brick_initiator.py | 19:02 |
e0ne | hemna: in shell | 19:03 |
*** dramakri has joined #openstack-cinder | 19:03 | |
e0ne | hemna: in python I'd got "file not found" error | 19:03 |
*** dramakri has quit IRC | 19:03 | |
*** cknight has joined #openstack-cinder | 19:03 | |
hemna | https://github.com/WaltHP/diediedie | 19:03 |
hemna | https://github.com/WaltHP/diediedie/blob/master/diediedie/brick_initiator.py | 19:03 |
*** ociuhandu has joined #openstack-cinder | 19:03 | |
smcginnis | :) | 19:03 |
hemna | :) | 19:04 |
hemna | kinda looks like sudo is dropped | 19:05 |
hemna | hrmm | 19:05 |
patrickeast | diablo_rojo: hey, sorry haven't had a chance to test it out, got side tracked with other stuff | 19:06 |
diablo_rojo | patrickeast: No worries. Think you have time today or tomorrow? The deadline is Friday for brick patches. | 19:07 |
*** jungleboyj has quit IRC | 19:07 | |
patrickeast | diablo_rojo: yea, should be able to | 19:08 |
diablo_rojo | patrickeast: Cool, so long as we are all happy with it by Friday we should be good. I appreciate the help :) | 19:08 |
*** krtaylor has joined #openstack-cinder | 19:09 | |
*** mylu has joined #openstack-cinder | 19:12 | |
alkhodos | Hi everyone! Can some of the cores take a look at this patch please? https://review.openstack.org/#/c/271537/ Thank you. | 19:14 |
ildikov | hemna: hi | 19:14 |
e0ne | hemna: too late for me to be useful on it. will try tomorrow morning | 19:15 |
ildikov | hemna: I just wanted to ask that when do you think you can write as that mail about your approach for disconnect_volume for multiattach? | 19:15 |
*** timcl1 has joined #openstack-cinder | 19:15 | |
hemna | ildikov, hey | 19:15 |
hemna | ildikov, crap, I'll do that today. | 19:15 |
ildikov | hemna: np, today is fine too :) | 19:16 |
*** esker has quit IRC | 19:16 | |
ildikov | hemna: I just would like to have the thread started | 19:16 |
ildikov | hemna: thanks! | 19:16 |
hemna | I'll start it now. | 19:16 |
hemna | sorry about that | 19:16 |
*** mylu has quit IRC | 19:17 | |
*** esker has joined #openstack-cinder | 19:17 | |
*** dims_ has joined #openstack-cinder | 19:17 | |
*** dims has quit IRC | 19:17 | |
*** timcl has quit IRC | 19:18 | |
ildikov | hemna: no probs, I don't think one day matters here | 19:18 |
ildikov | hemna: just don't tell anyone I said that ;) | 19:18 |
*** esker has quit IRC | 19:19 | |
*** mylu has joined #openstack-cinder | 19:20 | |
hemna | I'll get it out here before lunch. | 19:20 |
hemna | bad hemna | 19:20 |
*** mylu has quit IRC | 19:24 | |
*** dims_ has quit IRC | 19:27 | |
*** alonma has joined #openstack-cinder | 19:28 | |
*** JoseMello has quit IRC | 19:28 | |
*** SDub has joined #openstack-cinder | 19:29 | |
openstackgerrit | Kendall Nelson proposed openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 19:29 |
SDub | Struggling with this bug right now using CirrOS 0.3.4 and Juno: https://bugs.launchpad.net/cirros/+bug/1312199 | 19:29 |
openstack | Launchpad bug 1312199 in OpenStack Compute (nova) icehouse "cirros 0.3.1 fails to boot" [High,Fix released] - Assigned to Attila Fazekas (afazekas) | 19:29 |
SDub | on all accounts this issue shouldn't be happening but for some reason it is. | 19:29 |
*** dims has joined #openstack-cinder | 19:30 | |
openstackgerrit | Merged openstack/cinder: Added Keystone and RequestID headers to CORS middleware https://review.openstack.org/265409 | 19:31 |
*** alonma has quit IRC | 19:32 | |
openstackgerrit | Merged openstack/cinder: Adds support for configuring zoning in a virtual fabric https://review.openstack.org/275407 | 19:33 |
*** ChubYann has quit IRC | 19:33 | |
apoorvad | Hello cores! Please take a loot at the reviews https://review.openstack.org/#/c/268370/ and https://review.openstack.org/#/c/261126/ | 19:34 |
*** SDub has left #openstack-cinder | 19:34 | |
*** esker has joined #openstack-cinder | 19:34 | |
*** esker has quit IRC | 19:34 | |
openstackgerrit | Merged openstack/cinder: Port backup drivers to Python 3 https://review.openstack.org/277778 | 19:34 |
*** esker has joined #openstack-cinder | 19:34 | |
*** alonma has joined #openstack-cinder | 19:34 | |
openstackgerrit | Merged openstack/cinder: DRBD: Fix arguments for resize_volume DBus API call. https://review.openstack.org/277314 | 19:35 |
*** sheel has quit IRC | 19:37 | |
ildikov | hemna: :) | 19:37 |
*** alonma has quit IRC | 19:39 | |
*** can8dnSix has joined #openstack-cinder | 19:41 | |
*** dims has quit IRC | 19:41 | |
*** dims has joined #openstack-cinder | 19:42 | |
*** alonma has joined #openstack-cinder | 19:42 | |
*** jungleboyj has joined #openstack-cinder | 19:43 | |
*** ChubYann has joined #openstack-cinder | 19:44 | |
*** mriedem has quit IRC | 19:45 | |
*** dims_ has joined #openstack-cinder | 19:46 | |
*** alonma has quit IRC | 19:46 | |
*** mylu has joined #openstack-cinder | 19:46 | |
*** mriedem has joined #openstack-cinder | 19:47 | |
*** dims has quit IRC | 19:49 | |
*** e0ne has quit IRC | 19:49 | |
*** fernnest__ has quit IRC | 19:52 | |
hemna | ildikov, ok sent | 19:52 |
hemna | hopefully that makes sense. | 19:52 |
ildikov | hemna: cool, thanks much! | 19:52 |
*** fernnest has quit IRC | 19:52 | |
*** fernnest has joined #openstack-cinder | 19:52 | |
ildikov | hemna: I will check, I cannot promise I can put my brain together today to write a reply that makes sense, so I might do it tomorrow morning instead :) | 19:53 |
*** mylu has quit IRC | 19:53 | |
hemna | ok no problemo. | 19:53 |
hemna | I just described 1 single issue, which is determining how to know when to call brick's disconnect_volume or not. | 19:53 |
hemna | I figured it was best to not confuse everyone with the other somewhat related issues of storing the connector, etc. | 19:54 |
*** esker has quit IRC | 19:56 | |
*** dims_ has quit IRC | 19:57 | |
*** can8dnSix has quit IRC | 19:59 | |
*** dustins has joined #openstack-cinder | 20:00 | |
*** ndipanov has quit IRC | 20:02 | |
*** akshai has quit IRC | 20:02 | |
*** timcl1 has quit IRC | 20:03 | |
*** timcl has joined #openstack-cinder | 20:04 | |
*** jgriffith has quit IRC | 20:05 | |
openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage volume in ScaleIO driver https://review.openstack.org/268344 | 20:05 |
*** akshai has joined #openstack-cinder | 20:05 | |
*** jgriffith has joined #openstack-cinder | 20:06 | |
*** jgriffith is now known as Guest55421 | 20:06 | |
*** rcernin has joined #openstack-cinder | 20:08 | |
smcginnis | hemna: Good write up. | 20:13 |
*** jungleboyj has quit IRC | 20:13 | |
*** alonma has joined #openstack-cinder | 20:18 | |
*** krtaylor has quit IRC | 20:19 | |
*** martyturner has quit IRC | 20:19 | |
*** martyturner has joined #openstack-cinder | 20:21 | |
*** alonma has quit IRC | 20:23 | |
openstackgerrit | Merged openstack/cinder: Move wsgi to oslo_service.wsgi https://review.openstack.org/258541 | 20:24 |
*** Guest55421 is now known as jgriffith | 20:24 | |
openstackgerrit | Merged openstack/cinder: hacking: Fix false positive in C302 check https://review.openstack.org/277841 | 20:25 |
*** amoturi has left #openstack-cinder | 20:25 | |
jgriffith | did anybody see the requirements update yet? | 20:28 |
jgriffith | ie did it merge and I missed it :) | 20:29 |
smcginnis | jgriffith: Nope | 20:29 |
jgriffith | hmm | 20:29 |
jgriffith | annoying | 20:29 |
smcginnis | jgriffith: Yep | 20:29 |
jgriffith | of course, I'm assuming it's in process | 20:29 |
smcginnis | jgriffith: Top of the gate queue right now. | 20:29 |
jgriffith | smcginnis: :) thanks | 20:29 |
smcginnis | jgriffith: Looks like it's done and just wrapping up. Says 0 minutes left. | 20:29 |
*** mylu has joined #openstack-cinder | 20:29 | |
smcginnis | jgriffith: Taking way too long! :) | 20:29 |
*** raildo-afk is now known as raildo | 20:29 | |
*** ndipanov has joined #openstack-cinder | 20:31 | |
*** krtaylor has joined #openstack-cinder | 20:31 | |
*** shyama has quit IRC | 20:34 | |
*** esp has joined #openstack-cinder | 20:35 | |
*** salv-orl_ has quit IRC | 20:36 | |
*** esp has quit IRC | 20:39 | |
*** mylu has quit IRC | 20:40 | |
*** mylu has joined #openstack-cinder | 20:40 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during driver initialization https://review.openstack.org/271424 | 20:42 |
openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: [LVM] Restore target config during ensure_export https://review.openstack.org/271424 | 20:42 |
mc_nair | smcginnis, jgriffith, DuncanT - the nested quota updates are ready for review at https://review.openstack.org/#/c/274825/. Feedback certainly welcome on the code or overall approach | 20:46 |
*** esp has joined #openstack-cinder | 20:47 | |
smcginnis | mc_nair: Thanks, I'll try to take a look soon. | 20:48 |
*** mylu has quit IRC | 20:48 | |
mc_nair | shweet | 20:48 |
mc_nair | unfortunately it's a decent amount of code :/ but some of that is test coverage + releasenotes | 20:49 |
*** knikolla has quit IRC | 20:49 | |
*** knikolla has joined #openstack-cinder | 20:50 | |
*** esp has quit IRC | 20:51 | |
smcginnis | Yay, my CI can finally stack again! | 20:52 |
*** salv-orlando has joined #openstack-cinder | 20:53 | |
*** esker has joined #openstack-cinder | 20:53 | |
*** krtaylor has quit IRC | 20:53 | |
*** akerr has quit IRC | 20:54 | |
hemna | smcginnis, w00t! | 20:54 |
*** chris_morrell has quit IRC | 20:58 | |
jgriffith | smcginnis: sweeeet! | 20:59 |
jgriffith | smcginnis: now to send nasty emails to the eventlet team :) | 20:59 |
*** esker has quit IRC | 20:59 | |
smcginnis | jgriffith: I'm staying out of that one! | 20:59 |
Swanson | jgriffith, repl v2.1 patchset 4 due up today? | 21:00 |
jgriffith | smcginnis: well that's no fun :) | 21:00 |
jgriffith | Swanson: yes... sorry I'm behind | 21:00 |
jgriffith | Swanson: I'm sick today and keep nodding off :( | 21:00 |
smcginnis | Ugg | 21:01 |
Swanson | jgriffith, Not a problem. I'm off to a dr's appt myself in a few. Was just wondering if I should start stacking with what is up there now or wait until later. | 21:01 |
Swanson | jgriffith, get better, btw. | 21:01 |
jgriffith | Swanson: thanks | 21:01 |
jgriffith | well... there *will* be an update sometime today :) | 21:02 |
jgriffith | depending on your TZ :) | 21:02 |
*** dims has joined #openstack-cinder | 21:03 | |
*** krotscheck is now known as krotscheck_dcm | 21:03 | |
*** e0ne has joined #openstack-cinder | 21:03 | |
*** knikolla has quit IRC | 21:03 | |
jgriffith | sigh... 2 email inboxes, 2 calendars, 2 vpns; not fun | 21:04 |
jgriffith | even with forwarding... still painful | 21:04 |
*** timcl has quit IRC | 21:06 | |
smcginnis | jgriffith: Oh, just wait for the aquisition to get into full swing. | 21:06 |
*** krtaylor has joined #openstack-cinder | 21:07 | |
jgriffith | smcginnis: sigh | 21:11 |
jgriffith | smcginnis: maybe I can convince them to just move everything to Google for Biz :) | 21:11 |
smcginnis | jgriffith: Hah, sure. Let me know how that goes. :P | 21:12 |
jgriffith | smcginnis: yeah... kinda what I'm thinking. But hey! It would be EPIC!!! I could put it on my resume :) | 21:12 |
smcginnis | True! | 21:12 |
ildikov | hemna: read the mail finally, I think it is a good start, thanks! | 21:15 |
ildikov | hemna: I will reply to it tomorrow | 21:15 |
scottda | yes hemna , nice work | 21:16 |
ildikov | scottda: +1 :) | 21:16 |
*** lpetrut has quit IRC | 21:18 | |
*** alonma has joined #openstack-cinder | 21:19 | |
*** martyturner has quit IRC | 21:20 | |
*** alonma has quit IRC | 21:23 | |
*** martyturner has joined #openstack-cinder | 21:23 | |
*** erlon has quit IRC | 21:26 | |
*** alonma has joined #openstack-cinder | 21:27 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Move integrated tests to 'functional' directory https://review.openstack.org/267801 | 21:29 |
smcginnis | mc_nair: Thanks for jumping in there. | 21:31 |
*** jgriffith is now known as jgriffith_away | 21:31 | |
mc_nair | smcginnis: sure thing. Some common implementation sounds like the way to go in the future, hoping that if we fix up our nested quota support that could be leveraged as a starting base | 21:31 |
mc_nair | thanks diablo_rojo for tipping me off about the convo | 21:32 |
*** alonma has quit IRC | 21:32 | |
mc_nair | but regardless, I think we need to blaze forward on the current fixup plan for our existing nested quotas, since we've already released and are bit behind the 8ball because of that | 21:32 |
smcginnis | mc_nair: I agree. | 21:33 |
diablo_rojo | mc_nair: No problem. I had said basically the same as what you did that they should look at implementing quotas and then extending it, but I figured if they started asking more questions it would be better to have you there | 21:33 |
smcginnis | mc_nair: Then hopefully whatever that ends up looking like fits in with whatever cross project lands on. | 21:33 |
*** alonma has joined #openstack-cinder | 21:34 | |
*** baumann has quit IRC | 21:34 | |
mc_nair | smcginnis: yea - think if it's ends up coded nicely we should be able to pull out most of the logic and generalize it | 21:35 |
*** baumann has joined #openstack-cinder | 21:35 | |
*** tpsilva has quit IRC | 21:36 | |
*** alonma has quit IRC | 21:38 | |
openstackgerrit | Merged openstack/python-cinderclient: Remove debug statement https://review.openstack.org/275927 | 21:38 |
*** vgridnev has quit IRC | 21:40 | |
*** alonma has joined #openstack-cinder | 21:40 | |
ildikov | hemna: scottda: still here? | 21:40 |
scottda | hi | 21:40 |
ildikov | hey :) | 21:40 |
ildikov | I just had some thoughts about hemna's approach about which I wanted to ask questions before writing a silly mail :) | 21:41 |
scottda | ok | 21:42 |
*** laughterwym has joined #openstack-cinder | 21:42 | |
ildikov | basically if I remember correctly our issue with storing the target was that when initialize_connection is called we don't have the instance or smth like this | 21:42 |
ildikov | do I remember correctly? | 21:42 |
scottda | Yes, you are correct | 21:42 |
*** cknight has quit IRC | 21:43 | |
ildikov | ok, so what I was wondering about is that what if we add the possibility to the Cinder API to provide this information after attach? | 21:43 |
ildikov | I mean Nova gets the info, it should have everything and it can call out to Cinder to feed back the info if the attach is successful | 21:44 |
scottda | Sure, that's possible | 21:44 |
ildikov | we could store the info as an additional item for the attachment in the attachments list in the volume info | 21:44 |
*** alonma has quit IRC | 21:44 | |
ildikov | and then both components could easily check who's attached to which target | 21:45 |
ildikov | if the info is missing because of race condition or any other issue we can still fall back to those options we discussed already | 21:45 |
scottda | Yes, that all makes sense to me. | 21:45 |
ildikov | it could work with live migration too I think | 21:46 |
scottda | hemna: might have feedback, but it sounds like a possibility worth thinking about. | 21:46 |
ildikov | not sure about race conditions in that case though :( | 21:46 |
*** laughterwym has quit IRC | 21:46 | |
ildikov | ok, cool tnx | 21:46 |
ildikov | I wanted only a base sanity check, I will add this to the mail thread | 21:47 |
scottda | You do indeed sound sane :) | 21:47 |
ildikov | cool, I will refer to you if someone proves the opposite ;) | 21:47 |
*** raildo is now known as raildo-afk | 21:49 | |
*** nkrinner has quit IRC | 21:50 | |
*** martyturner has quit IRC | 21:54 | |
openstackgerrit | Merged openstack/cinder: Zfssaiscsi driver should return target_lun as int https://review.openstack.org/273741 | 21:54 |
*** martyturner has joined #openstack-cinder | 21:56 | |
*** alonma has joined #openstack-cinder | 21:57 | |
*** esker has joined #openstack-cinder | 22:00 | |
*** alonma has quit IRC | 22:01 | |
*** jwcroppe has joined #openstack-cinder | 22:02 | |
hemna | back | 22:02 |
hemna | sup | 22:02 |
hemna | the issue with storing the connector is live migration | 22:03 |
hemna | we need a new Cinder api to have cinder update the attachment | 22:03 |
hemna | during live migration. | 22:03 |
hemna | and to do that, we need microversions to land | 22:04 |
hemna | .... | 22:04 |
hemna | ... | 22:04 |
scottda | new patch set for microversions is coming soon....just one small fix.... | 22:04 |
*** esker has quit IRC | 22:05 | |
ildikov | what I propose is definitely a new API | 22:05 |
*** alonma has joined #openstack-cinder | 22:05 | |
hemna | ildikov, I had discussed the same thing in an earlier ML thread | 22:05 |
ildikov | hemna: I added this idea now to this thread as well | 22:05 |
ildikov | hemna: what was the outcome of the previous? | 22:06 |
*** gouthamr has quit IRC | 22:06 | |
hemna | the outcome was, we couldn't do it in M, because we needed microversions. | 22:06 |
hemna | It's on my N TODO list | 22:06 |
ildikov | it works in N as well if we have microversion | 22:06 |
ildikov | your initial proposal is needed as a fallback option anyway IMHO | 22:07 |
*** mylu has joined #openstack-cinder | 22:08 | |
*** mylu has quit IRC | 22:09 | |
*** alonma has quit IRC | 22:09 | |
ildikov | hemna: I guess based on the outcome of the ML thread we can add this item to the TODO list for Cinder for N, right? | 22:10 |
hemna | ildikov, that's the hope yah | 22:10 |
ildikov | hemna: scottda: how far is microversion from flying? | 22:11 |
*** cknight has joined #openstack-cinder | 22:11 | |
*** martyturner has quit IRC | 22:11 | |
scottda | ildikov: Not sure. I had failures in CI that I couldn't repro on my desktop, but I've fixed some things since then. I've a new patch just waiting on a unit test fix. Might be in OK shape for review as early as today. | 22:12 |
*** sinese_ has quit IRC | 22:12 | |
ildikov | scottda: ok, that sounds promising | 22:12 |
scottda | Yes, but note that I did not make any promises :) | 22:13 |
ildikov | scottda: I just wasn't sure as hemna mentioned his TODO list | 22:13 |
scottda | haha, no , it's on my TODO list. | 22:13 |
ildikov | but then I think I misunderstood which item is on that list :) | 22:13 |
scottda | api microversions is on my list | 22:14 |
ildikov | ok, I see now :) | 22:14 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: WIP!!!!! Fix Rally job https://review.openstack.org/278119 | 22:14 |
ildikov | scottda: thanks for the clarifications | 22:14 |
scottda | New API to update Cinder connector info (for live migration) is on hemna | 22:14 |
hemna | scottda, +1 | 22:14 |
hemna | we need to be able to hand the connector at attach time, and the new update_attach api | 22:15 |
ildikov | is that new API equals to what I wrote above? | 22:15 |
ildikov | I imagined the whole process as update by having the connector_info empty when the attachment created and then update as many times as needed | 22:16 |
*** porrua has quit IRC | 22:17 | |
e0ne | it's needed not only for live migration | 22:17 |
e0ne | see also thread "Cleanly detaching volumes from failed nodes" | 22:17 |
ildikov | but this part looks like a detail | 22:17 |
scottda | ildikov: had the idea of updating the connector with the instance id using the new API.... | 22:18 |
scottda | This info is not known at attach time... | 22:18 |
scottda | But could be updated later by Nova | 22:18 |
apoorvad | smcginnis patrickeast hemna : my CI is failing on test_volume_create_get_update_delete_as_clone does anyone of u know what changed? http://openstack-ci.tintri.com/tintri/refs-changes-35-277835-3/console.log.out | 22:18 |
ildikov | e0ne: you mean the connector_info would be needed in the failed nodes case, right? | 22:19 |
*** mylu has joined #openstack-cinder | 22:19 | |
e0ne | ildikov: yes | 22:19 |
hemna | e0ne, ildikov it's the same data needed to detach | 22:19 |
*** martyturner has joined #openstack-cinder | 22:19 | |
ildikov | hemna: yeah exactly, so it's failed nodes and multiattach, where we definitely need this data | 22:20 |
hemna | when either 1) nova evacuate is called or 2) when nova has no record of the instance anymore and cinder still thinks it's attached. | 22:20 |
smcginnis | apoorvad: Volume driver reported an error: Failed to create snapshot for volume /tintri/openstack/volume-4e0d19d0-715e-4943-bdcf-9fec1561a9b4 | 22:20 |
smcginnis | apoorvad: Looks like it's something on your array side. | 22:20 |
ildikov | hemna: yeah, agreed | 22:20 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: WIP!!!!! Fix Rally job https://review.openstack.org/278119 | 22:20 |
apoorvad | smcginnis: thanks for the help. I will look into it | 22:21 |
hemna | -1 excessive !'s 4 !'s is ok. 5....excessive | 22:21 |
smcginnis | Hah | 22:21 |
smcginnis | apoorvad: Yeah, looking through your screen-c-vol log it looks like errors in there, but I don't see anything logging exactly why it failed. | 22:22 |
ildikov | hemna: ok, cool, then I think we are on the same page regarding the slightly longer term approach here, let's see the outcome of the ML thread | 22:24 |
ildikov | I will ping the Nova folks later if no response from that side soon | 22:24 |
ildikov | aaand going to have some sleep now :) | 22:24 |
smcginnis | apoorvad: Ah, I at least see why there's nothing useful in the log. | 22:24 |
hemna | ildikov, ok cool. thanks for the updates in the ML | 22:24 |
smcginnis | apoorvad: You check the requests response.status_code but none of the other data returned. | 22:25 |
e0ne | #NOTE (e0ne): do not use 5 '!' in one place | 22:25 |
ildikov | hemna: np, it would be good to have all the options captured there | 22:25 |
hemna | ildikov, +1 | 22:25 |
*** diablo_rojo has quit IRC | 22:25 | |
smcginnis | apoorvad: Might want to do what I did here: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/dell/dell_storagecenter_api.py#L248 | 22:26 |
apoorvad | smcginnis: checking on the array what's going on | 22:26 |
ildikov | hemna: have a nice rest of the day :) | 22:26 |
hemna | ildikov, thanks :) get some Zzzss | 22:26 |
ildikov | hemna: I stop re-inventing ideas now :) | 22:26 |
ildikov | laters | 22:26 |
apoorvad | smcginnis; that would be helpful. thanks | 22:27 |
*** crose has quit IRC | 22:30 | |
*** julim has quit IRC | 22:32 | |
*** esker has joined #openstack-cinder | 22:34 | |
*** wN has quit IRC | 22:34 | |
*** jgregor has quit IRC | 22:35 | |
*** baumann has quit IRC | 22:35 | |
*** dustins has quit IRC | 22:35 | |
*** dschoenb__ has joined #openstack-cinder | 22:35 | |
*** dschoenb__ is now known as dustins | 22:35 | |
notmorgan | hmm | 22:39 |
*** dims_ has joined #openstack-cinder | 22:39 | |
*** salv-orl_ has joined #openstack-cinder | 22:40 | |
*** dims has quit IRC | 22:40 | |
*** wN has joined #openstack-cinder | 22:41 | |
*** wN has joined #openstack-cinder | 22:41 | |
*** jwcroppe has quit IRC | 22:42 | |
*** rlrossit has quit IRC | 22:42 | |
*** salv-orlando has quit IRC | 22:43 | |
*** esker has quit IRC | 22:46 | |
*** martyturner has quit IRC | 22:47 | |
*** dramakri has joined #openstack-cinder | 22:50 | |
*** esker has joined #openstack-cinder | 22:52 | |
*** daneyon has joined #openstack-cinder | 22:55 | |
*** mylu has quit IRC | 22:56 | |
*** tongli has quit IRC | 22:56 | |
*** rcernin has quit IRC | 22:58 | |
openstackgerrit | Vincent Hou proposed openstack/cinder: Migration: take the local_path for the source volume https://review.openstack.org/250220 | 22:58 |
*** daneyon_ has quit IRC | 22:58 | |
*** mylu has joined #openstack-cinder | 22:59 | |
*** dustins has quit IRC | 23:00 | |
*** edtubill has quit IRC | 23:01 | |
*** edtubill has joined #openstack-cinder | 23:02 | |
*** eharney has quit IRC | 23:03 | |
*** alonma has joined #openstack-cinder | 23:05 | |
*** edtubill has quit IRC | 23:06 | |
*** xyang1 has quit IRC | 23:09 | |
*** alonma has quit IRC | 23:10 | |
*** alonma has joined #openstack-cinder | 23:11 | |
*** alonma has quit IRC | 23:16 | |
*** alonma has joined #openstack-cinder | 23:17 | |
*** daneyon_ has joined #openstack-cinder | 23:18 | |
*** daneyon has quit IRC | 23:21 | |
*** alonma has quit IRC | 23:22 | |
*** mriedem has quit IRC | 23:22 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: WIP! Fix Rally job https://review.openstack.org/278119 | 23:24 |
*** jungleboyj has joined #openstack-cinder | 23:26 | |
hemna | only 1! | 23:26 |
hemna | +A | 23:26 |
hemna | :P | 23:26 |
e0ne | wait, you forgot -1 on commit message:) | 23:27 |
hemna | crap | 23:27 |
*** alonma has joined #openstack-cinder | 23:29 | |
e0ne | hemna: https://review.openstack.org/277788 - here is one more good candidate to +2 or -1 ;) | 23:30 |
hemna | *click* | 23:30 |
e0ne | all gate-tempest-dsvm-full are broken:( | 23:30 |
hemna | CI has been borked all day today | 23:31 |
hemna | sux | 23:31 |
e0ne | it's good time to sleep | 23:32 |
e0ne | I've raised a question about gates in #openstack-qa | 23:32 |
e0ne | [01:32:16] <jlvillal>Trying to figure out if it is my patch or something else. | 23:33 |
hemna | bed time at 3:30pm. sounds good... | 23:33 |
hemna | :P | 23:33 |
*** alonma has quit IRC | 23:34 | |
hemna | e0ne, so the unmount | 23:34 |
e0ne | 1.34am | 23:34 |
hemna | I think folks have rejected that in the past for some reason for NFS | 23:34 |
hemna | eharney might have some insight | 23:35 |
e0ne | hemna: I have to ask him | 23:35 |
hemna | I don't really see an issue with it. | 23:35 |
e0ne | hemna: because for now, nova does it | 23:35 |
e0ne | hemna: and I'm not sure how volume migration will work | 23:35 |
e0ne | hemna: volume could be attached after migration | 23:36 |
e0ne | it will be easier to get it landed in os-brick before nova will use this connector | 23:37 |
*** hongbin has joined #openstack-cinder | 23:38 | |
*** prithiv has joined #openstack-cinder | 23:39 | |
*** jwcroppe has joined #openstack-cinder | 23:40 | |
hemna | yah I'm sure it's left behind at this point | 23:41 |
hemna | well | 23:41 |
hemna | I dunno, because the nfs driver doesn't use the brick connector | 23:41 |
gus | hemna: I'm UTC+11 (Sydney) fwiw. | 23:43 |
hemna | gus, hey! :) | 23:43 |
hemna | gus, ok cool thanks, I wasn't sure. | 23:43 |
gus | :) | 23:43 |
hemna | gus, did you happen to see my last comment on the brick patch? | 23:44 |
hongbin | Hi Cinder team, prithiv attempts to attach a volume to VM (via Magnum) and encounter an error from Cinder. I guess he will show the details soon | 23:44 |
hemna | after I applied my privsep-helper patch | 23:44 |
*** jwcroppe has quit IRC | 23:44 | |
hemna | gus, http://paste.openstack.org/show/486460/ | 23:44 |
hemna | I'm not sure what's up with that | 23:44 |
prithiv | hi | 23:45 |
gus | I have the one-character fix for the latest bug - just trying to get a unittest together (because I constantly underestimate how many bugs can be hiding in "obviously correct" python code) | 23:45 |
hemna | I tried running my script as sudo and it still happens | 23:45 |
prithiv | http://paste.openstack.org/show/486501/ | 23:45 |
hemna | gus, :) | 23:45 |
*** jwcroppe has joined #openstack-cinder | 23:45 | |
hemna | prithiv, I'm not sure what is causing that. I've never seen that one | 23:46 |
*** smoriya_ has joined #openstack-cinder | 23:46 | |
prithiv | i am seeing this error when my magnum client tries to attach volume to vm | 23:47 |
prithiv | i see this error in c-api log | 23:47 |
*** wanghao has quit IRC | 23:47 | |
*** wanghao has joined #openstack-cinder | 23:48 | |
*** yhayashi has joined #openstack-cinder | 23:48 | |
prithiv | anyone have any idea about this ? | 23:49 |
hemna | prithiv, I've not tried that :( | 23:49 |
hemna | can you do an attach to a nova instance with the nova client ? | 23:49 |
*** ndipanov has quit IRC | 23:49 | |
*** jwcroppe has quit IRC | 23:49 | |
*** alonma has joined #openstack-cinder | 23:51 | |
*** alonma has quit IRC | 23:56 | |
*** mragupat_ has quit IRC | 23:57 | |
*** alonma has joined #openstack-cinder | 23:57 | |
openstackgerrit | Dina Belova proposed openstack/cinder: Use OSprofiler options consolidated in lib itself https://review.openstack.org/278155 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!