| *** spotz is now known as Guest2040 | 01:32 | |
| *** spotz_ is now known as spotz | 01:32 | |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Add Target Port Assignment for VSP Driver https://review.opendev.org/c/openstack/cinder/+/828060 | 01:58 |
|---|---|---|
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support AIX as host OS type https://review.opendev.org/c/openstack/cinder/+/828061 | 02:00 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Dropping explicit unicode literal https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/799564 | 03:34 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Dropping explicit unicode literal https://review.opendev.org/c/openstack/cinder/+/799631 | 03:34 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] Manage host attachment using portsets https://review.opendev.org/c/openstack/cinder/+/817351 | 06:31 |
| *** _pewp__ is now known as _pewp_ | 07:37 | |
| *** hemna3 is now known as hemna | 07:37 | |
| opendevreview | yuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key https://review.opendev.org/c/openstack/cinder/+/828085 | 08:00 |
| opendevreview | yuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key https://review.opendev.org/c/openstack/cinder/+/828085 | 08:09 |
| opendevreview | yuval proposed openstack/os-brick master: Followup - Reuse get_host_nqn code https://review.opendev.org/c/openstack/os-brick/+/828087 | 08:15 |
| opendevreview | yuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key https://review.opendev.org/c/openstack/cinder/+/828085 | 08:16 |
| opendevreview | yuval proposed openstack/os-brick master: Followup - Reuse get_host_nqn code https://review.opendev.org/c/openstack/os-brick/+/828087 | 08:26 |
| opendevreview | yuval proposed openstack/os-brick master: Followup - Reuse get_host_nqn code https://review.opendev.org/c/openstack/os-brick/+/828087 | 08:27 |
| yuval | GM - anyone know how I can re-trigger zuul? | 08:32 |
| yuval | had some dependency issue which I solved | 08:33 |
| opendevreview | yuval proposed openstack/cinder master: Followup - remove "hostnqn" extra key https://review.opendev.org/c/openstack/cinder/+/828085 | 10:40 |
| yuval | there is an issue with zuul - "Depends-On" feature | 10:51 |
| yuval | if os-brick depends on a patch from cinder and cinder depends on the same patch from os-brick | 10:52 |
| yuval | it will fail for circular dependency - but its a reasonable request - set cinder at branch "a" is-brick brach "b" | 10:53 |
| *** dviroel|out is now known as dviroel|river | 10:58 | |
| *** dviroel|river is now known as dviroel|rover | 10:58 | |
| *** dviroel|rover is now known as dviroel|ruck | 11:01 | |
| bingyanh_ | hi reviewers it need other +2 can help to take a look: https://review.opendev.org/c/openstack/cinder/+/819778 | 11:12 |
| smcginnis | yuval: I don't think a circular dependency is a reasonable request. One needs to merge before the other, so that should not be allowed. | 11:59 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVF] RevertToSnapshot for rep-volumes in group https://review.opendev.org/c/openstack/cinder/+/823015 | 12:09 |
| *** dviroel|ruck is now known as dviroel|ruck|afk | 12:23 | |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] RevertToSnapshot for rep-volumes in group https://review.opendev.org/c/openstack/cinder/+/823015 | 13:03 |
| opendevreview | Mounika Sreeram proposed openstack/cinder master: [SVf] update rccg name property to metadata https://review.opendev.org/c/openstack/cinder/+/810917 | 13:05 |
| opendevreview | Iago Filipe proposed openstack/cinder master: Improving delete volumes exception log https://review.opendev.org/c/openstack/cinder/+/828150 | 13:17 |
| raghavendrat | Hi reviewers, it would be great if someone can have a look at: | 13:22 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/786054 | 13:22 |
| raghavendrat | Move nimble driver code to hpe folder | 13:23 |
| raghavendrat | Zuul and CI have passed. | 13:23 |
| raghavendrat | Recently below patches were reviewed by me: | 13:23 |
| raghavendrat | 1. Lightbits LightOS driver https://review.opendev.org/c/openstack/cinder/+/821602 | 13:23 |
| raghavendrat | 2. [SVf] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 13:23 |
| *** dviroel|ruck|afk is now known as dviroel|ruck | 14:03 | |
| yuval | raghavendart: added my review | 14:03 |
| yuval | smcginnis: If i am changing a key in the connection_properties dictionary located in os-brick. this change need both os-brick and cinder for a patch which is dependent on each other | 14:07 |
| yuval | recently reviewed patches: | 14:18 |
| yuval | https://review.opendev.org/c/openstack/cinder/+/828150 | 14:18 |
| yuval | https://review.opendev.org/c/openstack/cinder/+/786054 | 14:19 |
| yuval | need review on: | 14:19 |
| yuval | https://review.opendev.org/c/openstack/cinder/+/828085 | 14:19 |
| yuval | https://review.opendev.org/c/openstack/os-brick/+/828087 | 14:19 |
| yuval | is that 2 review per patch or per request? I mean if I have 2 patches I should review at least 4? :o | 14:20 |
| rosmaita | 4 is better than 2, but 2 is a definite start! | 14:25 |
| *** Guest1862 is now known as dansmith | 14:47 | |
| yuval | https://review.opendev.org/c/openstack/cinder/+/828060 | 14:59 |
| yuval | https://review.opendev.org/c/openstack/cinder/+/800703 | 14:59 |
| *** dviroel|ruck is now known as dviroel|ruck|lunch | 15:13 | |
| *** dansmith is now known as Guest2102 | 15:29 | |
| *** Guest2102 is now known as dansmith | 15:37 | |
| *** dviroel|ruck|lunch is now known as dviroel|ruck | 16:09 | |
| *** dansmith is now known as Guest2108 | 16:45 | |
| *** Guest2108 is now known as dansmith | 16:46 | |
| *** _pewp__ is now known as _pewp_ | 19:19 | |
| *** hemna3 is now known as hemna | 19:20 | |
| *** noonedeadpunk_ is now known as noonedeadpunk | 19:34 | |
| opendevreview | Merged openstack/cinder master: Remove various unused code https://review.opendev.org/c/openstack/cinder/+/800703 | 19:52 |
| opendevreview | Iago Filipe proposed openstack/cinder master: Improving delete volumes exception log https://review.opendev.org/c/openstack/cinder/+/828150 | 19:54 |
| *** dviroel|ruck is now known as dviroel|out | 22:41 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!