*** tosky has quit IRC | 00:36 | |
*** eharney has quit IRC | 01:00 | |
*** eharney has joined #openstack-manila | 01:18 | |
*** vishalmanchanda has joined #openstack-manila | 04:23 | |
*** carloss has quit IRC | 08:14 | |
*** carloss has joined #openstack-manila | 08:14 | |
*** tbarron has quit IRC | 08:16 | |
*** johanssone has quit IRC | 08:19 | |
*** tbarron has joined #openstack-manila | 08:22 | |
*** johanssone has joined #openstack-manila | 08:23 | |
*** tosky has joined #openstack-manila | 08:38 | |
*** vhari has quit IRC | 08:43 | |
openstackgerrit | wushiming proposed openstack/manila-tempest-plugin master: Update TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/manila-tempest-plugin/+/766917 | 10:00 |
---|---|---|
*** lseki has joined #openstack-manila | 10:10 | |
*** vhari has joined #openstack-manila | 13:25 | |
*** _erlon_ has joined #openstack-manila | 14:01 | |
*** baojg has quit IRC | 15:34 | |
*** lbragstad_ has joined #openstack-manila | 16:38 | |
*** lbragstad has quit IRC | 16:41 | |
*** lbragstad__ has joined #openstack-manila | 17:13 | |
*** lbragstad_ has quit IRC | 17:16 | |
*** johanssone has quit IRC | 17:45 | |
*** johanssone has joined #openstack-manila | 17:47 | |
*** eharney has quit IRC | 18:19 | |
*** lbragstad__ has quit IRC | 18:24 | |
*** eharney has joined #openstack-manila | 18:31 | |
*** eharney has quit IRC | 18:41 | |
*** eharney has joined #openstack-manila | 18:41 | |
*** lbragstad__ has joined #openstack-manila | 18:42 | |
*** openstackgerrit has quit IRC | 18:50 | |
*** gmann is now known as gmann_lunch | 18:53 | |
*** baojg has joined #openstack-manila | 19:19 | |
*** hoonetorg has quit IRC | 19:41 | |
*** hoonetorg has joined #openstack-manila | 19:54 | |
*** arne_wiebalck has quit IRC | 20:03 | |
*** arne_wiebalck has joined #openstack-manila | 20:03 | |
gouthamr | vkmc: o/ can i have your review on: https://review.opendev.org/764075 | 20:05 |
gouthamr | when you get a chance :) | 20:05 |
*** gmann_lunch is now known as gmann | 20:08 | |
*** vishalmanchanda has quit IRC | 20:12 | |
*** raildo has quit IRC | 20:18 | |
*** lbragstad__ has quit IRC | 20:19 | |
*** raildo has joined #openstack-manila | 20:21 | |
*** lbragstad has joined #openstack-manila | 20:22 | |
*** rajinir has joined #openstack-manila | 20:25 | |
vkmc | gouthamr, of course | 20:33 |
gouthamr | ty vkmc | 20:38 |
*** openstackgerrit has joined #openstack-manila | 20:41 | |
openstackgerrit | Dina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI https://review.opendev.org/c/openstack/manila-ui/+/767017 | 20:41 |
openstackgerrit | Dina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI https://review.opendev.org/c/openstack/manila-ui/+/767017 | 20:52 |
*** baojg has quit IRC | 20:54 | |
*** baojg has joined #openstack-manila | 20:55 | |
*** raissa has joined #openstack-manila | 20:59 | |
*** raissa has quit IRC | 20:59 | |
*** raissa has joined #openstack-manila | 21:00 | |
openstackgerrit | Dina Saparbaeva proposed openstack/manila-ui master: Addition of share shrinking feature to Manila-UI https://review.opendev.org/c/openstack/manila-ui/+/767017 | 21:00 |
*** raissa has quit IRC | 21:00 | |
*** raissa has joined #openstack-manila | 21:03 | |
*** raissa has quit IRC | 21:03 | |
*** raissa has joined #openstack-manila | 21:04 | |
*** raissa has joined #openstack-manila | 21:05 | |
*** raissa has quit IRC | 21:05 | |
*** raissa has joined #openstack-manila | 21:11 | |
*** raissa has quit IRC | 21:11 | |
*** raildo has quit IRC | 21:29 | |
*** jmlowe has quit IRC | 21:43 | |
*** jmlowe has joined #openstack-manila | 21:45 | |
gouthamr | carloss dviroel tbarron: o/ pinging for reviews on the fix for broken lower-constraints jobs on stable/victoria and stable/ussuri: https://review.opendev.org/766670 and https://review.opendev.org/766674 :) | 23:04 |
dviroel | o/ | 23:05 |
*** baojg has quit IRC | 23:06 | |
*** baojg has joined #openstack-manila | 23:07 | |
vkmc | gouthamr, one question, was this fixed in the driver already? https://review.opendev.org/c/openstack/manila/+/764075 | 23:27 |
vkmc | I don't see it merged | 23:28 |
gouthamr | vkmc: a recent fix reinitialized the list, which makes our CI fail: https://review.opendev.org/q/I14835f6c54376737b41cbf78c94908ea1befde15 | 23:28 |
vkmc | yes, I see that | 23:29 |
gouthamr | vkmc: we unfortunately ended up backporting the bug when the CI was broken | 23:29 |
vkmc | but now you are removing the reinit... and we cannot the list of used volumes to continue growing | 23:29 |
vkmc | or what I am missing? | 23:30 |
vkmc | I read your suggestion to Lin to make the driver smarter by using the controller database to store state | 23:30 |
vkmc | but I don't see they added that fix yet | 23:30 |
vkmc | we cannot prevent* | 23:33 |
vkmc | so I assume that we are fixing the gate and then fixing that issue in a follow up? | 23:34 |
gouthamr | vkmc: sorry, stepped afk for a few | 23:41 |
gouthamr | vkmc: no, its a set, so additions are idempotent | 23:41 |
vkmc | gouthamr, oh sweet | 23:42 |
vkmc | yes, I got confused | 23:42 |
vkmc | cool | 23:42 |
gouthamr | vkmc: maintaining a set of volumes like that is definitely bad if you have multiple readers/writers that can't operate in a thread safe manner on the same data | 23:43 |
vkmc | gouthamr, so that's why you are suggesting to use the control database instead of having it in memory | 23:43 |
gouthamr | so a better fix is necessary, and hopefully Lin's able to provide it | 23:43 |
gouthamr | yep | 23:43 |
vkmc | nice | 23:43 |
vkmc | ok | 23:43 |
vkmc | thanks for the clarification | 23:43 |
gouthamr | you're welcome, thanks for reviewing vkmc | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!