*** EinstCrazy has quit IRC | 00:03 | |
*** zhangjn has quit IRC | 00:04 | |
*** salv-orlando has joined #openstack-cinder | 00:21 | |
*** salv-orl_ has joined #openstack-cinder | 00:26 | |
*** salv-orlando has quit IRC | 00:27 | |
*** daneyon has joined #openstack-cinder | 00:41 | |
*** chlong has joined #openstack-cinder | 00:42 | |
*** dimsum__ has joined #openstack-cinder | 00:45 | |
*** diegows has quit IRC | 00:46 | |
*** dimsum__ has quit IRC | 00:47 | |
*** ilackarms has quit IRC | 00:48 | |
*** ilackarms2 has joined #openstack-cinder | 00:48 | |
*** zhangjn has joined #openstack-cinder | 00:55 | |
*** daneyon_ has joined #openstack-cinder | 00:56 | |
*** EinstCrazy has joined #openstack-cinder | 00:57 | |
*** zhangjn has quit IRC | 00:57 | |
*** zhangjn has joined #openstack-cinder | 00:58 | |
*** daneyon has quit IRC | 00:59 | |
*** salv-orl_ has quit IRC | 01:00 | |
*** baojg has joined #openstack-cinder | 01:02 | |
*** mss has quit IRC | 01:03 | |
*** jwcroppe has joined #openstack-cinder | 01:09 | |
*** ilackarms2 has quit IRC | 01:10 | |
*** ilackarms2 has joined #openstack-cinder | 01:10 | |
*** stevemar has joined #openstack-cinder | 01:16 | |
*** daneyon_ has quit IRC | 01:23 | |
*** gouthamr_ has joined #openstack-cinder | 01:28 | |
*** leeantho has joined #openstack-cinder | 01:30 | |
*** gouthamr has quit IRC | 01:32 | |
*** garthb has joined #openstack-cinder | 01:34 | |
*** davechen has joined #openstack-cinder | 01:35 | |
*** davechen1 has joined #openstack-cinder | 01:39 | |
*** davechen has quit IRC | 01:41 | |
*** Lee1092 has joined #openstack-cinder | 01:42 | |
*** dimsum__ has joined #openstack-cinder | 01:45 | |
*** jwcroppe has quit IRC | 01:56 | |
*** dimsum__ has quit IRC | 02:00 | |
*** haomaiwa_ has joined #openstack-cinder | 02:07 | |
*** EinstCrazy has quit IRC | 02:09 | |
*** DericHorn-HP has joined #openstack-cinder | 02:18 | |
*** haomaiwa_ has quit IRC | 02:18 | |
*** EinstCrazy has joined #openstack-cinder | 02:19 | |
*** chenying1 has joined #openstack-cinder | 02:19 | |
*** DericHorn-HP has quit IRC | 02:20 | |
*** esker has quit IRC | 02:26 | |
*** DericHorn-HP has joined #openstack-cinder | 02:32 | |
*** EinstCrazy has quit IRC | 02:34 | |
*** mragupat has joined #openstack-cinder | 02:34 | |
*** mragupat_ has joined #openstack-cinder | 02:35 | |
*** DericHorn-HP has quit IRC | 02:36 | |
*** mragupat has quit IRC | 02:39 | |
*** haomaiwa_ has joined #openstack-cinder | 02:48 | |
*** gouthamr has joined #openstack-cinder | 02:51 | |
*** baojg has quit IRC | 02:52 | |
*** gouthamr_ has quit IRC | 02:52 | |
*** mragupat_ has quit IRC | 02:56 | |
*** baojg has joined #openstack-cinder | 02:59 | |
*** baojg has quit IRC | 02:59 | |
*** salv-orlando has joined #openstack-cinder | 03:00 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** dimsum__ has joined #openstack-cinder | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** salv-orlando has quit IRC | 03:04 | |
*** salv-orlando has joined #openstack-cinder | 03:06 | |
*** dimsum__ has quit IRC | 03:08 | |
*** mss has joined #openstack-cinder | 03:11 | |
*** chenying1 has quit IRC | 03:14 | |
*** mss has quit IRC | 03:15 | |
*** EinstCrazy has joined #openstack-cinder | 03:22 | |
*** IlyaG has quit IRC | 03:25 | |
*** baojg has joined #openstack-cinder | 03:26 | |
*** rhagarty__ has joined #openstack-cinder | 03:26 | |
*** rhagarty has quit IRC | 03:27 | |
*** rhagarty_ has quit IRC | 03:27 | |
*** rhagarty_ has joined #openstack-cinder | 03:27 | |
*** rhagarty has joined #openstack-cinder | 03:27 | |
*** rhagarty__ has quit IRC | 03:31 | |
*** chenying1 has joined #openstack-cinder | 03:35 | |
*** zhonghua has joined #openstack-cinder | 03:37 | |
*** salv-orlando has quit IRC | 03:43 | |
*** garthb has quit IRC | 03:44 | |
*** gouthamr has quit IRC | 03:58 | |
*** mss has joined #openstack-cinder | 03:58 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** mss has quit IRC | 04:02 | |
wanghao | ilackarms: what's the backend you use, LVM? | 04:05 |
---|---|---|
*** EinstCrazy has quit IRC | 04:06 | |
*** chenying1 has quit IRC | 04:26 | |
*** DericHorn-HP has joined #openstack-cinder | 04:40 | |
*** DericHorn-HP has quit IRC | 04:48 | |
*** baojg has quit IRC | 04:50 | |
*** garthb has joined #openstack-cinder | 04:54 | |
*** haomaiwang has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** dimsum__ has joined #openstack-cinder | 05:05 | |
*** dimsum__ has quit IRC | 05:11 | |
*** sgotliv has joined #openstack-cinder | 05:14 | |
*** baojg has joined #openstack-cinder | 05:15 | |
*** ankit_ag has joined #openstack-cinder | 05:20 | |
*** dsariel_ has joined #openstack-cinder | 05:21 | |
*** Yogi1 has quit IRC | 05:25 | |
*** shausy has joined #openstack-cinder | 05:25 | |
*** Swanson has quit IRC | 05:27 | |
*** deepakcs has joined #openstack-cinder | 05:28 | |
*** garthb has quit IRC | 05:33 | |
*** dsariel_ has quit IRC | 05:36 | |
DuncanT | ilackarms2: Check for a running dd process. My guess is that you've got secure delete turned on (the default) which causes volumes to be zeroed out when you delete them | 05:36 |
*** dsariel_ has joined #openstack-cinder | 05:36 | |
*** salv-orlando has joined #openstack-cinder | 05:39 | |
*** salv-orlando has quit IRC | 05:44 | |
*** salv-orlando has joined #openstack-cinder | 05:45 | |
*** edtubill has joined #openstack-cinder | 05:54 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** zhonghua has quit IRC | 06:01 | |
*** zhonghua has joined #openstack-cinder | 06:02 | |
*** chenying1 has joined #openstack-cinder | 06:04 | |
*** stevemar has quit IRC | 06:05 | |
*** e0ne has joined #openstack-cinder | 06:10 | |
*** haomaiwang has quit IRC | 06:12 | |
*** e0ne has quit IRC | 06:14 | |
*** _afazekas has joined #openstack-cinder | 06:20 | |
*** _afazekas has quit IRC | 06:26 | |
*** salv-orlando has quit IRC | 06:28 | |
*** shausy has quit IRC | 06:38 | |
*** edtubill has quit IRC | 06:38 | |
*** shausy has joined #openstack-cinder | 06:38 | |
*** dsariel_ has quit IRC | 06:48 | |
*** sgotliv has quit IRC | 06:49 | |
*** salv-orlando has joined #openstack-cinder | 06:50 | |
openstackgerrit | Sean McCully proposed openstack/cinder: Use of ast for integers doesn't changes type https://review.openstack.org/225558 | 06:53 |
*** vlaza has joined #openstack-cinder | 06:58 | |
*** nkrinner has joined #openstack-cinder | 06:59 | |
*** e0ne has joined #openstack-cinder | 07:06 | |
*** dimsum__ has joined #openstack-cinder | 07:08 | |
*** haomaiwang has joined #openstack-cinder | 07:09 | |
*** jamielennox is now known as jamielennox|away | 07:09 | |
*** dimsum__ has quit IRC | 07:13 | |
*** leeantho has quit IRC | 07:16 | |
*** chlong has quit IRC | 07:17 | |
*** yrabl has joined #openstack-cinder | 07:18 | |
*** geguileo has joined #openstack-cinder | 07:22 | |
wanghao | ilackarms2: Agree with DuncanT, there is an option configuration 'volume_clear' you can use it to turn off secure delete, volume_clear=none. | 07:31 |
*** ronis has joined #openstack-cinder | 07:33 | |
DuncanT | ilackarms2: wanghao: Nope that turning off volume_clear means that old volume contents can leak between tenants. You definitely *don't* want that in any production multi-tenant cloud | 07:35 |
wanghao | DuncanT: Yes, you're right, volume_clear=none should just be used for test environment if you don't want to wait the clear process. | 07:38 |
DuncanT | wanghao: Or just use lvm-thin. We're hoping to switch that to default soon. | 07:39 |
*** ndipanov has joined #openstack-cinder | 07:40 | |
wanghao | DuncanT: that's great, thanks duncan. I saw the code that we didn't clear volume too if using lvm-thin now. | 07:44 |
*** lpetrut has joined #openstack-cinder | 07:46 | |
*** lpetrut1 has joined #openstack-cinder | 07:49 | |
*** lpetrut has quit IRC | 07:50 | |
*** lpetrut1 is now known as lpetrut | 07:50 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwang has joined #openstack-cinder | 08:01 | |
*** salv-orlando has quit IRC | 08:03 | |
*** zhonghua is now known as zhonghua-lee | 08:07 | |
*** dsariel_ has joined #openstack-cinder | 08:09 | |
*** sgotliv has joined #openstack-cinder | 08:09 | |
*** pradipta has joined #openstack-cinder | 08:11 | |
*** Zhongjun has joined #openstack-cinder | 08:20 | |
*** haomaiwang has quit IRC | 08:21 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of getting LUN id in Huawei driver https://review.openstack.org/209749 | 08:25 |
*** svasheka has quit IRC | 08:33 | |
*** svasheka has joined #openstack-cinder | 08:38 | |
*** alexpilotti has joined #openstack-cinder | 08:43 | |
*** chlong has joined #openstack-cinder | 08:44 | |
*** asselin__ has quit IRC | 08:44 | |
openstackgerrit | Sergey Gotliv proposed openstack/python-cinderclient: Adding pagination capabilities to snapshots and backups lists https://review.openstack.org/225582 | 08:45 |
*** e0ne has quit IRC | 08:46 | |
*** anshul has joined #openstack-cinder | 08:48 | |
*** sweston has quit IRC | 08:53 | |
*** boris-42 has quit IRC | 08:53 | |
*** patrickeast has quit IRC | 08:53 | |
*** rex_lee_ has quit IRC | 08:53 | |
*** aix has quit IRC | 08:54 | |
*** salv-orlando has joined #openstack-cinder | 08:59 | |
*** geguileo has quit IRC | 09:07 | |
*** vgridnev has joined #openstack-cinder | 09:14 | |
*** sweston has joined #openstack-cinder | 09:18 | |
*** haomaiwang has joined #openstack-cinder | 09:20 | |
*** haomaiwang has quit IRC | 09:21 | |
*** haomaiwa_ has joined #openstack-cinder | 09:21 | |
*** ociuhandu has quit IRC | 09:21 | |
*** geguileo has joined #openstack-cinder | 09:21 | |
*** ociuhandu has joined #openstack-cinder | 09:22 | |
*** ociuhandu has quit IRC | 09:22 | |
*** haomaiwa_ has quit IRC | 09:23 | |
*** haomaiwa_ has joined #openstack-cinder | 09:23 | |
*** sayali has quit IRC | 09:24 | |
*** boris-42 has joined #openstack-cinder | 09:24 | |
*** aix has joined #openstack-cinder | 09:25 | |
*** sayali has joined #openstack-cinder | 09:26 | |
*** markvoelker has quit IRC | 09:27 | |
*** baojg has quit IRC | 09:35 | |
*** IanGovett has joined #openstack-cinder | 09:42 | |
*** rex_lee_ has joined #openstack-cinder | 09:42 | |
*** davechen1 has left #openstack-cinder | 09:43 | |
*** dimsum__ has joined #openstack-cinder | 09:47 | |
*** bluex-pl has joined #openstack-cinder | 09:49 | |
openstackgerrit | chenying proposed openstack/python-cinderclient: Fix a snapshots not being deleted by using name https://review.openstack.org/223067 | 09:51 |
*** patrickeast has joined #openstack-cinder | 09:53 | |
*** ociuhandu has joined #openstack-cinder | 09:56 | |
*** salv-orlando has quit IRC | 09:57 | |
*** e0ne has joined #openstack-cinder | 09:58 | |
*** haomaiwa_ has quit IRC | 10:01 | |
*** haomaiwa_ has joined #openstack-cinder | 10:01 | |
*** baojg has joined #openstack-cinder | 10:06 | |
*** baojg has quit IRC | 10:07 | |
*** zhangjn has quit IRC | 10:10 | |
*** zhangjn has joined #openstack-cinder | 10:17 | |
*** zhangjn has quit IRC | 10:19 | |
*** zhangjn has joined #openstack-cinder | 10:26 | |
*** markvoelker has joined #openstack-cinder | 10:28 | |
*** EinstCrazy has joined #openstack-cinder | 10:28 | |
*** chenying1 has quit IRC | 10:30 | |
*** markvoelker has quit IRC | 10:32 | |
*** zhangjn has quit IRC | 10:35 | |
*** zhangjn has joined #openstack-cinder | 10:35 | |
*** yangyapeng has joined #openstack-cinder | 10:45 | |
*** aagrawal has joined #openstack-cinder | 10:46 | |
*** ankit_ag has quit IRC | 10:50 | |
*** sayali has quit IRC | 10:57 | |
*** sayali has joined #openstack-cinder | 10:58 | |
e0ne | DuncanT: hi. are you around? I've got few questions about your comments to 'use-cinder-without-nova' spec | 10:58 |
*** salv-orlando has joined #openstack-cinder | 11:00 | |
*** haomaiwa_ has quit IRC | 11:01 | |
*** 6JTAB24P4 has joined #openstack-cinder | 11:01 | |
*** aagrawal has left #openstack-cinder | 11:19 | |
*** aagrawal has joined #openstack-cinder | 11:19 | |
*** aagrawal has quit IRC | 11:19 | |
*** ankit_ag has joined #openstack-cinder | 11:20 | |
DuncanT | e0ne: Hi | 11:22 |
*** salv-orlando has quit IRC | 11:23 | |
*** salv-orlando has joined #openstack-cinder | 11:24 | |
*** markvoelker has joined #openstack-cinder | 11:29 | |
*** pradipta has quit IRC | 11:30 | |
e0ne | DuncanT: I've posted question to your comments inline https://review.openstack.org/#/c/224124/1/specs/mikata/use-cinder-without-nova.rst | 11:30 |
DuncanT | e0ne: We don't need to do anything on the server side to support trusts AFAIK | 11:31 |
e0ne | DuncanT: it would be helpful if you as a user of such feature will comment Pavlo's proposal: Ideally though a simple access to generate-attach should mark the volume as such in Cinder (you never know when user will actually attach the volume, so it is safer that way) | 11:31 |
e0ne | I like this idea and I'm goint to add note about it in the next version of the spec | 11:31 |
DuncanT | e0ne: We definitely want to make the CLI client calls just attach and detach, and hide any multi-step work, yeah | 11:32 |
DuncanT | I'd just call it 'attach' rather than 'generate attach' - under this model there are no other possible semantics for attach AFAICT | 11:33 |
*** markvoelker has quit IRC | 11:33 | |
*** bswartz has quit IRC | 11:39 | |
e0ne | DuncanT: Ironic guys asked us at summit about case when they don't want to install any agent/client inside an instance | 11:40 |
e0ne | DuncanT: they want to be able to ask users to run some commands to attach a volume | 11:41 |
*** 6JTAB24P4 has quit IRC | 11:41 | |
DuncanT | e0ne: The API call is the same whether you're inside an instance or no: "Here are the connection details of the thing I want to attach, give me back the details I need to attach" - or do you want the client to actually give a list of iSCSI attach commands and such? That is rather distro etc specific, isn't it? | 11:42 |
*** ronis has quit IRC | 11:42 | |
*** ronis_ has joined #openstack-cinder | 11:42 | |
e0ne | DuncanT: "do you want the client to actually give a list of iSCSI attach commands and such?". TBH, not me, buy Ironic team asked it. I'm also worried about distro-specific commands | 11:44 |
DuncanT | I'd skip that on V1, it's a minefield. It is client-side only, so can easily be added later | 11:44 |
DuncanT | Get the basics working in the first apss | 11:44 |
DuncanT | s/apss/pass/ | 11:45 |
e0ne | DuncanT: I'm agree with you | 11:45 |
e0ne | thanks for feedback | 11:46 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Attach/detach volumes without Nova (draft) https://review.openstack.org/224124 | 11:49 |
e0ne | DuncanT: I've proposed few choises for new client name in the spec | 11:50 |
DuncanT | e0ne: As long as it isn't 'cinder' I don't much care, probably easier to let John pick one | 11:51 |
*** lpetrut has quit IRC | 11:51 | |
e0ne | DuncanT: thanks :) | 11:51 |
DuncanT | cinder-local is probably to most accurate out of those, but as I said, I really don't care | 11:52 |
*** jordanP has joined #openstack-cinder | 11:52 | |
*** adrianofr has joined #openstack-cinder | 11:52 | |
e0ne | it's not very important, but I don't want to have remaming import and repository issues | 11:53 |
*** bluex-pl has quit IRC | 11:54 | |
*** martyturner has joined #openstack-cinder | 12:06 | |
*** gouthamr has joined #openstack-cinder | 12:07 | |
*** bswartz has joined #openstack-cinder | 12:09 | |
*** markvoelker has joined #openstack-cinder | 12:11 | |
*** Lee1092 has quit IRC | 12:13 | |
dulek | DuncanT, e0ne: Can I get your opinion on how to approach https://bugs.launchpad.net/cinder/+bug/1496747 ? | 12:14 |
openstack | Launchpad bug 1496747 in Cinder "Removing project access to volume type will not work after 128+ access rule" [Undecided,In progress] - Assigned to Michal Dulko (michal-dulko-f) | 12:14 |
DuncanT | dulek: Options are: a schema migration to make the deleted column big enough to hold an id, or change the delete to just set deleted=1 like everywhere else in that code, and fix it properly in M | 12:15 |
dulek | DuncanT, e0ne: I can either just change the only use of oslo.db's soft_delete in our code to plain update or migrate all tables to have deleted column of the same type as id and use soft_delete everywhere. | 12:15 |
dulek | DuncanT: Ah, we're thinking of fixing that in the RC1? | 12:16 |
DuncanT | dulek: I'd go with getting rid of the soft_delete for RC1, less code to test | 12:16 |
DuncanT | dulek: Do the migration in M, it'll be a useful simple case for the live migration code :-) | 12:17 |
dulek | DuncanT: Okay, I'll do just that. | 12:17 |
dulek | DuncanT: Live migration or online DB schema migration? ;) | 12:18 |
DuncanT | dulek: You're right, I should call it online DB schema migration, or schema independence | 12:18 |
dulek | Naming new things is always the hard. ;) | 12:19 |
*** yangyapeng has quit IRC | 12:23 | |
*** bluex-pl has joined #openstack-cinder | 12:28 | |
*** edmondsw has joined #openstack-cinder | 12:31 | |
*** diablo_rojo has joined #openstack-cinder | 12:33 | |
*** bluex-pl has quit IRC | 12:40 | |
*** e0ne has quit IRC | 12:44 | |
*** timcl has joined #openstack-cinder | 12:46 | |
*** diablo_rojo1 has joined #openstack-cinder | 12:46 | |
*** dimsum__ has quit IRC | 12:47 | |
*** diablo_rojo2 has joined #openstack-cinder | 12:47 | |
*** diablo_rojo1 has quit IRC | 12:47 | |
*** dimsum__ has joined #openstack-cinder | 12:47 | |
*** xyang1 has joined #openstack-cinder | 12:48 | |
*** diablo_rojo2 has left #openstack-cinder | 12:48 | |
*** dimsum__ is now known as dims | 12:48 | |
*** diablo_rojo has quit IRC | 12:48 | |
*** diablo_rojo has joined #openstack-cinder | 12:49 | |
*** dims is now known as Guest31942 | 12:49 | |
*** Guest31942 is now known as dims__ | 12:49 | |
*** merooney has joined #openstack-cinder | 12:50 | |
*** merooeny_ has joined #openstack-cinder | 12:52 | |
*** deepakcs has quit IRC | 12:53 | |
*** akerr has joined #openstack-cinder | 12:53 | |
*** merooney has quit IRC | 12:55 | |
*** gouthamr has quit IRC | 13:01 | |
*** merooeny_ has quit IRC | 13:03 | |
*** jaypipes has joined #openstack-cinder | 13:05 | |
*** dustins has joined #openstack-cinder | 13:05 | |
*** julim has joined #openstack-cinder | 13:09 | |
ociuhandu | thingee: Hello, did you have time by any chance to evaluate the results of the Microsoft iSCSI CI and if there’s anything else we need to do in order to re-enable the CI? | 13:11 |
*** EinstCrazy has quit IRC | 13:11 | |
*** Yogi1 has joined #openstack-cinder | 13:13 | |
*** breitz has joined #openstack-cinder | 13:16 | |
*** gouthamr has joined #openstack-cinder | 13:17 | |
*** mfisch has quit IRC | 13:18 | |
*** charlesr has joined #openstack-cinder | 13:18 | |
*** charlesr has quit IRC | 13:19 | |
*** mfisch has joined #openstack-cinder | 13:19 | |
*** mfisch is now known as Guest61243 | 13:19 | |
*** raildo-afk is now known as raildo | 13:19 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Use function capsulation in Huawei driver https://review.openstack.org/222916 | 13:19 |
*** amoturi has joined #openstack-cinder | 13:21 | |
*** jaypipes has quit IRC | 13:22 | |
*** haomaiwang has joined #openstack-cinder | 13:23 | |
*** jgregor has joined #openstack-cinder | 13:24 | |
*** charlesr has joined #openstack-cinder | 13:26 | |
*** charlesr has quit IRC | 13:26 | |
*** crose has quit IRC | 13:27 | |
*** crose has joined #openstack-cinder | 13:27 | |
*** Yogi1 has quit IRC | 13:28 | |
*** ankit_ag has quit IRC | 13:33 | |
*** superdan is now known as dansmith | 13:34 | |
*** eharney has joined #openstack-cinder | 13:35 | |
*** sayali has quit IRC | 13:35 | |
*** zhangjn has quit IRC | 13:37 | |
*** mriedem has joined #openstack-cinder | 13:39 | |
*** lprice1 has joined #openstack-cinder | 13:41 | |
*** sayali has joined #openstack-cinder | 13:41 | |
*** jaypipes has joined #openstack-cinder | 13:42 | |
*** lprice has quit IRC | 13:43 | |
*** sayali has quit IRC | 13:43 | |
*** salv-orl_ has joined #openstack-cinder | 13:46 | |
*** r-daneel has joined #openstack-cinder | 13:46 | |
*** e0ne has joined #openstack-cinder | 13:47 | |
*** erlon has joined #openstack-cinder | 13:47 | |
*** salv-orlando has quit IRC | 13:47 | |
*** jordanP has quit IRC | 13:50 | |
*** Lee1092 has joined #openstack-cinder | 13:51 | |
gouthamr | DuncanT, e0ne : Thank you for reviewing https://review.openstack.org/#/c/222413/ , I updated the code addressing your comments. Would appreciate any more comments you may have. | 13:54 |
*** sayali has joined #openstack-cinder | 13:54 | |
DuncanT | gouthamr: I'll take another look. I notice many of the third party CIs are failing on that patch, can you check a few and see if they are related, please? | 13:56 |
gouthamr | sure thing DuncanT | 13:56 |
gouthamr | thank you. | 13:56 |
*** mdenny has joined #openstack-cinder | 13:56 | |
*** mc_nair has joined #openstack-cinder | 13:56 | |
*** jgregor has quit IRC | 13:56 | |
*** sseverson has quit IRC | 13:57 | |
*** edtubill has joined #openstack-cinder | 13:58 | |
*** timcl1 has joined #openstack-cinder | 13:59 | |
*** crose has quit IRC | 13:59 | |
*** jgregor has joined #openstack-cinder | 13:59 | |
*** salv-orl_ has quit IRC | 14:00 | |
*** martyturner has quit IRC | 14:00 | |
*** haomaiwang has quit IRC | 14:01 | |
DuncanT | gouthamr: I'm still confused by the options.py change. If the user has a list of volumes in netapp_volume_list, then you can't just treat that as a regular expression. Ditto a comma separated list in netapp_storage_pools in not a regex. How do you even tell the difference between a legacy list of volumes and a legacy list of pools? | 14:01 |
*** david-ly_ is now known as david-lyle | 14:01 | |
*** timcl has quit IRC | 14:01 | |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** martyturner has joined #openstack-cinder | 14:02 | |
*** wilson1 has joined #openstack-cinder | 14:02 | |
gouthamr | DuncanT: previously users who used to use netapp_volume_list for 7-mode and netapp_storage_pools for e-series could specify names of the pools as CSVs. It is true that we weren't doing regex matches on these names.. we were equating strings in the code and reporting only specified pools to the scheduler. | 14:05 |
DuncanT | gouthamr: My point is about back-compatability. If a user has a current config file that has one of those two lists specified, then as far as I can tell, the config file will not work - you'll not interpret it as you did before | 14:07 |
*** zhangjn has joined #openstack-cinder | 14:07 | |
gouthamr | DuncanT: With the new option, users can still use CSVs.. I'm not sure why we want to tell the difference between the list of pool names and regexes :) | 14:07 |
gouthamr | DuncanT: ah. we tested this against the legacy options.. the new option works with CSVs as well. We take each value, prefix-suffix with ^$ and match, so one-to-one matches still work | 14:09 |
*** e0ne has quit IRC | 14:10 | |
*** jordanP has joined #openstack-cinder | 14:10 | |
tbarron | gouthamr: DuncanT: yeah, you convert the csv to an equivalent regex under the covers, maintaining backwards compatibility. | 14:10 |
*** thangp has joined #openstack-cinder | 14:10 | |
DuncanT | gouthamr: Ah, ok, and it does something clever with pool .v. volume names in a list? I guess most people aren't perverse enough to create pools and volumes with the same names? | 14:10 |
*** rhagarty__ has joined #openstack-cinder | 14:10 | |
*** zhangjn has quit IRC | 14:12 | |
*** lpetrut has joined #openstack-cinder | 14:12 | |
gouthamr | DuncanT: we are filtering on the same entities as before -> FlexVols on teh backend.. netapp_volume_list was probably a poor name chosen in the implementation prior. | 14:13 |
*** krtaylor has quit IRC | 14:13 | |
*** 17SADM1EH has joined #openstack-cinder | 14:13 | |
*** wilson1 has quit IRC | 14:13 | |
DuncanT | gouthamr: Ok, thanks for clarifying | 14:13 |
gouthamr | DuncanT: you're welcome... | 14:13 |
*** sayali has quit IRC | 14:13 | |
gouthamr | DuncanT: Thanks for reviewing :) | 14:13 |
*** rhagarty_ has quit IRC | 14:13 | |
*** rhagarty has quit IRC | 14:13 | |
*** tpsilva has joined #openstack-cinder | 14:14 | |
*** wilson1 has joined #openstack-cinder | 14:14 | |
guitarzan | DuncanT: did I see you say something about quota-show being broken? | 14:14 |
*** e0ne has joined #openstack-cinder | 14:14 | |
guitarzan | (because quota-show seems to be broken) | 14:14 |
openstackgerrit | Tom Barron proposed openstack/cinder: NetApp volume/snapshot delete performance fix https://review.openstack.org/225306 | 14:14 |
*** rhagarty has joined #openstack-cinder | 14:14 | |
*** rhagarty_ has joined #openstack-cinder | 14:14 | |
guitarzan | with this nested quota stuff calling identity | 14:15 |
openstackgerrit | Michal Dulko proposed openstack/cinder: Replace soft_delete in volume_type_access_remove https://review.openstack.org/225701 | 14:17 |
*** sayali has joined #openstack-cinder | 14:18 | |
*** rhagarty__ has quit IRC | 14:18 | |
*** jordanP has quit IRC | 14:18 | |
*** jordanP has joined #openstack-cinder | 14:19 | |
*** zhangjn has joined #openstack-cinder | 14:19 | |
DuncanT | guitarzan: yes, the default rules for keystone break it | 14:21 |
guitarzan | ok, glad it's not just me | 14:22 |
*** krtaylor has joined #openstack-cinder | 14:22 | |
guitarzan | I'm a little sad that we crash even if you don't want to use nested quotas | 14:22 |
*** david-lyle has quit IRC | 14:22 | |
guitarzan | and pretty surprised that we use the key manager config to determine the identity endpoint | 14:23 |
*** wilson-1 has joined #openstack-cinder | 14:24 | |
*** yrabl has quit IRC | 14:24 | |
openstackgerrit | Monty Taylor proposed openstack/cinder: Change ignore-errors to ignore_errors https://review.openstack.org/225725 | 14:25 |
*** martyturner has quit IRC | 14:25 | |
*** vlaza has quit IRC | 14:26 | |
*** david-lyle has joined #openstack-cinder | 14:26 | |
erlon | DuncanT: tbarron: hi guys, do we have any etherpad or wiki with rc dates for Cinder? | 14:27 |
*** wilson1 has quit IRC | 14:27 | |
*** Yogi11 has joined #openstack-cinder | 14:28 | |
DuncanT | erlon: The RC will be cut when the blocker bugs are gone | 14:28 |
DuncanT | erlon: https://launchpad.net/cinder/+milestone/liberty-rc1 | 14:28 |
*** asselin_ has joined #openstack-cinder | 14:29 | |
erlon | DuncanT: hmm, so there's no fixed date | 14:29 |
*** 17SADM1EH has quit IRC | 14:29 | |
DuncanT | guitarzan: Yeah, that's a bug, Ollie is about to put a patch up to default those to None, and pull it out of the catalog by default instead, probably worth stealing part of that to fix. I'm tempted to revert the whole damn thing though, to be honest. I've been trying to do a partial revert but can't get anything to work. | 14:29 |
guitarzan | DuncanT: are we supposed to be able to run without keystone v3? | 14:30 |
guitarzan | (please say yes) :) | 14:30 |
*** cbader has joined #openstack-cinder | 14:30 | |
guitarzan | I'd prefer a much more graceful failure for nested quotas | 14:30 |
erlon | DuncanT: but all bugs in this list are listed with 'Fix commited', so there's no blocker for RC1 | 14:30 |
*** tsekiyama has joined #openstack-cinder | 14:31 | |
tbarron | erlon: I've been assuming RC1 is imminent for that reason ... | 14:31 |
erlon | tbarron: hmm | 14:31 |
smcginnis | erlon: Yeah, I think it's just up to thingee to decide when at this point. | 14:31 |
tbarron | smcginnis: DuncanT: so there are no other critical bug reviews that we need to attend to ATM, right? | 14:32 |
smcginnis | tbarron: I'm sure there probably are, just not that have been flagged. ;) | 14:33 |
tbarron | smcginnis: :-) I mean, that would hold up RC1 ? | 14:33 |
DuncanT | tbarron: erlon: Not all of the committed fixes actually fix the bugs :-( e.g. the quota fix doesn't actually fix anything | 14:33 |
smcginnis | This would probably be a good point for anyone to speak up if they have something that should hold on. | 14:33 |
smcginnis | The nested quota one is probably a good one to wait for. That's kind of a Bad Thing. | 14:34 |
DuncanT | https://bugs.launchpad.net/cinder/+bug/1491495 is not even nested quota, it's any quota | 14:34 |
openstack | Launchpad bug 1491495 in Cinder "cinderclient.quotas.get() fails against Cinder service in a Devstack" [Medium,Fix committed] - Assigned to Ivan Kolodyazhny (e0ne) | 14:34 |
smcginnis | DuncanT: In that case it's a Real Bad Thing. :) | 14:35 |
e0ne | DuncanT: AFAI, you were going to revert that patch with qoutas | 14:35 |
guitarzan | right, the nested quotas just introduced the bug | 14:35 |
erlon | DuncanT: the patch was committed and does not fix or it should be a partial-bug? | 14:35 |
guitarzan | breaking all quota shows | 14:35 |
guitarzan | erlon: the patch doesn't address the issue at all | 14:35 |
erlon | DuncanT: hmm, ok | 14:35 |
DuncanT | If somebody wants to propose a revert for that whole mess, that would be useful, I'm still trying to actually fix the bug, but failing (feel free to try that too, I'm not possessive of anything with less than a 4% ABV) | 14:36 |
*** jungleboyj has joined #openstack-cinder | 14:36 | |
guitarzan | hah! | 14:36 |
DuncanT | erlon: The patch committed is a partial fix at best | 14:36 |
*** asselin__ has joined #openstack-cinder | 14:38 | |
*** timcl has joined #openstack-cinder | 14:40 | |
*** timcl1 has quit IRC | 14:41 | |
openstackgerrit | Monty Taylor proposed openstack/os-brick: Change ignore-errors to ignore_errors https://review.openstack.org/225781 | 14:41 |
*** pck is now known as pckizer | 14:43 | |
openstackgerrit | Monty Taylor proposed openstack/python-cinderclient: Change ignore-errors to ignore_errors https://review.openstack.org/225808 | 14:44 |
*** eharney has quit IRC | 14:44 | |
*** jgregor has quit IRC | 14:44 | |
*** porrua has joined #openstack-cinder | 14:45 | |
*** zhangjn has quit IRC | 14:45 | |
*** mss has joined #openstack-cinder | 14:50 | |
*** zhangjn has joined #openstack-cinder | 14:51 | |
*** xek has quit IRC | 14:53 | |
e0ne | DuncanT, erlon: I've just realized, that project_get in Keystone API v2 is admin-only | 14:54 |
e0ne | DuncanT, erlon: but it works for non-admin user for API v3 | 14:54 |
DuncanT | e0ne: Thats... unhelpful | 14:55 |
DuncanT | e0ne: We can't rely on V3 yet, unfortunately | 14:55 |
e0ne | DuncanT: why? AFAIK, it's enabled by default on Liberty | 14:55 |
DuncanT | e0ne: We could just skip validation if we get back a policy error, which would give you the old behaviour | 14:56 |
DuncanT | e0ne: It won't be there for upgrade deploys though | 14:56 |
e0ne | DuncanT: understood. you're right | 14:56 |
*** EinstCrazy has joined #openstack-cinder | 14:57 | |
DuncanT | e0ne: I'd be happy with a skip on policy fail for now, and we can look at something better in M. It is better than a revert, though looking in the barbican configuration for the keystone API is still broken | 14:57 |
*** mss has quit IRC | 14:58 | |
*** daneyon has joined #openstack-cinder | 14:58 | |
*** EinstCrazy has quit IRC | 14:59 | |
*** jgregor has joined #openstack-cinder | 14:59 | |
*** albertom has quit IRC | 14:59 | |
*** haomaiwang has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
*** devlaps has joined #openstack-cinder | 15:02 | |
*** zhangjn has quit IRC | 15:05 | |
*** zhangjn has joined #openstack-cinder | 15:06 | |
*** albertom has joined #openstack-cinder | 15:06 | |
*** jungleboyj has quit IRC | 15:06 | |
*** zhangjn has quit IRC | 15:06 | |
*** zhangjn has joined #openstack-cinder | 15:07 | |
*** zhangjn has quit IRC | 15:08 | |
*** annegentle has joined #openstack-cinder | 15:08 | |
*** zhangjn has joined #openstack-cinder | 15:09 | |
*** zhangjn has quit IRC | 15:09 | |
e0ne | DuncanT: "skip on policy fail" sounds like an easy hack. I'll try it right now | 15:10 |
*** asselin__ has quit IRC | 15:10 | |
*** zhangjn has joined #openstack-cinder | 15:10 | |
*** lpetrut has quit IRC | 15:11 | |
*** lpetrut has joined #openstack-cinder | 15:11 | |
*** zhangjn has quit IRC | 15:11 | |
*** eharney has joined #openstack-cinder | 15:12 | |
*** zhangjn has joined #openstack-cinder | 15:12 | |
*** ronis_ has quit IRC | 15:14 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: WIP: Ignore Forbidden error on quotas-get for nested projects https://review.openstack.org/225891 | 15:16 |
e0ne | DuncanT: do you mean something like ^^ | 15:16 |
*** hemnafk is now known as hemna | 15:16 | |
e0ne | I've tested this on devstack | 15:16 |
*** harlowja_at_home has joined #openstack-cinder | 15:17 | |
DuncanT | e0ne Yes | 15:17 |
guitarzan | can we make it more generic? | 15:17 |
guitarzan | any fail? | 15:18 |
guitarzan | I don't want to even have to set the keymanager config option if I don't have to | 15:18 |
e0ne | DuncanT: ok. I'll fix other cases | 15:18 |
*** jungleboyj has joined #openstack-cinder | 15:18 | |
DuncanT | guitarzan: I want to get rid of that too, just working on a patch to grab it out of the catalog | 15:18 |
guitarzan | DuncanT: well, for example I don't even have a v3 endpoint | 15:18 |
*** martyturner has joined #openstack-cinder | 15:19 | |
guitarzan | granted, that might be an unreasonable ask, I'm not sure | 15:19 |
DuncanT | guitarzan: If you have a V2 endpoint in your catalog, then the call will fail at policy by default and skip | 15:19 |
*** devlaps has quit IRC | 15:19 | |
DuncanT | guitarzan: Or am I missing something | 15:20 |
DuncanT | guitarzan: We could skip on all failures I guess | 15:20 |
guitarzan | hmm, does noauth work with no keystone? | 15:21 |
hemna | morning | 15:21 |
*** rushil has joined #openstack-cinder | 15:22 | |
*** stevemar has joined #openstack-cinder | 15:22 | |
e0ne | hemna: evening:) | 15:23 |
*** dustins is now known as dustins|away | 15:23 | |
jungleboyj | hemna: Morning sir. | 15:24 |
hemna | e0ne, :) | 15:27 |
*** devlaps has joined #openstack-cinder | 15:28 | |
*** jdurgin1 has joined #openstack-cinder | 15:29 | |
*** daneyon has left #openstack-cinder | 15:30 | |
*** dsariel_ has quit IRC | 15:31 | |
*** jdurgin1 has quit IRC | 15:35 | |
*** mtanino has joined #openstack-cinder | 15:35 | |
*** EinstCrazy has joined #openstack-cinder | 15:40 | |
*** nkrinner has quit IRC | 15:42 | |
DuncanT | guitarzan: noauth doesn't work at all at the moment, unless you delete a bunch of code | 15:45 |
guitarzan | DuncanT: fair enough, I just have a fake auth | 15:46 |
DuncanT | guitarzan: I need to investigate the 'correct' fix for noauth / fakeauth some time, but deleting every line of code that got in the way made the PoC run :-) | 15:46 |
*** garthb has joined #openstack-cinder | 15:47 | |
guitarzan | :) | 15:47 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Ignore Forbidden error on quotas-get for nested projects https://review.openstack.org/225891 | 15:48 |
*** zhangjn has quit IRC | 15:50 | |
e0ne | DuncanT: I fixed quotas-get and qoutas-defautls. W/o unit-tests yet:( | 15:50 |
DuncanT | e0ne: Great, I'll take a look in a sec, I'm moments away from a draft of the catalog code | 15:50 |
*** stevemar has quit IRC | 15:51 | |
e0ne | DuncanT: thanks. I'll try to write few tests for it | 15:51 |
*** mc_nair has quit IRC | 15:52 | |
*** sayali has quit IRC | 15:52 | |
DuncanT | https://www.irccloud.com/pastebin/32fT5i5c/ | 15:52 |
DuncanT | I'm clearly doing something stupid, but I don't know what | 15:53 |
*** zhangjn has joined #openstack-cinder | 15:53 | |
DuncanT | https://www.irccloud.com/pastebin/EMh4NjC5/patch | 15:53 |
DuncanT | The second paste is the (untested) patch that I think should pull the info out of the catalog. Exception class needs creating tpp | 15:54 |
DuncanT | s/tpp/too/ | 15:54 |
*** haomaiwang has quit IRC | 15:54 | |
*** haomaiwang has joined #openstack-cinder | 15:55 | |
*** rhagarty__ has joined #openstack-cinder | 15:55 | |
DuncanT | We have this catalog grabbing code in four places now, really should make a helper util for it | 15:55 |
*** rhagarty__ has quit IRC | 15:56 | |
*** rhagarty_ has quit IRC | 15:56 | |
*** stevemar has joined #openstack-cinder | 15:57 | |
*** sseverson has joined #openstack-cinder | 15:58 | |
*** rhagarty_ has joined #openstack-cinder | 16:00 | |
*** haomaiwang has quit IRC | 16:01 | |
*** rhagarty__ has joined #openstack-cinder | 16:01 | |
*** haomaiwang has joined #openstack-cinder | 16:01 | |
*** Yogi11 has quit IRC | 16:01 | |
*** stevemar has quit IRC | 16:02 | |
DuncanT | Sorry, I've got to run for an hour, if somebody wants to try to beat that code into some form of correctness, feel free, if not I'll work on it later this evening | 16:02 |
*** jungleboyj has quit IRC | 16:02 | |
*** mudassirlatif has joined #openstack-cinder | 16:02 | |
*** diogogmt has joined #openstack-cinder | 16:02 | |
*** salv-orlando has joined #openstack-cinder | 16:03 | |
*** jungleboyj has joined #openstack-cinder | 16:03 | |
*** alejandrito has joined #openstack-cinder | 16:03 | |
*** mss has joined #openstack-cinder | 16:05 | |
*** breitz has quit IRC | 16:06 | |
*** hodos has joined #openstack-cinder | 16:06 | |
hodos | thingee: hi, do we have to resubmit new blueprints for M release for re-adding of Nexenta Drivers, cause we missed L due to CI instability | 16:07 |
*** leeantho has joined #openstack-cinder | 16:08 | |
*** breitz has joined #openstack-cinder | 16:09 | |
*** anshul has quit IRC | 16:09 | |
*** rhagarty has quit IRC | 16:10 | |
openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: Fix use of wrong storage pools for NetApp Drivers https://review.openstack.org/222413 | 16:12 |
*** zhangjn has quit IRC | 16:14 | |
*** EinstCrazy has quit IRC | 16:15 | |
*** shausy has quit IRC | 16:18 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Ignore Forbidden error on quotas-get for nested projects https://review.openstack.org/225891 | 16:21 |
*** rushil has quit IRC | 16:26 | |
*** Apoorva has joined #openstack-cinder | 16:27 | |
*** IlyaG has joined #openstack-cinder | 16:28 | |
*** vgridnev has quit IRC | 16:36 | |
*** stevemar has joined #openstack-cinder | 16:37 | |
*** annegentle has quit IRC | 16:38 | |
*** harlowja_at_home has quit IRC | 16:38 | |
*** annegentle has joined #openstack-cinder | 16:39 | |
*** e0ne has quit IRC | 16:41 | |
*** szhang has joined #openstack-cinder | 16:41 | |
*** rushil has joined #openstack-cinder | 16:42 | |
*** dustins|away is now known as dustins | 16:45 | |
*** ronis_ has joined #openstack-cinder | 16:48 | |
*** chlong has quit IRC | 16:50 | |
*** Yogi1 has joined #openstack-cinder | 16:50 | |
*** angela-s has joined #openstack-cinder | 16:52 | |
thingee | smcginnis, erlon waiting for the release team at this point. My request is in for a cut | 16:54 |
thingee | hodos: the next PTL will retarget the blueprint for M release | 16:55 |
*** sgotliv has quit IRC | 16:56 | |
*** _cjones_ has joined #openstack-cinder | 16:56 | |
*** haomaiwang has quit IRC | 16:58 | |
*** haomaiwang has joined #openstack-cinder | 16:59 | |
*** haomaiwang has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
*** haomaiwang has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:02 | |
*** chlong has joined #openstack-cinder | 17:02 | |
*** haomaiwang has quit IRC | 17:03 | |
*** haomaiwang has joined #openstack-cinder | 17:03 | |
*** haomaiwang has quit IRC | 17:11 | |
*** asselin has quit IRC | 17:12 | |
*** devlaps has quit IRC | 17:12 | |
*** jordanP has quit IRC | 17:15 | |
*** asselin has joined #openstack-cinder | 17:17 | |
openstackgerrit | Sean McCully proposed openstack/cinder: Use of ast for integers doesn't changes type https://review.openstack.org/225947 | 17:19 |
*** vgridnev has joined #openstack-cinder | 17:21 | |
*** e0ne has joined #openstack-cinder | 17:24 | |
*** aix has quit IRC | 17:29 | |
*** akerr has quit IRC | 17:34 | |
ociuhandu | thingee: Hello, did you have time by any chance to evaluate the results of the Microsoft iSCSI CI and if there’s anything else we need to do in order to re-enable the CI? | 17:35 |
*** akerr has joined #openstack-cinder | 17:35 | |
*** Zhongjun has quit IRC | 17:37 | |
*** subscope has joined #openstack-cinder | 17:37 | |
jungleboyj | Has anyone been seeing this unit test fail: cinder.tests.unit.test_cmd.TestCinderAllCmd.test_main_load_binary_exception | 17:40 |
dhellmann | thingee: can you confirm that os-brick 0.4.0 is what you want for your stable/liberty branch? | 17:42 |
thingee | dhellmann: ack, give me 10 minutes before I can look | 17:43 |
dhellmann | thingee: np | 17:43 |
jungleboyj | A fresh pull of master on ubuntu 14.04 fails that test case. | 17:43 |
*** lpetrut has quit IRC | 17:44 | |
hodos | thingee: thanks | 17:45 |
*** harlowja has joined #openstack-cinder | 17:46 | |
smcginnis | jungleboyj: I haven't seen that. (yet) | 17:50 |
*** amoturi has left #openstack-cinder | 17:50 | |
smcginnis | dhellmann, thingee: I believe there were changes in os-brick 0.4.0 that we need. That is the latest release. | 17:50 |
jungleboyj | I am thinking I have having issues because I ran devstack on here. Playing around with it. | 17:51 |
jungleboyj | smcginnis: Thanks. | 17:51 |
smcginnis | jungleboyj: Running on master locally (ubuntu 14.04.3). I'll let you know how it goes. | 17:52 |
*** timcl has quit IRC | 17:52 | |
hemna | dhellmann, thingee I think there are a few items that have gone into brick since 0.4.0 | 17:54 |
hemna | we might want to push a 0.4.1 | 17:54 |
hemna | eharney, or smcginnis can either of you +A this one? I don't want to ninja it, even though it has 2 +2's on it already. https://review.openstack.org/#/c/219779/ | 17:55 |
eharney | hemna: sure | 17:56 |
hemna | https://review.openstack.org/#/c/222998/ | 17:56 |
hemna | is another one | 17:56 |
thingee | hemna: ack | 17:56 |
*** mc_nair has joined #openstack-cinder | 17:56 | |
hemna | thingee, they are small fixes | 17:56 |
*** Apoorva_ has joined #openstack-cinder | 17:57 | |
smcginnis | hemna: On their way through. | 17:58 |
hemna | ok thanks | 17:58 |
e0ne | it was too quick for me | 17:58 |
thingee | hemna: I would like to get keyerror fix in | 17:58 |
smcginnis | e0ne: :) | 17:58 |
hemna | most of what has gone in since 0.4.0 is small log fixes | 17:58 |
thingee | device not found giving key errors might be common | 17:59 |
thingee | hemna: ^ | 17:59 |
hemna | thingee, https://review.openstack.org/#/c/220902/ that one ? | 17:59 |
thingee | hemna: yes | 17:59 |
hemna | yah that's been in for a bit | 17:59 |
hemna | but yah, I agree. | 18:00 |
*** Apoorva has quit IRC | 18:00 | |
hemna | coolio. thanks guys | 18:01 |
*** shyama has joined #openstack-cinder | 18:01 | |
thingee | dhellmann: I have a fix I'd like to see in the release. Could do a cut right now. we do have some stuff in flight gate, eta 1hr | 18:01 |
dhellmann | thingee: ok, the goal is to have the branches by eod so we can wait for a bit for that patch | 18:02 |
dhellmann | thingee: is that https://review.openstack.org/#/c/220902/ ? | 18:02 |
thingee | yes | 18:02 |
dhellmann | ok, noted | 18:02 |
openstackgerrit | Sean McCully proposed openstack/cinder: Use of ast for integers doesn't changes type https://review.openstack.org/225947 | 18:02 |
dhellmann | thingee: while we're at it, how about cinder client? | 18:02 |
thingee | dhellmann: switching to relmgr-office | 18:03 |
smcginnis | jungleboyj: Full UT run on 14.04. No test failures on master for me. | 18:04 |
hemna | https://bugs.launchpad.net/cinder/+bug/1491495 is still marked as in progress, but the patch for it has landed | 18:06 |
openstack | Launchpad bug 1491495 in Cinder "cinderclient.quotas.get() fails against Cinder service in a Devstack" [Medium,In progress] - Assigned to Ivan Kolodyazhny (e0ne) | 18:06 |
jungleboyj | Interesting. Ok, so something definitely wrong with my env. | 18:06 |
smcginnis | hemna: I think there's an update in flight. | 18:07 |
e0ne | hemna: we've found a root cause with DuncanT today | 18:07 |
hemna | ok | 18:07 |
e0ne | hemna: here is a new patch | 18:07 |
e0ne | https://review.openstack.org/225891 | 18:07 |
hemna | ah yah I see it now further down in the bug | 18:07 |
hemna | huh. why didn't we see this earlier ? | 18:08 |
hemna | are the CI tests not covering this? | 18:08 |
thingee | hemna: do we have quota tempest tests? | 18:09 |
jungleboyj | smcginnis: Yep, interesting. If there is a config file in /etc/cinder on the system where you are running unit test that fails. I will open a bug for that in case anyone else finds it. Something we should eventually fix. | 18:09 |
hemna | thingee, doesn't look like it | 18:10 |
hemna | volume_quotas_client | 18:10 |
thingee | hemna: I've recommended a tag for cinder client once that lands. | 18:12 |
thingee | smcginnis: I'm assuming you actually tried this patch? | 18:12 |
*** tsekiyam_ has joined #openstack-cinder | 18:15 | |
e0ne | jungleboyj: whicj tests does fail with cinder.conf? | 18:17 |
e0ne | s/whicj/which | 18:17 |
jungleboyj | e0ne: cinder.tests.unit.test_cmd.TestCinderAllCmd.test_main_load_binary_exception | 18:17 |
*** asselin_ has quit IRC | 18:18 | |
*** tsekiyama has quit IRC | 18:18 | |
*** tsekiyam_ has quit IRC | 18:19 | |
e0ne | jungleboyj: yes, the same on my env:( | 18:20 |
e0ne | jungleboyj: seems to incorrect CONF mock | 18:20 |
e0ne | jungleboyj: even more. it doesn't mock CONF object. it uses real config | 18:21 |
jungleboyj | e0ne: Yuck. | 18:22 |
jungleboyj | I will open a bug for that. | 18:22 |
e0ne | jungleboyj: could you please send me a link for filed bug? | 18:22 |
jungleboyj | Sure. | 18:23 |
*** timcl has joined #openstack-cinder | 18:27 | |
openstackgerrit | Kendall Nelson proposed openstack/cinder: Missing configuration opts from cinder.sample.conf https://review.openstack.org/225974 | 18:27 |
jungleboyj | angela-s: Can you take a look at this. I will merge if you are ok: https://review.openstack.org/#/c/218341/ | 18:29 |
*** Apoorva_ has quit IRC | 18:30 | |
*** Apoorva has joined #openstack-cinder | 18:31 | |
*** vgridnev has quit IRC | 18:32 | |
angela-s | jungleboyj: +1 | 18:34 |
hemna | jungleboyj, angela-s brocade CI looks happy with it | 18:35 |
jungleboyj | hemna: Yeah, I know. Just covering my bases. | 18:35 |
jungleboyj | angela-s: Thank you! | 18:35 |
*** martyturner has quit IRC | 18:35 | |
*** vgridnev has joined #openstack-cinder | 18:35 | |
angela-s | hemna: yes, CI tests were successful for zoning | 18:36 |
*** vgridnev has quit IRC | 18:38 | |
*** vgridnev has joined #openstack-cinder | 18:38 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Remove unused gettextutils from oslo-incubator https://review.openstack.org/225978 | 18:40 |
openstackgerrit | Kendall Nelson proposed openstack/cinder: Missing configuration opts from cinder.sample.conf https://review.openstack.org/225974 | 18:41 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Small optimization in Block Device driver https://review.openstack.org/220561 | 18:45 |
*** vgridnev has quit IRC | 18:45 | |
openstackgerrit | Eric Harney proposed openstack/cinder: Allow c-vol backends to start when some backends fail to load https://review.openstack.org/225982 | 18:46 |
*** cdelatte has joined #openstack-cinder | 18:50 | |
*** shyama has quit IRC | 18:53 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make rpc_client private method for VolumeCommands https://review.openstack.org/225985 | 18:53 |
*** vgridnev has joined #openstack-cinder | 18:53 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make rpc_client method private for VolumeCommands https://review.openstack.org/225985 | 18:54 |
*** rushil has quit IRC | 18:55 | |
*** vgridnev has quit IRC | 18:55 | |
*** rushil has joined #openstack-cinder | 18:57 | |
*** vgridnev has joined #openstack-cinder | 19:02 | |
*** annegentle has quit IRC | 19:03 | |
*** nikeshm has joined #openstack-cinder | 19:10 | |
*** ociuhandu has quit IRC | 19:10 | |
jungleboyj | e0ne: https://bugs.launchpad.net/cinder/+bug/1498148 You planning to look at that or should I have someone here take a look> | 19:13 |
openstack | Launchpad bug 1498148 in Cinder "test_main_load_binary_exception fails if there is an existing /etc/cinder/cinder.conf file" [Medium,New] | 19:13 |
e0ne | jungleboyj: thanks. if somebody will work on it - I'll be happy to verify and review | 19:14 |
jungleboyj | e0ne: Sounds good. | 19:14 |
e0ne | jungleboyj: if not, I'll try to fix it later this week | 19:14 |
e0ne | jungleboyj: TBH, there are other places where we don't mock config:( | 19:16 |
scottda | bswartz: Am I correct that using "latest" for api-microversion is frowned upon because it can inadvertently introduce backward incompatible changes , but is necessary for Tempest? | 19:18 |
bswartz | scottda: I started a thread about that topic | 19:20 |
scottda | thx. I'll look for it... | 19:20 |
bswartz | scottda: it's debatable whether tempest actually needs it, but that's the one use case where "latest" seems to have some value | 19:20 |
bswartz | scottda: IMO the only reason nova needs latest is because their functional tests are in a different git repo than their code | 19:21 |
scottda | hmmm...ok. So cinder may not need it. How about Manila? Do you need it ? | 19:21 |
*** mc_nair has quit IRC | 19:25 | |
bswartz | scottda: our tempest tests are in the same tree -- I think cinder has a more similar situation to nova where the tempest tests are out of tree | 19:27 |
bswartz | the challenge with microversions is that you want to test both the oldest and the newest microversions that the server supports, as well as any other intermediate versions if they have changed 2 or more times between oldest and newest | 19:28 |
*** ociuhandu has joined #openstack-cinder | 19:29 | |
scottda | OK. Thanks for the info | 19:29 |
*** breitz has quit IRC | 19:35 | |
*** Adriano_ has joined #openstack-cinder | 19:36 | |
*** breitz has joined #openstack-cinder | 19:37 | |
*** sgotliv has joined #openstack-cinder | 19:41 | |
*** DericHorn-HP has joined #openstack-cinder | 19:48 | |
gouthamr | DuncanT, smcginnis, e0ne : Made a minor modification to https://review.openstack.org/#/c/222413/ as smcginnis suggested. Could you please approve. | 19:49 |
*** amoturi has joined #openstack-cinder | 19:53 | |
*** e0ne has quit IRC | 19:59 | |
*** e0ne has joined #openstack-cinder | 20:00 | |
*** rushil has quit IRC | 20:09 | |
smcginnis | thingee: Sorry, busy afternoon. Which patch? | 20:10 |
*** annegentle has joined #openstack-cinder | 20:10 | |
*** amoturi has left #openstack-cinder | 20:12 | |
*** lpetrut has joined #openstack-cinder | 20:13 | |
*** thangp has quit IRC | 20:21 | |
*** porrua has quit IRC | 20:24 | |
*** jgregor has quit IRC | 20:25 | |
*** timcl has quit IRC | 20:25 | |
*** bswartz has quit IRC | 20:27 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Make rpc_client method private for VolumeCommands https://review.openstack.org/225985 | 20:27 |
*** e0ne has quit IRC | 20:29 | |
*** Apoorva_ has joined #openstack-cinder | 20:30 | |
*** mudassirlatif_ has joined #openstack-cinder | 20:33 | |
*** Apoorva has quit IRC | 20:34 | |
*** mudassirlatif has quit IRC | 20:35 | |
*** mudassirlatif_ is now known as mudassirlatif | 20:35 | |
*** tpsilva has quit IRC | 20:35 | |
*** merooney has joined #openstack-cinder | 20:38 | |
*** merooeny_ has joined #openstack-cinder | 20:40 | |
*** markvoelker has quit IRC | 20:41 | |
*** akerr has quit IRC | 20:41 | |
*** mudassirlatif has quit IRC | 20:42 | |
*** merooney has quit IRC | 20:43 | |
*** mudassirlatif has joined #openstack-cinder | 20:43 | |
*** ronis_ has quit IRC | 20:44 | |
* EmilienM | 20:49 | |
*** rhagarty__ has quit IRC | 20:53 | |
*** rhagarty_ has quit IRC | 20:53 | |
*** merooeny_ has quit IRC | 20:57 | |
*** aix has joined #openstack-cinder | 20:57 | |
*** dustins has quit IRC | 21:01 | |
*** mudassirlatif has quit IRC | 21:01 | |
*** mudassirlatif has joined #openstack-cinder | 21:02 | |
*** mss_ has joined #openstack-cinder | 21:02 | |
*** raildo is now known as raildo-afk | 21:02 | |
*** lpetrut has quit IRC | 21:03 | |
*** mss has quit IRC | 21:05 | |
*** Yogi1 has quit IRC | 21:06 | |
*** eharney has quit IRC | 21:06 | |
*** julim has quit IRC | 21:06 | |
*** r-daneel has quit IRC | 21:14 | |
*** mudassirlatif has quit IRC | 21:16 | |
*** stevemar has quit IRC | 21:19 | |
*** stevemar has joined #openstack-cinder | 21:20 | |
*** markvoelker has joined #openstack-cinder | 21:20 | |
*** mudassirlatif has joined #openstack-cinder | 21:22 | |
*** Lee1092 has quit IRC | 21:23 | |
*** gouthamr has quit IRC | 21:23 | |
*** stevemar has quit IRC | 21:24 | |
*** mudassirlatif has quit IRC | 21:25 | |
*** bnemec has quit IRC | 21:27 | |
*** IlyaG has quit IRC | 21:28 | |
*** IlyaG has joined #openstack-cinder | 21:28 | |
*** szhang has quit IRC | 21:29 | |
openstackgerrit | Sean McCully proposed openstack/cinder: Use of ast for integers doesn't changes type https://review.openstack.org/225947 | 21:30 |
*** mudassirlatif has joined #openstack-cinder | 21:32 | |
*** alexpilotti has quit IRC | 21:32 | |
*** IlyaG has quit IRC | 21:32 | |
*** cbader has quit IRC | 21:35 | |
*** bnemec has joined #openstack-cinder | 21:41 | |
*** mriedem is now known as mriedem_away | 21:43 | |
*** strony-zhang has joined #openstack-cinder | 21:57 | |
*** vgridnev has quit IRC | 21:59 | |
thingee | smcginnis: this one that you gave a +2 https://review.openstack.org/#/c/225891/ | 21:59 |
*** vgridnev has joined #openstack-cinder | 22:01 | |
*** mudassirlatif has quit IRC | 22:05 | |
*** jaypipes has quit IRC | 22:05 | |
*** alejandrito has quit IRC | 22:05 | |
*** vgridnev has quit IRC | 22:06 | |
*** mudassirlatif has joined #openstack-cinder | 22:06 | |
*** dsariel_ has joined #openstack-cinder | 22:08 | |
*** breitz has quit IRC | 22:08 | |
*** breitz has joined #openstack-cinder | 22:08 | |
*** jungleboyj has quit IRC | 22:15 | |
*** stevemar has joined #openstack-cinder | 22:16 | |
*** gouthamr has joined #openstack-cinder | 22:20 | |
*** stevemar has quit IRC | 22:20 | |
*** bswartz has joined #openstack-cinder | 22:22 | |
*** xyang1 has quit IRC | 22:24 | |
*** edtubill has quit IRC | 22:27 | |
*** alexpilotti has joined #openstack-cinder | 22:31 | |
*** diogogmt has quit IRC | 22:35 | |
*** IlyaG has joined #openstack-cinder | 22:35 | |
*** IlyaG_ has joined #openstack-cinder | 22:36 | |
*** dims__ has quit IRC | 22:37 | |
*** annegentle has quit IRC | 22:39 | |
*** IlyaG has quit IRC | 22:39 | |
*** nikeshm has quit IRC | 22:40 | |
*** IlyaG_ has quit IRC | 22:44 | |
*** markvoelker has quit IRC | 22:48 | |
*** cdelatte has quit IRC | 22:49 | |
*** angela-s has quit IRC | 22:55 | |
*** edtubill has joined #openstack-cinder | 22:55 | |
*** aix has quit IRC | 23:02 | |
*** mtanino has quit IRC | 23:05 | |
*** hemna is now known as hemnafk | 23:06 | |
*** DericHorn-HP has quit IRC | 23:09 | |
*** edmondsw has quit IRC | 23:13 | |
*** alexpilotti has quit IRC | 23:16 | |
*** dims has joined #openstack-cinder | 23:17 | |
*** mudassirlatif has quit IRC | 23:22 | |
*** mudassirlatif has joined #openstack-cinder | 23:22 | |
*** dims has quit IRC | 23:23 | |
*** jamielennox|away is now known as jamielennox | 23:23 | |
*** dims has joined #openstack-cinder | 23:23 | |
*** mudassirlatif has quit IRC | 23:26 | |
*** pv_ has joined #openstack-cinder | 23:28 | |
*** IanGovett has quit IRC | 23:28 | |
pv_ | do you guys know what cinder uses for device rescans? im using a shell script inside my attach code for a device rescan before our backend shows the appropriate LUN mapping, does openstack have a built in way to do the rescan>? | 23:28 |
pv_ | im testing my FC cinder driver | 23:28 |
*** mdenny has quit IRC | 23:29 | |
*** sgotliv has quit IRC | 23:38 | |
*** dims has quit IRC | 23:39 | |
jgriffith | cburgess_: https://github.com/j-griffith/cinder/tree/add_cache_to_icehouse | 23:39 |
*** dsariel_ has quit IRC | 23:39 | |
jgriffith | oops | 23:39 |
*** dims has joined #openstack-cinder | 23:40 | |
*** Apoorva_ has quit IRC | 23:42 | |
*** pv_ has quit IRC | 23:43 | |
*** dims_ has joined #openstack-cinder | 23:43 | |
*** Apoorva has joined #openstack-cinder | 23:43 | |
*** dims has quit IRC | 23:45 | |
*** zhangjn has joined #openstack-cinder | 23:54 | |
*** zhangjn has quit IRC | 23:54 | |
*** zhangjn has joined #openstack-cinder | 23:54 | |
*** zhangjn has quit IRC | 23:55 | |
*** zhangjn has joined #openstack-cinder | 23:55 | |
*** zhangjn has quit IRC | 23:55 | |
*** zhangjn has joined #openstack-cinder | 23:56 | |
*** markvoelker has joined #openstack-cinder | 23:56 | |
*** EinstCrazy has joined #openstack-cinder | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!