*** 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!