| markstur | Yeah. I'll hang on to it. It'll probably come in handy eventually. :) | 00:00 |
|---|---|---|
| *** alonma has joined #openstack-cinder | 00:01 | |
| * jungleboyj starts making my own sign | 00:01 | |
| *** ducttape_ has joined #openstack-cinder | 00:01 | |
| hemna | http://paste.openstack.org/show/597113/ | 00:02 |
| hemna | ^^ latest CI reporting results | 00:02 |
| hemna | now it shows the last success for each job in each CI | 00:03 |
| ildikov | markstur: free me from multi-attach ;) | 00:04 |
| hemna | so it exposes some jobs that are not running now for whatever reason | 00:04 |
| hemna | like | 00:04 |
| hemna | dsvm-tempest-cinder-hnas-iscsi | 00:05 |
| hemna | maybe they changed the name of the job? | 00:05 |
| *** alonma has quit IRC | 00:05 | |
| hemna | ildikov, so, now that the new API is in, shouldn't multi-attach be 'easy' ? :) | 00:05 |
| markstur | ildikov: That works. I guess it is better than incarceration | 00:06 |
| hemna | markstur, are you sure about that? | 00:06 |
| ildikov | hemna: it's not fully figured out yet still, but that's my hope too, yes :) | 00:07 |
| *** alonma has joined #openstack-cinder | 00:07 | |
| *** dims_ has joined #openstack-cinder | 00:10 | |
| *** dims has quit IRC | 00:11 | |
| *** alonma has quit IRC | 00:11 | |
| *** Julien-zte has joined #openstack-cinder | 00:16 | |
| *** kaisers has joined #openstack-cinder | 00:18 | |
| *** alonma has joined #openstack-cinder | 00:18 | |
| jgriffith | hemna might suggest filtering out the gate-* jobs when computing averages at least | 00:22 |
| *** kaisers has quit IRC | 00:22 | |
| *** alonma has quit IRC | 00:23 | |
| jgriffith | hemna it would also be interesting to get a tally/comparison of jobs run out of last n succesful gate runs | 00:23 |
| jgriffith | looking at those numbers just a quick glance and things don't quite add up | 00:23 |
| jgriffith | smcginnis so are we going to do anything like anything with a success rate < n% gets marked as unsupported? | 00:24 |
| jungleboyj | jgriffith, There was a comment about anything that was less than 40% the other day getting marked unsupported. | 00:25 |
| jungleboyj | That was just said in passing though. | 00:25 |
| jgriffith | hehe... 40%, setting the bar high :) | 00:26 |
| smcginnis | jgriffith: I think for now it's good just seeing who stops passing for a few weeks. | 00:27 |
| jgriffith | @smcginnis yeah | 00:27 |
| smcginnis | jgriffith: We probably can discuss at the PTG what we want for a next step of setting a minimum passing rate. | 00:27 |
| jgriffith | scary data IMO | 00:27 |
| smcginnis | Tru | 00:28 |
| jgriffith | I'm trying to figure out how the "out of 'n' runs " is calculated | 00:29 |
| jungleboyj | jgriffith, Yeah, 40 is high. It was thrown out there though. :-) | 00:30 |
| jgriffith | jungleboyj I was being sarcastic | 00:30 |
| jgriffith | :) | 00:31 |
| jgriffith | I think I'll bump my average up and not report on failure :) | 00:31 |
| jgriffith | then I can get 100% :) | 00:31 |
| jungleboyj | jgriffith, Well, there you go! That is being industrious. | 00:33 |
| jgriffith | jungleboyj :) | 00:33 |
| jungleboyj | Sadly, that is kind of a high bar though, isn't it right now? | 00:33 |
| jgriffith | jungleboyj I think it should be at least a passing grade honestly | 00:33 |
| jgriffith | 70% | 00:33 |
| jgriffith | no curve | 00:33 |
| jgriffith | and I also want to understand the 100% ers that I can't actually find any votes for out of the last 15 merged reviews I've checked | 00:34 |
| *** jdurgin has quit IRC | 00:34 | |
| jgriffith | hehe.. found my answer | 00:37 |
| jungleboyj | jgriffith, Ok, a C is a fair expectation. ;-) What was the answer? | 00:42 |
| *** zhurong has joined #openstack-cinder | 00:42 | |
| jgriffith | don't run anything but add a comment under the CI name | 00:43 |
| jgriffith | so you won't get anything in the ci results column up top, but if you toggle on ci comments there are comments logged with no info | 00:43 |
| *** zhurong has quit IRC | 00:44 | |
| jungleboyj | jgriffith, Oh, interesting. | 00:44 |
| smcginnis | Oh lovely. | 00:44 |
| *** crushil has quit IRC | 00:45 | |
| jgriffith | we should probably do something like implement testr-results and dump everything off into an ELK stack | 00:45 |
| smcginnis | If it's working the way my script was, I was checking the last 1000 comments. If they only reported 10, and all 10 of them passed, it would show up as 100%. | 00:45 |
| *** crushil has joined #openstack-cinder | 00:45 | |
| *** jose-phillips has quit IRC | 00:45 | |
| jgriffith | @smcginnis I know that's how I came up with my diabolical plan to achieve 100% :) | 00:46 |
| *** gouthamr has joined #openstack-cinder | 00:46 | |
| smcginnis | jgriffith: That's not a bad idea. But another pile of work. :/ | 00:46 |
| smcginnis | Oh, definitely. There are plenty of ways to cheat at this game. | 00:46 |
| jgriffith | smcginnis LOL, I know.. and I don't want to do it so never mind :) | 00:46 |
| *** rajinir has quit IRC | 00:46 | |
| smcginnis | ;) | 00:46 |
| *** jdurgin has joined #openstack-cinder | 00:48 | |
| *** mriedem has joined #openstack-cinder | 00:48 | |
| *** diablo_rojo has joined #openstack-cinder | 00:50 | |
| jgriffith | @smcginnis hemna jungleboyj to be clear, I think what you guys have been working on there and putting together is great work and very useful | 00:51 |
| jgriffith | didn't mean to detract from that | 00:51 |
| smcginnis | jgriffith: Hah, not taken that way at all. | 00:51 |
| jgriffith | just being cynical about 3'rd party CI | 00:51 |
| smcginnis | I've worked on it - and I'm still cynical about 3rd party CI. ;) | 00:52 |
| smcginnis | Mostly because I know what it's like trying to run one! | 00:52 |
| jgriffith | @smcginnis :) | 00:52 |
| * asselin__ rises from the dead | 00:52 | |
| smcginnis | asselin__: Hey! Haven't seen you in a long time. :) | 00:52 |
| asselin__ | what's up :) | 00:52 |
| jgriffith | I honestly don't know how to get much better, most of the failures I've checked as of late on mine have been things completely outside of the driver | 00:52 |
| jgriffith | which is weird | 00:53 |
| *** ducttape_ has quit IRC | 00:55 | |
| smcginnis | jgriffith: Same here. Though I do have a persistent issue with iSCSI that I swear is some day going to be discovered as the other guys in the lab stomping on my IP addresses. | 00:56 |
| smcginnis | Otherwise most are patches that need to be rebased, weird intermitent failures, or who knows what. | 00:56 |
| *** liverpooler has quit IRC | 00:58 | |
| jgriffith | @smcginnis LOL | 01:01 |
| jgriffith | I do have a problem after my virtual cluster gets about 50K volumes on it | 01:01 |
| smcginnis | jgriffith: At least no one has walked off with my gear. Yet. | 01:01 |
| jgriffith | @smcginnis consider yourself lucky!!! | 01:02 |
| smcginnis | 50k? How many CI jobs you runnin? :) | 01:02 |
| jgriffith | hehe | 01:02 |
| jgriffith | well, it's a long story | 01:02 |
| *** ducttape_ has joined #openstack-cinder | 01:06 | |
| *** jperry has quit IRC | 01:07 | |
| *** xyang has joined #openstack-cinder | 01:07 | |
| *** xyang has quit IRC | 01:08 | |
| *** diablo_rojo has quit IRC | 01:14 | |
| *** ducttape_ has quit IRC | 01:39 | |
| *** dpak has joined #openstack-cinder | 01:46 | |
| dpak | ? | 01:47 |
| *** dpak has left #openstack-cinder | 01:50 | |
| *** ducttape_ has joined #openstack-cinder | 01:54 | |
| *** knomura has joined #openstack-cinder | 02:01 | |
| *** knomura has quit IRC | 02:05 | |
| *** Apoorva has quit IRC | 02:06 | |
| *** Apoorva has joined #openstack-cinder | 02:06 | |
| *** Apoorva has quit IRC | 02:07 | |
| *** Jack_Iv has joined #openstack-cinder | 02:09 | |
| *** Apoorva has joined #openstack-cinder | 02:12 | |
| *** Apoorva has quit IRC | 02:13 | |
| *** ducttape_ has quit IRC | 02:13 | |
| *** Jack_Iv has quit IRC | 02:14 | |
| *** Apoorva has joined #openstack-cinder | 02:15 | |
| *** Apoorva has quit IRC | 02:17 | |
| *** Apoorva has joined #openstack-cinder | 02:18 | |
| *** kaisers has joined #openstack-cinder | 02:19 | |
| *** alonma has joined #openstack-cinder | 02:20 | |
| *** diablo_rojo_phon has quit IRC | 02:20 | |
| *** ducttape_ has joined #openstack-cinder | 02:20 | |
| *** ducttape_ has quit IRC | 02:20 | |
| *** ducttape_ has joined #openstack-cinder | 02:20 | |
| *** Apoorva has quit IRC | 02:22 | |
| *** kaisers has quit IRC | 02:23 | |
| *** alonma has quit IRC | 02:24 | |
| *** alonma has joined #openstack-cinder | 02:26 | |
| *** alonma has quit IRC | 02:30 | |
| *** Julien-z_ has joined #openstack-cinder | 02:32 | |
| *** Julien-zte has quit IRC | 02:34 | |
| *** Julien-z_ has quit IRC | 02:40 | |
| openstackgerrit | Merged openstack/cinder: Cluster volume group fix in lvm https://review.openstack.org/396269 | 02:47 |
| *** dhinesh has quit IRC | 02:48 | |
| *** raunak has joined #openstack-cinder | 03:00 | |
| openstackgerrit | jiamin proposed openstack/cinder: change the design of CG in DS8K driver https://review.openstack.org/419321 | 03:01 |
| *** xyang has joined #openstack-cinder | 03:18 | |
| *** xyang has quit IRC | 03:19 | |
| *** crushil has quit IRC | 03:28 | |
| openstackgerrit | Merged openstack/cinder: clean up replication volume on an xiv backend https://review.openstack.org/426763 | 03:28 |
| *** sticker_ has joined #openstack-cinder | 03:29 | |
| *** sticker has quit IRC | 03:32 | |
| *** mtanino has quit IRC | 03:43 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Move release notes to correct location https://review.openstack.org/427542 | 03:43 |
| *** nicolasbock has quit IRC | 03:46 | |
| *** tlian has quit IRC | 03:59 | |
| *** crushil has joined #openstack-cinder | 04:02 | |
| *** psachin has joined #openstack-cinder | 04:03 | |
| *** ducttape_ has quit IRC | 04:05 | |
| smcginnis | If anyone, like me, ignored this in the ML and now has stacking failures: http://lists.openstack.org/pipermail/openstack-dev/2017-January/111295.html | 04:07 |
| *** sudipto_ has joined #openstack-cinder | 04:08 | |
| *** sudipto has joined #openstack-cinder | 04:08 | |
| *** raunak has quit IRC | 04:10 | |
| *** mdnadeem has joined #openstack-cinder | 04:10 | |
| *** crushil has quit IRC | 04:20 | |
| *** crushil has joined #openstack-cinder | 04:21 | |
| *** arch-nemesis has quit IRC | 04:24 | |
| *** ducttape_ has joined #openstack-cinder | 04:28 | |
| *** crushil has quit IRC | 04:31 | |
| *** alonma has joined #openstack-cinder | 04:33 | |
| openstackgerrit | Merged openstack/cinder: Add create volume from image policy https://review.openstack.org/426728 | 04:36 |
| *** alonma has quit IRC | 04:38 | |
| *** ducttape_ has quit IRC | 04:38 | |
| *** pbandark has joined #openstack-cinder | 04:46 | |
| *** ducttape_ has joined #openstack-cinder | 04:47 | |
| *** nikeshm has quit IRC | 04:53 | |
| openstackgerrit | Vijay Ladani proposed openstack/cinder: 3PAR: rally test create-volume-from-snapshot fails https://review.openstack.org/421739 | 04:58 |
| *** kwathore has joined #openstack-cinder | 05:03 | |
| *** pgadiya has joined #openstack-cinder | 05:09 | |
| *** kaisers has joined #openstack-cinder | 05:10 | |
| *** ducttape_ has quit IRC | 05:13 | |
| *** kaisers has quit IRC | 05:15 | |
| *** raunak has joined #openstack-cinder | 05:15 | |
| *** lpetrut has joined #openstack-cinder | 05:16 | |
| *** kaisers has joined #openstack-cinder | 05:18 | |
| *** dave-mccowan has quit IRC | 05:18 | |
| *** gouthamr has quit IRC | 05:21 | |
| *** dave-mccowan has joined #openstack-cinder | 05:22 | |
| *** kaisers has quit IRC | 05:22 | |
| *** pdeore has joined #openstack-cinder | 05:22 | |
| *** nikeshm has joined #openstack-cinder | 05:32 | |
| *** dave-mcc_ has joined #openstack-cinder | 05:59 | |
| *** Jack_Iv has joined #openstack-cinder | 06:00 | |
| *** dave-mccowan has quit IRC | 06:01 | |
| *** raunak has quit IRC | 06:01 | |
| *** lkuchlan has joined #openstack-cinder | 06:04 | |
| *** knomura has joined #openstack-cinder | 06:07 | |
| *** knomura has quit IRC | 06:11 | |
| *** lpetrut has quit IRC | 06:13 | |
| openstackgerrit | Dinesh Bhor proposed openstack/cinder: Fix 500 error while listing manageable volumes and snapshots https://review.openstack.org/427560 | 06:16 |
| *** alonmarx has joined #openstack-cinder | 06:21 | |
| *** ducttape_ has joined #openstack-cinder | 06:27 | |
| *** rcernin has joined #openstack-cinder | 06:29 | |
| *** ducttape_ has quit IRC | 06:31 | |
| *** mjura has joined #openstack-cinder | 06:33 | |
| vladani | Please review these changes... | 06:33 |
| vladani | https://review.openstack.org/#/c/421739/ | 06:34 |
| vladani | https://review.openstack.org/#/c/424454/ | 06:34 |
| *** alonma has joined #openstack-cinder | 06:41 | |
| *** nkrinner_afk is now known as nkrinner | 06:44 | |
| *** alonma has quit IRC | 06:46 | |
| *** arne_r has joined #openstack-cinder | 06:53 | |
| *** lpetrut has joined #openstack-cinder | 06:56 | |
| *** fnordahl has quit IRC | 06:57 | |
| *** fnordahl has joined #openstack-cinder | 06:57 | |
| *** kaisers has joined #openstack-cinder | 07:09 | |
| *** alexey_weyl has joined #openstack-cinder | 07:13 | |
| alexey_weyl | Hi Guys.. | 07:13 |
| alexey_weyl | I am a core developer in the Vitrage project, and recently (from about 30 hours ago) we have a problem in our gate in gerrit due to Cinder and I wanted to ask your help | 07:14 |
| alexey_weyl | The problem goes like that.. | 07:14 |
| alexey_weyl | In our tempest we subscribe to the cinder.conf the following data: | 07:15 |
| alexey_weyl | notification_topics = notifications,vitrage_notifications | 07:15 |
| alexey_weyl | Due to that, when we create volumes and attached them etc.. Vitrage receives those events from the oslo bus and uses them. | 07:16 |
| alexey_weyl | But approximatelly from 30 hours ago we don't receive those events anymore, although I see in the logs that the volume is created and attached (and after wards deleted of course) | 07:17 |
| alexey_weyl | So I wanted to ask you guys if maybe you know due to what this problem happens? | 07:17 |
| *** alonma has joined #openstack-cinder | 07:17 | |
| *** tesseract has joined #openstack-cinder | 07:18 | |
| alexey_weyl | (it is happening only in when creating cinder, because when creating ports in neutron, instances in nova, alarms in aodh etc.. we receive the events) | 07:18 |
| *** pcaruana has joined #openstack-cinder | 07:19 | |
| *** alonma has quit IRC | 07:22 | |
| *** fnordahl has quit IRC | 07:27 | |
| *** fnordahl has joined #openstack-cinder | 07:27 | |
| *** Jack_Iv has quit IRC | 07:31 | |
| *** xek__ is now known as xek | 07:31 | |
| *** Jack_Iv has joined #openstack-cinder | 07:32 | |
| *** alonmarx has quit IRC | 07:34 | |
| *** alonma has joined #openstack-cinder | 07:35 | |
| *** sudipto has quit IRC | 07:37 | |
| *** sudipto_ has quit IRC | 07:37 | |
| *** Jack_Iv has quit IRC | 07:38 | |
| *** Jack_I has joined #openstack-cinder | 07:39 | |
| *** alonma has quit IRC | 07:40 | |
| *** ChubYann has quit IRC | 07:42 | |
| openstackgerrit | Merged openstack/cinder: Add cluster_name to temporary volumes https://review.openstack.org/427361 | 07:44 |
| *** alonma has joined #openstack-cinder | 07:45 | |
| *** shubham0d has joined #openstack-cinder | 07:45 | |
| *** lpetrut has quit IRC | 07:48 | |
| *** adisky_ has joined #openstack-cinder | 07:54 | |
| *** ducttape_ has joined #openstack-cinder | 07:57 | |
| *** ducttape_ has quit IRC | 08:02 | |
| *** lpetrut has joined #openstack-cinder | 08:06 | |
| *** bkopilov has quit IRC | 08:14 | |
| *** lpetrut has quit IRC | 08:20 | |
| *** dave-mcc_ has quit IRC | 08:24 | |
| *** bkopilov has joined #openstack-cinder | 08:24 | |
| *** s-shiono has joined #openstack-cinder | 08:48 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-cinder | 09:00 | |
| *** s-shiono has quit IRC | 09:00 | |
| *** alexey_weyl has quit IRC | 09:06 | |
| *** alexey_weyl has joined #openstack-cinder | 09:12 | |
| dulek | alexey_weyl: 24 h ago this was merged: https://review.openstack.org/#/c/426547/ | 09:15 |
| dulek | geguileo: ^ | 09:15 |
| geguileo | alexey_weyl: I'll have a look, as it should be working... | 09:16 |
| geguileo | dulek: thanks for the ping | 09:16 |
| geguileo | alexey_weyl: Can you point me to a sample config in the gate that is failing, please? | 09:17 |
| alexey_weyl | geguileo: yes | 09:19 |
| alexey_weyl | i am sending you a link to a tempest that failed | 09:20 |
| geguileo | alexey_weyl: thanks | 09:20 |
| alexey_weyl | you can go to this link http://logs.openstack.org/28/427128/4/check/gate-vitrage-dsvm-datasources-ubuntu-xenial/7189cf2/ | 09:20 |
| alexey_weyl | and here you can see in the vitrage-graph.log that we don't receive any cinder.volume notifications although we create a volume and attach it | 09:21 |
| alexey_weyl | dulek | 09:22 |
| alexey_weyl | dulek: yes, thats what I saw and thought that it is connected | 09:22 |
| geguileo | alexey_weyl: I see the problem | 09:22 |
| alexey_weyl | thats why I decided to ask you guys about that | 09:22 |
| *** e0ne has joined #openstack-cinder | 09:23 | |
| geguileo | alexey_weyl: there's no transport_url defined in oslo_messaging_notifications | 09:23 |
| alexey_weyl | geguileo | 09:23 |
| alexey_weyl | geguileo | 09:23 |
| alexey_weyl | geguileo: i am very curious | 09:23 |
| geguileo | probably oslo_messaging takes the value from the default when you define a driver in that section | 09:24 |
| *** mdnadeem has quit IRC | 09:24 | |
| *** shubham0d has quit IRC | 09:24 | |
| *** mdnadeem has joined #openstack-cinder | 09:24 | |
| geguileo | alexey_weyl: I'll investigate how oslo messaging is actually getting the value for oslo_messaging_motifications.transport_url and fix the check | 09:24 |
| geguileo | alexey_weyl: OK, apparently they decoupled the notification transport url in Mitaka, and Vitrage is using the old cfg | 09:27 |
| alexey_weyl | geguileo: what should be written in the transport_url? | 09:27 |
| geguileo | alexey_weyl: Oh, it's not your fault | 09:27 |
| alexey_weyl | I see.. | 09:28 |
| geguileo | alexey_weyl: I have to make the check compatible with the old configuration | 09:28 |
| alexey_weyl | in cinder you mean? | 09:28 |
| *** ducttape_ has joined #openstack-cinder | 09:28 | |
| geguileo | alexey_weyl: Sure you can fix it, but it was my mistake for not considering how oslo messaging was configured before Mitaka | 09:28 |
| geguileo | alexey_weyl: Yes, I have to fix it in Cinder | 09:28 |
| alexey_weyl | geguileo: That will be great. | 09:29 |
| alexey_weyl | geguileo: when do you think it will be possible to see the new change? | 09:30 |
| geguileo | alexey_weyl: Could you create a LP bug for me? | 09:30 |
| geguileo | alexey_weyl: I should be able to get it fixed this morning | 09:30 |
| *** stakeda has quit IRC | 09:31 | |
| alexey_weyl | geguileo: ok, I am opening an LP bug. | 09:31 |
| geguileo | alexey_weyl: thanks | 09:31 |
| *** lpetrut has joined #openstack-cinder | 09:32 | |
| alexey_weyl | geguileo: but just that I will know, what is supposed to be the transport_url value in the conf? | 09:32 |
| geguileo | alexey_weyl: it allows you to send the notifications to a different transport than the rpc | 09:32 |
| *** ducttape_ has quit IRC | 09:32 | |
| geguileo | alexey_weyl: As you can see, defining it in the default section is deprecated: https://github.com/openstack/oslo.messaging/blob/afaa4d935dd77760492f36dba622f6a01fe14fd9/oslo_messaging/notify/notifier.py#L44 | 09:33 |
| alexey_weyl | geguileo: ok, we are all waiting for your fix, because this is the end of the Ocata cycle and we have to push all our changes. | 09:33 |
| alexey_weyl | geguileo: thanks in advance | 09:33 |
| geguileo | alexey_weyl: so no pressure, right? ;-) | 09:34 |
| alexey_weyl | geguileo: hhhhhhhhh :) | 09:35 |
| *** zhonghua has quit IRC | 09:36 | |
| *** zhonghua has joined #openstack-cinder | 09:36 | |
| *** sudipto_ has joined #openstack-cinder | 09:38 | |
| *** sudipto has joined #openstack-cinder | 09:38 | |
| alexey_weyl | geguileo: I have opened a new bug. Hope it is ok the way I wrote it. here is the link: https://bugs.launchpad.net/cinder/+bug/1660928 | 09:39 |
| openstack | Launchpad bug 1660928 in Cinder "Backwards compatibility for oslo messaging in Cinder" [Undecided,New] | 09:39 |
| geguileo | alexey_weyl: Thanks | 09:42 |
| openstackgerrit | zhangsong proposed openstack/cinder: RBD:Move RBDVolume calls to a separate threads https://review.openstack.org/423184 | 09:42 |
| *** jordanP has joined #openstack-cinder | 09:46 | |
| *** yumiriam has joined #openstack-cinder | 09:48 | |
| *** xiangxinyong has quit IRC | 09:51 | |
| *** xiangxinyong has joined #openstack-cinder | 09:52 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Add check if copyoffload tool is available https://review.openstack.org/427613 | 09:52 |
| *** shubham0d has joined #openstack-cinder | 09:55 | |
| *** e0ne has quit IRC | 09:56 | |
| *** ebalduf has quit IRC | 10:08 | |
| *** ebalduf has joined #openstack-cinder | 10:11 | |
| erlon | geguileo: hey Gorka, when you can, please give a look on these: https://review.openstack.org/#/c/422733/, https://review.openstack.org/425100, https://review.openstack.org/427348 | 10:12 |
| *** shausy has joined #openstack-cinder | 10:18 | |
| *** Jack_Iv has joined #openstack-cinder | 10:20 | |
| *** Jack_I has quit IRC | 10:21 | |
| *** Jack_Iv has quit IRC | 10:23 | |
| openstackgerrit | jiamin proposed openstack/cinder: change the design of CG in DS8K driver https://review.openstack.org/419321 | 10:25 |
| *** Zack2 has joined #openstack-cinder | 10:26 | |
| Zack2 | hi all. how come there is a --tenant option for "cinder list" and "cinder snapshot-list", but no such option for "cinder backup-list"? | 10:28 |
| Zack2 | not having that option for backups hardens ones efforts to create any sort of "per tenant backups" overview script/report | 10:29 |
| kwathore | Please review below cheery pick patch | 10:36 |
| kwathore | https://review.openstack.org/#/c/407928/ | 10:36 |
| *** alexey_weyl has quit IRC | 10:37 | |
| *** sudipto has quit IRC | 10:40 | |
| *** sudipto_ has quit IRC | 10:40 | |
| *** alonma has quit IRC | 10:45 | |
| *** alonma has joined #openstack-cinder | 10:45 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration https://review.openstack.org/427348 | 10:45 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin https://review.openstack.org/422733 | 10:45 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs https://review.openstack.org/427632 | 10:45 |
| *** dalgaaf has quit IRC | 10:48 | |
| *** dalgaaf has joined #openstack-cinder | 10:49 | |
| *** alyson_ has joined #openstack-cinder | 10:49 | |
| *** alonma has quit IRC | 10:49 | |
| *** mvk has quit IRC | 10:49 | |
| *** alonma has joined #openstack-cinder | 10:51 | |
| *** shubham0d has quit IRC | 10:51 | |
| *** alonma has quit IRC | 10:55 | |
| *** alexey_weyl has joined #openstack-cinder | 10:58 | |
| *** ducttape_ has joined #openstack-cinder | 10:58 | |
| Zack2 | kwathore: so this is the case only when HP 3par is in question or it does not work in general. I mean I use ceph for volumes, snapshots and backups and I don't have any problems creating it | 10:59 |
| kwathore | yes | 10:59 |
| kwathore | it is releated to HP 3par | 10:59 |
| *** alexey_weyl has quit IRC | 11:01 | |
| *** kaisers1 has quit IRC | 11:01 | |
| *** kaisers1 has joined #openstack-cinder | 11:02 | |
| *** ducttape_ has quit IRC | 11:03 | |
| *** Jack_Iv has joined #openstack-cinder | 11:03 | |
| *** nicolasbock has joined #openstack-cinder | 11:04 | |
| *** shubham0d has joined #openstack-cinder | 11:06 | |
| *** alonma has joined #openstack-cinder | 11:07 | |
| *** jbernard has quit IRC | 11:11 | |
| *** alonma has quit IRC | 11:12 | |
| *** alonma has joined #openstack-cinder | 11:15 | |
| *** jbernard has joined #openstack-cinder | 11:17 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX driver - ignore service level and workload in xml https://review.openstack.org/427506 | 11:18 |
| *** mvk has joined #openstack-cinder | 11:19 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix notification short-circuit https://review.openstack.org/427647 | 11:21 |
| geguileo | dulek: ^ | 11:22 |
| *** e0ne has joined #openstack-cinder | 11:22 | |
| *** jamielennox is now known as jamielennox|away | 11:25 | |
| *** shubham0d has quit IRC | 11:27 | |
| dulek | geguileo: Looking. | 11:27 |
| *** sdague has joined #openstack-cinder | 11:28 | |
| geguileo | I have verified it in my env with the driver set to logging and setting a hard breakpoint in the code | 11:29 |
| *** mdnadeem has quit IRC | 11:36 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration https://review.openstack.org/427348 | 11:41 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs https://review.openstack.org/427632 | 11:41 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin https://review.openstack.org/422733 | 11:41 |
| *** mdnadeem has joined #openstack-cinder | 11:42 | |
| erlon | hemma: indeed, dsvm-tempest-cinder-hnas-iscsi -> hitachi-hnas-iscsi | 11:44 |
| *** ociuhandu has joined #openstack-cinder | 11:46 | |
| *** ociuhandu has quit IRC | 11:51 | |
| *** pdeore has quit IRC | 11:52 | |
| lyarwood | Morning all, hopefully a stupid question but none of the connectors in os-brick update the connection_info / connection_properties passed in by Nova/Cinder right? | 11:57 |
| lyarwood | I've found some code that's going out of it's way in nova to save connection_info after a call to connect_volume but I can't think why | 11:57 |
| openstackgerrit | jiamin proposed openstack/cinder: support bulk create for CG and VG https://review.openstack.org/424945 | 11:58 |
| *** adrianofr has quit IRC | 11:59 | |
| dulek | geguileo: What do you think about this: https://github.com/openstack/tempest/blob/bf73e0cb1c9eba5d62b9230b0535c970651dc1fa/tempest/api/volume/admin/v2/test_backends_capabilities.py#L36-L52 | 12:00 |
| dulek | geguileo: Tempest assumes that get-pools returns list that can be used as argument to get-capabilities. | 12:00 |
| dulek | geguileo: I'm leaning to the side that this is a rightful assumption and get-capabilities should accept clusters as well. | 12:01 |
| dulek | geguileo: I'll be happy to know what do you think about it. | 12:01 |
| *** ociuhandu has joined #openstack-cinder | 12:01 | |
| * geguileo checking | 12:02 | |
| *** Jack_Iv has quit IRC | 12:03 | |
| *** knomura has joined #openstack-cinder | 12:05 | |
| geguileo | dulek: You mean that it should it should look by host and cluster to find the match? | 12:08 |
| *** ociuhandu has quit IRC | 12:08 | |
| *** adrianofr has joined #openstack-cinder | 12:09 | |
| *** cdelatte has joined #openstack-cinder | 12:11 | |
| dulek | geguileo: Hm, I thought get-capabilities goes down to the c-vol, let my check this. | 12:12 |
| geguileo | dulek: It goes | 12:12 |
| geguileo | dulek: But first checks if the service exists | 12:12 |
| geguileo | dulek: And it's checking by host | 12:13 |
| dulek | geguileo: Oh right, this is where it fails. I don't see an issue with letting it check for cluster too. | 12:13 |
| dulek | Maybe only the fact that host means both things now. | 12:13 |
| dulek | mdulko:tempest/ (master) $ cinder get-capabilities | 12:14 |
| dulek | usage: cinder get-capabilities <host> | 12:14 |
| dulek | This <host> | 12:14 |
| geguileo | dulek: well, truth be told I don't think we have documentation for that | 12:14 |
| geguileo | dulek: Except in the client, and we can change that | 12:14 |
| geguileo | dulek: So it says host/cluster | 12:14 |
| dulek | geguileo: Right - URL has capabilities/chrzaszcz@nfs, so it's unnamed. | 12:15 |
| geguileo | dulek: yup ;-) | 12:15 |
| geguileo | dulek: So we can say it was always backend_id | 12:15 |
| dulek | chrząszcz is a beetle BTW. ;) | 12:16 |
| geguileo | lol | 12:16 |
| dulek | geguileo: Now there will be two ways of accessing same data - through cluster and host URL. I don't exactly see a serious issue with that too. | 12:16 |
| geguileo | dulek: I think that's ok, since they are not contradicting one another | 12:17 |
| geguileo | dulek: Though we'll send the request to the cluster if it's clustered | 12:17 |
| geguileo | dulek: You may ask for the capabilities of a specific host, but if it's clustered the request will go to the cluster | 12:18 |
| dulek | geguileo: This isn't that bad, unless someone is using this call to healthcheck individual nodes. | 12:18 |
| geguileo | dulek: Well, that will still work for A/P deployments | 12:19 |
| geguileo | dulek: And for A/A they'll have to do it using the new log level checking API | 12:19 |
| dulek | geguileo: Good point, if you're moving to A/A, then you expect your get-capabilities call to be A/A as well. | 12:19 |
| *** alonma has quit IRC | 12:20 | |
| geguileo | dulek: I can write a patch for this | 12:20 |
| geguileo | dulek: Or do you want to do it? | 12:20 |
| *** alonma has joined #openstack-cinder | 12:20 | |
| dulek | geguileo: I have some free cycles, so I can do it. I'll also upload a Tempest patch that fixes get-pools tests when using A/A. | 12:21 |
| geguileo | dulek: I was talking about the Cinder changes | 12:22 |
| dulek | geguileo: To get-capabilities? I meant I have some time, so I can write Cinder changes as well. :) | 12:22 |
| geguileo | dulek: Oh, OK, please add me to the review, since I already have the context in my head to review the patch | 12:22 |
| dulek | Okay! | 12:23 |
| *** alonma has quit IRC | 12:24 | |
| openstackgerrit | jiamin proposed openstack/cinder: support bulk create for CG and VG https://review.openstack.org/424945 | 12:25 |
| *** alonma has joined #openstack-cinder | 12:26 | |
| *** catintheroof has joined #openstack-cinder | 12:28 | |
| *** ducttape_ has joined #openstack-cinder | 12:29 | |
| *** alonma has quit IRC | 12:31 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix attachments after attached migration https://review.openstack.org/427348 | 12:31 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix multibackend-matrix confs https://review.openstack.org/427632 | 12:31 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix volume retype with migration as non-admin https://review.openstack.org/422733 | 12:31 |
| *** ociuhandu has joined #openstack-cinder | 12:31 | |
| *** ducttape_ has quit IRC | 12:33 | |
| *** pbandark has quit IRC | 12:33 | |
| *** ociuhandu has quit IRC | 12:36 | |
| Dinesh_Bhor | geguileo, smcginnis: Hi, whenever you get time please take a look at it: https://review.openstack.org/#/c/427560/ | 12:40 |
| geguileo | Dinesh_Bhor: lol, I had just opened it for review | 12:40 |
| Dinesh_Bhor | geguileo: aah, thanks | 12:41 |
| *** alonma has joined #openstack-cinder | 12:42 | |
| *** alonma has quit IRC | 12:45 | |
| *** alonma has joined #openstack-cinder | 12:46 | |
| *** gaurangt has joined #openstack-cinder | 12:46 | |
| *** mvk has quit IRC | 12:47 | |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants https://review.openstack.org/426747 | 12:48 |
| *** test123 has joined #openstack-cinder | 12:48 | |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants https://review.openstack.org/426748 | 12:49 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Use storage profile name in extra-spec https://review.openstack.org/427257 | 12:49 |
| *** test123 has quit IRC | 12:49 | |
| *** eharney has joined #openstack-cinder | 12:50 | |
| nikeshm | hi | 12:50 |
| nikeshm | any one facing the issue in their CIs https://review.openstack.org/#/c/427561/ | 12:51 |
| *** lkuchlan_ has joined #openstack-cinder | 12:52 | |
| *** lkuchlan has quit IRC | 12:53 | |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants https://review.openstack.org/426749 | 12:53 |
| openstackgerrit | Pooja Jadhav proposed openstack/cinder: Extracted HTTP response codes to constants https://review.openstack.org/426750 | 12:55 |
| *** ociuhandu has joined #openstack-cinder | 12:56 | |
| *** mtanino has joined #openstack-cinder | 12:56 | |
| arne_r | hi channel, quick question | 12:57 |
| *** ducttape_ has joined #openstack-cinder | 12:57 | |
| arne_r | is there al list of who participated in the PTGs, or who wants to be in the next one? | 12:57 |
| *** mvk has joined #openstack-cinder | 13:00 | |
| *** ociuhandu has quit IRC | 13:01 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Fall back to full clone if extend needed https://review.openstack.org/427683 | 13:02 |
| *** nikeshm has quit IRC | 13:02 | |
| *** wN has quit IRC | 13:02 | |
| *** Jack_Iv has joined #openstack-cinder | 13:03 | |
| *** knomura has quit IRC | 13:04 | |
| *** knomura has joined #openstack-cinder | 13:04 | |
| *** cdelatte has quit IRC | 13:05 | |
| *** wN has joined #openstack-cinder | 13:07 | |
| *** wN has quit IRC | 13:07 | |
| *** wN has joined #openstack-cinder | 13:07 | |
| *** bapalm has quit IRC | 13:07 | |
| *** cdelatte has joined #openstack-cinder | 13:07 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Use full clone if extend needed https://review.openstack.org/427683 | 13:08 |
| *** Jack_Iv has quit IRC | 13:10 | |
| *** bapalm has joined #openstack-cinder | 13:12 | |
| *** shausy has quit IRC | 13:12 | |
| *** mtanino has quit IRC | 13:13 | |
| *** ociuhandu has joined #openstack-cinder | 13:16 | |
| *** pgadiya has quit IRC | 13:21 | |
| *** akapil has joined #openstack-cinder | 13:21 | |
| *** bapalm has quit IRC | 13:24 | |
| *** jordanP has quit IRC | 13:27 | |
| smcginnis | arne_r: This is a the first PTG, so it's all new. | 13:28 |
| *** jordanP has joined #openstack-cinder | 13:28 | |
| smcginnis | arne_r: Last I heard we were close to registration being full. Something like 550-600 people. | 13:28 |
| openstackgerrit | Merged openstack/cinder: Revert "Mark the sheepdog driver as unsupported" https://review.openstack.org/425586 | 13:29 |
| jungleboyj | smcginnis, We have been talking about that a lot at Lenovo. Really wonder how this is going to go. | 13:29 |
| jungleboyj | Oooh, Sheepdog is back in huh? | 13:30 |
| smcginnis | jungleboyj: Yep, yep. :) | 13:30 |
| *** alonmarx has joined #openstack-cinder | 13:31 | |
| jungleboyj | smcginnis, Plus have you seen that they have made the statement that they are not doing events in the US due to the plitical crap. | 13:32 |
| smcginnis | jungleboyj: Yeah, after Boston, at least the next three summits are going to be international. | 13:32 |
| smcginnis | jungleboyj: I wonder if that means Australia, Canada, somewhere-in-Europe. | 13:33 |
| smcginnis | jungleboyj: Though south would be nice. Haven't gone that way yet. ;) | 13:33 |
| jungleboyj | smcginnis, That would be my guess. I hope Canada is still taking refugees. | 13:33 |
| jungleboyj | Brazil? | 13:33 |
| *** gouthamr has joined #openstack-cinder | 13:34 | |
| smcginnis | jungleboyj: That could work. We have a few contributors there I know of. | 13:34 |
| *** Jack_Iv has joined #openstack-cinder | 13:34 | |
| jungleboyj | smcginnis, I am surprised we haven't done South America yet. | 13:34 |
| arne_r | smcginnis: Any chance that the registration list for the PTG is public? | 13:35 |
| dulek | arne_r: I don't think so, but maybe diablo_rojo can give you some limited info if you need it. | 13:35 |
| smcginnis | arne_r: I highly doubt it. | 13:36 |
| dulek | geguileo: Vitrage gate is blocked until bug 1660928 gets fixed. I think it's worth it to just hit recheck on the patch and solve the unit tests problem in another patch. | 13:36 |
| openstack | bug 1660928 in Cinder "Backwards compatibility for oslo messaging in Cinder" [Undecided,In progress] https://launchpad.net/bugs/1660928 - Assigned to Gorka Eguileor (gorka) | 13:36 |
| *** dustins has joined #openstack-cinder | 13:37 | |
| *** knomura_ has joined #openstack-cinder | 13:37 | |
| arne_r | dulek: That would probably be a bit overkill. I am just curious if IBM folks will be present at the cinder-PTG | 13:37 |
| jungleboyj | arne_r, They will be. Why do you ask? | 13:38 |
| *** knomura has quit IRC | 13:39 | |
| *** gaurangt has left #openstack-cinder | 13:40 | |
| *** eharney has quit IRC | 13:41 | |
| dulek | geguileo: Uh-oh, I've just reproduced this recursion problem locally. | 13:41 |
| *** gaurangt has joined #openstack-cinder | 13:41 | |
| arne_r | jungleboyj: to put it diplomatic: my boss is considering wether he can afford to send guys from our team to the summit, and if other IBM teams do so as well, that would be an incentive | 13:42 |
| *** ducttape_ has quit IRC | 13:42 | |
| erlon | ildikov: jgriffith: shouldn't the attach APIs receive the volume name as optional? Im having this error: ERROR (BadRequest): Invalid input for field/attribute volumeId. Value: v1. u'v1' is not a 'uuid' (HTTP 400) (Request-ID: req-204b6b47-c6ce-4826-a87a-b3d961423c86) | 13:43 |
| *** mriedem has quit IRC | 13:44 | |
| dulek | arne_r: FYI: Eventbrite says that there are only 18 tickets left. | 13:46 |
| openstackgerrit | Merged openstack/cinder: 3PAR: Inconsistency in copied and source CG https://review.openstack.org/419335 | 13:48 |
| arne_r | dulek: that's probably why i was asked to learn it today :p | 13:49 |
| *** gaurangt has quit IRC | 13:51 | |
| *** gaurangt has joined #openstack-cinder | 13:54 | |
| *** tongli has joined #openstack-cinder | 13:55 | |
| *** kfarr has joined #openstack-cinder | 14:01 | |
| jungleboyj | arne_r, Yes, there will be a number of people from the Cloud Division there. | 14:02 |
| jungleboyj | hemna and scottda I believe are both going. | 14:02 |
| arne_r | jungleboyj: good to hear :) | 14:03 |
| arne_r | do you also know from people of the systems group? | 14:04 |
| jungleboyj | Ah, I know that there were a number that were planning to go. I am pretty sure that people from under Anne Funai will be going. | 14:05 |
| jungleboyj | Ping Joe Cropper or Gerald McBrearty for details on that side of the org. | 14:05 |
| jungleboyj | AFK for a bit. Need to get a boy to school. | 14:05 |
| *** tlian has joined #openstack-cinder | 14:07 | |
| *** dustins has quit IRC | 14:08 | |
| *** arne_r has quit IRC | 14:08 | |
| *** arne_r has joined #openstack-cinder | 14:09 | |
| arne_r | jungleboyj: my client crashed, could you resend your last message? | 14:09 |
| *** ducttape_ has joined #openstack-cinder | 14:09 | |
| jungleboyj | arne_r, Ping Joe Cropper or Gerald McBrearty for details on that side of the org. | 14:11 |
| arne_r | thank you very much :) | 14:11 |
| smcginnis | dulek, geguileo: Mind taking care of this one? https://review.openstack.org/#/c/427542/ | 14:11 |
| *** xyang1 has joined #openstack-cinder | 14:13 | |
| smcginnis | dulek: Thank you. | 14:14 |
| dulek | smcginnis: It's funny we haven't noticed earlier. | 14:14 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Make get_capabilities look for clusters https://review.openstack.org/427724 | 14:14 |
| smcginnis | dulek: I know! :) | 14:15 |
| geguileo | smcginnis: Sorry, I was out of my desk and forgot to set it to away | 14:15 |
| geguileo | dulek: I was able to reproduce it locally when I wrote the comment, I'll fix it after lunch | 14:15 |
| geguileo | dulek: That's how I know it's an interdependency, because it's caused by a specific order | 14:16 |
| geguileo | dulek: I need to pinpoint the exact tests that are interacting | 14:16 |
| *** mriedem has joined #openstack-cinder | 14:16 | |
| *** xyang has joined #openstack-cinder | 14:17 | |
| smcginnis | geguileo: Hah, no problem. | 14:18 |
| smcginnis | geguileo: I never set mine to away. Keep people guessing if I'm actually here or not. :D | 14:18 |
| dulek | smcginnis: It might be even better to keep it set to away all the time… :D | 14:19 |
| smcginnis | dulek: Hah, there you go. ;) | 14:20 |
| *** eharney has joined #openstack-cinder | 14:22 | |
| *** ducttape_ has quit IRC | 14:22 | |
| *** ducttape_ has joined #openstack-cinder | 14:22 | |
| *** ducttape_ has quit IRC | 14:23 | |
| *** liverpooler has joined #openstack-cinder | 14:24 | |
| *** porrua has joined #openstack-cinder | 14:24 | |
| *** mkoderer has joined #openstack-cinder | 14:24 | |
| *** cdelatte has quit IRC | 14:25 | |
| *** gaurangt has left #openstack-cinder | 14:26 | |
| xyang | geguileo: hi, we got this error in CI's: ERROR oslo_db.sqlalchemy.exc_filters [-] DBAPIError exception wrapped from (pymysql.err.ProgrammingError) (1064, u"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(6)' at line 1") [SQL: u'\nALTER TABLE workers CHANGE COLU | 14:26 |
| xyang | MN updated_at updated_at DATETIME(6)'] | 14:26 |
| xyang | geguileo: any idea what might be the problem? | 14:26 |
| *** ducttape_ has joined #openstack-cinder | 14:28 | |
| *** gaurangt has joined #openstack-cinder | 14:30 | |
| *** ducttape_ has quit IRC | 14:31 | |
| xyang | geguileo: upgrade to mysql > v5.5? | 14:32 |
| *** alonmarx has quit IRC | 14:32 | |
| *** crushil has joined #openstack-cinder | 14:32 | |
| *** gaurangt has left #openstack-cinder | 14:33 | |
| scottda | lyarwood: regarding Nova saving connection_info after calling connect_volume: is that during Nova live migration? It seems I recall doing that to keep track of old_connection while creating new_connection. | 14:33 |
| scottda | lyarwood: I think saving that info prevents some trickery to get the old info after the new connection is made. | 14:34 |
| xyang | geguileo: looking at your comments in the migration script, sounds like this error can be safely ignored? v5.5 should still work? | 14:34 |
| *** gaurangt has joined #openstack-cinder | 14:36 | |
| lyarwood | scottda: it was during swap_volume, turns out that nova is stashing the device['path'] that os_brick returns in the connection_info | 14:40 |
| lyarwood | scottda: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/iscsi.py#L67 | 14:40 |
| scottda | lyarwood: OK, I *believe* that's only called for a Cinder migration to a new backend, if the volume is attached... | 14:43 |
| scottda | lyarwood: So, maybe a similar situation, to save the old_connection info and use it after the new connection is made, in order to call terminate_connection on cinder properly. | 14:44 |
| scottda | lyarwood: But I'm not certain of that. | 14:44 |
| lyarwood | scottda: we can also call it via `nova volume-update` that switches two volumes, it isn't an issue really, we were just double saving the connection_info on the Nova side for no reason | 14:45 |
| lyarwood | scottda: do you know if retyping volumes multiple times is supported btw? | 14:45 |
| lyarwood | scottda: it's continously breaking for me with the second attempt | 14:46 |
| *** dave-mccowan has joined #openstack-cinder | 14:47 | |
| scottda | lyarwood: It seems I recall a bug for that...not sure if it has been fixed...just a minute.. | 14:47 |
| dulek | geguileo: Okay, so here's another fun tempest assumption: https://github.com/openstack/tempest/blob/9e6ee394184aedc777226b0723e5fe16d2b33ebe/tempest/api/volume/admin/v2/test_volume_pools.py#L29-L34 | 14:49 |
| dulek | geguileo: I thought I could fix this by looking for a cluster_name in volume-show output, but that's missing from there. | 14:50 |
| dulek | geguileo: Maybe this test should call service-show to decide if it should pass cluster_name or host to get-pools? | 14:50 |
| scottda | lyarwood: Old bug, it looks like it was fixed: https://bugs.launchpad.net/nova/+bug/1490236 | 14:52 |
| openstack | Launchpad bug 1490236 in OpenStack Compute (nova) "Swapping volume can't been swap again" [Low,Fix released] - Assigned to Takashi NATSUME (natsume-takashi) | 14:52 |
| lyarwood | scottda: sweet, thanks | 14:52 |
| lyarwood | scottda: ah that's the one we reverted | 14:53 |
| *** guyr-infinidat has joined #openstack-cinder | 14:53 | |
| lyarwood | scottda: introduced a regression | 14:53 |
| scottda | lyarwood: Yeah | 14:53 |
| *** cdelatte has joined #openstack-cinder | 14:53 | |
| scottda | FYI, we've introduced new APIs for Cinder attach/detach in Ocata. Trying to improve situations like this.. | 14:54 |
| scottda | lyarwood: With new APIs, Cinder will keep track of connection info, so less burden on Nova (or whomever) to track all that. | 14:54 |
| scottda | stay tuned... | 14:55 |
| *** dustins has joined #openstack-cinder | 14:55 | |
| lyarwood | scottda: yeah that's cool, looking forward to using them in Pike :) | 14:56 |
| geguileo | xyang: Yes v5.5 will still work | 15:04 |
| xyang | geguileo: ok, thanks | 15:05 |
| geguileo | xyang: But That exception should have been swallowed by the exception catching... | 15:05 |
| *** liverpooler has quit IRC | 15:05 | |
| geguileo | xyang: That happened during migration 85? | 15:06 |
| geguileo | dulek: I think we should be returning the value of cluster | 15:06 |
| geguileo | dulek: I saw it yesterday | 15:06 |
| *** arne_r has quit IRC | 15:06 | |
| xyang | geguileo: I have not got the full logs yet. I only saw a section of it | 15:06 |
| geguileo | dulek: But I haven't had time to add it | 15:06 |
| xyang | geguileo: by looking at the migration script, it should be from 85 | 15:07 |
| geguileo | xyang: I'd like to see it to be sure | 15:07 |
| xyang | geguileo: sure. I have asked for it. will let you know | 15:07 |
| geguileo | xyang: thanks | 15:07 |
| xyang | geguileo: np | 15:07 |
| dulek | geguileo: Yup, I have fun with microversions now. It's v2 API tests that are failing, so I don't really have any info about clusters available. | 15:08 |
| *** liverpooler has joined #openstack-cinder | 15:08 | |
| *** tongli has quit IRC | 15:09 | |
| geguileo | dulek: I'm having troubles running devstack, but I want to check one thing in case there's a workaround... | 15:09 |
| *** jperry has joined #openstack-cinder | 15:11 | |
| dulek | geguileo: I don't have an idea how to solve this, besides removing any asserts from the test. | 15:18 |
| *** cyberik has joined #openstack-cinder | 15:18 | |
| *** akapil has quit IRC | 15:26 | |
| *** liverpooler has quit IRC | 15:30 | |
| *** akapil has joined #openstack-cinder | 15:33 | |
| *** breitz has quit IRC | 15:33 | |
| *** mdnadeem has quit IRC | 15:33 | |
| *** breitz has joined #openstack-cinder | 15:34 | |
| *** ducttape_ has joined #openstack-cinder | 15:34 | |
| *** ducttape_ has quit IRC | 15:34 | |
| *** ducttape_ has joined #openstack-cinder | 15:35 | |
| ildikov | erlon: jgriffith: is volume name unique? | 15:37 |
| jungleboyj | ildikov, The display name? | 15:39 |
| ildikov | jungleboyj: just got a question earlier whether the new attach API should receive the volume name or not | 15:41 |
| ildikov | jungleboyj: does the volume have other name than the display name? | 15:42 |
| jungleboyj | ildikov, The volume name is not unique. Just the UUID is. You can have multiple volumes with the same name or no name at all. | 15:42 |
| ildikov | jungleboyj: tnx, that's what I thought | 15:42 |
| jungleboyj | ildikov, The 'display name' is the only name I am aware of. | 15:42 |
| jungleboyj | ildikov, You are welcome. :-) | 15:42 |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi https://review.openstack.org/425392 | 15:43 |
| ildikov | jungleboyj: haha, cool, thanks :) | 15:43 |
| jungleboyj | np | 15:44 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: HNAS: Cloned volume with different volume type https://review.openstack.org/426878 | 15:44 |
| ildikov | erlon: jgriffith: so I don't have a strong preference on the name question, although I don't really see that would add too much value to the API as we wouldn't use it to identify the volume anyhow | 15:44 |
| *** liverpooler has joined #openstack-cinder | 15:44 | |
| xyang | geguileo: http://publiclogs.emc.com/xio_ostack/EMC_XtremIO_FC/1444/logs/devstacklog.txt.gz | 15:45 |
| jungleboyj | ildikov, Is there anything helpful that it could be used for? | 15:45 |
| xyang | geguileo: looks like it only happened during migration 85 | 15:45 |
| ildikov | jungleboyj: I don't see anything at this point honestly | 15:45 |
| jungleboyj | Was there an explanation for the request? :-) | 15:46 |
| ildikov | just the question on this channel earlier | 15:47 |
| jungleboyj | I suppose it could be used in Logging which could be helpful. | 15:48 |
| geguileo | xyang: Yes that can be ignored | 15:49 |
| xyang | geguileo: thanks! | 15:49 |
| geguileo | xyang: We are catching the exception (because we anticipate it can fail) so it doesn't break the migration | 15:49 |
| geguileo | xyang: But SQLAlchemy still logs the error | 15:50 |
| xyang | geguileo: sure. I can see in the logs it actually continues with 86, 87, ... | 15:50 |
| geguileo | xyang: The only solution to prevent it from displaying would be to disable the SQLA logging durint that call | 15:50 |
| *** rcernin has quit IRC | 15:51 | |
| *** liverpooler has quit IRC | 15:51 | |
| xyang | geguileo: sure | 15:51 |
| *** liverpooler has joined #openstack-cinder | 15:53 | |
| *** mtanino has joined #openstack-cinder | 15:54 | |
| *** ircuser-1 has joined #openstack-cinder | 15:56 | |
| hemna | smcginnis, http://paste.openstack.org/show/597113/ | 15:59 |
| hemna | didn't know if you saw that yesterday | 15:59 |
| *** sv7577 has joined #openstack-cinder | 15:59 | |
| scottda | smcginnis: I just updated the meeting agenda , FYI | 15:59 |
| smcginnis | scottda: ack | 16:00 |
| smcginnis | hemna: Looks great! | 16:00 |
| *** lkuchlan_ has quit IRC | 16:01 | |
| openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: doc: verify all rst files https://review.openstack.org/310591 | 16:04 |
| *** stvnoyes has quit IRC | 16:05 | |
| *** stvnoyes has joined #openstack-cinder | 16:06 | |
| *** sv7577 has quit IRC | 16:07 | |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi https://review.openstack.org/425392 | 16:09 |
| *** _ducttape_ has joined #openstack-cinder | 16:09 | |
| erlon | ildikov: jgriffith: that is just the behaviour of most of Cinder commands, so, users expect consistency | 16:10 |
| *** JoseMello has joined #openstack-cinder | 16:10 | |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi https://review.openstack.org/425392 | 16:10 |
| *** ducttape_ has quit IRC | 16:13 | |
| ildikov | erlon: jgriffith: if there's a good use of that attribute I'm not against; although this one is a completely new API, therefore if we see it completely unnecessary we might choose not to add it | 16:13 |
| ildikov | erlon: jgriffith: currently the only thing we might use is logging, but as it's not unique the UUID is still needed | 16:14 |
| *** sv7577 has joined #openstack-cinder | 16:15 | |
| erlon | ildikov: jgriffith: the clients (not sure if is really the clients who check that) try to do an operation and if the name is unique for the tenant, the operation succeeds, its not only for logging, its for convenience I believe | 16:16 |
| *** sv7577 has quit IRC | 16:17 | |
| erlon | ildikov: jgriffith: if the checking is done in the API, so that is something we should work on as OSC heavily rely on that to keep consistency with the current behaviour | 16:17 |
| *** windmoon has joined #openstack-cinder | 16:20 | |
| pjm6 | Hi there | 16:22 |
| pjm6 | anyone here tried to run cinder in a Dell EqualLogic? I'm getting an error: "cli-settings confirmation off" | 16:22 |
| *** windmoon has quit IRC | 16:22 | |
| pjm6 | Bad or unexpected resposne from the storage volume backend API: SSH Command failed after '6' attemtps: 'cli-settings confirmation off' | 16:22 |
| pjm6 | if I do this in the Dell via SSH i have no problems | 16:23 |
| *** diablo_rojo_phon has joined #openstack-cinder | 16:24 | |
| *** alonma has quit IRC | 16:24 | |
| smcginnis | pjm6: Our meeting is taking place right now, but I might be able to help, or at least get you going in the right direction, in a little while. | 16:24 |
| *** alonma has joined #openstack-cinder | 16:25 | |
| pjm6 | thanks smcginnis :) | 16:25 |
| pjm6 | I think that just found the problem, I wasn't using the default Group name | 16:25 |
| pjm6 | and I did specified it, I just corrected and that error gone, will try to see if its working :) | 16:26 |
| smcginnis | pjm6: Great! | 16:27 |
| *** xinli has joined #openstack-cinder | 16:27 | |
| *** alonma has quit IRC | 16:29 | |
| pjm6 | so far, so good :) it seems working smcginnis | 16:31 |
| jgriffith | ildikov erlon huh? | 16:31 |
| jgriffith | Names are not unique, and should be banned if you ask me :) | 16:31 |
| jgriffith | UUID's FTW :) | 16:31 |
| ildikov | jgriffith: +1 :) | 16:32 |
| smcginnis | pjm6: Excellent! CI jobs run on that, so should be good. Feel free to ask if anything comes up though. | 16:32 |
| jgriffith | but I'm not following the conversation, and my scroll back seems truncated | 16:32 |
| erlon | jgriffith: O_o how that came to exist so far then? | 16:32 |
| * ildikov is not user friendly this morning :) | 16:32 | |
| jgriffith | erlon how what came to exist? | 16:32 |
| erlon | jgriffith: calling resources by their names | 16:32 |
| jgriffith | erlon don't | 16:33 |
| jgriffith | people whined and added that in but it's bad | 16:33 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix notification short-circuit https://review.openstack.org/427647 | 16:33 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix CopyVolumeToImageTestCase max recursion depth https://review.openstack.org/427800 | 16:33 |
| jgriffith | because we don't require/guarantee unique names and the whole thing sucks | 16:33 |
| jgriffith | just adds confusion | 16:33 |
| erlon | jgriffith: from a UPOV its a good idea | 16:33 |
| pjm6 | smcginnis: thank you :) | 16:33 |
| jgriffith | I don't know what UPOV is | 16:33 |
| geguileo | dulek: ^ That's the fix for the mas recursion error | 16:34 |
| jgriffith | I don't even know what your'e asking still :) | 16:34 |
| erlon | jgriffith: see? you know nothing about user experience :) | 16:34 |
| erlon | jgriffith: jgriffith: user point of view | 16:34 |
| jgriffith | but historically I will lose any argument along the lines of "banning names" :) | 16:34 |
| dulek | geguileo: I've already opened it. :) | 16:35 |
| jgriffith | erlon Yeah, I just actually talk to them instead of sitting on panels or making up silly groups and mettings of OS devs that talk about them and work in a vacuum | 16:35 |
| geguileo | dulek: thanks | 16:35 |
| erlon | jgriffith: but seriously, we should keep consistency as much as possible with the behaviour of the other commands of the CLI | 16:35 |
| ildikov | erlon: well, in smaller environments if users are careful enough to choose unique names it might make life easier, but anything different than that I agree with jgriffith here | 16:35 |
| erlon | users rely on that, it was the first thing I notice using the new command | 16:36 |
| jgriffith | erlon hoenstly if you want to go names, go names I don't care any more; but you need to add uniqueness requirements if you're going to do it | 16:36 |
| jgriffith | erlon and yes, what we have now thanks to a certain individual SUCKS because some Client things allow it, some don't | 16:36 |
| jgriffith | and that's crappy | 16:36 |
| ildikov | jgriffith: or an extra check that fails when the name is not unique and then the user can retry with the UUID | 16:37 |
| erlon | ildikov: even in big deployments, some tenants rely in the names to better management of their resources | 16:37 |
| jgriffith | erlon still unclear what you're proposing | 16:38 |
| erlon | ildikov: +1 thats how its done so far | 16:38 |
| jgriffith | erlon you want to convert everything to use names instead of UUIDs? | 16:38 |
| *** tongli has joined #openstack-cinder | 16:38 | |
| erlon | jgriffith: no no, you can still use the UUID but if the user passes volume-name and its is unique it needs to work | 16:39 |
| ildikov | jgriffith: to try to use volume name as an alternative to UUID to identify the volume, so the user does not have to look up the UUID, but use the nice and shiny names | 16:39 |
| jgriffith | erlon yeah you mean if they pass it instead of UUID? | 16:39 |
| *** jdurgin1 has joined #openstack-cinder | 16:40 | |
| erlon | jgriffith: yes | 16:40 |
| ildikov | jgriffith: so UUID stays, but if only the name is specified, look up the volumes by name, if you found multiple or none you fail, if one the request goes as if you would've had UUID specified there | 16:40 |
| erlon | jgriffith: try: cinder create 1 --name v1 | 16:40 |
| erlon | jgriffith: then cinder delete v1 | 16:40 |
| *** alonma has joined #openstack-cinder | 16:40 | |
| jgriffith | erlon I don't like it but whatever | 16:41 |
| erlon | ildikov: jgriffith, yes I believe by the way that there is already a function on cinder that does that | 16:41 |
| jgriffith | erlon keep in mind you're bighting off a pretty large chunk of work | 16:41 |
| *** nikeshm has joined #openstack-cinder | 16:41 | |
| jgriffith | snapshots, backups, volume-types, consistency-groups etc etc | 16:41 |
| jgriffith | erlon there's a function intended to do it; and it's half done work | 16:41 |
| jgriffith | so would be cool to finish it or remove it (I'm guessing the vote is finish it) | 16:42 |
| jgriffith | erlon FYI, I was AWS a good bit and have never had an issue with the fact that my resources don't have "names" to access them by | 16:42 |
| jgriffith | in fact I prefer it | 16:43 |
| jgriffith | s/was/use/ | 16:43 |
| ildikov | we can report a bug for this and add wishlist or higher priority, cores here can decide | 16:43 |
| jgriffith | and I'm not sure, but last time I checked there are a few people using that and have a pretty descent UPOV | 16:43 |
| *** knomura_ has quit IRC | 16:43 | |
| nikeshm | hi do any obe face CI failures due to this https://review.openstack.org/#/c/427561/ | 16:43 |
| nikeshm | s/obe/one | 16:44 |
| erlon | jgriffith: its not the end of life if you dont have this at all, but have in half of the commands and not the others is a big problem | 16:44 |
| jgriffith | erlon I'll agree with you on that | 16:44 |
| *** alonma has quit IRC | 16:45 | |
| erlon | about the chunk of work, that actually already works for snapshots, types, volumes, most of stuff | 16:46 |
| erlon | jgriffith: not sure about the consistency groups | 16:46 |
| *** knomura has joined #openstack-cinder | 16:47 | |
| ildikov | erlon: jgriffith: my vote is on creating a bug report to document this and then we can decide when to get it done | 16:49 |
| ildikov | erlon: jgriffith: unless either of you wants to look into it now | 16:49 |
| *** alonmarx has joined #openstack-cinder | 16:50 | |
| jgriffith | ildikov seems reasonable to me | 16:50 |
| *** tesseract has quit IRC | 16:56 | |
| *** knomura has quit IRC | 16:56 | |
| *** alonma has joined #openstack-cinder | 16:56 | |
| *** diablo_rojo has joined #openstack-cinder | 16:59 | |
| *** tongli has quit IRC | 17:00 | |
| *** psachin has quit IRC | 17:01 | |
| *** alonma has quit IRC | 17:01 | |
| pots | smcginnis i think you'd suggested you would revert the patch depending on how many CIs we got running before the freeze | 17:01 |
| pots | smcginnis i can submit a patch though, if you like | 17:01 |
| smcginnis | pots: Last I recall, I had said submitting a patch for the one that is reporting would be accetable. | 17:02 |
| smcginnis | pots: But the bigger issue here is there's going to need to be a new driver submitted for Pike, because the inherited DotHill one will be removed. | 17:02 |
| smcginnis | If CI isn't fixed on that very very soon. | 17:02 |
| pots | smcginnis the other CIs are very close, one is running in the sandbox now | 17:03 |
| smcginnis | pots: OK, great. | 17:04 |
| pots | smcginnis but there is probably no more need for a "dot hill" branded driver since that company was acquired | 17:04 |
| jungleboyj | pots, So, what is the status on the Lenovo CI? | 17:04 |
| pots | smcginnis so we need to remove the documentation, but 99.9% of the driver has to remain to support the other drivers. | 17:05 |
| pots | the Lenovo CI is running against the sandbox but I'm having trouble with the FC passthrough | 17:05 |
| pots | i think the docs are out of date for that? | 17:05 |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi https://review.openstack.org/425392 | 17:06 |
| pots | So I'd really like to keep the Lenovo driver in the 'supported' column since we're committed to getting the CI up and obviously to supporting the driver | 17:06 |
| * jungleboyj would like that too. :-) | 17:07 | |
| jungleboyj | pots, Can we at least start reporting the iSCSI results? | 17:07 |
| pots | yes, i can get that going today | 17:08 |
| *** e0ne has quit IRC | 17:08 | |
| *** guyr-infinidat has quit IRC | 17:08 | |
| *** jordanP has quit IRC | 17:08 | |
| jungleboyj | smcginnis, Thoughts? | 17:09 |
| *** kfarr has quit IRC | 17:09 | |
| pots | smcginnis sorry, i put too much stock in the fact that the drivers are all identical and didn't realize how much work it would be to duplicate the same CI with the latest OS release | 17:10 |
| smcginnis | jungleboyj: We can get the one marked supported, but we need to see CI on the rest. | 17:11 |
| erlon | ildikov: jgriffith: ok: https://bugs.launchpad.net/cinder/+bug/1661043 | 17:11 |
| openstack | Launchpad bug 1661043 in Cinder "Cinder attachment-* commands does not support names" [Undecided,New] | 17:11 |
| smcginnis | bbl | 17:11 |
| jungleboyj | smcginnis, Which one? | 17:12 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: [WIP][PoC] RCPAPI microversion constants https://review.openstack.org/427817 | 17:13 |
| *** nkrinner is now known as nkrinner_afk | 17:14 | |
| *** akapil has quit IRC | 17:14 | |
| *** akapil has joined #openstack-cinder | 17:14 | |
| *** lpetrut has quit IRC | 17:17 | |
| *** ggrabows has joined #openstack-cinder | 17:17 | |
| *** akapil_ has joined #openstack-cinder | 17:19 | |
| hemna | smcginnis, https://github.com/hemna/third-party-ci-tools/tree/hemna/monitoring/lastcomment-scoreboard | 17:20 |
| *** akapil has quit IRC | 17:21 | |
| jungleboyj | hemna, Nice. | 17:22 |
| erlon | smcginnis: Sean, can you give a look on this when you have a change? https://review.openstack.org/#/c/414076/ | 17:24 |
| ildikov | erlon: tnx | 17:25 |
| *** akapil_ has quit IRC | 17:25 | |
| hemna | soo...... | 17:26 |
| hemna | I just ran the generate_driver_list | 17:27 |
| geguileo | dulek: How about adding an attribute to show command like we have now the volume extension volume_host_attribute but for cluster? | 17:27 |
| hemna | and I'm seeing some drivers not having a CI_WIKI_NAME associated with them | 17:27 |
| nikeshm | hi do any one face CI failures due to this https://review.openstack.org/#/c/427561/ | 17:28 |
| nikeshm | i am facing it in Kaminario CI | 17:29 |
| nikeshm | pots: you might try nova passthrough instead of passthrough using scripts | 17:30 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: ZTE: Added CI_WIKI_NAME https://review.openstack.org/427823 | 17:30 |
| nikeshm | when i was maintaining those CIs, passthrough was failing for higher versions of Ubuntu | 17:31 |
| erlon | geguileo: dulek: guys, can you give a look in those changes when you have a chance? The migration-matrix is only failing on ceph (which apparently does not support attached migration): https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:fix-migration-bugs | 17:31 |
| nikeshm | with the passthrough scripts | 17:31 |
| erlon | ildikov: np | 17:31 |
| hemna | nikeshm, the FC passthrough was failing ? | 17:32 |
| *** Zack2 has left #openstack-cinder | 17:32 | |
| nikeshm | hemna: when i was trying seting up CI with FC passthrough scripts (more than one year back), i faced issues with higher versions of Ubuntu | 17:36 |
| *** raunak has joined #openstack-cinder | 17:36 | |
| nikeshm | not failing | 17:37 |
| *** raunak has quit IRC | 17:37 | |
| nikeshm | passthrough did not work | 17:37 |
| nikeshm | for me that time | 17:37 |
| *** ericyoung has joined #openstack-cinder | 17:39 | |
| hemna | I think we ran into that problem at HPE too for a while. | 17:40 |
| *** ericyoung has quit IRC | 17:45 | |
| *** ericyoung has joined #openstack-cinder | 17:45 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder: Removes getfattr from Quobyte Cinder driver https://review.openstack.org/427833 | 17:45 |
| *** ericyoung has quit IRC | 17:47 | |
| *** ericyoung has joined #openstack-cinder | 17:48 | |
| *** ericyoung has quit IRC | 17:49 | |
| *** ericyoung has joined #openstack-cinder | 17:49 | |
| *** esker has joined #openstack-cinder | 17:50 | |
| *** cyberik has quit IRC | 17:52 | |
| *** _ducttape_ has quit IRC | 17:54 | |
| *** sv7577 has joined #openstack-cinder | 17:54 | |
| *** ducttape_ has joined #openstack-cinder | 17:54 | |
| *** kaisers has quit IRC | 17:54 | |
| *** ducttape_ has quit IRC | 17:54 | |
| *** ducttape_ has joined #openstack-cinder | 17:55 | |
| *** ducttape_ has quit IRC | 17:55 | |
| *** ducttape_ has joined #openstack-cinder | 17:56 | |
| *** gaurangt has left #openstack-cinder | 17:56 | |
| *** aarefiev is now known as aarefiev_afk | 17:57 | |
| *** sv7577 has quit IRC | 18:01 | |
| *** harlowja_ has joined #openstack-cinder | 18:05 | |
| *** harlowja has quit IRC | 18:06 | |
| *** mkoderer has quit IRC | 18:06 | |
| openstackgerrit | Merged openstack/cinder: Move release notes to correct location https://review.openstack.org/427542 | 18:07 |
| *** Apoorva has joined #openstack-cinder | 18:10 | |
| *** xyang has quit IRC | 18:12 | |
| *** Jack_Iv has quit IRC | 18:12 | |
| *** kaisers has joined #openstack-cinder | 18:13 | |
| jgriffith | scottda umm, has anybody even tried running tempest tests with the Cinder API endpoint set to V3? | 18:14 |
| jgriffith | scottda it's kinda ugly | 18:14 |
| *** xyang has joined #openstack-cinder | 18:15 | |
| *** lpetrut has joined #openstack-cinder | 18:16 | |
| jgriffith | scottda I think I'm going to give up on this honestly | 18:17 |
| geguileo | smcginnis: If you have time for a couple of easy reviews: spureous erros in unit tests (https://review.openstack.org/#/c/427800/), and this one has broken vitrage CI (https://review.openstack.org/#/c/427647/) | 18:17 |
| smcginnis | hemna: Filtered out to the non-reporting CIs that we have not aleady marked nonsupported. | 18:17 |
| smcginnis | hemna: http://paste.openstack.org/show/597255/ | 18:17 |
| smcginnis | geguileo: Sure, I can take a look now. | 18:17 |
| smcginnis | jgriffith: Bloodbath? | 18:17 |
| geguileo | smcginnis: Awesome!, Thanks! | 18:17 |
| smcginnis | geguileo: Oh good, had meant to get back to checking on the short circuit one. | 18:18 |
| *** xyang has quit IRC | 18:18 | |
| *** dhinesh has joined #openstack-cinder | 18:19 | |
| geguileo | smcginnis: patrickeast Thanks! | 18:19 |
| geguileo | patrickeast: This one already had your +2 before I changed the commit message: https://review.openstack.org/#/c/424747/ | 18:20 |
| geguileo | ;-) | 18:20 |
| *** diablo_rojo has quit IRC | 18:21 | |
| geguileo | patrickeast: thanks! | 18:22 |
| patrickeast | geguileo: np | 18:22 |
| hemna | smcginnis, cool, that's a short list | 18:23 |
| hemna | which is good | 18:23 |
| smcginnis | hemna: Yes, much better than I had originally feared. | 18:24 |
| *** diablo_rojo has joined #openstack-cinder | 18:24 | |
| *** raunak has joined #openstack-cinder | 18:24 | |
| *** xyang has joined #openstack-cinder | 18:27 | |
| pots | nikeshm: thanks, i've tried both so far, not sure how to troubleshoot the nova passthrough | 18:27 |
| pots | hemna: do you remember how it was solved at HPE? | 18:28 |
| *** pcaruana has quit IRC | 18:28 | |
| *** xyang has quit IRC | 18:29 | |
| *** xyang has joined #openstack-cinder | 18:29 | |
| hemna | I think I had rewritten most of the FC passthrough stuff | 18:30 |
| hemna | the problem was that it wasn't correctly unbinding the HBA | 18:30 |
| hemna | I don't have access to any of that code now, unless asselin__ put it upstream | 18:30 |
| asselin__ | it's all in the 3rd party ci tools repo | 18:32 |
| pots | hemna: i think i tried those scripts but they didn't work for me and i couldn't find any doc to double-check my work. we use a version of them on the HP MSA CI successfully but that's an older version of everything. | 18:32 |
| hemna | asselin__, ok that's what I was hoping for | 18:32 |
| asselin__ | http://git.openstack.org/cgit/openstack/third-party-ci-tools/tree/provisioning_scripts/fibre_channel | 18:32 |
| pots | is anyone using the (current?) method of FC pass-through in Nova? | 18:33 |
| asselin__ | but....openstack nova *should* support this now out of the box. | 18:33 |
| asselin__ | those scripts were used back in kilo | 18:33 |
| *** sv7577 has joined #openstack-cinder | 18:34 | |
| smcginnis | patrickeast: Aren't you doing something with this? ^^ | 18:35 |
| smcginnis | For those interested - some old reviews sitting out there: http://cinderstats-dellstorage.rhcloud.com/cinder-openreviews.html | 18:35 |
| pots | asselin__: i followed the PCI passthrough instructions here: http://docs.openstack.org/admin-guide/compute-pci-passthrough.html | 18:36 |
| asselin__ | looks promising | 18:37 |
| sv7577 | Can someone review https://review.openstack.org/#/c/407123/ | 18:38 |
| openstackgerrit | Merged openstack/cinder: Reduced the complexity of the main() method https://review.openstack.org/250914 | 18:47 |
| *** sv7577 has quit IRC | 18:51 | |
| *** mvk has quit IRC | 18:57 | |
| *** mjura has quit IRC | 18:58 | |
| *** alonma has joined #openstack-cinder | 18:58 | |
| *** ksumit has joined #openstack-cinder | 18:59 | |
| *** yumiriam has quit IRC | 19:00 | |
| jungleboyj | Oh, look at that pots you are already talking to asselin__ :-) | 19:00 |
| jungleboyj | So, talk to asselin__ :-) | 19:01 |
| *** ducttape_ has quit IRC | 19:02 | |
| *** alonma has quit IRC | 19:02 | |
| *** ducttape_ has joined #openstack-cinder | 19:03 | |
| *** alonma has joined #openstack-cinder | 19:04 | |
| openstackgerrit | Rohan Arora proposed openstack/cinder: Nosec audit url open issue from Bandit https://review.openstack.org/424356 | 19:06 |
| *** knomura has joined #openstack-cinder | 19:08 | |
| *** alonma has quit IRC | 19:08 | |
| *** lpetrut has quit IRC | 19:08 | |
| *** adisky_ has quit IRC | 19:09 | |
| *** lpetrut has joined #openstack-cinder | 19:11 | |
| *** Jack_Iv has joined #openstack-cinder | 19:12 | |
| *** knomura has quit IRC | 19:13 | |
| *** kfarr has joined #openstack-cinder | 19:13 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Hitachi VSP: Specify compute nodes and copy ports https://review.openstack.org/414076 | 19:15 |
| *** Jack_Iv has quit IRC | 19:17 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc https://review.openstack.org/427435 | 19:19 |
| *** harlowja_ has quit IRC | 19:19 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Add --metadata option to API v2 cinder list command again https://review.openstack.org/427875 | 19:25 |
| patrickeast | asselin__: pots: yea the nova pci passthrough is working pretty well for me and the Pure CI, we still have some problems but its mostly switch related | 19:25 |
| *** ggrabows has quit IRC | 19:25 | |
| *** ggrabows has joined #openstack-cinder | 19:25 | |
| patrickeast | i definitely recommend doing that over using the passthrough scripts | 19:25 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Hitachi VSP: Specify compute nodes and copy ports https://review.openstack.org/414076 | 19:26 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc https://review.openstack.org/427435 | 19:29 |
| *** e0ne has joined #openstack-cinder | 19:30 | |
| *** ggrabows has quit IRC | 19:31 | |
| *** ociuhandu has quit IRC | 19:33 | |
| *** mvk has joined #openstack-cinder | 19:34 | |
| *** akapil has joined #openstack-cinder | 19:36 | |
| *** alonmarx has quit IRC | 19:36 | |
| *** xyang has quit IRC | 19:37 | |
| *** Apoorva has quit IRC | 19:38 | |
| *** xyang has joined #openstack-cinder | 19:39 | |
| *** esker has quit IRC | 19:40 | |
| *** ggrabows has joined #openstack-cinder | 19:40 | |
| *** akapil has quit IRC | 19:41 | |
| *** ggrabows has quit IRC | 19:43 | |
| *** ggrabows has joined #openstack-cinder | 19:44 | |
| *** ChubYann has joined #openstack-cinder | 19:45 | |
| openstackgerrit | Chris M proposed openstack/cinder: Add SUPPORTED flag to HP MSA driver https://review.openstack.org/427881 | 19:46 |
| *** ggrabows_ has joined #openstack-cinder | 19:49 | |
| *** ggrabows has quit IRC | 19:50 | |
| *** viks has joined #openstack-cinder | 19:51 | |
| *** mtanino has quit IRC | 19:51 | |
| *** ociuhandu has joined #openstack-cinder | 19:53 | |
| *** cknight has joined #openstack-cinder | 19:53 | |
| *** diablo_rojo has quit IRC | 19:54 | |
| *** e0ne has quit IRC | 19:55 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Add --metadata option to API v2 cinder list command again https://review.openstack.org/427875 | 19:55 |
| *** mtanino has joined #openstack-cinder | 19:59 | |
| *** e0ne has joined #openstack-cinder | 19:59 | |
| *** Jack_Iv has joined #openstack-cinder | 20:00 | |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Convert dict to OVO in backup export_record rpcapi https://review.openstack.org/425392 | 20:00 |
| openstackgerrit | Chris M proposed openstack/cinder: Add SUPPORTED flag to Lenovo iSCSI driver https://review.openstack.org/427886 | 20:00 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell EMC: Moved SC driver under dell_emc https://review.openstack.org/427435 | 20:01 |
| pots | patrickeast: thanks. do you ever see "Claim pci failed" errors when trying to pass through a device? Do you know what the disposition of the PCI device needs to be before Nova grabs it (e.g. bound to a stub driver or unbound)? | 20:03 |
| *** gouthamr has quit IRC | 20:03 | |
| *** crushil has quit IRC | 20:06 | |
| *** crushil has joined #openstack-cinder | 20:06 | |
| *** mtanino has quit IRC | 20:09 | |
| *** mtanino has joined #openstack-cinder | 20:12 | |
| *** ericyoung has quit IRC | 20:13 | |
| *** e0ne has quit IRC | 20:17 | |
| *** crushil has quit IRC | 20:20 | |
| *** crushil has joined #openstack-cinder | 20:20 | |
| *** diablo_rojo_phon has quit IRC | 20:20 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up release notes for Ocata https://review.openstack.org/427896 | 20:20 |
| *** porrua has quit IRC | 20:21 | |
| patrickeast | pots: I haven't seen that particular error before, but as far as state goes you just need to make sure that it *can* be stubbed out | 20:21 |
| patrickeast | pots: for mine, when something bad happens I just re-bind them all to their qla2xxx drivers | 20:21 |
| *** e0ne has joined #openstack-cinder | 20:21 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up release notes for Ocata https://review.openstack.org/427896 | 20:21 |
| patrickeast | pots: and nova/libvirt take care of it from there | 20:21 |
| pots | patrickeast: thanks | 20:22 |
| *** nikeshm has quit IRC | 20:26 | |
| *** timcl has quit IRC | 20:28 | |
| *** e0ne has quit IRC | 20:28 | |
| *** jamielennox|away is now known as jamielennox | 20:30 | |
| *** e0ne has joined #openstack-cinder | 20:32 | |
| *** timcl has joined #openstack-cinder | 20:33 | |
| *** JoseMello has quit IRC | 20:35 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up release notes for Ocata https://review.openstack.org/427896 | 20:36 |
| *** e0ne has quit IRC | 20:37 | |
| *** harlowja has joined #openstack-cinder | 20:39 | |
| *** e0ne has joined #openstack-cinder | 20:41 | |
| *** e0ne has quit IRC | 20:47 | |
| *** e0ne has joined #openstack-cinder | 20:51 | |
| *** esker has joined #openstack-cinder | 20:53 | |
| *** e0ne has quit IRC | 20:55 | |
| *** diablo_rojo has joined #openstack-cinder | 20:55 | |
| *** Jack_Iv has quit IRC | 20:57 | |
| *** Jack_Iv has joined #openstack-cinder | 20:57 | |
| *** Jack_Iv_ has joined #openstack-cinder | 20:58 | |
| *** _ducttape_ has joined #openstack-cinder | 20:58 | |
| smcginnis | pots: Not that you need another thing to worry about, but when you get some time, can you update your CI trigger: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_trigger_my_CI_to_rerun_on_gerrit_comments.3F | 20:58 |
| smcginnis | pots: Otherwise that rechecks the infra tests too. Which then rechecks all CIs that trigger on Jenkins +1. | 20:59 |
| *** e0ne has joined #openstack-cinder | 21:00 | |
| *** ggrabows_ has quit IRC | 21:01 | |
| pots | will do--i think i got it right on the Lenovo CI | 21:01 |
| *** liverpooler has quit IRC | 21:01 | |
| smcginnis | pots: Thanks! | 21:01 |
| *** ggrabows has joined #openstack-cinder | 21:01 | |
| *** ducttape_ has quit IRC | 21:02 | |
| *** Jack_Iv has quit IRC | 21:02 | |
| *** gouthamr has joined #openstack-cinder | 21:02 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Clean up release notes for Ocata https://review.openstack.org/427896 | 21:03 |
| *** jose-phillips has joined #openstack-cinder | 21:05 | |
| *** diablo_rojo has quit IRC | 21:11 | |
| *** alonmarx has joined #openstack-cinder | 21:18 | |
| *** erlon has quit IRC | 21:20 | |
| *** cknight has quit IRC | 21:20 | |
| *** alonma has joined #openstack-cinder | 21:20 | |
| *** ggrabows has quit IRC | 21:23 | |
| *** alonma has quit IRC | 21:25 | |
| *** eharney has quit IRC | 21:28 | |
| *** erlon has joined #openstack-cinder | 21:30 | |
| openstackgerrit | Merged openstack/cinder: Fix CopyVolumeToImageTestCase max recursion depth https://review.openstack.org/427800 | 21:30 |
| openstackgerrit | Yucong Feng proposed openstack/cinder: Resolve Brocade HTTPS connection error https://review.openstack.org/416371 | 21:30 |
| *** kfarr has quit IRC | 21:32 | |
| *** _ducttape_ has quit IRC | 21:37 | |
| *** catintheroof has quit IRC | 21:37 | |
| *** ducttape_ has joined #openstack-cinder | 21:37 | |
| *** catintheroof has joined #openstack-cinder | 21:37 | |
| *** Apoorva has joined #openstack-cinder | 21:38 | |
| *** Apoorva has quit IRC | 21:40 | |
| *** Apoorva has joined #openstack-cinder | 21:41 | |
| *** eharney has joined #openstack-cinder | 21:44 | |
| *** ggrabows has joined #openstack-cinder | 21:45 | |
| *** e0ne has quit IRC | 21:53 | |
| *** catintheroof has quit IRC | 21:57 | |
| *** catintheroof has joined #openstack-cinder | 21:58 | |
| *** diablo_rojo_phon has joined #openstack-cinder | 22:01 | |
| *** alyson_ has quit IRC | 22:07 | |
| *** gouthamr has quit IRC | 22:07 | |
| *** knomura has joined #openstack-cinder | 22:08 | |
| *** ducttape_ has quit IRC | 22:09 | |
| *** ducttape_ has joined #openstack-cinder | 22:10 | |
| *** ksumit has quit IRC | 22:10 | |
| *** knomura has quit IRC | 22:13 | |
| *** ggrabows has quit IRC | 22:13 | |
| *** ducttape_ has quit IRC | 22:15 | |
| *** lpetrut has quit IRC | 22:19 | |
| *** akapil has joined #openstack-cinder | 22:19 | |
| *** akapil has quit IRC | 22:24 | |
| *** nileo has joined #openstack-cinder | 22:28 | |
| *** gouthamr has joined #openstack-cinder | 22:33 | |
| *** ksumit has joined #openstack-cinder | 22:34 | |
| *** mtbar has joined #openstack-cinder | 22:35 | |
| *** catintheroof has quit IRC | 22:35 | |
| *** mtbar has quit IRC | 22:37 | |
| *** Jack_Iv_ has quit IRC | 22:38 | |
| *** crushil has quit IRC | 22:39 | |
| *** viks has quit IRC | 22:46 | |
| *** ducttape_ has joined #openstack-cinder | 22:47 | |
| *** ducttape_ has quit IRC | 22:49 | |
| *** _ducttape_ has joined #openstack-cinder | 22:49 | |
| *** tlian has quit IRC | 22:51 | |
| *** _ducttape_ has quit IRC | 22:53 | |
| *** dustins has quit IRC | 22:54 | |
| *** crushil has joined #openstack-cinder | 22:57 | |
| *** tlian has joined #openstack-cinder | 22:58 | |
| *** alonmarx has quit IRC | 22:59 | |
| hemna | smcginnis, https://github.com/hemna/third-party-ci-tools/tree/hemna/monitoring/lastcomment-scoreboard | 23:04 |
| *** raunak has quit IRC | 23:04 | |
| hemna | more reworking/refactoring | 23:04 |
| hemna | new cireporter.py script | 23:04 |
| hemna | that generates JSON output | 23:04 |
| smcginnis | hemna: Sweet | 23:05 |
| hemna | it can dump it to stdout or a file too | 23:05 |
| smcginnis | hemna: That should be handy. | 23:05 |
| hemna | thinking of writing a cinder script that runs the generate_driver_list, sucks out the CI names, and then uses that to populate the file needed to kick off the cireporter.py script and output as son. | 23:05 |
| hemna | json | 23:05 |
| smcginnis | That really would be ideal. Then we don't have to try to maintain a separate yaml file for things. | 23:06 |
| hemna | yah | 23:06 |
| smcginnis | Added this to my backlog to try to automate publishing results so anyone can look whenever they want. | 23:07 |
| hemna | the json output is nice to view in chrome w/ a json formatter | 23:07 |
| *** jperry has quit IRC | 23:09 | |
| *** tlian2 has joined #openstack-cinder | 23:10 | |
| *** tlian has quit IRC | 23:13 | |
| hemna | smcginnis, http://paste.openstack.org/show/597284/ and JSON http://paste.openstack.org/show/597285/ | 23:15 |
| *** tlian2 has quit IRC | 23:15 | |
| *** tlian has joined #openstack-cinder | 23:24 | |
| *** alonma has joined #openstack-cinder | 23:30 | |
| *** alonma has quit IRC | 23:34 | |
| *** xyang has quit IRC | 23:35 | |
| *** raunak has joined #openstack-cinder | 23:43 | |
| *** xyang has joined #openstack-cinder | 23:46 | |
| *** s-shiono has joined #openstack-cinder | 23:47 | |
| *** s-shiono_ has joined #openstack-cinder | 23:50 | |
| *** ducttape_ has joined #openstack-cinder | 23:51 | |
| *** s-shiono has quit IRC | 23:52 | |
| *** xyang has quit IRC | 23:54 | |
| *** xyang has joined #openstack-cinder | 23:57 | |
| *** xinli has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!