*** dave-mccowan has quit IRC | 00:02 | |
*** dr_gogeta86 has quit IRC | 00:04 | |
*** namnh has joined #openstack-cinder | 00:04 | |
*** edmondsw has joined #openstack-cinder | 00:07 | |
*** dr_gogeta86 has joined #openstack-cinder | 00:07 | |
*** clarkb has quit IRC | 00:09 | |
*** namnh has quit IRC | 00:10 | |
*** edmondsw has quit IRC | 00:12 | |
*** _ix has joined #openstack-cinder | 00:15 | |
*** tswanson has quit IRC | 00:21 | |
*** dave-mccowan has joined #openstack-cinder | 00:22 | |
*** mriedem has quit IRC | 00:29 | |
*** SatelliteN00b has quit IRC | 00:35 | |
*** _ix has quit IRC | 00:36 | |
*** zhaochao has joined #openstack-cinder | 00:55 | |
*** dave-mccowan has quit IRC | 00:55 | |
openstackgerrit | Merged openstack/cinder master: XtremIO: support multiattach https://review.openstack.org/554524 | 01:02 |
---|---|---|
*** EmilienM has quit IRC | 01:02 | |
*** EmilienM has joined #openstack-cinder | 01:03 | |
*** namnh has joined #openstack-cinder | 01:08 | |
*** namnh has quit IRC | 01:13 | |
*** itlinux_ has quit IRC | 01:26 | |
*** edmondsw has joined #openstack-cinder | 01:29 | |
*** namnh has joined #openstack-cinder | 01:35 | |
*** liuzz_ has quit IRC | 01:35 | |
*** dave-mccowan has joined #openstack-cinder | 01:47 | |
*** edmondsw has quit IRC | 01:50 | |
*** edmondsw has joined #openstack-cinder | 01:51 | |
*** edmondsw has quit IRC | 01:55 | |
*** namnh has quit IRC | 01:59 | |
*** namnh has joined #openstack-cinder | 02:08 | |
*** psachin has joined #openstack-cinder | 02:20 | |
*** e0ne has joined #openstack-cinder | 02:23 | |
*** e0ne has quit IRC | 02:23 | |
*** LutzB has quit IRC | 02:59 | |
*** gcb has quit IRC | 03:07 | |
*** tswanson has joined #openstack-cinder | 03:11 | |
*** edmondsw has joined #openstack-cinder | 03:19 | |
*** edmondsw has quit IRC | 03:24 | |
*** dave-mccowan has quit IRC | 03:33 | |
*** dalgaaf has quit IRC | 03:34 | |
*** namnh has quit IRC | 03:44 | |
*** fnordahl has joined #openstack-cinder | 05:05 | |
*** tswanson has quit IRC | 05:29 | |
*** Luzi has joined #openstack-cinder | 06:12 | |
openstackgerrit | yenai proposed openstack/cinder master: Add none check when do _retype_error https://review.openstack.org/588144 | 06:12 |
openstackgerrit | TommyLike proposed openstack/cinder master: Increase length limit on image metadata content https://review.openstack.org/586737 | 06:16 |
*** hoonetorg has quit IRC | 06:42 | |
*** hoonetorg has joined #openstack-cinder | 06:55 | |
*** rcernin has quit IRC | 07:03 | |
openstackgerrit | wu.chunyang proposed openstack/python-cinderclient master: update wrong link https://review.openstack.org/588156 | 07:11 |
*** gouthamr has quit IRC | 07:36 | |
*** devfaz has quit IRC | 07:45 | |
*** Tahvok has quit IRC | 07:45 | |
*** strigazi has quit IRC | 07:45 | |
*** mvk_ has quit IRC | 07:46 | |
*** tbarron has quit IRC | 07:46 | |
*** eandersson has quit IRC | 07:46 | |
*** Tahvok has joined #openstack-cinder | 07:46 | |
openstackgerrit | Vivek Soni proposed openstack/cinder master: VSA: Concurrent request handling in attachment https://review.openstack.org/588171 | 08:17 |
*** gnufied has quit IRC | 08:33 | |
*** pcaruana has joined #openstack-cinder | 09:00 | |
openstackgerrit | Merged openstack/cinder master: nimble: Fix missing ssl support https://review.openstack.org/587811 | 09:01 |
openstackgerrit | Boxiang Zhu proposed openstack/cinder master: replace status value with objects.fields https://review.openstack.org/588189 | 09:31 |
*** arne_wiebalck has joined #openstack-cinder | 09:44 | |
openstackgerrit | David Rabel proposed openstack/cinder master: Fix indentation in docs https://review.openstack.org/588193 | 09:44 |
*** edmondsw has joined #openstack-cinder | 09:48 | |
*** hoonetorg has quit IRC | 09:54 | |
*** VikasReddy1 has quit IRC | 09:59 | |
openstackgerrit | Boxiang Zhu proposed openstack/cinder master: replace status value with objects.fields https://review.openstack.org/588189 | 10:00 |
*** tbarron has joined #openstack-cinder | 10:05 | |
*** hoonetorg has joined #openstack-cinder | 10:09 | |
openstackgerrit | Boxiang Zhu proposed openstack/cinder master: remove some invalid variable statement https://review.openstack.org/588206 | 10:35 |
*** vivsoni has joined #openstack-cinder | 10:39 | |
openstackgerrit | Boxiang Zhu proposed openstack/cinder master: meet the pep8 N806 rule https://review.openstack.org/588210 | 10:51 |
*** ganso has joined #openstack-cinder | 10:55 | |
*** wolsen has quit IRC | 11:04 | |
*** fmccarthy has quit IRC | 11:04 | |
*** rajinir has quit IRC | 11:04 | |
*** lamt has quit IRC | 11:04 | |
*** simondodsley_ has quit IRC | 11:04 | |
*** simondodsley_ has joined #openstack-cinder | 11:05 | |
*** zzzeek has quit IRC | 11:07 | |
*** zzzeek has joined #openstack-cinder | 11:13 | |
*** dave-mccowan has joined #openstack-cinder | 11:27 | |
*** luizbag has joined #openstack-cinder | 11:32 | |
*** abishop has joined #openstack-cinder | 11:57 | |
openstackgerrit | caixiaoyu proposed openstack/cinder master: volume/rbd: Add judgment whether a rbd snapshot is protected from deletion https://review.openstack.org/588226 | 12:02 |
*** vivsoni_ has joined #openstack-cinder | 12:06 | |
*** vivsoni has quit IRC | 12:07 | |
*** tpsilva has joined #openstack-cinder | 12:15 | |
openstackgerrit | caixiaoyu proposed openstack/cinder master: volume/rbd: Judge protected attr when deleting snapshot https://review.openstack.org/588226 | 12:16 |
*** dustins has joined #openstack-cinder | 12:20 | |
*** EmilienM has left #openstack-cinder | 12:21 | |
*** mvenesio has quit IRC | 12:25 | |
*** jmlowe has quit IRC | 12:35 | |
*** cmart has quit IRC | 12:35 | |
*** mriedem has joined #openstack-cinder | 12:57 | |
*** eharney has quit IRC | 12:59 | |
openstackgerrit | Boxiang Zhu proposed openstack/cinder master: replace status value with objects.fields https://review.openstack.org/588189 | 13:33 |
*** Luzi has quit IRC | 13:35 | |
*** bswartz has quit IRC | 13:38 | |
*** gnufied has joined #openstack-cinder | 13:40 | |
*** _ix has joined #openstack-cinder | 13:41 | |
*** eharney has joined #openstack-cinder | 13:45 | |
*** _ix has quit IRC | 13:49 | |
hemna_ | morning | 13:55 |
smcginnis | Hey hemna_ o/ | 13:59 |
openstackgerrit | Merged openstack/cinder master: Fix indentation in docs https://review.openstack.org/588193 | 14:00 |
*** erlon has quit IRC | 14:04 | |
*** luizbag has quit IRC | 14:04 | |
jungleboyj | Morning all. | 14:12 |
mnaser | hi everyone | 14:15 |
*** luizbag has joined #openstack-cinder | 14:15 | |
*** erlon has joined #openstack-cinder | 14:16 | |
mnaser | we make heavy use of the iops_per_gb feature and one of our private cloud users runs very small volumes which means sometimes they might end up with a tiny volume that barely gets any iops | 14:16 |
mnaser | i'd like to propose a change that adds an iops_per_gb_min which is a minimum, so it gets either that or the multiplied value | 14:17 |
mnaser | i don't think it would be a huge change, probably a few lines, but is it too late in the cycle :( | 14:17 |
mnaser | (there should be a question mark there) | 14:17 |
eharney | mnaser: sounds like a reasonable thing to work on | 14:18 |
smcginnis | Yeah, I would expect that to be a small change. | 14:18 |
jungleboyj | ++ | 14:18 |
mnaser | i'm just wondering if that type of thing can still get merged or it's too late .. it would be nice to be able to get it for rocky rather than keep it only be able to get it in stein | 14:18 |
smcginnis | If we see a patch soon, I would assume it would be a low risk thing to still allow in. | 14:19 |
jungleboyj | Kind of sounds like a bug to me? ;-) | 14:19 |
mnaser | jungleboyj: sneaky, aha ;p | 14:20 |
mnaser | i'll push up something very shortly | 14:20 |
jungleboyj | Sounds good. Ping us and we will take a look. | 14:20 |
openstackgerrit | Mohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS https://review.openstack.org/588302 | 14:45 |
mnaser | jungleboyj, smcginnis, eharney ^ patch with tests, docs and release notes :> | 14:45 |
smcginnis | mnaser: That was quick! | 14:45 |
mnaser | it was pretty straight forward | 14:45 |
mnaser | ugh i touched one of the original tests | 14:46 |
mnaser | one sec | 14:46 |
openstackgerrit | Mohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS https://review.openstack.org/588302 | 14:46 |
mnaser | okay the diff should look much cleaner now | 14:46 |
openstackgerrit | Mohammed Naser proposed openstack/cinder master: Add ability to specify minimum value when using capacity based QoS https://review.openstack.org/588302 | 14:55 |
mnaser | smcginnis: thanks for that comment, TIL, pop'ing a value that didnt exist was something i was thinking about | 14:55 |
mnaser | but i didnt know you could also get a default value when popping so cool :> | 14:55 |
mnaser | jungleboyj: ^ | 14:56 |
smcginnis | mnaser: Always discovering ways python makes simple things even easier. ;) | 14:56 |
mnaser | absolutely :D | 14:56 |
*** gouthamr has joined #openstack-cinder | 15:02 | |
*** tomtom001 has joined #openstack-cinder | 15:14 | |
tomtom001 | Hello all, trying to setup the cinder backup service with swift for Openstack Newton. I set a user and key for the cinder user. I keep getting the following error message in the cinder-volume service whenever I try to make a backup. Container PUT failed: http://10.0.30.10:8080/v1/AUTH_744b62717c1946cb95d11909fea98a4e/ 405 Method Not Allowed -- Anyone know why the method is not allowed? Is | 15:17 |
tomtom001 | it just an auth thing? What .conf options should be defined at minimum to get it to authenticate properly? | 15:17 |
*** psachin has quit IRC | 15:21 | |
jungleboyj | tomtom001: Sounds like an authentication issue. | 15:23 |
jungleboyj | I haven't done much with backup. e0ne should pop on later and might be able to provide more guidance. | 15:23 |
*** tswanson has joined #openstack-cinder | 15:30 | |
*** cmart has joined #openstack-cinder | 15:53 | |
openstackgerrit | Merged openstack/cinder master: Update volume create schema to allow 'null' https://review.openstack.org/587821 | 16:07 |
*** gman-tx has joined #openstack-cinder | 16:09 | |
*** eandersson has joined #openstack-cinder | 16:10 | |
*** arne_wiebalck has quit IRC | 16:13 | |
*** walshh_ has joined #openstack-cinder | 16:16 | |
*** spotz has quit IRC | 16:18 | |
tomtom001 | jungleboyj - thank you, that's what I thought, I'll keep playing around with the settings to see if I can get it working. | 16:28 |
*** walshh_ has quit IRC | 16:30 | |
*** walshh_ has joined #openstack-cinder | 16:32 | |
openstackgerrit | Merged openstack/cinder master: tintri: Enable SSL with requests https://review.openstack.org/587445 | 16:34 |
openstackgerrit | Merged openstack/cinder master: NEC driver: Return non-random iscsi portal in initialize https://review.openstack.org/576390 | 16:34 |
openstackgerrit | Merged openstack/cinder master: [doc] Use openstack client commands to replace cinder client https://review.openstack.org/580935 | 16:34 |
openstackgerrit | Merged openstack/cinder master: docs: Write high availability devref https://review.openstack.org/571242 | 16:34 |
*** imacdonn has quit IRC | 16:36 | |
*** imacdonn has joined #openstack-cinder | 16:37 | |
openstackgerrit | Jay Bryant proposed openstack/cinder master: Add sphinx-feature-classification based support matrix https://review.openstack.org/580757 | 16:41 |
*** walshh_ has quit IRC | 16:41 | |
openstackgerrit | Jay Rubenstein proposed openstack/cinder master: SF ensure the correct volume is deleted https://review.openstack.org/588342 | 17:00 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-cinderclient master: __repr__ crashes when empty dict passed https://review.openstack.org/588346 | 17:15 |
*** gnufied has quit IRC | 17:31 | |
openstackgerrit | Merged openstack/cinder master: VSA: Concurrent request handling in attachment https://review.openstack.org/588171 | 17:34 |
*** gnufied has joined #openstack-cinder | 17:47 | |
*** itlinux has joined #openstack-cinder | 18:14 | |
*** gman-tx has quit IRC | 18:15 | |
*** tswanson has quit IRC | 18:37 | |
openstackgerrit | Merged openstack/cinder master: Fix remotefs driver report wrong value https://review.openstack.org/576975 | 19:03 |
luizbag | People, has anyone noticed failing stable/queens jobs due to os-brick on fibre channel? | 19:05 |
smcginnis | luizbag: Did it just start? | 19:06 |
eharney | luizbag: unit tests? | 19:06 |
luizbag | eharney, tempest tests | 19:06 |
luizbag | smcginnis, we started noticing it a few days ago | 19:07 |
smcginnis | luizbag: Did you change something in your environment? I don't think we've really backported much to stable/queens lately. | 19:07 |
luizbag | smcginnis, we haven't changed anything | 19:09 |
smcginnis | Nothing of significance changed there that I see for awhile: https://review.openstack.org/#/q/project:openstack/cinder+status:merged+branch:stable/queens | 19:09 |
openstackgerrit | Merged openstack/os-brick stable/queens: RemoteFS: don't fail in do_mount if already mounted https://review.openstack.org/583237 | 19:13 |
*** gnufied has quit IRC | 19:27 | |
*** dustins has quit IRC | 19:38 | |
*** gnufied has joined #openstack-cinder | 19:41 | |
*** eharney has quit IRC | 19:44 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: solidfire: Enable SSL with requests https://review.openstack.org/588365 | 19:44 |
*** lseki has joined #openstack-cinder | 19:44 | |
jungleboyj | luizbag: There was another question about tests timing out on stable/queens yesterday. | 20:00 |
*** zul has quit IRC | 20:01 | |
luizbag | interesting | 20:02 |
jungleboyj | smcginnis: So, the requirements on stable queens were for os-brick >= 2.2.0 . | 20:03 |
jungleboyj | We are up to 2.5.3 | 20:03 |
smcginnis | jungleboyj: upper-constraints limits that though. | 20:03 |
jungleboyj | So, would that be what is getting pulled in when running tempest since we didn't cap? | 20:03 |
smcginnis | https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L217 | 20:03 |
jungleboyj | smcginnis: Ok. That is good. | 20:04 |
*** masayukig has quit IRC | 20:18 | |
luizbag | the problem is with the os-brick version on upper-constraints.txt | 20:21 |
luizbag | it's freezing the version on 2.3.2 | 20:21 |
smcginnis | 2.3.2 has been out for a month and a half, so why would it start failing for you now? | 20:22 |
luizbag | because now we have patches to run on fibre channel | 20:23 |
smcginnis | luizbag: So you weren't running before and that's why you just noticed it fails? | 20:24 |
luizbag | no, the upper-constraints.txt is updated on master, but on stable/queens it's with the old version | 20:25 |
luizbag | we're working on a backport to stable/queens that's having this issue | 20:25 |
luizbag | the patches for master branch are passing | 20:25 |
smcginnis | luizbag: Right, this is stable/queens we're talking about, not master I thought. | 20:26 |
luizbag | should I open a bug for that? | 20:27 |
smcginnis | For what?! | 20:27 |
smcginnis | Are we talking stable/queens or are we talking master? | 20:27 |
luizbag | stable/queens | 20:27 |
luizbag | to update the upper-constraints.txt file | 20:28 |
smcginnis | luizbag: It has the most up to date in upper-constraints already. | 20:29 |
luizbag | or just create a patch to update it? | 20:29 |
luizbag | yes, on master branch | 20:29 |
luizbag | the stable/queens branch on requirements project is not up to date | 20:29 |
smcginnis | luizbag: Yes it is. | 20:30 |
luizbag | but I'm reading version 2.3.2 on it... | 20:30 |
smcginnis | Yes. | 20:31 |
smcginnis | That's the latest stable/queens os-brick version - https://github.com/openstack/releases/blob/master/deliverables/queens/os-brick.yaml#L35 | 20:31 |
luizbag | so I'm kinda lost here | 20:32 |
luizbag | how do I fix it? | 20:32 |
smcginnis | luizbag: It depends. What is actually broken? | 20:32 |
smcginnis | If it's something that's been fixed during rocky, then it should be backported to stable/queens and we can do another release. | 20:33 |
smcginnis | But other than it's broke and nothing was changed, I don't know what the problem is. :) | 20:33 |
luizbag | I've found a bug already reported: https://bugs.launchpad.net/os-brick/+bug/1774283 | 20:35 |
openstack | Launchpad bug 1774283 in os-brick "fc rescan issue when looking at specific hbas" [Undecided,Fix released] - Assigned to Patrick East (patrick-east) | 20:35 |
luizbag | that's the problem | 20:35 |
smcginnis | Yeah, that looks like a good thing to backport. | 20:38 |
jungleboyj | smcginnis: ++ | 20:38 |
jungleboyj | luizbag: Do you know how to backport? | 20:38 |
luizbag | yes | 20:39 |
smcginnis | luizbag: It you put up a patch to backport that, then jungleboyj and I can take a look and get it through if all's good. | 20:39 |
smcginnis | And get a new stable/queens release done. | 20:39 |
jungleboyj | Ok. Please propose the backport and then I will work on requesting another release. | 20:39 |
jungleboyj | Yeah, what smcginnis said. ;-) | 20:40 |
luizbag | ok, thanks, I'll do it asap | 20:46 |
*** luizbag has quit IRC | 20:47 | |
*** lseki has quit IRC | 21:02 | |
*** dcdamien has quit IRC | 21:17 | |
*** abishop has quit IRC | 21:24 | |
*** mriedem is now known as mriedem_afk | 21:28 | |
*** itlinux has quit IRC | 21:54 | |
*** mchlumsky has quit IRC | 22:02 | |
*** ganso has quit IRC | 22:24 | |
*** rcernin has joined #openstack-cinder | 22:32 | |
openstackgerrit | Chuck Short proposed openstack/cinder master: solidfire: Enable SSL with requests https://review.openstack.org/588365 | 22:34 |
*** tswanson has joined #openstack-cinder | 22:47 | |
*** tpsilva has quit IRC | 22:53 | |
*** edmondsw has quit IRC | 23:08 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!