| *** n-saito has joined #openstack-cinder | 00:24 | |
| *** Liang__ has joined #openstack-cinder | 01:09 | |
| openstackgerrit | melissaml proposed openstack/os-brick master: Remove translation sections from setup.cfg https://review.opendev.org/728321 | 01:29 |
|---|---|---|
| *** brinzhang_ has joined #openstack-cinder | 01:31 | |
| *** brinzhang has quit IRC | 01:34 | |
| *** sapd__x has joined #openstack-cinder | 02:01 | |
| *** ruffian_sheep has joined #openstack-cinder | 02:15 | |
| ruffian_sheep | whoami-rajat : hi,If you have time, please help to see if this feature submission meets the requirements . https://review.opendev.org/#/c/727711/ | 02:16 |
| *** Xuchu has joined #openstack-cinder | 02:22 | |
| ruffian_sheep | whoami-rajat : thxs. ;) | 02:28 |
| *** josephillips has quit IRC | 02:36 | |
| *** ruffian_sheep has quit IRC | 02:54 | |
| *** ociuhandu has joined #openstack-cinder | 03:05 | |
| *** ociuhandu has quit IRC | 03:09 | |
| *** rosmaita has left #openstack-cinder | 03:16 | |
| *** Xuchu has quit IRC | 03:20 | |
| *** kevinz has quit IRC | 03:37 | |
| *** psachin has joined #openstack-cinder | 03:39 | |
| *** kevinz has joined #openstack-cinder | 03:46 | |
| *** ykarel|away is now known as ykarel | 03:54 | |
| *** zzzeek has quit IRC | 04:23 | |
| *** zzzeek has joined #openstack-cinder | 04:24 | |
| *** Xuchu has joined #openstack-cinder | 04:26 | |
| *** ociuhandu has joined #openstack-cinder | 04:33 | |
| *** ociuhandu has quit IRC | 04:43 | |
| *** ociuhandu has joined #openstack-cinder | 04:44 | |
| *** ociuhandu has quit IRC | 04:50 | |
| *** vishalmanchanda has joined #openstack-cinder | 04:55 | |
| *** udesale has joined #openstack-cinder | 05:25 | |
| *** ociuhandu has joined #openstack-cinder | 05:52 | |
| *** dpawlik has joined #openstack-cinder | 05:58 | |
| *** ociuhandu has quit IRC | 05:59 | |
| *** ociuhandu has joined #openstack-cinder | 06:45 | |
| *** ykarel is now known as ykarel|afk | 06:49 | |
| *** tosky has joined #openstack-cinder | 07:35 | |
| *** ociuhandu has quit IRC | 07:51 | |
| *** ykarel|afk is now known as ykarel | 07:57 | |
| *** e0ne has joined #openstack-cinder | 07:57 | |
| *** e0ne has quit IRC | 08:00 | |
| *** tkajinam has quit IRC | 08:14 | |
| *** ykarel is now known as ykarel|lunch | 08:27 | |
| *** e0ne has joined #openstack-cinder | 08:37 | |
| *** martinkennelly has joined #openstack-cinder | 08:48 | |
| *** ociuhandu has joined #openstack-cinder | 08:49 | |
| *** ykarel|lunch is now known as ykarel | 09:07 | |
| *** brinzhang_ has quit IRC | 09:24 | |
| *** brinzhang has joined #openstack-cinder | 09:30 | |
| *** ircuser-1 has quit IRC | 09:32 | |
| *** n-saito has quit IRC | 09:41 | |
| *** brinzhang has quit IRC | 09:47 | |
| *** Xuchu has quit IRC | 09:51 | |
| whoami-rajat | gmann, sadly the json response is just for mv 3.0 and doesn't include responses for mv > 3.0. in this case user_id is a valid response in 3.41 and shouldn't be removed | 09:52 |
| *** Liang__ has quit IRC | 09:54 | |
| *** dpawlik has quit IRC | 10:27 | |
| *** dpawlik has joined #openstack-cinder | 10:28 | |
| *** breizhkoala has joined #openstack-cinder | 10:29 | |
| *** breizhkoala has quit IRC | 10:37 | |
| *** breizhkoala has joined #openstack-cinder | 10:37 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder master: Move get_volume_stats impl to the base volume driver https://review.opendev.org/656888 | 10:40 |
| *** ociuhandu has quit IRC | 11:01 | |
| *** ociuhandu has joined #openstack-cinder | 11:02 | |
| *** ociuhandu has quit IRC | 11:02 | |
| *** ociuhandu has joined #openstack-cinder | 11:03 | |
| *** ociuhandu has quit IRC | 11:13 | |
| *** ociuhandu has joined #openstack-cinder | 11:14 | |
| *** ociuhandu has quit IRC | 11:14 | |
| *** ociuhandu has joined #openstack-cinder | 11:15 | |
| *** ociuhandu has quit IRC | 11:25 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext master: Fix hacking min version to 3.0.1 https://review.opendev.org/728283 | 11:29 |
| *** threestrands has quit IRC | 11:46 | |
| *** andrebeltrami has joined #openstack-cinder | 11:56 | |
| *** ociuhandu has joined #openstack-cinder | 12:05 | |
| *** udesale_ has joined #openstack-cinder | 12:10 | |
| *** ociuhandu has quit IRC | 12:13 | |
| *** udesale has quit IRC | 12:13 | |
| walshh_ | is there anyone available to look at https://review.opendev.org/#/c/722667/ it has +2 and clean CI. Thank you | 12:26 |
| *** rosmaita has joined #openstack-cinder | 12:28 | |
| *** rosmaita has quit IRC | 12:28 | |
| *** rosmaita has joined #openstack-cinder | 12:37 | |
| openstackgerrit | Andreas Jaeger proposed openstack/cinder master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728272 | 12:37 |
| *** ociuhandu has joined #openstack-cinder | 12:39 | |
| *** enriquetaso has joined #openstack-cinder | 12:40 | |
| *** ykarel is now known as ykarel|afk | 12:56 | |
| *** sapd__x has quit IRC | 13:04 | |
| whoami-rajat | walshh_, done | 13:11 |
| gmann | whoami-rajat: but do we add user_id in lsit response. i did not find in code - https://github.com/openstack/cinder/blob/6b5d1f45e9f1833ddf8c994c4203dfef64a51591/cinder/api/views/snapshots.py#L38 | 13:12 |
| whoami-rajat | gmann, it's being added in v3 views, https://github.com/openstack/cinder/blob/6b5d1f45e9f1833ddf8c994c4203dfef64a51591/cinder/api/v3/views/snapshots.py#L34 | 13:14 |
| gmann | whoami-rajat: yeah that is for detail view used by GET detail, POST, PUT snapshot and summary view used by GET (no detail) snapshot does not have user_id | 13:15 |
| gmann | whoami-rajat: https://github.com/openstack/cinder/blob/6b5d1f45e9f1833ddf8c994c4203dfef64a51591/cinder/api/v3/snapshots.py#L122 | 13:16 |
| gmann | whoami-rajat: if you see original patch for 3.45, it was never added for sumamry_list but doc got updated - https://review.opendev.org/#/c/472545/ | 13:16 |
| whoami-rajat | gmann, oh yeah right. it's only for detailed response | 13:19 |
| whoami-rajat | i think we can remove user_id from summary | 13:20 |
| gmann | yeah, api-ref mismatch with what code return | 13:20 |
| *** throne82_ has joined #openstack-cinder | 13:20 | |
| gmann | also on microversion sample files, it will be easy to do like re-run or add new test for microversion change and maintain their sample file in versioned directory. and api-ref can show latest microversion sample example (or all).Nova doing those for all applicable microversion | 13:21 |
| gmann | whoami-rajat: that can test the response in more reliable way and can be seen very clearly when microversion happen | 13:22 |
| whoami-rajat | gmann, yeah i was going to finish it but due to other priorities i couldn't continue it. :( | 13:29 |
| gmann | whoami-rajat: +1 | 13:29 |
| whoami-rajat | it is a good thing to be in and if anyone would like to pursue it, i can guide through the existing code | 13:31 |
| gmann | +1 | 13:32 |
| gmann | whoami-rajat: rosmaita jungleboyj either of you can approve this. gate is passing (doc job is green in in-progress zuul run) now as req revert for openstackdocstheme is merged. - https://review.opendev.org/#/c/728281/1 | 13:33 |
| gmann | otherwise we might hit pep8 + doc failure again. | 13:33 |
| whoami-rajat | gmann, done. thanks! | 13:37 |
| rosmaita | gmann: that duplicates https://review.opendev.org/#/c/727654/ -- see my comment on that one | 13:38 |
| *** e0ne has quit IRC | 13:40 | |
| gmann | rosmaita: ah thanks for pointing. we need to remove hacking from lower_constraint, replied | 13:41 |
| gmann | rosmaita: how you would like to go fixing that in 728281/ or 727654/ , either is fine for me | 13:42 |
| rosmaita | gmann: thanks, will look -- i'm still not 100% clear on what lower-constraints is for | 13:42 |
| rosmaita | gmann: let's do your patch, the author of the other one hasn't replied since wednesday, and you're working on this now | 13:43 |
| *** e0ne has joined #openstack-cinder | 13:44 | |
| gmann | rosmaita: because hacking is blacklist req, so requirement job would not verify the lowe constraint of hacking so it is much possible to go out of date like this case. let's bump for now and I will check later with req team if removing is good idea or not. | 13:50 |
| gmann | updating | 13:50 |
| rosmaita | ok, sounds good to me | 13:50 |
| *** kevinz has quit IRC | 13:50 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/cinder master: Fix hacking min version to 3.0.1 https://review.opendev.org/728281 | 13:52 |
| gmann | rosmaita: whoami-rajat done ^^ | 13:53 |
| *** pcaruana has quit IRC | 13:58 | |
| *** breizhkoala has quit IRC | 13:59 | |
| rosmaita | gmann: thanks for your note to wenping on that other reveiw | 14:00 |
| gmann | rosmaita: i did cinder late i think as it was already capping pycodestyle but new hacking version broke otherwsie it could be on Tuesday similar to other projects which could have avoid duplicate. | 14:01 |
| *** ykarel|afk is now known as ykarel | 14:10 | |
| *** pcaruana has joined #openstack-cinder | 14:11 | |
| *** kevinz has joined #openstack-cinder | 14:28 | |
| *** Xuchu has joined #openstack-cinder | 15:01 | |
| openstackgerrit | Andreas Jaeger proposed openstack/cinder master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728272 | 15:03 |
| e0ne | do we have any spec how online db sync supposed to be working? | 15:08 |
| *** udesale_ has quit IRC | 15:10 | |
| *** ykarel is now known as ykarel|away | 15:16 | |
| smcginnis | e0ne: https://specs.openstack.org/openstack/cinder-specs/specs/mitaka/online-schema-upgrades.html | 15:17 |
| e0ne | smcginnis: thanks a lot! | 15:18 |
| *** Xuchu has quit IRC | 15:31 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder stable/stein: Fix online data migrations https://review.opendev.org/728476 | 15:40 |
| *** dpawlik has quit IRC | 15:41 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder stable/rocky: Fix online data migrations https://review.opendev.org/728478 | 15:45 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder stable/rocky: Fix online data migrations https://review.opendev.org/728478 | 15:46 |
| *** ociuhandu has quit IRC | 15:47 | |
| openstackgerrit | Merged openstack/cinder master: PowerMax Driver - Live migrate remove rep vol from sg https://review.opendev.org/722667 | 16:00 |
| openstackgerrit | Simon Dodsley proposed openstack/cinder master: [WIP] Add QoS Suport for Pure Storage https://review.opendev.org/727267 | 16:00 |
| *** jmlowe has quit IRC | 16:07 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-cinderclient master: Fix hacking min version to 3.0.1 https://review.opendev.org/728284 | 16:10 |
| openstackgerrit | Rajini Karthik proposed openstack/cinder master: Dell EMC Sc: Add support for whitelisting fault domains https://review.opendev.org/727892 | 16:10 |
| *** gnufied has joined #openstack-cinder | 16:18 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-brick master: Fix hacking min version to 3.0.1 https://review.opendev.org/728282 | 16:23 |
| *** kberger_ has joined #openstack-cinder | 16:31 | |
| *** KeithMnemonic has quit IRC | 16:31 | |
| *** kberger_ has quit IRC | 16:32 | |
| *** kberger_ has joined #openstack-cinder | 16:32 | |
| *** ociuhandu has joined #openstack-cinder | 16:36 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder stable/stein: Fix online data migrations https://review.opendev.org/728476 | 16:37 |
| *** ociuhandu has quit IRC | 16:41 | |
| *** ociuhandu has joined #openstack-cinder | 16:42 | |
| *** psachin has quit IRC | 17:02 | |
| *** ociuhandu has quit IRC | 17:30 | |
| *** ociuhandu has joined #openstack-cinder | 17:32 | |
| openstackgerrit | Andreas Jaeger proposed openstack/cinder master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728272 | 17:41 |
| *** ociuhandu has quit IRC | 17:42 | |
| *** ociuhandu has joined #openstack-cinder | 17:44 | |
| *** gnufied has quit IRC | 17:47 | |
| *** gnufied has joined #openstack-cinder | 17:52 | |
| *** ociuhandu has quit IRC | 17:54 | |
| *** ociuhandu has joined #openstack-cinder | 17:54 | |
| *** ociuhandu has quit IRC | 17:55 | |
| *** ociuhandu has joined #openstack-cinder | 17:55 | |
| smcginnis | Testing, but we may need https://review.opendev.org/#/c/725370/ to land to get that openstackdocstheme update to work. | 18:00 |
| *** ociuhandu has quit IRC | 18:05 | |
| *** ociuhandu has joined #openstack-cinder | 18:06 | |
| *** ociuhandu has quit IRC | 18:06 | |
| *** ociuhandu has joined #openstack-cinder | 18:06 | |
| openstackgerrit | Andreas Jaeger proposed openstack/python-cinderclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728541 | 18:14 |
| openstackgerrit | Andreas Jaeger proposed openstack/python-cinderclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728541 | 18:15 |
| *** ociuhandu has quit IRC | 18:16 | |
| *** ociuhandu has joined #openstack-cinder | 18:17 | |
| *** ociuhandu has quit IRC | 18:57 | |
| *** ociuhandu has joined #openstack-cinder | 18:58 | |
| *** ociuhandu has quit IRC | 18:59 | |
| *** ociuhandu has joined #openstack-cinder | 18:59 | |
| openstackgerrit | Sofia Enriquez proposed openstack/os-brick master: Retype from non encrypted to encrypted volume in ceph backend fails https://review.opendev.org/726467 | 19:05 |
| *** ociuhandu has quit IRC | 19:10 | |
| *** abhishekk is now known as abhishekk|offlin | 19:10 | |
| *** ociuhandu has joined #openstack-cinder | 19:12 | |
| *** ociuhandu has quit IRC | 19:17 | |
| *** kberger_ has quit IRC | 19:48 | |
| *** KeithMnemonic has joined #openstack-cinder | 19:48 | |
| *** e0ne has quit IRC | 20:42 | |
| *** ociuhandu has joined #openstack-cinder | 20:44 | |
| *** ociuhandu has quit IRC | 20:49 | |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/cinder master: Add log if resource_filters json does not exist https://review.opendev.org/724722 | 20:59 |
| *** martinkennelly has quit IRC | 21:21 | |
| *** ociuhandu has joined #openstack-cinder | 21:32 | |
| *** ociuhandu has quit IRC | 21:40 | |
| *** KeithMnemonic has quit IRC | 21:47 | |
| *** enriquetaso has quit IRC | 21:48 | |
| openstackgerrit | Merged openstack/cinder master: Fix hacking min version to 3.0.1 https://review.opendev.org/728281 | 23:07 |
| *** tosky has quit IRC | 23:39 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!