| *** ganso has quit IRC | 00:07 | |
| *** dklyle has quit IRC | 00:20 | |
| *** Nel1x has joined #openstack-cinder | 00:31 | |
| *** brinzhang has joined #openstack-cinder | 00:47 | |
| *** sapd1 has joined #openstack-cinder | 01:15 | |
| *** zhaochao has joined #openstack-cinder | 01:16 | |
| openstackgerrit | Merged openstack/cinder master: NetApp SolidFire: Fix NetApp SolidFire SSL option https://review.openstack.org/590586 | 01:18 |
|---|---|---|
| zul | jungleboyj: sorry i probably should have asked my question on irc | 01:28 |
| jungleboyj | zul: what was the question? | 01:29 |
| zul | jungleboyj: about the retries | 01:32 |
| zul | jungleboyj: doh...nm....i suck :) | 01:32 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Update doc for get-pools API https://review.openstack.org/591878 | 01:33 |
| *** tommylikehu has joined #openstack-cinder | 01:36 | |
| tommylikehu | check | 01:38 |
| openstackgerrit | TommyLike proposed openstack/cinder master: Update doc for get-pools API https://review.openstack.org/591878 | 01:40 |
| *** hemna_ has quit IRC | 01:59 | |
| *** hemna_ has joined #openstack-cinder | 02:09 | |
| *** openstack has joined #openstack-cinder | 02:37 | |
| *** ChanServ sets mode: +o openstack | 02:37 | |
| *** cmart has joined #openstack-cinder | 02:51 | |
| *** cmart has quit IRC | 03:01 | |
| *** Nel1x has quit IRC | 03:11 | |
| *** hemna_ has quit IRC | 03:15 | |
| *** cmart has joined #openstack-cinder | 03:32 | |
| *** cmart has quit IRC | 03:41 | |
| *** gnufied has quit IRC | 03:58 | |
| *** gnufied has joined #openstack-cinder | 04:00 | |
| *** eharney has quit IRC | 04:15 | |
| *** gnufied has quit IRC | 04:26 | |
| whoami-rajat | Hi jungleboyj , smcginnis , i'm confused about Nible Storage CI behaviour here https://review.openstack.org/#/c/589951/, initially it was failing both tests and currently passing one, i checked out other patches as well, some have both failing some have both passing, and the LOGS aren't available as well. | 04:26 |
| whoami-rajat | This is the recent behaviour i saw here https://review.openstack.org/#/c/591878/ | 04:26 |
| whoami-rajat | https://review.openstack.org/#/c/591878/ | 04:26 |
| whoami-rajat | FAILURE and UNSTABLE. | 04:26 |
| *** eharney has joined #openstack-cinder | 04:27 | |
| *** hemna_ has joined #openstack-cinder | 05:12 | |
| *** pcaruana has joined #openstack-cinder | 05:12 | |
| *** hemna_ has quit IRC | 05:17 | |
| *** e0ne has joined #openstack-cinder | 06:16 | |
| *** alexchadin has joined #openstack-cinder | 06:59 | |
| *** geguileo has quit IRC | 07:08 | |
| *** lennyb_ has joined #openstack-cinder | 07:34 | |
| *** lennyb_ is now known as lennyb | 07:35 | |
| *** lennyb has quit IRC | 07:35 | |
| openstackgerrit | Boxiang Zhu proposed openstack/python-cinderclient master: Use store_true action for attachment cli https://review.openstack.org/591952 | 08:06 |
| *** rha has quit IRC | 08:34 | |
| *** lennyb has joined #openstack-cinder | 08:35 | |
| *** alexchadin has quit IRC | 09:00 | |
| *** josecastroleon has quit IRC | 09:00 | |
| *** josecastroleon has joined #openstack-cinder | 09:00 | |
| *** andreaf has quit IRC | 09:16 | |
| *** andreaf has joined #openstack-cinder | 09:17 | |
| *** alexchadin has joined #openstack-cinder | 09:17 | |
| *** dirk_ has joined #openstack-cinder | 09:17 | |
| *** dirk has quit IRC | 09:18 | |
| *** wiggin15 has quit IRC | 09:18 | |
| *** dirk_ is now known as dirk | 09:18 | |
| *** gkadam has joined #openstack-cinder | 09:30 | |
| *** rtjure has joined #openstack-cinder | 09:35 | |
| *** josecastroleon has quit IRC | 10:09 | |
| *** zul has quit IRC | 10:25 | |
| *** alexchadin has quit IRC | 10:59 | |
| *** sapd1 has quit IRC | 10:59 | |
| *** alexchadin has joined #openstack-cinder | 11:00 | |
| *** alexchadin has quit IRC | 11:00 | |
| *** alexchadin has joined #openstack-cinder | 11:01 | |
| *** alexchadin has quit IRC | 11:01 | |
| *** alexchadin has joined #openstack-cinder | 11:01 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder master: Fixing security bug in scaleio driver https://review.openstack.org/592001 | 11:02 |
| *** alexchadin has quit IRC | 11:02 | |
| *** pvc has joined #openstack-cinder | 11:02 | |
| pvc | hi guys | 11:02 |
| pvc | Volume size extension is not supported by the hypervisor.: ExtendVolumeNotSupported: Volume size extension is not supported by the hypervisor. | 11:03 |
| *** alexchadin has joined #openstack-cinder | 11:33 | |
| *** alexchadin has quit IRC | 11:38 | |
| *** brinzhang has quit IRC | 11:39 | |
| *** alexchadin has joined #openstack-cinder | 11:56 | |
| *** eharney has quit IRC | 11:58 | |
| *** josecastroleon has joined #openstack-cinder | 12:00 | |
| *** abishop has joined #openstack-cinder | 12:08 | |
| *** gcb_ has joined #openstack-cinder | 12:19 | |
| *** alexchadin has quit IRC | 12:23 | |
| *** alexchadin has joined #openstack-cinder | 12:30 | |
| *** erlon has joined #openstack-cinder | 12:34 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder master: Fixing security bug in scaleio driver https://review.openstack.org/592001 | 12:41 |
| *** zul has joined #openstack-cinder | 12:43 | |
| *** josecastroleon has quit IRC | 12:54 | |
| *** josecastroleon has joined #openstack-cinder | 12:54 | |
| *** cmart has joined #openstack-cinder | 12:58 | |
| *** gnufied has joined #openstack-cinder | 13:00 | |
| *** eharney has joined #openstack-cinder | 13:09 | |
| *** mvkr has quit IRC | 13:10 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder master: Fixing security bug in scaleio driver https://review.openstack.org/592001 | 13:10 |
| *** markstur_ has joined #openstack-cinder | 13:18 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Fix api-ref title levels and index https://review.openstack.org/592026 | 13:20 |
| mnaser | has anyone seen a weird issue where cinder-volume for some reason doesnt send heartbeats to rabbitmq which results in the connection being dropped, or something along those lines.. | 13:28 |
| *** luizbag has joined #openstack-cinder | 13:29 | |
| *** luizbag has quit IRC | 13:30 | |
| mnaser | "closing AMQP connection <0.894.0> (10.1.1.1:49244 -> 10.1.1.101:5672 - cinder-volume:23478:a2a29ff1-a431-4db5-9d05-f3ffb68d431d): missed heartbeats from client, timeout: 60s" | 13:30 |
| *** dklyle has joined #openstack-cinder | 13:31 | |
| mnaser | and at the same time.. cinder "[b956eec0-15bb-4070-8a23-fd0f3c5e5a8e] AMQP server on 10.1.1.101:5672 is unreachable: <AMQPError: unknown error>. Trying again in 1 seconds." | 13:32 |
| *** rambo_li has joined #openstack-cinder | 13:35 | |
| *** rambo_li has quit IRC | 13:38 | |
| *** mvkr has joined #openstack-cinder | 13:40 | |
| openstackgerrit | Michael McAleer proposed openstack/cinder master: VMAX Driver - Initiator retrieval short hostname fix https://review.openstack.org/589772 | 13:40 |
| *** david-lyle has joined #openstack-cinder | 13:48 | |
| *** dklyle has quit IRC | 13:49 | |
| *** david-lyle has quit IRC | 13:49 | |
| *** david-lyle has joined #openstack-cinder | 13:49 | |
| KeithMnemonic | jungleboyj: smcginnis is this ready for WF +1 now? https://review.openstack.org/#/c/591759/ | 14:12 |
| smcginnis | KeithMnemonic: I'm not too comfortable with the versioning scheme there being the same as the now divergent later driver versions. | 14:16 |
| smcginnis | KeithMnemonic: It really would be better if there was some indication that this is different. | 14:16 |
| smcginnis | KeithMnemonic: It doens't need to conform to a full semver scheme. Just putting something like 2.0.13a or something to have an indication that this is a variant of 2.0.13 would be better IMO. | 14:17 |
| *** erlon has quit IRC | 14:21 | |
| KeithMnemonic | ok i can do that | 14:21 |
| KeithMnemonic | is my "not included" for 2.0.12 ok? | 14:21 |
| KeithMnemonic | i want to make sure that is clear | 14:22 |
| openstackgerrit | Chi Lo proposed openstack/cinder master: Add policy granularity to the encryption API https://review.openstack.org/571563 | 14:23 |
| smcginnis | KeithMnemonic: Yeah, that's probably fine to have that called out. | 14:24 |
| *** erlon has joined #openstack-cinder | 14:24 | |
| openstackgerrit | Keith Berger proposed openstack/cinder stable/ocata: HPE Lefthand: Fixed cloning operation https://review.openstack.org/591759 | 14:25 |
| KeithMnemonic | eharney: i did not quite get that. 2.0.11 is lined up with master, it is only 2.13 that becomes special | 14:30 |
| KeithMnemonic | and whar happnes with 2.14, 2.15, 2.16 | 14:30 |
| eharney | well ocata is 2.0.11, right? | 14:31 |
| KeithMnemonic | the ocata branch has 2.11, correct | 14:32 |
| *** alexchadin has quit IRC | 14:32 | |
| eharney | so if we branched the version numbers off for additional fixes, it would split there, not at 2.0.13 | 14:32 |
| KeithMnemonic | ok but dont i need to reference the original version that change was in | 14:33 |
| KeithMnemonic | so someone looking can line it up | 14:34 |
| eharney | standard versioning schemes would indicate that 2.0.13a includes everything that's in 2.0.13, but on ocata it doesn't | 14:35 |
| KeithMnemonic | so maybe 2.0.13_special? the problem is if i do 2.0.11a, then what is the version for the next fix 2.0.11b? | 14:36 |
| jungleboyj | pots: You around? | 14:37 |
| eharney | the next fix would be 2.0.11b, yes | 14:37 |
| KeithMnemonic | ok i can do that if that is the majority opinion ;-) | 14:37 |
| smcginnis | ¯\_(ツ)_/¯ | 14:38 |
| KeithMnemonic | so with that i remove the 2.0.12 "not included" entry? | 14:38 |
| KeithMnemonic | or just leave it at the end? | 14:39 |
| eharney | the ¯\_(ツ)_/¯ answer would be to just not use version numbers at all because they aren't really needed in drivers | 14:39 |
| *** zul has quit IRC | 14:39 | |
| eharney | but since we're already here... | 14:39 |
| jungleboyj | Where is the tableflipping bot when you need it? | 14:40 |
| smcginnis | Maybe we should document expectations for driver version numbers for backports. | 14:40 |
| jungleboyj | smcginnis: I don't remember this being an issue in the past. | 14:40 |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/queens: VMAX Queens driver - incorrect condition for an empty list. https://review.openstack.org/592065 | 14:40 |
| eharney | typically everyone just follows standard semantic version rules, or doesn't bump the version at all | 14:41 |
| KeithMnemonic | it looks funny leaving the 2.0.12 "not included entry" so i would just remove it if that is ok | 14:41 |
| openstackgerrit | Keith Berger proposed openstack/cinder stable/ocata: HPE Lefthand: Fixed cloning operation https://review.openstack.org/591759 | 14:42 |
| openstackgerrit | Helen Walsh proposed openstack/cinder stable/queens: VMAX Queens driver - incorrect condition for an empty list. https://review.openstack.org/592065 | 14:43 |
| *** david-lyle is now known as dklyle | 15:01 | |
| *** zhaochao has quit IRC | 15:08 | |
| *** masayukig has joined #openstack-cinder | 15:09 | |
| jungleboyj | smcginnis: eharney Ok, not going to be the go between here. | 15:29 |
| jungleboyj | Lets talk about config changes on backports. | 15:29 |
| eharney | sure | 15:29 |
| jungleboyj | Ok. reading smcginnis comments in the bug since I am naughty and haven't done that yet. | 15:31 |
| jungleboyj | eharney: Why do you feel that your solution is not backwards incompatible? | 15:32 |
| eharney | the goal is to support both the old and new option name | 15:33 |
| eharney | the goal i had in mind when writing that patch, i mean | 15:33 |
| smcginnis | jungleboyj: It is backwards compatible. | 15:33 |
| smcginnis | The only issue is that now we would have a config option that would be flagged as deprecated on older branches when it had not actually been deprecated. | 15:34 |
| jungleboyj | Ok ... | 15:34 |
| smcginnis | Hence my suggestion for the two step process to get the fix in in an easy way to backport, then get it deprecated. | 15:34 |
| smcginnis | Which was followed by multiple comments asking why it was separated. | 15:34 |
| eharney | i'm not sure that deprecated_name = flagged as deprecated? (maybe, honestly not sure) | 15:35 |
| jungleboyj | smcginnis: I am sorry. I hadn't see your comment. I understand now. | 15:35 |
| eharney | doesn't it just act as an alias? | 15:35 |
| smcginnis | I believe deprecated_name would cause the use of the old name to trigger warnings in the log that they are using a deprecated config option. | 15:35 |
| smcginnis | I can double check that though. | 15:35 |
| jungleboyj | smcginnis: And the concern is that suddenly people who have been using that in previous drivers will get a deprecation warning? | 15:37 |
| smcginnis | Yeah | 15:37 |
| smcginnis | We can't really retroactively deprecate something. | 15:38 |
| eharney | i think the warning may only happen if you set "deprecated_for_removal" https://git.openstack.org/cgit/openstack/oslo.config/tree/oslo_config/cfg.py?h=6.4.0#n1117 | 15:38 |
| jungleboyj | Ok... | 15:38 |
| eharney | i can go try some things to find out though | 15:39 |
| smcginnis | Not sure if this does it - https://git.openstack.org/cgit/openstack/oslo.config/tree/oslo_config/cfg.py?h=6.4.0#n1043 | 15:39 |
| smcginnis | If not, then I think we are OK using deprecated_name. | 15:39 |
| jungleboyj | So, the advantage of Eric's change is that people who have enabled this option automatically get coverage for both options. Right? | 15:40 |
| jungleboyj | But we are backporting a deprecation which seems kind-of wrong. | 15:40 |
| eharney | that is the theory, i'm hoping it actually works that way | 15:40 |
| eharney | i think deprecation is a misnomer for this case... it's a config parameter rename | 15:41 |
| jungleboyj | eharney: With the two option approach then the user will get the creation error until they add the new option or fix their backend config. | 15:42 |
| smcginnis | No, same behavior with the two opt option. | 15:43 |
| smcginnis | It just needs to check both opts instead of just one. | 15:44 |
| eharney | ok, just using deprecated_name does issue a warning at startup | 15:44 |
| eharney | so what's the ideal case here that we actually want? | 15:45 |
| jungleboyj | No comment. | 15:45 |
| smcginnis | What is the problem with the two step process I recommended? | 15:45 |
| jungleboyj | It is fine. It is the more correct way of doing this. | 15:46 |
| *** mvkr has quit IRC | 15:46 | |
| jungleboyj | I don't like adding more config options but more importantly we need to get this fixed. | 15:46 |
| eharney | it looks like it works fine | 15:46 |
| smcginnis | Yeah, it's kind of a hokey config dance, but it makes it clean to backport and gets us ready to remove the option going forward. We still end up where eharney's patch was going. | 15:48 |
| eharney | i'm a bit unsure about the logic in the patch | 15:49 |
| eharney | if you set allow_non_padded_volumes to True but allow_non_padded_thick_volumes to False, it passes the check | 15:50 |
| jungleboyj | Ok. So, if either of the options is set to trye it will say creation is safe. | 15:50 |
| smcginnis | eharney: Hmm, yeah. Looks like that part needs to be a little smarter. | 15:50 |
| smcginnis | And of course the unit tests need to be addressed. | 15:51 |
| jungleboyj | And there needs to be a release note. | 15:51 |
| *** ganso has joined #openstack-cinder | 15:51 | |
| eharney | the same concerns about compatibility for backports come up with changing the logic though :/ | 15:51 |
| eharney | (i think) | 15:52 |
| smcginnis | That part I think we are OK on since we are addressing a security issue and it will be called out in the release notes. | 15:52 |
| *** itlinux has joined #openstack-cinder | 15:52 | |
| jungleboyj | Ok. | 15:52 |
| smcginnis | "Why isn't this working now? Oh, because it was bad." :) | 15:52 |
| eharney | if someone has written tooling to set the old option, and we require the new option to be set anyway, then we introduce a compat issue | 15:53 |
| *** pvc_ has joined #openstack-cinder | 15:53 | |
| pvc_ | im currently using ceph backend for my voluems | 15:53 |
| pvc_ | but when i extending it on a attached state it have an error | 15:53 |
| eharney | but if we only allow one to be set -- then it's confusing what it means for thin volumes | 15:53 |
| pvc_ | Hypervisor is not supported the volume extension | 15:53 |
| eharney | only require* | 15:53 |
| pvc_ | any help guys? | 15:53 |
| pvc_ | my instance cant get the new size of the ceph volume :( | 15:53 |
| smcginnis | eharney: You're saying if someone has set tooling to allow them to do the unsecure way, now it changes that? | 15:54 |
| pvc_ | @smcginnis hi can you help me | 15:54 |
| pvc_ | hi | 15:54 |
| *** FracKen has joined #openstack-cinder | 15:54 | |
| smcginnis | pvc_: A little busy at the moment. | 15:54 |
| smcginnis | pvc_: But I don't think attached extend is supported with ceph. | 15:54 |
| eharney | if we try to preserve compatibility, we end up in a situation where allow_non_padded_volumes would apply to thick volumes, even if the old allow_non_padded_thick_volumes option is set to False | 15:54 |
| jungleboyj | With how it is now we don't have an issue but it is confusing as to what it really means. | 15:54 |
| pvc_ | So sad to hear that :( | 15:55 |
| eharney | which i guess works, but is kind of confusing | 15:55 |
| smcginnis | eharney: No, it should still be defaulted to not allow it even though that is a change in behavior. Since it's fixing a security issue. | 15:55 |
| eharney | i agree they should default to False, the question is what happens when you only set one of the options | 15:56 |
| jungleboyj | smcginnis: That makes more sense to me. | 15:56 |
| jungleboyj | I think we should probably throw an error if only one is set so they are aware they had an issue. | 15:57 |
| *** cmart has quit IRC | 15:57 | |
| pvc_ | anyone using ceph here as a volume? | 15:57 |
| jungleboyj | I realize that that kind-of goes against what I said earlier. | 15:57 |
| eharney | that sounds a lot like a deprecation | 15:57 |
| jungleboyj | But this is a security thing. | 15:57 |
| jungleboyj | jbernard: ^^^ Can you address the Ceph question? | 15:58 |
| pvc_ | Hi jbernard can help me? | 15:58 |
| jungleboyj | The support matrix has Attached Extend as supported but maybe that is wrong? If so please let me know so I can fix the matrix. | 15:58 |
| pvc_ | are you referring to me jungleboyj? | 15:59 |
| jungleboyj | pvc_: Yes. | 15:59 |
| pvc_ | when my cinder backend is LVM I can extend its size on a attached state | 15:59 |
| eharney | https://review.openstack.org/#/c/589925/ is in progress for ceph extend | 16:00 |
| eharney | needs testing/review | 16:00 |
| pvc_ | but when im using ceph rbd volume backend, my instance cant get it even if the horizon change its size. | 16:00 |
| jungleboyj | Ok, I should fix the existing infor for that then. | 16:01 |
| pvc_ | i see eharney thank you for this | 16:01 |
| eharney | so it seems like we may need a warning message if "allow_thick" is False but "allow" is True saying that this may be doing something unexpected? | 16:02 |
| eharney | (given the context that the original security notice said you could avoid this problem by using thin) | 16:02 |
| smcginnis | eharney: That might be the best approach. | 16:02 |
| eharney | and just keep the logic that's in the patch currently for how the options actually work? | 16:02 |
| pvc_ | hi eharney is that review applicable for Kolla Openstack deployment? thank you for your help | 16:03 |
| eharney | pvc_: it's part of supporting it in cinder/nova | 16:11 |
| pvc_ | thank you, so for now the on fly extension of ceph rbd is not yet supported? | 16:13 |
| *** sayalilunkad has quit IRC | 16:16 | |
| *** sayalilunkad has joined #openstack-cinder | 16:20 | |
| *** cmart has joined #openstack-cinder | 16:30 | |
| *** sayalilunkad has quit IRC | 16:32 | |
| *** clayc has joined #openstack-cinder | 16:33 | |
| jungleboyj | smcginnis: eharney I need to go get my boys from their Mom's. | 16:37 |
| jungleboyj | We need to land on an answer for the config options though so we can resolve the issue. | 16:37 |
| smcginnis | I commented on the patch that they should add a release note. | 16:39 |
| KeithMnemonic | eharney: hopefully this does the trick https://review.openstack.org/#/c/591759/ | 16:39 |
| smcginnis | If we are good with the two step plan, maybe good to follow up on there since right now there's still some dissent as the last comments. | 16:39 |
| smcginnis | And the email thread. | 16:39 |
| jungleboyj | Ok. Will look like when I get back. | 16:42 |
| *** pvc_ has quit IRC | 16:45 | |
| *** sayalilunkad has joined #openstack-cinder | 16:47 | |
| *** e0ne has quit IRC | 16:47 | |
| *** sayalilunkad has quit IRC | 16:51 | |
| *** sayalilunkad has joined #openstack-cinder | 16:52 | |
| *** openstackstatus has joined #openstack-cinder | 16:56 | |
| *** ChanServ sets mode: +v openstackstatus | 16:56 | |
| mnaser | does anyone know if this job is actually running in cinder ci ? https://github.com/openstack/cinder-tempest-plugin/blob/master/cinder_tempest_plugin/api/volume/test_volume_unicode.py | 17:02 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX Driver - Initiator retrieval short hostname fix https://review.openstack.org/589772 | 17:04 |
| mnaser | because that change is def. failing in OSA | 17:05 |
| *** gkadam has quit IRC | 17:05 | |
| mnaser | http://logs.openstack.org/85/592085/3/check/openstack-ansible-functional-ubuntu-xenial/99dbc7a/logs/openstack/cinder/cinder-api.service.journal.log.txt.gz#_Aug_15_16_25_52 with this stacktrace | 17:05 |
| *** zul has joined #openstack-cinder | 17:08 | |
| mnaser | looks like its manifesting because of this failure -- https://github.com/openstack/taskflow/blob/master/taskflow/types/notifier.py#L215-L217 | 17:08 |
| *** sayalilunkad has quit IRC | 17:09 | |
| mnaser | it looks like cinder_tempest_plugin is not actually ran anywhere? | 17:16 |
| *** gkadam has joined #openstack-cinder | 17:20 | |
| *** eharney has quit IRC | 17:26 | |
| *** dustins has joined #openstack-cinder | 17:33 | |
| *** markstur_ has quit IRC | 17:34 | |
| *** cmart has quit IRC | 17:36 | |
| *** markstur has joined #openstack-cinder | 17:41 | |
| *** markstur has quit IRC | 17:45 | |
| smcginnis | mnaser: Hmm, that would be concerning. I thought it was being pulled in, but it would appear otherwise. | 17:46 |
| mnaser | smcginnis: i checked a few recent commits and it's def not being ran | 17:46 |
| smcginnis | OK, we must be missing something somewhere that pulls that in. | 17:46 |
| mnaser | http://logs.openstack.org/08/591608/1/check/tempest-full/8fb702c/testr_results.html.gz | 17:47 |
| mnaser | recent run, dont see it there so not sure if that is where it should be | 17:47 |
| smcginnis | Looks like we may need a devstack change to pull it in for Cinder that is dependent on the fix for whatever is failing right now. | 17:48 |
| mnaser | well, i'd add it first and see if it fails at your side | 17:48 |
| mnaser | it could be an OSA specific thing, though i doubt it | 17:49 |
| smcginnis | Yeah, I would blame Cinder first until proven otherwise. | 17:49 |
| *** cmart has joined #openstack-cinder | 17:52 | |
| *** e0ne has joined #openstack-cinder | 17:56 | |
| *** mvkr has joined #openstack-cinder | 17:56 | |
| *** markstur has joined #openstack-cinder | 17:57 | |
| *** zul has quit IRC | 18:18 | |
| *** gkadam has quit IRC | 18:19 | |
| *** eharney has joined #openstack-cinder | 18:19 | |
| jungleboyj | smcginnis: eharney I have put my $0.02 in the patch. Really need Matan to address this and get a fix out there. | 18:40 |
| *** ganso has quit IRC | 18:41 | |
| eharney | jungleboyj: so what changes still remain there? | 18:52 |
| jungleboyj | Need a release note and I think they need to reconsider the logic for two config options eharney | 18:53 |
| eharney | sounds good | 18:53 |
| *** markstur has quit IRC | 18:57 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder master: Fixing security bug in scaleio driver https://review.openstack.org/592001 | 19:01 |
| hemna | anyone mess with gerrit searching much ? | 19:05 |
| smcginnis | hemna: A bit. What are you trying to do? | 19:07 |
| hemna | find all driver patches | 19:07 |
| smcginnis | mnaser: Doesn't look like any of the main projects get their tempest plugins pulled in to tempest-full. Just specific jobs. | 19:07 |
| hemna | file:"^*drivers*" doesn't seem to worky | 19:08 |
| smcginnis | hemna: Open ones? | 19:08 |
| hemna | yah | 19:08 |
| hemna | open or closed really | 19:08 |
| mnaser | smcginnis: yeah, that's what i noticed from codesearch.o.o | 19:08 |
| mnaser | hemna: https://review.openstack.org/#/q/project:openstack/cinder+file:%255Ecinder/volume/drivers.* seems to work? | 19:09 |
| smcginnis | hemna: Or https://review.openstack.org/#/q/project:openstack/cinder+file:drivers | 19:10 |
| hemna | hrmm | 19:10 |
| hemna | I guess *drivers* wasn't good enough | 19:10 |
| smcginnis | File instead of files for some reason. | 19:10 |
| *** markstur has joined #openstack-cinder | 19:11 | |
| hemna | smcginnis, thanks man | 19:12 |
| smcginnis | hemna: No problem. It's definitely not intuitive. | 19:12 |
| *** markstur has quit IRC | 19:14 | |
| *** markstur has joined #openstack-cinder | 19:14 | |
| *** openstackgerrit has quit IRC | 19:19 | |
| itlinux | hello guys, back to the IRC channel.. strange thing in Pike, and maybe it is a bug.. but here it is.. I tried every combo and set a default location but it never holds.. so now I have an az-1 and az2 and default to az-2 but for some reasons it still go to nova az which I do not have enabled and it's off so if I just do cinder create 1 , it looks like it is looping and sometimes it does create the volume the 1st time sometimes the 2time and | 19:21 |
| itlinux | sometimes the 3rd time.. kinda frustr.. on this .. any suggestions on really what I should be checking since I restarted all cinder services and the pacemaker just in case after every change. Thanks | 19:21 |
| jbernard | pvc: i don't think so, but ill add it to my list for stein | 19:30 |
| hemna | hmm, finding this info is going to be harder than I thought. | 19:33 |
| hemna | so I'm trying to find the total number of driver reviews per release vs non driver reviews | 19:33 |
| hemna | using branch alone doesn't cut it | 19:33 |
| hemna | since almost all new features and reviews were against master at one point. | 19:33 |
| hemna | :( | 19:34 |
| hemna | maybe reviews before the date when the stable/queens tag was added and after stable/ocata will give me the queens release ? | 19:38 |
| *** openstackgerrit has joined #openstack-cinder | 19:41 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: [2/2] Fix mock for sleep mocks in volume drivers tests https://review.openstack.org/537303 | 19:41 |
| smcginnis | hemna: queens and pike | 19:42 |
| hemna | yah | 19:42 |
| *** _alastor_ has joined #openstack-cinder | 19:42 | |
| smcginnis | There might be an easier way to query directly in git. | 19:42 |
| smcginnis | Sorry, need to drop for a minute. | 19:43 |
| hemna | man git pull from wifi on a plane at 35,000' takes a while | 19:45 |
| mnaser | latency is rough | 19:47 |
| *** e0ne has quit IRC | 20:03 | |
| hemna | ok that seems to be working, but the query only returns 500 results at a time. | 20:08 |
| hemna | boo | 20:09 |
| hemna | I really only want a total count, not the actual individual reviews | 20:09 |
| *** pocketprotector has quit IRC | 20:24 | |
| *** pocketprotector has joined #openstack-cinder | 20:26 | |
| *** markstur has quit IRC | 20:27 | |
| *** markstur has joined #openstack-cinder | 20:29 | |
| *** markstur has quit IRC | 20:33 | |
| *** markstur has joined #openstack-cinder | 20:35 | |
| *** markstur has quit IRC | 20:40 | |
| jungleboyj | hemna: What are you trying to get the data for? | 20:43 |
| *** markstur has joined #openstack-cinder | 20:47 | |
| *** markstur has quit IRC | 20:52 | |
| *** markstur has joined #openstack-cinder | 20:56 | |
| *** markstur has quit IRC | 20:56 | |
| *** masayukig has quit IRC | 21:12 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: ScaleIO: Disable volume creation without padding https://review.openstack.org/592001 | 21:24 |
| *** rcernin has joined #openstack-cinder | 21:29 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/589781 | 21:43 |
| *** KeithMnemonic has quit IRC | 21:44 | |
| *** erlon has quit IRC | 21:50 | |
| openstackgerrit | Merged openstack/cinder stable/ocata: HPE Lefthand: Fixed cloning operation https://review.openstack.org/591759 | 21:55 |
| openstackgerrit | Merged openstack/cinder master: Update document for metadata show API https://review.openstack.org/591263 | 21:55 |
| openstackgerrit | Merged openstack/cinder master: Default functional tests to use v3 API https://review.openstack.org/589955 | 21:55 |
| openstackgerrit | Merged openstack/cinder stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590464 | 21:55 |
| *** FracKen has left #openstack-cinder | 22:17 | |
| *** masayukig has joined #openstack-cinder | 22:18 | |
| *** itlinux has quit IRC | 22:22 | |
| *** cmart has quit IRC | 22:26 | |
| *** imacdonn has quit IRC | 22:38 | |
| *** imacdonn has joined #openstack-cinder | 22:38 | |
| *** erlon has joined #openstack-cinder | 22:52 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder stable/pike: Fix _attachment_reserve to not allow attaching an invalid status volume https://review.openstack.org/592212 | 22:53 |
| *** rcernin has quit IRC | 23:18 | |
| *** rcernin has joined #openstack-cinder | 23:19 | |
| openstackgerrit | Merged openstack/os-brick stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/586096 | 23:27 |
| *** dustins has quit IRC | 23:27 | |
| openstackgerrit | Merged openstack/python-cinderclient stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/589783 | 23:41 |
| openstackgerrit | Merged openstack/python-cinderclient stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/589784 | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!