*** baojg has quit IRC | 00:09 | |
*** baojg has joined #openstack-cinder | 00:10 | |
*** manoj_kumar_kata has quit IRC | 00:15 | |
*** manoj_kumar_kata has joined #openstack-cinder | 00:15 | |
*** devfaz has quit IRC | 00:35 | |
*** devfaz has joined #openstack-cinder | 00:35 | |
*** zhanglong has joined #openstack-cinder | 00:36 | |
*** rcernin_ has joined #openstack-cinder | 00:45 | |
*** rcernin has quit IRC | 00:47 | |
*** zhanglong has quit IRC | 01:31 | |
*** zhanglong has joined #openstack-cinder | 01:46 | |
*** ajitha has joined #openstack-cinder | 02:07 | |
*** baojg has quit IRC | 02:32 | |
*** baojg has joined #openstack-cinder | 02:33 | |
*** josephillips has quit IRC | 02:37 | |
*** zenkuro has quit IRC | 03:10 | |
*** rcernin_ has quit IRC | 03:19 | |
*** dunweifu has joined #openstack-cinder | 03:28 | |
*** rcernin_ has joined #openstack-cinder | 03:34 | |
*** dunweifu has quit IRC | 03:35 | |
*** psachin has joined #openstack-cinder | 03:35 | |
*** dunweifu has joined #openstack-cinder | 03:37 | |
*** dunweifu has quit IRC | 03:47 | |
*** whoami-rajat__ has joined #openstack-cinder | 03:55 | |
*** whoami-rajat has joined #openstack-cinder | 03:56 | |
*** zhanglong has quit IRC | 03:58 | |
*** udesale has joined #openstack-cinder | 04:16 | |
*** m75abrams has joined #openstack-cinder | 04:44 | |
*** ajitha has quit IRC | 04:47 | |
openstackgerrit | Venkata krishna Thumu proposed openstack/cinder master: [Storwize] Option to retain the auxiliary volume https://review.opendev.org/744683 | 04:49 |
---|---|---|
*** vishalmanchanda has joined #openstack-cinder | 04:55 | |
*** baojg has quit IRC | 05:12 | |
*** baojg has joined #openstack-cinder | 05:13 | |
openstackgerrit | Amar proposed openstack/cinder master: [Storwize] Support to primary volume Migration https://review.opendev.org/747425 | 05:17 |
*** bhagyashris|away is now known as bhagyashris | 05:27 | |
*** zhanglong has joined #openstack-cinder | 05:34 | |
*** kevinz has joined #openstack-cinder | 05:34 | |
*** dunweifu has joined #openstack-cinder | 05:55 | |
*** jopdorp has quit IRC | 06:01 | |
*** arne_wiebalck has quit IRC | 06:01 | |
*** simondodsley_ has joined #openstack-cinder | 06:02 | |
*** arne_wiebalck has joined #openstack-cinder | 06:02 | |
*** jopdorp has joined #openstack-cinder | 06:02 | |
*** simondodsley has quit IRC | 06:02 | |
*** simondodsley_ is now known as simondodsley | 06:02 | |
*** baojg has quit IRC | 06:02 | |
*** rajinir__ has joined #openstack-cinder | 06:02 | |
*** rajinir has quit IRC | 06:03 | |
*** rajinir__ is now known as rajinir | 06:03 | |
*** baojg has joined #openstack-cinder | 06:03 | |
*** dunweifu has quit IRC | 06:08 | |
*** benfelin has joined #openstack-cinder | 06:27 | |
*** rcernin_ has quit IRC | 06:51 | |
*** rcernin_ has joined #openstack-cinder | 06:54 | |
*** e0ne has joined #openstack-cinder | 07:26 | |
*** rcernin_ has quit IRC | 07:35 | |
*** brinzhang0 has joined #openstack-cinder | 07:35 | |
*** tosky has joined #openstack-cinder | 07:37 | |
*** brinzhang_ has quit IRC | 07:38 | |
*** baojg has quit IRC | 07:53 | |
*** baojg has joined #openstack-cinder | 07:53 | |
*** JaySee has quit IRC | 07:57 | |
*** baojg has quit IRC | 07:59 | |
*** martinkennelly has joined #openstack-cinder | 08:07 | |
*** rcernin_ has joined #openstack-cinder | 08:39 | |
*** JaySee has joined #openstack-cinder | 08:44 | |
*** rcernin_ has quit IRC | 08:46 | |
openstackgerrit | Sharath Kacham proposed openstack/cinder master: [storwize]:changes in create_group_from_src in SVC https://review.opendev.org/745293 | 08:50 |
openstackgerrit | Sharath Kacham proposed openstack/cinder master: [storwize]:changes in create_group_from_src in SVC https://review.opendev.org/745293 | 09:02 |
*** ajitha has joined #openstack-cinder | 09:11 | |
*** baojg has joined #openstack-cinder | 09:28 | |
*** zzzeek has quit IRC | 09:32 | |
*** zzzeek has joined #openstack-cinder | 09:33 | |
*** zzzeek has quit IRC | 09:48 | |
*** zzzeek has joined #openstack-cinder | 09:49 | |
*** baojg has quit IRC | 09:54 | |
*** zhanglong has quit IRC | 10:04 | |
*** zzzeek has quit IRC | 10:08 | |
*** zzzeek has joined #openstack-cinder | 10:10 | |
*** zenkuro has joined #openstack-cinder | 10:18 | |
zenkuro | hello, can anybody give a look at https://review.opendev.org/#/c/726906/ | 10:20 |
*** sapd1_x has joined #openstack-cinder | 10:21 | |
e0ne | zenkuro: hi. I'll take a look on it but this patch missed a deadline https://releases.openstack.org/victoria/schedule.html#cinder-new-driver-merge-deadline, so it could be merged only in next release | 10:24 |
zenkuro | e0ne: sad. But something is better then nothing. | 10:28 |
whoami-rajat__ | zenkuro: looking at the gate failures, you should run pep8 locally before updating new patchset | 10:39 |
*** rcernin_ has joined #openstack-cinder | 10:42 | |
*** lpetrut has joined #openstack-cinder | 10:47 | |
*** rcernin_ has quit IRC | 10:47 | |
*** brinzhang_ has joined #openstack-cinder | 10:48 | |
*** brinzhang0 has quit IRC | 10:51 | |
*** brinzhang has joined #openstack-cinder | 10:51 | |
*** brinzhang_ has quit IRC | 10:54 | |
*** zzzeek has quit IRC | 10:57 | |
*** zzzeek has joined #openstack-cinder | 11:00 | |
*** manoj_kumar_kata has quit IRC | 11:11 | |
*** manoj_kumar_kata has joined #openstack-cinder | 11:11 | |
*** elod has quit IRC | 11:15 | |
*** elod has joined #openstack-cinder | 11:16 | |
openstackgerrit | Jiri Suchomel proposed openstack/devstack-plugin-ceph master: zuul: Add devstack-plugin-ceph-compute-qcow2 job https://review.opendev.org/748212 | 11:42 |
*** Lucas_Gray has joined #openstack-cinder | 11:47 | |
*** Lucas_Gray has quit IRC | 11:57 | |
zenkuro | whoami-rajat__: there is a failure? it is somewhere on https://review.opendev.org/#/c/726906/ | 12:02 |
*** Lucas_Gray has joined #openstack-cinder | 12:05 | |
*** dave-mccowan has joined #openstack-cinder | 12:18 | |
*** udesale_ has joined #openstack-cinder | 12:18 | |
zenkuro | whoami-rajat: there is onlyt one fail cinder-plugin-ceph-tempest-mn-aa, and it looks like it is an issue with running devstack with ceph | 12:20 |
*** udesale has quit IRC | 12:20 | |
whoami-rajat | zenkuro, i was just giving a general suggestion as i saw a lot of previous gate fails due to pep8 | 12:23 |
zenkuro | whoami-rajat: actually this is interesting question, why sometimes it fails. is it normal? I run pep8 pylint and flake8 to check | 12:24 |
whoami-rajat | did you run pep8 with tox ? ``tox -epep8`` | 12:25 |
*** pcaruana has quit IRC | 12:30 | |
*** Lucas_Gray has quit IRC | 12:30 | |
rosmaita | eharney: geguileo: smcginnis: hemna: whoami-rajat: need some eyes on https://review.opendev.org/#/c/730376 (final os-brick patch for victoria) | 12:30 |
hemna | *click* | 12:30 |
rosmaita | e0ne can't do it because he is co-author | 12:30 |
e0ne | :( | 12:31 |
rosmaita | e0ne: you should give yourself a :D for your work on that patch | 12:31 |
smcginnis | rosmaita: We don't have testing for octopus in gate, do we? I wonder if we need periodic jobs that can test against different versions. | 12:31 |
smcginnis | Glad to see that passing though! | 12:31 |
hemna | ew | 12:31 |
hemna | custom code for a specific version of ceph :( | 12:32 |
* hemna has a sad | 12:32 | |
e0ne | rosmaita: fair enough | 12:32 |
e0ne | smcginnis: we don't have ceph backups in gates | 12:32 |
smcginnis | Is this only for the backup driver? | 12:32 |
rosmaita | smcginnis: we haven't been able to get it running, eharney had a patch up, but there's a problem because devstack installs logging from a package, and neutron needs a more recent version, and the stack fails | 12:32 |
eharney | i put up a test patch for octopus... it's kind of a mess so far | 12:32 |
eharney | right | 12:33 |
e0ne | smcginnis: honestly, I didn't check | 12:33 |
eharney | https://review.opendev.org/#/c/748245/ | 12:33 |
e0ne | smcginnis: I reproduced this issue with backups on a devstack + ubuntu 20.04 + ceph | 12:33 |
smcginnis | Cool. I'm good with it as is. Just seems like we should have some sort of test coverage if there is a significant difference between versions. | 12:33 |
hemna | do things fail if we always put [global] in the ini file? even with previous versions of ceph? | 12:35 |
*** pcaruana has joined #openstack-cinder | 12:35 | |
hemna | it would be nice if that worked, then we could avoid all this version checking and running ceph --version, etc | 12:35 |
rosmaita | the example configs in the ceph docs from mimic onward have a [global] section, couldn't find anything prior to that | 12:37 |
eharney | even the hammer docs reference [global] | 12:37 |
eharney | might be a good follow-up effort to test that and see if we can just use the same conf everywhere, not sure if there are subtle differences | 12:38 |
e0ne | eharney: +1 | 12:38 |
rosmaita | the way the gate has been, i'd prefer to do it as a follow-up | 12:38 |
eharney | yeah | 12:39 |
e0ne | I can test with nautilus now | 12:39 |
rosmaita | but you are right hemna, it would be good if we could simplify this | 12:39 |
*** Lucas_Gray has joined #openstack-cinder | 12:39 | |
eharney | isn't the gate testing it w/ nautilus? | 12:40 |
e0ne | eharney: it does, but without backups | 12:40 |
zenkuro | whoami-rajat: just checked --- 0 fails, all good. | 12:40 |
e0ne | do we have a list of supported ceph versions? | 12:40 |
whoami-rajat | cool | 12:41 |
eharney | i think not, but the driver intends to work with quite a lot of versions | 12:43 |
eharney | looking at the ceph release schedules, i think the answer is that we support all of them that are actively maintained | 12:46 |
e0ne | taking into account, tomorrow is a deadline to get os-brick released for victoria, I would like to get this patch merged "as is" | 12:47 |
e0ne | I mean with different config formats support | 12:47 |
rosmaita | e0ne: ++ | 12:47 |
e0ne | and we can fix everything else as a follow up patches | 12:48 |
hemna | hrmm | 12:48 |
hemna | as a bug fix later? | 12:48 |
e0ne | hemna: it looks like more like a code clean up | 12:48 |
hemna | how much time do we need to test always having [global] there? | 12:49 |
*** Reepicheep has quit IRC | 12:49 | |
e0ne | I'm testing with nautilus version now | 12:49 |
e0ne | I don't have other envs by the hand | 12:49 |
hemna | is that our minimum ceph version we support? | 12:49 |
smcginnis | We should be able to just add [default] and have the gate test it. | 12:50 |
rosmaita | i don't think we have a published min version, we should probably also look into that | 12:50 |
eharney | nautilus is not the minimum that we support | 12:50 |
hemna | yah | 12:50 |
smcginnis | If that works fine, I think it would be better to just skip this patch and do a simple update that always adds the default section header. | 12:50 |
hemna | smcginnis +1 | 12:50 |
smcginnis | We're getting close to the deadline for os-brick, but it's not going to take that much time. And more risk in adding all this conditional logic at the last minute anyway. | 12:51 |
hemna | yah I figure if we have a day, test it and then we can merge it if we have to tomorrow, but with the idea of a cleaner patch would be best. | 12:52 |
*** zhanglong has joined #openstack-cinder | 12:53 | |
rosmaita | i think we've got pretty good tests around the conditional logic ... my concern is that we're not going to be able to test with all relevant ceph versions | 12:55 |
rosmaita | the devstack-ceph-plugin only updated to nautilus very recently | 12:55 |
rosmaita | which makes me think there are a lot of luminous and mimic installations out there | 12:56 |
hemna | the devstack-ceph-plugin hasn't gotten a lot of attention in ages though | 12:56 |
eharney | ? | 12:57 |
hemna | in terms of upgrading the version of ceph being installed | 12:57 |
e0ne | confirmed: [global] work for nautilus too | 12:58 |
hemna | sweet | 12:58 |
rosmaita | so just h, i, j, k, l, m left to check | 12:59 |
e0ne | :) | 13:00 |
e0ne | it will be a much easier, if devstack-ceph-plugin can setup a needed version | 13:00 |
rosmaita | agreed | 13:00 |
hemna | yes | 13:00 |
hemna | I wanted to redo the plugin and use ceph-ansible | 13:01 |
hemna | in my 'spare time' | 13:01 |
hemna | heh | 13:01 |
e0ne | hemna: it's a good time to get a vacation for it ;) | 13:01 |
*** Reepicheep has joined #openstack-cinder | 13:02 | |
rosmaita | well, my opinion is that this issue has only been reported for octopus, so i don't want to make a change that could break earlier versions without more testing | 13:03 |
rosmaita | so i think we go with this patch and follow up once we know it's safe to simplify the code | 13:04 |
hemna | I'm 50/50 on it | 13:04 |
hemna | I think as a result of this we need to specify a minimum ceph version we support and test for that in the driver at some point. | 13:05 |
rosmaita | hemna: ++ | 13:07 |
*** johanssone has quit IRC | 13:11 | |
*** m75abrams has quit IRC | 13:13 | |
*** zhanglong has quit IRC | 13:15 | |
*** m75abrams has joined #openstack-cinder | 13:15 | |
*** zhanglong has joined #openstack-cinder | 13:16 | |
eharney | hemna: i think for anything we've added to the ceph driver, we ensure it doesn't break compat with older versions of ceph -- so like i said, i think the aim is to support any versions of ceph that ceph supports | 13:37 |
hemna | I suppose that's fine, I just don't care for the hacky nature of that patch that tests for versions and does special stuff | 13:38 |
*** zhanglong has quit IRC | 13:45 | |
*** enriquetaso has joined #openstack-cinder | 13:47 | |
*** kaisers has joined #openstack-cinder | 13:50 | |
rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:58 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^ | 13:58 |
*** Liang__ has joined #openstack-cinder | 13:58 | |
*** Liang__ is now known as LiangFang | 13:59 | |
*** m75abrams has quit IRC | 14:01 | |
*** sfernand has joined #openstack-cinder | 14:03 | |
*** Lucas_Gray has quit IRC | 14:05 | |
*** LiangFang has quit IRC | 14:12 | |
openstackgerrit | Merged openstack/os-brick master: Fix for rbd connector to work with ceph octopus https://review.opendev.org/730376 | 14:14 |
*** Liang__ has joined #openstack-cinder | 14:14 | |
*** zenkuro has quit IRC | 14:30 | |
*** Lucas_Gray has joined #openstack-cinder | 14:32 | |
*** johanssone has joined #openstack-cinder | 14:36 | |
*** rcernin_ has joined #openstack-cinder | 14:44 | |
*** rcernin_ has quit IRC | 14:49 | |
rosmaita | \o/ | 14:51 |
*** udesale_ has quit IRC | 14:59 | |
*** Liang__ has quit IRC | 15:29 | |
*** tosky has quit IRC | 15:31 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: Add mypy tox env https://review.opendev.org/733620 | 15:37 |
openstackgerrit | Eric Harney proposed openstack/cinder master: mypy: annotate volume manager https://review.opendev.org/733621 | 15:37 |
openstackgerrit | Eric Harney proposed openstack/cinder master: mypy: annotate api.py https://review.opendev.org/733622 | 15:37 |
*** zenkuro22 has joined #openstack-cinder | 15:43 | |
zenkuro22 | e0ne: or you are talking that this patch will not be merged at all in the next several month? | 15:44 |
e0ne | zenkuro22: new drivers could be merged not sooner than Wallaby cycle begins | 15:45 |
zenkuro22 | but can they be merged in code tree? | 15:45 |
zenkuro22 | e0ne: but can they be merged in code tree? | 15:45 |
rosmaita | zenkuro22: only after the stable/victoria branch is cut | 15:45 |
e0ne | zenkuro22: it's a week of Sep 21-Sep 25 | 15:47 |
zenkuro22 | so the branches is not separated yet, I know it is against the rules, but since code is ok and it is just a driver with basic features, like there is nothing that affects cinder itself been introduced | 15:48 |
zenkuro22 | may be it is possible to squise it in? | 15:48 |
zenkuro22 | I promise to support it on daily bases if ANY issue happen till 25 of sept | 15:49 |
openstackgerrit | Merged openstack/python-cinderclient master: Add support for Cinder API mv3.61 https://review.opendev.org/742994 | 15:50 |
zenkuro22 | + this code was previously cheked by Lucio Seki, Eric Harney, Sean McGinnis and Ive fixed coding style issues pointed out by them. | 15:53 |
*** lpetrut has quit IRC | 15:53 | |
rosmaita | smcginnis: when you have some time, can you look at the release patch for brick? not sure i have the branch point correct: https://review.opendev.org/749544 | 15:54 |
smcginnis | zenkuro22: It should be good to go once wallaby opens up. It is too late for victoria. | 15:55 |
smcginnis | rosmaita: Will do. Do you use new-release for those? There's a --stable-branch switch for that. | 15:55 |
smcginnis | rosmaita: It does look correct through. | 15:56 |
smcginnis | *though | 15:56 |
rosmaita | smcginnis: i will have to investigate new-release ... i just do it by hand the old fashioned way | 15:56 |
openstackgerrit | zhoulinhui proposed openstack/python-cinderclient master: Use importlib to take place of imp module https://review.opendev.org/748857 | 15:59 |
*** dsariel has quit IRC | 16:28 | |
*** enriquetaso has quit IRC | 16:40 | |
openstackgerrit | OpenStack Release Bot proposed openstack/os-brick stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749555 | 16:43 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-brick stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749556 | 16:43 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-brick master: Update master for stable/victoria https://review.opendev.org/749557 | 16:43 |
openstackgerrit | OpenStack Release Bot proposed openstack/os-brick master: Add Python3 wallaby unit tests https://review.opendev.org/749558 | 16:43 |
*** Lucas_Gray has quit IRC | 16:50 | |
*** enriquetaso has joined #openstack-cinder | 16:56 | |
*** rcernin_ has joined #openstack-cinder | 17:04 | |
*** rcernin_ has quit IRC | 17:08 | |
*** psachin has quit IRC | 17:09 | |
*** manoj_kumar_kata has quit IRC | 17:12 | |
*** manoj_kumar_kata has joined #openstack-cinder | 17:12 | |
*** manoj_kumar_kata has quit IRC | 17:14 | |
*** manoj_kumar_kata has joined #openstack-cinder | 17:14 | |
*** josephillips has joined #openstack-cinder | 17:15 | |
openstackgerrit | Merged openstack/os-brick master: Update master for stable/victoria https://review.opendev.org/749557 | 17:17 |
openstackgerrit | Merged openstack/os-brick stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/749555 | 17:17 |
openstackgerrit | Merged openstack/os-brick stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/749556 | 17:28 |
*** manoj_kumar_kata has quit IRC | 17:31 | |
*** manoj_kumar_kata has joined #openstack-cinder | 17:32 | |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: DNM Testing Manila gates https://review.opendev.org/747154 | 17:55 |
*** e0ne has quit IRC | 18:29 | |
*** vishalmanchanda has quit IRC | 18:43 | |
openstackgerrit | Fernando Ferraz proposed openstack/cinder master: NetApp SolidFire: Add storage assisted migration support https://review.opendev.org/746941 | 18:58 |
openstackgerrit | Fernando Ferraz proposed openstack/cinder master: NetApp SolidFire: Add storage assisted migration support https://review.opendev.org/746941 | 19:01 |
*** zzzeek has quit IRC | 19:17 | |
*** zzzeek has joined #openstack-cinder | 19:18 | |
*** manoj_kumar_kata has quit IRC | 19:21 | |
*** zzzeek has quit IRC | 19:25 | |
*** zzzeek has joined #openstack-cinder | 19:27 | |
*** e0ne has joined #openstack-cinder | 19:51 | |
*** zzzeek has quit IRC | 19:53 | |
*** zzzeek has joined #openstack-cinder | 19:56 | |
*** martinkennelly has quit IRC | 20:32 | |
*** Lucas_Gray has joined #openstack-cinder | 20:42 | |
*** e0ne has quit IRC | 20:44 | |
*** zzzeek has quit IRC | 20:50 | |
*** zzzeek has joined #openstack-cinder | 20:51 | |
*** sapd1_x has quit IRC | 21:04 | |
*** rcernin_ has joined #openstack-cinder | 21:05 | |
*** rcernin_ has quit IRC | 21:12 | |
*** sapd1_x has joined #openstack-cinder | 21:17 | |
openstackgerrit | Merged openstack/os-brick stable/train: Leverage the iSCSI mpath to get the WWN https://review.opendev.org/749334 | 21:41 |
openstackgerrit | Merged openstack/cinder master: RBD: remove rbd_keyring_conf option https://review.opendev.org/747494 | 21:43 |
*** ajitha has quit IRC | 21:50 | |
zenkuro22 | smcginnis: hi, is there specific format for re running ci? | 22:02 |
zenkuro22 | smcginnis: open-e triggers on run open-e-joviandss-ci | 22:03 |
*** rcernin_ has joined #openstack-cinder | 22:15 | |
smcginnis | zenkuro22: That should be fixed then. | 22:35 |
zenkuro22 | smcginnis: many thanks for review! I will fix it. | 22:35 |
smcginnis | zenkuro22: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_trigger_my_CI_to_rerun_on_gerrit_comments.3F | 22:35 |
smcginnis | zenkuro22: Looks pretty good overall. So I think once these little issues are addressed, things should be ready to go for Wallaby. | 22:35 |
zenkuro22 | smcginnis: thanks again \>_</ | 22:37 |
*** enriquetaso has quit IRC | 22:58 | |
*** zenkuro22 has quit IRC | 23:00 | |
*** rcernin_ has quit IRC | 23:04 | |
*** rcernin has joined #openstack-cinder | 23:04 | |
*** Lucas_Gray has quit IRC | 23:07 | |
*** lifeless has quit IRC | 23:29 | |
*** irclogbot_2 has quit IRC | 23:29 | |
*** irclogbot_3 has joined #openstack-cinder | 23:32 | |
*** whoami-rajat has quit IRC | 23:35 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!